Skip to content

feat: expose typed handoff reasoning effort - #138

Open
Will-Liang329 wants to merge 2 commits into
rebel0789:mainfrom
Will-Liang329:codex/typed-handoff-reasoning-effort
Open

Will-Liang329 wants to merge 2 commits into
rebel0789:mainfrom
Will-Liang329:codex/typed-handoff-reasoning-effort

Conversation

@Will-Liang329

Copy link
Copy Markdown
Contributor

Adds optional reasoning_effort to the typed handoff_to_agent MCP input and serializes it as Reasoning effort: <value> in the existing deterministic handoff preamble.

The value is opaque metadata: CodexPro does not apply provider-specific effort validation or execute a provider. When omitted, the generated handoff retains its existing format and behavior. This supports downstream runner/router contracts without introducing an AHR dependency.

Validation: npm run build and the complete npm run smoke suite. The smoke test checks tools/list exposes the optional string field, verifies the serialized preamble, and verifies omission compatibility.

@rebel0789 rebel0789 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The implementation and complete local smoke suite pass, but one public contract is currently inaccurate: the reasoning_effort schema says the local executor validates provider support, while this PR only serializes opaque metadata and execute-handoff does not parse, apply, or validate it. Please either change that description to say it is metadata that CodexPro does not validate/apply, matching the PR body, or implement the claimed executor behavior. After #141 lands, please also update the branch so the patched dependency audit and Windows matrix can run cleanly.

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants