@takagaki/cortex-decisions-viewer 0.4.83 → 0.4.84

package/dist/render.js

@@ -2083,6 +2083,25 @@ ${capabilityState.toString()}
   function isInternalSlack(s) {
     return s.type === "slack" && !s.workspace;
   }
+  /**
+   * Slack チャンネルの表示名。**# は表示のときだけ付ける**（保存値は素・2026-08-19）。
+   *
+   * 保存値に # を持たせていたせいで、人が手で # 無しに書いた行と Slack から引いた値が
+   * 「# の有無だけ」で食い違い、「名前が違います」と出た（実機で発生）。**保存は素に
+   * 決めた**ので、付けるのはここ1箇所だけ。決定の理由は infra/lambda/shared/slack-gate.mjs。
+   *
+   * **冪等にしてある**（先頭の # を落としてから付ける）。既存データには # 付きで
+   * 保存された行が10件あり、これが無いと「##tf-project-cortex」になる。冪等なので
+   * **データの移行を待たずにこの表示へ切り替えられる**（揃うのは名前が引き直された行から順に）。
+   *
+   * 印付きの名前（「（名前未取得: C0BQ…）」）には付けない——# は実名の印なので、
+   * 引けていない名前に付けると実名に見えてしまう。
+   */
+  function slackDisplayName(name) {
+    var s = String(name || "");
+    if (!s || s.indexOf("（名前未取得") === 0) return s;
+    return "#" + s.replace(/^#+/, "");
+  }
   /** 行に出す接続状態。no_token だけは経路（社内/顧客）で文面が分かれる */
   function dsGateOf(s) {
     if (s.gate === "no_token" && s.type === "slack") {
@@ -3084,104 +3103,99 @@ ${capabilityState.toString()}
   }
 
   /**
-   * 判定ごとの「次の一手」。**状態の語だけでは足りない**——今日の3件はどれも
-   * 「何が起きているか」は分かっても「次に何をすればいいか」が決まらなかった。
+   * 判定ごとの表示。**成功は1行で終わらせ、失敗は「理由＋次の一手」だけを残す。**
+   *
+   * ## 短くした理由（2026-08-19・オーナー指示）
+   *
+   * 「接続を確認を押して出てくる文言が長ったらしい」。実際、成功時に4行出ていた。
+   * **成功時は「読めている」以上の情報が要らない。**
+   *
+   * ## ただし失敗時は削りすぎない
+   *
+   * 短くしすぎると、今日と同じ「画面を見ても次に何をすればいいか決まらない」状態に戻る
+   * （それで半日かかった）。落とすのは**内部の仕組みの説明**——「設定ファイルの表示名だけが
+   * 書き換わります」「読み取りの対象・範囲は変わりません」のような、読む人の次の行動を
+   * 変えない文。残すのは「何が起きているか」と「次に何をするか」だけ。
+   *
+   * 状態の語そのものは DS_GATE / DS_GATE_NO_TOKEN が持つ（probeVerdict 経由）ので、
+   * ここが返すのは**それに足りない分だけ**。二重に言わない。
+   *
+   * @returns 本文に並べるノード（0〜2個）
    */
   function probeHelpNodes(s, r) {
     var p = function (t) { return el("p", { class: "ds-probe-help", text: t }); };
-    var out = [];
-    if (r.gate === "ok") {
-      // **表示名が引けたかまで言う。** 読めているのに名前が「（名前未取得: C0BQ…）」の
-      // ままだった案件があり、読めていること自体は正しかったので誰も疑わなかった
-      out.push(p(r.name
-        ? "読めています。Slack で確認できた表示名は「" + r.name + "」です。"
-        : "読めています。ただし表示名は引けませんでした（Slack の応答: " +
-          (r.error || "不明") + "）。一覧の名前は登録したときのままになります。"));
-      return out;
-    }
+    // 成功。**1行で終わり。** 状態の語（「接続OK」）が既に出ているので本文は空
+    if (r.gate === "ok") return [];
     if (r.gate === "not_in_channel") {
-      out.push(p("このチャンネルに Cortex を招待してください。公開チャンネルでも招待が必要で、" +
-        "招待していないと中身を1件も読めません。Slack でこのチャンネルを開き、次のコマンドを貼ります。" +
-        "招待したら、もう一度「接続を確認」を押して直ったことを確かめてください。"));
-      // **手で打たせない。** 短い文字列でも打ち間違いは黙って失敗し、
-      // 利用者には「招待したのに読まれない」としか見えない（copyRow の直上コメント）
-      out.push(copyRow("/invite @Cortex"));
-      return out;
+      // 状態の語が「Slackで招待してください」なので、説明は足さずコマンドだけ出す。
+      // **手で打たせない**——打ち間違いは黙って失敗し、「招待したのに読まれない」に見える
+      return [copyRow("/invite @Cortex")];
     }
     if (r.gate === "missing_scope") {
-      // **足りないスコープ名をそのまま出す（2026-08-19 の不具合の本体）。**
-      // Slack は needed で名指しで返しているのに、実測がそれを捨てて「不明」にしていた。
-      // **招待と言わない**——招待では直らないので、言うと直らない操作を繰り返させる
-      var line = el("p", { class: "ds-probe-help" });
-      line.appendChild(document.createTextNode("Cortex の Slack App に権限（スコープ）が足りません。不足しているのは "));
-      line.appendChild(el("code", { class: "ds-probe-scope", text: r.needed || "不明（Slack が名前を返しませんでした）" }));
-      line.appendChild(document.createTextNode(" です。"));
-      out.push(line);
-      out.push(p("これは招待では直りません。Slack App にこのスコープを追加して再インストールする必要があるので、" +
-        "cortex の担当者にこのスコープ名をそのまま伝えてください。" +
-        (r.readable
-          ? "なお、チャンネルの中身そのものは読めています（毎晩の取り込みは動いています）。直らないのは表示名です。"
-          : "")));
-      return out;
+      // 不足スコープは頭のバッジ横に出る（probeSection）。ここは次の一手だけ
+      return [p("招待では直りません。cortex の担当者にこのスコープ名をお伝えください。")];
     }
     if (r.gate === "no_token") {
-      // **社内/顧客の出し分けは既存の1箇所から取る。** ここで書き分けると必ずずれる
-      var help = probeVerdict(s, r).help;
-      if (help) out.push(p(help));
-      return out;
+      // **社内/顧客の出し分けは変えない。** 社内は画面から設定できるものが無いので
+      // 連絡先、顧客は接続へ誘導する（2026-08-18 の不具合の再発防止）。
+      // 長い説明は落とし、次の行動の1行だけ残す
+      return [p(isInternalSlack(s)
+        ? "cortex の担当者にご連絡ください。"
+        : "下の「お客様の Slack ワークスペースを接続」から接続してください。")];
     }
     if (r.gate === "unreachable") {
-      out.push(p("チャンネルが見つからないか、アーカイブされています。" +
-        "Slack でこのチャンネルがまだ存在するかを確かめてください。" +
-        "アーカイブしたチャンネルなら、この行の「登録を解除」で読み取り対象から外せます。" +
-        "存在するのに見つからない場合は、Cortex が入っていない別ワークスペースのチャンネルの可能性があるので、" +
-        "cortex の担当者にご連絡ください。"));
-      return out;
+      return [p("チャンネルが見つからないか、アーカイブされています。不要なら「登録を解除」してください。")];
     }
-    // unknown。**生のエラーを隠さない**——「不明」だけだと切り分けが始められない
-    out.push(p("読めるかどうかを確かめられませんでした。Slack の応答は「" + (r.error || "不明") + "」です。" +
-      "少し時間をおいてもう一度お試しください。同じ応答が続く場合は、この文言をそのまま cortex の担当者にお伝えください。"));
-    return out;
+    // unknown。**生のエラーは隠さない**——「不明」だけだと切り分けが始められない
+    return [p("Slack の応答：" + (r.error || "不明") + "。時間をおいて再度お試しください。")];
   }
 
   /**
-   * 表示名の取り直し。**確認とは別のボタンにしてある。**
+   * 表示名を Slack の実名へ**自動で揃える**（2026-08-19・オーナー判断）。
+   *
+   * ## なぜ自動なのか
+   *
+   * 「表示名は人が決めるものではなく、Slack から引いたものをそのまま出す」と決まった。
+   * 正解が1つしかないので、**人に選ばせるボタンは摩擦でしかない**（差分表示と
+   * 「表示名を更新する」ボタンは撤去した。Gold トグルを撤去したときと同じ理屈）。
+   *
+   * ## なぜサーバの ?op=probe ではなく画面から送るのか
    *
-   * 確認（?op=probe）は読むだけなので誰でも押してよいと決めた（オーナー判断・2026-08-19）。
-   * 名前の更新は案件リポジトリへのコミットなので、**設定変更と同じ入口（?op=config）**に
-   * 通す（sendConfig）。確認のついでに黙って書き込む形にすると、「読むだけだから誰でも
-   * 押してよい」という前提そのものが崩れる。**押す前に、何がどう変わるかを見せる。**
+   * **疎通確認そのものは読むだけ、を保ちたいから。** 「読むだけで設定を変えない」が
+   * ボタンを誰でも押せるようにした根拠なので、?op=probe を書き込む口にはしない。
+   * 書き込みは既存の ?op=config を通す——他案件チェック・コミットのリトライ・
+   * 画面への反映まで、設定変更と同じ守りがそのまま効く。
    *
-   * 出すのは**違っているときだけ**。同じ名前のときにボタンだけ出すと、押しても
-   * 「変更はありません」と返るだけの操作になる。
+   * **本来の置き場は夜間の接続状況生成（engine 側）。** そちらなら人の操作と無関係に
+   * 1日1回揃い、改名にも追従する。engine が今回の変更対象外なので、いまは押したときに揃える。
+   *
+   * ## 静かに失敗させる
+   *
+   * 揃えるのはおまけで、押した人の目的は「読めているか確かめる」こと。
+   * 失敗しても疎通確認の結果は出したままにする（名前が古いままなだけ）。
    */
-  function renameChannelRow(s, r, card) {
-    if (!r.name || r.name === s.name) return null;
-    var box = el("div");
-    box.appendChild(el("p", { class: "ds-probe-help",
-      text: "一覧の表示名「" + (s.name || "（未設定）") + "」は、Slack のいまの名前「" + r.name + "」と違います。" +
-        "更新すると、この案件の設定ファイル（チャット/channels.json）の表示名だけが書き換わります。" +
-        "読み取りの対象・範囲は変わりません。" }));
-    var row = el("div", { class: "ds-toggle" });
-    var btn = el("button", { class: "ds-toggle-btn", type: "button", text: "表示名を更新する" });
-    var msg = el("div", { class: "ds-toggle-msg" });
-    btn.addEventListener("click", function () {
-      // **名前は送らない。** サーバが Slack から引き直したものだけを書く（引けなければ
-      // 断って既存の名前を残す）。画面から任意の名前を送れる形にすると、
-      // 名前の取り直しの顔をした自由な改名になる
-      var payload = { op: "refresh-channel-name", ref: s.ref };
-      if (s.workspace) payload.workspace = s.workspace;
-      sendConfig(payload, msg, btn, function () {
-        // 一覧は夜間のスナップショットなので、その場では変わらない。行だけ追いつかせる
-        var nameEl = card.querySelector(".ds-name");
-        if (nameEl) nameEl.textContent = r.name;
-        btn.disabled = true;
-      });
-    });
-    row.appendChild(btn);
-    row.appendChild(msg);
-    box.appendChild(row);
-    return box;
+  function autoAlignName(s, r, card) {
+    // 引けていない／既に同じなら何もしない（押すたびにコミットを積まない）
+    if (!r.name || !s.ref) return;
+    if (String(s.name || "").replace(/^#+/, "") === String(r.name)) return;
+    // **channels.json に無い行は書き換えられない。** dev-sources.json だけに載っている
+    // 行（origin: explicit）へ送ると必ず404になる（「登録を解除」と同じ条件）
+    if (s.origin === "explicit") return;
+    var intake = SITE.intake;
+    if (!intake) return;
+    var payload = { op: "refresh-channel-name", ref: s.ref };
+    if (s.workspace) payload.workspace = s.workspace;
+    var pfx = intake.url + (intake.url.indexOf("?") === -1 ? "?" : "&");
+    fetch(pfx + "op=config&t=" + encodeURIComponent(intake.token), {
+      method: "POST",
+      headers: { "content-type": "application/json" },
+      body: JSON.stringify(payload),
+    }).then(function (res) {
+      if (!res.ok) return;
+      // 一覧は夜間のスナップショットなので、その場では変わらない。行だけ追いつかせる
+      var nameEl = card.querySelector(".ds-name");
+      if (nameEl) nameEl.textContent = slackDisplayName(r.name);
+    }).catch(function () { /* 揃えられなくても疎通確認の結果は出したまま */ });
   }
 
   /**
@@ -3192,9 +3206,9 @@ ${capabilityState.toString()}
    * になり、この節で3回直してきた食い違いを画面の側から作ることになる。
    * 置き換えたことが分かるよう、いつ測ったかを必ず添える。
    *
-   * **結果は保存しない。** fleet-status.json は夜間のスナップショットのまま置く
-   * （押すたびにコミットが増えるのを避ける）。表示名の取り直しだけが例外で、
-   * そちらは正本（channels.json）を直す価値があるので別のボタンで書き込む。
+   * **判定は保存しない。** fleet-status.json は夜間のスナップショットのまま置く
+   * （押すたびにコミットが増えるのを避ける）。表示名だけが例外で、そちらは
+   * Slack の実名が正なので自動で揃える（autoAlignName）。
    */
   function probeSection(s, card, gateEl) {
     var wrap = el("div", { class: "ds-probe" });
@@ -3234,19 +3248,24 @@ ${capabilityState.toString()}
           btn.disabled = false;
           return;
         }
-        msg.textContent = "";
+        msg.className = "ds-toggle-msg" + (j.gate === "ok" ? " is-ok" : " is-error");
+        msg.textContent = j.gate === "ok" ? "✓" : "✗";
         var v = probeVerdict(s, j);
         gateEl.className = "ds-gate " + v.cls;
         gateEl.textContent = v.text;
+        // 不足スコープは**状態の語のすぐ横**に出す（「権限が足りません：groups:read」）。
+        // 本文へ落とすと、いちばん要る情報が説明文に埋もれる
+        if (j.gate === "missing_scope") {
+          head.appendChild(el("code", { class: "ds-probe-scope", text: j.needed || "不明" }));
+        }
         at.textContent = "いま測った結果（" + (dsFmtTime(j.checkedAt) || "たった今") + "）";
         probeHelpNodes(s, j).forEach(function (n) { body.appendChild(n); });
-        // **登録ファイルが channels.json の行にだけ出す。** 書き換える先がそこなので、
-        // dev-sources.json だけに載っている行（origin: explicit）では必ず 404 になる
-        // ——「登録を解除」が同じ理由で同じ条件を持っている
-        if (s.origin !== "explicit") {
-          var rename = renameChannelRow(s, j, card);
-          if (rename) body.appendChild(rename);
-        }
+        // **夜間の判定に添えてあった案内は隠す。** 置き換えた判定と食い違う案内が
+        // 下に残ると、同じ行に2つのことが書いてある状態になる
+        var stale = card.querySelector(".ds-gate-help");
+        if (stale) stale.hidden = true;
+        // 表示名を Slack の実名へ自動で揃える（差分表示も更新ボタンも出さない）
+        autoAlignName(s, j, card);
       } catch (e) {
         msg.className = "ds-toggle-msg is-error";
         msg.textContent = "✗ 通信に失敗しました。";
@@ -4327,7 +4346,8 @@ ${capabilityState.toString()}
       el("div", { class: "ds-card-main" }, [
         el("span", { class: "ds-icon", text: meta.icon }),
         el("span", { class: "ds-type", text: meta.label }),
-        dsNameEl(s.name || "", s.url),
+        // Slack だけ # を表示側で付ける（保存値は素）。他のアダプタの名前には触らない
+        dsNameEl(s.type === "slack" ? slackDisplayName(s.name) : (s.name || ""), s.url),
       ]),
       badges,
       gateEl,

package/package.json

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