fix(runner): list codex in the /health harnesses payload

[WahidSaeed]

Summary

• The runner's HARNESS_KINDS list in services/runner/src/version.ts was not extended when Codex became a harness of its own (its own model catalog, its own CODEX_HOME setup, routed through run-plan.ts like pi_core/claude/pi_agenta).
• Its only consumer is runnerInfo().harnesses, surfaced on GET /health — nothing is gated on the list, but a client probing /health concludes Codex is unsupported on a runner where it actually works.

Fixes #5693

Demo

Backend-only, non-visual change (one string added to a hardcoded array) — the demo below is real captured output: the actual pre-fix line (git show of the base commit), the actual runnerInfo() JSON after the fix (run directly against the fixed code), and the actual passing test run for version.test.ts.

Test plan

• Added services/runner/tests/unit/version.test.ts asserting HARNESS_KINDS includes codex and runnerInfo().harnesses surfaces it.
• pnpm vitest run --project unit: 1479/1479 tests pass across 97 files (full runner unit suite, not just the new file).
• tsc --noEmit: no errors.

[vercel]

@WahidSaeed is attempting to deploy a commit to the agenta projects Team on Vercel.

A member of the Team first needs to authorize it.

[github-actions]

✅ Thanks @WahidSaeed! This PR now meets the contribution requirements and has been reopened. A maintainer will review it soon.

[CLAassistant]

All committers have signed the CLA.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d7d61c35-ad09-4ccd-8341-99bb3b3089a0

📥 Commits

Reviewing files that changed from the base of the PR and between e73fb2e and 09bf666.

⛔ Files ignored due to path filters (1)

• .github/pr-assets/5693-health-codex-harness.png is excluded by !**/*.png

📒 Files selected for processing (2)

• services/runner/src/version.ts
• services/runner/tests/unit/version.test.ts

🚧 Files skipped from review as they are similar to previous changes (2)

• services/runner/tests/unit/version.test.ts
• services/runner/src/version.ts

---

📝 Walkthrough

Summary by CodeRabbit

• New Features

  • Added Codex as a supported runner harness.
  • Codex availability is now shown in runner health information.

• Tests

  • Added coverage for Codex support and health reporting.

Walkthrough

The runner now includes codex in HARNESS_KINDS. Unit tests verify that the harness list and runnerInfo().harnesses expose Codex.

Changes

Codex harness identity

Layer / File(s)	Summary
Harness identity and validation services/runner/src/version.ts, services/runner/tests/unit/version.test.ts	The exported harness list includes codex. Unit tests verify the list and the runner health identity payload.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

• Agenta-AI/agenta#5706: Both PRs update harness-kind handling and explicitly support codex in related code paths.
• Agenta-AI/agenta#5716: Both PRs update HARNESS_KINDS and validate Codex in runner health data.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes adding Codex to the runner health harnesses payload.
Description check	✅ Passed	The description directly explains the Codex health payload bug, the fix, and the validation performed.
Linked Issues check	✅ Passed	The changes satisfy issue #5693 by adding codex to HARNESS_KINDS and testing its exposure in runnerInfo().harnesses.
Out of Scope Changes check	✅ Passed	The changes are limited to the required harness list update and focused unit tests for the linked issue.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[WahidSaeed]

Hi maintainers — this PR was auto-closed for missing a demo, but I think it's a mismatch between the bot and the written policy: CONTRIBUTING.md and the first-pr guide both say a demo is required for UI/UX changes, and that non-visual changes should mark the demo section "not applicable." This change is a one-line addition to a hardcoded string array (HARNESS_KINDS in services/runner/src/version.ts) surfaced verbatim on GET /health — there's no UI or visual behavior to record.

The PR description already includes the concrete before/after of the only observable output (the /health JSON payload) plus full test evidence (1479/1479 unit tests passing, new regression test, clean typecheck). Happy to add anything else that would help review — could a maintainer take a look and reopen if this qualifies for the non-visual exemption?

[coderabbitai]

Note

GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer.