@takagaki/cortex-decisions-viewer 0.12.32 → 0.12.33

package/dist/render.js

@@ -16,6 +16,7 @@ exports.overlayStateOnFleet = overlayStateOnFleet;
 exports.chatRowsToAdd = chatRowsToAdd;
 exports.gchatRowsToAdd = gchatRowsToAdd;
 exports.devRowsToAdd = devRowsToAdd;
+exports.markExcludedPendingRows = markExcludedPendingRows;
 exports.designRowsToAdd = designRowsToAdd;
 exports.materialRowsToAdd = materialRowsToAdd;
 exports.issuesRefOf = issuesRefOf;
@@ -1011,6 +1012,44 @@ function devRowsToAdd(externals, rows) {
     }
     return out;
 }
+/**
+ * 仮置きで足した GitHub の行に、「読み取り停止中」の印を写す（#409 の後追い）。
+ *
+ * 🔴 **仮置きの行は印を持たない。** `pendingExternalRow` が作れるのは payload から
+ * 切り出せる値だけで、その repo が `exclude` に居るかは登録ファイル側にしかない。
+ *
+ * それが問題になるのはこの一続きのときだけ:
+ * 読み取りを止めた repo を**もう一度追加すると、サーバは「既に登録されています」と断る**
+ *（`sources` には残っているため）。#409 で画面はその断りを事実として受け取り行を出すように
+ * なったが、**印が無いので「読み取り中の行」として並ぶ**——止めたはずのものが動いて
+ * 見えるので、**止め忘れたと思ってもう一度止めに行く**ことになる。まとめ表にも
+ * 即時読み取りの `dev` にもその行は居ない（除外は `devExcluded` に分かれている）ので、
+ * 他のどこからも印は付かない。
+ *
+ * **クロージャを持たない純関数に保つこと**（`.toString()` でクライアントへ渡す）。
+ */
+function markExcludedPendingRows(view, reg) {
+    const v = view || {};
+    const rows = v.externalSources;
+    if (!reg || !Array.isArray(reg.devExcluded) || !Array.isArray(rows))
+        return v;
+    const excluded = {};
+    for (const r of reg.devExcluded)
+        if (r && r.ref)
+            excluded[String(r.ref).toLowerCase()] = true;
+    let changed = false;
+    const next = rows.map(function (s) {
+        if (!s || s.pending !== true || !s.ref || s.goldState)
+            return s;
+        if (String(s.type || "").indexOf("github") !== 0)
+            return s;
+        if (excluded[String(s.ref).toLowerCase()] !== true)
+            return s;
+        changed = true;
+        return Object.assign({}, s, { goldState: "excluded" });
+    });
+    return changed ? Object.assign({}, v, { externalSources: next }) : v;
+}
 /** デザインの行で、まとめ表がまだ知らないもの（鍵はファイルキー） */
 function designRowsToAdd(files, rows) {
     const have = {};
@@ -3604,6 +3643,7 @@ ${sanitizeRegisteredNow.toString()}
 ${chatRowsToAdd.toString()}
 ${gchatRowsToAdd.toString()}
 ${devRowsToAdd.toString()}
+${markExcludedPendingRows.toString()}
 ${designRowsToAdd.toString()}
 ${materialRowsToAdd.toString()}
 ${issueRowsToAdd.toString()}
@@ -4053,6 +4093,58 @@ ${deriveCapTile.toString()}
   function codeUrl(prompt) {
     return claudeCodeUrl(prompt, REPO_SLUG, fleetVal("externalSources") || []);
   }
+  /**
+   * 🔴 **「1件も登録されていない」と「確かめられなかった」を分ける（#409）。**
+   *
+   * サーバは登録ファイルを読めなかった能力を、**キーごと応答から落とす**
+   *（「読めなかった」を「0件」と混ぜないため。intake の buildRegisteredNow）。
+   * ところが画面はそれを「足す行が無い」としか受け取っていなかったので、
+   * **確かめられなかった結果が「登録されていません」として出ていた**。
+   * オーナーはそれを見て未登録だと読み、追加して「既に登録されています」と断られた。
+   *
+   * 🔴 **能力ごとのキーの有無で判定してはいけない（実装レビューの指摘）。** サーバは
+   * 「読めなかった」ときだけでなく **「そのファイルがまだ無い」ときにも同じくキーを落とす**
+   * （読んだ本文が null でも undefined でも、文字列でなければ同じ枝に落ちる）。
+   * キーの有無で見ると、**その能力をまだ使っていない案件**——つまり立ち上げ直後の、
+   * この画面が一番よく見られる状態——に**永久に消えない誤警報**が出る
+   * （「再読み込みしてください」と言われても、設定ファイルは再読み込みでは生えない）。
+   * しかも製品としては、その状態は「登録されていません」と言い切ると決めてある
+   * （engine の capabilities: ファイルの有無は実装の都合で、利用者にはどちらも「まだ登録していない」）。
+   *
+   * **だから応答まるごとで見る。** 登録ファイルの一覧（registeredNow）と
+   * 能力の状態（capabilitiesNow）は**サーバの同じ return から出る**ので、
+   * **両方とも無いときだけ**が「そもそも読みに行けなかった」——今回の症状の
+   * 3経路（ツリー取得の失敗・ツリーの打ち切り・応答そのものの失敗）は全部これに当たる。
+   *
+   * ⚠ この粒度では、**ファイル1本だけが読めなかった回**（8秒の打ち切り・単発の失敗）は
+   * 拾えない。そちらはサーバがログを出す経路なので、まず記録から追えるようにしてある。
+   */
+  function registrationUnreadable() {
+    return liveAnswered && LIVE_REGISTERED === null && LIVE_CAPS === null;
+  }
+  /**
+   * 一覧に添える1行。**帯は症状・行は一手・繰り返さない。**
+   * 応答がまだ届いていない間は出さない（確かめる前に「確かめられなかった」と言わない）。
+   *
+   * **0件のときは一覧の代わり**——空の一覧そのものを出さない（オーナーが踏んだのは
+   * 「空の一覧を見て未登録だと読んだ」ことなので）。
+   *
+   * 🔴 **行があるときも黙らない。** まとめ表が行を持っていれば一覧は出るが、
+   * **その日足したばかりの行だけが欠けたまま、完全な一覧に見える**——利用者の読み違い方は
+   * 0件のときと同じで、「自分が足したものは登録されていない」と読む。行は本物なので消さず、
+   * **欠けているかもしれないことだけを添える**。
+   *
+   * ⚠ 「表示は前の晩の内容です」とは言わない。一覧にはまとめ表（前の晩）だけでなく
+   * **このブラウザが覚えている反映待ちの行も混ざる**ので、言い切ると画面自身が嘘をつく。
+   * 利用者が知りたいのは日付ではなく「自分が足したものが抜けているかどうか」。
+   */
+  function unverifiedListNote(noun, hasRows) {
+    if (!registrationUnreadable()) return null;
+    var head = "登録されている" + noun + "を確認できませんでした。";
+    return el("div", { class: "ds-note", text: hasRows
+      ? head + "最近足したものが抜けているかもしれません。再読み込みしてください。"
+      : head + "再読み込みしてください。" });
+  }
   // AI 導線の a はここでだけ作る。href を決めてよいのは codeUrl だけ（#404）。
   // 指示文を属性に残すのは、接続状況の応答が届いたときに href を組み直すため（syncAiLinks）
   function aiLink(cls, prompt, text) {
@@ -6499,7 +6591,7 @@ ${deriveCapTile.toString()}
       // 状態ストアの答えは常に勝つ（まとめ表は gate を焼き込まない。overlayStateOnFleet の項）
       if (!MERGED) {
         MERGED = overlayStateOnFleet(
-          applyPendingOps(mergeRegisteredFleet(rawView(), LIVE_REGISTERED), pend),
+          markExcludedPendingRows(applyPendingOps(mergeRegisteredFleet(rawView(), LIVE_REGISTERED), pend), LIVE_REGISTERED),
           LIVE_STATE,
         );
       }
@@ -7431,44 +7523,6 @@ ${deriveCapTile.toString()}
      * （素の Decisions ビュー）で行も無い能力に空の箱を積むと、押しても何も無いモーダルになる。
      */
     function liveListWanted(kind) { return !!SITE.intake || liveExternals(kind).length > 0; }
-    /**
-     * 🔴 **「1件も登録されていない」と「確かめられなかった」を分ける（#409）。**
-     *
-     * サーバは登録ファイルを読めなかった能力を、**キーごと応答から落とす**
-     *（「読めなかった」を「0件」と混ぜないため。intake の buildRegisteredNow）。
-     * ところが画面はそれを「足す行が無い」としか受け取っていなかったので、
-     * **確かめられなかった結果が「登録されていません」として出ていた**。
-     * オーナーはそれを見て未登録だと読み、追加して「既に登録されています」と断られた。
-     *
-     * 🔴 **能力ごとのキーの有無で判定してはいけない（実装レビューの指摘）。** サーバは
-     * 「読めなかった」ときだけでなく **「そのファイルがまだ無い」ときにも同じくキーを落とす**
-     * （読んだ本文が null でも undefined でも、文字列でなければ同じ枝に落ちる）。
-     * キーの有無で見ると、**その能力をまだ使っていない案件**——つまり立ち上げ直後の、
-     * この画面が一番よく見られる状態——に**永久に消えない誤警報**が出る
-     * （「再読み込みしてください」と言われても、設定ファイルは再読み込みでは生えない）。
-     * しかも製品としては、その状態は「登録されていません」と言い切ると決めてある
-     * （engine の capabilities: ファイルの有無は実装の都合で、利用者にはどちらも「まだ登録していない」）。
-     *
-     * **だから応答まるごとで見る。** 登録ファイルの一覧（registeredNow）と
-     * 能力の状態（capabilitiesNow）は**サーバの同じ return から出る**ので、
-     * **両方とも無いときだけ**が「そもそも読みに行けなかった」——今回の症状の
-     * 3経路（ツリー取得の失敗・ツリーの打ち切り・応答そのものの失敗）は全部これに当たる。
-     *
-     * ⚠ この粒度では、**ファイル1本だけが読めなかった回**（8秒の打ち切り・単発の失敗）は
-     * 拾えない。そちらはサーバがログを出す経路なので、まず記録から追えるようにしてある。
-     */
-    function registrationUnreadable() {
-      return liveAnswered && LIVE_REGISTERED === null && LIVE_CAPS === null;
-    }
-    /**
-     * 空の一覧の代わりに出す1行。**帯は症状・行は一手・繰り返さない。**
-     * 空の一覧そのものを出さない——今回オーナーが踏んだのは「空の一覧を見て未登録だと読んだ」ことなので。
-     * 応答がまだ届いていない間は出さない（確かめる前に「確かめられなかった」と言わない）。
-     */
-    function unverifiedListNote(noun) {
-      if (!registrationUnreadable()) return null;
-      return el("div", { class: "ds-note", text: "登録されている" + noun + "を確認できませんでした。再読み込みしてください。" });
-    }
     // **箱にはアダプタのタグを付ける（#153）。** pushCap の無タグは「どのタイルにも出す」
     // 既定なので、タグを忘れると Slack の一覧が Google Chat モーダルに丸ごと出る
     if (liveListWanted("slack")) pushCap("チャット", liveList(
@@ -7476,11 +7530,10 @@ ${deriveCapTile.toString()}
       function () { return externalsSig("slack", slackWorkspaces()); },
       function (box) {
         var items = liveExternals("slack");
-        if (!items.length) {
-          var noteSlack = unverifiedListNote("チャンネル");
-          if (noteSlack) box.appendChild(noteSlack);
-          return;
-        }
+        // 注記は**行より前**に置く（0件なら一覧の代わり・行があれば添え書き）
+        var noteSlack = unverifiedListNote("チャンネル", items.length > 0);
+        if (noteSlack) box.appendChild(noteSlack);
+        if (!items.length) return;
         var meta = DS_TYPE["slack"] || { icon: "🔗", label: "Slack" };
         slackRows(items, meta).forEach(function (n) { box.appendChild(n); });
       },
@@ -7492,11 +7545,10 @@ ${deriveCapTile.toString()}
       function () { return externalsSig("google-chat", null); },
       function (box) {
         var items = liveExternals("google-chat");
-        if (!items.length) {
-          var noteGchat = unverifiedListNote("スペース");
-          if (noteGchat) box.appendChild(noteGchat);
-          return;
-        }
+        // 注記は**行より前**に置く（0件なら一覧の代わり・行があれば添え書き）
+        var noteGchat = unverifiedListNote("スペース", items.length > 0);
+        if (noteGchat) box.appendChild(noteGchat);
+        if (!items.length) return;
         var meta = DS_TYPE["google-chat"] || { icon: "💬", label: "Google Chat" };
         items.forEach(function (s) { box.appendChild(extSourceCard(s, meta)); });
       },
@@ -7506,11 +7558,10 @@ ${deriveCapTile.toString()}
       function () { return externalsSig("github", githubOwnersView() || null); },
       function (box) {
         var items = liveExternals("github");
-        if (!items.length) {
-          var noteDev = unverifiedListNote("リポジトリ");
-          if (noteDev) box.appendChild(noteDev);
-          return;
-        }
+        // 注記は**行より前**に置く（0件なら一覧の代わり・行があれば添え書き）
+        var noteDev = unverifiedListNote("リポジトリ", items.length > 0);
+        if (noteDev) box.appendChild(noteDev);
+        if (!items.length) return;
         // **meta は行ごとに引く**（issues と discussions でアイコンとラベルが違う）
         githubRows(items).forEach(function (n) { box.appendChild(n); });
       },
@@ -9512,10 +9563,14 @@ ${deriveCapTile.toString()}
         box.appendChild(liveList("共有資料", function () {
           var d = driveNow();
           // 仮置きの印（drivePending）も署名に入れる——外れたときに印だけ残さない（#388）
-          return JSON.stringify([d.driveFolderIds || [], d.driveFolderNames || {}, d.driveState || "", d.drivePending || {}]);
+          // 「確かめられたか」も混ぜる（#409。混ぜないと注記の要否が変わっても作り直されない）
+          return JSON.stringify([d.driveFolderIds || [], d.driveFolderNames || {}, d.driveState || "", d.drivePending || {}, registrationUnreadable()]);
         }, function (listBox) {
         var src = driveNow();
         var folders = src.driveFolderIds || [];
+        // 兄弟の穴を一緒に塞ぐ（#409）。ここも同じ registeredNow から行を足している
+        var noteDrive = unverifiedListNote("フォルダ", folders.length > 0);
+        if (noteDrive) listBox.appendChild(noteDrive);
         // 名前は任意。**無ければIDを出す**（既存の案件は名前を持っていない）
         var fnames = src.driveFolderNames || {};
         if (folders.length) {
@@ -9651,9 +9706,13 @@ ${deriveCapTile.toString()}
       // **署名に取り込みの状態も混ぜる（段C第二弾）。** 一覧のJSONだけだと、応答が返る前に
       // モーダルを開いた人には取り込みが1件も出ない（designIngestSigOf の項に経緯）
       figGroup.appendChild(liveList("デザイン", function () {
-        return JSON.stringify(figsOf()) + "|" + designIngestSigOf(figsOf());
+        // 「確かめられたか」も混ぜる（#409。混ぜないと注記の要否が変わっても作り直されない）
+        return JSON.stringify(figsOf()) + "|" + designIngestSigOf(figsOf()) + "|" + registrationUnreadable();
       }, function (listBox) {
       var figs = figsOf();
+      // 兄弟の穴を一緒に塞ぐ（#409）。ここも同じ registeredNow から行を足している
+      var noteFig = unverifiedListNote("ファイル", figs.length > 0);
+      if (noteFig) listBox.appendChild(noteFig);
       // **一覧から消えた鍵は「外し済み」から落とす**（figsRemoved の項に経緯）。
       // サーバが削除を認めた印なので、ここから先に同じ鍵が現れたら**貼り直された別の登録**
       var figKeys = figs.map(function (f) { return f.key; });

package/package.json

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