fix(#324): config spec re-baseline; fugu-ultra manual-only everywhere#326
Conversation
…y everywhere Operator ruling (2026-07-15): the codex-fugu baseline is fugu; premium tiers (fugu-ultra, fable) are high-cost, special-use, manual-only — never a default, workerMix pin, modelByHarness baseline, or captured spec value. - live ao config fixed first (workerMix fugu-ultra→fugu; deprecated respawn stanza dropped), then all four project specs re-baselined from live via ops/project-config.mjs capture (adds agent-vault spec, absorbs #302's per-harness pin + canonical-name normalization) - knownModelsForHarness: baseline models listed first — fugu for codex-fugu (was fugu-ultra ONLY), gpt-5.5 for codex (was only the legacy *-codex names #302's DefaultCodexModel comment documents as bogus on this account) - routing-label descriptions: agent:fugu says (fugu), agent:codex says (gpt-5.5) - model-tier policy recorded in ops/project-config/README.md - every casual fugu-ultra test fixture neutralized (fugu or an explicitly-fake premium name); catalog test asserts the baseline is listed and truthfully unknown when unprobed Closes #324 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K1BedAdnAVFqVQTBoVyyp5
b10fc86 to
5dfb79b
Compare
|
Reviewer finding (workspace |
|
Final review verdict for head |
Summary
Executes #324 plus the operator's 2026-07-15 ruling: codex-fugu baseline is
fugu; premium tiers (fugu-ultra,fable) are manual-only — never a default, mix pin, baseline, or captured spec value.fugu-ultra→fuguand the deprecatedrespawnstanza dropped (CAS write against fresh ETag). Verified: live config has zerofugu-ultra, zerorespawn.capture), including a new trackedagent-vault.json; absorbs Project config write safety and daemon defaults (#298 P1) #302's per-harness pins + canonical names.knownModelsForHarnesslisted ONLYfugu-ultrafor codex-fugu (no baseline at all — the likely origin of the bad spec pin) and only legacy*-codexnames for codex without the realgpt-5.5; both now baseline-first with explanatory comments. Routing-label descriptions corrected (agent:fugu (fugu),agent:codex (gpt-5.5)).ops/project-config/README.mdrecords the tier ruling; every casualfugu-ultrafixture in tests neutralized; the catalog test now asserts the baseline is listed and truthfullyunknownwhen unprobed.Post-merge:
node ops/project-config.mjs check --allis expected green (verified locally pre-PR against live).Test plan
npm run ci:backend— PASS (build, vet, race tests, golangci-lint 0 issues; a stale-cache false failure from a deleted worktree was diagnosed and cleared — cache clean, not a code change)npm run test:ops— PASS (474/474)format:check+ prettier over staged files — PASSCloses #324. Refs #302, #313, #320.
Need help on this PR? Tag
/codesmithwith what you need. Autofix is disabled.