Skip to content

[5693] fix(backend): report Codex in runner health - #5716

Closed
huige66631 wants to merge 1 commit into
Agenta-AI:mainfrom
huige66631:codex/fix-5693-codex-health
Closed

[5693] fix(backend): report Codex in runner health#5716
huige66631 wants to merge 1 commit into
Agenta-AI:mainfrom
huige66631:codex/fix-5693-codex-health

Conversation

@huige66631

Copy link
Copy Markdown

Context

The runner's /health endpoint omitted codex from the harnesses list, even though Codex runs are supported. The response therefore under-reported the runner capabilities.

Changes

Add codex to HARNESS_KINDS, the source used to build the health response. Extend the /health unit test to assert that the returned harness list includes codex.

Fixes #5693

Tests / notes

  • corepack pnpm exec vitest run --project unit tests/unit/server.test.ts
  • corepack pnpm run typecheck
  • git diff --check

The focused test and typecheck pass. The full suite was also attempted, but this Windows sparse checkout reports unrelated failures involving POSIX permissions, path separators, and cross-tree fixtures.

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 3, 2026
@vercel

vercel Bot commented Aug 3, 2026

Copy link
Copy Markdown

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

A member of the Team first needs to authorize it.

@dosubot dosubot Bot added the backend label Aug 3, 2026
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Hi @huige66631, thanks for opening a pull request. 🙏

This PR was automatically closed because it does not yet meet our contribution requirements:

  • The Summary section is missing or empty. Describe what changed and why using the PR template.
  • This PR changes functional code (SDK, API, or frontend) but includes no demo. Add a screenshot or short video of the change. Only test-only, docs-only, or chore changes may skip it.

We ask for this so every change is documented and demonstrably tested before review.

How to get it reopened
Update the PR description (and add a demo recording if your change touches functional code). The bot reopens the PR automatically once the requirements are met. No need to open a new one.

See the Contributing guide and Creating your first PR. If you think this was closed in error, leave a comment and a maintainer will take a look.

@github-actions github-actions Bot added the incomplete pr PR is missing required template sections or a demo recording label Aug 3, 2026
@github-actions github-actions Bot closed this Aug 3, 2026
@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: bfcc295b-e1ae-44bf-8405-d3c3ab3dc9ea

📥 Commits

Reviewing files that changed from the base of the PR and between 69e2165 and 3dca654.

📒 Files selected for processing (2)
  • services/runner/src/version.ts
  • services/runner/tests/unit/server.test.ts

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Summary by CodeRabbit

  • New Features
    • Added support for the Codex harness.
    • Health status information now reports Codex among the available harnesses.

Walkthrough

The runner now includes "codex" in its exported harness list. The /health unit test verifies that the response reports Codex.

Changes

Codex health reporting

Layer / File(s) Summary
Codex harness contract and validation
services/runner/src/version.ts, services/runner/tests/unit/server.test.ts
HARNESS_KINDS includes "codex". The /health test verifies that body.harnesses contains "codex".

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

Possibly related PRs

✨ 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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend incomplete pr PR is missing required template sections or a demo recording size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(bug) The runner's /health endpoint doesn't list codex as a harness

1 participant