c0de-agent 1.13.1 → 1.14.0

package/dist/server/routes/files.js

@@ -1,14 +1,17 @@
+import { existsSync } from 'node:fs';
 import { access, mkdir, readdir, readFile, writeFile } from 'node:fs/promises';
 import { dirname, join, relative } from 'node:path';
 import { Hono } from 'hono';
 import trash from 'trash';
 import { createSummarizer } from '../../core/compact.js';
 import { loadConfigScopes, mergeConfig } from '../../core/config.js';
+import { buildRegistryFromConfig } from '../../core/provider-registry.js';
 import { buildFallbackChain } from '../../llm/routing.js';
 import { getProject } from '../../project/project.js';
-import { appendToGitignore, checkIgnored, checkoutGitBranch, createGitBranch, getGitBranch, getGitDiffSummary, getGitLastCommit, getGitStatus, listGitBranches, performGitCommit, } from '../../project/resolve.js';
+import { appendToGitignore, checkIgnored, checkoutGitBranch, createGitBranch, getGitBranch, getGitDiffSummary, getGitLastCommit, getGitStatus, isValidBranchName, listGitBranches, performGitCommit, } from '../../project/resolve.js';
+import { headChars } from '../../shared/utils/string.js';
 import { apiError } from '../middleware/error.js';
-import { buildRegistryFromConfig } from '../registry-config.js';
+import { readJsonObject } from '../util/json-body.js';
 import { safeResolve } from '../util/safe-path.js';
 /** 递归搜索时跳过的目录（体积大/为元数据噪音，避免递归进入）。
  *  P3：补齐常见构建/缓存产物目录——此前只跳 .git/node_modules，
@@ -34,7 +37,11 @@ const SEARCH_MAX_RESULTS = 5000;
 /** 递归收集文件列表（用于搜索）。P3：深度上限 5 → 8，深层文件此前搜不到。
  *  P3-14：达结果上限即停止遍历，防大仓库全量 walk。 */
 async function collectFiles(dir, basePath, maxDepth = 8, budget = SEARCH_MAX_RESULTS) {
-    if (maxDepth < 0 || budget <= 0)
+    // maxDepth 语义：能列举条目的调用层级。`maxDepth < 0` 会放 maxDepth=0 的
+    // 调用进来多收集一层（上限 8 时第 9 层的文件/目录仍被搜到）——守卫与
+    // 「深度上限」契约差一层。maxDepth <= 0 使收集层数恰为声明的上限：
+    // 首次调用 maxDepth=D 列举第 1 层，递归递减，第 D 层之后不再列举。
+    if (maxDepth <= 0 || budget <= 0)
         return [];
     const results = [];
     let entries;
@@ -135,6 +142,29 @@ function rejectCommitMessage(c, reason, mode) {
 }
 function createFilesRoute(ctx) {
     const app = new Hono();
+    /**
+     * 从请求 URL 提取工作区相对文件路径（逐段 decodeURIComponent）。
+     *
+     * 不能直接用 Hono 的 `c.req.path`：它经 `decodeURI`，只解码非 reserved 转义，
+     * `? # : @ & = + $ , ;` 的转义被原样保留。客户端 encodeURIComponent 后
+     * `a%3Fb.txt` 在服务端永远对不上磁盘上的 `a?b.txt`——读取 404，写入/删除更会
+     * 落到另一个字面名为 `a%3Fb.txt` 的幽灵文件（界面报成功、用户改动丢失）。
+     * 基于原始 URL 逐段解码（而非整串解码）保证 `%2F` 不会拆出新的路径层级。
+     * 非法转义（`%zz` 等）返回 null，由调用方转 400。
+     */
+    const extractFilePath = (c) => {
+        try {
+            return new URL(c.req.url).pathname
+                .replace(/^\/api\/files\/?/, '')
+                .replace(/^\/+/, '')
+                .split('/')
+                .map((segment) => decodeURIComponent(segment))
+                .join('/');
+        }
+        catch {
+            return null;
+        }
+    };
     // git 状态：返回 path → 状态分类 的映射（非 git 返回空对象）
     app.get('/git-status', async (c) => {
         const projectId = c.req.query('projectId');
@@ -190,9 +220,10 @@ function createFilesRoute(ctx) {
             return apiError(c, 400, 'NO_CHANGES', 'No changes to commit');
         }
         // 可选 body：mode / message / suggestions
-        const body = await c.req
-            .json()
-            .catch(() => ({}));
+        const parsedBody = await readJsonObject(c);
+        if (!parsedBody.ok)
+            return parsedBody.response;
+        const body = parsedBody.body;
         // --- mode: force — 跳过检查，用传入 message 直接提交 ---
         if (body.mode === 'force') {
             const check = checkCommitMessage(body.message);
@@ -252,7 +283,7 @@ Reply as JSON ONLY:
 
 If no files need ignoring, return an empty array for ignoreSuggestions.
 
-${summary.diff.slice(0, 8000)}`;
+${headChars(summary.diff, 8000)}`;
         let raw;
         try {
             // P1-1：项目级 provider 注册表（项目配置的 provider 才能路由）。
@@ -333,10 +364,20 @@ ${summary.diff.slice(0, 8000)}`;
             }
             root = project.worktree;
         }
-        const body = await c.req.json().catch(() => ({}));
-        const branch = body.branch;
+        const parsed = await readJsonObject(c);
+        if (!parsed.ok)
+            return parsed.response;
+        // branch 必须是非空字符串：数字此前经 as 断言进 spawnSync 参数抛
+        // ERR_INVALID_ARG_TYPE → 500。
+        const branch = typeof parsed.body.branch === 'string' ? parsed.body.branch : '';
         if (!branch)
             return apiError(c, 400, 'BAD_REQUEST', 'branch is required');
+        // 分支名合法性在入口显式 400：以 `-` 开头的名字会被 git 当作**选项**解释
+        //（`git checkout -B<name>` 静默重置分支指针、`--detach` 让 HEAD 脱离分支），
+        // 且这类输入属客户端错误而非服务端故障（resolve 层同样兜底拒绝）。
+        if (!isValidBranchName(branch)) {
+            return apiError(c, 400, 'INVALID_BRANCH', `非法分支名：${branch}`);
+        }
         const result = checkoutGitBranch(root, branch);
         if ('error' in result) {
             return apiError(c, 500, 'CHECKOUT_FAILED', result.error);
@@ -354,10 +395,17 @@ ${summary.diff.slice(0, 8000)}`;
             }
             root = project.worktree;
         }
-        const body = await c.req.json().catch(() => ({}));
-        const name = body.name;
+        const parsed = await readJsonObject(c);
+        if (!parsed.ok)
+            return parsed.response;
+        const name = typeof parsed.body.name === 'string' ? parsed.body.name : '';
         if (!name)
             return apiError(c, 400, 'BAD_REQUEST', 'name is required');
+        // 与 /git-checkout 同口径：非法分支名（含以 `-` 开头）显式 400，
+        // 不透出 git 的 stderr（那是输入非法而非 git 故障）。
+        if (!isValidBranchName(name)) {
+            return apiError(c, 400, 'INVALID_BRANCH', `非法分支名：${name}`);
+        }
         const result = createGitBranch(root, name);
         if ('error' in result) {
             return apiError(c, 500, 'BRANCH_CREATE_FAILED', result.error);
@@ -431,9 +479,9 @@ ${summary.diff.slice(0, 8000)}`;
     // 读取文件
     // projectId 指定时按对应项目 worktree 解析，否则回退 ctx.cwd（向后兼容）。
     app.get('/*', async (c) => {
-        const path = c.req.path.replace(/^\/api\/files\//, '').replace(/^\//, '');
-        const raw = path.endsWith('/raw');
-        const filePath = raw ? path.slice(0, -'/raw'.length) : path;
+        const path = extractFilePath(c);
+        if (path === null)
+            return apiError(c, 400, 'BAD_REQUEST', '非法的 URL 转义序列');
         const projectId = c.req.query('projectId');
         let root = ctx.cwd;
         if (projectId) {
@@ -443,6 +491,13 @@ ${summary.diff.slice(0, 8000)}`;
             }
             root = project.worktree;
         }
+        // /raw 后缀是媒体预览的协议标记，但仅当「完整路径不存在」时才按 raw 模式
+        // 解释——真实存在的 .../raw 文件按普通文件字面读取。此前纯后缀判定优先，
+        // 名为 raw 的文件（data/raw 等）恒被误判为 raw 模式，去后缀父路径必为目录
+        // （readFile 抛 EISDIR → 404），该文件永远读不到。
+        const fullResolved = safeResolve(root, path);
+        const raw = path.endsWith('/raw') && !(fullResolved !== null && existsSync(fullResolved));
+        const filePath = raw ? path.slice(0, -'/raw'.length) : path;
         const resolved = safeResolve(root, filePath);
         if (!resolved) {
             return apiError(c, 403, 'FORBIDDEN', 'Path outside workspace');
@@ -476,7 +531,9 @@ ${summary.diff.slice(0, 8000)}`;
     // 写入文件
     // projectId 指定时按对应项目 worktree 解析，否则回退 ctx.cwd（向后兼容）。
     app.put('/*', async (c) => {
-        const path = c.req.path.replace(/^\/api\/files\//, '').replace(/^\//, '');
+        const path = extractFilePath(c);
+        if (path === null)
+            return apiError(c, 400, 'BAD_REQUEST', '非法的 URL 转义序列');
         const projectId = c.req.query('projectId');
         let root = ctx.cwd;
         if (projectId) {
@@ -512,7 +569,9 @@ ${summary.diff.slice(0, 8000)}`;
     // 删除文件/目录（移入系统回收站）
     // projectId 指定时按对应项目 worktree 解析，否则回退 ctx.cwd（向后兼容）。
     app.delete('/*', async (c) => {
-        const path = c.req.path.replace(/^\/api\/files\//, '').replace(/^\//, '');
+        const path = extractFilePath(c);
+        if (path === null)
+            return apiError(c, 400, 'BAD_REQUEST', '非法的 URL 转义序列');
         const projectId = c.req.query('projectId');
         let root = ctx.cwd;
         if (projectId) {

package/dist/server/routes/filesystem.js

@@ -42,7 +42,11 @@ async function searchDirectories(directory, query, limit = 50, maxDepth = 5) {
     const lowerQuery = query.toLowerCase();
     const results = [];
     async function walk(dir, depth) {
-        if (results.length >= limit || depth > maxDepth)
+        // depth 语义：walk(base, 0) 处理的是第 1 层条目。守卫必须挡住
+        // depth === maxDepth 的调用——`depth > maxDepth` 会放它进来多收集一层
+        // （maxDepth=1 实测返回 ['a','a/b']）。depth >= maxDepth 使收集层数恰为
+        // maxDepth：walk 只在 depth ∈ [0, maxDepth) 时推入条目。
+        if (results.length >= limit || depth >= maxDepth)
             return;
         let entries;
         try {
@@ -93,7 +97,10 @@ function createFilesystemRoute(ctx) {
     app.get('/search', async (c) => {
         const rawDir = c.req.query('directory') ?? '';
         const q = c.req.query('q') ?? '';
-        const limitRaw = Number.parseInt(c.req.query('limit') ?? '', 10);
+        // limit 仅接受十进制正整数整体：parseInt('2abc') 会折成 2 静默截断结果集。
+        // 非法/缺失回落默认 50（与 read/grep 的数值参数显式校验同口径）。
+        const limitParam = c.req.query('limit') ?? '';
+        const limitRaw = /^\d+$/.test(limitParam) ? Number(limitParam) : Number.NaN;
         const limit = Number.isFinite(limitRaw) && limitRaw > 0 ? Math.min(limitRaw, 200) : 50;
         const directory = expandPath(rawDir);
         const items = await searchDirectories(directory, q, limit);

package/dist/server/routes/kanban.js

@@ -3,11 +3,50 @@
 // P2-5：/deleted* 为看板回收站端点（必须注册在 /:projectId 之前避免被参数路由吞掉）。
 import { existsSync } from 'node:fs';
 import { Hono } from 'hono';
-import { createKanbanStore, getDeletedKanbanBoard, KanbanCardNotFoundError, KanbanColumnInUseError, KanbanColumnNotFoundError, KanbanInvalidPriorityError, listDeletedKanbanBoards, mergeKanbanBoard, permanentlyDeleteKanbanBoard, restoreKanbanBoard, } from '../../kanban/index.js';
+import { createKanbanStore, getDeletedKanbanBoard, KanbanCardNotFoundError, KanbanColumnInUseError, KanbanColumnNotFoundError, KanbanInvalidPositionError, KanbanInvalidPriorityError, listDeletedKanbanBoards, mergeKanbanBoard, permanentlyDeleteKanbanBoard, restoreKanbanBoard, } from '../../kanban/index.js';
 import { fromDirectory, getProject } from '../../project/index.js';
+import { isUuid } from '../../shared/utils/string.js';
 import { apiError } from '../middleware/error.js';
+import { readJsonObject } from '../util/json-body.js';
+// ── body 字段类型守卫 ──────────────────────────────────────────────
+// 畸形 JSON 此前直接 as 断言后进 store/DB：数字 title → .trim TypeError 500、
+// columns/labels 数组含 null/缺字段 → 读 null.id TypeError 500、labels 字符串
+// → 持久化毒化卡片（后续每轮前端 .map 崩溃）。
+const VALID_PRIORITIES = new Set(['high', 'medium', 'low']);
+function isValidColumnDef(v) {
+    return (v !== null &&
+        typeof v === 'object' &&
+        typeof v.id === 'string' &&
+        typeof v.name === 'string');
+}
+function isValidLabelDef(v) {
+    return (v !== null &&
+        typeof v === 'object' &&
+        typeof v.id === 'string' &&
+        typeof v.name === 'string' &&
+        typeof v.color === 'string');
+}
+function isValidStringArray(v) {
+    return Array.isArray(v) && v.every((x) => typeof x === 'string');
+}
+/** 列/标签配置含重复 id：拖拽目标歧义、前端按 id 索引互踩——导入/更新前拒绝。 */
+function hasDuplicateIds(items) {
+    return new Set(items.map((x) => x.id)).size !== items.length;
+}
 function createKanbanRoute(ctx) {
     const app = new Hono();
+    // 看板 id 是 uuid 列：非 UUID 直接 404（同 guardProject 的理由——此前非 UUID
+    // 经 drizzle 查询在 PG 侧抛 22P02，Hono 兜成 500 且回显 SQL 错误细节）。
+    // 注册在 /deleted* 路由之前：Hono 按注册序分派，守卫必须先于具体处理器。
+    // '/deleted'（单段）不匹配本守卫模式，回收站列表/清空不受影响。
+    const guardBoardId = async (c, next) => {
+        if (!isUuid(c.req.param('boardId') ?? '')) {
+            return apiError(c, 404, 'BOARD_NOT_FOUND', '看板不存在或不在回收站');
+        }
+        await next();
+    };
+    app.use('/deleted/:boardId', guardBoardId);
+    app.use('/deleted/:boardId/*', guardBoardId);
     // GET /deleted — 回收站看板列表（项目删除软删除的看板，60 天保留期）。
     app.get('/deleted', async (c) => {
         return c.json({ boards: await listDeletedKanbanBoards(ctx.db) });
@@ -17,7 +56,10 @@ function createKanbanRoute(ctx) {
     // （目标无板时等价普通恢复）。此前 409 指引「删除目标看板」但产品无此能力，
     // 恢复是死胡同。
     app.post('/deleted/:boardId/restore', async (c) => {
-        const body = (await c.req.json().catch(() => ({})));
+        const parsed = await readJsonObject(c);
+        if (!parsed.ok)
+            return parsed.response;
+        const body = parsed.body;
         const boardId = c.req.param('boardId');
         const merge = body.merge === true;
         // 重建原项目：目录仍存在时重建项目记录并恢复到该目录（与会话存储同语义）。
@@ -92,6 +134,13 @@ function createKanbanRoute(ctx) {
     };
     app.use('/:projectId', guardProject);
     app.use('/:projectId/*', guardProject);
+    // 卡片 id 同为用户可控路径参数（uuid 列）：非 UUID → 404 而非 PG 22P02 → 500。
+    app.use('/:projectId/cards/:cardId', async (c, next) => {
+        if (!isUuid(c.req.param('cardId') ?? '')) {
+            return apiError(c, 404, 'CARD_NOT_FOUND', '卡片不存在');
+        }
+        await next();
+    });
     // GET /:projectId — full board with cards
     app.get('/:projectId', async (c) => {
         const projectId = c.req.param('projectId');
@@ -125,6 +174,18 @@ function createKanbanRoute(ctx) {
         if (body?.version !== 1 || !Array.isArray(body.columns) || !Array.isArray(body.cards)) {
             return apiError(c, 400, 'INVALID_EXPORT', '无效的看板导出 JSON：需要 version/columns/cards 字段');
         }
+        // 条目形状校验：columns 数组含 null/缺 name 此前直接读 col.id 击穿 500，
+        // 缺字段列被持久化后前端渲染/拖拽静默出错。
+        if (!body.columns.every(isValidColumnDef)) {
+            return apiError(c, 400, 'INVALID_EXPORT', 'columns 条目必须是 { id, name } 对象');
+        }
+        if (Array.isArray(body.labels) && !body.labels.every(isValidLabelDef)) {
+            return apiError(c, 400, 'INVALID_EXPORT', 'labels 条目必须是 { id, name, color } 对象');
+        }
+        const columns = body.columns;
+        if (hasDuplicateIds(columns)) {
+            return apiError(c, 400, 'INVALID_EXPORT', 'columns 包含重复的 id');
+        }
         // P3：导入体积上限——与会话导入（messages/archives 上限）同口径。
         // 无上限的数组会让 replaceBoard 的单事务逐条重建 OOM/长事务拖垮 PGLite。
         const MAX_IMPORT_COLUMNS = 100;
@@ -139,7 +200,6 @@ function createKanbanRoute(ctx) {
         if (body.cards.length > MAX_IMPORT_CARDS) {
             return apiError(c, 400, 'IMPORT_TOO_LARGE', `导入卡片数 ${body.cards.length} 超过上限 ${MAX_IMPORT_CARDS}`);
         }
-        const columns = body.columns;
         const labels = Array.isArray(body.labels) ? body.labels : [];
         // 宽松校验卡片：只保留结构完整的条目（导入宁少勿坏，与会话导入一致）。
         const cards = body.cards
@@ -156,7 +216,7 @@ function createKanbanRoute(ctx) {
             priority: (card.priority === 'high' || card.priority === 'medium' || card.priority === 'low'
                 ? card.priority
                 : 'medium'),
-            position: typeof card.position === 'number' ? card.position : 0,
+            position: typeof card.position === 'number' && Number.isFinite(card.position) ? card.position : 0,
             labels: Array.isArray(card.labels)
                 ? card.labels.filter((l) => typeof l === 'string')
                 : [],
@@ -174,7 +234,28 @@ function createKanbanRoute(ctx) {
     // PATCH /:projectId — update board columns/labels config
     app.patch('/:projectId', async (c) => {
         const projectId = c.req.param('projectId');
-        const body = await c.req.json().catch(() => ({}));
+        const parsed = await readJsonObject(c);
+        if (!parsed.ok)
+            return parsed.response;
+        const body = parsed.body;
+        // 形状校验：columns 非数组/条目含 null 此前在 store .map 处击穿 500；
+        // 缺 name 的列/标签被持久化后前端渲染静默出错。
+        if (body.columns !== undefined) {
+            if (!Array.isArray(body.columns) || !body.columns.every(isValidColumnDef)) {
+                return apiError(c, 400, 'INVALID_COLUMNS', 'columns 必须是 { id, name } 对象数组');
+            }
+            if (hasDuplicateIds(body.columns)) {
+                return apiError(c, 400, 'INVALID_COLUMNS', 'columns 包含重复的 id');
+            }
+        }
+        if (body.labels !== undefined) {
+            if (!Array.isArray(body.labels) || !body.labels.every(isValidLabelDef)) {
+                return apiError(c, 400, 'INVALID_LABELS', 'labels 必须是 { id, name, color } 对象数组');
+            }
+            if (hasDuplicateIds(body.labels)) {
+                return apiError(c, 400, 'INVALID_LABELS', 'labels 包含重复的 id');
+            }
+        }
         const store = createKanbanStore(ctx.db, projectId);
         try {
             const board = await store.updateBoard({
@@ -193,10 +274,30 @@ function createKanbanRoute(ctx) {
     // POST /:projectId/cards — add a card
     app.post('/:projectId/cards', async (c) => {
         const projectId = c.req.param('projectId');
-        const body = await c.req.json().catch(() => ({}));
-        const title = body.title?.trim();
+        const parsed = await readJsonObject(c);
+        if (!parsed.ok)
+            return parsed.response;
+        const body = parsed.body;
+        // 类型校验：数字 title 此前经 (as string).trim 击穿 500；labels 字符串/
+        // description 数字会被持久化成毒化卡片（jsonb 存非数组，前端 .map 崩溃）。
+        const title = typeof body.title === 'string' ? body.title.trim() : '';
         if (!title)
             return apiError(c, 400, 'INVALID_INPUT', 'title is required');
+        if (body.description !== undefined &&
+            body.description !== null &&
+            typeof body.description !== 'string') {
+            return apiError(c, 400, 'INVALID_INPUT', 'description must be a string or null');
+        }
+        if (body.columnId !== undefined && typeof body.columnId !== 'string') {
+            return apiError(c, 400, 'INVALID_INPUT', 'columnId must be a string');
+        }
+        if (body.priority !== undefined &&
+            (typeof body.priority !== 'string' || !VALID_PRIORITIES.has(body.priority))) {
+            return apiError(c, 400, 'INVALID_PRIORITY', 'priority must be high/medium/low');
+        }
+        if (body.labels !== undefined && !isValidStringArray(body.labels)) {
+            return apiError(c, 400, 'INVALID_LABELS', 'labels must be an array of strings');
+        }
         const store = createKanbanStore(ctx.db, projectId);
         try {
             const card = await store.addCard({
@@ -222,7 +323,30 @@ function createKanbanRoute(ctx) {
     app.patch('/:projectId/cards/:cardId', async (c) => {
         const projectId = c.req.param('projectId');
         const cardId = c.req.param('cardId');
-        const body = await c.req.json().catch(() => ({}));
+        const parsed = await readJsonObject(c);
+        if (!parsed.ok)
+            return parsed.response;
+        const body = parsed.body;
+        // 类型校验：数字 title/description/labels 字符串此前被 as 断言后直接
+        // update 进 DB（text/jsonb 类型失配 500 或持久化毒化字段）。
+        if (body.title !== undefined && (typeof body.title !== 'string' || body.title.trim() === '')) {
+            return apiError(c, 400, 'INVALID_INPUT', 'title must be a non-empty string');
+        }
+        if (body.description !== undefined &&
+            body.description !== null &&
+            typeof body.description !== 'string') {
+            return apiError(c, 400, 'INVALID_INPUT', 'description must be a string or null');
+        }
+        if (body.priority !== undefined &&
+            (typeof body.priority !== 'string' || !VALID_PRIORITIES.has(body.priority))) {
+            return apiError(c, 400, 'INVALID_PRIORITY', 'priority must be high/medium/low');
+        }
+        if (body.labels !== undefined && !isValidStringArray(body.labels)) {
+            return apiError(c, 400, 'INVALID_LABELS', 'labels must be an array of strings');
+        }
+        if (body.columnId !== undefined && typeof body.columnId !== 'string') {
+            return apiError(c, 400, 'INVALID_INPUT', 'columnId must be a string');
+        }
         const store = createKanbanStore(ctx.db, projectId);
         try {
             // columnId + position → moveCard; otherwise field update.
@@ -250,6 +374,9 @@ function createKanbanRoute(ctx) {
             if (err instanceof KanbanInvalidPriorityError) {
                 return apiError(c, 400, 'INVALID_PRIORITY', err.message);
             }
+            if (err instanceof KanbanInvalidPositionError) {
+                return apiError(c, 400, 'INVALID_POSITION', err.message);
+            }
             throw err;
         }
     });
@@ -258,7 +385,17 @@ function createKanbanRoute(ctx) {
         const projectId = c.req.param('projectId');
         const cardId = c.req.param('cardId');
         const store = createKanbanStore(ctx.db, projectId);
-        await store.deleteCard(cardId);
+        try {
+            // 作用域内删除：卡片不属于本项目板（他板 id / 已不存在）→ 404，
+            // 不再对任意 id 回 ok:true（删除失败静默无反馈）。
+            await store.deleteCard(cardId);
+        }
+        catch (err) {
+            if (err instanceof KanbanCardNotFoundError) {
+                return apiError(c, 404, 'CARD_NOT_FOUND', err.message);
+            }
+            throw err;
+        }
         return c.json({ ok: true });
     });
     return app;

package/dist/server/routes/permissions.js

@@ -2,6 +2,7 @@ import { eq } from 'drizzle-orm';
 import { Hono } from 'hono';
 import { sessions } from '../../db/schema.js';
 import { getSession } from '../../session/session.js';
+import { isUuid } from '../../shared/utils/string.js';
 import { apiError } from '../middleware/error.js';
 /** GET/PUT /api/permissions — 授权模式（default 逐个确认 / auto 自动放行 ask 工具）。
  *  - 根路径：全局默认模式（启动时取 config.permission.defaultMode；
@@ -66,6 +67,21 @@ async function persistSessionMetadata(ctx, sessionId, mutate) {
 }
 function createPermissionsRoute(ctx) {
     const app = new Hono();
+    // 路径参数 sessionId 是 sessions.id（uuid 列）：非 UUID 直接 404。
+    // 此前未守卫：PUT /:sessionId 的 getSession 把非 UUID 直接送进 drizzle
+    // 参数化查询，PG 侧抛 22P02（invalid input syntax for type uuid），Hono 兜成
+    // 500 并把 SQL 错误细节回给客户端——与 session/kanban/todo 路由已修的
+    // 同型缺陷。「格式非法」与「会话不存在」同属资源不存在，统一 404。
+    // /:sessionId/pending 只读内存 permissionStore（不触 DB），不受守卫约束。
+    const guardSessionId = async (c, next) => {
+        if (!isUuid(c.req.param('sessionId') ?? '')) {
+            return apiError(c, 404, 'NOT_FOUND', '会话不存在或已删除');
+        }
+        await next();
+    };
+    app.use('/:sessionId', guardSessionId);
+    app.use('/:sessionId/always-allow', guardSessionId);
+    app.use('/:sessionId/always-allow/*', guardSessionId);
     // GET / — 默认授权模式
     app.get('/', (c) => {
         return c.json({ mode: ctx.permissionMode });
@@ -95,6 +111,12 @@ function createPermissionsRoute(ctx) {
     // GET /:sessionId — 会话实际生效模式（覆盖优先，回退默认）+ 始终允许白名单
     app.get('/:sessionId', async (c) => {
         const sessionId = c.req.param('sessionId');
+        // 与 PUT /:sessionId 同口径：不存在/回收站会话 404——返回全局默认模式
+        // 会冒充「该会话的模式」，调用方无法区分「无覆盖」与「会话不存在」。
+        const session = await getSession(ctx.db, sessionId);
+        if (!session || session.deletedAt) {
+            return apiError(c, 404, 'NOT_FOUND', '会话不存在或已删除');
+        }
         await ensureSessionModeLoaded(ctx, sessionId);
         await ensureSessionAllowLoaded(ctx, sessionId);
         return c.json({
@@ -144,6 +166,12 @@ function createPermissionsRoute(ctx) {
     // POST /:sessionId/always-allow — 追加工具到会话白名单（幂等）
     app.post('/:sessionId/always-allow', async (c) => {
         const sessionId = c.req.param('sessionId');
+        // P3-11 同口径（此前只修了 PUT /:sessionId）：不存在的会话此前写入返回
+        // 200 且污染内存 Map——用户以为白名单已生效，实际会话已删除。
+        const session = await getSession(ctx.db, sessionId);
+        if (!session || session.deletedAt) {
+            return apiError(c, 404, 'NOT_FOUND', '会话不存在或已删除');
+        }
         const body = (await c.req.json().catch(() => null));
         const tool = body?.tool;
         if (typeof tool !== 'string' || tool.length === 0) {
@@ -162,6 +190,10 @@ function createPermissionsRoute(ctx) {
     // DELETE /:sessionId/always-allow/:tool — 从会话白名单移除
     app.delete('/:sessionId/always-allow/:tool', async (c) => {
         const sessionId = c.req.param('sessionId');
+        const session = await getSession(ctx.db, sessionId);
+        if (!session || session.deletedAt) {
+            return apiError(c, 404, 'NOT_FOUND', '会话不存在或已删除');
+        }
         const tool = c.req.param('tool');
         await ensureSessionAllowLoaded(ctx, sessionId);
         const list = (ctx.sessionAlwaysAllow.get(sessionId) ?? []).filter((t) => t !== tool);

package/dist/server/routes/project.js

@@ -9,6 +9,7 @@ import { fromDirectory, getByDirectory, getProject, listProjects, relocateProjec
 import { clearTrashMarks } from '../../session/session.js';
 import { generateId } from '../../shared/index.js';
 import { apiError } from '../middleware/error.js';
+import { readJsonObject } from '../util/json-body.js';
 import { expandPath } from './filesystem.js';
 /** realpath 失败（目录不存在等）时返回 null，调用方回退原路径。 */
 function realpathSafe(p) {
@@ -28,8 +29,12 @@ function createProjectRoute(ctx) {
     const app = new Hono();
     // 解析目录并创建/更新项目记录
     app.post('/from-directory', async (c) => {
-        const body = await c.req.json().catch(() => ({}));
-        const directory = body.directory;
+        const parsed = await readJsonObject(c);
+        if (!parsed.ok)
+            return parsed.response;
+        // directory 必须是非空字符串：数字/对象此前经 as 断言进 expandPath 的
+        // startsWith 抛 TypeError → 500。
+        const directory = typeof parsed.body.directory === 'string' ? parsed.body.directory : '';
         if (!directory)
             return apiError(c, 400, 'BAD_REQUEST', 'directory is required');
         // 展开 ~ 前缀为 home 绝对路径（前端可能传 ~/... 形式）
@@ -68,8 +73,10 @@ function createProjectRoute(ctx) {
     });
     // 更新项目名
     app.patch('/:id', async (c) => {
-        const body = await c.req.json().catch(() => ({}));
-        const name = body.name;
+        const parsed = await readJsonObject(c);
+        if (!parsed.ok)
+            return parsed.response;
+        const name = typeof parsed.body.name === 'string' ? parsed.body.name : '';
         if (!name)
             return apiError(c, 400, 'BAD_REQUEST', 'name is required');
         const project = await updateProjectName(ctx.db, c.req.param('id'), name);
@@ -90,8 +97,10 @@ function createProjectRoute(ctx) {
     // 目标目录已注册为另一项目 → 409；有活跃 run → 拒绝（agent 工作目录悬空）。
     app.post('/:id/relocate', async (c) => {
         const id = c.req.param('id');
-        const body = (await c.req.json().catch(() => ({})));
-        const directory = typeof body.directory === 'string' ? body.directory.trim() : '';
+        const parsed = await readJsonObject(c);
+        if (!parsed.ok)
+            return parsed.response;
+        const directory = typeof parsed.body.directory === 'string' ? parsed.body.directory.trim() : '';
         if (!directory)
             return apiError(c, 400, 'DIRECTORY_REQUIRED', 'directory is required');
         if (!existsSync(directory)) {

package/dist/server/routes/provider.js

@@ -1,10 +1,12 @@
 import { Hono } from 'hono';
 import { loadConfigScopes, mergeConfig } from '../../core/config.js';
+import { buildRegistryFromConfig } from '../../core/provider-registry.js';
 import { decryptSecret } from '../../core/secret.js';
 import { resolveRoute } from '../../llm/registry.js';
 import { getProject } from '../../project/project.js';
+import { headChars } from '../../shared/utils/string.js';
 import { apiError } from '../middleware/error.js';
-import { buildRegistryFromConfig } from '../registry-config.js';
+import { readJsonObject } from '../util/json-body.js';
 /**
  * 用给定 baseURL/apiKey 探测 OpenAI 兼容的 /models 端点。
  * baseURL 应自行包含版本路径（如 https://api.openai.com/v1）。
@@ -25,7 +27,7 @@ async function probeModels(baseURL, apiKey) {
         });
         if (!res.ok) {
             const text = await res.text().catch(() => '');
-            return { ok: false, error: `HTTP ${res.status} ${text.slice(0, 200) || res.statusText}` };
+            return { ok: false, error: `HTTP ${res.status} ${headChars(text, 200) || res.statusText}` };
         }
         const json = (await res.json());
         const models = Array.isArray(json.data)
@@ -96,10 +98,13 @@ function createProviderRoute(ctx) {
     });
     // 连接测试：用请求体里的凭据探测 /models，不污染 registry
     app.post('/test', async (c) => {
-        const body = await c.req.json().catch(() => ({}));
-        const { baseURL, apiKey } = body;
-        if (!baseURL)
+        const parsed = await readJsonObject(c);
+        if (!parsed.ok)
+            return parsed.response;
+        const { baseURL, apiKey } = parsed.body;
+        if (typeof baseURL !== 'string' || !baseURL) {
             return apiError(c, 400, 'BAD_REQUEST', 'baseURL is required');
+        }
         // P3：仅允许 http/https——服务端按调用方提供的 URL 发起请求，非 http 协议
         // 不应由「测试连接」端点代理（本机 localhost/Ollama 等 http 服务不受影响）。
         const scheme = /^([a-z][a-z0-9+.-]*):/i.exec(baseURL)?.[1]?.toLowerCase();