@takagaki/cortex-decisions-viewer 0.10.5 → 0.10.7

package/dist/render.js

@@ -12,6 +12,7 @@ exports.chatRowsToAdd = chatRowsToAdd;
 exports.devRowsToAdd = devRowsToAdd;
 exports.designRowsToAdd = designRowsToAdd;
 exports.materialRowsToAdd = materialRowsToAdd;
+exports.issuesRefOf = issuesRefOf;
 exports.issueRowsToAdd = issueRowsToAdd;
 exports.unsettledRegisteredCount = unsettledRegisteredCount;
 exports.registeredRowsToAdd = registeredRowsToAdd;
@@ -734,6 +735,25 @@ function materialRowsToAdd(ids, rows) {
     }
     return out;
 }
+/**
+ * 課題管理（Backlog）の接続先1件を1つの鍵にする。**ドメインは小文字・キーは大文字。**
+ *
+ * 🔴 **この規則の正本はここ1つ。** 設定ファイルの値は人が書いたもので大小が揃って
+ * いないので、揃えずに突き合わせると**同じ接続先を別物と見る**。それが起きる場所が
+ * 3つある——行の重複判定（`issueRowsToAdd`）、登録から消えた行の引き算
+ * （`subtractRemovedRegistered`）、そして接続状態ストアの行との突き合わせ
+ * （画面の `ingestedStateOf`。ストアの `ref` も engine 側がこの規則で書く）。
+ *
+ * **クロージャを持たないこと**（クライアントへ `.toString()` で流し込む）。
+ *
+ * ⚠ 画面の `targetsGenerationOf`（サーバへ送る世代）は同じ規則を持つが、この関数を
+ * 呼べない——配信物から関数1つを切り出して直接呼ぶテストがあり、外の名前を参照した
+ * 瞬間にそのテストが動かなくなる。あちらを直すときはここも直す。
+ */
+function issuesRefOf(domain, projectKey) {
+    return String(domain == null ? "" : domain).toLowerCase() + "/" +
+        String(projectKey == null ? "" : projectKey).toUpperCase();
+}
 /**
  * 課題管理の行で、まとめ表がまだ知らないもの。
  *
@@ -742,7 +762,7 @@ function materialRowsToAdd(ids, rows) {
  * 二重に並ぶ（片方は削除ボタンを持たない）。
  */
 function issueRowsToAdd(targets, rows) {
-    const key = (d, k) => String(d).toLowerCase() + "/" + String(k).toUpperCase();
+    const key = issuesRefOf;
     const have = {};
     for (const t of targets || [])
         if (t && t.domain && t.projectKey)
@@ -899,7 +919,7 @@ function subtractRemovedRegistered(view, reg) {
     if (Array.isArray(reg.issues)) {
         // 鍵の畳み方は足す側（issueRowsToAdd）と同じにする——別の畳み方にすると
         // 大小の違いで「引いた直後に別綴りで足し直される」形になる
-        const issueKey = (t) => String(t.domain).toLowerCase() + "/" + String(t.projectKey).toUpperCase();
+        const issueKey = (t) => issuesRefOf(t.domain, t.projectKey);
         const keys = {};
         for (const r of reg.issues)
             if (r)
@@ -2051,6 +2071,12 @@ a.ds-name:hover { text-decoration: underline; }
 .ds-all-connected { color: var(--muted); }
 .ds-gate { display: inline-flex; align-items: center; gap: 5px; font-size: 12.5px; font-weight: 600; flex: 0 0 auto; }
 .ds-gate-at { font-size: 11px; color: var(--muted); flex: 0 0 auto; }
+/* 接続先1件ごとの「取り込み」（段C・2026-08-24）。**「最終同期」（.ds-sync・run単位）とは
+   別の行に並ぶ**ので、同じ強さにしない——run の時刻より弱い添え物として読ませる。
+   失敗だけは .ds-gate-warn と同じ色にする（同じ画面で警告の色が2種類あると、
+   どちらが重いのか読めない） */
+.ds-ingest { font-size: 11px; color: var(--muted); flex: 0 0 auto; font-variant-numeric: tabular-nums; }
+.ds-ingest-failed { color: #b45309; font-weight: 600; }
 .ds-gate::before { content: ""; width: 8px; height: 8px; border-radius: 999px; background: currentColor; }
 .ds-gate-ok { color: #1f7a45; }
 .ds-gate-warn { color: #b45309; }
@@ -2324,6 +2350,7 @@ ${capabilityAdapter.toString()}
 ${capabilityRawState.toString()}
 ${capabilityDisplayState.toString()}
 ${backlogTargetOf.toString()}
+${issuesRefOf.toString()}
 ${sourceNeedsAttention.toString()}
 ${capabilityState.toString()}
   // 反映待ち（⏳ 反映中）の判定。**期限・間隔・対応表もビルド時の定義から流し込む**
@@ -5608,6 +5635,7 @@ ${nextStateDelay.toString()}
    * ——食い違わないことは backlog-targets-ui.test.mjs が実物どうしで突き合わせている。
    *
    * **クロージャを持たない純関数に保つこと**（テストが配信物から切り出して直接呼ぶ）。
+   * だから issuesRefOf を呼べない（規則の正本はあちら側。直すときは両方直す）。
    */
   function targetsGenerationOf(list) {
     return (list || [])
@@ -5710,6 +5738,116 @@ ${nextStateDelay.toString()}
     return domains.length === 1 ? domains[0] : "";
   }
 
+  /**
+   * その接続先の「取り込み」の行（接続状態ストア・段C）。**無ければ null。**
+   *
+   * 🔴 **overlayStateOnFleet は通らない。** あれは externalSources（チャット / 開発）に
+   * 重ねる仕組みで、**Backlog の行はそこに存在しない**（まとめ表を作る側が作るのは
+   * slack と github 系だけ）。だから接続先の行を描くこちらがストアを直接引く。
+   *
+   * **突き合わせは両側とも issuesRefOf を通す。** 設定ファイルの値は人が書いたもので
+   * 大小が揃っておらず（"CM1.Backlog.JP" / "pj_a" が実在する）、素の文字列で比べると
+   * **取り込めているのに何も出ない**——「測っていないものは何も言わない」設計なので、
+   * 突き合わせに失敗しても画面は静かに黙るだけで、誰も気づけない。
+   */
+  function ingestedStateOf(domain, projectKey) {
+    var rows = LIVE_STATE;
+    if (!rows || !rows.length || !domain || !projectKey) return null;
+    var want = issuesRefOf(domain, projectKey);
+    for (var i = 0; i < rows.length; i++) {
+      var r = rows[i];
+      // cap と question の両方で絞る。片方だけだと、同じ ref を持つ別の問い
+      //（将来 issues に別の question が増えたとき）の行を取り込みとして読む
+      if (!r || r.cap !== "issues" || r.question !== "ingested") continue;
+      var at = String(r.ref || "").indexOf("/");
+      if (at < 0) continue;
+      if (issuesRefOf(String(r.ref).slice(0, at), String(r.ref).slice(at + 1)) === want) return r;
+    }
+    return null;
+  }
+
+  /**
+   * 接続先1件の「取り込み」の表示。**測っていなければ何も出さない（null）。**
+   *
+   * 段A・段Bと同じ原則。行が無いのは「取り込みが動いていない」ではなく
+   * 「まだ誰も書いていない」（engine の版が古い案件・一度も同期していない案件）で、
+   * そこに「未取り込み」と書くと**直す必要があると読ませる嘘**になる。
+   *
+   * **「最終同期」（.ds-sync）を置き換えない。** あちらは run 全体の最終成功時刻で、
+   * どの接続先のことか分からない。こちらは接続先1件の成否。意味が違うので併記する。
+   */
+  function ingestedEl(r) {
+    if (!r) return null;
+    var when = rtAgo(r.at);
+    var text = "";
+    if (r.status === "ok") text = when ? "取り込み: " + when + "に成功" : "取り込み: 成功";
+    else if (r.status === "failed") text = when ? "取り込み: " + when + "に失敗" : "取り込み: 失敗";
+    else if (r.status === "skipped") text = "取り込み: スキップ";
+    if (!text) return null;
+    return el("span", {
+      class: "ds-ingest" + (r.status === "failed" ? " ds-ingest-failed" : ""),
+      text: text,
+    });
+  }
+
+  /**
+   * いま一覧に出ている接続先の「取り込み」をひとつの文字列にする（差し替えの要否の判定用）。
+   *
+   * 🔴 **接続先の世代（targetsGenerationOf）だけでは足りない。** モーダルを開くと
+   * その能力の状況をもう一度取りに行くので、**応答が返る前に開いた人は取り込みの行を
+   * 1件も持っていない**。届いた応答で接続先の集合が変わっていなければ、世代だけを見る
+   * 早期return が描き直しを飛ばし、**閉じて開き直すまで空のまま**になる。
+   * 空表示は「まだ誰も測っていない」と区別が付かないので、**取り込みの失敗を黙る**という
+   * この機能が殺そうとしている欠陥そのものが、別の経路で復活する。
+   */
+  function ingestSigOf(list) {
+    return (list || []).map(function (t) {
+      var r = ingestedStateOf(t.domain, t.projectKey);
+      return r ? r.status + "@" + r.at : "-";
+    }).join(",");
+  }
+
+  /**
+   * Figmaファイル1件の「取り込み」の行（接続状態ストア・段C第二弾）。**無ければ null。**
+   * 表示は Backlog と同じ ingestedEl を通す（状態から文言を決める規則は1つだけ持つ）。
+   *
+   * 🔴 **鍵は正規化しない**——ingestedStateOf（課題管理）との意図的な違い。Figma の
+   * ファイルキーは**大小を区別する不透明ID**で、揃えにいくと別のファイルを同じものと見る。
+   * logic.mjs の figmaFileKey も trim だけで大小を保ち、applyAddFigmaFile の重複判定も完全一致。
+   * 突き合わせが成立する根拠は「人が大小を揃えること」ではなく、**書き手（同期ジョブ）と
+   * 画面（figmaFiles）が同じ figma.json の同じ値を読むこと**。
+   */
+  function designIngestedStateOf(key) {
+    var rows = LIVE_STATE;
+    var want = String(key || "");
+    if (!rows || !rows.length || !want) return null;
+    for (var i = 0; i < rows.length; i++) {
+      var r = rows[i];
+      // cap と question の両方で絞る。片方だけだと、同じ ref を持つ別の問いの行
+      //（別能力の行・将来 design に増える別の question）を取り込みとして読む
+      if (!r || r.cap !== "design" || r.question !== "ingested") continue;
+      if (r.ref === want) return r;
+    }
+    return null;
+  }
+
+  /**
+   * いま一覧に出ている Figma ファイルの「取り込み」をひとつの文字列にする（liveList の sig 用）。
+   *
+   * 🔴 **ファイル一覧のJSONだけでは足りない。** モーダルを開くとその能力の状況をもう一度
+   * 取りに行くので、**応答が返る前に開いた人は取り込みの行を1件も持っていない**。届いた応答で
+   * ファイルの集合は変わらないため、一覧だけを署名にすると liveList の早期return が描き直しを
+   * 飛ばし、**閉じて開き直すまで空のまま**になる。空表示は「まだ誰も測っていない」と
+   * 見分けが付かないので、この機能が殺そうとしている「取り込みの失敗を黙る」欠陥が
+   * 別の経路で復活する（課題管理側の ingestSigOf と同じ理由）。
+   */
+  function designIngestSigOf(list) {
+    return (list || []).map(function (f) {
+      var r = designIngestedStateOf(f && f.key);
+      return r ? r.status + "@" + r.at : "-";
+    }).join(",");
+  }
+
   /**
    * 「課題管理」カードの接続先（一覧＋追加＋削除・#127）。
    *
@@ -5768,6 +5906,8 @@ ${nextStateDelay.toString()}
           el("span", { class: "ds-type", text: "Backlog" }),
           dsNameEl(t.domain + " / " + t.projectKey, "https://" + t.domain + "/projects/" + t.projectKey),
         ]),
+        // この接続先の取り込みが最後にどうなったか（段C）。測っていなければ null
+        ingestedEl(ingestedStateOf(t.domain, t.projectKey)),
       ]);
       if (!editable) return row;
       var btn = el("button", { class: "ds-toggle-btn ds-danger", type: "button", text: "この接続先を外す" });
@@ -5912,6 +6052,8 @@ ${nextStateDelay.toString()}
     // 箱が無いと最初の描画で「未設定」の行が出ない
     var box = el("div", { "data-live-cap": REGISTERED_CAP_NAMES.issues }, [listEl]);
     paint();
+    /** いま描いてある取り込みの状態（段C）。接続先の集合とは別に変わるので、別に覚える */
+    var lastIngestSig = ingestSigOf(targets);
     /**
      * 取り直しで新しい値が届いたときの差し替え（段階1）。**一覧だけを描き直す。**
      *
@@ -5923,8 +6065,13 @@ ${nextStateDelay.toString()}
       // 🔴 応答に添えられた一覧（adoptTargets）のほうが新しい。上書きしない
       if (adopted) return;
       var next = backlogTargetsNow(internalSourceOf(REGISTERED_CAP_NAMES.issues));
-      if (targetsGenerationOf(next) === targetsGenerationOf(targets)) return;
+      // 🔴 **変化が無いと言えるのは、接続先の集合と取り込みの状態が両方同じときだけ（段C）。**
+      // 世代だけで判断すると、**応答を待たずにモーダルを開いた人には取り込みが永久に出ない**
+      //（届いた応答で集合は変わらないので、ここで飛ばされる）。ingestSigOf の項に経緯
+      var sig = ingestSigOf(next);
+      if (targetsGenerationOf(next) === targetsGenerationOf(targets) && sig === lastIngestSig) return;
       targets = next;
+      lastIngestSig = sig;
       paint();
     };
     // 接続先がまだ無い案件の受け皿は、0件の1行の直後に置く（行が無いので中へ入れられない）
@@ -6189,15 +6336,57 @@ ${nextStateDelay.toString()}
       // **一覧は開いたままでも差し替わる箱に入れる（段階1）。** 追加は「モーダルを開いたまま」
       // 行われるので、ここが固定だと足したファイルは閉じるまで画面に出ない
       var figsOf = function () { return (internalSourceOf("デザイン") || {}).figmaFiles || []; };
-      box.appendChild(liveList("デザイン", function () { return JSON.stringify(figsOf()); }, function (listBox) {
+      /**
+       * 外し終えたファイルの鍵。**一覧の箱の外に置く（段C第二弾）。**
+       *
+       * 外した行には取り消し線を引き、ボタンだけを無効にする。その印は行の closure に
+       * あるので**描き直しで作り直される**——取り込みの状態を署名に混ぜたことで
+       * LIVE_STATE の到着でも描き直しが走るようになり、ここに覚えておかないと
+       * **外したばかりの行が生き返り、二度押しで「登録されていません」になる**
+       *（削除は取り消せないので、押せる見た目に戻すのがいちばん悪い）。
+       *
+       * **効くのは短い。** 段A補（subtractRemovedRegistered）以降、外した行は次の
+       * op=state が返った時点で一覧から消える。だからこの印が要るのは
+       * **①保存の応答から次の op=state までの数秒**と、**②op=state が取れない案件**
+       *（古い Lambda・通信失敗で、夜間の一覧が翌朝まで残る）だけ。
+       *
+       * 🔴 **一覧から消えた鍵は落とす（下の prune）。** 消えた＝サーバが削除を認めた
+       * ということなので、この印は役目を終えている。**持ち続けると、押し間違いに気づいて
+       * 同じファイルを貼り直した人の行が、復活した瞬間に「外し済み」で描かれて操作不能に
+       * なる**（回復手段はモーダルを開き直すことだけで、その手掛かりが画面に無い）。
+       * 追加の口から鍵を落とす手もあるが、あちらが受け取るのは URL で、鍵に直すには
+       * サーバ側の figmaFileKey を画面にもう1つ持つことになる（同じ規則が2箇所に増える）。
+       */
+      var figsRemoved = [];
+      /**
+       * いま送信中の行（鍵 → 行の節点）。**描き直しでも作り直さず、この節点を使い続ける。**
+       *
+       * 取り込みの状態を署名に混ぜたことで、**op=config の応答待ちの最中に op=state が
+       * 届くと行が作り直される**窓ができた。作り直すと「保存中…」と押せない状態を持つ
+       * 節点が捨てられ、**返事が来る前にもう一度押せてしまう**。押している当人の行だけを
+       * 据え置き、他の行の取り込み表示は普段どおり追いつかせる（removeButton の onBusy）。
+       */
+      var figSending = {};
+      // **署名に取り込みの状態も混ぜる（段C第二弾）。** 一覧のJSONだけだと、応答が返る前に
+      // モーダルを開いた人には取り込みが1件も出ない（designIngestSigOf の項に経緯）
+      box.appendChild(liveList("デザイン", function () {
+        return JSON.stringify(figsOf()) + "|" + designIngestSigOf(figsOf());
+      }, function (listBox) {
       var figs = figsOf();
+      // **一覧から消えた鍵は「外し済み」から落とす**（figsRemoved の項に経緯）。
+      // サーバが削除を認めた印なので、ここから先に同じ鍵が現れたら**貼り直された別の登録**
+      var figKeys = figs.map(function (f) { return f.key; });
+      figsRemoved = figsRemoved.filter(function (k) { return figKeys.indexOf(k) >= 0; });
       // **残り件数は「押した時点」で数える。** 一覧（figs）は夜間バッチのスナップショットで、
       // このページを開いている間はもちろん、その日のうちに再読み込みしても更新されない。
       // 描画時点の件数で判定すると、3件を1件ずつ外していったとき最後の1件でも警告が出ない。
-      var figsLeft = figs.length;
+      // **描き直しでも外した分を引いたままにする**（figsRemoved の項に経緯）
+      var figsLeft = figs.filter(function (f) { return figsRemoved.indexOf(f.key) < 0; }).length;
       if (figs.length) {
         var flist = el("div", { class: "ds-list" });
         figs.forEach(function (f) {
+          // 送信中の行は作り直さない（figSending の項に経緯）
+          if (figSending[f.key]) { flist.appendChild(figSending[f.key]); return; }
           // **Slack / GitHub の行と同じ構造にする（ds-card > ds-card-main >
           // ds-icon + ds-type + ds-name）。** extSourceCard は呼ばない——
           // あちらは type / gate / origin など外部ソース固有の判断を抱えている。
@@ -6208,6 +6397,8 @@ ${nextStateDelay.toString()}
               el("span", { class: "ds-type", text: "Figma" }),
               dsNameEl(f.name || f.key, "https://www.figma.com/design/" + f.key),
             ]),
+            // このファイルの取り込みが最後にどうなったか（段C）。測っていなければ null
+            ingestedEl(designIngestedStateOf(f.key)),
           ]);
           row.appendChild(removeButton({
             label: "このファイルを外す",
@@ -6233,9 +6424,21 @@ ${nextStateDelay.toString()}
             onDone: function () {
               // **クラスのハード代入をやめる**（片方だけ直し忘れる形を構造的に消す）
               row.classList.add("ds-removed");
+              if (figsRemoved.indexOf(f.key) < 0) figsRemoved.push(f.key);
               figsLeft -= 1;
             },
+            // 確認〜応答のあいだ、この行だけ描き直しの対象から外す（figSending の項に経緯）
+            onBusy: function (on) {
+              if (on) figSending[f.key] = row;
+              else delete figSending[f.key];
+            },
           }));
+          // 描き直しのあとも「外した」ことが分かる形に戻す（figsRemoved の項に経緯）
+          if (figsRemoved.indexOf(f.key) >= 0) {
+            row.classList.add("ds-removed");
+            var doneBtn = row.querySelector(".ds-toggle-btn");
+            if (doneBtn) doneBtn.disabled = true;
+          }
           flist.appendChild(row);
         });
         listBox.appendChild(flist);
@@ -6669,7 +6872,14 @@ ${nextStateDelay.toString()}
    * 成功しても一覧からは消さない。接続状況は夜間バッチのスナップショットで、消したつもりの行が
    * 翌朝まで残るのが正しい状態のため——行に取り消し線を引き、ボタンだけを無効にする
    * （二度押しすると「登録されていません」で失敗するので、押せなくする）。
-   * @param opts { label, confirmLines(), build(), onDone() }
+   * **押している間（確認〜応答）を呼び出し側に知らせる（onBusy）。**
+   * 一覧を差し替える口では、この間に描き直しが走るとこの行ごと作り直され、
+   * 「保存中…」と押せない状態を持つ節点が捨てられる——**二度押しできる見た目に戻る**。
+   * 削除は取り消せないので、行を差し替える側はこの間だけ元の節点を使い続けること。
+   * **確認の前から数える**（確認モーダルの戻り先もこの節点なので、途中で捨てると
+   * 「いいえ」を押した人のフォーカスの戻り先が消える）。
+   *
+   * @param opts { label, confirmLines(), build(), onDone(), onBusy(bool) }
    */
   function removeButton(opts) {
     var wrap = el("div", { class: "ds-toggle" });
@@ -6677,11 +6887,18 @@ ${nextStateDelay.toString()}
     var msg = el("div", { class: "ds-toggle-msg" });
 
     btn.addEventListener("click", async function () {
-      if (!(await confirmDanger(opts.confirmLines(), { returnTo: btn }))) { msg.textContent = ""; return; }
-      sendConfig(opts.build(), msg, btn, function () {
-        btn.disabled = true;
-        if (opts.onDone) opts.onDone();
-      });
+      if (opts.onBusy) opts.onBusy(true);
+      // **finally で必ず倒す。** 断られた・通信に失敗した経路で立てっぱなしにすると、
+      // 呼び出し側はその行を永久に差し替えられなくなる（sendConfig は自分で捕まえるので投げない）
+      try {
+        if (!(await confirmDanger(opts.confirmLines(), { returnTo: btn }))) { msg.textContent = ""; return; }
+        await sendConfig(opts.build(), msg, btn, function () {
+          btn.disabled = true;
+          if (opts.onDone) opts.onDone();
+        });
+      } finally {
+        if (opts.onBusy) opts.onBusy(false);
+      }
     });
 
     wrap.appendChild(btn);

package/package.json

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