@takagaki/cortex-decisions-viewer 0.12.77 → 0.12.78

package/dist/build.js

@@ -40,6 +40,7 @@ exports.PROJECT_ICON_EXTS = void 0;
 exports.sanitizeBody = sanitizeBody;
 exports.findProjectIcon = findProjectIcon;
 exports.readFleetSources = readFleetSources;
+exports.attachRederived = attachRederived;
 exports.readGoldProposals = readGoldProposals;
 exports.buildHomeGrounding = buildHomeGrounding;
 exports.collectMeetings = collectMeetings;
@@ -672,7 +673,9 @@ async function readFleetSources(repoRoot) {
  * 🔴 **突き合わせはビルド時にここで閉じる**（`proposals` と同じ）。画面側でパスを組み立て直すと、
  * `repoPathOf` が `editUrl` 由来なので **`repoBaseUrl` が無い案件で帯が黙って消える**。
  */
-async function attachRederived(records, recordsDir, ctx, nowIso) {
+async function attachRederived(records, recordsDir, ctx, nowIso, 
+// 番人テストが「git に渡った key」を数えるための差し替え口。本番は常に既定の `readBlobsAt`
+readBlobs = repo_1.readBlobsAt) {
     if (!ctx)
         return records;
     // 🔴 **レコードのパスは `repoRoot` から作る**（`toRepoRelative` の `process.cwd()` 起点で
@@ -684,32 +687,46 @@ async function attachRederived(records, recordsDir, ctx, nowIso) {
     // ——ここで別のものを渡すと、画面の札と食い違う（「未確認」の札が付いた記録に帯が出る）
     const statusOf = (r) => r.rstatus;
     // 決定表の行 8（中身が変わったか）の材料。**印の候補になったレコードのぶんだけ**、
-    // `<基準コミット>:<path>` をまとめて 1 回引く。
+    // 今の版を読み、`<基準コミット>:<path>` をまとめて 1 回引く。
     // 🔴 **キーが 0 件の区画では git を起こさない**（この関数は区画ごとに呼ばれる）
     const keyOf = new Map();
-    for (const r of records) {
-        const key = (0, rederivation_1.rederivedContentKey)(pathOf(r), statusOf(r), ctx.ledgers);
-        if (key)
-            keyOf.set(r, key);
-    }
-    const bases = keyOf.size ? (0, repo_1.readBlobsAt)(ctx.repoRoot, [...keyOf.values()]) : new Map();
     // 今の版は**作業ツリーから読む**——画面が描いているのはこのファイルであって HEAD ではない
     const now = new Map();
-    for (const r of keyOf.keys()) {
+    for (const r of records) {
+        const key = (0, rederivation_1.rederivedContentKey)(pathOf(r), statusOf(r), ctx.ledgers);
+        if (!key)
+            continue;
+        const p = pathOf(r);
         try {
-            now.set(pathOf(r), await node_fs_1.promises.readFile(path.join(ctx.repoRoot, pathOf(r)), "utf8"));
+            now.set(p, await node_fs_1.promises.readFile(path.join(ctx.repoRoot, p), "utf8"));
         }
         catch {
             // 読めなければ「引けなかった」側（下の verdict が unknown に倒す）
         }
+        // 🔴 **帳簿の指紋が今の中身と一致する記録は git の key に入れない**（engine #546）。
+        // 答えは下の verdict で `same` になるので、基準版を引く意味が無い——ここで外さないと、
+        // engine が指紋を書いても用語 1400 件の案件で毎晩 1400 個の blob を読み続ける。
+        // 指紋が無い・食い違う・今の版が読めない記録は今までどおり git へ落とす
+        const nowRaw = now.get(p);
+        const sha = ctx.ledgers.confirmation.get(p)?.contentSha;
+        if (nowRaw != null && sha && sha === (0, rederivation_1.contentShaOf)(nowRaw)) {
+            if (ctx.blobs)
+                ctx.blobs.skipped++;
+            continue;
+        }
+        keyOf.set(r, key);
     }
+    if (ctx.blobs)
+        ctx.blobs.read += keyOf.size;
+    const bases = keyOf.size ? readBlobs(ctx.repoRoot, [...keyOf.values()]) : new Map();
     const verdict = (p, baseCommit) => {
         const nowRaw = now.get(p);
         if (nowRaw == null) {
             ctx.miss.unreadable.add(p);
             return "unknown";
         }
-        // 🔴 **帳簿が中身の指紋を持つならそれを先に使う**（engine が後の便で書く予定）。
+        // 🔴 **帳簿が中身の指紋を持つならそれを先に使う**（engine #546 が書く。一致した記録は
+        // 上で git の key から外してある）。
         // 一致したときだけ「変わっていない」と言い、食い違ったら git の二段目へ落とす——
         // engine が別の綴りで指紋を作っていた場合に、全レコードへ誤報が出るのを避ける
         const conf = ctx.ledgers.confirmation.get(p);
@@ -968,10 +985,18 @@ async function loadGenericRecords(dirAbs, exclude, repoBaseUrl, branch) {
         records.push(await parseGeneric(raw, e.name, repoRelDir, repoBaseUrl, branch));
     }
     // 日付降順 → タイトル昇順（用語集のように日付が同一でも安定して並ぶ）
+    //
+    // 🔴 **第 2 鍵（コード順）を外さない**（#576）。`localeCompare(…, "ja")` は**別の綴りでも
+    // 0 を返す**（実測: `が`/`ガ`・`サーバ`/`さーば`・`AI`/`ＡＩ`・`あ行`/`ア行`）。0 のまま
+    // 安定ソートに任せると、その 2 語の順を決めるのは**`readdir` が返した順**になり、Node は
+    // その順を保証しない——中身を 1 文字も変えていない日に、用語集の並びが環境で変わる。
+    // cortex-engine の記憶の目次（`scripts/memory-index.mjs` の `cmpTerm`）はこの並びに
+    // **上位 20 件の集合**を合わせているので、ここが揺れると目次との一致も揺れる。
+    // 引き分けだったところに順を決めるだけで、**決まっていた並びは 1 つも変えていない**。
     records.sort((a, b) => {
         if (a.date !== b.date)
             return a.date < b.date ? 1 : -1;
-        return a.title.localeCompare(b.title, "ja");
+        return a.title.localeCompare(b.title, "ja") || (a.title < b.title ? -1 : a.title > b.title ? 1 : 0);
     });
     return records;
 }
@@ -1959,6 +1984,7 @@ async function build(opts) {
             repoRoot: repoRoot,
             identities: ledgers.confirmation.size ? (0, repo_1.collectGoldCommitIdentities)(repoRoot) : new Map(),
             miss: { unresolved: new Set(), unreadable: new Set() },
+            blobs: { read: 0, skipped: 0 },
         }
         : null;
     const decisions = [];
@@ -2051,6 +2077,12 @@ async function build(opts) {
             `（浅い clone か、確認の後に改名・削除された。変わったかを言えないので印は出しません）: ` +
             paths.slice(0, 3).join(", ") + (paths.length > 3 ? " ほか" : ""));
     }
+    // 🔴 **指紋（engine #546）が効いているかを 1 行残す。** 一致した記録は git を読まないので、
+    // 「読んだ件数」が候補の数より減っていれば効いている。警告ではないので console.log
+    if (rederive?.blobs && rederive.blobs.read + rederive.blobs.skipped > 0) {
+        console.log(`gold-confirmation.json: 人が確認した版の中身を git から ${rederive.blobs.read} 件読みました` +
+            `（中身の指紋が一致した ${rederive.blobs.skipped} 件は読んでいません）`);
+    }
     const unmatched = [...PROPOSALS_BY_PATH.keys()].filter((k) => !PROPOSAL_PATHS_MATCHED.has(k));
     if (unmatched.length) {
         console.warn(`gold-proposals.json: レコードに突き合わない提案が ${unmatched.length} 件ありました（表示しません）: ` +

package/dist/rederivation.js

@@ -141,7 +141,7 @@ async function readRederivationLedgers(repoRoot) {
             const changedBy = Array.isArray(row?.changed_by)
                 ? row.changed_by.map((s) => String(s)).filter((s) => s !== "")
                 : [];
-            // 中身の指紋は**文字列のときだけ**持つ（engine がまだ書かないので、いまは常に無い）
+            // 中身の指紋は**文字列のときだけ**持つ（engine #546 が書く。古い帳簿の行には無い）
             const contentSha = typeof confirmed.content_sha === "string" ? confirmed.content_sha : "";
             out.confirmation.set(p, {
                 at: confirmed.at != null ? String(confirmed.at) : "",
@@ -311,9 +311,12 @@ function sameRederivedContent(a, b) {
 /**
  * 「説明文と本文」の指紋。**帳簿の `confirmed.content_sha` と突き合わせる綴り**。
  *
- * 🔴 engine がこの欄を書き始めるまで、この値と比べる相手は存在しない。**一致したときだけ
+ * 🔴 比べる相手は engine #546 が書く `confirmed.content_sha`。**一致したときだけ
  * 「変わっていない」と言い、食い違ったら git へ落とす**（`attachRederived`）——engine が
  * 別の綴りで指紋を作っていた場合に、全レコードへ誤報が出るのを避けるため。
+ *
+ * engine の写し: `cortex-engine/scripts/rederive/content-sha.mjs`・一致の番人:
+ * `cortex-engine/test/rederive-history.test.mjs` の #546 の節（ここを変えたら engine の `VIEWER_SHA` を計算し直す）
  */
 function contentShaOf(raw) {
     const c = comparableContent(raw);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@takagaki/cortex-decisions-viewer",
-  "version": "0.12.77",
+  "version": "0.12.78",
   "description": "Cortexの意思決定記録（Decisions/*.md）を静的サイトにビルドして閲覧する。各レコードに「Edit on GitHub」リンクを付与する。",
   "license": "UNLICENSED",
   "private": false,