@takagaki/cortex-decisions-viewer 0.12.59 → 0.12.60

package/dist/render.js

@@ -19,6 +19,8 @@ exports.overlayStateOnFleet = overlayStateOnFleet;
 exports.chatRowsToAdd = chatRowsToAdd;
 exports.gchatRowsToAdd = gchatRowsToAdd;
 exports.overlayGchatNotify = overlayGchatNotify;
+exports.overlaySlackNotify = overlaySlackNotify;
+exports.overlayNotifyAuto = overlayNotifyAuto;
 exports.devRowsToAdd = devRowsToAdd;
 exports.markExcludedPendingRows = markExcludedPendingRows;
 exports.designRowsToAdd = designRowsToAdd;
@@ -887,9 +889,11 @@ const REGISTERED_CAP_NAMES = {
  * `orgUnknown` で同じ穴を1つ埋めた（0.12.1）のに、同族の3欄が残っていた。
  */
 const REGISTERED_FIELDS = {
-    chat: ["ref", "name", "workspace", "orgUnknown", "origin", "goldState", "notify"],
+    // notifyAuto（#589）: 接続が確認できたので自動で通知先になった行の印。
+    // **`notify` と対で読む**——サーバは通知先である行にしか付けない
+    chat: ["ref", "name", "workspace", "orgUnknown", "origin", "goldState", "notify", "notifyAuto"],
     // notify（#572）: スペースも通知先にできる。真偽値の扱いは chat の notify と同じ篩
-    "google-chat": ["ref", "name", "notify"],
+    "google-chat": ["ref", "name", "notify", "notifyAuto"],
     dev: ["ref", "type"],
     // 🔴 **意図的に外した開発リポジトリ（#391）。** サーバは前から別キーで返していたが、
     // ここに無いので篩が丸ごと落としていた。**これが読めないと「読み取り停止中」の行を
@@ -974,8 +978,9 @@ function sanitizeRegisteredNow(v) {
                 // **false はキーごと落とす。** 読み手は `=== true` で見る（`s.notify === true` /
                 // `r.orgUnknown === true`）ので、false を載せても意味は同じ——載せないほうが
                 // 「印が無い」と「印が false」の2通りを画面に持ち込まない。
-                // 例外はこの2欄だけ——他の欄まで真偽値を通すと、許可リストが篩の意味を失う
-                if (f === "orgUnknown" || f === "notify") {
+                // 例外はこの3欄だけ（#589 で `notifyAuto` を足した）——他の欄まで真偽値を通すと、
+                // 許可リストが篩の意味を失う
+                if (f === "orgUnknown" || f === "notify" || f === "notifyAuto") {
                     if (fv === true)
                         item[f] = true;
                     continue;
@@ -1190,6 +1195,10 @@ function chatRowsToAdd(externals, rows) {
         // **真偽値の印は true のときだけ載せる**（読み手は `s.notify === true`。篩の項を参照）
         if (r.notify === true)
             row.notify = true;
+        // 自動で設定された通知先の印（#589）。**足す行にも載せる**——載せないと、追加して
+        // 接続を確認した直後（一番この説明が要る瞬間）だけ「自動で設定」が出ない
+        if (r.notify === true && r.notifyAuto === true)
+            row.notifyAuto = true;
         out.push(row);
     }
     return out;
@@ -1216,6 +1225,9 @@ function gchatRowsToAdd(externals, rows) {
         // 通知先の印（#572）。真偽値の印は true のときだけ載せる（chatRowsToAdd と同じ）
         if (r.notify === true)
             row.notify = true;
+        // 自動で設定された通知先の印（#589・chatRowsToAdd と同じ条件）
+        if (r.notify === true && r.notifyAuto === true)
+            row.notifyAuto = true;
         out.push(row);
     }
     return out;
@@ -1264,6 +1276,106 @@ function overlayGchatNotify(externals, rows) {
     });
     return changed ? out : list;
 }
+/**
+ * まとめ表が既に持っている Slack の行に、live（registeredNow）が「通知先」と言う印を**足す**（#589）。
+ *
+ * 🔴 **#589 で、まとめ表を待たずに通知先が増えるようになった。** まとめ表（fleet-status）は
+ * 夜間の生成物なので、「接続を確認」で自動的に通知先になったチャンネルは**翌朝まで
+ * `notify` を持たない**——札が出ないだけでなく、トグルが「通知先にする」のままになる。
+ * **通知は届いているのに画面は通知先でないと言う**、という取り違えが最長 24 時間続く。
+ *
+ * 🔴 **足すだけで、外さない（`overlayGchatNotify` との違い）。** 画面の `registeredNow` からは
+ * 「宣言を読んだうえで通知先でなかった」と「宣言を読めなかった」が区別できない
+ * （どちらも `notify` のキーが無い。サーバ側 `buildRegisteredNow` は notify.json を読めなかった
+ * 便では**全行から印を落とす**）。外す向きにも効かせると、GitHub が不調だった 1 回の読み取りで
+ * **本物の通知先が画面から「通知先でない」に化ける**——通知の宛先を取り違えさせる方向なので、
+ * 足す向きだけに限る。外す向きが遅れる（最長 24 時間「通知先」の札が残る）のは #589 より前からの
+ * Slack の挙動で、こちらは悪化させない。Google Chat が両向きなのは、まとめ表が
+ * その欄を**そもそも持たない**ので「読めなかった」と混ざりようが無いから。
+ *
+ * 新しく足す行の印は `chatRowsToAdd` が持つ（こちらは既にある行だけを直す）。
+ *
+ * **クロージャを持たない純関数に保つこと**（`.toString()` でクライアントへ渡す）。
+ */
+function overlaySlackNotify(externals, rows) {
+    const list = externals || [];
+    if (!Array.isArray(rows))
+        return list;
+    const live = {};
+    for (const r of rows) {
+        if (!r || !r.ref || r.notify !== true)
+            continue;
+        // 鍵はワークスペース込み（チャンネルIDはワークスペースの中でしか一意でない・#104）
+        live[String(r.workspace || "") + "\t" + String(r.ref)] = true;
+    }
+    let changed = false;
+    const out = list.map(function (s) {
+        if (!s || s.type !== "slack" || !s.ref || s.notify === true)
+            return s;
+        if (live[String(s.workspace || "") + "\t" + String(s.ref)] !== true)
+            return s;
+        changed = true;
+        return Object.assign({}, s, { notify: true });
+    });
+    return changed ? out : list;
+}
+/**
+ * 「自動で設定された通知先」の印（#589）を、まとめ表の行に重ねる。
+ *
+ * 🔴 **まとめ表（fleet-status）はこの印を持たない。** 印の置き場は案件リポの
+ * `チャット/notify.json` の `autoDefault` で、それを読んで行に付けるのは中央（intake）の
+ * `registeredNow` だけ。重ねないと、**自動で通知先になった行が「通知先」としか出ず、
+ * 担当者は自分が設定した覚えのない通知先を前にして、外してよいのかも分からない**
+ * （オーナー決定 4「外せる」が画面から読めない）。
+ *
+ * **印だけを重ねる。** `notify` そのものには触らない——通知先かどうかを live で重ねるのは
+ * `overlayGchatNotify`（Google Chat）と `overlaySlackNotify`（Slack）の仕事で、**この関数より
+ * 先に走る**。ここで二重に決めると「どちらが正か」が読めなくなる。印は `notify` が立っている
+ * 行にしか付けない（サーバ側も同じ条件で付けている）。
+ *
+ * live に該当の能力のキーが無ければ（読めていない）何も変えない。
+ *
+ * **クロージャを持たない純関数に保つこと**（`.toString()` でクライアントへ渡す）。
+ */
+function overlayNotifyAuto(externals, reg) {
+    const list = externals || [];
+    if (!reg || typeof reg !== "object")
+        return list;
+    const auto = {};
+    const known = {};
+    const scan = (rows, type, withWorkspace) => {
+        if (!Array.isArray(rows))
+            return;
+        for (const r of rows) {
+            if (!r || !r.ref)
+                continue;
+            const key = type + "\t" + (withWorkspace ? String(r.workspace || "") : "") + "\t" + String(r.ref);
+            known[key] = true;
+            auto[key] = r.notifyAuto === true && r.notify === true;
+        }
+    };
+    scan(reg.chat, "slack", true);
+    scan(reg["google-chat"], "google-chat", false);
+    let changed = false;
+    const out = list.map(function (s) {
+        if (!s || !s.ref || (s.type !== "slack" && s.type !== "google-chat"))
+            return s;
+        const key = s.type + "\t" + (s.type === "slack" ? String(s.workspace || "") : "") + "\t" + String(s.ref);
+        if (!Object.prototype.hasOwnProperty.call(known, key))
+            return s;
+        const want = auto[key] === true && s.notify === true;
+        if ((s.notifyAuto === true) === want)
+            return s;
+        changed = true;
+        const row = Object.assign({}, s);
+        if (want)
+            row.notifyAuto = true;
+        else
+            delete row.notifyAuto;
+        return row;
+    });
+    return changed ? out : list;
+}
 /**
  * 開発の行で、まとめ表がまだ知らないもの。**鍵は `owner/repo` の生文字列。**
  *
@@ -1670,6 +1782,10 @@ function mergeRegisteredFleet(view, reg) {
         return out;
     // Google Chat の行の「通知先」は live が正（#572・まとめ表は持っていない）。足す前に重ねる
     out.externalSources = overlayGchatNotify(v.externalSources, reg["google-chat"]);
+    // Slack の行の「通知先」も live が正（#589・まとめ表は夜間の生成物で、自動設定に追いつかない）
+    out.externalSources = overlaySlackNotify(out.externalSources, reg.chat);
+    // 「自動で設定」の印も live が正（#589・まとめ表は持っていない）。`notify` を重ねた後に置く
+    out.externalSources = overlayNotifyAuto(out.externalSources, reg);
     const add = registeredRowsToAdd(v, reg);
     if (add.externalSources.length)
         out.externalSources = (out.externalSources || []).concat(add.externalSources);
@@ -5075,6 +5191,8 @@ ${sanitizeRegisteredNow.toString()}
 ${chatRowsToAdd.toString()}
 ${gchatRowsToAdd.toString()}
 ${overlayGchatNotify.toString()}
+${overlaySlackNotify.toString()}
+${overlayNotifyAuto.toString()}
 ${devRowsToAdd.toString()}
 ${markExcludedPendingRows.toString()}
 ${designRowsToAdd.toString()}
@@ -11248,8 +11366,19 @@ ${calendarAddUrl.toString()}
    * 「通知先」の札。**組み立てる場所を1つにする**——下のトグルで後から付け外しするので、
    * 描画時と押した後で別々に作ると、押した行だけ見た目の違う札が並ぶ。
    */
-  function notifyBadge() {
-    return el("span", { class: "badge ds-notify", text: "通知先" });
+  /**
+   * @param auto 接続が確認できたので**自動で**通知先になった行か（#589）。
+   *   語を増やしすぎない——札は「通知先（自動で設定）」の1つで、外せることと
+   *   「外したら戻さない」は hover の説明に置く（行に長い定型文を入れない）。
+   *   **押して通知先にした行には付けない**（トグルが作る札は常に素の「通知先」）
+   */
+  function notifyBadge(auto) {
+    if (auto !== true) return el("span", { class: "badge ds-notify", text: "通知先" });
+    return el("span", {
+      class: "badge ds-notify",
+      text: "通知先（自動で設定）",
+      title: "接続が確認できたので、自動でこのチャンネルを通知先にしました。外すには「通知を止める」を押してください（外したら自動では戻しません）。",
+    });
   }
 
   /**
@@ -11293,10 +11422,13 @@ ${calendarAddUrl.toString()}
         // 行の元データも合わせる。ライブの読み直しで同じ行が描き直されるまでの間、
         // ここだけが「いまどちらか」を持っている
         s.notify = next;
+        // **押した時点で「自動で設定」ではなくなる（#589）。** 人が入り切りした行に
+        // 「自動で設定」と出したままだと、外して押し直した行が自動で戻ったように見える
+        s.notifyAuto = false;
         btn.textContent = labelOf();
         var badge = badges.querySelector(".ds-notify");
         // 並びは描画時と同じ（札の先頭）。後ろに足すと、押した行だけ順番が変わる
-        if (on && !badge) badges.insertBefore(notifyBadge(), badges.firstChild);
+        if (on && !badge) badges.insertBefore(notifyBadge(false), badges.firstChild);
         if (!on && badge) badges.removeChild(badge);
       });
     });
@@ -13802,7 +13934,7 @@ ${calendarAddUrl.toString()}
         title: "このリポジトリは読み取り対象から除外されています（開発の登録ファイル dev-sources.json の exclude）。この画面からは戻せません。",
       }));
     }
-    if (s.notify === true) badges.appendChild(notifyBadge());
+    if (s.notify === true) badges.appendChild(notifyBadge(s.notifyAuto === true));
     // **どのワークスペースのチャンネルかを、必ず行に出す（#104・2026-08-20 に社内も）。**
     //
     // 折りたたみの見出しでまとめるのをやめた（オーナー指摘「複雑になっちゃってる」）ので、

package/package.json

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