Skip to content

feat(mcp): expose scheduled task run-now - #10562

Merged
juliusmarminge merged 1 commit into
agents/mcp-thin/thread-searchfrom
agents/mcp-thin/scheduled-tasks
Sep 7, 2026
Merged

feat(mcp): expose scheduled task run-now#10562
juliusmarminge merged 1 commit into
agents/mcp-thin/thread-searchfrom
agents/mcp-thin/scheduled-tasks

Conversation

@juliusmarminge

@juliusmarminge juliusmarminge commented Sep 7, 2026

Copy link
Copy Markdown
Member

Part 12/16 of the shared-core and MCP stack. Based on #10561. Next: #10563.

Expose ScheduledTaskService.runNow for tasks in the calling project, using the existing manual-run execution and bookkeeping.

Requires a full-access/default caller. Each call is a new manual run, not an idempotent retry. Returned bookkeeping is not provider turn completion; durable keyed retries and scheduler hardening are separate.

MCP-only rebuild of #8731, preserving attribution to Julius Marminge's original work. Original branches remain available for separate service follow-ups.

Validation: The composed stack passes 94 tests across 12 focused files, including shared core MCP and real MCP/V2 integration, attachment intake, project RPC/service contracts, and client model command selection. Server/contracts/shared/client-runtime typechecks and targeted format/lint/diff checks pass. New behavior coverage lives with the shared operation; no per-tool mock suite was added. Current-head CI is shown below.

Layer size: 3 files, +54/-0. No domain-service production implementation or documentation files change in this MCP layer.

Prepared with Codex in the OpenAI agent runtime.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M 30-99 changed lines (additions + deletions). labels Sep 7, 2026
@juliusmarminge juliusmarminge changed the title feat(mcp): expose manual scheduled task execution feat(mcp): expose scheduled task run-now Sep 7, 2026
@macroscopeapp

macroscopeapp Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR adds a new destructive MCP operation that can dispatch scheduled-task provider work and update durable run bookkeeping. Although access is bounded and existing scheduler logic is reused, the new user-facing production workflow has material side effects and warrants human review.

No code changes detected at 06de990. Prior analysis still applies.

You can add or adjust custom eligibility rules. Learn more.

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Thread transfer impact

⚠️ The latest CI run did not produce a thread transfer result for 06de990.

This comment will update automatically after the next completed run.

Comment thread apps/server/src/mcp/toolkits/thread/handlers.ts
@juliusmarminge
juliusmarminge force-pushed the agents/mcp-thin/scheduled-tasks branch from 97d1493 to c43f14e Compare September 7, 2026 19:44
@juliusmarminge
juliusmarminge force-pushed the agents/mcp-thin/scheduled-tasks branch from c43f14e to fae4158 Compare September 7, 2026 20:39
@juliusmarminge
juliusmarminge force-pushed the agents/mcp-thin/scheduled-tasks branch 2 times, most recently from 5345006 to 47b90f0 Compare September 7, 2026 20:48
@juliusmarminge
juliusmarminge force-pushed the agents/mcp-thin/scheduled-tasks branch from 47b90f0 to 06de990 Compare September 7, 2026 20:55
@juliusmarminge
juliusmarminge merged commit f7eb40c into t3code/codex-turn-mapping Sep 7, 2026
43 of 54 checks passed
@juliusmarminge
juliusmarminge deleted the agents/mcp-thin/scheduled-tasks branch September 7, 2026 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant