GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — v0.7.36: hide preview-block-owned triggers from copilot VFS by TheodoreSpeaks · Pull Request #5677 · simstudioai/sim

Conversation

@TheodoreSpeaks

@TheodoreSpeaks TheodoreSpeaks commented Jul 14, 2026

Copy link
Copy Markdown
Contributor
  • fix(mothership): hide preview-block-owned triggers from copilot VFS (fix(mothership): hide preview-block-owned triggers from copilot VFS #5676)

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@TheodoreSpeaks
fix(mothership): hide preview-block-owned triggers from copilot VFS (#…
103ff7c
…5676)
@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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@cursor

cursor Bot commented Jul 14, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@TheodoreSpeaks
TheodoreSpeaks merged commit 50364ed into main Jul 14, 2026
38 checks passed

Uh oh!

There was an error while loading. Please reload this page.

This branch was previously deployed

1 inactive deployment
Preview — 103ff7cf Deployed Jul 14, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

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.

1 participant

@TheodoreSpeaks
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.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.