@takagaki/cortex-decisions-viewer 0.12.76 → 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);
@@ -462,23 +465,39 @@ function rederivedBandOf(recordPath, status, ledgers, identities, verdict, miss)
     };
 }
 /**
- * 用語集の並び（#432 裁定 5-5）。**確認済みは先頭群・群の中は活性度の降順。**
+ * 用語集の並び（#432 裁定 5-5 → #690 で群分けを撤廃）。**活性度の降順だけで決める。**
+ *
+ * 🔴 **確認済みかどうかは並びに使わない**（便 #690・オーナー決定 2026-09-21 逐語
+ * 「触れられた回数とか、携わってる人、携わってる回数が多い人が上に出てくる方がいい。
+ * **アクティブ、ドラフト関係なく**」）。engine の関係者の並び（`scripts/engagement/tiers.mjs`
+ * の `order`・便 #543）に当てたのと同じ形で、**同じ画面の中で並び方を揃える**。
+ *
+ * 前はここで「確認済みが先頭群」を先に見ていたので、**群の境目が活性度より強く**、
+ * ほとんど触られていない確認済みの語が先頭に並び、**いま出てきたばかりの下書きが下に沈んだ**。
+ * 理由は 2 つ:
+ * - 「確認済みかどうか」は**札で分かる**（`render.ts` の「AI生成・未確認」）ので、
+ *   並びで二重に表現する必要がない。下書きが上に来るのは「**新しく出てきた言葉**」なので、
+ *   むしろ見たいもの
+ * - 🔴 **この並びは「索引の上から N 件だけを AI に読ませる」構想の土台**（engine #544）。
+ *   「上に来るものが重要」でなければ、その仕組みが成り立たない
  *
- * 🔴 **「確認済み」は `status` だけで決まる**（#667 のオーナー決定）。**帳簿は見ない**
- * ——先頭群に来るのは「AI生成・未確認」の札が付かない記録で、画面の札と 1 文字も同じ判定
- * （`isConfirmedStatus`）を使う。前は `ledgers.confirmation.has(p)`（＝人が 1 度でも触ったか）
- * で決めていたので、**札は「未確認」なのに先頭群に居る**記録が出ていた（実測: cortex-context
- * の用語集で 25 件）。帳簿の役目は「どの版に戻すか」だけになった。
+ * 🔴 **`isConfirmedStatus` は消さない**——このファイルの**印の門（`rederivedContentKey`）と
+ * 帯の門（`rederivedBandOf`）**が使う。`render.ts` の札は `rec.rstatus === "draft"` の
+ * **別実装**で、2 つが一致していることは `rederived-mark-ui.test.mjs` の `[札との一致]` が見る。
+ * ここで変えるのは**並びだけ**で、札・帯・印・プロンプトに渡る材料は 1 つも変えていない。
  *
  * - 活性度を持つレコードが 1 件も無い区画は **1 バイトも触らない**（帳簿が無い案件は従来順）
- * - 活性度を知らないレコードは群の末尾へ、**元の並びのまま**（安定）
+ * - 活性度を知らないレコードは**末尾**へ、**元の並びのまま**（安定）
+ * - 同じ活性度なら元の並び（日付降順 → タイトル昇順がそのまま残る）
  * - 用語集だけ（メンバーの並びは便 14・Decisions と Rules は対象外）
  *
- * ℹ️ `ledgers` は活性度のために引き続き受け取る（`ledgers.activity` だけを渡す形にすると
- * 「帳簿の判定に戻す」退行がコンパイルで止まる代わりに、**変異検証でそれを実演できなくなる**）。
+ * ℹ️ `ledgers` と `statusOf` は**受け取り続ける**。`ledgers` は活性度のために要る。
+ * `statusOf` は並びに使わなくなったが、**引数から落とすと「群分けを足し戻す」退行が
+ * コンパイルで止まる代わりに、変異検証でそれを実演できなくなる**（`ledgers` と同じ流儀）。
+ * 番人（`rederivation-ledgers.test.mjs` の `[並び]`）はこの 1 行の足し戻しで赤くなる。
  *
  * @param records 区画のレコード（`repoPathOf` で引けるパスを持つもの。並びは変えずに新しい配列を返す）
- * @param statusOf そのレコードの `rstatus`（`String()` 済み）を引く関数
+ * @param statusOf そのレコードの `rstatus`（`String()` 済み）を引く関数。**並びには使わない**（上の ℹ️）
  */
 function orderByActivity(records, pathOf, statusOf, ledgers, nowIso) {
     if (!records.length)
@@ -490,7 +509,6 @@ function orderByActivity(records, pathOf, statusOf, ledgers, nowIso) {
         return {
             r: r,
             i: i,
-            confirmed: inSection && isConfirmedStatus(statusOf(r)),
             score: inSection ? activityScore(ledgers.activity.get(p), nowIso) : null,
         };
     });
@@ -499,15 +517,12 @@ function orderByActivity(records, pathOf, statusOf, ledgers, nowIso) {
     return scored
         .slice()
         .sort((a, b) => {
-        // 1. 確認済みは先頭群（群の境目は活性度より強い）
-        if (a.confirmed !== b.confirmed)
-            return a.confirmed ? -1 : 1;
-        // 2. 群の中は活性度の降順。知らないものは群の末尾
+        // 1. 活性度の降順だけ（便 #690・確認済みかどうかでは分けない）。知らないものは末尾
         if (a.score != null && b.score != null && a.score !== b.score)
             return b.score - a.score;
         if ((a.score == null) !== (b.score == null))
             return a.score == null ? 1 : -1;
-        // 3. 同点は元の並び（安定・日付降順 → タイトル昇順がそのまま残る）
+        // 2. 同点は元の並び（安定・日付降順 → タイトル昇順がそのまま残る）
         return a.i - b.i;
     })
         .map((s) => s.r);

package/package.json

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