@takagaki/cortex-decisions-viewer 0.12.59 → 0.12.61

package/dist/render.js

@@ -19,6 +19,7 @@ exports.overlayStateOnFleet = overlayStateOnFleet;
 exports.chatRowsToAdd = chatRowsToAdd;
 exports.gchatRowsToAdd = gchatRowsToAdd;
 exports.overlayGchatNotify = overlayGchatNotify;
+exports.overlaySlackNotify = overlaySlackNotify;
 exports.devRowsToAdd = devRowsToAdd;
 exports.markExcludedPendingRows = markExcludedPendingRows;
 exports.designRowsToAdd = designRowsToAdd;
@@ -887,6 +888,11 @@ const REGISTERED_CAP_NAMES = {
  * `orgUnknown` で同じ穴を1つ埋めた（0.12.1）のに、同族の3欄が残っていた。
  */
 const REGISTERED_FIELDS = {
+    // ⚠ **自動で入った行の印を足し戻さない（オーナー指示 2026-09-12・#592）。** #589 の初版は
+    // 札の語を出し分けるためにその欄（`notifyAuto`）を通していたが、1 日で撤回された
+    // ——自動で入るのが既定の動きなので、画面で人が設定した行と区別する意味が無い。
+    // この定数はクライアントへ `JSON.stringify` で渡る（コメントは付いて行かない）ので、
+    // 消した欄の名前をここに残しても画面には出ない
     chat: ["ref", "name", "workspace", "orgUnknown", "origin", "goldState", "notify"],
     // notify（#572）: スペースも通知先にできる。真偽値の扱いは chat の notify と同じ篩
     "google-chat": ["ref", "name", "notify"],
@@ -975,6 +981,9 @@ function sanitizeRegisteredNow(v) {
                 // `r.orgUnknown === true`）ので、false を載せても意味は同じ——載せないほうが
                 // 「印が無い」と「印が false」の2通りを画面に持ち込まない。
                 // 例外はこの2欄だけ——他の欄まで真偽値を通すと、許可リストが篩の意味を失う
+                // （#589 で足した自動設定の印は #592 で落とした。上の許可リストの項を参照）
+                // ⚠ この行はクライアントへ丸ごと流し込まれる。**印の欄の名前をここに書かない**
+                // ——画面にその語が 1 度も出ないことを notify-auto-gone.test.mjs が見張っている
                 if (f === "orgUnknown" || f === "notify") {
                     if (fv === true)
                         item[f] = true;
@@ -1264,6 +1273,49 @@ 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;
+}
 /**
  * 開発の行で、まとめ表がまだ知らないもの。**鍵は `owner/repo` の生文字列。**
  *
@@ -1670,6 +1722,8 @@ 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);
     const add = registeredRowsToAdd(v, reg);
     if (add.externalSources.length)
         out.externalSources = (out.externalSources || []).concat(add.externalSources);
@@ -5075,6 +5129,7 @@ ${sanitizeRegisteredNow.toString()}
 ${chatRowsToAdd.toString()}
 ${gchatRowsToAdd.toString()}
 ${overlayGchatNotify.toString()}
+${overlaySlackNotify.toString()}
 ${devRowsToAdd.toString()}
 ${markExcludedPendingRows.toString()}
 ${designRowsToAdd.toString()}
@@ -11248,6 +11303,12 @@ ${calendarAddUrl.toString()}
    * 「通知先」の札。**組み立てる場所を1つにする**——下のトグルで後から付け外しするので、
    * 描画時と押した後で別々に作ると、押した行だけ見た目の違う札が並ぶ。
    */
+  /**
+   * ⚠ **自動で入った行にも同じ札を出す（オーナー指示 2026-09-12・#592）。** #589 の初版は
+   * 自動で入った行だけ札の語を変え、外し方を hover で説明していたが、1 日で撤回された
+   * ——自動で入るのが既定の動きなので、画面で区別する意味が無い。外したい人はどちらの行でも
+   * 「通知を止める」を押せばよい。**出し分けを足し戻さない**（notify-auto-gone.test.mjs が番人）
+   */
   function notifyBadge() {
     return el("span", { class: "badge ds-notify", text: "通知先" });
   }

package/package.json

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