@takagaki/cortex-decisions-viewer 0.12.0 → 0.12.2

package/dist/render.js

@@ -517,9 +517,14 @@ const REGISTERED_CAP_NAMES = {
     materials: "共有資料",
     issues: "課題管理",
 };
-/** 通してよい欄。**ここに無い欄は1つも通さない**（Lambda 側の列挙と同じ形） */
+/**
+ * 通してよい欄。**ここに無い欄は1つも通さない**（Lambda 側の列挙と同じ形）
+ *
+ * 🔴 **`orgUnknown` は真偽値**（他の欄は全部文字列）。**この1欄だけの例外**で、
+ * 篩の側にも `=== true` の分岐がある——`sanitizeRegisteredNow` を参照
+ */
 const REGISTERED_FIELDS = {
-    chat: ["ref", "name", "workspace"],
+    chat: ["ref", "name", "workspace", "orgUnknown"],
     dev: ["ref", "type"],
     design: ["key", "name"],
     materials: ["id", "name"],
@@ -559,6 +564,17 @@ function sanitizeRegisteredNow(v) {
             const item = {};
             for (const f of REGISTERED_FIELDS[cap]) {
                 const fv = r[f];
+                // 🔴 **真偽値を通すのは orgUnknown だけ**（2026-08-25）。葉を文字列に限った篩が
+                // この印を落としており、**追加した直後のチャンネルから「Slack ワークスペースに
+                // 接続する」の導線が消えていた**（顧客Slackを繋ぐ唯一の入口）。
+                // **`=== true` に限る。** 文字列の "true" や 1 を真と読むと、意図しない値が
+                // 「組織不明」の案内を呼び、直すものが無い人に接続を勧めることになる。
+                // 例外はこの1欄だけ——他の欄まで真偽値を通すと、許可リストが篩の意味を失う
+                if (f === "orgUnknown") {
+                    if (fv === true)
+                        item[f] = true;
+                    continue;
+                }
                 if (typeof fv !== "string" || fv === "")
                     continue; // 葉は文字列だけ
                 item[f] = fv;
@@ -668,6 +684,10 @@ function overlayStateOnFleet(view, rows, generatedAt) {
  * 🔴 **鍵はチャンネルID＋ワークスペース。** チャンネルIDはワークスペースの中でしか
  * 一意でないので、IDだけで突き合わせると別ワークスペースの同IDを同じものと見て
  * 行が出ない（#104 で同じ理由の事故がある）。
+ *
+ * 🔴 **`orgUnknown` の印をここで落とさない（2026-08-25）。** 落とすと、この関数が作る行
+ * （＝追加した直後の主役）が `groupSlackByOrg` で社内の組に落ち、
+ * **「Slack ワークスペースに接続する」が画面から消える**——顧客Slackを繋ぐ唯一の入口。
  */
 function chatRowsToAdd(externals, rows) {
     const have = {};
@@ -685,6 +705,8 @@ function chatRowsToAdd(externals, rows) {
         const row = { type: "slack", name: r.name || r.ref, ref: r.ref };
         if (r.workspace)
             row.workspace = r.workspace;
+        if (r.orgUnknown === true)
+            row.orgUnknown = true;
         out.push(row);
     }
     return out;
@@ -2438,6 +2460,10 @@ a.ds-name:hover { text-decoration: underline; }
   min-width: 0; white-space: normal; overflow-wrap: anywhere;
 }
 .ds-toggle { display: flex; align-items: center; gap: 10px; flex-wrap: wrap; margin-top: 6px; }
+/* 常設の接続の入口に添える「なぜ押すのか」（0.12.2）。**必ず自分の行を占める**
+   （flex: 1 1 100%）——.ds-toggle は横並びなので、隣の「誰が承認できるか」の説明と
+   横に並ぶと、どちらも半分の幅に潰れる（.ds-gate-help と同じ理由） */
+.ds-connect-lead { flex: 1 1 100%; }
 .ds-toggle-btn { font-size: 12px; padding: 3px 10px; border-radius: 999px; border: 1px solid var(--border); background: #fff; color: var(--text); cursor: pointer; }
 .ds-toggle-btn:hover:not(:disabled) { border-color: var(--accent); color: var(--accent); }
 .ds-toggle-btn:disabled { opacity: .6; cursor: default; }
@@ -2507,6 +2533,12 @@ a.ds-name:hover { text-decoration: underline; }
 .ds-probe-at { font-size: 11.5px; color: var(--muted); }
 .ds-probe-help { margin: 6px 0 0; font-size: 12px; line-height: 1.7; color: #6b7280;
   min-width: 0; white-space: normal; overflow-wrap: anywhere; }
+/* 「アクセス不可」の原因の候補（0.12.2）。**箇条書きにする**——候補が3つあるので、
+   一続きの文章にすると読む人が「自分がどれなのか」を自分で切り出すことになる
+   （組織不明の案内が同じ形で並べているのと同じ理由）。
+   色と大きさは .ds-probe-help を併せて付けることで他の本文に揃える */
+.ds-probe-why { padding-left: 1.25em; list-style: disc; }
+.ds-probe-why li { margin: 2px 0; }
 /* Slack が名指しした不足スコープ。**そのまま読める形で出す**（言い換えると検索できない） */
 .ds-probe-scope { font-family: ui-monospace, SFMono-Regular, Menlo, monospace; background: #fef3c7;
   border: 1px solid #fde68a; border-radius: 6px; padding: 1px 6px; user-select: all; }
@@ -3625,8 +3657,17 @@ ${deriveCapTile.toString()}
    * お客様の Slack なら、クラスメソッドの人間は押せないことが多い
    * （その Slack の管理者にお願いすることになる）。それを書かないと、
    * 押しても進まない人が「壊れている」と読む。
+   *
+   * ## 常設の入口として置くときは lead を渡す（0.12.2）
+   *
+   * 「組織不明」の案内の中では、直前の箇条書きが**なぜ接続するのか**を言っている。
+   * 追加フォームの後に単独で置くときは**その文脈が無い**ので、押す理由が画面から
+   * 読み取れない——下の説明文は「誰が承認できるか」の話しかしていない。
+   * 1行だけ添える口を開けておく。**見出しは付けない**（節を増やすと一覧が遠くなる）。
+   *
+   * @param lead 省略可。押す理由の1行（案内の中から呼ぶときは渡さない）
    */
-  function slackConnectButton() {
+  function slackConnectButton(lead) {
     var wrap = el("div", { class: "ds-toggle" });
     // **「この」を付けない（2026-08-20・オーナー指摘）。**
     // どのワークスペースを指すのか読み取れない——そもそも**指定できない**
@@ -3664,6 +3705,9 @@ ${deriveCapTile.toString()}
         btn.disabled = false;
       }
     });
+    // **lead は「なぜ押すのか」・下の説明は「誰が押せるのか」。** 別のことを言っているので
+    // 1文にまとめない（まとめると、どちらの話なのか読み取れない長文になる）
+    if (lead) wrap.appendChild(el("p", { class: "ds-group-note ds-connect-lead", text: lead }));
     wrap.appendChild(el("p", {
       class: "ds-group-note",
       text: "ボタンを押すと Slack の承認画面が開きます。承認できるのは、その Slack の管理者だけです"
@@ -5419,6 +5463,40 @@ ${deriveCapTile.toString()}
     // 読み取れていない組として現れ、そこに接続ボタンが出る。
     // 0件なら繋ぐ相手もまだ無いので、入口が要る場面にならない。
     pushCap("チャット", adders.querySelector('[data-adder="slack"]'));
+    /* -------------------------------------------------------------------------
+     * 🔴 接続の入口を常設する（0.12.2・実オンボーディングで踏んだ）
+     * -------------------------------------------------------------------------
+     * 直前の版まで、このボタンは**「組織不明」の案内の中にしか無かった**。
+     * 2026-08-20 に独立した節をやめたときの理屈は「チャンネルを追加すれば
+     * 読み取れていない組として現れ、そこに接続ボタンが出る」だったが、
+     * **印が付かない経路で登録された行には、そこへ行く道が1本も無い**:
+     *
+     *   - channels.json に手で書き足した行は orgUnknown を持たない
+     *     （印を載せるのはサーバの登録経路だけ）ので、お客様の Slack の
+     *     チャンネルなのに「クラスメソッド」の並びに落ちる
+     *   - Slack コネクトの共有チャンネルは **URL がどちらの組織のドメインにもなりうる**
+     *     ので、そもそも分類が当てにならない。「クラスメソッド」に分類された行の
+     *     正しい対処が接続（OAuth）のこともある（実測: 招待しようとしたら
+     *     「組織外のユーザー」で断られた）
+     *
+     * **お客様の Slack かどうかは、行の状態から決まらない。** だから入口は
+     * 行の状態と無関係に置く——**組織不明の組が1つも無くても出る**。
+     *
+     * ⚠ **「組織不明」の案内の中のボタンは残す。** あの文脈では、直前の箇条書きが
+     * 「なぜ接続するのか」を言っており、次の一手として正しい。同じ関数を2箇所から
+     * 呼ぶだけで、押したときの動きは1つ（承認の口も1つ）。
+     *
+     * **置き場所は追加フォームの後。** フォームの手順3が「組織不明と出たら…接続する」と
+     * 言っているので、フォームより前に出すと読む順と操作の順が食い違う。
+     *
+     * **投入の口が無い構成では出さない。** 承認の開始は intake Lambda を叩くので、
+     * 無い案件では押しても始まらない（renderExternalAdders も同じ条件で空を返す）。
+     * ----------------------------------------------------------------------- */
+    if (SITE.intake) {
+      pushCap("チャット", slackConnectButton(
+        "お客様の Slack や、Slack コネクトで共有されたチャンネルを読むには、先にワークスペースの接続が要ります。",
+      ));
+    }
     // **組織ごとの接続状況は、行と一体にした（2026-08-20・Slack からの横展開）。**
     //
     // それまでは「リポジトリの一覧」と「組織ごとの接続状況」が**別々に並んで**おり、
@@ -6103,6 +6181,59 @@ ${deriveCapTile.toString()}
     return [el("p", { class: "ds-probe-help", text: "GitHub の応答：" + (r.error || "不明") + "。時間をおいて再度お試しください。" })];
   }
 
+  /**
+   * 「アクセス不可」の次の一手（0.12.2・実オンボーディングで踏んだ）。
+   *
+   * ## なぜ2つの導線を併記するのか
+   *
+   * それまでは「見つからないか、アーカイブされています。不要なら登録を解除」だけを
+   * 出していた。**この行がまさに、まだ読めていない主役の行**なのに、
+   * 生きているチャンネルを読めるようにする道が1本も書かれていなかった。
+   *
+   * 原因は**行の分類では絞れない**:
+   *
+   *   - 非公開チャンネルは、招待されるまで「存在しない」と返る（招待で直る）
+   *   - Slack コネクトの共有チャンネルは、**URL がどちらの組織のドメインにもなりうる**。
+   *     だから「クラスメソッド」に分類された行の正しい対処が
+   *     **ワークスペース接続（OAuth）** のこともある
+   *     （実測: 招待しようとしたら「組織外のユーザー」で断られた）
+   *
+   * **どちらかに賭けて片方だけ出すと、外したほうの人は詰まる。** 招待だけを出すと
+   * 何度招待しても直らないものを繰り返させ、接続だけを出すと招待で済む人に
+   * 「直らない処方」を渡す。**両方出して、読む人に選ばせる**のが正しい
+   * （組織不明の案内が同じ判断をしている）。
+   *
+   * ## 社内/顧客で出し分けない
+   *
+   * no_token は出し分けている（社内は画面から設定できるものが本当に無い）が、
+   * こちらは**分類そのものが当てにならない**のが問題の本体なので、
+   * 同じ理屈を持ち込むと直したい誤読を画面の側から作り直すことになる。
+   */
+  function slackUnreachableHelpNodes() {
+    var out = [];
+    out.push(el("p", { class: "ds-probe-help", text: "Cortex からこのチャンネルが見えませんでした。次のどれかです。" }));
+    var ul = el("ul", { class: "ds-probe-help ds-probe-why" });
+    // **押したあと何をするかまで書く。** 直したあと表示が変わらないと「効かなかった」と
+    // 読まれる——この画面の一覧は夜間のスナップショットなので、その場で確かめる手段は
+    // 「接続を確認」をもう一度押すこと以外に無い
+    ul.appendChild(el("li", {
+      text: "非公開チャンネル … 下のコマンドをそのチャンネルに貼って Cortex を招待してから、もう一度「接続を確認」を押してください",
+    }));
+    ul.appendChild(el("li", {
+      text: "お客様の Slack、または Slack コネクトで共有されたチャンネル … "
+        + "「クラスメソッド」と出ている行でもこちらのことがあります。"
+        + "下の「Slack ワークスペースに接続する」から接続してから、もう一度「接続を確認」を押してください",
+    }));
+    ul.appendChild(el("li", {
+      text: "アーカイブ・削除された、URL が違う … 「登録を解除」を押してください",
+    }));
+    out.push(ul);
+    // **手で打たせない**（copyRow のコメント）。招待の候補を出したなら、
+    // そのコマンドも同じ場所に無いと、貼る文字列を探しに行くことになる
+    out.push(copyRow("/invite @Cortex"));
+    return out;
+  }
+
   /**
    * 判定ごとの表示。**成功は1行で終わらせ、失敗は「理由＋次の一手」だけを残す。**
    *
@@ -6146,7 +6277,9 @@ ${deriveCapTile.toString()}
         : "「Slack ワークスペースに接続する」から接続してください。")];
     }
     if (r.gate === "unreachable") {
-      return [p("チャンネルが見つからないか、アーカイブされています。不要なら「登録を解除」してください。")];
+      // **ここだけ本文が2つ以上になる（0.12.2）。** 原因が3つあり、しかも
+      // 行の分類では絞れないので、候補と導線を並べる（関数側に理由がある）
+      return slackUnreachableHelpNodes();
     }
     // unknown。**生のエラーは隠さない**——「不明」だけだと切り分けが始められない
     return [p("Slack の応答：" + (r.error || "不明") + "。時間をおいて再度お試しください。")];

package/package.json

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