Skip to content

feat(agent): expose mode on AgentExecutionConfig - #165

Open
dineshreddy91 wants to merge 1 commit into
mainfrom
feat/agent-execution-config-mode
Open

feat(agent): expose mode on AgentExecutionConfig#165
dineshreddy91 wants to merge 1 commit into
mainfrom
feat/agent-execution-config-mode

Conversation

@dineshreddy91

@dineshreddy91 dineshreddy91 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add mode?: "agent" | "program" to AgentExecutionConfig / AgentExecutionConfigParams
  • Serialize it as mode in toJSON() (omitted when unset)
  • Unit tests cover forward + omit behavior on /agent/execute

This matches the Python SDK (vlmrun>=0.7.1, #205) and the /v1/agent/execute OpenAPI schema. Without it, cookbook notebook 18’s mode: "program" path cannot be expressed from the Node SDK, so grounded document.extract pipelines fall back to agent mode and lose *_metadata.bboxes.

Test plan

  • npx jest tests/unit/client/agent.test.ts (38 passed)
  • Publish npm release after merge

Made with Cursor


Open in Devin Review

Serialize Orion-2 program/agent execution mode on the wire so
client.agent.execute({ config: { mode: "program" } }) matches the
Python SDK and API schema.

Co-authored-by: Cursor <cursoragent@cursor.com>

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

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.

1 participant