Skip to content

feat(codex): Codex harness backend - #11

Merged
RobertCrupa merged 2 commits into
mainfrom
feat/codex-harness
Sep 29, 2026
Merged

RobertCrupa merged 2 commits into
mainfrom
feat/codex-harness

Conversation

@RobertCrupa

@RobertCrupa RobertCrupa commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

Adds Codex as a second harness adapter alongside Claude Code: isolated sessions with MCP and process limits (temp `CODEX_HOME`, empty working directory, shell tools disabled, read-only sandbox, pre-approved allowlisted MCP tools so `call-actor` isn't rejected), a JSONL event parser with captured fixtures, and the runner/docs wiring to select it via `harness.kind`.

Depends on scenario suites from #10. Last PR in this stack.

@RobertCrupa
RobertCrupa added this pull request to stack #12 September 21, 2026 13:26
@RobertCrupa
RobertCrupa force-pushed the feat/codex-harness branch 3 times, most recently from d57c6c0 to eb9d3f6 Compare September 29, 2026 12:45
Base automatically changed from feat/scenario-suites to main September 29, 2026 12:46
Adds Codex as a second harness adapter alongside Claude Code: isolated
sessions with MCP and process limits (temp CODEX_HOME, empty working
directory, shell tools disabled, read-only sandbox, pre-approved allowlisted
MCP tools so call-actor is not rejected), a JSONL event parser with captured
fixtures, and workspace test/documentation wiring. Builds on
feat/scenario-suites.
The repo's own suites write `input.prompt`; the mcp-server-evals datasets
write `input.query`. Reading only `prompt` fell through to
`JSON.stringify(input)`, so an item from those datasets handed the agent
the literal string `{"query":"..."}` and still produced a trace, a rubric
verdict and scores that all looked valid.

Temporary. The fix is one key across every dataset, asserted at sync time
rather than reinterpreted at run time: apify/ai-team#327. Drop the `query`
fallback once the datasets are normalised.
@apify-service-account apify-service-account added the tested Temporary label used only programatically for some analytics. label Sep 29, 2026
@apify-service-account apify-service-account added the t-ai Issues owned by the AI team. label Sep 29, 2026
@RobertCrupa
RobertCrupa merged commit 03b1dfb into main Sep 29, 2026
3 of 4 checks passed
@RobertCrupa
RobertCrupa deleted the feat/codex-harness branch September 29, 2026 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-ai Issues owned by the AI team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants