feat(session-flow)!: rename orchestration-brief skill to orchestrate#234
Conversation
## What - Rename the session-flow skill `orchestration-brief` to `orchestrate` (`git mv` preserves history): directory, frontmatter `name`, every `/session-flow:orchestration-brief` slash token, eval `skill_name` and prompts, the plugin.json skill enumeration, and both READMEs. The invocation is now `/session-flow:orchestrate`. - Add a seventh standing imperative, CALIBRATE TO CONDITIONS: size the whole orchestration (whether to delegate at all, fan-out width, nesting depth) to the active model's capability, advisor/verifier availability, context pressure, and concurrent-session / rate-limit headroom, with small/medium/large fan-out sizing and single-agent as the floor. Backed in context/sources.md with per-factor sourcing (canonical vs authoring convention marked, like imperative 6). - Bump session-flow 0.5.0 to 0.6.0 and record both changes in the plugin CHANGELOG. ## Why - The old name foregrounded the secondary export brief and undersold the skill's default action — arm/prime the current session. `orchestrate` is an action verb, matching this repo's action-skill naming convention (docs/MIGRATION-PLAYBOOK.md). Export modes (`handoff` / `worker`) are unchanged, and the exported document is still an orchestration brief. - The six imperatives specified how to delegate but not how to size delegation to the conditions in play; the calibration imperative closes that gap. Fresh-docs mandate: the skill invocation token derives from the directory name (https://code.claude.com/docs/en/skills, fetched 2026-07-16), so the rename is a directory move plus a matching display-`name`. Version-bump-as- delivery and the per-plugin CHANGELOG follow docs/MIGRATION-PLAYBOOK.md. BREAKING CHANGE: `/session-flow:orchestration-brief` no longer resolves; use `/session-flow:orchestrate`. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013QfME4XhdvpZXAUinfomr5
|
Claude finished @kyle-sexton's task in 1m 44s —— View job PR Review
SummaryThis is a clean, well-executed rename + feature addition. The reference sweep is complete, the new imperative is grounded, and the delivery mechanics (version bump, CHANGELOG, evals) are consistent. Two minor observations below; neither is a blocker. Rename sweep — cleanZero stale slash-token references in tracked files outside of The retained New imperative 7 — CALIBRATE TO CONDITIONSThe imperative text is well-formed and the sourcing in Relationship to imperative 1 (DELEGATE / FAN OUT) — not an overlap: imperative 1 governs whether to delegate (context flooding, genuinely independent paths, specialist tool); imperative 7 governs how wide and deep to go given current conditions (model strength, context pressure, rate-limit headroom). They address adjacent but distinct questions. The distinction is clear on reading. One observation: the imperative body is noticeably longer than imperatives 1–6 and introduces four orthogonal sizing factors in a single run-on sub-clause. A user loading a Eval coverage gap (minor)Evals 1–4 each exercise a distinct action/modifier combination. Eval 5 tests that the skill declines to actually orchestrate. None of the five evals exercises imperative 7 directly — e.g., a scenario where the model is asked to fan out aggressively but conditions (small ask, thin context) should suppress delegation. The existing evals are sufficient for the current behavior contract, but a sixth eval targeting calibration specifically would catch regressions if the imperative text is later trimmed. Mechanics — all correct
No security concerns. No correctness bugs. The eval gap is low-risk since the imperative is self-contained instruction text, not branching logic. Ready to merge pending the author's judgment on the eval note. |
…-actor exception (#858) ## What - ADR 0002 step-3 addendum: `security-review-gate` org ruleset ACTIVE (github-iac#193 ruleset + #194 App-grant widening, production deploy verified: `security-review / security-review` in `required_status_checks` on this repo's main); operator-ratified skip-actor exception (dependabot[bot], melodic-standards-sync[bot]) with rationale and bounds; the PR-branch caller-file bound stated plainly. New revisit trigger: any skip-actors addition re-opens the exception. - Roadmap PLAN: Phase II [DONE] + step-3 status note. Closes #509. ## Related - ci-workflows#189, standards#229/#234, #825, #828, github-iac#193/#194 - Deploy run: https://github.com/melodic-software/github-iac/actions/runs/29822200383 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
What
orchestration-brief→orchestrate(
git mv, history preserved). The sweep covered every reference form:directory name, frontmatter
name, all/session-flow:orchestration-briefslash tokens, eval
skill_name+ prompts, theplugin.jsonskillenumeration, and both READMEs. Invocation is now
/session-flow:orchestrate.orchestration (whether to delegate at all, fan-out width, nesting depth) to
the active model's capability, advisor/verifier availability, context
pressure, and concurrent-session / rate-limit headroom, with
small/medium/large fan-out sizing and single-agent as the floor. Wired into
the standing instructions in the existing imperatives' voice, the export
template count, the compact-mode count, the frontmatter description, and a
new
context/sources.mdsection (per-factor sourcing, canonical vs authoringconvention marked like imperative 6).
0.5.0→0.6.0and record both changes inthe plugin
CHANGELOG.md.Why
skill's default action — arm/prime the current session.
orchestrateis anaction verb, matching this repo's action-skill naming convention
(
docs/MIGRATION-PLAYBOOK.md). Export modes (handoff/worker) areunchanged, and the exported document is still an orchestration brief — so the
"brief" noun is retained where it names that artifact (export header,
sources.mdprose) and one trigger phrase, while every skill-invocationreference moved to
orchestrate.delegation to the conditions in play; the calibration imperative closes that
gap.
Reference sweep evidence
Ran the
docs-hygiene:rename-referencesaudit pattern library (slash-token,path, dot-form, glob, chain-forward/backward, numbered-row, frontmatter) plus
an any-separator
orchestration.briefregex over tracked files. After therename, zero
orchestration-brieftoken forms remain except the intentionalCHANGELOG.mdrename record. The surviving space-formorchestration brief/ORCHESTRATION BRIEFhits are the deliberate export-artifact noun and theretained
'orchestration brief'trigger — classified, not missed..work/working notes are untracked and out of scope.
Validation
node scripts/validate-plugin-contracts.mjs— pass (1365 files checked)node scripts/generate-catalog.mjs --check— catalog in sync (root READMEregenerated from
plugin.json, only the session-flow line changed)markdownlint-cli2(repo config) — 0 errors across changed markdowntypos(repo_typos.toml) — cleanFresh-docs mandate
Confirmed the skill invocation token derives from the directory name
(https://code.claude.com/docs/en/skills, fetched 2026-07-16), so the rename is
a directory move plus a matching display-
name. Version-bump-as-delivery andthe per-plugin
CHANGELOG.mdfollowdocs/MIGRATION-PLAYBOOK.md.BREAKING:
/session-flow:orchestration-briefno longer resolves; use/session-flow:orchestrate.🤖 Generated with Claude Code
https://claude.ai/code/session_013QfME4XhdvpZXAUinfomr5