Verify coding-engine capabilities before dispatch - #592
Conversation
|
@codex review |
|
@greptileai review |
Greptile SummaryAdds a canonical coding-engine readiness contract and enforces it across setup, diagnostics, and dispatch.
Confidence Score: 5/5The PR appears safe to merge. No blocking failures remain.
What T-Rex did
|
| Filename | Overview |
|---|---|
| lib/agent_runner/engine_registry.py | Introduces canonical engine descriptors, capability metadata, sanitized protocol probes, authentication checks, and bounded readiness caching. |
| lib/agent_runner/process.py | Gates real Claude and Codex adapters on readiness and adds hermetic, ephemeral Codex invocation flags. |
| lib/agent_runner/config.py | Derives supported engine choices from the registry and converts invalid operator configuration into a non-dispatching state. |
| lib/server/setup.py | Replaces executable-presence detection with canonical readiness inventory in setup status. |
| bin/alfred | Adds engine diagnostics and discovers configured engine-aware agents from deployed scheduler state. |
| clients/desktop/src/components/onboarding/EngineStep.tsx | Distinguishes detected engines from compatible, authenticated engines and exposes detailed probe states. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
Config[Configured engine] --> Registry[Capability registry]
Registry --> Probe[Protocol, version, and auth probe]
Probe -->|Ready| Dispatch[Engine adapter dispatch]
Probe -->|Not ready| Failure[Structured readiness failure]
Registry --> Setup[Setup API and desktop readiness UI]
Registry --> Doctor[CLI engine doctor]
Reviews (10): Last reviewed commit: "fix engine readiness and doctor roster p..." | Re-trigger Greptile
Greptile SummaryAdds a canonical coding-engine capability registry and exposes protocol/authentication readiness throughout setup and desktop surfaces.
Confidence Score: 2/5This should not merge until autonomous dispatch enforces engine readiness and Codex authentication is not inferred from API-key presence alone. Scheduled runners still invoke engines after binary-only preflight, while an arbitrary nonempty OPENAI_API_KEY marks Codex ready and routes work to it before real authentication occurs. lib/agent_runner/config.py, lib/agent_runner/engine_registry.py, and lib/agent_runner/process.py
What T-Rex did
Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart LR
Registry[Engine capability registry] --> Probe[Protocol and auth probes]
Probe --> Setup[Setup status]
Setup --> Desktop[Onboarding and settings]
Setup --> Converse[Conversation engine discovery]
Config[Scheduled agent configuration] --> Preflight[Binary-only preflight]
Preflight --> Invoke[Autonomous engine invocation]
Probe -. readiness not enforced .-> Preflight
Reviews (1): Last reviewed commit: "feat(engines): verify harness capabiliti..." | Re-trigger Greptile |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9c9d8d2f79
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
@greptileai review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a823e35b7a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
@greptileai review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ddf7aa8bcb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
@greptileai review |
|
Codex Review: Something went wrong. Try again later by commenting “@codex review”. ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4b7f438390
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Something went wrong. Try again later by commenting “@codex review”. ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
|
Codex Review: Something went wrong. Try again later by commenting “@codex review”. ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5f94a7c98b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@greptile review |
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 82862e6cf5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@greptile review |
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ce32095866
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ce32095866
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@greptile review Exact head: |
|
@codex review Exact head: |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@greptile review Exact head: |
|
@greptileai review Please review exact HEAD |
|
@codex review |
|
@greptileai review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 71cbd0b6e5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
@greptileai review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c3980812d8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
@greptileai review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 381b547fb2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
@greptileai review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1e67d954bb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| markers=( | ||
| "--output-last-message", | ||
| "--sandbox", | ||
| "--cd", | ||
| "--skip-git-repo-check", | ||
| "--ignore-user-config", | ||
| "--ephemeral", | ||
| "-c", | ||
| ), |
There was a problem hiding this comment.
Probe every Codex option used at dispatch
When a Codex CLI or wrapper exposes the options checked here but lacks --add-dir or --dangerously-bypass-approvals-and-sandbox, this probe reports it ready even though codex_invoke later supplies those options. This affects concrete fleet paths: the planner supplies both options, while the reviewer and senior developer supply --add-dir, so they fail only after dispatch instead of being rejected by the readiness gate. Include every dispatched option in the exec --help markers, including --model if configured models are supported.
Useful? React with 👍 / 👎.
What changed
Verification