Skip to content

TUI applies events from other directories when several TUIs share one server #39181

Closed as not planned

Description

@chriswritescode-dev

Description

I run one opencode serve and attach several TUIs to it, each in a different directory. Two things go wrong.

The branch in the bottom right of the sidebar sometimes shows a branch belonging to a different project. The directory stays correct, only the branch is wrong, and it changes on its own when I switch branches in one of the other checkouts. The second one is that a session switch driven from the server navigates the wrong TUI, or more than one at a time.

Both come from the same place. The TUI subscribes to /global/event. That route is registered on RootHttpApi without the directory middleware, so the ?directory= the SDK sends is ignored and every instance's events reach every connected client. On the client side the handlers only compare workspace, and that is undefined on both the event and the store unless experimental workspaces are enabled, so undefined === undefined passes and each TUI applies the other TUIs' events. packages/tui/src/context/event.ts receives directory in the metadata but never filters on it.

Where it shows up:

  • vcs.branch.updated in packages/tui/src/context/sync.tsx writes any directory's branch into every TUI's store. The sidebar guard at packages/tui/src/feature-plugins/sidebar/footer.tsx:23 compares directories, not branches, so it renders my own directory next to another project's branch. packages/tui/src/context/directory.ts:14 has no guard at all.
  • tui.session.select in packages/tui/src/app.tsx navigates every attached TUI instead of the one it was aimed at. Same for tui.toast.show, tui.command.execute and tui.prompt.append.

packages/app already partitions this same stream by directory in src/context/server-sync.tsx, the TUI does not.

Worth flagging that #31469 is the same line with the opposite symptom. Its analysis says the event is "already scoped to the correct project by event.ts:19", which is not the case, event.ts only forwards the metadata. Removing the workspace guard as suggested there would make this leak worse unless directory scoping lands first.

I have a fix and will open a PR against this.

Plugins

opencode-forge. It triggers the session switch symptom because it is the only thing calling tui.selectSession programmatically, but the fan-out itself is not plugin specific.

OpenCode version

1.18.5

Steps to reproduce

  1. start a server with opencode serve
  2. attach a TUI in repo A, on branch main
  3. attach a second TUI in repo B
  4. in repo B run git checkout -b feature
  5. repo A's sidebar now reads ~/repo-a:feature

Screenshot and/or share link

Not included.

Operating System

macOS 26.5.2

Terminal

Ghostty

Activity

  1. github-actions commented on Jul 27, 2026

    @github-actions
    Contributor

    This issue might be a duplicate of existing issues. Please check:

    Your issue adds a concrete multi-TUI reproduction path and specifically identifies the ?directory= parameter being ignored on /global/event, which complements the analysis in those issues.

  2. chriswritescode-dev commented on Jul 27, 2026

    @chriswritescode-dev
    Author

    Same root cause family, but I do not think this is a duplicate. Two differences: the transport and the event types.

    Those three are all about the v2 /api/event stream in packages/server, and all three are labelled 2.0. #37792 points at packages/tui/src/context/client.tsx, which does not exist on dev yet, it arrives with #37486/#37487.

    What I hit is the v1 /global/event route in packages/opencode/src/server/routes/instance/httpapi/handlers/global.ts, which is what the shipping 1.18.x TUI actually subscribes to at packages/tui/src/context/sdk.tsx:91. On dev the v2 DataProvider is fed off that same stream through useEvent at packages/tui/src/context/data.tsx:75, so packages/tui/src/context/event.ts is the single seam today.

    The other difference is the axis. #36443 and #37792 scope by session interest. Neither of my symptoms carries a session:

    • vcs.branch.updated has no session at all
    • tui.session.select is deliberately addressed at a TUI, not at a session

    So a session interest gate does not filter either one. The branch would still be wrong and the wrong TUI would still get navigated. #36443 does list location scoping in its acceptance criteria, which would cover this, but on the v2 transport whenever that lands.

    Last one is that these are correctness bugs on a released build rather than fan-out cost. Wrong branch rendered in the sidebar, wrong TUI navigated on a session switch.

    Happy to close this and fold it under #36441 if you would rather keep it all in one place. Otherwise I have a fix for the v1 path that scopes vcs.branch.updated, tui.session.select, tui.toast.show, tui.command.execute and tui.prompt.append by directory and leaves session and message events unfiltered, since a TUI can legitimately hold sessions from sibling directories of the same project.

  3. BB-84C commented on Aug 11, 2026

    @BB-84C
    SponsorContributor

    Follow-up on the promised fix (comment of 2026-07-27): as of today no PR references this issue. A fix is now live in PR #41842: client-side guards in packages/tui/src/context/sync.tsx that compare the event envelope's directory against the TUI's own instance directory for vcs.branch.updated and session.updated, plus tests.

    Additional reproduction detail confirmed independently: switching the branch externally (GitHub Desktop, plain git in a shell) triggers the same cross-workspace statusline contamination without any TUI involvement — the fs watcher publishes through the same /global/event stream.

    The remaining items documented here (tui.session.select / tui.toast.show / tui.command.execute / tui.prompt.append fan-out) are not covered by #41842 and stay open.

  4. BB-84C commented on Aug 11, 2026

    @BB-84C
    SponsorContributor

    We have submitted PR #41842 implementing the client-side directory guards for this issue: packages/tui/src/context/sync.tsx now compares the event envelope's directory against the TUI's own instance directory for both vcs.branch.updated and session.updated, with tests covering the foreign-directory rejection cases. The PR body references this issue (Closes #39181).

  5. github-actions commented on Oct 11, 2026

    @github-actions
    Contributor

    To stay organized issues are automatically closed after 60 days of no activity. If the issue is still relevant please open a new one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions