Repository navigation
Replies: 1 comment
|
Hi! I'd like to work on this issue as one of my first open-source contributions. I understand the proposed fix is to add an exception to Would you be open to a small, focused PR for this? I will keep the change limited to the intended behavior and verify it before submitting. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Before submitting
Area
packages/provider-core, apps/server
Steps to reproduce
AGENTS.md(or user prompt) mandates a repository-owned CLI review runner (for example, a read-only shell command that prepares an isolated review packet and runs a cross-provider reviewer CLI with--strict-mcp-configand structured verdict parsing), start a Codex turn in T3 Code with thet3-codeMCP server attached.t3_code_orchestration(T3_CODE_ORCHESTRATION_INSTRUCTIONS) injected inturn/start.additionalContext.Expected behavior
When the user or a documented repository workflow/skill explicitly names a dedicated CLI runner or script for a review or check, the agent should follow that workflow rather than refusing it on the ground that
T3_CODE_ORCHESTRATION_INSTRUCTIONSrequiresdelegate_task.T3_CODE_BROWSER_TOOL_INSTRUCTIONSin the same file (packages/provider-core/src/server/orchestrationInstructions.tsL49–54) already includes this exact carve-out for browser tools:Actual behavior
T3_CODE_ORCHESTRATION_INSTRUCTIONS(packages/provider-core/src/server/orchestrationInstructions.tsL9–11) currently states unconditionally:Also use delegate_task for cross-provider or explicitly T3-owned child tasks.For every T3 delegated review round, call delegate_task again.For Codex turns (
apps/server/src/orchestration-v2/Adapters/CodexAdapterV2.tsL766–775 →buildCodexAdditionalContextinapps/server/src/provider/CodexDeveloperInstructions.tsL146–164),t3_code_orchestrationis injected as{ kind: "application", value: T3_CODE_ORCHESTRATION_INSTRUCTIONS }, which Codex renders as a developer-level<t3_code_orchestration>message. Because developer instructions outrankAGENTS.mdand user messages in Codex's instruction hierarchy, Codex sessions refuse to run a repository's shell-based review runner even when bothAGENTS.mdand the user explicitly tell them to use the repository runner instead ofdelegate_task.Suggested fix: in
T3_CODE_ORCHESTRATION_INSTRUCTIONS(packages/provider-core/src/server/orchestrationInstructions.ts), add a short exception parallel toT3_CODE_BROWSER_TOOL_INSTRUCTIONS—for example, noting that when the user or a documented repository workflow/skill explicitly names a specific CLI runner or script for a review or check, the agent should follow that workflow instead ofdelegate_task.Impact
Blocks core workflow
Version or commit
0.0.46-nightly.20261014.3132; still present onmain@42da166d13Environment
macOS, Apple Silicon. T3 Code Desktop Nightly
0.0.46-nightly.20261014.3132. Provider: Codex (gpt-5.4/gpt-6.1-sol).All reactions