feat: add session rename functionality to TUI modal#1821
Merged
thdxr merged 6 commits intoanomalyco:devfrom Aug 12, 2025
Merged
feat: add session rename functionality to TUI modal#1821thdxr merged 6 commits intoanomalyco:devfrom
thdxr merged 6 commits intoanomalyco:devfrom
Conversation
Enable users to rename sessions directly from the session modal by pressing 'r' key. - Add PATCH /session/:id API endpoint for updating session title - Extend Go SDK with UpdateSession method and SessionUpdateParams type - Add UpdateSession method to TUI app for API integration - Implement rename mode in session modal with textinput component - Update modal help text to show 'r rename' option - Handle Enter/Esc for confirm/cancel rename operations 🤖 Generated with [opencode](https://opencode.ai) Co-Authored-By: opencode <noreply@opencode.ai>
Change the session modal title from "Switch Session" to "Rename Session" when user enters rename mode, providing clearer context for the current operation. 🤖 Generated with [opencode](https://opencode.ai) Co-Authored-By: opencode <noreply@opencode.ai>
…g modal When in session rename mode, pressing ESC now exits rename mode and returns to the session list view instead of closing the modal entirely. This provides a more intuitive user experience. - Add ReopenSessionModalMsg to handle rename mode ESC behavior - Modify sessionDialog.Close() to detect rename mode and reopen modal - Add TUI handler for ReopenSessionModalMsg to recreate session modal - Remove explicit ESC handling from rename mode (now handled by Close method) 🤖 Generated with [opencode](https://opencode.ai) Co-Authored-By: opencode <noreply@opencode.ai>
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Line 332-340: This violates the AGENTS.md guideline: "DO NOT do unnecessary destructuring of variables". The const updates = c.req.valid("json") could be used directly in the Session.update call. Also violates "AVOID else statements" - the if statement could be restructured to avoid conditional logic.
github-actions
bot
force-pushed
the
dev
branch
from
9aa0700 to
5141fe8
Compare
yokowasis
pushed a commit
to yokowasis/opencode-sst
that referenced
this pull request
Aug 14, 2025
Co-authored-by: opencode <noreply@opencode.ai> Co-authored-by: Dax Raad <d@ironbay.co> Co-authored-by: Dax <mail@thdxr.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add session rename functionality to the TUI session modal, allowing users to rename sessions by
pressing the 'r' key.