feat(table): hide new workflow column feature#4414
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Copilot contract rollback: the generated Test stability: Reviewed by Cursor Bugbot for commit 7523570. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThis PR temporarily hides the workflow column feature to unblock staging: the column-type picker now filters out
Confidence Score: 3/5Not safe to merge without fixing the mapping schema mismatch that breaks null-skip for CSV import via copilot. One P1 defect (schema type contradicts the description it ships alongside) keeps the ceiling at 4/5, and it affects an active non-workflow code path (CSV import), pulling the score to 3/5. apps/sim/lib/copilot/generated/tool-catalog-v1.ts and apps/sim/lib/copilot/generated/tool-schemas-v1.ts — the mapping.additionalProperties.type must be restored to ['string', 'null'] or the description updated to remove the null instruction. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[User opens Column Sidebar] --> B{existingGroup?}
B -- Yes --> C[Show all type options including workflow]
B -- No --> D[Filter out workflow type option]
C --> E[User selects type]
D --> E
E --> F{type === workflow?}
F -- Yes --> G[Show workflow config UI]
F -- No --> H[Show scalar column UI]
I[Copilot: import_file with mapping] --> J{mapping value null?}
J -- Yes --> K[Schema validation fails - type string rejects null]
J -- No --> L[Maps CSV header to column]
Reviews (1): Last reviewed commit: "fix lint" | Re-trigger Greptile |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 1fbac96. Configure here.

Summary
Temporarily hide workflow column feature to unblock staging. Also reverted the copilot type contract so mothership doesn't try to create one.
Type of Change
Testing
Checklist
Screenshots/Videos