@takagaki/cortex-decisions-viewer 0.10.5 → 0.10.6

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,75 @@ ${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(",");
+  }
+
   /**
    * 「課題管理」カードの接続先（一覧＋追加＋削除・#127）。
    *
@@ -5768,6 +5865,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 +6011,8 @@ ${nextStateDelay.toString()}
     // 箱が無いと最初の描画で「未設定」の行が出ない
     var box = el("div", { "data-live-cap": REGISTERED_CAP_NAMES.issues }, [listEl]);
     paint();
+    /** いま描いてある取り込みの状態（段C）。接続先の集合とは別に変わるので、別に覚える */
+    var lastIngestSig = ingestSigOf(targets);
     /**
      * 取り直しで新しい値が届いたときの差し替え（段階1）。**一覧だけを描き直す。**
      *
@@ -5923,8 +6024,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行の直後に置く（行が無いので中へ入れられない）

package/package.json

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