Skip to content

feat(execute_code): add native/poll/auto execution modes for long-run… - #28

Open
rgonzalo-orellana wants to merge 1 commit into
mainfrom
feat/execute-code-progress-and-poll
Open

feat(execute_code): add native/poll/auto execution modes for long-run…#28
rgonzalo-orellana wants to merge 1 commit into
mainfrom
feat/execute-code-progress-and-poll

Conversation

@rgonzalo-orellana

Copy link
Copy Markdown
Collaborator

…ning work

Introduce an ExecutionService orchestration layer so MCP tools stay thin, and add three execution modes for code that outlives a single request:

  • native: SEP-1686 MCP tasks with real-time progress notifications
  • poll: background JobStore with server-paced get_job_result polling
  • auto: picks native or poll per client capability (default)

Details:

  • New get_job_result tool for clients without task support.
  • Inject a progress(message) callback into the sandbox for user feedback.
  • Report executor_busy from get_status; configurable executor lock timeout.
  • Bundle burner_redis in the PyInstaller spec (needed by docket/native tasks).
  • Skill + manifest guidance on choosing a mode (quick->native, heavy->poll).

Tests: add tests/test_execution.py; repoint test_connection patch target.

…ning work

Introduce an ExecutionService orchestration layer so MCP tools stay thin,
and add three execution modes for code that outlives a single request:

- native: SEP-1686 MCP tasks with real-time progress notifications
- poll:   background JobStore with server-paced get_job_result polling
- auto:   picks native or poll per client capability (default)

Details:
- New get_job_result tool for clients without task support.
- Inject a progress(message) callback into the sandbox for user feedback.
- Report executor_busy from get_status; configurable executor lock timeout.
- Bundle burner_redis in the PyInstaller spec (needed by docket/native tasks).
- Skill + manifest guidance on choosing a mode (quick->native, heavy->poll).

Tests: add tests/test_execution.py; repoint test_connection patch target.
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