@takagaki/cortex-decisions-viewer 0.12.30 → 0.12.32

package/dist/pending-ops.js

@@ -22,17 +22,20 @@ exports.PENDING_OPS_LIMITS = void 0;
 exports.pendingKindsTable = pendingKindsTable;
 exports.pendingOpsKey = pendingOpsKey;
 exports.pendingSlackChannelId = pendingSlackChannelId;
+exports.pendingIsSpaceRef = pendingIsSpaceRef;
 exports.pendingGoogleChatSpaceRef = pendingGoogleChatSpaceRef;
 exports.pendingRepo = pendingRepo;
 exports.pendingFigmaKey = pendingFigmaKey;
 exports.pendingDriveFolderId = pendingDriveFolderId;
 exports.backlogTargetOf = backlogTargetOf;
+exports.pendingIsSlackTeamId = pendingIsSlackTeamId;
 exports.pendingIssuesRef = pendingIssuesRef;
 exports.pendingLabel = pendingLabel;
 exports.pendingOpOf = pendingOpOf;
 exports.sanitizePendingOps = sanitizePendingOps;
 exports.rememberPendingOp = rememberPendingOp;
 exports.pendingSiteTimeMs = pendingSiteTimeMs;
+exports.pendingStale = pendingStale;
 exports.pendingOpBuildCaughtUp = pendingOpBuildCaughtUp;
 exports.pendingLiveRowMatches = pendingLiveRowMatches;
 exports.pendingOpLiveAgrees = pendingOpLiveAgrees;
@@ -101,24 +104,42 @@ function pendingSlackChannelId(input) {
     if (/^[CG][A-Z0-9]{6,}$/.test(s))
         return s;
     const m = s.match(/\/archives\/([CG][A-Z0-9]{6,})/);
-    return m ? m[1] : null;
+    if (m)
+        return m[1];
+    // ブラウザのアドレスバーの形（#398）。**team_id は使わない**——どのワークスペースかは
+    // サーバが実測して決める。ここで取るのはチャンネルIDだけ
+    // （team_id の位置の形は shared/secret-names.mjs の isSlackTeamId と同じ。値は採らない）
+    const client = s.match(/^https?:\/\/app\.slack\.com\/client\/[TE][A-Z0-9]{6,20}\/([CG][A-Z0-9]{6,})/);
+    return client ? client[1] : null;
+}
+/**
+ * スペース ref の形（サーバの `GOOGLE_CHAT_SPACE_REF` の写し）。ずれは parity テストが固定する。
+ *
+ * **写しがこのファイルに 2 つあったのを 1 つに寄せた（#398）。** ビューアは infra を
+ * import できないので写し自体は無くせないが、**同じ判定を 2 度書かない**。
+ */
+function pendingIsSpaceRef(v) {
+    return /^spaces\/[A-Za-z0-9_-]+$/.test(String(v == null ? "" : v));
 }
 function pendingGoogleChatSpaceRef(input) {
     const s = String(input == null ? "" : input).trim();
     if (!s || s.length > 500)
         return null;
+    // 素の `spaces/<ID>`（#389）。URL ではないのでホスト検査より前に見る
+    if (pendingIsSpaceRef(s))
+        return s;
     if (!/^https?:\/\/(?:chat|mail)\.google\.com\//i.test(s))
         return null;
-    if (/^https?:\/\/chat\.google\.com\/(?:u\/\d+\/)?dm\//i.test(s) ||
+    if (/^https?:\/\/chat\.google\.com\/(?:u\/\d+\/)?(?:dm|app\/dm)\//i.test(s) ||
         /^https?:\/\/mail\.google\.com\/[^#]*#chat\/dm\//i.test(s)) {
         return null;
     }
-    const m = s.match(/^https?:\/\/chat\.google\.com\/(?:u\/\d+\/)?room\/([^/?#]+)/i) ||
+    const m = s.match(/^https?:\/\/chat\.google\.com\/(?:u\/\d+\/)?(?:room|app\/(?:chat|join|management))\/([^/?#]+)/i) ||
         s.match(/^https?:\/\/mail\.google\.com\/(?:chat|mail)\/u\/\d+\/[^#]*#chat\/space\/([^/?#]+)/i);
     if (!m)
         return null;
     const ref = "spaces/" + m[1];
-    return /^spaces\/[A-Za-z0-9_-]+$/.test(ref) ? ref : null;
+    return pendingIsSpaceRef(ref) ? ref : null;
 }
 function pendingRepo(input) {
     let s = String(input == null ? "" : input).trim();
@@ -171,14 +192,35 @@ function backlogTargetOf(input) {
         return num.test(view[1]) ? null : { domain: domain, projectKey: view[1].toUpperCase() };
     return null;
 }
+/**
+ * Slack の team_id の形。**サーバの正本（shared/secret-names.mjs の `isSlackTeamId`）の写し**で、
+ * ずれは parity テストが固定する。
+ *
+ * 🔴 **狭くしないこと。** `T…` だけに絞ると **Enterprise Grid の `E…` を画面が弾き**、
+ * その組織のチャンネルだけ仮置きの行に札が出ない（サーバは受けているのに画面が捨てる形）。
+ */
+function pendingIsSlackTeamId(v) {
+    return /^[TE][A-Z0-9]{6,20}$/.test(String(v == null ? "" : v));
+}
 /** Backlog の接続先の鍵（render.ts の issuesRefOf と同形: ドメイン小文字 / キー大文字） */
 function pendingIssuesRef(domain, projectKey) {
     return String(domain == null ? "" : domain).toLowerCase() + "/" + String(projectKey == null ? "" : projectKey).toUpperCase();
 }
 // ---- エントリを作る（op → kind / action / targetId の対応表はここ 1 つ） ------------
-/** 表示名の整形（サーバの cleanLabel と同じ考え方: 制御文字を落とし、長すぎれば切る） */
+/**
+ * 表示名の整形（サーバの cleanLabel の写し: 制御文字を落とし、連続空白を1つに畳み、
+ * 長すぎれば切る）。**空なら空文字**——サーバは null を返すが、画面側は `|| id` の
+ * 落とし先があるので文字列のまま扱う（parity テストは `|| null` にして突き合わせる）。
+ *
+ * 🔴 **連続空白の圧縮を落とさないこと（#391）。** サーバだけが畳むと、名前に連続空白を
+ * 含むチャンネルの update 仮置きが pendingOpLiveAgrees の名前比較で永久に同意せず、
+ * 24 時間（TTL）残り続ける。写しのずれはこの1行だけで起きる。
+ */
 function pendingLabel(s, max) {
-    const v = String(s == null ? "" : s).replace(/[\u0000-\u001f\u007f]/g, " ").trim();
+    const v = String(s == null ? "" : s)
+        .replace(/[\u0000-\u001f\u007f]/g, " ")
+        .replace(/\s+/g, " ")
+        .trim();
     const lim = max || 120;
     return v.length > lim ? v.slice(0, lim) : v;
 }
@@ -218,6 +260,15 @@ function pendingOpOf(payload, ctx) {
             const data = { name: pendingLabel(payload.name) || id };
             if (payload.workspace)
                 data.workspace = String(payload.workspace);
+            // **どの組織のチャンネルかは、サーバの実測を応答から受け取る（#391）。**
+            // 画面はワークスペースを送らない（プルダウンは 2026-08-20 に廃止）ので、
+            // これが無いと仮置きの行にだけ組織の札が出ない＝本物の行と見た目が食い違う。
+            // 🔴 **文字列 1 欄で運ぶ**（"unknown" / "internal" / team_id）。真偽値を増やすと
+            // sanitizePendingOps の許可リストの例外が増え、篩が篩の意味を失う。
+            // 応答に無ければ載せない——載せないことが「まだ分からない」の表明になる
+            const org = ctx.response && typeof ctx.response.slackOrg === "string" ? ctx.response.slackOrg : "";
+            if (org)
+                data.slackOrg = org;
             return make("slack", "add", id, data);
         }
         case "remove-channel":
@@ -303,7 +354,7 @@ function sanitizePendingOps(raw, projectKey) {
         const src = e.payload && typeof e.payload === "object" ? e.payload : {};
         // 通してよい payload の欄（葉は文字列か、notify だけ真偽値）。関数の中に置くのは、
         // クライアントへ toString() で流し込んだ先でも同じ一覧を参照できるようにするため
-        const fields = ["name", "workspace", "notify", "domain", "projectKey", "type"];
+        const fields = ["name", "workspace", "notify", "domain", "projectKey", "type", "slackOrg"];
         for (const f of fields) {
             if (!Object.prototype.hasOwnProperty.call(src, f))
                 continue;
@@ -387,8 +438,46 @@ function pendingSiteTimeMs(s) {
     return Date.parse(s.replace(" UTC", "Z").replace(" ", "T"));
 }
 /**
- * 本物の一覧の世代が追いついたか。**判定の形はアイコン（iconPendingStale）・トゥームストーンと同じ**
- * ——いま画面が持っている世代が、保存時刻＋余裕 と 保存時に見ていた世代＋余裕 の両方より新しい。
+ * **「操作より後のビルドが来たか」の唯一の判定（#391）。**
+ *
+ * 削除のトゥームストーン・アイコンの反映待ち・操作タブの仮置きは、どれも「押した結果が
+ * 次のビルドで本物になるまでブラウザが覚えておく」という同じ性質の待ちで、破棄の条件も
+ * 同じだった。**同じものが 3 実装あると、片方だけ直したときに挙動が割れる**（値が同じ
+ * うちは誰も気づかない）ので、ここ 1 つに寄せてある。
+ *
+ * 判定: いま画面が持っている世代（nowMs）が、**押した時刻＋余裕**と**押したとき見ていた
+ * 世代＋余裕**の両方より新しい。generatedAt はコミット時点ではない——押す前に始まった
+ * ビルドが後から完成すると、世代は押した時刻より新しいのに操作は入っていない。
+ * ビルドの所要時間とクライアント時計のずれを見て余裕（既定 30 分）を取る。
+ *
+ * **主軸はサーバ時刻同士の比較**（現行の世代と、押した画面の gen0）。クライアントの時計が
+ * 大きく狂っていても機能が丸ごとは死なない（進んだ時計なら破棄が遅れるだけ）。
+ * gen0 を持たない古い形式のエントリは at だけで見る（後方互換・安全側）。
+ * nowMs が読めない（NaN）ときは false ＝**隠し続ける／残し続ける側に倒す**（嘘の再表示より安全）。
+ *
+ * @param at     押した時刻（ms・クライアント時計）
+ * @param gen0   押した画面の世代（generatedAt の文字列）。読めなければ at だけで見る
+ * @param nowMs  いまの世代（ms）
+ *
+ * ⚠ 余裕は引数にしない。3 経路とも 30 分で、可変にすると**呼び出しが 1 つも無い引数**が
+ * 残る（壊しても何も起きない＝変異検証の網から静かに漏れる形）。
+ */
+function pendingStale(at, gen0, nowMs) {
+    const margin = PENDING_OPS_MARGIN_MS;
+    if (typeof nowMs !== "number" || isNaN(nowMs))
+        return false;
+    if (typeof at !== "number" || !isFinite(at))
+        return false;
+    if (!(nowMs > at + margin))
+        return false;
+    const g = pendingSiteTimeMs(gen0);
+    if (isNaN(g))
+        return true;
+    return nowMs > g + margin;
+}
+/**
+ * 本物の一覧の世代が追いついたか。**判定の実体は pendingStale**（トゥームストーン・
+ * アイコンの反映待ちと同じもの・#391）。
  *
  * 🔴 **世代は「一覧の下地」のもの（fleet-status の generatedAt）を渡すこと。** ページの
  * generatedAt（SITE.generatedAt）は記録の追加でも進むが、一覧の下地は夜間の自己チェックの
@@ -396,13 +485,9 @@ function pendingSiteTimeMs(s) {
  * 生成時刻はコミット時点ではない（保存前に始まった実行が後から完成する）ので余裕を取る。
  */
 function pendingOpBuildCaughtUp(e, siteNowMs) {
-    if (!e || typeof siteNowMs !== "number" || isNaN(siteNowMs))
+    if (!e)
         return false;
-    const byAt = siteNowMs > e.committedAt + PENDING_OPS_MARGIN_MS;
-    const gen0 = pendingSiteTimeMs(e.baseBuildId);
-    if (isNaN(gen0))
-        return byAt;
-    return byAt && siteNowMs > gen0 + PENDING_OPS_MARGIN_MS;
+    return pendingStale(e.committedAt, e.baseBuildId, siteNowMs);
 }
 /** registeredNow の行が、このエントリの対象か */
 function pendingLiveRowMatches(kind, r, targetId) {
@@ -443,8 +528,22 @@ function pendingOpLiveAgrees(e, live, liveAt) {
             found = r;
             break;
         }
-    if (e.action === "remove")
-        return !found;
+    if (e.action === "remove") {
+        if (found)
+            return false;
+        // 🔴 **github は「読み取り停止中」（exclude）の行が live の dev に載らない（#391）。**
+        // dev だけを見ると、exclude された行の remove は**保存が効く前から**「無い＝同意」に
+        // なり、5 分で仮置きが捨てられて、まとめ表の行が次のビルドまで戻る（点滅する）。
+        // devExcluded を返すサーバなら、そちらにも無いことまで確かめてから同意する。
+        // 返さない古いサーバでは従来どおり（dev だけ）——判断材料が無いのに厳しくすると、
+        // 普通のリポジトリの解除まで 24 時間残る
+        if (e.kind === "github" && Array.isArray(live.devExcluded)) {
+            for (const r of live.devExcluded)
+                if (pendingLiveRowMatches(e.kind, r, e.targetId))
+                    return false;
+        }
+        return true;
+    }
     if (e.action === "add")
         return !!found;
     if (!found)
@@ -538,6 +637,24 @@ function pendingExternalRow(e) {
             row.workspace = p.workspace;
         if (p.notify === true)
             row.notify = true;
+        // **サーバが実測した組織を、本物の行と同じ欄に展開する（#391）。**
+        // 展開できたときだけ pendingOrg を立てる——立っていない行に札を出すと、
+        // 測れていないのに「クラスメソッド」と名乗る（画面側の既定がそれなので）
+        // 値は 3 通りだけ通す（"unknown" / "internal" / team_id）。**"internal" を team_id として
+        // 札に出さない**ための門で、形の検査はサーバの正本（pendingIsSlackTeamId）と同じにする
+        // ——狭くすると Enterprise Grid（E…）の組織だけ札が出ない
+        const org = typeof p.slackOrg === "string" ? p.slackOrg : "";
+        if (org === "unknown") {
+            row.pendingOrg = true;
+            row.orgUnknown = true;
+        }
+        else if (org === "internal") {
+            row.pendingOrg = true;
+        }
+        else if (pendingIsSlackTeamId(org)) {
+            row.pendingOrg = true;
+            row.workspace = org;
+        }
     }
     else if (e.kind === "google-chat") {
         row.type = "google-chat";
@@ -602,6 +719,12 @@ function applyPendingOps(view, list) {
                 // dev-sources.json 側にも宣言された行（both / explicit）は channels.json から消しても
                 // エンジンがまだ読む——消すと「読まれているのに画面に無い」逆向きの嘘になり、
                 // live が追いついた後に復活する（設計レビュー②の中5）。そういう行は従来どおり取り消し線のまま
+                //
+                // **「読み取り停止中」（goldState: "excluded"）の GitHub 行も、押したら消す（#391）。**
+                // サーバの applyRemoveGithubSource は sources と exclude の両方から実際に消すので、
+                // 保存は完全に効いている。ここで守ると「押しても何も起きない」＝保存が失敗したのと
+                // 区別が付かない。消えたままにする仕掛けは 2 つ: 同意の判定（pendingOpLiveAgrees が
+                // devExcluded も見る）と、引き算（subtractRemovedRegistered が devExcluded で判断する）
                 const removable = function (s) {
                     return pendingExternalMatches(e.kind, s, e.targetId) && (e.kind !== "slack" || s.origin === "derived");
                 };