v5.6.0 — Fable→Opus + reviewer context wiring (incl. v5.5.0 hybrid Workflow acceleration) - #81
Merged
Merged
Conversation
Opt-in saved workflows (deployed to ~/.claude/workflows/ via checksum sync): - /superflow-review: unified review fan-out (product + technical in parallel, fenced-JSON verdicts, fail-closed parsing, light-mode product:false, codex runs inside the sprint worktree via workdir arg) - /superflow-wave: implementation-only parallel wave; orchestrator keeps review/docs/PAR/ship per sprint references/workflow-orchestration.md is the single Workflow authority: documented API surface, opt-in policy, permission gates by mode, limits, runtime conventions verified by live smoke runs (top-level body, parallel(thunks), meta pure literal, args may arrive as a JSON string), undocumented-API warning, /goal watchdog, Codex fallback chain. Integration: context.use_workflows opt-in at Phase 1 plan approval (state + charter); /goal ready-to-paste suggestion at Phase 2 launch (user-only command, honestly documented); provider enum gains workflow-review; Codex runtime unaffected (no Workflow tool, no /goal — existing spawn_agent paths). Verification: live smoke runs of both scripts (fail-closed path: 0 agents/12ms; full pipeline: 1 agent, parsed position-bound result); DAG verify 33/33; dual review -> fixes -> scoped re-review APPROVE. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Fable access is blocked. All deep judgment roles (deep spec/code/product reviewers + deep-analyst) move model: fable -> opus; depth differentiated by effort (deep=max, standard=high). Implementers stay on Sonnet — deliberately not pushed into judgment roles. - Remove the model_profile selection entirely (Phase 1 Step 2c, state schema, charter frontmatter, plan recap, all dispatch docs). - Pin every Claude secondary to claude-opus-4-8 (SKILL.md, codex/AGENTS.md, codex dispatch patterns + phase overlays); PAR provider strings updated. - Fix stale "inherits Fable" warnings -> inherits the orchestrator session model (Opus). Reviewer context wiring (spec/plan/charter actually reach reviewers): - code-reviewer (deep+standard+mirror) gains <spec_or_plan> + <autonomy_charter> slots; product-reviewer gains <product_brief> + <autonomy_charter>. - /superflow-review takes optional spec_path/plan_path/brief_path, wired into both lenses; missing inputs flagged unverifiable, not passed silently. - review-unified.md dispatch contract made explicit. Deployed copies synced to ~/.claude and ~/.codex. CI forbidden-token gate hardening is split into a follow-up commit (needs GitHub `workflow` scope). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…claude-opus-4-8 Regression guard so the blocked Fable model cannot be reintroduced. Pushing this requires the GitHub `workflow` OAuth scope. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Owner
Author
v5.6.0 added on top of this branchFable access blocked → Opus everywhere
Reviewer context wiring
|
Owner
Author
|
✅ Follow-up |
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.
Summary
Hybrid (not a rewrite): the markdown DAG and Agent-based dispatch stay the default everywhere and the only path on Codex runtime. Two opt-in saved workflows accelerate the hottest Phase 2 fan-outs, and Phase 2 launch now suggests a
/goalwatchdog.Saved workflows (deployed to
~/.claude/workflows/via the v5.4.0 checksum sync)/superflow-review— unified review fan-out: product + technical reviewers in parallel; fenced-JSON verdicts parsed fail-closed in the script;passcomputed in code, not model memory.workdirarg makes codex review the sprint worktree checkout (not the repo root);product: falsehandles light-governance single-reviewer cells. Provider value:workflow-review./superflow-wave— implementation-only parallel wave per the v5.4.0 dispatch rule; orchestrator keeps review → docs → PAR → ship per sprint.references/workflow-orchestration.md— single Workflow authorityDocumented API facts with official-doc citations, opt-in policy, permission gates by mode, limits, runtime conventions verified by live smoke runs (top-level script body,
parallel(thunks),metapure literal,argsmay arrive as a JSON string → normalization required), an UNDOCUMENTED-API warning box, the/goalwatchdog section, and the Codex fallback chain.Integration
context.use_workflowsopt-in captured at Phase 1 plan approval (state + Autonomy Charter), with an honest permission-mode heads-up./goalsuggestion (user-only command — documented as such; subagent goal-direction remains charter injection)./goal— existingspawn_agentpaths.Test plan
/superflow-reviewwith empty args → instant{pass: false, error: "missing required args: …"}, 0 agents, 12 ms/superflow-wavewith a fake sprint → 1 agent dispatched, fenced JSON parsed, position-bound result returned exactly per contractargsarrive as a JSON string → defensive normalization shipped in both scriptstools/verify-phase2-dag.sh33/33 PASS;jqschema checks; llms.txt line counts swept🤖 Generated with Claude Code