docs: add /health and /checkpoint to README (v0.16.2.2)#952
Closed
ignsm wants to merge 8 commits intogarrytan:mainfrom
Closed
docs: add /health and /checkpoint to README (v0.16.2.2)#952ignsm wants to merge 8 commits intogarrytan:mainfrom
ignsm wants to merge 8 commits intogarrytan:mainfrom
Conversation
./setup was emitting a non-fatal but ugly error during the Windows Node.js fallback bundle step: Building Node-compatible server bundle... error: cannot write multiple output files without an output directory Root cause: browse/src/server.ts dynamically imports @ngrok/ngrok for /pair-agent tunneling. Bun's bundler statically sees the import and tries to inline ngrok's prebuilt native .node binaries (darwin-universal 19MB + darwin-arm64 9MB + win32-x64 etc.) as separate asset files. But build-node-server.sh uses --outfile, which only supports a single output file. Bun fails with "cannot write multiple output files" before writing anything. Fix: mark @ngrok/ngrok as --external in the Node bundle build, matching how we already handle playwright and playwright-core. Windows Node users who want pair-agent tunneling install @ngrok/ngrok separately — same model as playwright. Verified: bash browse/scripts/build-node-server.sh now produces a clean single server-node.mjs (0.31 MB, 23 modules) with no errors. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
…t /plan-devex-review /devex-review The README had two separate skill lists that drifted out of sync: 1. Install command block (line 49) listed /connect-chrome (legacy name), /plan-devex-review, /devex-review but was missing /pair-agent. 2. CLAUDE.md example block (line 396) listed /open-gstack-browser, /pair-agent but was missing /plan-devex-review and /devex-review. Fix: both lists now contain the same 34 skills in the same order: - Replace legacy /connect-chrome with canonical /open-gstack-browser (the skill was renamed; /connect-chrome is still a symlink alias but shouldn't be the advertised name). - Add /pair-agent to the install block. - Add /plan-devex-review and /devex-review to the CLAUDE.md example. Users pasting either block into a new project's CLAUDE.md now get the full set of available skills without gaps. Note: /checkpoint and /health are real user-facing skills that exist on disk and get linked by ./setup but are still not mentioned anywhere in the README (not in either skill list block, not in the skill table). Left for a follow-up PR — this one stays narrowly scoped to aligning the two existing list strings. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Both skills exist on disk, get linked by ./setup, and show up in ~/.claude/skills/ — but they're not mentioned anywhere in the README (no install-block entry, no CLAUDE.md example entry, no skill table row). This TODO captures the follow-up PR that would add them in all three places. Kept out of this branch to preserve narrow scope: the two skill-list strings are now aligned, but a proper fix for /checkpoint and /health needs two new rows in the big skill table too, with descriptions matching the existing table voice. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Both skills exist on disk at health/SKILL.md and checkpoint/SKILL.md, get registered by ./setup on every install, and show up in ~/.claude/skills/ as working slash commands. But they were completely absent from the README big skill table — new users had no way to discover them. Placement rationale: - /health slots after /benchmark (both are measurement/dashboard tools that produce a score for the codebase). - /checkpoint slots after /learn (both are session/state persistence tools — /learn for cross-session learnings, /checkpoint for within- project resume state). Specialist roles: - /health → "Quality Engineer" (sits alongside "Performance Engineer" for /benchmark and "Staff Engineer" for /review). - /checkpoint → "Session Bookmark" (avoids collision with /learn's "Memory" role since they're adjacent in the table). Voice matches the existing table: one-line role, two-to-three-sentence description emphasizing what the user can DO and the concrete trigger phrase. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Now that the big skill table has rows for both, keep the two skill-list strings (the install command block and the CLAUDE.md example block) consistent with the table. Both strings list the same 36 skills in the same order. Placement: - /health slots after /benchmark (same rationale as the table row commit: both measurement/dashboard tools). - /checkpoint slots after /learn (same rationale: session/state tools grouped at the end of the list). Users who paste either block into a new project's CLAUDE.md now get the full set of 36 skills that ./setup actually installs. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Remove the TODO added in commit 1680079 (the ## Documentation section holding the single "Add /checkpoint and /health to README" entry) and move it to the ## Completed section at the bottom with the v0.16.2.2 marker. This keeps TODOS.md tidy: the ## Documentation section existed only to hold this one item, so removing it collapses cleanly. The ## Completed entry documents what shipped so the history is still visible. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Closed
5 tasks
Author
|
Closing this along with its parent #951. The README work from this PR (adding The split into a parent + stacked follow-up didn't make sense once #951 turned out to duplicate open build-fix PRs #924 and #947. #953 is the correct final shape: one branch from fresh main, three bisected commits, no overlap with any build-fix work in flight. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Follow-up to #951. Both
/healthand/checkpointhave been working skills for a while —health/SKILL.mdandcheckpoint/SKILL.mdlive on disk, have clean frontmatter with trigger phrases, and./setupalready registers them into~/.claude/skills/on every install. But they were completely absent from the README in all three user-facing places: the install command skill list (line 49), the CLAUDE.md example skill list (line 398), and the big skill table (around line 186). New users had no way to discover them.This PR closes that gap. All three places now mention both skills.
Stacking note
This is a stacked PR on top of #951 (
fix/ngrok-external-in-node-bundle). #951 aligned the two README skill-list blocks and added/pair-agent,/plan-devex-review,/devex-review. This PR builds on that cleanup to add the remaining two skills in one coherent pass.Until #951 merges, this PR's diff will show all 4 commits from #951 plus the 4 new commits here. Once #951 merges to main, GitHub will auto-rebase this PR and only the 4 new commits will remain. No manual rebase needed.
If you want to review just this PR's delta: check commits
12c7bbf1,0f7e5222,89fe0a4c,fd6d4f6e.Bisection
Four clean commits, each independently revertable:
12c7bbf1—docs: add /health and /checkpoint rows to README skill table/healthafter/benchmark(both measurement/dashboard tools)./checkpointafter/learn(both session/state persistence tools)./health→ "Quality Engineer" (sits alongside "Performance Engineer" for/benchmark)./checkpoint→ "Session Bookmark" (avoids collision with/learn's "Memory" role since they're adjacent).0f7e5222—docs: add /health and /checkpoint to both README skill-list blocks/healthafter/benchmark,/checkpointafter/learn.89fe0a4c—docs: mark /checkpoint + /health README TODO as done## Documentationsection from TODOS.md (it existed only to hold this one entry) and moves the entry to## Completedwith a v0.16.2.2 marker.fd6d4f6e—chore: bump version and changelog (v0.16.2.2)(VERSION + CHANGELOG.md)Test Coverage
No new code paths — this is a pure documentation change across 3 files (README.md, TODOS.md, CHANGELOG.md) and a VERSION bump. No application logic, no build scripts, no runtime behavior changes.
Both skills were manually verified to exist and register correctly on the
/Users/ignat/GitHub/gstackcheckout before writing the table rows:health/SKILL.mdlines 1-13 (frontmatter with description, allowed-tools, trigger phrases)checkpoint/SKILL.mdlines 1-13 (frontmatter with description, allowed-tools, trigger phrases)./setupoutput on the upgrade session that surfaced this gap confirmed both skills land in~/.claude/skills/checkpoint/SKILL.mdand~/.claude/skills/health/SKILL.mdas working top-level slash commands.Pre-Landing Review
No issues. 5 substantive line edits across README.md (2 new table rows, 2 skill-list string updates) + 14 lines added/removed in TODOS.md + 3 lines in VERSION/CHANGELOG. No attack surface, no runtime effect, no regression risk.
Voice calibration
Both descriptions were written to match the existing skill table voice:
/learnis already "Memory", so/checkpointis "Session Bookmark"./benchmarkis already "Performance Engineer", so/healthis "Quality Engineer".Test plan
grep "^## \[" CHANGELOG.mdshows contiguous version sequence (verified — 0.16.2.2, 0.16.2.1, 0.16.2.0, 0.16.1.0, ...)/healthand/checkpointappear in all three places: install block (line 49), CLAUDE.md example (line 398-403), big skill table (after/benchmarkand after/learnrespectively)health/SKILL.mdexists,checkpoint/SKILL.mdexists,./setuplinks both into~/.claude/skills/(confirmed from prior setup run on the upgrade session)🤖 Generated with Claude Code