Skip to content

fix(bin): allow max Codex effort and harden supervision - #2665

Open
V4f1k wants to merge 6 commits into
kunchenguid:mainfrom
V4f1k:fm/codex-max-allowlist-fix
Open

fix(bin): allow max Codex effort and harden supervision#2665
V4f1k wants to merge 6 commits into
kunchenguid:mainfrom
V4f1k:fm/codex-max-allowlist-fix

Conversation

@V4f1k

@V4f1k V4f1k commented Aug 20, 2026

Copy link
Copy Markdown

Intent

Propustit effort max pro Codex ve firstmate dispatch nástrojích, protože allowlist je zastaralý vůči instalovanému codex-cli. Nezávisle ověřit instalovaný codex-cli verzi, codex debug models a levnou pozitivní sondu codex exec --skip-git-repo-check --sandbox read-only -m gpt-5.6-luna -c model_reasoning_effort="max", plus negativní kontrolu s bogus, a doložit, že max je podporovaná hodnota model_reasoning_effort. V bin/fm-spawn.sh ve funkci effort_flag_for_harness přijmout max v codex větvi a opravit zastaralý komentář. Shared firstmate effort vocabulary je low, medium, high, xhigh, max; ultra ani nižší Codex hodnoty se nemají zavádět. V bin/fm-bootstrap.sh má effort_ok pro codex přijímat max. Aktualizovat .agents/skills/harness-adapters/SKILL.md v launch-profile tabulce a codex sekci podle instalované verze, data 2026-08-20 a ověření codex-cli 0.147.0, včetně max; uvést ultra jako katalogový fakt pro gpt-5.6-sol a gpt-5.6-terra, ale firstmate ho zatím nemapuje. Projít a upravit nebo doplnit kolokované tests/ pro effort validaci a spawn flagy a zachovat shellcheck-clean bin skripty. Needitovat config/crew-dispatch.json ani žádný gitignored config. Akceptace: fm-spawn s codex a effort max emituje -c model_reasoning_effort="max"; bootstrap validace codex:max prochází; dokumentace odpovídá instalované verzi a datu; ostatní harnessy nemění chování. Změna je úmyslně omezena na Codex allowlist, bootstrap validaci, odpovídající dokumentaci a regresní testy.

What Changed

  • Allow Codex dispatch profiles to accept max in bootstrap validation and emit -c 'model_reasoning_effort="max"'; update harness guidance and runtime evidence for codex-cli 0.147.0 while keeping Firstmate's shared vocabulary bounded at max.
  • Strengthen generated worker briefs and supervision flow with same-turn helper cleanup, continuation/wait contracts, resolved status handling, and Claude pending:handling recovery recognition without weakening strict turn-end checks.
  • Make Linux Herdr remote-doctor startup detach safely from caller descriptors, bound and reap never-ready starts, and document and regression-test the readiness behavior.

Risk Assessment

🚨 High: Codex změna je úzká a konzistentní, ale reviewed branch obsahuje další funkční změny mimo autoritativně omezený kontrakt a mění chování jiných harnessů.

Testing

Spawn a bootstrap behavior testy i živé sondy codex-cli 0.147.0 potvrdily max, odmítnutí bogus, katalogové ultra jen pro Sol/Terra a zachování ostatních harnessů; shellcheck nebyl v této testovací fázi spuštěn podle jejího omezení.

Evidence: Codex effort CLI transcript

Source: Codex effort CLI transcript

$ codex --version
codex-cli 0.147.0
$ codex debug models | jq -r ...
gpt-5.6-sol	low,medium,high,xhigh,max,ultra
gpt-5.6-terra	low,medium,high,xhigh,max,ultra
gpt-5.6-luna	low,medium,high,xhigh,max
$ codex exec ... model_reasoning_effort="max" ...
exit=0
OpenAI Codex v0.147.0
model: gpt-5.6-luna
reasoning effort: max
OK
OK
$ codex exec ... model_reasoning_effort="bogus" ...
exit=1
    "message": "[ReasoningEffortParam] [reasoning.effort] [invalid_enum_value] Invalid value: 'bogus'. Supported values are: 'none', 'minimal', 'low', 'medium', 'high', 'xhigh', and 'max'.",
    "message": "[ReasoningEffortParam] [reasoning.effort] [invalid_enum_value] Invalid value: 'bogus'. Supported values are: 'none', 'minimal', 'low', 'medium', 'high', 'xhigh', and 'max'.",
Evidence: fm-spawn Codex max launch

Source: fm-spawn Codex max launch

focused test: test_codex_threads_max_effort
test_exit=0
+ launch='env -u CURSOR_AGENT -u CURSOR_INVOKED_AS codex --model '\''gpt-5'\'' -c '\''model_reasoning_effort="max"'\'' --dangerously-bypass-approvals-and-sandbox -c "notify=[\"bash\",\"-c\",\"touch '\''/private/var/folders/mz/3dbzbvkd1375djgpnq7r7hj40000gp/T/fm-spawn-dispatch-profile.3GPqH2/profile-codex-max/home/state/profile-codex-max-z4.turn-ended'\''\"]" "$('\''/Users/computer/.no-mistakes/worktrees/2f3dd046dbd1/01M0EY2QMSWD9NFZRSYB2C1Y10/bin/fm-operational-input.sh'\'' encode launch-brief < '\''/var/folders/mz/3dbzbvkd1375djgpnq7r7hj40000gp/T//fm-spawn-dispatch-profile.3GPqH2/profile-codex-max/home/data/profile-codex-max-z4/brief.md'\'')"'
ok - codex receives max through model_reasoning_effort
Evidence: fm-bootstrap Codex max validation

Source: fm-bootstrap Codex max validation

codex:max exit=0 output=<empty>
codex:ultra exit=0 output=CREW_DISPATCH: invalid config/crew-dispatch.json - invalid effort: codex:ultra

Pipeline

Updates from git push no-mistakes

✅ **intent** - passed

✅ No issues found.

⚠️ **Rebase** - 1 warning

Push main to origin, or rebase your branch onto origin/main, before gating.

⚠️ **Review** - 1 error
  • 🚨 bin/backends/herdr.sh:1445 - Intent vyžaduje: „Změna je úmyslně omezena na Codex allowlist, bootstrap validaci, odpovídající dokumentaci a regresní testy“ a ostatní harnessy nemají měnit chování. Reviewed rozsah ale přidává změnu Herdr lifecycle na bin/backends/herdr.sh:1445, Claude/ostatní supervision logiku na bin/fm-wake-lib.sh:174 a nové brief kontrakty na bin/fm-brief.sh:301. Nejmenší konformní alternativa je ponechat pouze Codex změnu, bootstrap, dokumentaci a relevantní testy; zachování těchto hunků mění další runtime kontrakty. Prosím potvrdit, zda je toto rozšíření záměrné, nebo odstranit nesouvisející commity.
✅ **Test** - passed

✅ No issues found.

  • bash tests/fm-spawn-dispatch-profile.test.sh
  • bash tests/fm-bootstrap.test.sh
  • Focused test_codex_threads_max_effort s zachyceným launch příkazem
  • Izolované spuštění fm-bootstrap.sh pro codex:max a codex:ultra
  • codex --version
  • codex debug models
  • Pozitivní codex exec ... model_reasoning_effort=&#34;max&#34;
  • Negativní codex exec ... model_reasoning_effort=&#34;bogus&#34;
✅ **Document** - passed

✅ No issues found.

✅ **Lint** - passed

✅ No issues found.

✅ **Push** - passed

✅ No issues found.

V4f1k and others added 6 commits August 14, 2026 20:59
* fix: odpojit Linux Herdr server od zděděných deskriptorů

* no-mistakes(review): fix: Zombie-safe liveness predikáty v Linux regresním testu

* no-mistakes(document): Aktualizována dokumentace bounded Herdr startu

* test: počkat na vlastnictví setsid skupiny
* fix(brief): preserve worker decision and evidence contracts

* no-mistakes(review): Fix brief continuation and pipeline-entry contracts

* no-mistakes(review): Fix post-response pipeline driving contract

* no-mistakes(document): Align ask-user guidance with same-turn waiting
* fix(brief): require same-turn helper cleanup

* no-mistakes(document): Align Codex task status documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant