feat(workflow): deploy workflows from resource panel - #6338
feat(workflow): deploy workflows from resource panel#6338BillLeoutsakosvl346 wants to merge 11 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview The shared
Reviewed by Cursor Bugbot for commit 1b58be2. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR adds a compact Deploy action to workflow resource tabs while reusing the editor’s deployment controller and modal. The latest changes also address both previously reported findings:
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains; the lock-metadata loading guard and absolute test import address the previously reported issues.
|
| Filename | Overview |
|---|---|
| apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-content/resource-content.tsx | Adds the compact Deploy action and disables it while lock metadata is loading or the workflow is effectively locked. |
| apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/deploy/deploy.tsx | Adds compact icon rendering, accessible labeling, external disabled-state support, and workflow-specific hydration checks while retaining the existing deployment flow. |
| apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/deploy/deploy.test.tsx | Covers compact labels, disabled states, and modal opening using the repository’s configured absolute import alias. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
A[Workflow resource tab] --> B[Load workflow and folder maps]
B --> C{Loading or effectively locked?}
C -->|Yes| D[Disable compact Deploy]
C -->|No| E[Run deployment readiness checks]
E --> F[Deployment mutation]
F --> G[Existing DeployModal]
Reviews (4): Last reviewed commit: "Merge remote-tracking branch 'origin/sta..." | Re-trigger Greptile
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 78171b7. Configure here.
…ow-resource-panel

Summary
Adds a compact Deploy action to workflow resource tabs on the Home resource panel. It reuses the editor deployment controller, readiness checks, permission handling, deployment mutation, and existing DeployModal.
No backend, API, schema, or migration changes are included.
Type of Change
Testing
cd apps/sim && bun run type-check: passed.cd apps/sim && bun run lint:check: passed with existing unrelated Zoho suppression warnings.bun run check:client-boundary: passed.bun run check:icon-paths: passed.Review focus: resource-panel action placement, compact accessibility labels/tooltips, lock/permission disabled state, and reuse of the existing deployment modal.
Checklist
Screenshots/Videos
Browser screenshots were not available because the execution environment exposed no browser session.