@supen-ai/cli 1.3.2 → 1.3.3

package/daemon/dist/http/routes/system.js

@@ -1615,6 +1615,16 @@ function coerceSingleQueryParam(value) {
         return value[0] || null;
     return value || null;
 }
+function codexThreadsListRoute(pathname) {
+    return /^\/api\/computers\/\{computer_id\}\/agents\/[^/]+\/codex\/threads$/.test(pathname);
+}
+function codexThreadActionRoute(pathname, action) {
+    const suffix = action === 'messages' ? 'messages' : action;
+    return pathname.match(new RegExp(`^/api/computers/\\{computer_id\\}/agents/[^/]+/codex/threads/([^/]+)/${suffix}$`));
+}
+function codexProjectsOpenRoute(pathname) {
+    return /^\/api\/computers\/\{computer_id\}\/agents\/[^/]+\/codex\/projects\/open$/.test(pathname);
+}
 function collectSpaceLogEntries(filters = {}) {
     const spaceId = currentSpaceId();
     const limit = Math.max(1, Math.min(filters.limit || SPACE_LOG_EVENT_LIMIT, 500));
@@ -1764,6 +1774,15 @@ const MANAGED_CODING_CLI_INSTALL_COMMANDS = {
     codex: CODING_CLI_INSTALL_COMMANDS.codex,
     gemini: CODING_CLI_INSTALL_COMMANDS.gemini,
 };
+function resolveManagedCodingCliInstallCommand(cli, current) {
+    if (cli === 'codex' && current.install_source === 'homebrew') {
+        return {
+            command: 'sh',
+            args: ['-lc', 'brew upgrade codex || brew upgrade --cask codex || brew reinstall --cask codex'],
+        };
+    }
+    return MANAGED_CODING_CLI_INSTALL_COMMANDS[cli];
+}
 const CODING_CLI_NAMES = ['codex', 'gemini', 'claude'];
 const DETECTABLE_SPACE_APPS = [
     { id: 'libreoffice', command: 'libreoffice', managed: false },
@@ -1971,7 +1990,7 @@ function inferCliInstallSource(name, executablePath, resolvedPath) {
         return { install_source: 'pnpm', update_supported: false };
     }
     if (normalized.includes('/homebrew/') || normalized.includes('/opt/homebrew/') || normalized.includes('/cellar/')) {
-        return { install_source: 'homebrew', update_supported: false };
+        return { install_source: 'homebrew', update_supported: name === 'codex' };
     }
     if (name === 'codex') {
         const npmRoot = detectGlobalNpmRoot();
@@ -2763,31 +2782,6 @@ export async function handleSystemRoutes(req, res, url, pathname, method) {
         writeJson(res, 200, await readCodexAgentStatusPayload());
         return true;
     }
-    if (pathname === '/api/computers/{computer_id}/runtime-models' && method === 'GET') {
-        writeJson(res, 200, readRuntimeModelStatusPayload());
-        return true;
-    }
-    if (pathname === '/api/computers/{computer_id}/runtime-models/default' && method === 'PUT') {
-        try {
-            const parsed = await readJsonBody(req);
-            const model = typeof parsed === 'object' && parsed && typeof parsed.model === 'string'
-                ? String(parsed.model).trim()
-                : '';
-            if (!model) {
-                writeProtocolError(res, 400, 'validation_error', 'missing_model', 'Request body must include a non-empty "model" string.');
-                return true;
-            }
-            writeCodexDefaultModel(model);
-            writeJson(res, 200, {
-                ok: true,
-                status: readRuntimeModelStatusPayload(),
-            });
-        }
-        catch (err) {
-            writeProtocolError(res, 500, 'config_error', 'codex_model_default_failed', err?.message || 'Failed to set Codex default model');
-        }
-        return true;
-    }
     const codexTransportDefaultMatch = pathname.match(/^\/api\/computers\/\{computer_id\}\/agents\/codex\/transports\/([^/]+)\/default$/);
     if (codexTransportDefaultMatch && method === 'PUT') {
         try {
@@ -2822,7 +2816,7 @@ export async function handleSystemRoutes(req, res, url, pathname, method) {
         }
         return true;
     }
-    if (pathname === '/api/computers/{computer_id}/agents/codex/install' && method === 'POST') {
+    if (pathname === '/api/computers/{computer_id}/agents/codex/update' && method === 'POST') {
         try {
             const cli = 'codex';
             const current = detectCliInstalled(cli);
@@ -2830,7 +2824,7 @@ export async function handleSystemRoutes(req, res, url, pathname, method) {
                 writeProtocolError(res, 400, 'validation_error', 'coding_cli_update_not_supported', `${cli} is installed via ${current.install_source || current.resolved_path || current.path || 'an unknown source'}; update it with that installer on this computer.`);
                 return true;
             }
-            const installSpec = MANAGED_CODING_CLI_INSTALL_COMMANDS.codex;
+            const installSpec = resolveManagedCodingCliInstallCommand(cli, current);
             const result = spawnSync(installSpec.command, installSpec.args, {
                 encoding: 'utf8',
                 timeout: 120_000,
@@ -2946,15 +2940,15 @@ export async function handleSystemRoutes(req, res, url, pathname, method) {
         writeJson(res, 200, buildHubSnapshotForSpace(spaceId));
         return true;
     }
-    if (pathname === '/api/computers/{computer_id}/usage' && method === 'GET') {
+    if (pathname === '/api/computers/{computer_id}/agents/codex/usage' && method === 'GET') {
         writeJson(res, 200, getGlobalUsage());
         return true;
     }
-    if (pathname === '/api/computers/{computer_id}/usage/daily' && method === 'GET') {
+    if (pathname === '/api/computers/{computer_id}/agents/codex/usage/daily' && method === 'GET') {
         writeJson(res, 200, { daily: getDailyUsage() });
         return true;
     }
-    if (pathname === '/api/computers/{computer_id}/codex/threads' && method === 'GET') {
+    if (codexThreadsListRoute(pathname) && method === 'GET') {
         const limitRaw = coerceSingleQueryParam(url.searchParams.get('limit'));
         const limit = limitRaw ? Number.parseInt(limitRaw, 10) : MIRRORED_THREAD_LIMIT;
         writeJson(res, 200, readMirroredTaskProjects(Number.isFinite(limit) ? limit : MIRRORED_THREAD_LIMIT));
@@ -2974,7 +2968,7 @@ export async function handleSystemRoutes(req, res, url, pathname, method) {
         serveRemoteFile(req, res, filePath);
         return true;
     }
-    const mirroredThreadArchiveMatch = pathname.match(/^\/api\/computers\/\{computer_id\}\/codex\/threads\/([^/]+)\/archive$/);
+    const mirroredThreadArchiveMatch = codexThreadActionRoute(pathname, 'archive');
     if (mirroredThreadArchiveMatch && method === 'POST') {
         const threadId = decodeURIComponent(mirroredThreadArchiveMatch[1] || '').trim();
         if (!threadId) {
@@ -2988,7 +2982,7 @@ export async function handleSystemRoutes(req, res, url, pathname, method) {
         writeJson(res, 200, { ok: true, archived: true });
         return true;
     }
-    const mirroredThreadStreamMatch = pathname.match(/^\/api\/computers\/\{computer_id\}\/codex\/threads\/([^/]+)\/stream$/);
+    const mirroredThreadStreamMatch = codexThreadActionRoute(pathname, 'stream');
     if (mirroredThreadStreamMatch && method === 'GET') {
         const threadId = decodeURIComponent(mirroredThreadStreamMatch[1] || '').trim();
         if (!threadId) {
@@ -3031,7 +3025,7 @@ export async function handleSystemRoutes(req, res, url, pathname, method) {
         res.flush?.();
         return true;
     }
-    const mirroredThreadHistoryMatch = pathname.match(/^\/api\/computers\/\{computer_id\}\/codex\/threads\/([^/]+)\/messages$/);
+    const mirroredThreadHistoryMatch = codexThreadActionRoute(pathname, 'messages');
     if (mirroredThreadHistoryMatch && method === 'GET') {
         const threadId = decodeURIComponent(mirroredThreadHistoryMatch[1] || '').trim();
         const limitRaw = coerceSingleQueryParam(url.searchParams.get('limit'));
@@ -3046,7 +3040,7 @@ export async function handleSystemRoutes(req, res, url, pathname, method) {
         writeJson(res, 200, history);
         return true;
     }
-    const mirroredThreadAdoptMatch = pathname.match(/^\/api\/computers\/\{computer_id\}\/codex\/threads\/([^/]+)\/adopt$/);
+    const mirroredThreadAdoptMatch = codexThreadActionRoute(pathname, 'adopt');
     if (mirroredThreadAdoptMatch && method === 'POST') {
         const threadId = decodeURIComponent(mirroredThreadAdoptMatch[1] || '').trim();
         const parsed = await readJsonBody(req);
@@ -3063,7 +3057,7 @@ export async function handleSystemRoutes(req, res, url, pathname, method) {
         writeJson(res, 200, { thread: serializeAdoptedMirroredThread(session) });
         return true;
     }
-    if (pathname === '/api/computers/{computer_id}/codex/projects/open' && method === 'POST') {
+    if (codexProjectsOpenRoute(pathname) && method === 'POST') {
         try {
             const parsed = await readJsonBody(req);
             const projectPath = parsed && typeof parsed === 'object' && typeof parsed.path === 'string'