@takagaki/cortex-decisions-viewer 0.10.2 → 0.10.3

package/dist/build.js

@@ -166,6 +166,10 @@ async function readFleetSources(repoRoot) {
             goldState: s.goldState != null ? String(s.goldState) : undefined,
             notify: typeof s.notify === "boolean" ? s.notify : undefined,
             gate: s.gate != null ? String(s.gate) : undefined,
+            // gate が missing_scope のときに Slack が名指しした不足スコープ（段B）。
+            // **列挙し忘れると静かに消える**（この対応は素通しではなく明示の許可リスト）。
+            // 落とすと「権限が足りません」とだけ出て、何を頼めばよいか画面から分からない
+            needed: s.needed != null ? String(s.needed) : undefined,
             // どこから来たか（derived / explicit / both）。**削除を出してよいかの判断に要る。**
             // 自動導出されたものは登録ファイルから消しても翌日また現れるので、
             // explicit のときだけ削除ボタンを出す。

package/dist/render.js

@@ -601,7 +601,20 @@ function sanitizeStateNow(v) {
 /**
  * ストアの「届くか」を行に重ねる。**まとめ表より新しいときだけ**——古い確認結果で
  * 新しい実測を潰さない（時刻の比較は ISO 文字列の辞書順で足りる）。
- * 対象はチャットの行だけ（段A。確認ボタンが書くのがチャットだけのため）。
+ *
+ * 対象はチャットと開発の行（段B。夜間チェックがこの2つを書く）。
+ *
+ * 🔴 **突き合わせの鍵は能力ごとに違う。**
+ *   - chat … ワークスペース + チャンネルID。チャンネルIDはワークスペースの中でしか
+ *     一意でないので、IDだけで突き合わせると別ワークスペースの同IDを同じものと見る
+ *   - dev  … owner/repo の生文字列（devRowsToAdd と同じ）。**ワークスペースは持たない。**
+ *     同じリポジトリの github-issues と github-discussions は**同じ行**を指すので、
+ *     2つの行に同じ判定が載るのが正しい（書く側が SK で dedupe している）
+ *
+ * **needed（足りないスコープ）も載せる。** 載せないと、夜間が missing_scope を
+ * 書いた行だけスコープ名が消え、「権限が足りません」とだけ出て何を直せばよいか
+ * 分からない状態に戻る。上書きするときは空でも必ず載せる——夜間の古い needed が
+ * 新しい判定の横に残ると、直したはずのスコープ名がいつまでも出る
  */
 function overlayStateOnFleet(view, rows, generatedAt) {
     const v = view || {};
@@ -609,21 +622,33 @@ function overlayStateOnFleet(view, rows, generatedAt) {
         return v;
     const byKey = {};
     for (const r of rows) {
-        if (r.cap !== "chat" || r.question !== "reachable")
+        if (r.question !== "reachable")
             continue;
-        byKey[String(r.workspace || "") + "\t" + String(r.ref)] = r;
+        if (r.cap === "chat")
+            byKey["chat\t" + String(r.workspace || "") + "\t" + String(r.ref)] = r;
+        else if (r.cap === "dev")
+            byKey["dev\t" + String(r.ref)] = r;
     }
     let changed = false;
     const externals = v.externalSources.map((s) => {
-        if (!s || s.type !== "slack" || !s.ref)
+        if (!s || !s.ref)
             return s;
-        const hit = byKey[String(s.workspace || "") + "\t" + String(s.ref)];
+        let hit = null;
+        if (s.type === "slack")
+            hit = byKey["chat\t" + String(s.workspace || "") + "\t" + String(s.ref)];
+        else if (String(s.type || "").indexOf("github") === 0)
+            hit = byKey["dev\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 Object.assign({}, s, {
+            gate: hit.status,
+            gateAt: hit.at,
+            gateBy: hit.by,
+            needed: hit.needed || "",
+        });
     });
     return changed ? Object.assign({}, v, { externalSources: externals }) : v;
 }
@@ -5486,6 +5511,12 @@ ${nextStateDelay.toString()}
         // 古い「X分前に確認」を残さない（下に出るいまの結果と、時刻が2つ並ぶ）
         var oldAt = card.querySelector(".ds-gate-at");
         if (oldAt) oldAt.remove();
+        // 🔴 **古いスコープ名の札も必ず消してから出し直す。** 行には夜間の判定に添えた
+        // 札が出ていることがあり（段B）、置き換えた「接続OK」の横に古いスコープ名が
+        // 残ると、同じ行が2つのことを言う——この節が何度も直してきた形そのもの。
+        // 押し直すたびに札が増えていくのも、まとめてここで断つ
+        var oldScopes = card.querySelectorAll(".ds-probe-scope");
+        for (var oi = 0; oi < oldScopes.length; oi++) oldScopes[oi].remove();
         // 不足スコープは**状態の語のすぐ横**に出す（「権限が足りません：groups:read」）。
         // 本文へ落とすと、いちばん要る情報が説明文に埋もれる
         if (j.gate === "missing_scope") {
@@ -6939,6 +6970,17 @@ ${nextStateDelay.toString()}
       badges,
       gateEl,
     ]);
+    // 足りないスコープは**状態の語のすぐ横**に出す（「権限が足りません：groups:read」）。
+    //
+    // **押した瞬間の結果と同じ見た目にする**（probeSection が同じ class を付けている）。
+    // 出どころは2つあるが、読む人にとっては同じ情報:
+    //   - 夜間チェックが書いた needed（fleet-status.json 経由）
+    //   - 状態ストアの行が上書きした needed（overlayStateOnFleet 経由）
+    // ここが無いと「権限が足りません」とだけ出て、**どのスコープを頼めばよいか
+    // 画面から分からない**——押して確かめた人にしか分からない状態に戻る
+    if (s.gate === "missing_scope" && s.needed) {
+      card.appendChild(el("code", { class: "ds-probe-scope", text: String(s.needed) }));
+    }
     if (gateAtEl) card.appendChild(gateAtEl);
     // チャンネルの登録解除。書き込み経路（intake Lambda）がある案件でだけ出す。
     // **Slack だけ。** github は登録ファイルが別（dev-sources.json）で、下のトグルと

package/package.json

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