Skip to content

Auto-review follow-ups for PR #233: feat(extension): dial configured remote bridges alongside loopback#234

Description

@github-actions

Tracking auto-review follow-ups for #233 (verdict: warn).

馃煛 Nits

  • server-hello.ts: a rejected hello keeps the mcpId鈫抣ink binding it created, so a refused id is held until the link drops and a hello flood grows mcpLink unboundedly
  • session-scope.ts: clearAllSessionScopes has no production caller after per-link teardown, and its doc comment still claims it is called on WS teardown
  • socket.ts/popup.ts: the connection badge stays green whenever any link is open, and bridge rows carry no connected bit, so a dead loopback link is invisible

When addressing the review on #233, resolve each item and add Closes #<this issue> to the PR once everything is genuinely fixed; deferred items stay open here. See the auto-review follow-up convention in CLAUDE.md.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions