Feat/compare models mode

.github/workflows/docker-publish.yml

@@ -7,6 +7,11 @@ on:
         description: 'Tag to build and publish'
         required: false
         type: string
+      skip_ci:
+        description: 'Skip CI'
+        required: false
+        type: boolean
+        default: false
   workflow_dispatch:
     inputs:
       tag:
@@ -20,7 +25,7 @@ env:
 
 jobs:
   ci:
-    if: github.event_name != 'workflow_call'
+    if: github.event_name != 'workflow_call' && inputs.skip_ci != true
     uses: ./.github/workflows/ci.yml
     permissions:
       contents: read

.github/workflows/electron-release.yml

@@ -7,6 +7,11 @@ on:
         description: 'Tag to build and publish'
         required: false
         type: string
+      skip_ci:
+        description: 'Skip CI'
+        required: false
+        type: boolean
+        default: false
   workflow_dispatch:
     inputs:
       tag:
@@ -19,7 +24,7 @@ permissions:
 
 jobs:
   ci:
-    if: github.event_name != 'workflow_call'
+    if: github.event_name != 'workflow_call' && inputs.skip_ci != true
     uses: ./.github/workflows/ci.yml
 
   build:

.github/workflows/release.yml

@@ -22,6 +22,7 @@ jobs:
     uses: ./.github/workflows/docker-publish.yml
     with:
       tag: ${{ inputs.tag }}
+      skip_ci: true
     secrets: inherit
     permissions:
       contents: read
@@ -32,6 +33,7 @@ jobs:
     uses: ./.github/workflows/electron-release.yml
     with:
       tag: ${{ inputs.tag }}
+      skip_ci: true
     secrets: inherit
     permissions:
       contents: write

.husky/pre-commit

@@ -0,0 +1 @@
+npx lint-staged

backend/Dockerfile

@@ -28,6 +28,7 @@ COPY --chown=node:node package*.json ./
 RUN apt-get update && apt-get install -y --no-install-recommends \
     gosu \
     libsqlite3-0 \
+    sqlite3 \
     ca-certificates \
     && rm -rf /var/lib/apt/lists/*
 

backend/__tests__/conversations.test.js

@@ -170,6 +170,23 @@ describe('GET /v1/conversations', () => {
   assert.equal(items[1].id, 'c2');
   assert.equal(items[1].provider_id, 'p2');
   });
+
+  test('excludes linked comparison conversations from list results', async () => {
+    createConversation({ id: 'parent', sessionId, userId: testUser.id, title: 'parent' });
+    createConversation({
+      id: 'linked',
+      sessionId,
+      userId: testUser.id,
+      title: 'linked',
+      parentConversationId: 'parent',
+    });
+    createConversation({ id: 'standalone', sessionId, userId: testUser.id, title: 'standalone' });
+
+    const app = makeApp();
+    const res = await request(app).get('/v1/conversations').set('x-session-id', sessionId);
+    const ids = res.body.items.map((item) => item.id).sort();
+    assert.deepEqual(ids, ['parent', 'standalone']);
+  });
 });
 
 // --- GET /v1/conversations/:id ---
@@ -235,6 +252,76 @@ describe('GET /v1/conversations/:id', () => {
   });
 });
 
+// --- GET /v1/conversations/:id/linked ---
+describe('GET /v1/conversations/:id/linked', () => {
+  test('returns linked comparison conversations for the parent', async () => {
+    createConversation({ id: 'parent', sessionId, userId: testUser.id, title: 'parent' });
+    createConversation({
+      id: 'child-1',
+      sessionId,
+      userId: testUser.id,
+      title: 'child 1',
+      parentConversationId: 'parent',
+    });
+    createConversation({
+      id: 'child-2',
+      sessionId,
+      userId: testUser.id,
+      title: 'child 2',
+      parentConversationId: 'parent',
+    });
+
+    const app = makeApp();
+    const res = await request(app)
+      .get('/v1/conversations/parent/linked')
+      .set('x-session-id', sessionId);
+
+    assert.equal(res.status, 200);
+    const ids = res.body.conversations.map((item) => item.id).sort();
+    assert.deepEqual(ids, ['child-1', 'child-2']);
+  });
+
+  test('returns 404 when parent conversation is not owned by the user', async () => {
+    const otherUser = createUser({
+      email: 'other@example.com',
+      passwordHash: 'pw',
+      displayName: 'Other User',
+    });
+    createConversation({ id: 'other-parent', sessionId, userId: otherUser.id, title: 'other' });
+
+    const app = makeApp();
+    const res = await request(app)
+      .get('/v1/conversations/other-parent/linked')
+      .set('x-session-id', sessionId);
+
+    assert.equal(res.status, 404);
+  });
+});
+
+// --- DELETE /v1/conversations/:id ---
+describe('DELETE /v1/conversations/:id', () => {
+  test('soft deletes linked comparison conversations with the parent', async () => {
+    createConversation({ id: 'parent', sessionId, userId: testUser.id, title: 'parent' });
+    createConversation({
+      id: 'linked',
+      sessionId,
+      userId: testUser.id,
+      title: 'linked',
+      parentConversationId: 'parent',
+    });
+
+    const app = makeApp();
+    const res = await request(app).delete('/v1/conversations/parent').set('x-session-id', sessionId);
+    assert.equal(res.status, 204);
+
+    const db = getDb();
+    const parent = db.prepare('SELECT deleted_at FROM conversations WHERE id = ?').get('parent');
+    const linked = db.prepare('SELECT deleted_at FROM conversations WHERE id = ?').get('linked');
+    assert.ok(parent.deleted_at);
+    assert.ok(linked.deleted_at);
+  });
+});
+
 // --- DELETE /v1/conversations/:id ---
 describe('DELETE /v1/conversations/:id', () => {
   test('soft deletes an existing conversation and returns 204', async () => {

backend/package.json

@@ -11,6 +11,7 @@
     "start": "NODE_ENV=production node src/index.js",
     "test": "NODE_OPTIONS=--experimental-vm-modules jest",
     "lint": "eslint .",
+    "lint:fix": "eslint . --fix",
     "format": "prettier --write .",
     "migrate": "node scripts/migrate.js",
     "backfill:message-events": "node scripts/backfill-message-events.js"

backend/src/db/conversations.js

@@ -15,12 +15,13 @@ export function createConversation({
   reasoningEffort = null,
   verbosity = null,
   metadata = {},
+  parentConversationId = null,
 }) {
   const db = getDb();
   const now = new Date().toISOString();
   db.prepare(
-    `INSERT INTO conversations (id, session_id, user_id, title, provider_id, model, metadata, streaming_enabled, tools_enabled, quality_level, reasoning_effort, verbosity, created_at, updated_at)
-     VALUES (@id, @session_id, @user_id, @title, @provider_id, @model, @metadata, @streaming_enabled, @tools_enabled, @quality_level, @reasoning_effort, @verbosity, @now, @now)`
+    `INSERT INTO conversations (id, session_id, user_id, title, provider_id, model, metadata, streaming_enabled, tools_enabled, quality_level, reasoning_effort, verbosity, parent_conversation_id, created_at, updated_at)
+     VALUES (@id, @session_id, @user_id, @title, @provider_id, @model, @metadata, @streaming_enabled, @tools_enabled, @quality_level, @reasoning_effort, @verbosity, @parent_conversation_id, @now, @now)`
   ).run({
     id,
     session_id: sessionId,
@@ -34,6 +35,7 @@ export function createConversation({
     quality_level: qualityLevel,
     reasoning_effort: reasoningEffort,
     verbosity,
+    parent_conversation_id: parentConversationId || null,
     now,
   });
 }
@@ -44,7 +46,7 @@ export function getConversationById({ id, userId }) {
   }
 
   const db = getDb();
-  const query = `SELECT id, title, provider_id, model, metadata, streaming_enabled, tools_enabled, quality_level, reasoning_effort, verbosity, created_at FROM conversations
+  const query = `SELECT id, title, provider_id, model, metadata, streaming_enabled, tools_enabled, quality_level, reasoning_effort, verbosity, parent_conversation_id, created_at FROM conversations
            WHERE id=@id AND user_id=@user_id AND deleted_at IS NULL`;
   const result = db.prepare(query).get({ id, user_id: userId });
 
@@ -190,8 +192,9 @@ export function listConversations({ userId, cursor, limit }) {
   const safeLimit = clampLimit(limit, { fallback: 20, min: 1, max: 100 });
   const { cursorCreatedAt, cursorId } = parseCreatedAtCursor(cursor);
 
+  // Exclude linked/comparison conversations (those with parent_conversation_id)
   let sql = `SELECT id, title, provider_id, model, created_at FROM conversations
-         WHERE user_id=@userId AND deleted_at IS NULL`;
+         WHERE user_id=@userId AND deleted_at IS NULL AND parent_conversation_id IS NULL`;
   const params = { userId, cursorCreatedAt, cursorId, limit: safeLimit + 1 };
 
   sql = appendCreatedAtCursor(sql, { cursorCreatedAt, cursorId });
@@ -204,13 +207,40 @@ export function listConversations({ userId, cursor, limit }) {
   return { items, next_cursor };
 }
 
+/**
+ * Get linked/comparison conversations for a parent conversation
+ * @param {string} parentId - Parent conversation ID
+ * @param {string} userId - User ID
+ * @returns {Array} Array of linked conversation metadata
+ */
+export function getLinkedConversations({ parentId, userId }) {
+  if (!userId) {
+    throw new Error('userId is required');
+  }
+
+  const db = getDb();
+  const query = `SELECT id, title, provider_id, model, created_at, updated_at FROM conversations
+         WHERE parent_conversation_id=@parentId AND user_id=@userId AND deleted_at IS NULL
+         ORDER BY datetime(created_at) ASC`;
+
+  return db.prepare(query).all({ parentId, userId });
+}
+
 export function softDeleteConversation({ id, userId }) {
   if (!userId) {
     throw new Error('userId is required');
   }
 
   const db = getDb();
   const now = new Date().toISOString();
+
+  // Also soft-delete any linked comparison conversations
+  db.prepare(
+    `UPDATE conversations SET deleted_at=@now, updated_at=@now
+     WHERE parent_conversation_id=@id AND user_id=@userId AND deleted_at IS NULL`
+  ).run({ id, userId, now });
+
+  // Delete the parent conversation
   const query = `UPDATE conversations SET deleted_at=@now, updated_at=@now WHERE id=@id AND user_id=@userId AND deleted_at IS NULL`;
   const info = db.prepare(query).run({ id, userId, now });
   return info.changes > 0;

backend/src/db/migrations/023-parent-conversation-id.js

@@ -0,0 +1,22 @@
+/**
+ * Migration: Add parent_conversation_id column for linked comparison conversations
+ *
+ * This enables storing comparison/secondary model responses as separate conversations
+ * that are linked to a primary conversation. Conversations with a non-null parent_conversation_id
+ * are excluded from the main conversation list.
+ */
+
+export default {
+  version: 23,
+  up: `
+    -- Add parent_conversation_id column for linked comparison conversations
+    ALTER TABLE conversations ADD COLUMN parent_conversation_id TEXT DEFAULT NULL;
+
+    -- Create index for efficient lookups of child conversations
+    CREATE INDEX IF NOT EXISTS idx_conversations_parent_id ON conversations(parent_conversation_id);
+  `,
+  down: `
+    -- Note: SQLite doesn't support DROP COLUMN, so the column will remain but be unused
+    DROP INDEX IF EXISTS idx_conversations_parent_id;
+  `
+};

backend/src/lib/openaiProxy.js

@@ -337,6 +337,24 @@ function handleProxyError(error, req, res, persistence) {
 async function handleRequest(context, req, res) {
   const { body, bodyIn, flags, provider, providerId, persistence, userId, abortContext } = context;
 
+  if (bodyIn?.parent_conversation_id && Array.isArray(bodyIn.messages)) {
+    const summarized = bodyIn.messages.map((msg) => ({
+      role: msg?.role,
+      id: msg?.id,
+      contentLen:
+        typeof msg?.content === 'string'
+          ? msg.content.length
+          : Array.isArray(msg?.content)
+            ? msg.content.length
+            : 0,
+    }));
+    logger.debug('[openaiProxy] comparison request message history', {
+      parentConversationId: bodyIn.parent_conversation_id,
+      count: summarized.length,
+      summary: summarized,
+    });
+  }
+
   if (flags.hasTools) {
     // Tool orchestration path
     if (flags.streamToFrontend) {

backend/src/lib/persistence/ConversationManager.js

@@ -81,7 +81,8 @@ export class ConversationManager {
       qualityLevel: params.qualityLevel || null,
       reasoningEffort: params.reasoningEffort || null,
       verbosity: params.verbosity || null,
-      metadata: params.metadata || {}
+      metadata: params.metadata || {},
+      parentConversationId: params.parentConversationId || null,
     });
 
     return conversationId;

backend/src/lib/retryUtils.js

@@ -20,7 +20,7 @@ const DEFAULT_RETRY_CONFIG = {
   maxDelayMs: 60000,
   backoffMultiplier: 2,
   jitterFactor: 0.1,
-  shouldRetry: (error, attempt) => {
+  shouldRetry: (error, _attempt) => {
     // Only retry on 429 (rate limit) and 5xx server errors
     if (error?.status === 429) return true;
     if (error?.status >= 500 && error?.status < 600) return true;
@@ -99,7 +99,7 @@ export async function retryWithBackoff(fn, userConfig = {}) {
         try {
           const clonedResponse = result.clone();
           errorBody = await clonedResponse.text();
-        } catch (e) {
+        } catch (_e) {
           errorBody = 'Could not read error body';
         }
 

backend/src/lib/simplifiedPersistence.js

@@ -157,10 +157,13 @@ export class SimplifiedPersistence {
     // Create new conversation if needed
     if (isNewConversation) {
       const settings = await this.persistenceConfig.extractRequestSettingsAsync(bodyIn, userId);
+      // Support linked comparison conversations via parent_conversation_id
+      const parentConversationId = bodyIn.parent_conversation_id || null;
       conversationId = this.conversationManager.createNewConversation({
         sessionId,
         userId,
         providerId: this.providerId,
+        parentConversationId,
         ...settings
       });
       convo = this.conversationManager.getConversation(conversationId, userId);
@@ -176,7 +179,23 @@ export class SimplifiedPersistence {
    * @private
    */
   async _processMessageHistory(sessionId, userId, bodyIn, isNewConversation) {
-    const messages = this.persistenceConfig.filterNonSystemMessages(bodyIn.messages || []);
+    let messages = this.persistenceConfig.filterNonSystemMessages(bodyIn.messages || []);
+    const emptyAssistantMessages = messages.filter(
+      (msg) =>
+        msg?.role === 'assistant' &&
+        (msg.content === '' || (Array.isArray(msg.content) && msg.content.length === 0)) &&
+        (!Array.isArray(msg.tool_calls) || msg.tool_calls.length === 0) &&
+        (!Array.isArray(msg.tool_outputs) || msg.tool_outputs.length === 0)
+    );
+    if (emptyAssistantMessages.length > 0) {
+      logger.debug('[SimplifiedPersistence] Dropping empty assistant messages from client history', {
+        conversationId: this.conversationId,
+        parentConversationId: bodyIn?.parent_conversation_id ?? null,
+        count: emptyAssistantMessages.length,
+        ids: emptyAssistantMessages.map((msg) => msg?.id).filter(Boolean),
+      });
+      messages = messages.filter((msg) => !emptyAssistantMessages.includes(msg));
+    }
     const maxSeq = messages
       .map(msg => msg.seq)
       .filter(seq => typeof seq === 'number' && seq > 0)
@@ -192,9 +211,10 @@ export class SimplifiedPersistence {
       const latestUserMapping = [...this._latestSyncMappings].reverse().find(mapping => mapping.role === 'user');
       this.userMessageId = latestUserMapping?.persistedId != null ? String(latestUserMapping.persistedId) : null;
 
-      // Generate title only if this is the first message in a new conversation
-      // Fire-and-forget to avoid blocking the response
-      if (isNewConversation) {
+      // Generate title when the conversation is new or still lacks a title.
+      // Fire-and-forget to avoid blocking the response.
+      const needsTitle = isNewConversation || !this.conversationMeta?.title;
+      if (needsTitle) {
         const lastUser = ConversationTitleService.findLastUserMessage(messages);
         if (lastUser) {
           // Extract the model being used for the chat to use the same model for title generation