aizuchi 0.2.0 → 0.3.0

package/dist/server/app.js

@@ -1,21 +1,24 @@
 import { promises as fs, existsSync } from 'node:fs';
+import os from 'node:os';
 import path from 'node:path';
 import { Hono } from 'hono';
 import { bodyLimit } from 'hono/body-limit';
 import { HTTPException } from 'hono/http-exception';
 import { streamSSE } from 'hono/streaming';
-import { commentInputSchema, configSchema, fileSaveInputSchema, memberRenameInputSchema, renameInputSchema, findMemberViolations, moveInputSchema, newTaskInputSchema, NEW_TASK_BODY, rejectInputSchema, saveInputSchema, summarizeZodError, taskFrontmatterSchema, taskIdSchema, } from '../shared/schema.js';
+import { commentInputSchema, configSchema, fileSaveInputSchema, memberRenameInputSchema, renameInputSchema, findMemberViolations, moveInputSchema, newTaskInputSchema, NEW_TASK_BODY, rejectInputSchema, restoreInputSchema, reviseInputSchema, saveInputSchema, summarizeZodError, taskFrontmatterSchema, taskIdSchema, } from '../shared/schema.js';
 import { loadArchive, loadBoard, loadConfig, loadTaskView } from './board.js';
 import { createEventBus } from './events.js';
 import { allocateNextId, createSerialQueue } from './lib/id.js';
-import { assertDirInsideKanban, FileExistsError, PathValidationError, readKanbanFile, resolveKanbanReal, safeWriteFile, statKanbanFile, } from './lib/fs-safe.js';
+import { assertDirInsideKanban, collectKanbanFilesNoSymlink, FileExistsError, PathValidationError, readKanbanFile, resolveKanbanReal, safeWriteFile, statKanbanFile, } from './lib/fs-safe.js';
 import { buildDocsTree, readDoc } from './lib/docs.js';
 import { findCommitsForTask } from './lib/git.js';
-import { buildCatalog, buildFileTree, fileMtime, readConfigFile, saveConfigFile } from './lib/files.js';
+import { buildCatalog, buildFileTree, buildGlobalCatalog, fileMtime, globalFileMtime, readConfigFile, readGlobalConfigFile, saveConfigFile, saveGlobalConfigFile, } from './lib/files.js';
 import { applyPinpointPatch, renameTaskId, replaceDependsOnId, replaceMemberKeys, verifyMemberRename, verifyPinpointResult, verifyRename, } from './lib/pinpoint.js';
-import { appendCommentLine, appendWorkLogLine, replaceBody } from './lib/proposal.js';
+import { appendCommentLine, appendWorkLogLine, replaceBody, reviseWorkLogText } from './lib/proposal.js';
+import { REVISABLE_STATUSES } from './lib/task-ops.js';
 import { parseTaskFile, serializeTaskFile, TaskFileParseError } from './lib/task-file.js';
 import { archiveMonth, nowIso, workLogTimestamp } from './lib/time.js';
+import { DEFAULT_TRASH_KEEP, listTrash, snapshotToTrash } from './lib/trash.js';
 /**
  * Hono アプリ (REST API + SSE)。要件 §6.7 のうち Phase 1 スコープの
  * board / tasks / events のみを実装する (アーカイブ・承認・却下 API は Phase 2)。
@@ -125,6 +128,54 @@ export function createApp(options) {
         await fs.rm(path.join(tasksDir, `${id}.md`), { force: true });
         return path.relative(projectRoot, archivePath);
     };
+    /**
+     * archive/**\/<id>.md を tasks/<id>.md へ復元する (moveToArchive の対称・unarchive)。
+     * 「復元先 (tasks/) を排他作成 → 成功後に archive 元を rm」順序でタスク消失を防ぐ (archive と対称)。
+     * archive 配下に id 一致が無ければ null を返す (呼び出し側で 404)。tasks/ に既存があれば
+     * safeWriteFile の exclusive が FileExistsError を投げる (呼び出し側で 409)。
+     *
+     * パス安全 (N-2): archive ルートにも resolveKanbanReal で .kanban の realpath 信頼境界を確立し、
+     * collectKanbanFilesNoSymlink (realpath containment + symlink 非追従) で探す。読出は readKanbanFile、
+     * 書込は safeWriteFile (許可リスト = .kanban) で、symlink / 中間 dir symlink 経由の外部読書を遮断する。
+     */
+    const restoreFromArchive = async (id) => {
+        const kanbanReal = await getKanbanReal(); // .kanban 自体が symlink なら throw
+        const archiveDir = path.join(kanbanReal, 'archive');
+        // archive 配下を symlink 非追従 + realpath containment で走査し、id 一致 (<id>.md) を探す
+        const files = await collectKanbanFilesNoSymlink(archiveDir, kanbanReal);
+        const archivePath = files.find((abs) => path.basename(abs).slice(0, -3) === id);
+        if (archivePath === undefined)
+            return false; // 不在 → 404
+        const content = await readKanbanFile(archivePath, kanbanReal); // 中間 dir / 最終 symlink を遮断
+        const destPath = path.join(tasksDir, `${id}.md`);
+        // 復元先を排他作成 (既存があれば FileExistsError → 409)。成功後に archive 元を消す (消失防止)
+        await safeWriteFile(destPath, content, [kanbanDir], { exclusive: true });
+        await fs.rm(archivePath, { force: true });
+        return true;
+    };
+    /**
+     * 破壊的書込 (PATCH/PUT/approve/revise/rename/member rename/comment) の直前に、現内容を
+     * `.kanban/.trash/<id>.<epoch_ms>.md` へコピー退避する安全網 (ごみ箱)。誤上書き / 誤操作からの復元用。
+     *
+     * - **ベストエフォート**: 退避失敗 (try/catch) は本処理を止めない。退避は元を消さない「コピー」なので、
+     *   退避が出来なくても従来どおり書込は進む (機能追加で既存挙動を壊さない)。
+     * - **comment / log 追記も退避する (SEC-N3・2026-06-16)**: append 系も実装は read→全文上書きのため、
+     *   外部編集と交錯して上書きで失われた場合に復元できるよう、他の破壊的書込と同様に直前退避する。
+     * - trashKeep は config 由来 (既定 DEFAULT_TRASH_KEEP)。世代刈り取りは snapshotToTrash 内で行う。
+     * - id は退避ファイル名 (basename) に使う。連鎖更新 (rename の readdir 由来 name.slice) や member rename
+     *   経由では taskIdSchema 未検証の値が渡るが、退避先 basename は safeWriteFile が SAFE_BASENAME_RE +
+     *   containment で再検証するためパス安全 (検証済み前提に依存しない)。
+     */
+    const snapshot = async (id, currentContent, kanbanReal) => {
+        try {
+            const { config } = await loadConfig(projectRoot);
+            const keep = config.trashKeep ?? DEFAULT_TRASH_KEEP;
+            await snapshotToTrash(id, currentContent, kanbanDir, kanbanReal, keep);
+        }
+        catch {
+            // 退避はベストエフォート安全網。失敗しても本処理 (破壊的書込) は止めない
+        }
+    };
     // セキュリティミドルウェア (CSP / Host / CSRF / bodyLimit) は overview と共通 (R14-5)
     applySecurityMiddleware(app);
     app.onError((error, c) => {
@@ -284,6 +335,7 @@ export function createApp(options) {
             if (!check.success) {
                 return c.json({ error: 'ステータス変更後のタスクが不正', issues: summarizeZodError(check.error) }, 400);
             }
+            await snapshot(idResult.data, raw, await getKanbanReal()); // 旧版を .trash へ退避 (ベストエフォート)
             await safeWriteFile(filePath, patched, [kanbanDir]);
             return c.json(await loadTaskView(projectRoot, filePath));
         });
@@ -335,6 +387,7 @@ export function createApp(options) {
             if (check.data.id !== idResult.data) {
                 return c.json({ error: `frontmatter の id (${check.data.id}) は変更できない` }, 400);
             }
+            await snapshot(idResult.data, serverContent, await getKanbanReal()); // 旧版を .trash へ退避 (ベストエフォート)
             await safeWriteFile(filePath, input.data.content, [kanbanDir]);
             return c.json(await loadTaskView(projectRoot, filePath));
         });
@@ -393,6 +446,7 @@ export function createApp(options) {
             // モード違反検知 (board.ts) の対象から外れる
             const logLine = `${workLogTimestamp()} [approve] 承認: proposed → todo`;
             const finalRaw = replaceBody(patched, appendWorkLogLine(patchedParsed.body, logLine));
+            await snapshot(idResult.data, raw, await getKanbanReal()); // 旧版を .trash へ退避 (ベストエフォート)
             await safeWriteFile(filePath, finalRaw, [kanbanDir]);
             return c.json(await loadTaskView(projectRoot, filePath));
         });
@@ -489,6 +543,237 @@ export function createApp(options) {
         });
     });
     // ---------------------------------------------------------------------
+    // POST /api/tasks/:id/unarchive — archive のタスクを board (tasks/) へ復元 (moveToArchive の対称)
+    // archive/**/<id>.md を探して tasks/<id>.md へ排他作成し、成功後に archive 元を rm する (消失防止)。
+    // 不在は 404 / 復元先 (tasks/) 衝突は 409。本文は変更せずそのまま戻す (所在のみ board へ移す)。
+    // ---------------------------------------------------------------------
+    app.post('/api/tasks/:id/unarchive', async (c) => {
+        const idResult = taskIdSchema.safeParse(c.req.param('id'));
+        if (!idResult.success) {
+            return c.json({ error: '不正なタスク ID' }, 400);
+        }
+        return enqueue(async () => {
+            try {
+                const found = await restoreFromArchive(idResult.data);
+                if (!found) {
+                    return c.json({ error: `archive にタスクが見つからない: ${idResult.data}` }, 404);
+                }
+                // 復元 (ファイル移動) は完了済み。応答用の loadTaskView は壊れた frontmatter で throw しうるが、
+                // その場合でも復元自体は成功しているので 500 にせず軽量応答を返す (board は lenient parse で警告表示)。
+                try {
+                    return c.json(await loadTaskView(projectRoot, path.join(tasksDir, `${idResult.data}.md`)));
+                }
+                catch {
+                    return c.json({ id: idResult.data, restored: true });
+                }
+            }
+            catch (error) {
+                if (error instanceof FileExistsError) {
+                    return c.json({ error: `復元先に同じ ID が既に存在します: ${idResult.data}` }, 409);
+                }
+                throw error;
+            }
+        });
+    });
+    // ---------------------------------------------------------------------
+    // GET /api/trash — ごみ箱の退避一覧 (read-only)。破壊的書込の直前に .trash へコピー退避した旧版を
+    // 軽量メタ (id / trashedAt / title / status) + プレビュー用本文で返す。新しい退避が先頭。
+    // N-2: .trash も resolveKanbanReal + collectKanbanFilesNoSymlink + readKanbanFile (containment + symlink 非追従)。
+    // .kanban が symlink / .trash 不在は空一覧に倒す (graceful)。
+    // ---------------------------------------------------------------------
+    app.get('/api/trash', async (c) => {
+        let kanbanReal;
+        try {
+            kanbanReal = await getKanbanReal(); // .kanban 自体が symlink / 不在なら throw → 空一覧
+        }
+        catch {
+            return c.json({ items: [] });
+        }
+        const entries = await listTrash(kanbanDir, kanbanReal);
+        // 新しい退避が先頭 (trashedAt 降順)。同 epoch は id で安定化
+        entries.sort((a, b) => b.trashedAt - a.trashedAt || a.id.localeCompare(b.id));
+        const items = [];
+        for (const entry of entries) {
+            let raw;
+            try {
+                raw = await readKanbanFile(entry.absPath, kanbanReal); // containment + symlink 非追従
+            }
+            catch {
+                continue; // 走査と読出の間に消えた / symlink はスキップ
+            }
+            // frontmatter が parse できれば title / status を出す。壊れていても本文は出せる限り出す
+            let title = null;
+            let status = null;
+            let body = raw;
+            try {
+                const parsed = parseTaskFile(raw);
+                body = parsed.body;
+                const fm = taskFrontmatterSchema.safeParse(parsed.data);
+                if (fm.success) {
+                    title = fm.data.title;
+                    status = fm.data.status;
+                }
+            }
+            catch {
+                // parse 不能 (YAML 構文エラー等) は title/status=null・body=全文
+            }
+            items.push({ id: entry.id, trashedAt: entry.trashedAt, title, status, body });
+        }
+        return c.json({ items });
+    });
+    // ---------------------------------------------------------------------
+    // POST /api/tasks/:id/restore — ごみ箱の退避版を tasks/<id>.md へ書き戻す (誤上書きからの復元)。
+    // body.trashedAt 指定でその世代、省略時は当該 id の最新世代を復元する。
+    // tasks/<id>.md が既存のとき: overwrite=true なら現行を .trash へ退避してから上書き復元、
+    // overwrite 未指定 / false なら 409 (上書きしない・unarchive と対称)。退避が無ければ 404。
+    // N-2: .trash 走査 (collectKanbanFilesNoSymlink) + 読出 (readKanbanFile) + 書込 (safeWriteFile)。
+    // 復元は退避版を消さない (コピー・世代として残す。本処理が再び snapshot を積むのとも整合)。
+    // ---------------------------------------------------------------------
+    app.post('/api/tasks/:id/restore', async (c) => {
+        const idResult = taskIdSchema.safeParse(c.req.param('id'));
+        if (!idResult.success) {
+            return c.json({ error: '不正なタスク ID' }, 400);
+        }
+        const input = restoreInputSchema.safeParse(await c.req.json().catch(() => ({})));
+        if (!input.success) {
+            return c.json({ error: '入力が不正', issues: summarizeZodError(input.error) }, 400);
+        }
+        const id = idResult.data;
+        return enqueue(async () => {
+            const kanbanReal = await getKanbanReal(); // .kanban 自体が symlink なら throw → onError で 400
+            // 当該 id の退避世代を集め、指定 (trashedAt) or 最新 (epoch 最大) を選ぶ
+            const generations = (await listTrash(kanbanDir, kanbanReal, id)).sort((a, b) => b.trashedAt - a.trashedAt);
+            const target = input.data.trashedAt !== undefined
+                ? generations.find((g) => g.trashedAt === input.data.trashedAt)
+                : generations[0];
+            if (target === undefined) {
+                return c.json({ error: `ごみ箱に退避が見つからない: ${id}` }, 404);
+            }
+            let content;
+            try {
+                content = await readKanbanFile(target.absPath, kanbanReal); // containment + symlink 非追従
+            }
+            catch {
+                return c.json({ error: `退避を読み込めない: ${id}` }, 404);
+            }
+            const destPath = path.join(tasksDir, `${id}.md`);
+            if (input.data.overwrite === true) {
+                // 上書き復元: 現行があれば破壊的書込なので、まず現行を .trash へ退避してから非排他で上書き。
+                // 退避版は消さず世代として残す (コピー復元) + 現行も新たな世代として退避され取り戻せる。
+                let current = null;
+                try {
+                    current = await readKanbanFile(destPath, kanbanReal);
+                }
+                catch {
+                    current = null; // 現行が無い / 読めない (壊れ) は退避不要でそのまま上書き復元
+                }
+                if (current !== null) {
+                    await snapshot(id, current, kanbanReal); // 現行を .trash へ退避 (ベストエフォート)
+                }
+                await safeWriteFile(destPath, content, [kanbanDir]);
+            }
+            else {
+                try {
+                    // 復元先を排他作成 (既存があれば 409)。退避版は消さず世代として残す (コピー復元)
+                    await safeWriteFile(destPath, content, [kanbanDir], { exclusive: true });
+                }
+                catch (error) {
+                    if (error instanceof FileExistsError) {
+                        return c.json({ error: `復元先に同じ ID が既に存在します: ${id}` }, 409);
+                    }
+                    throw error;
+                }
+            }
+            // 復元自体は完了済み。応答用 loadTaskView は壊れた frontmatter で throw しうるので軽量応答に倒す
+            try {
+                return c.json(await loadTaskView(projectRoot, destPath));
+            }
+            catch {
+                return c.json({ id, restored: true });
+            }
+        });
+    });
+    // ---------------------------------------------------------------------
+    // POST /api/tasks/:id/revise — やり直し依頼 (差し戻し)。todo/in_progress/pending/done → todo へ戻し、
+    // 理由を作業ログに残す。pending から戻す場合は blocked_* を null へ (reject の blocked クリア相当)。
+    // 既に todo の場合も理由ログだけは追記する (status 変更なしでも差し戻し意図を残せる)。
+    // proposed / backlog は対象外 (approve 承認フロー迂回防止のため 409。REVISABLE_STATUSES 参照)。
+    // ---------------------------------------------------------------------
+    app.post('/api/tasks/:id/revise', async (c) => {
+        const idResult = taskIdSchema.safeParse(c.req.param('id'));
+        if (!idResult.success) {
+            return c.json({ error: '不正なタスク ID' }, 400);
+        }
+        const input = reviseInputSchema.safeParse(await c.req.json().catch(() => null));
+        if (!input.success) {
+            return c.json({ error: '入力が不正', issues: summarizeZodError(input.error) }, 400);
+        }
+        const filePath = path.join(tasksDir, `${idResult.data}.md`);
+        return enqueue(async () => {
+            let stat;
+            let raw;
+            try {
+                // R17-D: realpath containment + symlink 非追従で stat/read する (symlink/中間dirは 404)。
+                // mtime 競合検知のため stat も取得する (PUT /api/tasks と同方式)
+                const kanbanReal = await getKanbanReal();
+                stat = await statKanbanFile(filePath, kanbanReal);
+                raw = await readKanbanFile(filePath, kanbanReal);
+            }
+            catch {
+                return c.json({ error: `タスクが見つからない: ${idResult.data}` }, 404);
+            }
+            // baseMtimeMs があれば mtime 厳密比較 (N-3、PATCH/PUT と同じく外部変更を 409 で弾く)
+            if (input.data.baseMtimeMs !== undefined && stat.mtimeMs !== input.data.baseMtimeMs) {
+                return c.json({ error: '外部で変更されています', serverContent: raw, serverMtimeMs: stat.mtimeMs }, 409);
+            }
+            let current;
+            try {
+                current = parseTaskFile(raw);
+            }
+            catch (error) {
+                const message = error instanceof TaskFileParseError ? error.message : 'タスクファイルを解析できない';
+                return c.json({ error: `差し戻しできない (${message})` }, 409);
+            }
+            // 源 status ガード: 差し戻しは「進行中・保留・完了・既に todo」のタスクのみを対象にする。
+            // proposed / backlog から todo へ直接行けると approve 承認フロー (proposed→todo) を迂回し、
+            // 恒久 mode_violation を生む。approve/reject が status==='proposed' を必須化するのと対称。
+            if (!REVISABLE_STATUSES.has(current.data.status)) {
+                return c.json({ error: 'この状態のタスクは差し戻せません' }, 409);
+            }
+            // status を todo へ行置換 (本文・他フィールドは保全)。pending から戻す場合は
+            // blocked_* も null へ戻す (reject ハンドラの blocked クリア相当)。verify-after-patch を通す
+            const patch = { status: 'todo', updated: nowIso() };
+            if (current.data.blocked_type != null || current.data.blocked_reason != null) {
+                patch.blocked_type = null;
+                patch.blocked_reason = null;
+            }
+            let patched;
+            let patchedParsed;
+            try {
+                patched = applyPinpointPatch(raw, patch);
+                patchedParsed = parseTaskFile(patched);
+            }
+            catch (error) {
+                const message = error instanceof TaskFileParseError ? error.message : error instanceof Error ? error.message : '差し戻しに失敗';
+                return c.json({ error: `差し戻しできない (${message})` }, 409);
+            }
+            const drift = verifyPinpointResult(current, patchedParsed, patch);
+            if (drift !== null) {
+                return c.json({ error: `差し戻しできない (${drift})` }, 409);
+            }
+            const check = taskFrontmatterSchema.safeParse(patchedParsed.data);
+            if (!check.success) {
+                return c.json({ error: '差し戻し後のタスクが不正', issues: summarizeZodError(check.error) }, 400);
+            }
+            // 差し戻し理由を作業ログに残す (frontmatter は pinpoint 済みなので body だけ更新する。approve と同型)
+            const logLine = `${workLogTimestamp()} [human] ${reviseWorkLogText(input.data.reason)}`;
+            const finalRaw = replaceBody(patched, appendWorkLogLine(patchedParsed.body, logLine));
+            await snapshot(idResult.data, raw, await getKanbanReal()); // 旧版を .trash へ退避 (ベストエフォート)
+            await safeWriteFile(filePath, finalRaw, [kanbanDir]);
+            return c.json(await loadTaskView(projectRoot, filePath));
+        });
+    });
+    // ---------------------------------------------------------------------
     // POST /api/tasks/:id/rename — 重複 ID を空き番号へリネーム (F-6-5)
     // ---------------------------------------------------------------------
     app.post('/api/tasks/:id/rename', async (c) => {
@@ -571,6 +856,9 @@ export function createApp(options) {
                 }
                 const updated = replaceDependsOnId(content, oldId, newId);
                 if (updated !== content) {
+                    // 連鎖更新は他タスクの depends_on を破壊的に書き換えるので、その旧版も退避する
+                    // (旧 id のリネーム自体は newPath に同内容が残るため退避不要・audit 参照)
+                    await snapshot(name.slice(0, -3), content, chainKanbanReal);
                     await safeWriteFile(p, updated, [kanbanDir]);
                 }
             }
@@ -644,6 +932,7 @@ export function createApp(options) {
                 // 書込失敗 (fs エラー等) も全体を中断させず skipped に積んで続行する —
                 // 途中失敗で renamed/skipped の進捗が失われると部分適用の把握ができなくなる
                 try {
+                    await snapshot(name.slice(0, -3), content, kanbanReal); // 旧版 (created_by/assignee 改変前) を .trash へ退避
                     await safeWriteFile(p, patched, [kanbanDir]);
                     renamed += 1;
                 }
@@ -747,8 +1036,12 @@ export function createApp(options) {
             return c.json({ error: 'path クエリが必要です' }, 400);
         }
         const { config } = await loadConfig(projectRoot);
+        const isGlobal = c.req.query('scope') === 'global';
         try {
-            return c.json(await readConfigFile(projectRoot, config, relPath));
+            const result = isGlobal
+                ? await readGlobalConfigFile(os.homedir(), relPath)
+                : await readConfigFile(projectRoot, config, relPath);
+            return c.json(result);
         }
         catch (error) {
             if (error instanceof PathValidationError)
@@ -765,15 +1058,20 @@ export function createApp(options) {
             return c.json({ error: '入力が不正', issues: summarizeZodError(input.error) }, 400);
         }
         const { config } = await loadConfig(projectRoot);
+        const isGlobal = input.data.scope === 'global';
         // 直列化キューで mtime 検査 → 保存 (プロセス内 write-write 競合を閉じる)
         return enqueue(async () => {
             try {
-                const current = await fileMtime(projectRoot, config, input.data.path);
+                const current = isGlobal
+                    ? await globalFileMtime(os.homedir(), input.data.path)
+                    : await fileMtime(projectRoot, config, input.data.path);
                 // 既存ファイルの場合のみ mtime 厳密比較 (新規作成は current=null で素通り)
                 if (current !== null && current !== input.data.baseMtimeMs) {
                     return c.json({ error: '外部で変更されています', serverMtimeMs: current }, 409);
                 }
-                const saved = await saveConfigFile(projectRoot, config, input.data.path, input.data.content);
+                const saved = isGlobal
+                    ? await saveGlobalConfigFile(os.homedir(), input.data.path, input.data.content)
+                    : await saveConfigFile(projectRoot, config, input.data.path, input.data.content);
                 return c.json(saved);
             }
             catch (error) {
@@ -788,8 +1086,11 @@ export function createApp(options) {
         const { config } = await loadConfig(projectRoot);
         return c.json({ trees: await buildFileTree(projectRoot, config) });
     });
-    // GET /api/files/catalog — skill / agent の frontmatter カタログ (F-4-4)
+    // GET /api/files/catalog — skill / agent の frontmatter カタログ (F-4-4)。scope=global で ~/.claude/skills,agents
     app.get('/api/files/catalog', async (c) => {
+        if (c.req.query('scope') === 'global') {
+            return c.json({ items: await buildGlobalCatalog(os.homedir()) });
+        }
         const { config } = await loadConfig(projectRoot);
         return c.json({ items: await buildCatalog(projectRoot, config) });
     });
@@ -851,9 +1152,10 @@ export function createApp(options) {
         }
         const filePath = path.join(tasksDir, `${idResult.data}.md`);
         return enqueue(async () => {
+            const kanbanReal = await getKanbanReal();
             let raw;
             try {
-                raw = await readKanbanFile(filePath, await getKanbanReal()); // R17-D: containment + symlink 非追従 (symlink/中間dirは 404)
+                raw = await readKanbanFile(filePath, kanbanReal); // R17-D: containment + symlink 非追従 (symlink/中間dirは 404)
             }
             catch {
                 return c.json({ error: `タスクが見つからない: ${idResult.data}` }, 404);
@@ -869,6 +1171,9 @@ export function createApp(options) {
             // 本文の ## やりとり セクションに `- 時刻 [member] msg` を追記する (frontmatter はバイト保全)
             const line = `${workLogTimestamp()} [${input.data.member}] ${input.data.message}`;
             const newBody = appendCommentLine(current.body, line);
+            // SEC-N3: コメント追記も read→全文上書きのため、上書き直前に現内容を .trash へ退避する
+            // (他の破壊的書込経路と同様)。外部編集と交錯して失われても復元できるようにする (ベストエフォート)。
+            await snapshot(idResult.data, raw, kanbanReal);
             await safeWriteFile(filePath, replaceBody(raw, newBody), [kanbanDir]);
             return c.json(await loadTaskView(projectRoot, filePath));
         });

package/dist/server/board.js

@@ -63,10 +63,19 @@ async function collectArchiveIds(kanbanDir, kanbanReal) {
     }
     return ids;
 }
+/** archive/<...>/<id>.md の絶対パス + kanbanReal から所在月 <YYYY-MM> を抽出する (無ければ null) */
+function extractArchiveMonth(abs, kanbanReal) {
+    const archiveRoot = path.join(kanbanReal, 'archive');
+    const rel = path.relative(archiveRoot, abs); // 例: 2026-06/TASK-001.md
+    const first = rel.split(path.sep)[0];
+    return /^\d{4}-\d{2}$/.test(first) ? first : null;
+}
 /**
- * archive/** の全タスクの本文を読む (日次ダイジェストが作業ログを集計するため)。
- * frontmatter 検証はせず本文を取り出す (アーカイブは所在で表現され様々な status がありうる)。
- * パース不能なファイルはスキップする。
+ * archive/** の全タスクを読む。`id` / `body` は常に返し (後方互換)、frontmatter が parse でき検証を
+ * 通れば表示用フィールド (title / status / assignee / labels / updated) と所在月 month を追加する。
+ * frontmatter が壊れていても本文は取り出せる限り返す (id + body + month のみ・title 等は欠落許容)。
+ * 本文も parse できない (YAML 構文エラー等) ファイルや symlink はスキップする。
+ * 検証はせず所在で表現する点は従来通り (アーカイブは様々な status がありうる)。
  */
 export async function loadArchive(projectRoot) {
     // R17-D: .kanban の realpath 信頼境界を確立 (.kanban 自体が symlink / 不在なら [] に倒す)
@@ -84,12 +93,33 @@ export async function loadArchive(projectRoot) {
     const files = await collectKanbanFilesNoSymlink(archiveDir, kanbanReal);
     const result = [];
     for (const abs of files) {
+        let parsed;
         try {
             const raw = await readKanbanFile(abs, kanbanReal);
-            result.push({ id: path.basename(abs).slice(0, -3), body: parseTaskFile(raw).body });
+            parsed = parseTaskFile(raw);
         }
         catch {
-            // パース不能 (壊れた archive) / symlink はスキップ
+            // 本文すら取り出せない (YAML 構文エラー等) / symlink はスキップ
+            continue;
+        }
+        const id = path.basename(abs).slice(0, -3);
+        const month = extractArchiveMonth(abs, kanbanReal);
+        // frontmatter が検証を通れば表示用フィールドを付ける。通らなければ id + body のみ (壊れタスク許容)
+        const fm = taskFrontmatterSchema.safeParse(parsed.data);
+        if (fm.success) {
+            result.push({
+                id,
+                body: parsed.body,
+                month,
+                title: fm.data.title,
+                status: fm.data.status,
+                assignee: fm.data.assignee,
+                labels: fm.data.labels,
+                updated: fm.data.updated,
+            });
+        }
+        else {
+            result.push({ id, body: parsed.body, month });
         }
     }
     return result;

package/dist/server/lib/config-paths.js

@@ -1,19 +1,49 @@
 import { promises as fs } from 'node:fs';
 import path from 'node:path';
 import { hasSafeConfigExt, isHardDeniedConfigPath, isSecretPath, isStrictlyInside, PathValidationError, sanitizeConfigDirEntries, } from './fs-safe.js';
-/** 要求された相対パスが aiConfigPaths のどの許可に該当するかを判定し、正規化済み絶対パスを返す */
-export async function resolveConfigPath(projectRoot, config, requestedRelPath) {
-    const root = path.resolve(projectRoot);
-    // 要求パスを正規化し、プロジェクトルート配下であることを確認する (絶対パス / ../ を弾く)
+/**
+ * target の祖先のうち realpath できる最も近いものと、そこからの未作成 segment 列を返す。
+ * 未作成 segment は実体が無く symlink になり得ないため、containment 判定は実在祖先の realpath だけで足りる。
+ * これにより「許可リスト内だが親ディレクトリが未作成」のパスを、403 (PathValidationError) ではなく
+ * 呼び出し側の stat ENOENT → 404 として扱える (symlink トラバーサルは実在祖先の realpath で従来どおり検知)。
+ */
+async function resolveNearestExistingAncestor(target) {
+    const missing = [];
+    let cur = target;
+    for (;;) {
+        try {
+            return { real: await fs.realpath(cur), missing };
+        }
+        catch (error) {
+            if (error.code !== 'ENOENT')
+                throw error;
+            const parent = path.dirname(cur);
+            if (parent === cur)
+                throw new PathValidationError('親ディレクトリが解決できません');
+            missing.unshift(path.basename(cur));
+            cur = parent;
+        }
+    }
+}
+/** 既定のグローバル browseDirs (~ からの相対)。グローバル skill/agent の閲覧・編集対象 (N-2 をホーム配下へ限定拡張) */
+export const GLOBAL_BROWSE_DIRS = ['.claude/skills', '.claude/agents'];
+/**
+ * root を基準に「完全一致ファイル + 接頭辞一致 browseDirs」の許可リスト検証を行う共通コア (N-2 の本丸)。
+ * プロジェクト (resolveConfigPath) とグローバル (resolveGlobalConfigPath) が同一のハードン済みロジックを共有する。
+ * 正規化 → root 配下か → 許可リスト該当か → deny-list (2 層) → 親 realpath containment → 対象 symlink 拒否。
+ */
+async function resolveAllowlistedPath(rootInput, exactFilesRel, browseDirsRel, requestedRelPath) {
+    const root = path.resolve(rootInput);
+    // 要求パスを正規化し、root 配下であることを確認する (絶対パス / ../ を弾く)
     const resolved = path.resolve(root, requestedRelPath);
     if (resolved !== root && !resolved.startsWith(root + path.sep)) {
-        throw new PathValidationError('プロジェクトルート外へのアクセスは許可されていません');
+        throw new PathValidationError('許可範囲外へのアクセスは許可されていません');
     }
-    // 許可リスト (config 由来) を正規化する。各エントリが root の真の配下であることを強制し
+    // 許可リストを正規化する。各エントリが root の真の配下であることを強制し
     // (`.` / `..` / 絶対パスで root 全域を許可リストに載せる攻撃を無効化)。deny-list はここでは効かせず、
     // 解決後パスに対して下のゲートで 2 層適用する (R14-1 再修正)。
-    const exactFiles = new Set(sanitizeConfigDirEntries(root, [...config.aiConfigPaths.settingsFiles, ...config.aiConfigPaths.rootFiles]));
-    const browseDirs = sanitizeConfigDirEntries(root, config.aiConfigPaths.browseDirs);
+    const exactFiles = new Set(sanitizeConfigDirEntries(root, exactFilesRel));
+    const browseDirs = sanitizeConfigDirEntries(root, browseDirsRel);
     let kind = null;
     if (exactFiles.has(resolved)) {
         kind = 'file';
@@ -22,24 +52,23 @@ export async function resolveConfigPath(projectRoot, config, requestedRelPath) {
         kind = 'dir';
     }
     if (kind === null) {
-        throw new PathValidationError('許可リスト (aiConfigPaths) 外へのアクセスは許可されていません');
+        throw new PathValidationError('許可リスト外へのアクセスは許可されていません');
     }
     // deny-list を解決後パス (lexical) に適用する (R14-1 再修正・残存脆弱性 #2/#4/#7)。
     // 強い deny (.git/** + 鍵類) は **完全一致 (exactFiles) でも** 遮断する (悪意 config が settingsFiles に
     // `.git/hooks/pre-commit` / `.env` / `id_rsa` を列挙しても read/write させない = RCE/N-8 防止)。
     assertNotDenied(resolved, kind);
-    // 親ディレクトリの realpath を解決し、symlink 経由で許可範囲の外へ出ていないか確認する
-    let parentReal;
-    try {
-        parentReal = await fs.realpath(path.dirname(resolved));
-    }
-    catch {
-        throw new PathValidationError('親ディレクトリが存在しません');
-    }
+    // 親ディレクトリの realpath を解決し、symlink 経由で許可範囲の外へ出ていないか確認する。
+    // 親 (や中間) が未作成なら「最も近い実在祖先」で containment を判定する: 実在しない segment は
+    // symlink になり得ないので安全で、かつ未作成ファイルを 403 でなく 404 (呼び出し側の stat ENOENT) に
+    // できる (許可リスト内なのに親未作成だけで 403 を返していた不整合の解消)。
     const rootReal = await fs.realpath(root).catch(() => root);
-    if (parentReal !== rootReal && !parentReal.startsWith(rootReal + path.sep)) {
-        throw new PathValidationError('symlink 経由でプロジェクトルート外を指しています');
+    const { real: ancestorReal, missing } = await resolveNearestExistingAncestor(path.dirname(resolved));
+    if (ancestorReal !== rootReal && !ancestorReal.startsWith(rootReal + path.sep)) {
+        throw new PathValidationError('symlink 経由で許可範囲外を指しています');
     }
+    // 親 canonical = 実在祖先 realpath + 未作成 segment (未作成部分は symlink 不可なので lexical 連結で安全)
+    const parentReal = missing.length > 0 ? path.join(ancestorReal, ...missing) : ancestorReal;
     // 接頭辞一致 (browseDirs) の場合、realpath 解決後も許可ディレクトリ配下か再確認する
     if (kind === 'dir') {
         const allowedReals = await Promise.all(browseDirs.map((dir) => fs.realpath(dir).catch(() => dir)));
@@ -68,6 +97,18 @@ export async function resolveConfigPath(projectRoot, config, requestedRelPath) {
     }
     return { absPath: canonical, kind };
 }
+/** プロジェクトの aiConfigPaths 許可リスト検証 (N-2)。完全一致 = settingsFiles/rootFiles、接頭辞 = browseDirs */
+export async function resolveConfigPath(projectRoot, config, requestedRelPath) {
+    return resolveAllowlistedPath(projectRoot, [...config.aiConfigPaths.settingsFiles, ...config.aiConfigPaths.rootFiles], config.aiConfigPaths.browseDirs, requestedRelPath);
+}
+/**
+ * グローバル (~/.claude) の skill/agent 許可リスト検証。プロジェクトと同一コアを root=home で再利用する。
+ * 対象は GLOBAL_BROWSE_DIRS (~/.claude/skills, ~/.claude/agents) の配下のみ。deny-list / 拡張子許可 / symlink 拒否は同一。
+ * requestedRelPath は home からの相対 (例: `.claude/skills/foo.md`)。
+ */
+export async function resolveGlobalConfigPath(homeDir, requestedRelPath) {
+    return resolveAllowlistedPath(homeDir, [], [...GLOBAL_BROWSE_DIRS], requestedRelPath);
+}
 /**
  * 解決後パスを allow-list (主ゲート) + deny-list (defense-in-depth) で検査する (R17-E + R14-1 再修正)。
  * - **allow-list (R17-E・#E-1)**: 拡張子が SAFE_CONFIG_EXT に無ければ完全一致 / 接頭辞一致を問わず拒否。