onboarding: informative status line (installer + doc)#55
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Adds DRGScripts-side onboarding wiring for the shared Claude Code status line (implemented in agent-skills), including an installer that surgically sets ~/.claude/settings.json’s .statusLine, documentation for members, and CI linting for the installer script.
Changes:
- Add
onboarding/statusline/install.shto locate the groupcc-statusline.sh, back upsettings.json, and set.statusLineviajq. - Add
onboarding/statusline/README.mdexplaining segments, install/customize/troubleshoot/uninstall. - Update onboarding/maintaining docs and CI to reference and shellcheck the installer.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| onboarding/statusline/README.md | Member-facing documentation for the status line behavior and setup. |
| onboarding/statusline/install.sh | Installer that patches ~/.claude/settings.json to point .statusLine at the group script. |
| onboarding/ONBOARDING.md | Updates step 5 to use the installer and expands the smoke test expectations. |
| onboarding/MAINTAINING.md | Updates artifact map/decisions to reflect the new installer + shared lib design. |
| .github/workflows/ci.yml | Adds a CI job to shellcheck the new installer script. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Give group members the same colour-coded, location-aware status line the PI runs. The script lives in agent-skills (single source of truth); this adds the wiring and docs. - onboarding/statusline/install.sh: surgically points ~/.claude/settings.json .statusLine at the group cc-statusline.sh — only that key, backup first, idempotent, cross-platform. Requires jq. - onboarding/statusline/README.md: annotated breakdown of every segment, plus install / customise / troubleshoot / uninstall. - ONBOARDING.md step 5 + smoke test: use the installer, describe the richer line. - MAINTAINING.md: artifact map + decisions updated (settings.json now sets the .statusLine key via the installer; statusline shares a lib with the PI variant). - ci.yml: shellcheck the installer. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
alongd
force-pushed
the
statusline-onboarding
branch
from
6f2b473 to
9be10c9
Compare
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.
Gives group members the same colour-coded, location-aware Claude Code status line the PI runs. The script itself is the single source of truth in agent-skills; this PR adds the DRGScripts wiring + docs.
What it shows
<25/ yellow25–40/ bold red≥40.repo@branch, red*dirty flag, and[wt:name]/ dim[primary]/ bold-red[!primary](editing the primary checkout — the work-in-a-worktree anti-pattern).What changes
onboarding/statusline/install.sh(new) — points~/.claude/settings.json.statusLineat the groupcc-statusline.sh. Surgical (only that key), backs the file up first, idempotent, cross-platform. Requires jq.onboarding/statusline/README.md(new) — annotated breakdown + install / customise / troubleshoot / uninstall.ONBOARDING.md— step 5 uses the installer; smoke test updated.MAINTAINING.md— artifact map + decision rows (settings.json now sets.statusLinevia the surgical installer; the statusline shares a lib with the PI variant).ci.yml— new job shellchecks the installer.Depends on DanaResearchGroup/agent-skills#14 (the script + shared lib). Merge that first so
install.shhas something to point at.Verification
🤖 Generated with Claude Code