@takagaki/cortex-decisions-viewer 0.12.73 → 0.12.74

package/dist/build.js

@@ -676,13 +676,16 @@ function attachRederived(records, recordsRel, ctx, nowIso) {
     if (!ctx)
         return records;
     const pathOf = (r) => (recordsRel ? `${recordsRel}/${r.fileName}` : r.fileName);
+    // 🔴 **「確認済み」は `status` だけで決まる**（#667）。帯も並びも同じ `rstatus` を見る
+    // ——ここで別のものを渡すと、画面の札と食い違う（「未確認」の札が付いた記録に帯が出る）
+    const statusOf = (r) => r.rstatus;
     for (const r of records) {
-        const band = (0, rederivation_1.rederivedBandOf)(pathOf(r), ctx.ledgers, ctx.identities, ctx.miss);
+        const band = (0, rederivation_1.rederivedBandOf)(pathOf(r), statusOf(r), ctx.ledgers, ctx.identities, ctx.miss);
         // **帯が立たないレコードにはキーごと付けない**（空の器を通さない）
         if (band)
             r.rederived = band;
     }
-    return (0, rederivation_1.orderByActivity)(records, pathOf, ctx.ledgers, nowIso);
+    return (0, rederivation_1.orderByActivity)(records, pathOf, statusOf, ctx.ledgers, nowIso);
 }
 /**
  * AIからの更新提案（`.cortex/gold-proposals.json`）を読む。

package/dist/rederivation.js

@@ -4,6 +4,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.emptyLedgers = exports.LEDGER_VERSION = void 0;
+exports.isConfirmedStatus = isConfirmedStatus;
 exports.parseLedger = parseLedger;
 exports.readRederivationLedgers = readRederivationLedgers;
 exports.parseInjectedNow = parseInjectedNow;
@@ -49,6 +50,24 @@ const ACTR_DECAY = 0.5;
  * 0.5 日＝「今日触れられた」を半日前として数える（ACT-R の実装で普通に使う丸め）。
  */
 const MIN_ELAPSED_DAYS = 0.5;
+/**
+ * その記録が「確認済み」か。**「AI生成・未確認」の札（`render.ts` の `draftBadge`）の裏返し**
+ * ——札が出ない記録が確認済み側（#667 のオーナー決定「確認済みは `status` だけで判断する」）。
+ *
+ * 🔴 **trim も小文字化もしない。** 札は `rec.rstatus === "draft"` の厳密比較なので、こちらを
+ * 1 文字でも緩めると「札は出ないのに未確認扱い」「札は出るのに確認済み扱い」が生まれる。
+ * `status` 欄が無い記録（`rstatus` が `""`）と `superseded` は**確認済み側**——札が出ないから。
+ *
+ * 🔴 **渡すのは `String()` 済みの `rstatus` だけ**（`build.ts:1048`）。生の frontmatter
+ * （`rec.fm.status`）を渡さない——YAML は真偽値も配列も返すので、札と同じ文字列にならない。
+ *
+ * ⚠️ **「確認済み」の門はもう 1 つある。** 削除の口の出し分けは `render.ts` の
+ * `humanConfirmedStatus`（`active|superseded|inactive` の列挙）で、`""` の扱いがここと逆。
+ * 揃えるかどうかは別の便（実データでは全 721 件が `status` を持つので今日の差は 0 件）。
+ */
+function isConfirmedStatus(status) {
+    return status !== "draft";
+}
 const emptyLedgers = () => ({
     confirmation: new Map(),
     rederivation: new Map(),
@@ -107,8 +126,11 @@ async function readRederivationLedgers(repoRoot) {
             if (!confirmed)
                 continue;
             // 🔴 **基準コミットが空の行も持つ**（実装レビュー）。帯は作れない（差分も戻すも
-            // 指す先が無い）が、**「人が確認した」という事実は並びが使う**——ここで落とすと、
-            // 確認済みなのに先頭群から黙って外れる。帯の側（`rederivedBandOf`）で落とす
+            // 指す先が無い）が、行そのものは落とさずに持つ——落とす判断は帯の側
+            // （`rederivedBandOf`）に 1 か所で寄せる。
+            // ⚠️ #667 より前は「並びが『人が確認した』事実として使う」ためでもあったが、**並びは
+            // もう帳簿を見ない**（確認済みかどうかは `status` だけで決まる・`orderByActivity`）。
+            // この帳簿の役目は「どの版に戻すか」で、「確認済みかどうか」ではない
             const commit = confirmed.commit != null ? String(confirmed.commit) : "";
             const changedBy = Array.isArray(row?.changed_by)
                 ? row.changed_by.map((s) => String(s)).filter((s) => s !== "")
@@ -233,6 +255,7 @@ function activityScore(entry, nowIso) {
  *
  * | 順 | 条件 | 結果 |
  * | --- | --- | --- |
+ * | **0** | **その記録の `status` が `draft`（＝「AI生成・未確認」の札が出る）** | **立てない**（#667） |
  * | 1 | path が用語・メンバーのレコードでない | 立てない |
  * | 2 | `confirmation` に行が無い／基準コミットが空／確認日が日時として読めない | 立てない |
  * | 3 | `changed_by` が空 | 立てない |
@@ -246,9 +269,15 @@ function activityScore(entry, nowIso) {
  * | 6 | 根拠は、帳簿の行の `commit` が `changed_by` に入っているときだけ出す | 出す／出さない |
  * | 7 | 基準コミットが削除のコミットなら「戻す」を出さない（`baseDeleted`） | — |
  *
+ * @param status その記録の `rstatus`（`String()` 済み）。**2 番目の必須引数**にしてある——
+ *   末尾の任意引数にすると、渡し忘れが「行 0 の門が静かに開く」になる（設計レビュー 🔴3）
  * @param miss 立たなかった理由を数える器（呼び出し側で 1 つ作って全レコードで共有する）
  */
-function rederivedBandOf(recordPath, ledgers, identities, miss) {
+function rederivedBandOf(recordPath, status, ledgers, identities, miss) {
+    // 0. 「AI生成・未確認」の札が出る記録には帯を重ねない（#667）。**綴りは 1 か所**
+    //（ここで `status === "draft"` と直書きすると、判定が 2 つに増えて一致が崩れる）
+    if (!isConfirmedStatus(status))
+        return null;
     if (!BAND_SECTION_RE.test(recordPath))
         return null;
     const conf = ledgers.confirmation.get(recordPath);
@@ -318,7 +347,11 @@ function rederivedBandOf(recordPath, ledgers, identities, miss) {
     const baseDeleted = base ? commit_classify_1.DELETE_SUBJECT_RE.test(base.subject) : false;
     return {
         confirmedAt: conf.at,
-        confirmedBy: conf.by,
+        // 🔴 **券の無い確認では名前を器に入れない**（設計レビュー 🟡4）。画面が出さないだけだと、
+        // 名前は `site-data` の JSON として全案件のページに配られ続ける（`commit-classify.ts`
+        // の `Operated-By`「値そのものを持ち回さない」と同じ流儀）。しかも engine の `by` は
+        // `confirmer || authorName` の合成で、**券が無ければ「ただ触った人」でしかない**
+        confirmedBy: conf.verified ? conf.by : "",
         confirmedVerified: conf.verified,
         baseCommit: conf.commit,
         changedAt: newest.at,
@@ -329,13 +362,23 @@ function rederivedBandOf(recordPath, ledgers, identities, miss) {
 /**
  * 用語集の並び（#432 裁定 5-5）。**確認済みは先頭群・群の中は活性度の降順。**
  *
+ * 🔴 **「確認済み」は `status` だけで決まる**（#667 のオーナー決定）。**帳簿は見ない**
+ * ——先頭群に来るのは「AI生成・未確認」の札が付かない記録で、画面の札と 1 文字も同じ判定
+ * （`isConfirmedStatus`）を使う。前は `ledgers.confirmation.has(p)`（＝人が 1 度でも触ったか）
+ * で決めていたので、**札は「未確認」なのに先頭群に居る**記録が出ていた（実測: cortex-context
+ * の用語集で 25 件）。帳簿の役目は「どの版に戻すか」だけになった。
+ *
  * - 活性度を持つレコードが 1 件も無い区画は **1 バイトも触らない**（帳簿が無い案件は従来順）
  * - 活性度を知らないレコードは群の末尾へ、**元の並びのまま**（安定）
  * - 用語集だけ（メンバーの並びは便 14・Decisions と Rules は対象外）
  *
+ * ℹ️ `ledgers` は活性度のために引き続き受け取る（`ledgers.activity` だけを渡す形にすると
+ * 「帳簿の判定に戻す」退行がコンパイルで止まる代わりに、**変異検証でそれを実演できなくなる**）。
+ *
  * @param records 区画のレコード（`repoPathOf` で引けるパスを持つもの。並びは変えずに新しい配列を返す）
+ * @param statusOf そのレコードの `rstatus`（`String()` 済み）を引く関数
  */
-function orderByActivity(records, pathOf, ledgers, nowIso) {
+function orderByActivity(records, pathOf, statusOf, ledgers, nowIso) {
     if (!records.length)
         return records;
     // この区画が並べ替えの対象か（用語集で、活性度の行が 1 件以上あるか）
@@ -345,7 +388,7 @@ function orderByActivity(records, pathOf, ledgers, nowIso) {
         return {
             r: r,
             i: i,
-            confirmed: inSection && ledgers.confirmation.has(p),
+            confirmed: inSection && isConfirmedStatus(statusOf(r)),
             score: inSection ? activityScore(ledgers.activity.get(p), nowIso) : null,
         };
     });

package/dist/render.js

@@ -6236,13 +6236,20 @@ ${calendarAddUrl.toString()}
    * 「AI による最近の変更」が変わった行そのものを持っている。
    */
 
-  /** 帯に出す 1 行。券の無い確認（自己申告）は名前を権威として出さない（#432 5-A） */
+  /**
+   * 帯に出す 1 行。**帯は起きたこと（機械が書き直した日）を言う。**
+   *
+   * 🔴 **確認した人と日は、券のある確認（confirmedVerified）のときだけ添える**（#667）。
+   * 「確認済み」は frontmatter の status だけで決まるようになり、その大半は
+   * 「AIで編集」か直接編集で active になったもの＝**画面から確認したわけではない**。
+   * そこに「○○さんが 9/1 に確認」と書くと、起きていないことを言い切ることになる。
+   * 券があるときだけ名前と日を出すので、「自己申告」の断り書きは要らなくなった。
+   */
   function rederivedMetaText(b) {
     var parts = [];
-    var who = b.confirmedBy
-      ? "（" + b.confirmedBy + (b.confirmedVerified ? "" : "・自己申告") + "）"
-      : "";
-    parts.push("確認 " + ahDate(b.confirmedAt) + who);
+    if (b.confirmedVerified) {
+      parts.push("確認 " + ahDate(b.confirmedAt) + (b.confirmedBy ? "（" + b.confirmedBy + "）" : ""));
+    }
     parts.push("書き直し " + ahDate(b.changedAt));
     // **「根拠」は言えるときだけ**（この帯のコミットの記録が帳簿に無ければ出さない）
     if (typeof b.evidence === "number") parts.push("根拠 " + b.evidence + " 件");

package/package.json

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