@takagaki/cortex-decisions-viewer 0.12.56 → 0.12.57

package/dist/pending-ops.js

@@ -202,9 +202,17 @@ function backlogTargetOf(input) {
 function pendingIsSlackTeamId(v) {
     return /^[TE][A-Z0-9]{6,20}$/.test(String(v == null ? "" : v));
 }
-/** Backlog の接続先の鍵（render.ts の issuesRefOf と同形: ドメイン小文字 / キー大文字） */
+/**
+ * Backlog の接続先の鍵（render.ts の issuesRefOf と同形: ドメイン小文字 / キー大文字）。
+ *
+ * 🔴 **規則の正本は `infra/lambda/shared/connection-state-keys.mjs` の `issuesRefOf`**（#556）。
+ * ここも render.ts も**写し**で、ずれると仮置きの札が実測の行に重ならない
+ * （押した直後の札が消えないまま、別の行に本物が出る）。
+ * ずれを捕まえる網は `viewer/test/state-ref-parity.test.mjs`。
+ */
 function pendingIssuesRef(domain, projectKey) {
-    return String(domain == null ? "" : domain).toLowerCase() + "/" + String(projectKey == null ? "" : projectKey).toUpperCase();
+    return String(domain == null ? "" : domain).trim().toLowerCase() + "/" +
+        String(projectKey == null ? "" : projectKey).trim().toUpperCase();
 }
 // ---- エントリを作る（op → kind / action / targetId の対応表はここ 1 つ） ------------
 /**
@@ -277,8 +285,13 @@ function pendingOpOf(payload, ctx) {
             const name = pendingLabel(payload.name);
             return name ? make("slack", "update", payload.ref, { name: name }) : null;
         }
-        case "set-notify":
-            return typeof payload.value === "boolean" ? make("slack", "update", payload.ref, { notify: payload.value }) : null;
+        case "set-notify": {
+            if (typeof payload.value !== "boolean")
+                return null;
+            // op は 1 つ。ref の形（spaces/…）で Google Chat と決める（サーバの notifyRefKind と同じ・#572）
+            const kind = pendingIsSpaceRef(payload.ref) ? "google-chat" : "slack";
+            return make(kind, "update", payload.ref, { notify: payload.value });
+        }
         case "add-google-chat-space": {
             const ref = pendingGoogleChatSpaceRef(payload.url);
             // 名前は画面に欄が無い（サーバが引く）。live が来るまでは ref で出す
@@ -660,6 +673,9 @@ function pendingExternalRow(e) {
         row.type = "google-chat";
         row.name = p.name || e.targetId;
         row.ref = e.targetId;
+        // add に畳まれた update（通知先 ON）を落とさない（#572・Slack と同じ）
+        if (p.notify === true)
+            row.notify = true;
     }
     else {
         row.type = p.type || "github-issues";
@@ -734,14 +750,23 @@ function applyPendingOps(view, list) {
                 extChanged = true;
             }
             else if (e.action === "add") {
-                if (idx >= 0)
+                if (idx >= 0) {
+                    // 本物の行が先に来ている。**add に畳まれた update（通知先の向き）だけは当てる**
+                    // （#572）——畳んだまま捨てると、追加 → 5 分以内に本物の行 → 通知先 ON の順で
+                    // 押した人の札が、据え置きの live で戻る
+                    if (typeof p.notify === "boolean" && (externals[idx].notify === true) !== p.notify) {
+                        externals[idx] = Object.assign({}, externals[idx], { notify: p.notify });
+                        extChanged = true;
+                    }
                     continue;
+                }
                 externals.push(pendingExternalRow(e));
                 extChanged = true;
             }
-            else if (e.kind === "slack" && idx >= 0) {
+            else if ((e.kind === "slack" || e.kind === "google-chat") && idx >= 0) {
+                // update: Slack は name / notify、Google Chat は notify だけ（名前はサーバが引く・#572）
                 const patch = {};
-                if (typeof p.name === "string" && p.name)
+                if (e.kind === "slack" && typeof p.name === "string" && p.name)
                     patch.name = p.name;
                 if (typeof p.notify === "boolean")
                     patch.notify = p.notify;

package/dist/render.js

@@ -18,6 +18,7 @@ exports.sanitizeStateNow = sanitizeStateNow;
 exports.overlayStateOnFleet = overlayStateOnFleet;
 exports.chatRowsToAdd = chatRowsToAdd;
 exports.gchatRowsToAdd = gchatRowsToAdd;
+exports.overlayGchatNotify = overlayGchatNotify;
 exports.devRowsToAdd = devRowsToAdd;
 exports.markExcludedPendingRows = markExcludedPendingRows;
 exports.designRowsToAdd = designRowsToAdd;
@@ -854,7 +855,8 @@ const REGISTERED_CAP_NAMES = {
  */
 const REGISTERED_FIELDS = {
     chat: ["ref", "name", "workspace", "orgUnknown", "origin", "goldState", "notify"],
-    "google-chat": ["ref", "name"],
+    // notify（#572）: スペースも通知先にできる。真偽値の扱いは chat の notify と同じ篩
+    "google-chat": ["ref", "name", "notify"],
     dev: ["ref", "type"],
     // 🔴 **意図的に外した開発リポジトリ（#391）。** サーバは前から別キーで返していたが、
     // ここに無いので篩が丸ごと落としていた。**これが読めないと「読み取り停止中」の行を
@@ -1177,10 +1179,58 @@ function gchatRowsToAdd(externals, rows) {
             continue;
         if (have[String(r.ref)])
             continue;
-        out.push({ type: "google-chat", name: r.name || r.ref, ref: r.ref });
+        const row = { type: "google-chat", name: r.name || r.ref, ref: r.ref };
+        // 通知先の印（#572）。真偽値の印は true のときだけ載せる（chatRowsToAdd と同じ）
+        if (r.notify === true)
+            row.notify = true;
+        out.push(row);
     }
     return out;
 }
+/**
+ * まとめ表が既に持っている Google Chat の行に、live（registeredNow）の「通知先」の印を重ねる（#572）。
+ *
+ * 🔴 **engine のまとめ表（fleet-status）は Google Chat の行に `notify` を付けない**
+ * （`resolve-external-sources.mjs` は Slack にだけ付ける）。重ねないと、押した直後の仮置きが
+ * 切れたあと（最長 24 時間後）に、通知先のスペースが「通知先にする」に戻って見える
+ * ——画面では通知先でないのに投稿される、という取り違えになる。
+ *
+ * **Slack の行には触らない**（まとめ表が正しく持っている・挙動を変えない）。live に
+ * `google-chat` のキーが無ければ（読めていない）何も変えない。live に同じ ref の行があれば
+ * その `notify` を正とし（true なら付け、そうでなければ外す）、live に無い行はそのまま
+ * （消す判断は `subtractRemovedRegistered` の責務）。
+ *
+ * **クロージャを持たない純関数に保つこと**（`.toString()` でクライアントへ渡す）。
+ */
+function overlayGchatNotify(externals, rows) {
+    const list = externals || [];
+    if (!Array.isArray(rows))
+        return list;
+    const live = {};
+    for (const r of rows) {
+        if (!r || !r.ref)
+            continue;
+        live[String(r.ref)] = r.notify === true;
+    }
+    let changed = false;
+    const out = list.map(function (s) {
+        if (!s || s.type !== "google-chat" || !s.ref)
+            return s;
+        if (!Object.prototype.hasOwnProperty.call(live, String(s.ref)))
+            return s;
+        const want = live[String(s.ref)];
+        if ((s.notify === true) === want)
+            return s;
+        changed = true;
+        const row = Object.assign({}, s);
+        if (want)
+            row.notify = true;
+        else
+            delete row.notify;
+        return row;
+    });
+    return changed ? out : list;
+}
 /**
  * 開発の行で、まとめ表がまだ知らないもの。**鍵は `owner/repo` の生文字列。**
  *
@@ -1295,10 +1345,17 @@ function materialRowsToAdd(ids, rows) {
  * ⚠ 画面の `targetsGenerationOf`（サーバへ送る世代）は同じ規則を持つが、この関数を
  * 呼べない——配信物から関数1つを切り出して直接呼ぶテストがあり、外の名前を参照した
  * 瞬間にそのテストが動かなくなる。あちらを直すときはここも直す。
+ *
+ * 🔴 **規則の正本は `infra/lambda/shared/connection-state-keys.mjs` の `issuesRefOf`**（#556）。
+ * ここはその**写し**で、`viewer/` は独立した npm パッケージ（`rootDir: src`）なので
+ * 原本を import できない。**ずれを捕まえる網は契約表**——
+ * `infra/test/fixtures/connection-state-contract.json` の `issuesRefOf` の節を、
+ * 原本（infra 側のテスト）と写し（viewer/test/state-ref-parity.test.mjs）の**両方**に流す。
+ * 片方だけ直したら、直していない側が赤くなる。
  */
 function issuesRefOf(domain, projectKey) {
-    return String(domain == null ? "" : domain).toLowerCase() + "/" +
-        String(projectKey == null ? "" : projectKey).toUpperCase();
+    return String(domain == null ? "" : domain).trim().toLowerCase() + "/" +
+        String(projectKey == null ? "" : projectKey).trim().toUpperCase();
 }
 /**
  * 課題管理の行で、まとめ表がまだ知らないもの。
@@ -1578,9 +1635,11 @@ function mergeRegisteredFleet(view, reg) {
     };
     if (!reg)
         return out;
+    // Google Chat の行の「通知先」は live が正（#572・まとめ表は持っていない）。足す前に重ねる
+    out.externalSources = overlayGchatNotify(v.externalSources, reg["google-chat"]);
     const add = registeredRowsToAdd(v, reg);
     if (add.externalSources.length)
-        out.externalSources = (v.externalSources || []).concat(add.externalSources);
+        out.externalSources = (out.externalSources || []).concat(add.externalSources);
     if (add.design.length || add.materials.length) {
         const internals = (v.internalSources || []).slice();
         const put = (kind, patch) => {
@@ -2055,12 +2114,16 @@ function fleetLastSyncOf(internal) {
  * `issuesRefOf` はこれより**前**に流し込んである）。
  */
 function stateRefKey(capId, ref, workspace) {
-    var s = String(ref == null ? "" : ref);
+    // 🔴 **`normalizeStateRef`（connection-state-keys.mjs）の写し**（#556）。ワークスペースを
+    // 足すところだけが画面固有で、ref の畳み方そのものは向こうが正本
+    // （契約表 `normalizeStateRef` の節を両側に流して突き合わせる）
+    var s = String(ref == null ? "" : ref).trim();
     if (!s)
         return "";
     if (capId === "issues") {
         var at = s.indexOf("/");
-        if (at < 0)
+        // ドメインもキーも欠けた値から `issues` の鍵は組めない（原本と同じ判断）
+        if (at <= 0 || at === s.length - 1)
             return "";
         return issuesRefOf(s.slice(0, at), s.slice(at + 1));
     }
@@ -2094,24 +2157,27 @@ function registeredRefKeys(regKey, rows) {
                 out[issuesRefOf(r.domain, r.projectKey)] = true;
             continue;
         }
+        // 🔴 **相方（`stateRefKey`）と同じだけ畳む（#556）。** あちらが前後の空白を落とすので、
+        // ここが落とさないと**登録してあるのに「登録簿に無い」と判定される**行ができる
+        // （いまは登録簿の値がサーバ側で trim 済みなので起きないが、片側だけ畳む形を残さない）
         if (regKey === "design") {
             if (r.key)
-                out[String(r.key)] = true;
+                out[String(r.key).trim()] = true;
             continue;
         }
         if (regKey === "materials") {
             if (r.id)
-                out[String(r.id)] = true;
+                out[String(r.id).trim()] = true;
             continue;
         }
         if (!r.ref)
             continue;
         if (regKey === "chat") {
-            out[String(r.workspace == null ? "" : r.workspace) + "\t" + String(r.ref)] = true;
-            out["\t" + String(r.ref)] = true;
+            out[String(r.workspace == null ? "" : r.workspace) + "\t" + String(r.ref).trim()] = true;
+            out["\t" + String(r.ref).trim()] = true;
             continue;
         }
-        out[String(r.ref)] = true;
+        out[String(r.ref).trim()] = true;
     }
     return out;
 }
@@ -4545,6 +4611,7 @@ var PROBE_WAIT_TIMEOUT_TEXT = ${JSON.stringify(exports.PROBE_WAIT_TIMEOUT_TEXT)}
 ${sanitizeRegisteredNow.toString()}
 ${chatRowsToAdd.toString()}
 ${gchatRowsToAdd.toString()}
+${overlayGchatNotify.toString()}
 ${devRowsToAdd.toString()}
 ${markExcludedPendingRows.toString()}
 ${designRowsToAdd.toString()}
@@ -10624,6 +10691,9 @@ ${calendarAddUrl.toString()}
           "次回の Gold 昇格（毎晩22時ごろ）から、このスペースは読み取られなくなります。",
           "",
           "※ 既に Gold 層に入った内容は消えません（Decision・用語集はそのまま残ります）。",
+          // 解除は通知先の宣言（notify.json）を触らない（#572）。通知先のまま解除すると
+          // 行が消えて画面から止められなくなる——先に止める手順をここで言う
+          "※ 通知先にしている場合は、先に「通知を止める」を押してください（解除しても通知先は変わりません）。",
           "",
           "解除しますか？",
         ];
@@ -10682,6 +10752,12 @@ ${calendarAddUrl.toString()}
    *
    * **クロージャを持たない純関数に保つこと**（テストが配信物から切り出して直接呼ぶ）。
    * だから issuesRefOf を呼べない（規則の正本はあちら側。直すときは両方直す）。
+   *
+   * ⚠ **ここは前後の空白を落とさない。行の鍵（issuesRefOf）とは別物**（#556）。
+   * 揃えたくなるが、**揃えなくても食い違わない**——サーバは受け取った世代を
+   * normalizeGeneration に通してから比べ、そちらが空白を落とすため。逆に、ここで
+   * 落とす形に変えるなら**サーバの両方**（targetsGeneration と normalizeGeneration）を
+   * 同時に見ること。善意で片側だけ揃えると、正しい操作が毎回 409 になる。
    */
   function targetsGenerationOf(list) {
     return (list || [])
@@ -10931,9 +11007,9 @@ ${calendarAddUrl.toString()}
       // cap と question の両方で絞る。片方だけだと、同じ ref を持つ別の問い
       //（将来 issues に別の question が増えたとき）の行を取り込みとして読む
       if (!r || r.cap !== "issues" || r.question !== "ingested") continue;
-      var at = String(r.ref || "").indexOf("/");
-      if (at < 0) continue;
-      if (issuesRefOf(String(r.ref).slice(0, at), String(r.ref).slice(at + 1)) === want) return r;
+      // 行側も畳み方の正本（stateRefKey ＝ normalizeStateRef の写し）を通す（#556）。
+      // ここで indexOf を自前で切っていた間は、**切り方が5本目**になっていた
+      if (stateRefKey("issues", r.ref) === want) return r;
     }
     return null;
   }
@@ -13227,6 +13303,11 @@ ${calendarAddUrl.toString()}
     // だけなので、Slack のような origin の出し分けは不要
     if (SITE.intake && s.type === "google-chat" && s.ref) {
       card.appendChild(removeSpaceButton(s, card));
+      // 通知先の入り切り（#572）。Slack と同じ notifyToggle・同じ文言・同じ op（サーバは
+      // ref の形（spaces/…）で Google Chat と判定する）。スペースの行に workspace は無いので
+      // 「読み取り専用の接続」の分岐には入らず、常にボタンが出る。仮置きの行（本物の行が
+      // 来るまで）には Slack と同じく出さない
+      if (s.pending !== true) card.appendChild(notifyToggle(s, badges));
     }
     // GitHubリポの登録から削除。**ON/OFF のトグルは撤去した（2026-08-21）**
     //（stateToggle のコメント参照）。

package/package.json

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