Skip to content

chore: add internal e2e skill - #112

Merged
lodev09 merged 1 commit into
mainfrom
chore/e2e-skill
Nov 10, 2026
Merged

lodev09 merged 1 commit into
mainfrom
chore/e2e-skill

Conversation

@lodev09

@lodev09 lodev09 commented Nov 10, 2026

Copy link
Copy Markdown
Member

Summary

Adds an internal e2e agent skill for testing @lugg/maps on the iOS simulator, a physical iPhone, the Android emulator, and web. It is based on the TrueSheet e2e skill and on what we learned while testing #110 and #111.

  • SKILL.md:
    • Building Debug and Release for each target.
    • What each control in the example app exercises.
    • A launch-argument hook for fixed benchmark scenarios.
    • A/B testing against HEAD.
    • Pitfalls, and the finish and cleanup steps.
  • references/benchmark.md:
    • CPU per process on the simulator and the device.
    • Time Profiler breakdowns per thread.
    • How to tell fewer frames from less work.
    • Side-by-side builds under different bundle ids.
    • Thermal limits, and reference numbers.
  • references/native-debugging.md: temporary MAPDBG logs, counting map objects that are added and removed, and crash triage.
  • scripts/:
    • sim-cpu.sh: CPU on the simulator.
    • device-cpu.py: CPU on a physical iPhone, from an Activity Monitor trace, with the thermal state.
    • thread-cpu.py: CPU per thread and per frame, from a time profile.

The skill lives in .agents/skills/e2e, and .claude/skills/e2e links to it. .gitignore now ignores .claude/* except .claude/skills.

The skill is marked metadata.internal: true, so npx skills add lugg/maps doesn't install it. The npm package doesn't include it either, because files excludes dot-folders.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Test Plan

  • npx skills@latest add <repo> --list lists only maps-usage. With INSTALL_INTERNAL_SKILLS=1, it also lists e2e.
  • npm pack --dry-run includes nothing from .agents or .claude.
  • The scripts pass bash -n and py_compile. They are cleaned-up versions of the scripts used to benchmark feat: move markers along paths natively #110; I haven't run these exact files on a device yet.

Screenshots / Videos

Checklist

  • I tested on iOS
  • I tested on Android
  • I tested on Web
  • I updated the documentation (if needed)

@vercel

vercel Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
lugg-maps Ready Ready Preview Nov 10, 2026 8:30pm UTC

Request Review

@lodev09
lodev09 merged commit 2e4cd77 into main Nov 10, 2026
5 checks passed
@lodev09
lodev09 deleted the chore/e2e-skill branch November 10, 2026 20:31

This branch was successfully deployed

1 active deployment
Preview — 1e8ad975 Deployed Nov 10, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant