Skip to content

models: ♻️ Route prose to Opus and review lanes to sol #319

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
yasyf merged 1 commit into from
Oct 5, 2026
Merged

models: ♻️ Route prose to Opus and review lanes to sol #319

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
115 changes: 71 additions & 44 deletions captain_hook/builtin_packs/general/hooks/models.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@
INLINE_EDIT_MIN_CHARS = 400
SUSTAINED_BROWSER_CALLS = 5
BROWSER_DRIVER = re.compile(r"(?i)\b(agent-browser|playwright)\b")
CODEX_AGENTS = ("codex-wrapper", "codex:codex-wrapper", "codex-wrapper-async", "codex:codex-wrapper-async")
SOURCE_FILE_GLOBS = tuple(
pattern for language, patterns in LANG_GLOBS.items() if language != "md" for pattern in patterns
)
Expand Down Expand Up @@ -132,7 +133,7 @@ class DelegatedSpawn:

tag: str = "delegated_spawn"
required: bool = True
unpinned_note: str = "(none — inherits the session model, fable)"
unpinned_note: str = "(none — inherits the session model, opus)"

def content(self, evt: BaseHookEvent) -> str | None:
if (call := evt.as_input(TaskCall)) is None or not call.prompt:
Expand All @@ -158,7 +159,7 @@ def content(self, evt: BaseHookEvent) -> str | None:
class ProseSpawn(DelegatedSpawn):
"""Gating context: the pending spawn, present only when its prompt asks to produce a prose artifact."""

unpinned_note: str = "(none — an unpinned subagent runs opus; subagents never inherit fable)"
unpinned_note: str = "(none — an unpinned subagent runs opus)"

def content(self, evt: BaseHookEvent) -> str | None:
# WORKAROUND: zero-arg super() breaks under @dataclass(slots=True), which rebuilds the class.
Expand Down Expand Up @@ -218,53 +219,69 @@ def content(self, evt: BaseHookEvent) -> str | None:
deliverable_rubric=str(Prompt.load("fragments/deliverable_rubric", verdict_attr="block")),
),
message=(
"Prose deliverables are written by gpt-6-astra through codex, never by a Claude subagent. "
"Spawn `subagent_type: 'codex:codex-wrapper'` with the writing brief, or state in the prompt "
"that every sentence is delegated to codex and landed verbatim."
"Prose deliverables are written by Claude Opus, never by codex, astra, sonnet, haiku, or fable. "
"Re-spawn with `model='opus'` or no `model` pin, as a Claude agent that writes the prose itself."
),
contexts=[ProseSpawn()],
events=Event.PreToolUse,
only_if=[Tool("Agent|Task")],
skip_if=[
ToolInput("prompt", r"(?i)\b(codex|astra)\b"),
And(
Not(ToolInput("model", r"(?i)\b(sonnet|haiku|fable)\b")),
Not(ToolInput("prompt", r"(?i)\b(codex|astra)\b")),
Not(Agent(*CODEX_AGENTS)),
),
ToolInput("prompt", r"(?i)\b(classif|label|tag|categoriz|count|extract|mechanical)"),
Agent("Explore|claude-code-guide|codex-wrapper|codex:codex-wrapper"),
Agent("Explore|claude-code-guide"),
],
agent=False,
transcript=False,
max_context=16_000,
tests={
Input(model="sonnet", prompt="Write the README quickstart for this repo"): Block(),
Input(model="opus", prompt="draft the release notes for v2"): Block(),
Input(model="haiku", prompt="update the CHANGELOG entry for the fix"): Block(),
Input(model="fable", prompt="write the README quickstart"): Block(),
Input(prompt="write the README quickstart"): Block(),
Input(prompt="draft the release notes for v2"): Block(),
Input(
agent_type="codex:codex-wrapper",
prompt="Rewrite the README quickstart in the technical-builder voice",
): Block(),
Input(
model="opus",
prompt="Orchestrate the incident-retro revision: rewrite the release notes and redraft "
"the guide. Every sentence is written by gpt-6-astra at xhigh via the codex skill and "
"landed verbatim; you orchestrate and never write the prose yourself.",
): Block(),
Input(model="opus", prompt="draft the release notes for v2"): Allow(),
Input(model="claude-opus-5-5", prompt="Write the README quickstart for this repo"): Allow(),
Input(prompt="write the README quickstart"): Allow(),
Input(prompt="draft the release notes for v2"): Allow(),
Input(
agent_type="ship-pr",
prompt="Write the PR description below to a body file and open the PR.\ntitle: fix retry\nbody: ...",
): Allow(),
Input(
agent_type="codex:codex-wrapper",
prompt="Rewrite the README quickstart in the technical-builder voice",
agent_type="ship-pr",
model="opus",
prompt="Write the PR description below to a body file and open the PR.\ntitle: fix retry\nbody: ...",
): Allow(),
Input(
model="opus",
prompt="Fix the import in cli.py, have the codex skill review the diff, then draft the "
"CHANGELOG entry yourself.",
llm={"block": False},
): Allow(),
Input(model="sonnet", prompt="review the README for factual errors"): Allow(),
Input(model="sonnet", prompt="update the retry backoff config"): Allow(),
Input(prompt="update the retry backoff config"): Allow(),
Input(model="haiku", prompt="label each README section with its Diataxis mode"): Allow(),
Input(agent_type="Explore", model="sonnet", prompt="find where the README quickstart is written"): Allow(),
Input(agent_type="claude-code-guide", model="sonnet", prompt="explain how the docs get updated"): Allow(),
Input(model="opus", prompt="Update the retry backoff config per the spec in docs/plan.md"): Allow(),
Input(
model="opus",
model="sonnet",
prompt="Fix the failing test in cli.py. Do NOT edit the CHANGELOG — a sibling owns updating it",
): Allow(),
Input(
model="opus",
prompt="Fix the failing test in cli.py; do NOT edit CHANGELOG.md. Then draft the release notes.",
llm={"block": False},
agent_type="codex:codex-wrapper",
prompt="Review the diff for correctness; return findings as file:line JSON",
): Allow(),
},
)
Expand All @@ -285,8 +302,8 @@ def content(self, evt: BaseHookEvent) -> str | None:
llm_nudge(
Prompt.load("models/implementation_spawn_nudge"),
message=(
"Implementation subagents run on `model='opus'` and repetitive N-unit sweeps on gpt-6-astra via "
"`codex:codex-wrapper`; fable is only for sensitive or error-prone code. "
"Implementation subagents run on `model='opus'` and repetitive N-unit sweeps on gpt-6.1-sol via "
"`codex:codex-wrapper`; fable is only for the most sensitive code. "
"Re-spawn with that route, at `effort='high'` for a bounded change and `effort='xhigh'` otherwise."
),
contexts=[DelegatedSpawn()],
Expand All @@ -307,7 +324,7 @@ def content(self, evt: BaseHookEvent) -> str | None:
),
Input(
prompt="Convert the eleven test modules under tests/legacy/ to pytest, one per lane, per the worked example"
): Warn(pattern="gpt-6-astra"),
): Warn(pattern="gpt-6.1-sol"),
Input(model="opus", prompt="implement the pagination endpoint in api/users.py"): Allow(),
Input(model="sonnet", prompt="scan the repo for TODO markers"): Allow(),
Input(agent_type="Explore", prompt="find where the config loader lives"): Allow(),
Expand All @@ -319,7 +336,7 @@ def content(self, evt: BaseHookEvent) -> str | None:
Prompt.load("models/inline_edit_nudge"),
message=(
"Sizable implementation is delegated, not edited inline on the main loop. "
"Spawn an `Agent` with `model='opus'`, or a typed `model='fable'` subagent for sensitive code, "
"Spawn an `Agent` with `model='opus'`, or a typed `model='fable'` subagent for the most sensitive code, "
"and hand it this change."
),
contexts=[InlineEdit()],
Expand Down Expand Up @@ -405,7 +422,7 @@ def content(self, evt: BaseHookEvent) -> str | None:
Prompt.load("models/review_routing_spawn_nudge"),
label="review_routing_spawn",
message=(
"Code review, security audit, and bug diagnosis route to gpt-6-astra through codex, not a Claude subagent. "
"Code review, security audit, and bug diagnosis route to gpt-6.1-sol through codex, not a Claude subagent. "
"Spawn `subagent_type: 'codex:codex-wrapper'` with the self-contained question, or run `Skill(codex)` "
"from the main conversation."
),
Expand All @@ -425,7 +442,7 @@ def content(self, evt: BaseHookEvent) -> str | None:
agent=False,
transcript=False,
tests={
Input(prompt="Review the diff for correctness and concurrency issues"): Warn(pattern="gpt-6-astra"),
Input(prompt="Review the diff for correctness and concurrency issues"): Warn(pattern="gpt-6.1-sol"),
Input(model="fable", prompt="Adversarially refute this finding: the retry loop is wrong"): Warn(
pattern="codex"
),
Expand All @@ -450,7 +467,7 @@ def content(self, evt: BaseHookEvent) -> str | None:
model="opus",
prompt="Synthesize the confirmed review findings and decide which to fix",
): Allow(),
Input(prompt="Audit auth/session.py for security vulnerabilities"): Warn(pattern="gpt-6-astra"),
Input(prompt="Audit auth/session.py for security vulnerabilities"): Warn(pattern="gpt-6.1-sol"),
Input(prompt="Verify the input-validation change blocks path traversal"): Warn(pattern="codex"),
Input(prompt="Verify the pagination change renders the last page correctly"): Allow(),
Input(
Expand Down Expand Up @@ -488,33 +505,43 @@ def content(self, evt: BaseHookEvent) -> str | None:
deliverable_rubric=DELIVERABLE_NUDGE_RUBRIC,
),
message=(
"Workflow prose stages are written by gpt-6-astra through codex, which no `model:` pin reaches. "
"Give the stage `agentType: 'codex:codex-wrapper'` with a self-contained writing brief."
"Workflow prose stages are written by Claude Opus, never by codex, astra, sonnet, haiku, or fable. "
"Pin each prose stage `model: 'opus'` or leave it unpinned, with no `codex:codex-wrapper` agentType."
),
contexts=[ProseWorkflowScript()],
events=Event.PreToolUse,
only_if=[Tool("Workflow")],
skip_if=[
And(
Not(WorkflowScript(model=r"(?i)sonnet|haiku|fable")),
Not(WorkflowScript(pattern=r"(?i)\b(codex|astra)\b")),
)
],
max_fires=2,
max_context=16_000,
agent=False,
transcript=False,
tests={
Input(script="steps:\n - agent: write the README intro\n model: 'sonnet'\n"): Warn(pattern="astra"),
Input(script="steps:\n - agent: write the README intro\n"): Warn(pattern="astra"),
Input(script="steps:\n - agent: write the README intro\n model: 'fable'\n"): Warn(pattern="astra"),
Input(
script="agent('Rewrite the README quickstart', {agentType: 'codex:codex-wrapper'})",
llm={"fire": False},
): Allow(),
Input(script="steps:\n - agent: write the README intro\n model: 'sonnet'\n"): Warn(pattern="opus"),
Input(script="steps:\n - agent: write the README intro\n model: 'fable'\n"): Warn(pattern="opus"),
Input(script="agent('Rewrite the README quickstart', {agentType: 'codex:codex-wrapper'})"): Warn(
pattern="opus"
),
Input(script="steps:\n - agent: write the README intro\n"): Allow(),
Input(script="agent('Rewrite the README quickstart', {model: 'opus'})"): Allow(),
Input(script="steps:\n - agent: fix the retry backoff\n model: 'sonnet'\n"): Allow(),
Input(script="agent('Audit docs/architecture.md for stale claims', {model: 'opus'})"): Allow(),
Input(
script="agent('recon the module map', {model: 'sonnet'})\n"
"// every prose stage goes to codex:codex-wrapper\n"
script="agent('recon the module map', {model: 'sonnet'})\n// every prose stage runs on opus\n"
): Allow(),
Input(
script="agent('Fix the import in cli.py. Do NOT edit CHANGELOG.md — a sibling owns it', {model: 'opus'})",
): Allow(),
Input(
script="agent('Review the diff for correctness', {agentType: 'codex:codex-wrapper'})\n"
"agent('Write the CHANGELOG entry for the fix', {model: 'opus'})",
llm={"fire": False},
): Allow(),
},
)

Expand All @@ -526,7 +553,7 @@ def content(self, evt: BaseHookEvent) -> str | None:
),
label="review_routing_workflow",
message=(
"Workflow review, security-audit, and bug-diagnosis stages route to gpt-6-astra through codex. "
"Workflow review, security-audit, and bug-diagnosis stages route to gpt-6.1-sol through codex. "
"Give each such stage `agentType: 'codex:codex-wrapper'` with the self-contained question as its prompt."
),
contexts=[WorkflowScriptSource()],
Expand All @@ -544,7 +571,7 @@ def content(self, evt: BaseHookEvent) -> str | None:
pattern="codex"
),
Input(script="agent(`Adversarially refute: ${f.title}`, {model: 'fable', effort: 'max'})"): Warn(
pattern="gpt-6-astra"
pattern="gpt-6.1-sol"
),
Input(
script="agent('Write a self-contained codex prompt reviewing this diff, "
Expand All @@ -571,24 +598,24 @@ def content(self, evt: BaseHookEvent) -> str | None:
llm={"fire": False},
): Allow(),
Input(script="agent(`Audit the login flow for auth bypass and injection; return findings as JSON`)"): Warn(
pattern="gpt-6-astra"
pattern="gpt-6.1-sol"
),
Input(script="agent('Verify the CLI renders the last page correctly')"): Allow(),
Input(
script=(
"const astraOrOpus = async (prompt, key) => {\n"
"const solOrOpus = async (prompt, key) => {\n"
" const r = await agent(prompt, { agentType: 'codex:codex-wrapper', "
"label: `${key}:astra`, phase: 'Review', schema: REVIEW })\n"
" if (r) return { ...r, lane_model: 'astra' }\n"
" log(`${key}: astra empty — opus fallback`)\n"
"label: `${key}:sol`, phase: 'Review', schema: REVIEW })\n"
" if (r) return { ...r, lane_model: 'sol' }\n"
" log(`${key}: sol empty — opus fallback`)\n"
" const f = await agent(prompt, { label: `${key}:opus`, phase: 'Review', schema: REVIEW })\n"
" return f ? { ...f, lane_model: 'opus' } : null\n"
"}"
),
llm={"fire": False},
): Allow(),
Input(
script="export const meta = { description: 'refuter pass; astra lane unavailable — "
script="export const meta = { description: 'refuter pass; sol lane unavailable — "
"opus escalation per models table' }\n"
"const f = await agent(`Adversarially refute: ${finding.title}`)",
llm={"fire": False},
Expand Down
4 changes: 2 additions & 2 deletions ...in_hook/builtin_packs/general/hooks/prompts/fragments/workflow_script_header.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
Its header quotes verbatim excerpts around the script's model pins; … marks
elided text within an excerpt. A header reading "excerpts around every model
pin in this script" is complete: a stage not quoted there carries no pin and so
runs opus — workflow stages never inherit fable, so an unpinned stage is
opus-routed, not fable-routed. A header reading "excerpts around the first model
runs opus — workflow stages never inherit the root's model, so an unpinned stage
is opus-routed. A header reading "excerpts around the first model
pins in this script" and ending in a "… [+N more model pins not excerpted]" line
was capped: absence from that header no longer proves a stage is unpinned — check
the script source before concluding a stage carries no pin.
4 changes: 2 additions & 2 deletions ...ain_hook/builtin_packs/general/hooks/prompts/models/browser_delegation_nudge.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,15 +1,15 @@
Decide whether the main agent should delegate this sustained browser automation
instead of driving it inline.

The main loop runs on fable-5; this session has been driving the browser directly —
The main loop is the opus root orchestrator; this session has been driving the browser directly —
a run of `agent-browser` / `playwright` calls (click, fill, snapshot, scrape, QA
step) through the recent tool calls shown below.

The Model Routing rubric: sustained tool-driving — browser automation, QA sweeps,
and bulk extract/fill/snapshot — belongs on opus at xhigh. Delegate it to a
model='opus' subagent at effort='xhigh' to drive agent-browser and return findings.
When the site needs the user's own login, use an agent-browser-with-cookies
teammate on opus at xhigh. The session's fable model does not change that route.
teammate on opus at xhigh. The root's own model does not change that route.
Keep browser work inline only for a single gated, stateful, or authenticated
interaction the main agent just decided to run: a go/no-go verification, one
confirming screenshot, or a login+2FA flow it must hold open.
Expand Down
Loading
Loading