@loicngr/kobo 1.2.0 → 1.4.0

package/AGENTS.md

@@ -171,6 +171,19 @@ See the "Notion integration" section of the README for the end-user setup guide.
 
 **Dependencies** — root `package.json` covers backend + tests. `src/client/package.json` is a separate npm tree. Install both.
 
+## Internationalization (i18n)
+
+The frontend uses `vue-i18n` v10 with 5 supported locales: English (`en`), French (`fr`), German (`de`), Spanish (`es`), Italian (`it`). Translation files live in `src/client/src/i18n/`.
+
+**Mandatory rules for all frontend code:**
+
+- **NEVER hardcode user-visible text** in Vue templates or scripts. Always use `$t('key')` in templates and `t('key')` in `<script setup>` (via `const { t } = useI18n()`).
+- When adding or modifying a text, **update ALL 5 locale files** (`en.ts`, `fr.ts`, `de.ts`, `es.ts`, `it.ts`) with the corresponding translation.
+- Keys follow the pattern `'component.label'` (e.g. `'git.push'`, `'settings.title'`, `'common.save'`). Use the existing key structure as reference.
+- Keep technical terms in English across all locales when that's the industry convention (Git, PR, Push, Diff, Commit, Branch, Tokens, etc.).
+- Placeholders like `{count}`, `{n}`, `{query}` must remain intact in all translations.
+- The language selector is in the Settings page (Global tab). The locale is auto-detected from the browser on first visit and persisted in `localStorage('kobo:locale')`.
+
 ## Testing discipline
 
 - **TDD for backend** — write the failing test, confirm it fails for the right reason, implement minimally, confirm it passes, commit. One commit per logical unit. See existing tests in `src/__tests__/workspace-service.test.ts` for the setup pattern (fresh in-memory DB per test via `resetDb()`).
@@ -225,3 +238,4 @@ The human user of this repository prefers French for conversational exchanges. C
 - Don't introduce ORMs, query builders, or schema validation libraries — the project is small enough for raw prepared statements and hand-written mappers.
 - Don't break the single-source-of-truth of `CLAUDE.md` → `AGENTS.md` symlink. Edit `AGENTS.md`; `CLAUDE.md` follows automatically.
 - Don't skip `try/catch` swallowing on best-effort cleanup (agent stop, dev-server stop, worktree removal). These must never break the primary operation.
+- Don't hardcode user-visible text in the frontend. Every string must go through `$t()` / `t()` with keys in all 5 locale files. See [Internationalization (i18n)](#internationalization-i18n).

package/dist/mcp-server/kobo-tasks-server.js

@@ -20,6 +20,7 @@ let db;
 try {
     db = new Database(dbPath, { readonly: false });
     db.pragma('journal_mode = WAL');
+    db.pragma('busy_timeout = 5000');
     db.pragma('foreign_keys = ON');
 }
 catch (err) {
@@ -205,6 +206,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
                 properties: {
                     status: {
                         type: 'string',
+                        enum: ['idle', 'completed', 'error'],
                         description: 'New status (e.g. idle, completed)',
                     },
                 },

package/dist/server/db/index.js

@@ -11,6 +11,7 @@ export function getDb(dbPath) {
     }
     instance = new Database(resolvedPath);
     instance.pragma('journal_mode=WAL');
+    instance.pragma('busy_timeout=5000');
     instance.pragma('foreign_keys=ON');
     return instance;
 }

package/dist/server/db/migrations.js

@@ -1,20 +1,82 @@
 import { initSchema } from './schema.js';
-export const SCHEMA_VERSION = 1;
+export const migrations = [
+    {
+        version: 2,
+        name: 'add-permission-mode',
+        migrate: (db) => {
+            db.exec("ALTER TABLE workspaces ADD COLUMN permission_mode TEXT NOT NULL DEFAULT 'auto-accept'");
+        },
+    },
+    {
+        version: 3,
+        name: 'add-workspace-id-indexes',
+        migrate: (db) => {
+            db.exec(`
+        CREATE INDEX IF NOT EXISTS idx_tasks_workspace_id ON tasks(workspace_id);
+        CREATE INDEX IF NOT EXISTS idx_agent_sessions_workspace_id ON agent_sessions(workspace_id);
+        CREATE INDEX IF NOT EXISTS idx_ws_events_workspace_id ON ws_events(workspace_id);
+      `);
+        },
+    },
+];
+/** Current schema version — always equals the highest migration version. */
+export const SCHEMA_VERSION = migrations.length > 0 ? migrations[migrations.length - 1].version : 1;
 export function runMigrations(db) {
+    // Create the history table (replaces the old single-row schema_version table).
     db.exec(`
-    CREATE TABLE IF NOT EXISTS schema_version (
-      version INTEGER NOT NULL
+    CREATE TABLE IF NOT EXISTS schema_migrations (
+      version  INTEGER PRIMARY KEY,
+      name     TEXT NOT NULL,
+      applied_at TEXT NOT NULL
     )
   `);
-    const row = db.prepare('SELECT version FROM schema_version LIMIT 1').get();
-    const currentVersion = row?.version ?? 0;
-    if (currentVersion < 1) {
+    // ── Backward compat: migrate from legacy schema_version table ──────────────
+    const hasLegacy = db.prepare("SELECT count(*) as c FROM sqlite_master WHERE type='table' AND name='schema_version'").get().c > 0;
+    if (hasLegacy) {
+        const legacyRow = db.prepare('SELECT version FROM schema_version LIMIT 1').get();
+        const legacyVersion = legacyRow?.version ?? 0;
+        // Back-fill history for all migrations that were already applied under the old system.
+        if (legacyVersion >= 1) {
+            const now = new Date().toISOString();
+            // Version 1 = initSchema (always applied if legacyVersion >= 1)
+            db.prepare('INSERT OR IGNORE INTO schema_migrations (version, name, applied_at) VALUES (?, ?, ?)').run(1, 'init-schema', now);
+            for (const m of migrations) {
+                if (m.version <= legacyVersion) {
+                    db.prepare('INSERT OR IGNORE INTO schema_migrations (version, name, applied_at) VALUES (?, ?, ?)').run(m.version, m.name, now);
+                }
+            }
+        }
+        db.exec('DROP TABLE schema_version');
+    }
+    // ── Determine current state ────────────────────────────────────────────────
+    const applied = new Set(db.prepare('SELECT version FROM schema_migrations').all().map((r) => r.version));
+    // Fresh install — no migrations applied yet.
+    if (!applied.has(1)) {
         initSchema(db);
-        if (currentVersion === 0) {
-            db.prepare('INSERT INTO schema_version (version) VALUES (?)').run(1);
+        const now = new Date().toISOString();
+        db.prepare('INSERT INTO schema_migrations (version, name, applied_at) VALUES (?, ?, ?)').run(1, 'init-schema', now);
+        // Mark all incremental migrations as applied (initSchema creates the latest shape).
+        for (const m of migrations) {
+            db.prepare('INSERT INTO schema_migrations (version, name, applied_at) VALUES (?, ?, ?)').run(m.version, m.name, now);
         }
-        else {
-            db.prepare('UPDATE schema_version SET version = ?').run(1);
+        return;
+    }
+    // Apply pending migrations sequentially.
+    for (const m of migrations) {
+        if (!applied.has(m.version)) {
+            m.migrate(db);
+            db.prepare('INSERT INTO schema_migrations (version, name, applied_at) VALUES (?, ?, ?)').run(m.version, m.name, new Date().toISOString());
         }
     }
 }
+/** Return the full migration history (for diagnostics / admin UI). */
+export function getMigrationHistory(db) {
+    try {
+        return db
+            .prepare('SELECT version, name, applied_at FROM schema_migrations ORDER BY version')
+            .all();
+    }
+    catch {
+        return [];
+    }
+}

package/dist/server/db/schema.js

@@ -10,6 +10,7 @@ export function initSchema(db) {
       notion_url TEXT,
       notion_page_id TEXT,
       model TEXT NOT NULL DEFAULT 'claude-opus-4-6',
+      permission_mode TEXT NOT NULL DEFAULT 'auto-accept',
       dev_server_status TEXT NOT NULL DEFAULT 'stopped',
       archived_at TEXT,
       created_at TEXT NOT NULL,
@@ -45,5 +46,9 @@ export function initSchema(db) {
       session_id TEXT,
       created_at TEXT NOT NULL
     );
+
+    CREATE INDEX IF NOT EXISTS idx_tasks_workspace_id ON tasks(workspace_id);
+    CREATE INDEX IF NOT EXISTS idx_agent_sessions_workspace_id ON agent_sessions(workspace_id);
+    CREATE INDEX IF NOT EXISTS idx_ws_events_workspace_id ON ws_events(workspace_id);
   `);
 }

package/dist/server/index.js

@@ -5,7 +5,7 @@ import path from 'node:path';
 import { serve } from '@hono/node-server';
 import { Hono } from 'hono';
 import { WebSocketServer } from 'ws';
-import { getDb } from './db/index.js';
+import { closeDb, getDb } from './db/index.js';
 import { runMigrations } from './db/migrations.js';
 import devServerRouter from './routes/dev-server.js';
 import gitRouter from './routes/git.js';
@@ -13,12 +13,13 @@ import imagesRouter from './routes/images.js';
 import notionRouter from './routes/notion.js';
 import settingsRouter from './routes/settings.js';
 import workspacesRouter from './routes/workspaces.js';
-import { getAvailableSkills, sendMessage, setBackendPort, startAgent, stopAgent } from './services/agent-manager.js';
+import { getAvailableSkills, sendMessage, setBackendPort, startAgent, startWatchdog, stopAgent, stopWatchdog, } from './services/agent-manager.js';
 import { startDevServer, stopDevServer } from './services/dev-server-service.js';
+import { startPrWatcher, stopPrWatcher } from './services/pr-watcher-service.js';
 import { emit, handleConnection, setMessageHandler } from './services/websocket-service.js';
 import { getLatestSession, getWorkspace, updateWorkspaceStatus } from './services/workspace-service.js';
-import { getClientSpaPath, getKoboHome } from './utils/paths.js';
-import { initProcessCleanup } from './utils/process-tracker.js';
+import { getClientSpaPath, getKoboHome, getPackageVersion } from './utils/paths.js';
+import { initProcessCleanup, killAll as killAllTrackedProcesses } from './utils/process-tracker.js';
 // 0. Runtime prerequisite check — warn if claude CLI is missing. Don't block
 // startup: the user may still want to configure settings or browse workspaces
 // before installing Claude Code.
@@ -32,12 +33,14 @@ console.log(`[kobo] Kōbō home: ${getKoboHome()}`);
 // 1. Initialize DB + run migrations
 const db = getDb();
 runMigrations(db);
-// 2. Initialize process cleanup
+// 2. Initialize process cleanup, agent watchdog, and PR watcher
 initProcessCleanup();
+startWatchdog();
+startPrWatcher();
 // 3. Create Hono app
 const app = new Hono();
 // Health check (root / is handled by the SPA catch-all below)
-app.get('/api/health', (c) => c.json({ status: 'ok', version: '0.1.0' }));
+app.get('/api/health', (c) => c.json({ status: 'ok', version: getPackageVersion() }));
 // 4. Mount route sub-routers
 app.route('/api/workspaces', workspacesRouter);
 app.route('/api/workspaces', imagesRouter);
@@ -47,7 +50,7 @@ app.route('/api/settings', settingsRouter);
 app.route('/api/dev-server', devServerRouter);
 // Skills endpoint
 app.get('/api/skills', (c) => c.json(getAvailableSkills()));
-const PORT = process.env.PORT ? parseInt(process.env.PORT, 10) : 3000;
+const PORT = parseInt(process.env.SERVER_PORT || process.env.PORT || '3000', 10);
 // 9. Serve static files from the built SPA if present (production mode).
 // The path is resolved relative to the package install directory, so this
 // works both in dev (tsx running from src/) and when installed via npm / npx
@@ -118,7 +121,7 @@ setMessageHandler((type, payload) => {
                 const workspace = getWorkspace(p.workspaceId);
                 if (workspace) {
                     const worktreePath = `${workspace.projectPath}/.worktrees/${workspace.workingBranch}`;
-                    startAgent(p.workspaceId, worktreePath, p.content, workspace.model, true);
+                    startAgent(p.workspaceId, worktreePath, p.content, workspace.model, true, workspace.permissionMode);
                     updateWorkspaceStatus(p.workspaceId, 'executing');
                 }
             }
@@ -136,7 +139,7 @@ setMessageHandler((type, payload) => {
             }
             const worktreePath = `${workspace.projectPath}/.worktrees/${workspace.workingBranch}`;
             const prompt = p.prompt ?? 'Continue the previous task where you left off.';
-            startAgent(p.workspaceId, worktreePath, prompt, workspace.model);
+            startAgent(p.workspaceId, worktreePath, prompt, workspace.model, false, workspace.permissionMode);
         }
         catch (err) {
             console.error('[ws] Failed to start agent:', err);
@@ -179,3 +182,54 @@ server.on('upgrade', (request, socket, head) => {
         socket.destroy();
     }
 });
+// 9. Graceful shutdown handler
+let isShuttingDown = false;
+function gracefulShutdown(signal) {
+    if (isShuttingDown)
+        return;
+    isShuttingDown = true;
+    console.log(`\n[kobo] Received ${signal}, shutting down gracefully…`);
+    // 1. Stop accepting new connections
+    wss.close(() => {
+        console.log('[kobo] WebSocket server closed');
+    });
+    server.close(() => {
+        console.log('[kobo] HTTP server closed');
+    });
+    // 2. Stop background services
+    try {
+        stopWatchdog();
+    }
+    catch {
+        // Best-effort
+    }
+    try {
+        stopPrWatcher();
+    }
+    catch {
+        // Best-effort
+    }
+    // 3. Kill all tracked child processes (agents, dev servers)
+    try {
+        killAllTrackedProcesses();
+        console.log('[kobo] Tracked processes killed');
+    }
+    catch {
+        // Best-effort
+    }
+    // 4. Close database
+    try {
+        closeDb();
+        console.log('[kobo] Database closed');
+    }
+    catch {
+        // Best-effort
+    }
+    // 4. Give a short grace period for in-flight requests, then exit
+    setTimeout(() => {
+        console.log('[kobo] Shutdown complete');
+        process.exit(0);
+    }, 2000).unref();
+}
+process.on('SIGTERM', () => gracefulShutdown('SIGTERM'));
+process.on('SIGINT', () => gracefulShutdown('SIGINT'));