Skip to content

feat(forking): excluded workflows#5727

Merged
icecrasher321 merged 2 commits into
stagingfrom
staging-v5
Jul 17, 2026
Merged

feat(forking): excluded workflows#5727
icecrasher321 merged 2 commits into
stagingfrom
staging-v5

Conversation

@icecrasher321

Copy link
Copy Markdown
Collaborator

Summary

Allow excluding workflows from push/pull -- equivalent to gitignores.

Type of Change

  • New feature

Testing

Tested manually

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

vercel Bot commented Jul 17, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Jul 17, 2026 3:17am

Request Review

@cursor

cursor Bot commented Jul 17, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Changes force-push/pull and fork-copy semantics for deployed workflows; mistakes could leave environments out of sync, but access is admin-only and behavior is covered by promote-plan and API tests.

Overview
Adds a per-workspace forkSyncExcluded flag so admins can treat deployed workflows like a sync .gitignore: they are not copied on new forks, not pushed/pulled as sources, and never overwritten or archived on the target when excluded locally.

Data & API: Migration 0263 adds workflow.fork_sync_excluded. Workflow list/update contracts and queries expose the flag; toggling requires workspace admin (batch PUT …/fork/excluded-workflows with audit/analytics, or single-workflow PUT alongside lock-style policy that bypasses mutability on locked workflows).

Fork engine: listDeployedWorkflows omits excluded workflows; promote planning skips replace/archive for excluded targets while keeping identity mappings for re-inclusion. Fork diff returns excludedSourceWorkflows / excludedTargetWorkflows for the sync UI.

UI & docs: New Excluded workflows section on the Forks settings page (folder tree, immediate toggles with optimistic cache). Sync preview lists excluded names greyed with tooltips. Enterprise forks docs and resource tables updated.

Reviewed by Cursor Bugbot for commit 3b4c689. Configure here.

Comment thread apps/sim/ee/workspace-forking/lib/promote/promote-plan.ts
@greptile-apps

greptile-apps Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds workspace-specific workflow exclusions for fork creation and synchronization. The main changes are:

  • Admin controls and APIs for excluding deployed workflows.
  • Sync planning that skips excluded sources and protects excluded targets.
  • Preview details for source-side and target-side exclusions.
  • Database, audit, analytics, documentation, and test updates.

Confidence Score: 5/5

This looks safe to merge.

  • The diff response now includes excluded source and target workflows.
  • The shared contract preserves both fields for the client.
  • Protected targets are omitted from replacement and archival operations.
  • No blocking issues were found in the updated code.

Important Files Changed

Filename Overview
apps/sim/app/api/workspaces/[id]/fork/diff/route.ts Adds source and target exclusion names to the fork diff response.
apps/sim/lib/api/contracts/workspace-fork.ts Adds backward-compatible exclusion arrays to the shared diff contract.
apps/sim/ee/workspace-forking/lib/promote/promote-plan.ts Prevents excluded targets from being replaced or archived and reports them in the plan.
apps/sim/ee/workspace-forking/components/fork-sync/fork-sync-view.tsx Shows excluded workflows in push and pull previews.

Reviews (2): Last reviewed commit: "improve sync preview" | Re-trigger Greptile

Comment thread apps/sim/ee/workspace-forking/lib/promote/promote-plan.ts
@icecrasher321

Copy link
Copy Markdown
Collaborator Author

bugbot run

@icecrasher321

Copy link
Copy Markdown
Collaborator Author

@greptile

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ 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 3b4c689. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant