Skip to content

feat: enabling session id button for all agents - #59

Closed
MCBoarder289 wants to merge 1 commit into
kenn-io:mainfrom
MCBoarder289:session-id-ui-fix
Closed

feat: enabling session id button for all agents#59
MCBoarder289 wants to merge 1 commit into
kenn-io:mainfrom
MCBoarder289:session-id-ui-fix

Conversation

@MCBoarder289

Copy link
Copy Markdown
Contributor

Minor change that allows for copying session ids for all agents. Previously it was gated for claude and codex only. Non-prefixed session ids are gracefully handled already.

This is exactly the kind of tool I was looking for, so thank you for building this! I am using Copilot CLI, and was looking for this session id button since it was mentioned in release notes. Hopefully this is a straightforward enough change, but let me know if you have any additional feedback!

Minor change that allows for copying session ids for all agents.
Previously it was gated for claude and codex only.
Non-prefixed session ids are gracefully handled already.
@roborev-ci

roborev-ci Bot commented Feb 26, 2026

Copy link
Copy Markdown

roborev: Combined Review (8fc9e44)

Summary Verdict: All agents agree the code is clean and introduces no issues.

Commit 8fc9e44 successfully simplifies the visibility condition
for the session ID button in frontend/src/App.svelte by checking for the presence of session.id rather than specific agent types. No security vulnerabilities or functional defects were detected.


Synthesized from 4 reviews (agents: codex, gemini | types: default, security)

@wesm

wesm commented Feb 26, 2026

Copy link
Copy Markdown
Member

This was done already in #58, releasing soon

@wesm wesm closed this Feb 26, 2026
@MCBoarder289
MCBoarder289 deleted the session-id-ui-fix branch February 26, 2026 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants