@takagaki/cortex-decisions-viewer 0.9.0 → 0.10.0

package/dist/render.js

@@ -6,6 +6,8 @@ exports.livePending = livePending;
 exports.nextCheckDelay = nextCheckDelay;
 exports.settledPending = settledPending;
 exports.sanitizeRegisteredNow = sanitizeRegisteredNow;
+exports.sanitizeStateNow = sanitizeStateNow;
+exports.overlayStateOnFleet = overlayStateOnFleet;
 exports.chatRowsToAdd = chatRowsToAdd;
 exports.devRowsToAdd = devRowsToAdd;
 exports.designRowsToAdd = designRowsToAdd;
@@ -564,6 +566,66 @@ function sanitizeRegisteredNow(v) {
     }
     return any ? out : null;
 }
+/** 接続状態ストアの語彙（サーバ側 connection-state.mjs と同じ。増やさない） */
+const STATE_STATUSES = {
+    reachable: ["ok", "not_in_channel", "missing_scope", "no_token", "unreachable", "unknown"],
+    ingested: ["ok", "failed", "skipped"],
+};
+const STATE_ROW_FIELDS = ["cap", "ref", "workspace", "question", "status", "at", "by", "needed", "lastOkAt", "lastFailAt"];
+/** stateNow はライブ専用キーで build.ts の網にかからない。画面側でも許可リストを通す */
+function sanitizeStateNow(v) {
+    if (!Array.isArray(v))
+        return null;
+    const out = [];
+    for (const r of v) {
+        if (out.length >= 500)
+            break;
+        if (r == null || typeof r !== "object" || Array.isArray(r))
+            continue;
+        const item = {};
+        for (const f of STATE_ROW_FIELDS) {
+            const fv = r[f];
+            if (typeof fv === "string" && fv !== "")
+                item[f] = fv;
+        }
+        const vocab = STATE_STATUSES[item.question];
+        if (!vocab || vocab.indexOf(item.status) < 0)
+            continue; // 知らない語は「不明」に化けさせず落とす
+        if (!item.cap || !item.ref || !item.at)
+            continue;
+        out.push(item);
+    }
+    return out.length ? out : null;
+}
+/**
+ * ストアの「届くか」を行に重ねる。**まとめ表より新しいときだけ**——古い確認結果で
+ * 新しい実測を潰さない（時刻の比較は ISO 文字列の辞書順で足りる）。
+ * 対象はチャットの行だけ（段A。確認ボタンが書くのがチャットだけのため）。
+ */
+function overlayStateOnFleet(view, rows, generatedAt) {
+    const v = view || {};
+    if (!rows || !rows.length || !Array.isArray(v.externalSources))
+        return v;
+    const byKey = {};
+    for (const r of rows) {
+        if (r.cap !== "chat" || r.question !== "reachable")
+            continue;
+        byKey[String(r.workspace || "") + "\t" + String(r.ref)] = r;
+    }
+    let changed = false;
+    const externals = v.externalSources.map((s) => {
+        if (!s || s.type !== "slack" || !s.ref)
+            return s;
+        const hit = byKey[String(s.workspace || "") + "\t" + String(s.ref)];
+        if (!hit)
+            return s;
+        if (generatedAt && hit.at <= generatedAt)
+            return s;
+        changed = true;
+        return Object.assign({}, s, { gate: hit.status, gateAt: hit.at, gateBy: hit.by });
+    });
+    return changed ? Object.assign({}, v, { externalSources: externals }) : v;
+}
 /**
  * チャットの行で、まとめ表がまだ知らないもの。
  *
@@ -1830,6 +1892,7 @@ a.ds-name:hover { text-decoration: underline; }
 /* 全組織が接続済みのときに、トークン入力欄の代わりに出す1行（#118） */
 .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; }
 .ds-gate::before { content: ""; width: 8px; height: 8px; border-radius: 999px; background: currentColor; }
 .ds-gate-ok { color: #1f7a45; }
 .ds-gate-warn { color: #b45309; }
@@ -2124,6 +2187,10 @@ ${internalOf.toString()}
 ${registeredRowsToAdd.toString()}
 ${unsettledRegisteredCount.toString()}
 ${mergeRegisteredFleet.toString()}
+var STATE_STATUSES = ${JSON.stringify(STATE_STATUSES)};
+var STATE_ROW_FIELDS = ${JSON.stringify(STATE_ROW_FIELDS)};
+${sanitizeStateNow.toString()}
+${overlayStateOnFleet.toString()}
 ${nextStateDelay.toString()}
   // 「はじめに」セクションのチュートリアルデータ（ビルド時に埋め込み。ボタンURL・チャレンジ4の決定は解決済み）
   var TUTORIAL = (function () {
@@ -3520,6 +3587,8 @@ ${nextStateDelay.toString()}
    * 指すのか説明できなくなる（realtime を別に持っているのと同じ理由）。
    */
   var LIVE_REGISTERED = null;
+  // 接続状態ストアの行（届くか・取り込めたか）。op=state の stateNow から
+  var LIVE_STATE = null;
   /** マージ済みの3キー（作り直しは setLive* だけが行う） */
   var MERGED = null;
   /** マージの対象になるキー。これ以外は素の値をそのまま返す */
@@ -3547,8 +3616,16 @@ ${nextStateDelay.toString()}
    * 足す場所を描画側に散らすと「タイルは1件・一覧は2件」のような食い違いが必ず出る。
    */
   function fleetVal(key) {
-    if (LIVE_REGISTERED && MERGE_KEYS[key]) {
-      if (!MERGED) MERGED = mergeRegisteredFleet(rawView(), LIVE_REGISTERED);
+    if ((LIVE_REGISTERED || LIVE_STATE) && MERGE_KEYS[key]) {
+      // 登録の追加 → 状態の上書き、の順で1回だけ合成する。
+      // 状態ストアの答えは、まとめ表より新しいときだけ勝つ（overlayStateOnFleet が判定）
+      if (!MERGED) {
+        MERGED = overlayStateOnFleet(
+          mergeRegisteredFleet(rawView(), LIVE_REGISTERED),
+          LIVE_STATE,
+          String(rawFleetVal("generatedAt") || ""),
+        );
+      }
       return MERGED[key];
     }
     return rawFleetVal(key);
@@ -3599,10 +3676,19 @@ ${nextStateDelay.toString()}
    * 変わったとき（例えば pending の語をやめたとき）に「接続OK」が復活しないようにする。
    */
   function stateCtx() {
+    // 🔴 **ストアの上書き（overlayStateOnFleet）だけは適用する。** 行の一覧は fleetVal
+    // （上書き済み）で描くので、ここが素のままだと「行は接続OK・タイルは⚠️」という
+    // 同じ画面での食い違いが出る（実装レビューで指摘）。registeredNow の**行の追加**を
+    // 混ぜない、という上の注意はそのまま——追加はここでは適用しない
+    var ov = overlayStateOnFleet(
+      { externalSources: rawFleetVal("externalSources") || [] },
+      LIVE_STATE,
+      String(rawFleetVal("generatedAt") || ""),
+    );
     return {
       capabilities: rawFleetVal("capabilities") || {},
       internals: rawFleetVal("internalSources") || [],
-      externals: rawFleetVal("externalSources") || [],
+      externals: ov.externalSources || [],
       pipes: rawFleetVal("pipelines") || [],
     };
   }
@@ -5107,6 +5193,9 @@ ${nextStateDelay.toString()}
         var v = probeVerdict(s, j);
         gateEl.className = "ds-gate " + v.cls;
         gateEl.textContent = v.text;
+        // 古い「X分前に確認」を残さない（下に出るいまの結果と、時刻が2つ並ぶ）
+        var oldAt = card.querySelector(".ds-gate-at");
+        if (oldAt) oldAt.remove();
         // 不足スコープは**状態の語のすぐ横**に出す（「権限が足りません：groups:read」）。
         // 本文へ落とすと、いちばん要る情報が説明文に埋もれる
         if (j.gate === "missing_scope") {
@@ -6081,7 +6170,7 @@ ${nextStateDelay.toString()}
       // 返ってきて**足りない行として一覧に出る**——さらに「まとめ表がまだ答えていない行」が
       // できるので、まとめ表が追いつくまで取り直しが続く。
       // 取得中なら何もしない（応答後に catchUpOpenCaps が拾う）
-      if (j.changed !== false) loadLiveFleet();
+      if (j.changed !== false) loadLiveFleet(true);
       // **何も変わっていないなら「追加した」と見せない。** 既に同じものが入っていた場合
       // サーバは 200 + changed:false を返す。ここを res.ok だけで判定すると
       // 「変更はありません」と言いながら追加した行が出る
@@ -6530,6 +6619,10 @@ ${nextStateDelay.toString()}
     // **「接続を確認」が押されたら、この要素を live の判定で置き換える**（probeSection）。
     // 2つの判定を同じ行に並べないため、参照を持っておく
     var gateEl = el("span", { class: "ds-gate " + gate.cls, text: gate.text });
+    // 状態ストア由来の判定には「いつ確かめたか」を必ず添える
+    var gateAtEl = s.gateAt
+      ? el("span", { class: "ds-gate-at", text: rtAgo(s.gateAt) + "に確認" })
+      : null;
     var card = el("div", { class: "ds-card" }, [
       el("div", { class: "ds-card-main" }, [
         el("span", { class: "ds-icon", text: meta.icon }),
@@ -6540,6 +6633,7 @@ ${nextStateDelay.toString()}
       badges,
       gateEl,
     ]);
+    if (gateAtEl) card.appendChild(gateAtEl);
     // チャンネルの登録解除。書き込み経路（intake Lambda）がある案件でだけ出す。
     // **Slack だけ。** github は登録ファイルが別（dev-sources.json）で、下のトグルと
     // 「登録から削除」が担う。
@@ -6805,7 +6899,19 @@ ${nextStateDelay.toString()}
    * （しかも次の取り直しは未確定の行が無いので張られない＝二度と追いつかない）。
    */
   var liveFleetSentCaps = [];
-  function loadLiveFleet() {
+  /**
+   * 次の取得で据え置きを迂回するか（保存が効いた直後の1回だけ）。
+   *
+   * 🔴 **通常のポーリングには付けない。** サーバの60秒の据え置きは、GitHub の枠を
+   * 守るために効かせたままにする（枠は組織で共有していて、枯渇すると設定変更・
+   * Amplifyビルド・Backlog同期が同時に止まる）。
+   *
+   * **取得中に保存が終わった分を取りこぼさない。** 取得中の loadLiveFleet は何もせずに
+   * 戻るので、旗として残し、応答後にもう一度取りに行く
+   */
+  var freshWanted = false;
+  function loadLiveFleet(fresh) {
+    if (fresh) freshWanted = true;
     if (liveFleetRequested) return;
     var intake = SITE.intake;
     if (!intake) return;
@@ -6814,13 +6920,17 @@ ${nextStateDelay.toString()}
     if (typeof fetch !== "function") return;
     liveFleetRequested = true;
     liveFleetSentCaps = OPEN_CAPS.slice();
+    var useFresh = freshWanted;
+    freshWanted = false;
     var pfx = intake.url + (intake.url.indexOf("?") === -1 ? "?" : "&");
     fetch(pfx + "op=state&t=" + encodeURIComponent(intake.token), {
       method: "POST",
       headers: { "content-type": "application/json" },
       // **開いている能力を伝える。** 知らない値は Lambda 側が黙って捨てる（断らない）ので、
       // 画面と Lambda の版が食い違っても接続状況の表示は消えない
-      body: JSON.stringify({ caps: OPEN_CAPS }),
+      // **保存直後だけ据え置きを迂回させる。** これが無いと、サーバが保存前の答えを返し、
+      // 足りない行が0件になって取り直しも張られない＝手で再読み込みするまで追いつかない
+      body: JSON.stringify(useFresh ? { caps: OPEN_CAPS, fresh: true } : { caps: OPEN_CAPS }),
     })
       // **非2xxも失敗として扱う。** ビューアとLambdaの版は独立に進むので、
       // op=state 未対応のLambdaは 400 を返す。通信エラーだけを見ていると空表示になる。
@@ -6833,6 +6943,7 @@ ${nextStateDelay.toString()}
         // （＝立ち上げ直後＝この仕組みが一番効く案件）は下で throw するので、
         // ここより後に置くと**その案件だけ素通り**する
         LIVE_REGISTERED = sanitizeRegisteredNow(j && j.registeredNow);
+        LIVE_STATE = sanitizeStateNow(j && j.stateNow);
         MERGED = null;
         if (!j || !j.fleet) throw new Error("no fleet");
         LIVE_FLEET = normalizeFleet(j.fleet);
@@ -6847,6 +6958,8 @@ ${nextStateDelay.toString()}
         repaintOpenLists();
         // まだ足りない行があるなら取り直しを続ける（無くなったら止める）
         refreshStateTimer();
+        // 取得中に保存が効いていたら、迂回付きでもう一度取りに行く
+        if (freshWanted) loadLiveFleet();
         catchUpOpenCaps();
       })
       .catch(function () {
@@ -6861,6 +6974,8 @@ ${nextStateDelay.toString()}
         // **失敗しても取り直しは続ける**（期限が来れば止まる）。ここで止めると、
         // 一時的な失敗のあと画面が二度と追いつかない
         refreshStateTimer();
+        // 取得中に保存が効いていたら、迂回付きでもう一度取りに行く
+        if (freshWanted) loadLiveFleet();
         catchUpOpenCaps();
       });
   }

package/package.json

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