Skip to content

feat(preview): bridge breadcrumb navigation - #1008

Merged
jeonghun-jj-lee merged 1 commit into
mainfrom
amico/pr-949-host-bridge-rebase
Sep 11, 2026
Merged

feat(preview): bridge breadcrumb navigation#1008
jeonghun-jj-lee merged 1 commit into
mainfrom
amico/pr-949-host-bridge-rebase

Conversation

@jeonghun-jj-lee

@jeonghun-jj-lee jeonghun-jj-lee commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Ports the extension-host half of #949 onto current main without restoring its superseded Preview overlay.

  • Relays opaque Explorer icon-theme assets and validates Preview visible-children requests.
  • Preserves current Preview overlay provenance and current Harmoniqs-provider relay paths.
  • Adds focused coverage for both chat render relays and the workspace-root boundary.

Supersedes #949, whose app-overlay implementation conflicts with the newer Preview implementation already merged through #978.

Validation

  • pnpm --filter amicode test -- chat_bridge.test.ts chat_panel.test.ts sidebar_view.test.ts preview_visible_children.test.ts
  • pnpm --filter amicode run typecheck
  • git diff --check

@coderabbitai

coderabbitai Bot commented Sep 11, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: c17ecf73-7959-452d-8f4d-a18912412ec9


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jeonghun-jj-lee
jeonghun-jj-lee merged commit 76458f5 into main Sep 11, 2026
10 checks passed
@jeonghun-jj-lee
jeonghun-jj-lee deleted the amico/pr-949-host-bridge-rebase branch September 11, 2026 04:14
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