@takagaki/cortex-decisions-viewer 0.11.0 → 0.11.1

package/dist/render.js

@@ -26,6 +26,7 @@ exports.sanitizeCapabilitiesNow = sanitizeCapabilitiesNow;
 exports.sanitizeGithubOwnersNow = sanitizeGithubOwnersNow;
 exports.splitNeededParts = splitNeededParts;
 exports.ingestedAtOf = ingestedAtOf;
+exports.usageFactMissing = usageFactMissing;
 exports.renderConnectionMap = renderConnectionMap;
 /** SiteDataを単一の自己完結HTML（埋め込みデータ+CSS+JS）にレンダリングする */
 function renderSite(site) {
@@ -1291,6 +1292,47 @@ function ingestedAtOf(rows, capId) {
     }
     return best;
 }
+/**
+ * 「取り込みの事実」でタイルと線を決める能力（画面の能力名 → ストアの能力名）。
+ *
+ * **この2つだけ。** 会議と共有資料は中央 Apps Script が取り込むので、設定が済んでいても
+ * （capabilities が `on` でも）**一度も取り込まれていない**ことが普通に起きる。他の4能力は
+ * 判定の材料が違う（夜間の reachable・パイプラインの成否）ので、ここに足さないこと——
+ * 足した瞬間、実績の無い案件が一斉に「未使用」になる。
+ */
+const FACT_CAPS = { "会議": "meetings", "共有資料": "materials" };
+/**
+ * その能力が「設定は済んでいるのに、まだ一度も取り込めていない」か（#235 の表示層）。
+ *
+ * 🔴 **`seen`（応答に `stateNow` の欄があったか）と `rows`（その中身）は別物。**
+ * サーバは**読めたなら0件でも欄を付ける**ので、この2つで3つの状態が区別できる:
+ *
+ *   - `seen === false` … 分からない（古い Lambda・ストアの読み取りに失敗）→ **偽**。
+ *     従来表示のままにする。分からないことを「未使用」と言い切らない
+ *   - `seen === true` かつ行が0件 … 読めて空＝**まだ一度も取り込めていない**→ 真。
+ *     初期化直後の案件がここに来る（この機能が一番効いてほしい案件）
+ *   - `seen === true` かつ行がある … その能力の取り込み実績で決める
+ *
+ * **`rows` の空と `seen` を混ぜないこと。** `sanitizeStateNow` は0件を `null` に畳むので、
+ * `rows` だけを見ると「読めて空」が「分からない」に化け、**まさに未使用の案件でだけ
+ * 機能しない**（サーバ側を直した意味が消える）。
+ *
+ * **成功の事実は `ingestedAtOf` に一本化する。** あちらは `lastOkAt` も見るので、
+ * 「一度取り込めた後に失敗した」能力を未使用に落とさない。判定を写経すると、そこがずれる。
+ *
+ * **クロージャを持たない純関数に保つこと**（`.toString()` でクライアントへ渡す。
+ * `FACT_CAPS` は CLIENT_JS 側にも `var` で流し込んである）。
+ */
+function usageFactMissing(cap, rows, seen) {
+    var capId = FACT_CAPS[cap];
+    if (!capId)
+        return false;
+    if (!seen)
+        return false;
+    if (!Array.isArray(rows) || !rows.length)
+        return true;
+    return ingestedAtOf(rows, capId) === "";
+}
 function capabilityState(cap, ctx) {
     // **能力→パイプラインの対応はここに持つ。** 呼び出し側に渡させると、
     // クライアント側（この関数を .toString() で受け取る側）が同じ表を書き写すことになり、
@@ -1536,8 +1578,19 @@ function renderConnectionMap(site) {
         //
         // cm-warn は引く。**壊れているのは「繋がったうえで失敗している」状態**で、
         // 経路自体は存在する（線を消すと、設定が無いのと区別が付かなくなる）。
+        //
+        // **`data-cm-cap` は線を後から取り消すための取っ手（#235 の表示層）。** この図は
+        // ビルド時に焼くので、**取り込みの事実（stateNow）はここでは分からない**——分かるのは
+        // ライブ応答が届いた後だけ。届いてから会議・共有資料の線を外せるように、線に
+        // 能力名を持たせておく（外す側は applyUsageFactToConnMap）。
+        // 位置で数えさせないこと——線は ok/warn の能力ぶんしか積まれないので、
+        // 並び順は案件ごとに変わる。
+        //
+        // 🔴 **`data-cap` は使わない。** あちらは能力ブロック（`.datasources` の中）が
+        // 既に名乗っていて、図はページの**上**にあるので `[data-cap="会議"]` で引くと
+        // 図の線のほうが先に当たる。実際、最初にそう書いてテストが12件落ちた。
         if (state === "cm-ok" || state === "cm-warn") {
-            links.push(`<path class="cm-link ${state}" d="M ${cx} 78 C ${cx} 152, ${REPO_X} 162, ${REPO_X} 238"></path>`);
+            links.push(`<path class="cm-link ${state}" data-cm-cap="${escapeHtml(c.cap)}" d="M ${cx} 78 C ${cx} 152, ${REPO_X} 162, ${REPO_X} 238"></path>`);
         }
     });
     // **中央ノードは GitHub のリポジトリ名そのもの（オーナー指示・2026-08-12）。**
@@ -2631,6 +2684,8 @@ ${sanitizeCapabilitiesNow.toString()}
 ${sanitizeGithubOwnersNow.toString()}
 ${splitNeededParts.toString()}
 ${ingestedAtOf.toString()}
+var FACT_CAPS = ${JSON.stringify(FACT_CAPS)};
+${usageFactMissing.toString()}
   // 「はじめに」セクションのチュートリアルデータ（ビルド時に埋め込み。ボタンURL・チャレンジ4の決定は解決済み）
   var TUTORIAL = (function () {
     var elx = document.getElementById("tutorial-data");
@@ -4075,6 +4130,15 @@ ${ingestedAtOf.toString()}
   var LIVE_REGISTERED = null;
   // 接続状態ストアの行（届くか・取り込めたか）。op=state の stateNow から
   var LIVE_STATE = null;
+  /**
+   * 応答に 「stateNow」 の欄があったか（＝サーバがストアを読めたか）。
+   *
+   * 🔴 **LIVE_STATE が null かどうかでは代用できない。** sanitize は0件を null に畳むので、
+   * 「読めて0件」と「そもそも読めていない」が同じ値になる。サーバは**読めたなら0件でも
+   * 欄を付ける**ので、区別できるのはこの旗だけ——潰すと、初期化直後の案件
+   * （＝会議・共有資料の「未使用」表示が一番効いてほしい案件）でだけ機能が死ぬ。
+   */
+  var LIVE_STATE_SEEN = false;
   /**
    * 鍵がいま登録されているか（op=state の secretsNow・列2）。取得前・取得失敗時は null。
    *
@@ -5364,15 +5428,30 @@ ${ingestedAtOf.toString()}
      *   節を作り直せない場面（モーダルを開いたまま）でも、**届いた一覧で
      *   引き直す**ために同じ関数を使う——写経すると片方だけ古くなる
      */
-    function tileSub(cap, st, derived, adapter, ins, srcs) {
+    /**
+     * @param noFact 「設定は済んでいるが、まだ一度も取り込めていない」と言い切れるか
+     *   （#235 の表示層）。**呼び出し側が渡す**——ここで LIVE_STATE から引き直すと、
+     *   タイルの色（st）を決めた材料と 2 回引くことになり、片方だけずれる余地ができる
+     */
+    function tileSub(cap, st, derived, adapter, ins, srcs, noFact) {
       var internal = ins.filter(function (s) { return s.kind === cap; })[0];
       var live = srcs.filter(function (s) { return externalCapability(s.type) === cap; });
       var parts = [];
+      // 🔴 **取り込みの実績が無いなら、まずそれを言う（#235 の表示層）。**
+      // 設定は済んでいる（capabilities は on）ので「未接続」ではないし、設定ミスでもないので
+      // 警告色にもしない（#230 の決定どおり中立表示）。ここを通るのは会議・共有資料だけで、
+      // しかも呼び出し側が st === "ok" のときにしか立てない——OFF・設定異常・同期失敗の
+      // 語を、この一文で上書きしないため。
+      //
+      // 実績が無い＝ st は "idle" に降りているので、下の「件数」「最終同期」は自動で消える
+      // （繋がっていないものに読み取り範囲と同期時刻を並べない、という既存の規律に乗る）
+      if (noFact) {
+        parts.push((CAP_STATE_MARK[st] || "") + "未使用（まだ取り込みがありません）");
       // **導出が on 以外なら、導出の語を優先する（#125②）。** broken は状態としては
       // 警告（⚠️）だが、原因は「設定そのものが壊れている」であって同期の失敗ではない。
       // 下の warn の文（同期が失敗しています／N件が読めていません）を出すと、
       // 直す場所が違うところを指す
-      if (derived && derived !== "on") {
+      } else if (derived && derived !== "on") {
         parts.push((CAP_STATE_MARK[st] || "") + (CAP_STATE_TEXT[derived] || "状態が判定できません"));
       } else if (st === "warn") {
         if (live.length) {
@@ -5489,6 +5568,15 @@ ${ingestedAtOf.toString()}
         // カタログにあるが導出が答えていないアダプタ（今日は発生しない。#107 の前方互換）
         ? "idle"
         : capabilityState(cap, stateCtx());
+      // 🔴 **「接続OK」と言うのは、取り込めた事実があるときだけ（#235 の表示層）。**
+      // 会議・共有資料は中央 Apps Script が取り込むので、設定が済んでいるのに一度も
+      // 取り込まれていない案件が普通にある。そこへ「● 接続OK」と出すと、**画面が
+      // 動いていないものを動いていると言う**——顧客がそれを信じて資料を置かなくなる。
+      //
+      // **降ろすのは ok からだけ。** warn（繋がったうえで失敗している）は隠さないし、
+      // idle / off はもともと繋がっていると言っていない。
+      var noFact = st === "ok" && usageFactMissing(cap, LIVE_STATE, LIVE_STATE_SEEN);
+      if (noFact) st = "idle";
       // **ロゴはタイルの左に独立して置く。** 名前の行に入れると名前の高さに
       // 縛られて小さくなる。ロゴで能力を見分けられるのが一覧の主な用途
       var logo = ADAPTER_ICONS[adapter];
@@ -5506,7 +5594,7 @@ ${ingestedAtOf.toString()}
       var pend = capPending(cap);
       var subEl = el("div", {
         class: "ds-tile-sub " + (pend ? DS_GATE_PENDING.cls : "ds-gate-" + st),
-        text: pend ? DS_GATE_PENDING.text : tileSub(cap, st, derived, adapter, internals, sources),
+        text: pend ? DS_GATE_PENDING.text : tileSub(cap, st, derived, adapter, internals, sources, noFact),
       });
       // **反映が届いたら、届いたスナップショットから引き直す。** 下地へ戻すと
       // 「⏳ 反映中」→「⦿ 未接続」になる——データが正しくなったあとに嘘をつく。
@@ -5519,10 +5607,14 @@ ${ingestedAtOf.toString()}
         var stNow = dNow === "unset" && capabilityDisplayState(cap, capsNow) !== "unset"
           ? "idle"
           : capabilityState(cap, stateCtx());
+        // 実績の判定も、タイルを作るときと同じ式で引き直す（#235 の表示層）。
+        // ここを省くと、反映が届いた瞬間だけ「● 接続OK」へ戻る
+        var noFactNow = stNow === "ok" && usageFactMissing(cap, LIVE_STATE, LIVE_STATE_SEEN);
+        if (noFactNow) stNow = "idle";
         var pendNow = capPending(cap);
         n.className = "ds-tile-sub " + (pendNow ? DS_GATE_PENDING.cls : "ds-gate-" + stNow);
         return {
-          text: pendNow ? DS_GATE_PENDING.text : tileSub(cap, stNow, dNow, adapter, insNow, srcsNow),
+          text: pendNow ? DS_GATE_PENDING.text : tileSub(cap, stNow, dNow, adapter, insNow, srcsNow, noFactNow),
           hidden: false,
         };
       });
@@ -7958,6 +8050,8 @@ ${ingestedAtOf.toString()}
         // ここより後に置くと**その案件だけ素通り**する
         LIVE_REGISTERED = sanitizeRegisteredNow(j && j.registeredNow);
         LIVE_STATE = sanitizeStateNow(j && j.stateNow);
+        // **欄の有無をそのまま覚える。** sanitize を通した後では「読めて0件」が消える
+        LIVE_STATE_SEEN = !!(j && Array.isArray(j.stateNow));
         // 🔴 **鍵・能力・組織も fleet より先に読む（工程1.5）。** まとめ表を
         // まだ持たない案件は下で throw するので、ここより後に置くと
         // **立ち上げ直後の案件＝この修正が一番効く案件**だけ素通りする
@@ -7971,6 +8065,10 @@ ${ingestedAtOf.toString()}
         liveFleetRequested = false;
         MERGED = null;
         refreshDataSources();
+        // 🔴 **図の降格は refreshDataSources の外で呼ぶ（#235 の表示層）。** あちらは
+        // モーダルが開いている間・入力中は何もせずに戻るので、中に入れると
+        // **モーダルを開いたまま最初の応答が届いた回だけ、図が「接続OK」のまま残る**
+        applyUsageFactToConnMap();
         // **モーダルを開いたままでも「反映中」だけは追いつかせる。** refreshDataSources は
         // モーダルが開いている間・入力中は何もしない（下地を入れ替えると開いているモーダルの
         // 中身が切り離される）ので、局所的に差し替える口をここに置く
@@ -8485,6 +8583,55 @@ ${ingestedAtOf.toString()}
     });
     app.appendChild(wrap);
     updateConnMapAsOf();
+    applyUsageFactToConnMap();
+  }
+
+  /**
+   * 取り込みの実績が無い能力を、接続マップから降ろす（#235 の表示層）。
+   *
+   * **図はビルド時に焼かれている。** 焼いた時点では 「stateNow」 を持っていないので、
+   * 会議・共有資料は capabilities が 「on」 なだけで「● 接続OK・線あり」になる。
+   * ライブ応答が届いてから、実績の無い2能力だけを**その場で**降ろす。
+   *
+   * - 線は消す（「data-cm-cap」 で引く。位置で数えると案件ごとに並びが変わる）
+   * - ノードは 「cm-off」（灰・破線）にして、3行目を「未使用」にする。図の語彙は
+   *   renderConnectionMap の off と同じにする——同じ画面で図とタイルの語を割らない
+   *
+   * **降ろすのは 「cm-ok」 からだけ**（タイル側と同じ規律。「cm-warn」 は隠さない）。
+   *
+   * ⚠ **戻す道は無い。** 画面を開いたまま初回の取り込みが完走した場合、下の一覧だけが
+   * 「接続OK」に変わり、図は次に開くまで「未使用」のまま。図の遅れは
+   * 「この図は◯◯時点です（下の一覧は最新）」で既に断ってあるので、ここでは受け入れる
+   * ——「cm-ok」 を条件にしているので、何度呼んでも二重に降ろすことはない。
+   */
+  function applyUsageFactToConnMap() {
+    var wrap = app.querySelector(".conn-map");
+    if (!wrap) return;
+    for (var cap in FACT_CAPS) {
+      if (!Object.prototype.hasOwnProperty.call(FACT_CAPS, cap)) continue;
+      if (!usageFactMissing(cap, LIVE_STATE, LIVE_STATE_SEEN)) continue;
+      var node = wrap.querySelector('[data-target="ds-' + cap + '"]');
+      if (!node || node.getAttribute("class").indexOf("cm-ok") < 0) continue;
+      node.setAttribute("class", "cm-node cm-off");
+      var sync = node.querySelector(".cm-sync");
+      if (!sync) {
+        // 3行目そのものが無いノード（最終同期を持たないまま ok になっていた）。
+        // **状態を書く場所を作る**——灰色になっただけで理由が読めないのは、
+        // 「名前だけ出て状態が出ない」という #124 の症状に戻ること
+        var label = node.querySelector(".cm-label");
+        sync = document.createElementNS("http://www.w3.org/2000/svg", "text");
+        sync.setAttribute("class", "cm-sync");
+        sync.setAttribute("x", label ? label.getAttribute("x") : "0");
+        sync.setAttribute("y", "67");
+        sync.setAttribute("text-anchor", "middle");
+        node.appendChild(sync);
+      }
+      // 時刻の整形（renderConnMap の [data-ts]）が後から書き戻さないよう、印ごと外す
+      sync.removeAttribute("data-ts");
+      sync.textContent = "未使用";
+      var link = wrap.querySelector('.cm-link[data-cm-cap="' + cap + '"]');
+      if (link && link.parentNode) link.parentNode.removeChild(link);
+    }
   }
 
   /**

package/package.json

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