Repository navigation
Conversation
Finding the current session's ID means opening a dialog or the session list, yet it is what the CLI, the API and a bug report ask for. session.sidebar_id shows it below the title in the sidebar, in the muted color, where it can be selected and copied. Off by default, and listed in the settings dialog.
|
The following comment was made by an LLM, it may be inaccurate: |
![opencode-agent[bot]](https://avatars.githubusercontent.com/in/1549082?s=60&v=4){kind=small}
There was a problem hiding this comment.
This works as described, and I found no issues that should block it. I ran the TUI from source with the simulated model and session.sidebar_id: true (screenshots below). With this PR, the session ID appears in muted text under the title and fits inside the sidebar. With the same config on the base, no ID is shown. bun test test/config-v2.test.tsx and bun run typecheck in packages/tui both pass.
Public API: this adds one optional config key, session.sidebar_id (boolean, off by default), also exposed in the settings dialog and documented in the CLI config docs. It's backwards compatible because existing configs are unaffected.
|
Closing this for now: I'm limiting myself to at most 3 open PRs on this repo to keep the review load reasonable. I may reopen it later. Thanks for your time! |
Issue for this PR
Related: #53662
Type of change
What does this PR do?
The current session's ID is what the CLI, the API and bug reports ask for, but finding it means opening a dialog or the session list.
This adds
session.sidebar_id(off by default, also in the settings dialog). When on, the sidebar shows the ID under the session title in the muted color, where it can be selected and copied.How did you verify your code works?
bun run typecheckandbun test test/config-v2.test.tsxinpackages/tui. I also ran it in a real terminal: a TUI from this branch (serve --stdioplus--server) on a throwaway tmux server, scratch data dir, and a fake OpenAI-compatible provider on loopback. Withsession.sidebar_id: true, the sidebar showsses_...under the title. The same check fails ondev.Screenshots / recordings
Text captures are in the description above.
Checklist
This PR is AI generated, but under direct supervision and on request of @Nowaker.
Added by the opencode bot (AI-generated).
Related: #47746, #53662
Public API: adds the optional
session.sidebar_idboolean config key (default off; backwards compatible)