Reject live mode for table workflow tools - #7301
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile SummaryThis PR rejects legacy
Confidence Score: 5/5The PR appears safe to merge, with the intended legacy live-mode requests rejected before any workflow resolution or table mutation. The new condition matches the documented legacy payload, returns an actionable failure, and tests both affected workflow-group operations while confirming mutation is not invoked.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/copilot/tools/server/table/user-table.ts | Adds a narrowly scoped pre-mutation guard for legacy live-mode workflow-group requests; no actionable defect identified. |
| apps/sim/lib/copilot/tools/server/table/user-table.test.ts | Replaces silent-stripping coverage with explicit rejection assertions for both affected operations and verifies no resolution or creation occurs. |
Reviews (1): Last reviewed commit: "fix(copilot): reject live table workflow..." | Re-trigger Greptile
|
Closing per rollout decision: Mothership will deploy alongside Sim, so we do not need special handling for stale deploymentMode calls. |
Summary
This closes the rolling-version case where an older Mothership could request draft semantics and Sim would silently report success while actually using deployed state.
Companion Mothership PR: https://github.com/simstudioai/mothership/pull/466
Tests