fix(skills): retire stale steward supplier refs; sharpen origin-skill provenance (#74)#75
Conversation
…ll provenance (0.18.1, #74) Land the intended fix for the devague origin-skill family at source, before rollout-cli resumes its paused mesh broadcast (24 PRs open, 46 pending) — so the fix propagates on the downstream re-pull instead of being chased across 70+ copies. - steward→guildmaster: the three wrapper scripts (`think.sh` / `spec-to-plan.sh` / `assign-to-workforce.sh`) and the `test_think_skill` docstring still named `steward` as the mesh supplier. That role moved to `guildmaster` at the 2026-05-24 cutover, and every `SKILL.md` already says so; these were the last stale references and would have shipped verbatim downstream. - other→inbound: the `think` / `spec-to-plan` / `assign-to-workforce` / `deviate` descriptions said they are "not vendored like the other skills here." The other origin skills are also not vendored from guildmaster, so "other" was imprecise — reworded to "inbound", matching `scope` and `summarize-delivery`. - scope provenance: clarified that its "fourth" ordinal is authoring order while its role is the opening leg of the flow, and added the never-re-vendored-back upstream clause its younger siblings already carry. Docs/comments only; 614 tests pass. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LYjiRW7pVrEjTpZYGvPaEg
|
/agentic_review |
|
PR Summary by QodoFix origin-skill provenance text and update mesh supplier refs (0.18.1)
AI Description
Diagram
High-Level Assessment
Files changed (11)
|



What
Lands the intended fix for the devague origin-skill family at source, so
rollout-cli can resume its paused mesh broadcast (24 PRs open, 46 pending) and
the fix propagates on the downstream re-pull — rather than being chased across
70+ downstream copies. Closes #74.
Docs/comments only. Three small cleanups across the six origin skills:
1.
steward→guildmaster(stale mesh-supplier name)The three wrapper scripts (
think.sh,spec-to-plan.sh,assign-to-workforce.sh) and thetest_think_skilldocstring still namedstewardas the mesh supplier in their# Origin:headers. That role moved toguildmaster at the 2026-05-24 cutover, and every
SKILL.mdalready says so— these were the last stale references, and they would have shipped verbatim
to every downstream copy on the broadcast.
2.
other→inboundskills (imprecise wording)The
think/spec-to-plan/assign-to-workforce/deviatedescriptionssaid they are "not vendored from guildmaster like the other skills here."
The other origin skills are also not vendored from guildmaster, so "other" was
imprecise. Reworded to "inbound", matching
scopeandsummarize-delivery.3.
scopeprovenance refreshscope's Provenance called it "the fourth in the outbound family after/think, /spec-to-plan, and /assign-to-workforce" — which reads as if it
pre-dates the two later siblings. Clarified that the "fourth" ordinal is
authoring order while
scope's role is the opening leg of the flow, andadded the never-re-vendored-back upstream clause its younger siblings
(
summarize-delivery,deviate) already carry.Testing
uv run pytest -n auto— 614 passedmarkdownlint-cli2 CHANGELOG.md— cleanbash -non all three wrapper scripts — cleanblack --check+flake8on the touched test — cleanVersion
Patch bump
0.18.0→0.18.1with a### FixedCHANGELOG entry.Follow-up for rollout-cli
Once merged, this unblocks #74: rollout-cli can refresh the 24 open PRs from the
fixed source and complete the remaining 46 pending repos.