@takagaki/cortex-decisions-viewer 0.10.6 → 0.10.7

package/dist/render.js

@@ -5807,6 +5807,47 @@ ${nextStateDelay.toString()}
     }).join(",");
   }
 
+  /**
+   * Figmaファイル1件の「取り込み」の行（接続状態ストア・段C第二弾）。**無ければ null。**
+   * 表示は Backlog と同じ ingestedEl を通す（状態から文言を決める規則は1つだけ持つ）。
+   *
+   * 🔴 **鍵は正規化しない**——ingestedStateOf（課題管理）との意図的な違い。Figma の
+   * ファイルキーは**大小を区別する不透明ID**で、揃えにいくと別のファイルを同じものと見る。
+   * logic.mjs の figmaFileKey も trim だけで大小を保ち、applyAddFigmaFile の重複判定も完全一致。
+   * 突き合わせが成立する根拠は「人が大小を揃えること」ではなく、**書き手（同期ジョブ）と
+   * 画面（figmaFiles）が同じ figma.json の同じ値を読むこと**。
+   */
+  function designIngestedStateOf(key) {
+    var rows = LIVE_STATE;
+    var want = String(key || "");
+    if (!rows || !rows.length || !want) return null;
+    for (var i = 0; i < rows.length; i++) {
+      var r = rows[i];
+      // cap と question の両方で絞る。片方だけだと、同じ ref を持つ別の問いの行
+      //（別能力の行・将来 design に増える別の question）を取り込みとして読む
+      if (!r || r.cap !== "design" || r.question !== "ingested") continue;
+      if (r.ref === want) return r;
+    }
+    return null;
+  }
+
+  /**
+   * いま一覧に出ている Figma ファイルの「取り込み」をひとつの文字列にする（liveList の sig 用）。
+   *
+   * 🔴 **ファイル一覧のJSONだけでは足りない。** モーダルを開くとその能力の状況をもう一度
+   * 取りに行くので、**応答が返る前に開いた人は取り込みの行を1件も持っていない**。届いた応答で
+   * ファイルの集合は変わらないため、一覧だけを署名にすると liveList の早期return が描き直しを
+   * 飛ばし、**閉じて開き直すまで空のまま**になる。空表示は「まだ誰も測っていない」と
+   * 見分けが付かないので、この機能が殺そうとしている「取り込みの失敗を黙る」欠陥が
+   * 別の経路で復活する（課題管理側の ingestSigOf と同じ理由）。
+   */
+  function designIngestSigOf(list) {
+    return (list || []).map(function (f) {
+      var r = designIngestedStateOf(f && f.key);
+      return r ? r.status + "@" + r.at : "-";
+    }).join(",");
+  }
+
   /**
    * 「課題管理」カードの接続先（一覧＋追加＋削除・#127）。
    *
@@ -6295,15 +6336,57 @@ ${nextStateDelay.toString()}
       // **一覧は開いたままでも差し替わる箱に入れる（段階1）。** 追加は「モーダルを開いたまま」
       // 行われるので、ここが固定だと足したファイルは閉じるまで画面に出ない
       var figsOf = function () { return (internalSourceOf("デザイン") || {}).figmaFiles || []; };
-      box.appendChild(liveList("デザイン", function () { return JSON.stringify(figsOf()); }, function (listBox) {
+      /**
+       * 外し終えたファイルの鍵。**一覧の箱の外に置く（段C第二弾）。**
+       *
+       * 外した行には取り消し線を引き、ボタンだけを無効にする。その印は行の closure に
+       * あるので**描き直しで作り直される**——取り込みの状態を署名に混ぜたことで
+       * LIVE_STATE の到着でも描き直しが走るようになり、ここに覚えておかないと
+       * **外したばかりの行が生き返り、二度押しで「登録されていません」になる**
+       *（削除は取り消せないので、押せる見た目に戻すのがいちばん悪い）。
+       *
+       * **効くのは短い。** 段A補（subtractRemovedRegistered）以降、外した行は次の
+       * op=state が返った時点で一覧から消える。だからこの印が要るのは
+       * **①保存の応答から次の op=state までの数秒**と、**②op=state が取れない案件**
+       *（古い Lambda・通信失敗で、夜間の一覧が翌朝まで残る）だけ。
+       *
+       * 🔴 **一覧から消えた鍵は落とす（下の prune）。** 消えた＝サーバが削除を認めた
+       * ということなので、この印は役目を終えている。**持ち続けると、押し間違いに気づいて
+       * 同じファイルを貼り直した人の行が、復活した瞬間に「外し済み」で描かれて操作不能に
+       * なる**（回復手段はモーダルを開き直すことだけで、その手掛かりが画面に無い）。
+       * 追加の口から鍵を落とす手もあるが、あちらが受け取るのは URL で、鍵に直すには
+       * サーバ側の figmaFileKey を画面にもう1つ持つことになる（同じ規則が2箇所に増える）。
+       */
+      var figsRemoved = [];
+      /**
+       * いま送信中の行（鍵 → 行の節点）。**描き直しでも作り直さず、この節点を使い続ける。**
+       *
+       * 取り込みの状態を署名に混ぜたことで、**op=config の応答待ちの最中に op=state が
+       * 届くと行が作り直される**窓ができた。作り直すと「保存中…」と押せない状態を持つ
+       * 節点が捨てられ、**返事が来る前にもう一度押せてしまう**。押している当人の行だけを
+       * 据え置き、他の行の取り込み表示は普段どおり追いつかせる（removeButton の onBusy）。
+       */
+      var figSending = {};
+      // **署名に取り込みの状態も混ぜる（段C第二弾）。** 一覧のJSONだけだと、応答が返る前に
+      // モーダルを開いた人には取り込みが1件も出ない（designIngestSigOf の項に経緯）
+      box.appendChild(liveList("デザイン", function () {
+        return JSON.stringify(figsOf()) + "|" + designIngestSigOf(figsOf());
+      }, function (listBox) {
       var figs = figsOf();
+      // **一覧から消えた鍵は「外し済み」から落とす**（figsRemoved の項に経緯）。
+      // サーバが削除を認めた印なので、ここから先に同じ鍵が現れたら**貼り直された別の登録**
+      var figKeys = figs.map(function (f) { return f.key; });
+      figsRemoved = figsRemoved.filter(function (k) { return figKeys.indexOf(k) >= 0; });
       // **残り件数は「押した時点」で数える。** 一覧（figs）は夜間バッチのスナップショットで、
       // このページを開いている間はもちろん、その日のうちに再読み込みしても更新されない。
       // 描画時点の件数で判定すると、3件を1件ずつ外していったとき最後の1件でも警告が出ない。
-      var figsLeft = figs.length;
+      // **描き直しでも外した分を引いたままにする**（figsRemoved の項に経緯）
+      var figsLeft = figs.filter(function (f) { return figsRemoved.indexOf(f.key) < 0; }).length;
       if (figs.length) {
         var flist = el("div", { class: "ds-list" });
         figs.forEach(function (f) {
+          // 送信中の行は作り直さない（figSending の項に経緯）
+          if (figSending[f.key]) { flist.appendChild(figSending[f.key]); return; }
           // **Slack / GitHub の行と同じ構造にする（ds-card > ds-card-main >
           // ds-icon + ds-type + ds-name）。** extSourceCard は呼ばない——
           // あちらは type / gate / origin など外部ソース固有の判断を抱えている。
@@ -6314,6 +6397,8 @@ ${nextStateDelay.toString()}
               el("span", { class: "ds-type", text: "Figma" }),
               dsNameEl(f.name || f.key, "https://www.figma.com/design/" + f.key),
             ]),
+            // このファイルの取り込みが最後にどうなったか（段C）。測っていなければ null
+            ingestedEl(designIngestedStateOf(f.key)),
           ]);
           row.appendChild(removeButton({
             label: "このファイルを外す",
@@ -6339,9 +6424,21 @@ ${nextStateDelay.toString()}
             onDone: function () {
               // **クラスのハード代入をやめる**（片方だけ直し忘れる形を構造的に消す）
               row.classList.add("ds-removed");
+              if (figsRemoved.indexOf(f.key) < 0) figsRemoved.push(f.key);
               figsLeft -= 1;
             },
+            // 確認〜応答のあいだ、この行だけ描き直しの対象から外す（figSending の項に経緯）
+            onBusy: function (on) {
+              if (on) figSending[f.key] = row;
+              else delete figSending[f.key];
+            },
           }));
+          // 描き直しのあとも「外した」ことが分かる形に戻す（figsRemoved の項に経緯）
+          if (figsRemoved.indexOf(f.key) >= 0) {
+            row.classList.add("ds-removed");
+            var doneBtn = row.querySelector(".ds-toggle-btn");
+            if (doneBtn) doneBtn.disabled = true;
+          }
           flist.appendChild(row);
         });
         listBox.appendChild(flist);
@@ -6775,7 +6872,14 @@ ${nextStateDelay.toString()}
    * 成功しても一覧からは消さない。接続状況は夜間バッチのスナップショットで、消したつもりの行が
    * 翌朝まで残るのが正しい状態のため——行に取り消し線を引き、ボタンだけを無効にする
    * （二度押しすると「登録されていません」で失敗するので、押せなくする）。
-   * @param opts { label, confirmLines(), build(), onDone() }
+   * **押している間（確認〜応答）を呼び出し側に知らせる（onBusy）。**
+   * 一覧を差し替える口では、この間に描き直しが走るとこの行ごと作り直され、
+   * 「保存中…」と押せない状態を持つ節点が捨てられる——**二度押しできる見た目に戻る**。
+   * 削除は取り消せないので、行を差し替える側はこの間だけ元の節点を使い続けること。
+   * **確認の前から数える**（確認モーダルの戻り先もこの節点なので、途中で捨てると
+   * 「いいえ」を押した人のフォーカスの戻り先が消える）。
+   *
+   * @param opts { label, confirmLines(), build(), onDone(), onBusy(bool) }
    */
   function removeButton(opts) {
     var wrap = el("div", { class: "ds-toggle" });
@@ -6783,11 +6887,18 @@ ${nextStateDelay.toString()}
     var msg = el("div", { class: "ds-toggle-msg" });
 
     btn.addEventListener("click", async function () {
-      if (!(await confirmDanger(opts.confirmLines(), { returnTo: btn }))) { msg.textContent = ""; return; }
-      sendConfig(opts.build(), msg, btn, function () {
-        btn.disabled = true;
-        if (opts.onDone) opts.onDone();
-      });
+      if (opts.onBusy) opts.onBusy(true);
+      // **finally で必ず倒す。** 断られた・通信に失敗した経路で立てっぱなしにすると、
+      // 呼び出し側はその行を永久に差し替えられなくなる（sendConfig は自分で捕まえるので投げない）
+      try {
+        if (!(await confirmDanger(opts.confirmLines(), { returnTo: btn }))) { msg.textContent = ""; return; }
+        await sendConfig(opts.build(), msg, btn, function () {
+          btn.disabled = true;
+          if (opts.onDone) opts.onDone();
+        });
+      } finally {
+        if (opts.onBusy) opts.onBusy(false);
+      }
     });
 
     wrap.appendChild(btn);

package/package.json

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