Repository navigation
S26-E0.5: remove the self-written Codex harness - #280
Closed
michaellchung wants to merge 2 commits into
Closed
michaellchung wants to merge 2 commits into
michaellchung wants to merge 2 commits into
Conversation
Harbor (task loading, sandbox via the boat provider, agent, verifier) plus the records proxy replace it. Removed: yeto/rl/harness/codex (agent, sandbox broker, TB2 provider, tool exec, grading, Codex bridge), the Miles codex glue, SecRLEnv retry, the RLF family, the TB2 reward-env adapter and Modal reward sandbox, the old eval runners, the launcher Codex bundle/contract path, the SSH-harness Codex/SecRLEnv daemon path, and their tests. Kept or moved: the Responses SSE writer into the records proxy, refresh_segment_counts into yeto.rl.compaction, the SecRLEnv outcome check into yeto.rl.secrlenv_outcome (trajectory evidence of that kind still verifies), the CompactionRL launch check into yeto.launch.harness_env. Removed Codex/SecRLEnv launcher flags are refused. The last commit with the old harness is tagged s26-pre-harness-removal. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The removed agents made these paths unreachable: - --codex-backend-profile, --codex-reasoning-effort, --codex-exclude-reasoning-from-loss and --secrlenv-max-infrastructure-replacements become launcher-only flags (still refused); the island no longer parses them or --codex-harness-contract, and no longer copies them into miles_args. - The SSH harness drops its SecRLEnv-only eval path: eval checkpoint contract, prepare options, checkpoint deploy, eval-only argv, syncer lifecycle evidence, eval collect/verify, and the codex fields of the plan. Plans that still carry eval, daemon or Codex fields are refused. - The island checkpoint config keeps its codex/secrlenv keys as None so checkpoints written before the removal still resume. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
Author
|
Closing: all commits of |
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.
Removes the legacy self-written Codex harness (task loading, sandbox, tool execution, grading, Codex runtime; ~29k lines incl. tests). The Harbor + recording proxy + boat path replaces it.
refresh_segment_countsintoyeto/rl/compaction.py, SecRLEnv outcome checks intoyeto/rl/secrlenv_outcome.py.s26-pre-harness-removal.Tests: full suite 6179 passed / 0 failed (remote CPU runner); Harbor, boat, recording proxy and tools/eval tests pass.
🤖 Generated with Claude Code