Skip to content

v0.7.36: hide preview-block-owned triggers from copilot VFS#5677

Merged
TheodoreSpeaks merged 1 commit into
mainfrom
staging
Jul 14, 2026
Merged

v0.7.36: hide preview-block-owned triggers from copilot VFS#5677
TheodoreSpeaks merged 1 commit into
mainfrom
staging

Conversation

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator

@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jul 14, 2026 10:33pm

Request Review

@cursor

cursor Bot commented Jul 14, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Copilot static discovery/visibility only; no runtime workflow execution or auth changes.

Overview
External trigger schema files in the copilot workspace VFS now follow the same preview/kill-switch rules as their owning blocks, mirroring how integration tool schemas are already gated.

At static build time the change records which block(s) expose each components/triggers/{provider}/{id}.json path (from triggers.available). When the VFS is stamped per viewer, those paths are skipped if every owner is hidden; if a trigger is shared by a GA block and a preview block, it stays visible until all owners are hidden.

The shared components/triggers/triggers.md overview no longer lists external triggers that would be hidden with no viewer context (e.g. owned only by unrevealed preview blocks), consistent with built-in preview trigger blocks already being omitted from that file.

Reviewed by Cursor Bugbot for commit 103ff7c. Bugbot is set up for automated code reviews on this repo. Configure here.

@greptile-apps

greptile-apps Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR hides preview-owned external triggers from the Copilot VFS. The main changes are:

  • Adds ownership tracking for external trigger schema paths.
  • Applies block visibility checks to external trigger files.
  • Filters preview-only external triggers from the generated trigger overview.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
apps/sim/lib/copilot/vfs/workspace-vfs.ts Adds trigger-path ownership tracking and uses it to hide preview-only external trigger schemas and overview entries.

Reviews (1): Last reviewed commit: "fix(mothership): hide preview-block-owne..." | Re-trigger Greptile

@TheodoreSpeaks
TheodoreSpeaks merged commit 50364ed into main Jul 14, 2026
38 checks passed
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