fix(tables): tolerate row deletion during run cancellation

[TheodoreSpeaks]

Summary

• ignore rows deleted during cancellation writes and tombstone insertion
• preserve fail-fast behavior for unrelated database failures
• add regression coverage for both deletion races

Type of Change

• Bug fix

Testing

• bun run test lib/table/workflow-columns.test.ts
• bun run type-check
• bunx biome lint apps/sim/lib/table/workflow-columns.ts apps/sim/lib/table/workflow-columns.test.ts
• bun run check:api-validation

Checklist

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

[vercel]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Aug 12, 2026 4:45am

[cursor]

PR Summary

Medium Risk
Touches cancellation write/error handling for in-flight table runs; incorrect swallowing could hide real DB failures, though the change is narrowly scoped and covered by tests.

Overview
Makes cancelWorkflowGroupRuns resilient when a row is deleted mid-cancellation.

Cancellation writes now swallow TableRowNotFoundError (including transaction-wrapped causes) so a deleted row no longer fails the whole cancel pass. Unrelated write errors still fail fast.

Tombstone inserts similarly ignore Postgres FK violations on the row FK (table_row_executions_row_id_user_table_rows_id_fk) when the row is already gone, while still rethrowing other constraint failures.

Adds regression coverage for both deletion races.

^{Reviewed by Cursor Bugbot for commit f008e38. Bugbot is set up for automated code reviews on this repo. Configure here.}

[greptile-apps]

Greptile Summary

The PR makes workflow-group cancellation tolerate rows deleted concurrently while preserving fail-fast behavior for unrelated database errors.

• Finds TableRowNotFoundError anywhere in the cancellation write’s wrapped cause chain.
• Ignores only the expected row foreign-key violation during tombstone insertion.
• Adds regression tests for both deletion races and unrelated-error propagation.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename	Overview
apps/sim/lib/table/workflow-columns.ts	Adds narrowly scoped handling for wrapped row-not-found errors and row-FK tombstone failures without suppressing unrelated write errors.
apps/sim/lib/table/workflow-columns.test.ts	Adds focused coverage for direct and transaction-wrapped deletion races plus fail-fast behavior for unrelated failures.

_{Reviews (2): Last reviewed commit: "fix(tables): unwrap row deletion errors" | Re-trigger Greptile}

[TheodoreSpeaks]

@greptile

[TheodoreSpeaks]

@cursor review

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit f008e38. Configure here.}