GeezerVison is showing github.com — v0.7.36: hide preview-block-owned triggers from copilot VFS by TheodoreSpeaks · Pull Request #5677 · simstudioai/sim
Conversation
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
greptile-apps[bot] reacted with thumbs up emoji
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
| Project |
Deployment |
Actions |
Updated (UTC) |
| docs |
Skipped |
|
Jul 14, 2026 10:33pm |

|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
cursor
Bot
commented
Jul 14, 2026
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
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.
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
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
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
This branch was previously deployed
1 inactive
deployment
Uh oh!
There was an error while loading. Please reload this page.
Assignees
No one assigned
Milestone
No milestone
Development
Successfully merging this pull request may close these issues.
Uh oh!
There was an error while loading. Please reload this page.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.