docs(onboarding): add gstack install step + author attribution#52
Merged
Conversation
CLAUDE.global.md and the smoke test both reference gstack skills (/browse, /review, /ship, …) but no onboarding step installed gstack — it's a separate repo (garrytan/gstack), git-ignored inside agent-skills, so members who only did steps 4-5 never got it. A prior CC session read this gap plus the "gstack hook" line in MAINTAINING.md as "gstack is PI-only," which is wrong: only the settings.json session-update hook is PI-only, the skills are member-facing. - ONBOARDING: new step 6 "gstack skills" (clone garrytan/gstack into ~/.claude/skills/gstack + ./setup, with the bun prerequisite noted); renumber steps 6-11 -> 7-12 and fix cross-refs; smoke test points at step 6. - ONBOARDING: author-attribution note (Alon Grinberg Dana wrote this runbook + CLAUDE.global.md) so future CC sessions don't guess who "the PI" is. - MAINTAINING: disambiguate the settings.json gstack *hook* (PI-only) from the gstack *skills* (member-installed, step 6); add a decision row. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Problem
CLAUDE.global.mdand the smoke test both reference gstack skills (/browse,/review,/ship, …), but no onboarding step actually installs gstack. gstack is a separate repo (garrytan/gstack), git-ignored insideagent-skills, so a member who does only steps 4–5 never gets it — and thetype /smoke-test check silently fails.A prior Claude Code session hit this and concluded "gstack is PI-only, drop it from the doc." That's wrong: only the settings.json session-update hook is PI-only — the skills are member-facing and meant to be installed (per the agent-skills README's own step 3).
Changes
ONBOARDING.md— new step 6 "gstack skills": clonegarrytan/gstackinto~/.claude/skills/gstackand run./setup, with thebunprerequisite flagged (./setupverified to require it). Steps 6–11 renumbered → 7–12, all cross-references fixed, smoke test now names concrete skills and points at step 6.ONBOARDING.md— author-attribution note: Alon Grinberg Dana authored the runbook +CLAUDE.global.md, so future CC sessions don't guess who "the PI" / "personal infra" refers to.MAINTAINING.md— disambiguates the settings.json gstack hook (PI-only) from the gstack skills (member-installed, step 6); adds a decision row so this doesn't get misread again.Notes
alongd/→DanaResearchGroup/) is already onmain(PR onboarding: point agent-skills URLs at DanaResearchGroup #51), so this branch is rebased ontomainand omits it — no duplication.MAINTAINING.md) run clean on the diff.🤖 Generated with Claude Code