fix(bin): allow max Codex effort and harden supervision - #2665
Open
V4f1k wants to merge 6 commits into
Open
Conversation
* 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
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.
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
maxin 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 atmax.resolvedstatus handling, and Claudepending:handlingrecovery recognition without weakening strict turn-end checks.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éultrajen 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
Evidence: fm-spawn Codex max launch
Source: fm-spawn Codex max launch
Evidence: fm-bootstrap Codex max validation
Source: fm-bootstrap Codex max validation
Pipeline
Updates from git push no-mistakes
✅ **intent** - passed
✅ No issues found.
.agents/skills/afk/SKILL.md- branch carries 4 commit(s) that exist on your local main branch but were never pushed to origin/main; rebasing would bundle this unrelated work (104 file(s)) into the PR:Push main to origin, or rebase your branch onto origin/main, before gating.
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 nabin/backends/herdr.sh:1445, Claude/ostatní supervision logiku nabin/fm-wake-lib.sh:174a nové brief kontrakty nabin/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.shbash tests/fm-bootstrap.test.shFocusedtest_codex_threads_max_efforts zachyceným launch příkazemIzolované spuštěnífm-bootstrap.shprocodex:maxacodex:ultracodex --versioncodex debug modelsPozitivnícodex exec ... model_reasoning_effort="max"Negativnícodex exec ... model_reasoning_effort="bogus"✅ **Document** - passed
✅ No issues found.
✅ **Lint** - passed
✅ No issues found.
✅ **Push** - passed
✅ No issues found.