@takagaki/cortex-decisions-viewer 0.10.2 → 0.10.4

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;
 }
@@ -2143,6 +2168,8 @@ a.ds-name:hover { text-decoration: underline; }
 .rt-card { margin-top: 10px; }
 .rt-steps { margin: 0; padding-left: 1.4em; color: var(--text); font-size: 13px; line-height: 1.9; }
 .rt-steps li { margin: 2px 0; }
+/* 通知イベントのチェック名（10個）。1行に並べると読めないので、種別ごとに折る */
+.rt-steps li ul { margin: 2px 0 6px; padding-left: 1.1em; list-style: disc; color: var(--muted); line-height: 1.7; }
 /* URLは長いので、入力欄を伸ばしてボタンを右に置く（折り返しても崩れないように wrap） */
 .rt-url-row { display: flex; flex-wrap: wrap; align-items: center; gap: 8px; }
 .rt-copy { margin: 6px 0 10px; }
@@ -5486,6 +5513,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 +6972,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）で、下のトグルと
@@ -7386,7 +7430,20 @@ ${nextStateDelay.toString()}
       el("ol", { class: "rt-steps" }, [
         el("li", { text: "Backlog のプロジェクト設定 → インテグレーション → Webhook を開く" }),
         el("li", { text: "「WebHook URL」に下の案件専用URLを貼り付ける" }),
-        el("li", { text: "通知イベントで「課題の追加」「課題の更新」「課題にコメント」等にチェックを入れて保存する" }),
+        // **チェック名は受信側が処理する10種を全部並べる。** 「等」で省くと、担当者は
+        // 課題の3つだけにチェックして「設定した」と思い、Wikiとドキュメントの更新は
+        // 夜間まで届かない（画面は稼働中と出るので、欠けていることに気づけない）。
+        // ここを増減するときは受信側（backlog-webhook/logic.mjs の HANDLED_TYPES）と
+        // 揃える——ずれると realtime-sync-card.test.mjs が赤くなる。
+        // **「他はチェック不要」とは言い切らない。** 未対応の type（ドキュメントの更新等）は
+        // 「無視した」ログから見つける設計なので、外させると発見の経路が塞がる
+        el("li", { text: "通知イベントで次の10個にチェックを入れて保存する（他のイベントはチェックしても取り込まれません）" }, [
+          el("ul", {}, [
+            el("li", { text: "課題:「課題の追加」「課題の更新」「課題にコメント」「課題の削除」「課題をまとめて更新」" }),
+            el("li", { text: "Wiki:「Wikiページを追加」「Wikiページを更新」「Wikiページを削除」" }),
+            el("li", { text: "ドキュメント:「ドキュメントの追加」「ドキュメントの削除」" }),
+          ]),
+        ]),
       ]),
       el("div", { class: "rt-url-row" }, [field, copyBtn]),
       msg,

package/package.json

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