Repository navigation
State the Bootstrap's Archive Live Channel in Two Skills - #2237
Conversation
check-this-repo and merge-and-release framed the Claude Code live channel only as a registered git checkout. A bootstrapped host registers the tree the bootstrap keeps, which --report describes as vcs archive with branch null, so both skills now state that case and name re-running the bootstrap's skills action as what refreshes it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
live_channel emits vcs only for an archive tree, so the git case is now the otherwise branch. The archive case also covers a null commit and a tree kept under a non-default --dir, which a rerun has to name. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
A live block with no vcs is a git checkout only where it carries a commit. An unregistered channel, a failed listing, and a directory git cannot read are their own shapes rather than a detached checkout. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #2237 +/- ##
==========================================
Coverage ? 56.61%
==========================================
Files ? 16
Lines ? 7482
Branches ? 0
==========================================
Hits ? 4236
Misses ? 3246
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
It is a documentation-only Skill update whose new text I verified as accurate against skills_install.py and the bootstrap scripts, consistent across all generated copies, and conformant with the repository's prose conventions.
Review effort: Balanced
Findings: None
What changed in this PR
This documentation-only change updates two agent Skills to describe the Claude Code "live" skills channel when it is served by an archive tree (a tree the host bootstrap keeps) rather than a git checkout. Since a bootstrapped host registers the plugin against a tree that has no git, skills_install.py --report describes that channel as live.vcs: archive with branch: null, and the prior text framed the live channel only as a git checkout. The two Skills now read the live block by shape so an agent diagnoses and remediates the archive case correctly (re-running bootstrap.sh --skills / bootstrap.ps1 -Skills) instead of treating branch: null as a detached checkout.
Changes:
check-this-repostep 1 now branches onliveshape:vcs: archive(serves the kept tree; null/behindlive.commitis the finding; bootstrap re-run is the fix, with--dir/-Dirwhere kept off-default), the git case (novcs+ non-nullcommit), and a catch-all that readsreason.merge-and-releasestep 7 states that an archive live channel holds what the bootstrap fetched until the bootstrap's skills action runs again, which that step does not do.- Distributions regenerated: the
.github/skills/and.claude-plugin/fleet-skills/copies and the two.claude-pluginsource digests were updated from the.agents/skills/source.
I verified the new prose against the implementation: live_channel() returns vcs: archive, branch: null, and a nullable commit for bootstrap trees (scripts/skills_install.py:413-420); --dir/-Dir names the parent of the kept skills-tree directory (host-setup/bootstrap.sh:67, host-setup/bootstrap.ps1:112); and --skills/-Skills flags exist. The three SKILL.md copies are byte-identical and the prose is ASCII-clean, free of mid-sentence semicolons, and within the ~100-column wrap used across .agents/skills/.
| File | Description |
|---|---|
.agents/skills/check-this-repo/SKILL.md |
Source: rewrites step 1 to read the live block by shape and state the archive remedy. |
.agents/skills/merge-and-release/SKILL.md |
Source: step 7 now describes the archive live channel and its refresh trigger. |
.github/skills/check-this-repo/SKILL.md |
Generated copy, identical to source. |
.github/skills/merge-and-release/SKILL.md |
Generated copy, identical to source. |
.claude-plugin/fleet-skills/skills/check-this-repo/SKILL.md |
Generated copy, identical to source. |
.claude-plugin/fleet-skills/skills/merge-and-release/SKILL.md |
Generated copy, identical to source. |
.claude-plugin/fleet-skills/.source-digests/check-this-repo |
Regenerated digest for the updated source. |
.claude-plugin/fleet-skills/.source-digests/merge-and-release |
Regenerated digest for the updated source. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
## Summary Promotes develop to main, carrying the pull requests below, each already reviewed and merged into develop. - [#2203](#2203): Strip every heredoc body a line opens in the wait-loop rule. - [#2216](#2216): Point the charset-unknown message at where a tier is classified. - [#2218](#2218): Name the leftover old tree when the Bash loader cannot remove it. - [#2227](#2227): End the guard's stdin redirect scan at a trailing comment. - [#2230](#2230): Tell apart the causes a labels payload is refused for. - [#2232](#2232): Correct the task names and pointers in the Python tasks snippet header. - [#2234](#2234): Drop the stale private-repository note from PhotoCleaner's registry entry. - [#2237](#2237): State the bootstrap's archive live channel in two skills. - [#2239](#2239): Stop the promotion count on a failed fetch in backlog-burndown. - [#2241](#2241): Render the include walk once per `build_dist.py --check` run. - [#2243](#2243): Have the tree check call `escapes_repo_root` and refuse a symlinked component. - [#2247](#2247): Refuse a Windows drive component anywhere in a tree path. - [#2222](#2222): Tick the adopted repos in the merge-bot and gate rollout stages. - [#2121](#2121): Refuse a dangling remote-tracking target in `local_review`. - [#2249](#2249): Ignore a quoted separator when the guard finds a loop's `done`. Closes #2112 Closes #2171 Closes #1790 Closes #2152 Closes #1372 Closes #2149 Closes #1082 Closes #1772 Closes #1310 Closes #1379 Closes #1452 Closes #2244 Closes #2168 Closes #2106 Closes #2207 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Summary
check-this-repo"What it checks" step 1 now reads theliveblock by its shape. Avcs: archivetree is the one the hub bootstrap keeps:branch: nullthere does not mean detached, alive.commitbehindmainor null is the finding, and re-runningbootstrap.sh --skillsorbootstrap.ps1 -Skillsis the fix, with--diror-Dirwhere the tree was kept outside the default. The git case is keyed on novcsand a non-nullcommit, and every other shape is an unregistered or unreadable channel.merge-and-releasestep 7 states that an archive live channel holds what the bootstrap fetched until the bootstrap's skills action runs again, which that step does not do.scripts/build_dist.py, and--checkis clean.The same framing in
docs/host-setup.md"Fleet Skills Install" is filed as #2236.Closes on promotion: #1772
🤖 Generated with Claude Code