Skip to content

feat: allow renaming conversations from history#211

Open
TimSkoglund wants to merge 4 commits intoeneo-ai:developfrom
TimSkoglund:feat/chat-history-rename
Open

feat: allow renaming conversations from history#211
TimSkoglund wants to merge 4 commits intoeneo-ai:developfrom
TimSkoglund:feat/chat-history-rename

Conversation

@TimSkoglund
Copy link
Copy Markdown

Summary

Adds support for renaming conversations directly from the history view.

Backend

  • Adds PATCH /conversations/{session_id}/name/
  • Uses existing session_service.update_session
  • Introduces ConversationRenameRequest model

Frontend

  • Adds rename dialog in HistoryActions
  • Adds renameConversation() in ChatService
  • Adds rename endpoint in intric-js client

Fix

Replaces pushState with goto in chat page navigation.
Fixes router initialization error when opening conversations from history.

Design goals

  • Minimal changes to existing code
  • No behavioural changes outside rename feature
  • Reuses existing service layer
  • Keeps UI consistent with delete dialog

Design notes

  • Follows existing Dialog.Trigger pattern used by delete action
  • Avoids introducing new UI components
  • Keeps rename logic inside ChatService to match architecture

Testing

  • Rename from history works
  • Rename active conversation works
  • Delete after rename works
  • Page reload preserves renamed title
  • No regression in navigation or streaming

@TimSkoglund
Copy link
Copy Markdown
Author

Happy to adjust anything if needed 🙂

@MaxEriksson2000
Copy link
Copy Markdown
Collaborator

Tack för PR:n, rename-flödet ser bra ut överlag. Innan merge ser vi gärna några korrigeringar.

  1. i18n i UI
    I rename-dialogen är flera texter hårdkodade på svenska.
    Flytta dem gärna till paraglide (m.*) så vi behåller språkstöd (filer sv.json och en.json)?

    • frontend/apps/web/src/lib/features/chat/components/history/HistoryActions.svelte
  2. Synka API-kontrakt/typer
    Nya endpointen /api/v1/conversations/{session_id}/name/ används via @ts-expect-error.
    Kan du uppdatera/generera schema-typer så endpointen finns i schema.d.ts, och ta bort @ts-expect-error?

    • frontend/packages/intric-js/src/endpoints/conversations.js
    • frontend/packages/intric-js/src/types/schema.d.ts
  3. Server-side validering av namn
    ConversationRenameRequest validerar inte name (t.ex. trim/min/max).
    Kan du lägga till backend-validering så vi inte enbart förlitar oss på frontend?

    • backend/src/intric/conversations/conversation_models.py
    • backend/src/intric/conversations/conversations_router.py
image Se bild ovan: uppdatera gärna designen så det blir tydligare vart fältet är i ljust läge.

@TimSkoglund TimSkoglund force-pushed the feat/chat-history-rename branch from fa4913b to f853a07 Compare February 18, 2026 08:54
TimSkoglund and others added 4 commits February 23, 2026 12:11
* feat: add voice-to-text input button using Web Speech API

  Adds a microphone button to the chat input that uses the browser's
  SpeechRecognition API for real-time speech-to-text transcription.
  Supports Chromium and Safari, with a disabled indicator for                                           unsupported browsers.

* Revert "feat: add voice-to-text input button using Web Speech API"

This reverts commit 3fd3e30.
@TimSkoglund TimSkoglund force-pushed the feat/chat-history-rename branch from 72a772c to c8f96fd Compare February 23, 2026 12:14
Sign up for free to 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.

3 participants