Skip to content

fix(app): avoid preview child MCP bootstraps#25244

Open
mattgenious wants to merge 8 commits intoanomalyco:devfrom
mattgenious:fix/sidebar-preview-mcp-bootstrap
Open

fix(app): avoid preview child MCP bootstraps#25244
mattgenious wants to merge 8 commits intoanomalyco:devfrom
mattgenious:fix/sidebar-preview-mcp-bootstrap

Conversation

@mattgenious
Copy link
Copy Markdown

@mattgenious mattgenious commented May 1, 2026

Issue for this PR

Closes #25243

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Preview child stores are created with bootstrap: false when hovering/sidebar-previewing projects, but they still registered status queries for path, MCP, LSP, and providers. Those queries can hit directory-scoped backend routes and bootstrap the preview instance anyway.

This PR keeps preview-only child stores from registering those status queries until the child is actually bootstrapped. Normal child store bootstrapping still enables the same queries, including when an existing preview child is later requested with bootstrap enabled.

How did you verify your code works?

  • bun test --preload ./happydom.ts ./src/context/global-sync/child-store.test.ts
  • bun run test:unit ./src/context/global-sync/child-store.test.ts
  • bun typecheck from packages/app
  • pre-push hook: bun turbo typecheck

Screenshots / recordings

N/A. This is a non-visual behavior fix.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@mattgenious mattgenious requested a review from adamdotdevin as a code owner May 1, 2026 05:18
@github-actions github-actions Bot added needs:compliance This means the issue will auto-close after 2 hours. and removed needs:compliance This means the issue will auto-close after 2 hours. labels May 1, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

Thanks for updating your PR! It now meets our contributing guidelines. 👍

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.

Sidebar preview child stores bootstrap MCP/LSP status queries despite bootstrap: false

1 participant