[waleedlatif1]

Summary

• Flag run_block and run_from_block as requiresApproval in the generated tool catalog, so all four workflow run tools draw the approval card
• The gate (toolCallNeedsApproval) is a pure catalog lookup; without the flag an agent steered into run_from_block ran everything downstream of its start block, or ran a Function block it had just added with edit_workflow, with no approval decision
• Applied the exact two-line generator delta rather than a full mship-tools:generate, which would pull in unrelated in-flight catalog drift
• New invariant test: every tool in WORKFLOW_TOOL_NAMES carries requiresApproval, so a re-sync that drops the flag fails CI
• Source of truth fixed in simstudioai/mothership#899

Type of Change

• Bug fix

Testing

• New test fails on the pre-fix catalog (lists exactly run_block, run_from_block), passes after
• permission.test.ts, router.test.ts, run-tool-execution.test.ts, copilot/confirm/route.test.ts, async-runs/repository.test.ts, run-workflow-result-budget.test.ts, request/handlers, tool-executor, copilot/tools suites: 269 + 38 passing
• Existing "every gated tool renders an answerable row" invariant still passes (neither tool is hidden or internal)
• bun run lint clean; type-check, check:audits, and the full suite left to CI

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing (new tests pass the test-audit authoring gate)
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Companion: simstudioai/mothership#899

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Nov 2, 2026 8:16pm UTC

[cubic-dev-ai]

No issues found across 2 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Turn on auto-fix | Re-trigger cubic}

[github-actions]

⚠️ Cross-repo companion check

One or more companion PRs aren't merged into staging yet. Merging this without them will leave copilot and sim out of sync — merge them in lockstep.

• ❌ simstudioai/mothership#899 — OPEN, not merged (targets staging) — fix(catalog): require approval for run_block and run_from_block

[greptile-apps]

Confidence Score: 5/5

[Medium impact] This PR appears safe to merge.

Summary

Adds approval prompts for run_block and run_from_block, matching the other workflow run tools.

• Block-based workflow runs now wait for approval.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Workflow tool call] --> B{Approval needed?}
  B -->|No| E[Start execution]
  B -->|Yes| C[Show approval card]
  C --> D{User decision}
  D -->|Allow| E
  D -->|Skip or timeout| F[Finish without running]

Loading

_{Reviews (1) · Last reviewed commit: "fix(copilot): require approval for run_b..." · Reviewed by Greptile}