@takagaki/cortex-decisions-viewer 0.4.70 → 0.4.71

package/dist/render.js

@@ -107,18 +107,19 @@ const ADAPTER_CATALOG = {
     "開発": ["github"],
 };
 /**
- * 導出された状態（5値）→ 画面に出す文言。**マップとタイルで同じ語を使う。**
+ * 導出された状態 → 画面に出す文言。**マップとタイルで同じ語を使う。**
  *
  * - `off` は「OFFに設定されています」。**接続へ誘わない**——意図的に止めたものに
  *   「接続しましょう」と出すのは、設定する必要が無いものを押させるのと同じ誤り。
- * - `unset` は「未接続」。こちらは入口を出してよい（まだ何も決めていない状態）。
- * - `empty` を `unset` に潰さない。設定ファイルはあるので「置き直す」ではなく「中身を足す」。
+ * - `unset` は「未接続」。こちらは入口を出してよい。
+ *
+ * **`empty` の行は無い。** 表示に入る前に `capabilityDisplayState` が unset へ畳むので、
+ * ここへ来ることはない（オーナー決定・2026-08-12。理由はその関数の項に書いてある）。
  */
 const CAP_STATE_TEXT = {
     unset: "未接続",
     off: "OFFに設定されています",
     broken: "設定に異常があります",
-    empty: "設定はありますが、対象がありません",
     on: "接続OK",
 };
 /** 状態の印。CAP_STATE_TEXT の前に付ける（マップは付けない——⚠️バッジと二重になる） */
@@ -243,6 +244,27 @@ function capabilityRawState(cap, caps) {
     var s = c.state == null ? "" : String(c.state);
     return s ? s : null;
 }
+/**
+ * **画面が使う状態。`empty` を `unset` に畳む。**（オーナー決定・2026-08-12）
+ *
+ * scaffold が設定ファイルを必ず作るので、「設定ファイルはある」は利用者にとって無情報。
+ * ダミーキーのまま置かれている＝繋がっていない、が実態で、`unset` と `empty` の違いは
+ * **実装の都合**でしかない。利用者にとっての答えはどちらも「未接続」。
+ * 実機の受け入れテストで「設定はありますが、対象がありません」が何を指すのか伝わらず、
+ * 次の一手（＝接続する）に繋がらなかった。
+ *
+ * **engine の状態機械は変えない。** `empty` は今後も導出されてくる（プレースホルダを
+ * 実キーと数えない判定は engine の責務で、そこを潰すと無駄な同期が走る #125①）。
+ * 畳むのは表示だけ——**この1関数だけ**が写像を持つので、「やっぱり区別したい」に
+ * なったらここの1行を消せば5値の表示へ戻る。
+ *
+ * 畳んだ先が unset なので、文言・接続の入口の案内・マップの線の有無・グレーの見た目は
+ * すべて unset と同一になる。`off` / `broken` の区別は従来どおり残す。
+ */
+function capabilityDisplayState(cap, caps) {
+    var s = capabilityRawState(cap, caps);
+    return s === "empty" ? "unset" : s;
+}
 /**
  * 貼られたURLから Backlog の接続先を読む。
  *
@@ -317,7 +339,11 @@ function capabilityState(cap, ctx) {
                 : "ok";
         }
         // unset / empty / 未知の値。**未知を ok にも off にも寄せない**——どちらも嘘になる。
-        // 「まだ繋がっていない」側に倒すのが安全（繋がっていないのに線を引かない）
+        // 「まだ繋がっていない」側に倒すのが安全（繋がっていないのに線を引かない）。
+        //
+        // ここは**生の状態**（capabilityRawState）で見ている。empty と unset は元々どちらも
+        // idle に落ちるので、表示側の写像（capabilityDisplayState が empty を unset に畳む）と
+        // 答えが割れることはない。**この2つを分けるなら、写像とここの両方を直すこと。**
         return "idle";
     }
     // ---- ここから下は capabilities を持たない古い fleet-status.json のための劣化表示 ----
@@ -406,7 +432,7 @@ function renderConnectionMap(site) {
         const idle = st === "idle";
         const warn = st === "warn";
         const state = "cm-" + st;
-        const derived = capabilityRawState(c.cap, caps);
+        const derived = capabilityDisplayState(c.cap, caps);
         // **ラベルは能力名（会議・課題管理…）。ツール名は下に小さく置く。**
         // ツールは「その能力をいま何で実現しているか」という差し替え可能なアダプタで、
         // 主語ではない。ツール名を主にすると、Teams のように複数の能力に使われるものが
@@ -1157,8 +1183,24 @@ a.ds-name:hover { text-decoration: underline; }
 .ds-check-detail { font-size: 12px; color: var(--muted); }
 .ds-tile-desc { font-size: 12.5px; color: var(--muted); }
 .ds-tile-sub { font-size: 12.5px; margin-top: 2px; }
-.ds-tile-chev { color: var(--muted); }
+.ds-tile-chev { color: var(--muted); flex: 0 0 auto; }
 .ds-tile-note { font-size: 12.5px; color: #a15c00; margin-top: 2px; }
+/* ---- 可変長の文字列は必ず折り返す（0.4.71） ----
+   実機の受け入れテストで、状態の文言がタイル幅を超えて溢れた（スシローのデザインタイル）。
+   ここに出るのは engine の detail・チェックの直し方・チャンネル名・ファイルパスなど
+   **長さが読めない**文字列で、増えるたびに個別に手当てしていたら必ず取りこぼす。
+   **省略（ellipsis）ではなく折り返しで受ける**——何が足りないのかを伝える文なので、
+   途中で切ると画面の目的そのものが消える（名前だけの .ds-name は従来どおり省略のまま）。
+     white-space: normal  … button の中に置く行があるので UA 既定に委ねない
+     overflow-wrap: anywhere … 空白の無い連なり（課題管理/issues-config.json・ID・URL）を切る
+     min-width: 0         … flex の子が min-content より縮めるようにする（無いと折り返しても溢れる） */
+.ds-tile-name, .ds-tile-desc, .ds-tile-sub, .ds-tile-note,
+.ds-check-label, .ds-check-action, .ds-check-detail,
+.ds-pipe-label, .ds-pipe-todo, .ds-owner-state,
+.ds-empty, .sec-how, .modal-desc, .ds-toggle-msg,
+.ds-drive-note, .ds-inactive-note, .ds-legacy-chip, .ds-keys-code {
+  min-width: 0; white-space: normal; overflow-wrap: anywhere;
+}
 .badge.ds-undeclared { background: #fff4e5; color: #9a6700; }
 .ds-toggle { display: flex; align-items: center; gap: 10px; flex-wrap: wrap; margin-top: 6px; }
 .ds-toggle-btn { font-size: 12px; padding: 3px 10px; border-radius: 999px; border: 1px solid var(--border); background: #fff; color: var(--text); cursor: pointer; }
@@ -1366,6 +1408,7 @@ var CAP_STATE_MARK = ${JSON.stringify(CAP_STATE_MARK)};
 ${externalCapability.toString()}
 ${capabilityAdapter.toString()}
 ${capabilityRawState.toString()}
+${capabilityDisplayState.toString()}
 ${backlogTargetOf.toString()}
 ${sourceNeedsAttention.toString()}
 ${capabilityState.toString()}
@@ -2347,13 +2390,20 @@ ${capabilityState.toString()}
         }));
         return;
       }
-      // **未接続（unset）の能力には入力欄を出さない（#125②）。** 鍵は「繋いだものを読むため」の
-      // ものなので、繋ぐ先が無い状態で入れても効かない。何も出さないと「鍵はどこで入れるのか」が
-      // 分からなくなるので、順序だけを1行で伝える
-      if (capabilityRawState(SECRET_CAP[k], caps) === "unset") {
+      // **意図的にOFFにした能力にだけ、入力欄を出さない（0.4.71）。** 使わないと決めたものに
+      // 鍵を求めるのは、設定する必要が無いものに設定を促す偽の警告と同じ。
+      // 何も出さないと「トークンはどこで入れるのか」が分からなくなるので、理由を1行だけ残す
+      // （上の GitHub の畳み方と同じ流儀）。
+      //
+      // **未接続（unset・畳んだ empty）には出す。** 0.4.70 では「繋ぐ先が無い状態で鍵を
+      // 入れても効かない」として畳んでいたが、engine の checks は「この画面の入力欄に
+      // 貼ってください」と案内しており、**入力欄が無いのは案内の嘘**だった（cortex-context の
+      // デザインで実際にそうなっていた）。「未接続＝これから繋げる入口」なら、その入口に
+      // 最初の一歩を置けるほうが整合する。broken も出す——鍵を貼り直して直る場合がある
+      if (capabilityDisplayState(SECRET_CAP[k], caps) === "off") {
         pushCap(SECRET_CAP[k], el("p", {
-          class: "sec-how",
-          text: "この能力はまだ接続されていません。接続すると、ここに鍵の入力欄が出ます。",
+          class: "sec-how ds-all-connected",
+          text: "この能力はOFFに設定されています。使う場合は設定ファイルで有効にしてください（鍵の登録はそのあとで構いません）。",
         }));
         return;
       }
@@ -2525,7 +2575,7 @@ ${capabilityState.toString()}
         // その状態を割り当て、残りは未接続——「知らない」を「繋がっている」に寄せない
         var active = capabilityAdapter(cap, caps);
         adapters.forEach(function (a, ai) {
-          renderAdapterTile(cap, items, a, a === active ? capabilityRawState(cap, caps) : "unset", ai);
+          renderAdapterTile(cap, items, a, a === active ? capabilityDisplayState(cap, caps) : "unset", ai);
         });
       } else if (items.length || declared) {
         // 旧データ: アダプタのタイルは宣言か行があるときだけ。手動投入だけが理由でここへ来た
@@ -2545,7 +2595,7 @@ ${capabilityState.toString()}
      */
     function renderAdapterTile(cap, items, adapter, derived, ai) {
       var meta = CAP_META[cap] || { icon: "🔗", desc: "" };
-      var st = derived === "unset" && capabilityRawState(cap, caps) !== "unset"
+      var st = derived === "unset" && capabilityDisplayState(cap, caps) !== "unset"
         // カタログにあるが導出が答えていないアダプタ（今日は発生しない。#107 の前方互換）
         ? "idle"
         : capabilityState(cap, { tools: tools, capabilities: caps, internals: internals, externals: sources, pipes: pipes });
@@ -2564,10 +2614,16 @@ ${capabilityState.toString()}
         el("div", { class: "ds-tile-sub ds-gate-" + st, text: tileSub(cap, st, derived, adapter) }),
       ]);
       // **なぜその状態なのかを添える（#125②）。** engine が書いた一文（detail）で、
-      // 「未接続」だけでは分からない「何が無いのか」を出す。on のときは
-      // 読み取り範囲（件数）が既に同じ行にあるので添えない
+      // 「何が無いのか」を出す。on のときは読み取り範囲（件数）が既に同じ行にある。
+      //
+      // **未接続（unset・畳んだ empty）はタイル表面に出さない（0.4.71）。** 未接続の答えは
+      // 「まだ繋がっていない」の一言で足り、files が空です／figma.json がありません、の
+      // 区別は押した先で読めばよい実装の話。表面に置くと一覧が理由文で埋まるうえ、
+      // 実機で幅を超えて溢れた。off（何件残して止めたか）と broken（異常）は
+      // 押さずに気づけないと困るので、表面に残す
       var capInfo = caps[cap];
-      if (derived && derived !== "on" && capInfo && capInfo.detail && adapter === capabilityAdapter(cap, caps)) {
+      var detailOnTile = derived === "off" || derived === "broken";
+      if (detailOnTile && capInfo && capInfo.detail && adapter === capabilityAdapter(cap, caps)) {
         body.appendChild(el("div", { class: "ds-tile-note ds-tile-todo", text: capInfo.detail }));
       }
       // トークン未設定は**状態とは別軸**。状態機械に混ぜると判定が1箇所でなくなる。
@@ -2583,10 +2639,13 @@ ${capabilityState.toString()}
       // 案件に未設定の警告を出すと、直す必要が無いものを直させることになる。
       // 読めていない組織があるときは、この下の checks とタイルの状態（warn）が拾う
       var tokenIsFallback = sk === "external-sources" && !githubTokenNeeded();
-      // **#125②: 導出があるときは導出で判断する。** OFF（意図的に止めた）と unset（未接続で、
-      // 鍵の入力欄そのものを案内文に畳んである）には出さない——入れる先が無い、あるいは
-      // 入れる必要が無いものについて設定を促す偽の警告になる
-      var tokenRelevant = derived != null ? derived !== "off" && derived !== "unset" : tools[cap] !== "none";
+      // **#125②: 導出があるときは導出で判断する。** OFF（意図的に止めた）にだけ出さない
+      // ——入れる必要が無いものについて設定を促す偽の警告になる。
+      //
+      // **注記の条件は、入力欄を出す条件と同じにする（0.4.71）。** 別々にすると
+      // 「入力欄はあるのに未設定だと言われない」「言われるのに入力欄が無い」のどちらかが起きる。
+      // 0.4.70 は unset で入力欄を畳んでいたので注記も抑えていたが、案A で欄を出すようにしたため揃えた
+      var tokenRelevant = derived != null ? derived !== "off" : tools[cap] !== "none";
       if (!tokenIsFallback && tokenRelevant && sk && (SITE.secretKinds || []).indexOf(sk) >= 0 && !((fleetVal("secretSources") || {})[sk])) {
         body.appendChild(el("div", { class: "ds-tile-note", text: "⚠ トークンが未設定です" }));
       }

package/package.json

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