omnikey-cli 1.1.0 → 1.2.0

package/backend-dist/agent/agentServer.js

@@ -603,19 +603,19 @@ async function runAgentTurnInternal(sessionId, subscription, clientMessage, send
             // is not the final turn (e.g. plain-text conclusion after terminal
             // output). Treat it as a final answer so the client is never left
             // hanging.
-            log.info('Agent returned untagged content on a non-final turn; treating as final answer', {
+            log.info('Agent returned untagged content on a non-final turn; treating as assistant response and looping the function again.', {
                 sessionId,
                 subscriptionId: subscription.id,
                 turn: session.turns,
             });
             (0, utils_1.pushToSessionHistory)(log, session, { role: 'assistant', content });
             await persistSessionToDB(sessionId, session);
-            send({
-                session_id: sessionId,
+            await runAgentTurnInternal(sessionId, subscription, {
                 sender: 'agent',
-                content: `<final_answer>\n${content}\n</final_answer>`,
-            });
-            void (0, sessionGrouping_1.updateSessionGroup)(sessionId, subscription.id);
+                session_id: sessionId,
+                content: '',
+                is_web_call: true,
+            }, send, logger_1.logger, options);
         }
         else {
             log.warn('Agent returned empty content with no recognized tags; sending error', {
@@ -797,7 +797,12 @@ function buildTranscript(raw) {
                 break;
             }
         }
-        currentAssistant.text = finalText || blocks.map((b) => b.text).join('\n\n').trim();
+        currentAssistant.text =
+            finalText ||
+                blocks
+                    .map((b) => b.text)
+                    .join('\n\n')
+                    .trim();
         messages.push(currentAssistant);
         currentAssistant = null;
     };

package/backend-dist/db.js

@@ -44,6 +44,68 @@ async function runSQLiteMigrations(logger) {
             logger.info(`SQLite migration: added column ${table}.${column}`);
         }
     }
+    // mcp_servers was originally created with UNIQUE on both subscription_id and
+    // name as column-level constraints (SQLite auto-indexes). These can't be
+    // dropped with DROP INDEX — the only fix is to recreate the table with the
+    // correct schema (composite unique on subscription_id+name only).
+    await migrateMcpServersTableIfNeeded(logger);
+}
+async function migrateMcpServersTableIfNeeded(logger) {
+    // Check if the old schema is still in place by inspecting the CREATE TABLE sql.
+    const rows = (await sequelize.query(`SELECT sql FROM sqlite_master WHERE type='table' AND name='mcp_servers'`))[0];
+    if (!rows.length)
+        return; // table doesn't exist yet — sync() will create it correctly
+    const createSql = rows[0].sql;
+    // Old schema has UNIQUE on subscription_id at the column level.
+    // New schema only has the composite index mcp_servers_subscription_id_name.
+    const needsMigration = /`subscription_id`[^,]*UNIQUE/i.test(createSql);
+    if (!needsMigration)
+        return;
+    logger.info('SQLite migration: recreating mcp_servers table to remove stale UNIQUE constraints');
+    await sequelize.query('PRAGMA foreign_keys = OFF');
+    try {
+        await sequelize.query('BEGIN TRANSACTION');
+        await sequelize.query(`
+      CREATE TABLE \`mcp_servers_new\` (
+        \`id\` VARCHAR(255) NOT NULL PRIMARY KEY,
+        \`subscription_id\` VARCHAR(255) NOT NULL REFERENCES \`subscriptions\` (\`id\`) ON DELETE CASCADE ON UPDATE CASCADE,
+        \`name\` VARCHAR(100) NOT NULL,
+        \`description\` VARCHAR(500),
+        \`transport\` VARCHAR(16) NOT NULL DEFAULT 'stdio',
+        \`command\` VARCHAR(500),
+        \`args\` JSON NOT NULL DEFAULT '[]',
+        \`env\` JSON NOT NULL DEFAULT '{}',
+        \`url\` VARCHAR(1000),
+        \`headers\` JSON NOT NULL DEFAULT '{}',
+        \`is_enabled\` TINYINT(1) NOT NULL DEFAULT 1,
+        \`last_connected_at\` DATETIME,
+        \`last_error\` TEXT,
+        \`createdAt\` DATETIME NOT NULL,
+        \`updatedAt\` DATETIME NOT NULL
+      )
+    `);
+        await sequelize.query(`
+      INSERT INTO \`mcp_servers_new\`
+        SELECT id, subscription_id, name, description, transport, command, args, env,
+               url, headers, is_enabled, last_connected_at, last_error, createdAt, updatedAt
+        FROM \`mcp_servers\`
+    `);
+        await sequelize.query('DROP TABLE `mcp_servers`');
+        await sequelize.query('ALTER TABLE `mcp_servers_new` RENAME TO `mcp_servers`');
+        await sequelize.query(`
+      CREATE UNIQUE INDEX IF NOT EXISTS \`mcp_servers_subscription_id_name\`
+        ON \`mcp_servers\` (\`subscription_id\`, \`name\`)
+    `);
+        await sequelize.query('COMMIT');
+        logger.info('SQLite migration: mcp_servers table recreated successfully');
+    }
+    catch (err) {
+        await sequelize.query('ROLLBACK');
+        throw err;
+    }
+    finally {
+        await sequelize.query('PRAGMA foreign_keys = ON');
+    }
 }
 async function initDatabase(logger) {
     try {

package/backend-dist/index.js

@@ -107,7 +107,7 @@ app.get('/macos/appcast', (req, res) => {
 // ── Windows distribution endpoints ───────────────────────────────────────────
 // These should match the values in windows/OmniKey.Windows.csproj
 // <Version> and windows/build_release_zip.ps1 $APP_VERSION.
-const WIN_VERSION = '1.12';
+const WIN_VERSION = '1.13';
 const WIN_ZIP_FILENAME = 'OmniKeyAI-windows-win-x64.zip';
 const WIN_ZIP_PATH = path_1.default.join(process.cwd(), 'windows', WIN_ZIP_FILENAME);
 // Serves the pre-built ZIP produced by windows/build_release_zip.ps1.
@@ -152,15 +152,19 @@ app.get('/windows/update', (req, res) => {
         releaseNotes: [
             `What's new in ${WIN_VERSION}`,
             ``,
-            `• Brand-new WPF shell — Agent Chat, OmniAgent Session, Task Instructions, Scheduled Jobs, Job Run History, MCP Servers, Manual, Subscription, and Check Updates all reachable from a unified sidebar.`,
-            `• Agent Chat with session sidebar, search, hover-to-delete, context-window indicator, default task-instruction picker (with a built-in "No task instructions" option).`,
-            `• OmniAgent Session replaces the old thinking window — sticky session picker, streaming collapsible timeline (web search, MCP, reasoning, terminal) and a copyable final-answer card. Selecting an existing session previews its last turn in-place.`,
-            `• Scheduled Jobs redesigned: clearer labels, self-contained Active toggle, restructured Save / Run now / Reset / Delete action bar, tooltips throughout.`,
-            `• Theme reworked to match the app icon — cyan + light purple. The Windows system-accent leak (orange/red buttons) is gone everywhere.`,
-            `• Modern typography (Aptos / Inter / Segoe UI Variable Display).`,
-            `• Mouse-wheel scroll fixed across Chat and OmniAgent pages.`,
-            `• Production-default backend URL: shipping binary defaults to https://omnikeyai.ca.`,
-            `• Internals: removed ~5,200 lines of unwired WinForms code; build is warning-free.`,
+            `• Projects: chats are now grouped by project in the sidebar — collapsible "folder" headers per group, a session count badge, and per-header collapse state that survives streaming turns.`,
+            `• Projects: new project picker in the composer toolbar (next to the task-instruction selector) — pick the project for your next turn, mirrors the macOS "Select project" menu. Auto-hides until the backend has classified at least one group.`,
+            `• Projects: the chosen project is stamped onto the outbound message and the optimistic session placeholder, so new chats appear under the right header immediately.`,
+            `• Chat: messages now sit in a centered 820 DIP reading column on large monitors and stretch edge-to-edge on smaller windows — matches the macOS layout exactly. User bubbles stay pinned right, assistant content stays pinned left, on every viewport.`,
+            `• Chat: new Final Answer card with a soft "paper" surface (mirrors macOS), copy button anchored bottom-right so it no longer overlaps long markdown headings, and an "Answer" tooltip on copy.`,
+            `• Chat: animated typing indicator (pulsing sparkle + three staggered dots) appears the moment you send your first message — matches macOS TypingDotsView.`,
+            `• Chat: extra breathing room between thinking-timeline steps so the agent's intermediate reasoning reads as discrete actions instead of a cramped wall.`,
+            `• Markdown: brand-new Nord-themed renderer — no more white-background leaks from the underlying MdXaml engine on paragraphs, blockquotes, lists, tables, or inline code.`,
+            `• Markdown: bullets and numbered lists are no longer clipped on the left edge.`,
+            `• Markdown: inline code now renders as a soft pill (BadgeFill) instead of a dark slab; fenced code blocks keep their rounded macOS-style chrome with language label + copy.`,
+            `• MCP Servers: editor now supports custom HTTP headers — one Key: Value per line, monospace input, persisted alongside the URL. Authorization headers are unredacted on edit so they round-trip cleanly, and stale fetches won't clobber what you're typing.`,
+            `• Composer: capped + centered at 820 DIP on wide monitors for a balanced layout, full pane width on smaller windows.`,
+            `• Theme: shared interactive-surface brushes (Hover, Press, CodeBackground, UserBubble, AssistantText, DangerSoft, FinalAnswerSurface, BadgeFill) promoted to NordTheme.xaml so every page stays in visual lockstep.`,
         ].join('\n'),
     });
 });

package/backend-dist/mcpServerRoutes.js

@@ -122,7 +122,12 @@ function mcpServerRouter() {
                 return res.status(400).json({ error: 'Invalid MCP server data.' });
             }
             if (err?.name === 'SequelizeUniqueConstraintError') {
-                return res.status(409).json({ error: 'An MCP server with that name already exists.' });
+                const isNameConflict = err?.fields?.includes('name') || err?.errors?.some((e) => e.path === 'name');
+                return res.status(409).json({
+                    error: isNameConflict
+                        ? 'An MCP server with that name already exists.'
+                        : 'Failed to create MCP server due to a conflict.',
+                });
             }
             res.status(500).json({ error: 'Failed to create MCP server.' });
         }

package/package.json

@@ -4,7 +4,7 @@
     "access": "public",
     "registry": "https://registry.npmjs.org/"
   },
-  "version": "1.1.0",
+  "version": "1.2.0",
   "description": "CLI for onboarding users to Omnikey AI and configuring OPENAI_API_KEY. Use Yarn for install/build.",
   "engines": {
     "node": ">=14.0.0",