Skip to content

feat(app): add session search and rename functionality - #6973

Closed
iso2kx wants to merge 1 commit into
anomalyco:devfrom
iso2kx:feat/session-rename-and-search
Closed

feat(app): add session search and rename functionality#6973
iso2kx wants to merge 1 commit into
anomalyco:devfrom
iso2kx:feat/session-rename-and-search

Conversation

@iso2kx

Copy link
Copy Markdown
Contributor

Summary

  • Add searchable session list dialog with date grouping and inline actions
  • Add session rename dialog
  • Add keyboard shortcuts: Cmd+O for session list, Cmd+Shift+E for rename
  • Fix corrupted custom-elements.d.ts files in app and enterprise packages

Changes

New Components

  • DialogSessionList - Searchable session list with:
    • Server-side search via sdk.client.session.list({ search })
    • Date grouping (Today, Yesterday, older dates)
    • Inline rename and delete buttons with click-to-confirm delete
    • Current session highlighting
  • DialogSessionRename - Simple rename dialog using sdk.client.session.update()

Modified

  • session-header.tsx - Replaced <Select> dropdown with <Button> that opens session list dialog, added rename button
  • session.tsx - Added command registrations for session.list and session.rename

Fixed

  • Restored corrupted custom-elements.d.ts files that contained file paths instead of actual TypeScript declarations

- Add DialogSessionList component with search, date grouping, and inline actions
- Add DialogSessionRename component for renaming sessions
- Replace session dropdown with button that opens searchable session list dialog
- Add keyboard shortcuts: Cmd+O for session list, Cmd+Shift+E for rename
- Fix corrupted custom-elements.d.ts file
@github-actions

Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

Duplicate PR Search Results

Good news! I found 1 potentially related PR that may be addressing similar session management features:

Related PR:

All other search results returned only the PR you're checking (#6973 - the current PR being analyzed), indicating no other directly duplicate PRs exist.

Recommendation:

Check PR #6769 to ensure the new session search/list functionality doesn't conflict with archived session viewing features and that both PRs can coexist harmoniously.

@github-actions

Copy link
Copy Markdown
Contributor

Closing this pull request because it has had no updates for more than 60 days. If you plan to continue working on it, feel free to reopen or open a new PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@iso2kx