@takagaki/cortex-decisions-viewer 0.4.76 → 0.4.77

package/dist/build.js

@@ -312,18 +312,37 @@ async function readFleetSources(repoRoot) {
     // **画面に現在値を出すために要る。** 「接続先を変更」フォームが入力欄だけだったため、
     // 実機で「何もセットされていない」と読まれ、同じURLが貼り直された（#127）。
     //
-    // **domain / projectKey しか通さない。** 生成側は configPath / sections / strays /
-    // features も持っているが、画面が使うのは接続先の2つだけ。素通しにすると、engine が
+    // **接続先そのものしか通さない。** 生成側は configPath / sections / strays /
+    // features も持っているが、画面が使うのは接続先だけ。素通しにすると、engine が
     // このオブジェクトに足したものが顧客の画面データへ黙って流れる（Lambda 側の
     // FLEET_PUBLIC_KEYS と同じ考え方で、こちらも許可リストにする）。
+    // 配列の要素も同じ2つに絞る——要素だけ素通しにすると、許可リストを通したつもりで
+    // 中身が漏れる（externalSources で実際にやった間違い）。
+    //
+    // **接続先は複数ありうる（#127）。** 1案件が複数の Backlog プロジェクトを同期できる
+    // ようになったので、`targets` があればそれを一覧の正本にする。**単数の2つは残す**
+    // （先頭と同じ値）——`targets` を載せない古いスナップショットでも現在値を出せるように、
+    // 読み手の両対応と同じ約束を画面データでも守る。
     //
     // **値が無い形は落とす。** 正本もミラーも持たない案件では `{ source: "secrets" }` だけが
     // 来る。空の器を通すと、画面は「接続先の情報がある」と判断して空文字を並べることになる
     const bt = obj.backlogTarget != null && typeof obj.backlogTarget === "object" && !Array.isArray(obj.backlogTarget)
         ? obj.backlogTarget
         : null;
-    const backlogTarget = bt && typeof bt.domain === "string" && bt.domain && typeof bt.projectKey === "string" && bt.projectKey
-        ? { domain: bt.domain, projectKey: bt.projectKey }
+    const oneTarget = (v) => {
+        if (v == null || typeof v !== "object")
+            return null;
+        const o = v;
+        return typeof o.domain === "string" && o.domain && typeof o.projectKey === "string" && o.projectKey
+            ? { domain: o.domain, projectKey: o.projectKey }
+            : null;
+    };
+    const btTargets = Array.isArray(bt?.targets)
+        ? bt.targets.map(oneTarget).filter((t) => t !== null)
+        : [];
+    const btHead = oneTarget(bt) ?? btTargets[0] ?? null;
+    const backlogTarget = btHead
+        ? { ...btHead, ...(btTargets.length ? { targets: btTargets } : {}) }
         : undefined;
     const secretSources = obj.secretSources && typeof obj.secretSources === "object"
         ? obj.secretSources

package/dist/render.js

@@ -1304,6 +1304,12 @@ a.ds-name:hover { text-decoration: underline; }
 .ds-current-label { color: var(--muted); flex: 0 0 auto; }
 .ds-current-value { color: var(--text); font-weight: 600; word-break: break-all; }
 .ds-current-value.is-unset { color: var(--muted); font-weight: 400; }
+/* 接続先の一覧（#127）。**1件のときは従来どおりラベルの右に1行**——単数の案件で
+   見た目が変わると、複数対応と関係のない全案件に「何か変わった」と読ませる */
+.ds-target-list { display: flex; flex-direction: column; gap: 6px; min-width: 0; }
+.ds-target-row { display: flex; align-items: baseline; gap: 10px; flex-wrap: wrap; }
+/* 行の中のボタンは行に揃える（.ds-toggle の縦マージンは節と節の間隔なので効かせない） */
+.ds-target-row .ds-toggle-btn { flex: 0 0 auto; }
 /* 入力欄のすぐ下。貼る直前に読める位置に置く */
 .sec-note { color: var(--muted); font-size: 12px; line-height: 1.7; margin: 6px 0 0; }
 .sec-how p { margin: 0; }
@@ -2875,31 +2881,200 @@ ${capabilityState.toString()}
   }
 
   /**
-   * いま同期している Backlog の接続先（#127）。
+   * いま同期している Backlog の接続先の一覧（#127）。**1件でも配列で返す。**
    *
-   * **2つの経路から引く。** 正本は fleet の backlogTarget（engine が正本
-   * issues-config.json → ミラーの順で解決した値）だが、**このキーを載せているのは
-   * 新しい engine だけ**で、艦隊の多くは @v1 のまま持っていない。無いときは
-   * 内部ソースの url（同じ値から機械導出されたプロジェクトURL）を解析して補う。
+   * **3つの経路から引く。** 正本は fleet の backlogTarget で、接続先が複数ある案件では
+   * targets に全件が入る。targets を載せていないスナップショット（複数対応より前に
+   * 生成されたもの）では単数の2つを1件として読み、それも無ければ内部ソースの url
+   * （同じ値から機械導出されたプロジェクトURL）を解析して補う。
    * ここを backlogTarget だけにすると、**設定済みの案件に「未設定」と出す**
-   * ——今回直している誤読（何もセットされていないと読んで貼り直す）を、
-   * 画面の側からもう一度作ることになる。
+   * ——直したい誤読（何もセットされていないと読んで貼り直す）を画面の側から作ることになる。
    *
-   * **解析できないときは黙らない。** 「未設定」と明示する。行ごと消すと、
-   * 「表示が無い＝まだ何も無い」なのか「出し忘れ」なのかが読み手に区別できない。
+   * **半端な要素は落とす。** ドメインかキーの片方しか無い接続先は行として読めない
+   * （「cm1.backlog.jp / 」が並ぶ）。空になったら「未設定」に倒す。
    */
-  function currentBacklogTargetLine(s) {
+  function backlogTargetsNow(s) {
     var t = fleetVal("backlogTarget");
-    if (!t || !t.domain || !t.projectKey) t = (s && s.url && backlogTargetOf(s.url)) || null;
-    var p = el("p", { class: "ds-current" });
-    p.appendChild(el("span", { class: "ds-current-label", text: "現在の接続先" }));
-    p.appendChild(
-      el("span", {
-        class: t ? "ds-current-value" : "ds-current-value is-unset",
-        text: t ? t.domain + " / " + t.projectKey : "未設定",
-      }),
-    );
-    return p;
+    var raw = t && Array.isArray(t.targets) ? t.targets : t ? [t] : [];
+    var out = [];
+    for (var i = 0; i < raw.length; i++) {
+      var x = raw[i];
+      if (x && x.domain && x.projectKey) out.push({ domain: String(x.domain), projectKey: String(x.projectKey) });
+    }
+    if (out.length) return out;
+    var u = (s && s.url && backlogTargetOf(s.url)) || null;
+    return u ? [u] : [];
+  }
+
+  /**
+   * 接続先の一覧を1つの文字列にする（**サーバへ送る「いま画面が見ている世代」**・#127）。
+   *
+   * 追加・削除は一覧に対する操作なので、後勝ちで上書きすると**他の人が足した接続先が
+   * 黙って消える**。サーバは受け取った世代といまの正本を突き合わせ、違えば 409 で断る。
+   *
+   * **並びと大小は問わない**（サーバ側の targetsGeneration と同じ畳み方にすること）。
+   * 片方だけが大文字化していると、正しい操作が毎回 409 になって**再読み込みしても直らない**
+   * ——食い違わないことは backlog-targets-ui.test.mjs が実物どうしで突き合わせている。
+   *
+   * **クロージャを持たない純関数に保つこと**（テストが配信物から切り出して直接呼ぶ）。
+   */
+  function targetsGenerationOf(list) {
+    return (list || [])
+      .filter(function (t) { return t && t.domain && t.projectKey; })
+      .map(function (t) { return String(t.domain).toLowerCase() + "/" + String(t.projectKey).toUpperCase(); })
+      .sort()
+      .join(",");
+  }
+
+  /**
+   * 他の人が先に一覧を変えていたときの文面（#127）。
+   *
+   * **サーバの文面をそのまま出さない。** サーバは「画面を再読み込みして…」と案内するが、
+   * 画面は応答に添えられた一覧でその場で追いついている。再読み込みを促すと、
+   * 済んでいることをやらせる（そして再読み込みしても同じ表示になる＝堂々巡りに見える）。
+   */
+  var TARGETS_RACE_NOTE =
+    "他の人が先に接続先を変更しました。最新の一覧に更新しましたので、内容を確かめてもう一度お試しください。";
+
+  /**
+   * 「課題管理」カードの接続先（一覧＋追加＋削除・#127）。
+   *
+   * **一覧は応答で描き直す。** 追加・削除の応答には必ずいまの一覧が添えられている
+   * （成功も、重複で断られたときも、世代がずれていたときも）。夜間スナップショットの
+   * 到着を待つと、押した直後は何も変わらないので「効いていない」と読まれ、もう一度
+   * 押して「すでに登録されています」になる——他の追加フォームで実際に起きたこと。
+   *
+   * **結果の表示は一覧の外に置く。** 行の中に置くと、一覧を描き直した瞬間に
+   * 押した結果が消える（Gold の先出しバッジで踏んだのと同じ罠）。
+   */
+  function backlogTargetsSection(s) {
+    var targets = backlogTargetsNow(s);
+    // 書き込み先（正本の置き場）が決まらない案件では操作を出さない。判定は呼び出し側と
+    // 同じ s.url（同期ミラーが1つでもあるか）で、フォームを出さない条件と揃える
+    var editable = !!(s && s.url);
+
+    var listEl = el("div", { class: "ds-target-list" });
+    var current = el("div", { class: "ds-current" }, [
+      el("span", { class: "ds-current-label", text: "現在の接続先" }),
+      listEl,
+    ]);
+    // 追加・削除の結果はここに1つだけ出す（行を描き直しても消えない位置）
+    var msg = el("div", { class: "ds-toggle-msg" });
+
+    function removeRow(t) {
+      var row = el("div", { class: "ds-target-row" }, [
+        el("span", { class: "ds-current-value", text: t.domain + " / " + t.projectKey }),
+      ]);
+      if (!editable) return row;
+      var btn = el("button", { class: "ds-toggle-btn ds-danger", type: "button", text: "この接続先を外す" });
+      btn.addEventListener("click", function () {
+        // **最後の1件かどうかは「押した時点」で数える。** 描画時の件数で決めると、
+        // 2件を1件ずつ外したとき最後の1件で確認が出ない（Figma で踏んだのと同じ）
+        var last = targets.length === 1;
+        if (!confirmDanger(removeTargetConfirmLines(t, last))) { msg.textContent = ""; return; }
+        var payload = {
+          op: "backlog-target-remove",
+          // **projectKey では送らない。** body の projectKey は案件キー（cortex の
+          // 案件識別子）に予約されていて、トークンから解決した案件と食い違えば 403 で断られる
+          backlogProjectKey: t.projectKey,
+          baseGeneration: targetsGenerationOf(targets),
+        };
+        // **常には送らない。** 最後の1件の重さを利用者に確かめてもらった証がこのフラグで、
+        // 無条件に付けると「一覧から1行消すだけ」の見た目で同期が止まる
+        if (last) payload.confirmLast = true;
+        sendConfig(payload, msg, btn, null, targetsHooks());
+      });
+      row.appendChild(btn);
+      return row;
+    }
+
+    function paint() {
+      while (listEl.firstChild) listEl.removeChild(listEl.firstChild);
+      if (!targets.length) {
+        // **行ごと消さない。** 表示が無いと「まだ何も無い」のか「出し忘れ」なのか区別できない
+        listEl.appendChild(el("span", { class: "ds-current-value is-unset", text: "未設定" }));
+        return;
+      }
+      for (var i = 0; i < targets.length; i++) listEl.appendChild(removeRow(targets[i]));
+    }
+
+    /**
+     * 応答に添えられた一覧を取り込む。**画面が見ていた集合と違っていたら true**
+     * （＝他の人が先に変えていた。409 の文面をこちらの言葉に差し替える判断に使う）。
+     */
+    function adoptTargets(list) {
+      var before = targetsGenerationOf(targets);
+      var next = [];
+      for (var i = 0; i < list.length; i++) {
+        var x = list[i];
+        if (x && x.domain && x.projectKey) next.push({ domain: String(x.domain), projectKey: String(x.projectKey) });
+      }
+      targets = next;
+      paint();
+      return targetsGenerationOf(targets) !== before;
+    }
+
+    function targetsHooks() {
+      return { onTargets: adoptTargets, staleNote: TARGETS_RACE_NOTE };
+    }
+
+    paint();
+    var box = el("div", {}, [current]);
+    if (editable) {
+      box.appendChild(addForm({
+        // **「追加」に戻す（#127）。** 1件しか持てなかった頃は「保存」（置き換え）だったが、
+        // 接続先は本当に足せるようになった。押した結果は一覧の行として出る
+        title: "＋ 接続先を追加",
+        placeholder: "https://<スペース>.backlog.jp/projects/<プロジェクトキー>",
+        requiredNote: "BacklogのURLを入力してください（プロジェクトキーが入った形。数字だけのURLは受け付けません）。",
+        hint: "Backlog でこの案件のプロジェクトを開き、ブラウザのURLをそのまま貼ってください（課題の画面のURLでも構いません）。APIキーは下の「Backlog のAPIキー」から別に登録します。",
+        confirmLines: function (v) {
+          // **解析した結果を見せる。** 貼ったURLをそのまま出すと、どのプロジェクトを
+          // 指すことになるかが確定前に分からない。実際 /projects/PJ-CORTEX は
+          // ハイフンで切れて PJ と解釈される（黙って別プロジェクトを指す）
+          var t = backlogTargetOf(v);
+          return [
+            "この案件が同期する Backlog プロジェクトを追加します。",
+            "",
+            t ? t.domain + " / " + t.projectKey : v,
+            "",
+            "このプロジェクトの課題・ドキュメント・Wiki が、次の同期から取り込まれます。",
+            "貼ったURLがこの案件のものか、もう一度確認してください。",
+            "",
+            "※ 増えるのは同期先だけです。ビューアからの起票先は変わりません。リアルタイム同期を使う場合は、追加したプロジェクトの Backlog 側にも案件専用URLの登録が要ります。",
+            "",
+            "追加しますか？",
+          ];
+        },
+        build: function (url) {
+          return { op: "backlog-target-add", url: url, baseGeneration: targetsGenerationOf(targets) };
+        },
+        // 応答の一覧で本物の行が増えるので、フォーム側の控え（「追加済み・反映中」）は出さない
+        hooks: targetsHooks,
+      }));
+    }
+    box.appendChild(msg);
+    return box;
+  }
+
+  /** 接続先を外すときの確認。**最後の1件は、止まるものを名指しで伝える** */
+  function removeTargetConfirmLines(t, last) {
+    var lines = [
+      "「" + t.domain + " / " + t.projectKey + "」を同期の対象から外します。",
+      "",
+    ];
+    if (last) {
+      // 一覧から1行消す見た目と、実際に起きることの重さが釣り合わないので必ず言う
+      lines.push("これが最後の接続先です。この接続先を外すと課題の同期が止まります。よろしいですか？");
+    } else {
+      lines.push(
+        "次回の同期から、このプロジェクトの課題・ドキュメント・Wiki は取り込まれなくなります。",
+        "取り込み済みの内容はリポジトリに残ります（不要なら手で削除してください）。",
+        "",
+        "外しますか？",
+      );
+    }
+    return lines;
   }
 
   /**
@@ -2969,46 +3144,21 @@ ${capabilityState.toString()}
       // 画面のどこにも出ていなかった。保存直後に出る楽観的なチップはリロードで消えるので、
       // **利用者は「何もセットされていない」と読み、同じURLを貼り直した**。
       // 設定を変える口の隣には、いまの値が必ず要る。
-      box.appendChild(currentBacklogTargetLine(s));
-      // **設定ファイルが無い案件にはフォームを出さない。** 書き込み先が無いので
-      // 貼っても 404 で終わる。三菱電機は tools が backlog なのに
+      // **設定ファイルが無い案件には追加・削除を出さない。** 書き込み先が無いので
+      // 押しても 404 で終わる。三菱電機は tools が backlog なのに
       // backlog-settings.json が1つも無い（固定IP経由の同期で、UIからの投入は
       // 鍵と同じく対応していない）。会議・共有資料は unset / broken を文言で
-      // 伝えてフォームを出さない形なので、課題管理だけ不揃いだった
+      // 伝えてフォームを出さない形なので、課題管理だけ不揃いだった。
+      // 一覧そのもの（現在の接続先）は**そういう案件にも出す**——読めているものを隠さない
       //
       // **接続先の正本は issues-config.json（#111）**だが、その置き場は
       // 同期ミラー（backlog-settings.json）の位置から決めている。ミラーが1つも
       // 無い案件では正本をどこへ置けばよいかも決められないので、条件は変わらない
+      box.appendChild(backlogTargetsSection(s));
       if (!s.url) {
         box.appendChild(el("p", { class: "sec-how", text: "この案件はまだ一度も Backlog と同期していないため、この画面からは接続先を設定できません。先に初回の同期を実行するか、cortex の担当者にご連絡ください。" }));
         used = true;
-      } else box.appendChild(addForm({
-        title: "接続先を変更",
-        // **「追加」ではなく「保存」。** 接続先は1件しか持てない置き換えで、
-        // 「追加」だと2つ目を足せると読める（実機テストでの指摘・複数対応は #127）
-        submitLabel: "保存",
-        placeholder: "https://<スペース>.backlog.jp/projects/<プロジェクトキー>",
-        requiredNote: "BacklogのURLを入力してください（プロジェクトキーが入った形。数字だけのURLは受け付けません）。",
-        hint: "Backlog でこの案件のプロジェクトを開き、ブラウザのURLをそのまま貼ってください（課題の画面のURLでも構いません）。APIキーは下の「Backlog のAPIキー」から別に登録します。",
-        confirmLines: function (v) {
-          // **解析した結果を見せる。** 貼ったURLをそのまま出すと、どのプロジェクトを
-          // 指すことになるかが確定前に分からない。実際 /projects/PJ-CORTEX は
-          // ハイフンで切れて PJ と解釈される（黙って別プロジェクトを指す）
-          var t = backlogTargetOf(v);
-          return [
-            "この案件が同期する Backlog プロジェクトを設定します。",
-            "",
-            "別のプロジェクトを指すと、次の同期で別案件の課題が入ります。",
-            "",
-            t ? t.domain + " / " + t.projectKey : v,
-            "",
-            "※ 変わるのは同期先だけです。ビューアからの起票先と Webhook の配送先は別途の対応が要ります。",
-            "",
-            "設定しますか？",
-          ];
-        },
-        build: function (url) { return { op: "set-backlog-target", url: url }; },
-      }));
+      }
       // **接続先の下に置く。** リアルタイム同期は「どのプロジェクトを見るか」が
       // 決まった後の話で、順序が逆だと、接続先を設定していない案件が先に
       // Webhook を貼りに行く（貼っても届かない）
@@ -3391,8 +3541,15 @@ ${capabilityState.toString()}
    * 押した内容を「（反映中）」として先に出す。出してよいのは (1) 保存の成否がサーバで
    * 確定していて推測が入らない (2) **確定した表示と見た目で区別できる**、の両方を満たす
    * ときだけ。区別しないと、リロードで消えたときに「戻された」と読まれる。
+   *
+   * **もう1つの例外が接続先の一覧（#127）。** 追加・削除の応答にはいまの一覧が必ず
+   * 添えられていて（成功も、重複で断られたときも、世代がずれていたときも）、これは
+   * スナップショットではなく**サーバが書いた直後の確定値**なので、その場で描き直してよい。
+   * @param hooks { onTargets(list) → 画面が見ていた集合と違ったか, staleNote }
+   *   onTargets … 応答に添えられた一覧を画面へ取り込む。**成否を問わず先に呼ぶ**——
+   *     断られたときこそ、画面を正しい状態へ追いつかせないと堂々巡りになる
    */
-  async function sendConfig(payload, msg, btn, onDone) {
+  async function sendConfig(payload, msg, btn, onDone, hooks) {
     var intake = SITE.intake;
     if (!intake) return;
     var pfx = intake.url + (intake.url.indexOf("?") === -1 ? "?" : "&");
@@ -3406,10 +3563,14 @@ ${capabilityState.toString()}
         body: JSON.stringify(payload),
       });
       var j = await res.json().catch(function () { return {}; });
+      // 一覧が添えられていれば、断られていても先に画面を追いつかせる（#127）
+      var raced = hooks && hooks.onTargets && Array.isArray(j.targets) ? hooks.onTargets(j.targets) : false;
       if (!res.ok) {
-        // サーバの文言をそのまま出す（貼り方の案内・取り違えの警告を含んでいる）
+        // サーバの文言をそのまま出す（貼り方の案内・取り違えの警告を含んでいる）。
+        // **例外は「他の人が先に変えていた」だけ**——サーバは再読み込みを案内するが、
+        // 画面は既に追いついているので、済んでいることをやらせる案内になる（#127）
         msg.className = "ds-toggle-msg is-error";
-        msg.textContent = "✗ " + (j.message || "保存できませんでした。");
+        msg.textContent = "✗ " + ((res.status === 409 && raced && hooks.staleNote) || j.message || "保存できませんでした。");
         btn.disabled = false;
         return;
       }
@@ -3520,12 +3681,14 @@ ${capabilityState.toString()}
   /**
    * 「＋…を追加」のフォーム。URLをそのまま貼れるようにする（IDだけを要求すると転記ミスが増える）。
    *
-   * **ボタンの語は呼び出し側が決める（既定は「追加」）。** 課題管理の「接続先を変更」だけは
-   * 1件しか持てない設定の**置き換え**なのに、ボタンが「追加」だと**複数登録できる**と読める
-   * （実機テストでの指摘。複数接続先の対応は #127 の将来要件で、いまは持てない）。
-   * 押した結果の表示（＋/追加済み）も同じ理由で語を揃える。
+   * **ボタンの語は呼び出し側が決める（既定は「追加」）。** 置き換えの設定に「追加」と書くと
+   * 2つ目を足せると読めるため、そういう口だけ submitLabel で語を変える。押した結果の
+   * 表示（＋/追加済み）も同じ理由で語を揃える。
    *
-   * @param opts { title, placeholder, hint, needsName, confirmLines, build(values), submitLabel }
+   * @param opts { title, placeholder, hint, needsName, confirmLines, build(values), submitLabel, hooks }
+   *   hooks … sendConfig へ渡す追い付き処理を返す関数（接続先の一覧・#127）。
+   *     **これがある口では、その場の控え（「追加済み・反映中」）を出さない**——
+   *     応答で本物の一覧が増えるので、控えを併記すると同じものが2回出る
    */
   function addForm(opts) {
     var main = el("input", { class: "sec-input", type: "text", placeholder: opts.placeholder, autocomplete: "off" });
@@ -3573,6 +3736,8 @@ ${capabilityState.toString()}
         // 追加できたら入力欄を空にする（同じものを二度押ししないように）
         main.value = "";
         if (name) name.value = "";
+        // 応答で本物の一覧が描き直される口では、控えを出さない（同じものが2回並ぶ）
+        if (opts.hooks) return;
         // **追加したものをその場に出す。** 上の一覧は fleet-status.json（夜間スナップショット）
         // から描かれているので、追加してもその場では変わらない。何も変わらないと
         // 「追加できていない」と読まれ、もう一度押して「既に登録されています」になる
@@ -3584,7 +3749,7 @@ ${capabilityState.toString()}
             el("span", { class: "ds-pending-note", text: submitLabel + "済み・反映中" }),
           ]),
         );
-      });
+      }, opts.hooks ? opts.hooks() : null);
     });
 
     // このセッションで追加したものを並べる場所（リロードすると消える＝正式な一覧は上に出る）

package/package.json

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