Merge to Develop

.github/workflows/backend-release.yml

@@ -2,7 +2,7 @@ name: Backend Release
 
 on:
   push:
-    branches: [master]
+    branches: [master, develop]
     paths:
       - 'apps/backend/**'
       - 'packages/shared/**'
@@ -18,17 +18,13 @@ on:
         type: boolean
         default: false
 
-permissions:
-  contents: write
-  issues: write
-  pull-requests: write
-  id-token: write
-
 jobs:
   test:
     name: Test & Build
     runs-on: ubuntu-latest
     if: ${{ !inputs.skip_tests }}
+    permissions:
+      contents: read
     steps:
       - name: Checkout
         uses: actions/checkout@v4
@@ -60,8 +56,13 @@ jobs:
   release:
     name: Release & Publish
     needs: test
-    if: ${{ github.event_name != 'pull_request' && (always() && (needs.test.result == 'success' || inputs.skip_tests)) }}
+    if: ${{ github.ref == 'refs/heads/master' && github.event_name != 'pull_request' && (always() && (needs.test.result == 'success' || inputs.skip_tests)) }}
     runs-on: ubuntu-latest
+    permissions:
+      contents: write
+      issues: write
+      pull-requests: write
+      id-token: write
     steps:
       - name: Checkout
         uses: actions/checkout@v4

.github/workflows/extension-release.yml

@@ -2,7 +2,7 @@ name: Extension Release
 
 on:
   push:
-    branches: [master]
+    branches: [master, develop]
     paths:
       - 'apps/extension/**'
       - 'packages/shared/**'
@@ -65,7 +65,7 @@ jobs:
   release:
     name: Release
     needs: test
-    if: ${{ github.event_name != 'pull_request' && (always() && (needs.test.result == 'success' || inputs.skip_tests)) }}
+    if: ${{ github.ref == 'refs/heads/master' && github.event_name != 'pull_request' && (always() && (needs.test.result == 'success' || inputs.skip_tests)) }}
     runs-on: ubuntu-latest
     steps:
       - name: Checkout

.gitignore

@@ -7,6 +7,10 @@ dist/
 .output/
 *.tsbuildinfo
 
+# TypeScript declaration files
+*.d.ts
+*.d.ts.map
+
 # WXT
 .wxt/
 

apps/backend/package.json

@@ -43,41 +43,41 @@
     "test": "vitest",
     "test:coverage": "vitest --coverage",
     "typecheck": "tsc --noEmit",
-    "lint": "eslint src/",
+    "lint": "biome check src/",
     "clean": "rm -rf dist",
     "prepublishOnly": "pnpm run build",
     "release": "semantic-release"
   },
   "dependencies": {
     "@fastify/cors": "^10.0.2",
-    "@github/copilot": "^1.0.7",
-    "@github/copilot-sdk": "^0.1.32",
+    "@github/copilot": "^1.0.20",
+    "@github/copilot-sdk": "^0.2.1",
     "better-sqlite3": "^11.7.0",
-    "drizzle-orm": "^0.38.4",
-    "fastify": "^5.8.1",
+    "drizzle-orm": "^0.41.0",
+    "fastify": "^5.8.4",
     "pino": "^9.6.0",
     "pino-pretty": "^13.0.0",
     "sharp": "^0.34.5",
-    "zod": "^3.24.1"
+    "zod": "^3.25.0"
   },
   "devDependencies": {
     "@devmentorai/shared": "workspace:*",
     "@semantic-release/changelog": "^6.0.3",
     "@semantic-release/commit-analyzer": "^13.0.1",
     "@semantic-release/exec": "^7.1.0",
     "@semantic-release/git": "^10.0.1",
-    "@semantic-release/github": "^12.0.5",
+    "@semantic-release/github": "^12.0.6",
     "@semantic-release/npm": "^13.1.4",
     "@semantic-release/release-notes-generator": "^14.1.0",
     "@types/better-sqlite3": "^7.6.12",
     "@types/node": "^22.10.5",
     "@types/sharp": "^0.32.0",
     "conventional-changelog-conventionalcommits": "^9.1.0",
-    "drizzle-kit": "^0.30.2",
+    "drizzle-kit": "^0.31.0",
     "semantic-release": "^25.0.3",
     "tsup": "^8.5.1",
     "tsx": "^4.19.2",
     "typescript": "^5.7.3",
-    "vitest": "^2.1.8"
+    "vitest": "^4.1.4"
   }
 }

apps/backend/src/cli.ts

@@ -12,11 +12,11 @@
  *   doctor   Check system requirements
  */
 
+import { doctorCommand } from './cli/doctor.js';
+import { logsCommand } from './cli/logs.js';
 import { startCommand } from './cli/start.js';
-import { stopCommand } from './cli/stop.js';
 import { statusCommand } from './cli/status.js';
-import { logsCommand } from './cli/logs.js';
-import { doctorCommand } from './cli/doctor.js';
+import { stopCommand } from './cli/stop.js';
 
 const VERSION = '1.0.0';
 
@@ -60,7 +60,17 @@ async function main(): Promise<void> {
     process.exit(0);
   }
 
-  const options = parseOptions(args.slice(command === 'start' || command === 'stop' || command === 'status' || command === 'logs' || command === 'doctor' ? 1 : 0));
+  const options = parseOptions(
+    args.slice(
+      command === 'start' ||
+        command === 'stop' ||
+        command === 'status' ||
+        command === 'logs' ||
+        command === 'doctor'
+        ? 1
+        : 0
+    )
+  );
 
   try {
     switch (command) {
@@ -102,8 +112,8 @@ function parseOptions(args: string[]): CliOptions {
   for (let i = 0; i < args.length; i++) {
     switch (args[i]) {
       case '--port':
-        options.port = parseInt(args[++i], 10);
-        if (isNaN(options.port)) {
+        options.port = Number.parseInt(args[++i], 10);
+        if (Number.isNaN(options.port)) {
           console.error('Error: --port requires a valid number');
           process.exit(1);
         }
@@ -114,7 +124,7 @@ function parseOptions(args: string[]): CliOptions {
         break;
       case '--lines':
       case '-n':
-        options.lines = parseInt(args[++i], 10);
+        options.lines = Number.parseInt(args[++i], 10);
         break;
     }
   }

apps/backend/src/cli/doctor.ts

@@ -6,8 +6,8 @@
 import { execSync } from 'node:child_process';
 import fs from 'node:fs';
 import net from 'node:net';
-import { DATA_DIR, LOG_DIR, IMAGES_DIR } from '../lib/paths.js';
 import { DEFAULT_CONFIG } from '@devmentorai/shared';
+import { DATA_DIR, IMAGES_DIR, LOG_DIR } from '../lib/paths.js';
 
 const DEFAULT_PORT = DEFAULT_CONFIG.DEFAULT_PORT;
 
@@ -90,9 +90,17 @@ function checkPort(port: number): Promise<CheckResult> {
     const server = net.createServer();
     server.once('error', (err: NodeJS.ErrnoException) => {
       if (err.code === 'EADDRINUSE') {
-        resolve({ name: `Port ${port}`, status: 'warn', message: `Port ${port} is in use (server may already be running)` });
+        resolve({
+          name: `Port ${port}`,
+          status: 'warn',
+          message: `Port ${port} is in use (server may already be running)`,
+        });
       } else {
-        resolve({ name: `Port ${port}`, status: 'fail', message: `Cannot bind port ${port}: ${err.message}` });
+        resolve({
+          name: `Port ${port}`,
+          status: 'fail',
+          message: `Cannot bind port ${port}: ${err.message}`,
+        });
       }
     });
     server.once('listening', () => {
@@ -106,12 +114,19 @@ function checkPort(port: number): Promise<CheckResult> {
 
 function checkCopilotCli(): CheckResult {
   try {
-    const version = execSync('github-copilot --version 2>/dev/null || copilot --version 2>/dev/null', {
-      encoding: 'utf-8',
-      timeout: 5000,
-    }).trim();
+    const version = execSync(
+      'github-copilot --version 2>/dev/null || copilot --version 2>/dev/null',
+      {
+        encoding: 'utf-8',
+        timeout: 5000,
+      }
+    ).trim();
     return { name: 'Copilot CLI', status: 'pass', message: version || 'installed' };
   } catch {
-    return { name: 'Copilot CLI', status: 'warn', message: 'Not found — server will run in mock mode' };
+    return {
+      name: 'Copilot CLI',
+      status: 'warn',
+      message: 'Not found — server will run in mock mode',
+    };
   }
 }

apps/backend/src/cli/logs.ts

@@ -4,8 +4,8 @@
  */
 
 import fs from 'node:fs';
-import { LOG_FILE } from '../lib/paths.js';
 import type { CliOptions } from '../cli.js';
+import { LOG_FILE } from '../lib/paths.js';
 
 export async function logsCommand(options: CliOptions): Promise<void> {
   const lines = options.lines || 50;

apps/backend/src/cli/start.ts

@@ -3,10 +3,10 @@
  * Starts the DevMentorAI server in background or foreground mode.
  */
 
+import { DEFAULT_CONFIG, checkForUpdate } from '@devmentorai/shared';
 import type { CliOptions } from '../cli.js';
 import { isServerRunning, spawnServer, waitForHealthy } from '../lib/daemon.js';
 import { LOG_FILE } from '../lib/paths.js';
-import { DEFAULT_CONFIG, checkForUpdate } from '@devmentorai/shared';
 import { BACKEND_VERSION } from '../version.js';
 
 const DEFAULT_PORT = DEFAULT_CONFIG.DEFAULT_PORT;
@@ -16,8 +16,8 @@ async function showUpdateNotice(): Promise<void> {
     const info = await checkForUpdate('backend', BACKEND_VERSION);
     if (info.hasUpdate) {
       console.log(`\n  ⚠ Update available: ${BACKEND_VERSION} → ${info.latestVersion}`);
-      console.log(`    Run: npx devmentorai-server@latest`);
-      console.log(`    Or: npm install -g devmentorai-server@latest`);
+      console.log('    Run: npx devmentorai-server@latest');
+      console.log('    Or: npm install -g devmentorai-server@latest');
       console.log(`    ${info.releaseUrl}\n`);
     }
   } catch {
@@ -30,7 +30,7 @@ async function showAuthNotice(port: number): Promise<void> {
     const response = await fetch(`http://127.0.0.1:${port}/api/account/auth`);
     if (!response.ok) return;
 
-    const payload = await response.json() as {
+    const payload = (await response.json()) as {
       success?: boolean;
       data?: { isAuthenticated?: boolean; login?: string | null };
     };
@@ -85,13 +85,13 @@ export async function startCommand(options: CliOptions): Promise<void> {
   const healthy = await waitForHealthy(port);
 
   if (healthy) {
-    console.log(`✓ Server started successfully`);
+    console.log('✓ Server started successfully');
     console.log(`  → http://127.0.0.1:${port}`);
     console.log(`  Logs: ${LOG_FILE}\n`);
     await showAuthNotice(port);
     await showUpdateNotice();
   } else {
-    console.error(`✗ Server started but healthcheck failed`);
+    console.error('✗ Server started but healthcheck failed');
     console.error(`  Check logs: ${LOG_FILE}\n`);
     process.exit(1);
   }

apps/backend/src/cli/status.ts

@@ -3,9 +3,9 @@
  * Shows the current status of the DevMentorAI server.
  */
 
-import { isServerRunning, readPid, healthcheck } from '../lib/daemon.js';
-import { PID_FILE, LOG_FILE, DATA_DIR } from '../lib/paths.js';
 import { DEFAULT_CONFIG } from '@devmentorai/shared';
+import { healthcheck, isServerRunning, readPid } from '../lib/daemon.js';
+import { DATA_DIR, LOG_FILE, PID_FILE } from '../lib/paths.js';
 
 const DEFAULT_PORT = DEFAULT_CONFIG.DEFAULT_PORT;
 
@@ -26,7 +26,7 @@ export async function statusCommand(): Promise<void> {
   const pid = readPid();
   const health = await healthcheck(DEFAULT_PORT);
 
-  console.log(`  Status:  ✓ running`);
+  console.log('  Status:  ✓ running');
   console.log(`  PID:     ${pid || 'unknown'}`);
   console.log(`  Port:    ${DEFAULT_PORT}`);
   console.log(`  URL:     http://127.0.0.1:${DEFAULT_PORT}`);
@@ -38,7 +38,9 @@ export async function statusCommand(): Promise<void> {
       console.log(`  Uptime:  ${formatUptime(data.uptime as number)}`);
     }
     if (data.copilotConnected !== undefined) {
-      console.log(`  Copilot: ${data.copilotConnected ? '✓ connected' : '⊘ disconnected (mock mode)'}`);
+      console.log(
+        `  Copilot: ${data.copilotConnected ? '✓ connected' : '⊘ disconnected (mock mode)'}`
+      );
     }
   }
 

apps/backend/src/cli/stop.ts

@@ -3,8 +3,8 @@
  * Stops the running DevMentorAI server.
  */
 
-import { isServerRunning, stopServer, readPid } from '../lib/daemon.js';
 import { DEFAULT_CONFIG } from '@devmentorai/shared';
+import { isServerRunning, readPid, stopServer } from '../lib/daemon.js';
 
 const DEFAULT_PORT = DEFAULT_CONFIG.DEFAULT_PORT;
 

apps/backend/src/db/index.ts

@@ -1,7 +1,7 @@
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
 import Database from 'better-sqlite3';
-import path from 'path';
-import os from 'os';
-import fs from 'fs';
 
 const DB_DIR = path.join(os.homedir(), '.devmentorai');
 const DB_PATH = path.join(DB_DIR, 'devmentorai.db');
@@ -15,10 +15,10 @@ export function initDatabase(): Database.Database {
   }
 
   const db = new Database(DB_PATH);
-  
+
   // Enable WAL mode for better performance
   db.pragma('journal_mode = WAL');
-  
+
   // Create tables
   db.exec(`
     CREATE TABLE IF NOT EXISTS sessions (
@@ -64,6 +64,34 @@ export function initDatabase(): Database.Database {
     CREATE INDEX IF NOT EXISTS idx_session_contexts_extracted_at ON session_contexts(extracted_at);
   `);
 
+  // Migration: Add tone, explain_tradeoffs, reasoning_effort columns if they don't exist
+  try {
+    db.exec(`
+      ALTER TABLE sessions ADD COLUMN tone TEXT DEFAULT 'balanced';
+    `);
+    console.log('[DB] Migration: Added tone column');
+  } catch {
+    // Column already exists
+  }
+
+  try {
+    db.exec(`
+      ALTER TABLE sessions ADD COLUMN explain_tradeoffs INTEGER DEFAULT 0;
+    `);
+    console.log('[DB] Migration: Added explain_tradeoffs column');
+  } catch {
+    // Column already exists
+  }
+
+  try {
+    db.exec(`
+      ALTER TABLE sessions ADD COLUMN reasoning_effort TEXT;
+    `);
+    console.log('[DB] Migration: Added reasoning_effort column');
+  } catch {
+    // Column already exists
+  }
+
   return db;
 }