@takagaki/cortex-decisions-viewer 0.4.53 → 0.4.55

package/dist/build.js

@@ -38,6 +38,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.readFleetSources = readFleetSources;
 exports.build = build;
+exports.createRefResolver = createRefResolver;
 const node_fs_1 = require("node:fs");
 const path = __importStar(require("node:path"));
 const gray_matter_1 = __importDefault(require("gray-matter"));
@@ -172,6 +173,13 @@ async function readFleetSources(repoRoot) {
             driveFolderIds: Array.isArray(s.driveFolderIds)
                 ? s.driveFolderIds.map((x) => String(x)).filter(Boolean)
                 : undefined,
+            // フォルダの表示名（ID→名前）。**ここに列挙し忘れると、型エラーも例外も出さずに
+            // 静かに消える**（origin / figmaFiles / driveFolderIds で3回踏んでいる）。
+            driveFolderNames: s.driveFolderNames != null && typeof s.driveFolderNames === "object" && !Array.isArray(s.driveFolderNames)
+                ? Object.fromEntries(Object.entries(s.driveFolderNames)
+                    .map(([k, v]) => [k, String(v ?? "").trim()])
+                    .filter(([, v]) => v))
+                : undefined,
             // 登録済みFigmaファイルの一覧。**1件ずつ外せるようにするために要る**
             // （先頭1件のURLしか無かった頃は、6ファイル持つ案件でどれも選べなかった）。
             figmaFiles: Array.isArray(s.figmaFiles)
@@ -344,10 +352,57 @@ async function walkMd(dirAbs) {
  * - 課題キー（例: PJ_CORTEX-13）→ 同期mdに記載された Backlog Issue Link（実Backlog URL）
  * - minute:{定例名}:{YYYYMMDD} → 議事録ファイルのGitHub blobリンク
  */
+/**
+ * マーカーファイルの在り処からディレクトリ名を求める（`cortex-engine/scripts/fleet-status.mjs`
+ * の `findDirByMarker` と同じ考え方）。
+ *
+ * **ディレクトリ名をハードコードしない。** `customize-tooling` で改名している案件が実在する
+ * （ある案件は `課題管理/`→`Backlog/`・`会議/`→`MTG/`・`デザイン/`→`Figma/`）。
+ * 決め打ちだと、その案件では課題キーもFigmaも**永久に解決できない**。
+ */
+async function findDirByMarker(repoRoot, marker, fallback) {
+    try {
+        for (const d of await node_fs_1.promises.readdir(repoRoot, { withFileTypes: true })) {
+            if (!d.isDirectory() || d.name === "node_modules" || d.name.startsWith("."))
+                continue;
+            try {
+                await node_fs_1.promises.stat(path.join(repoRoot, d.name, marker));
+                return d.name;
+            }
+            catch {
+                // 直下に無ければ1階層下も見る（backlog-settings.json は issues/ の下に置かれる）
+            }
+            try {
+                for (const sub of await node_fs_1.promises.readdir(path.join(repoRoot, d.name), { withFileTypes: true })) {
+                    if (!sub.isDirectory())
+                        continue;
+                    try {
+                        await node_fs_1.promises.stat(path.join(repoRoot, d.name, sub.name, marker));
+                        return d.name;
+                    }
+                    catch {
+                        // 次へ
+                    }
+                }
+            }
+            catch {
+                // 読めないディレクトリは飛ばす
+            }
+        }
+    }
+    catch {
+        // repoRoot が読めなければ既定名で進む
+    }
+    return fallback;
+}
 async function buildTargetUrlMap(repoRoot, repoBaseUrl, branch) {
     const map = new Map();
-    // 課題キー → Backlog URL（課題管理/issues/ の同期mdの基本情報から抽出）
-    for (const file of await walkMd(path.join(repoRoot, "課題管理", "issues"))) {
+    const issuesDir = await findDirByMarker(repoRoot, "backlog-settings.json", "課題管理");
+    const designDir = await findDirByMarker(repoRoot, "figma.json", "デザイン");
+    const meetingDir = await findDirByMarker(repoRoot, "ingest-config.json", "会議");
+    const materialsDir = await findDirByMarker(repoRoot, "materials-config.json", "共有資料");
+    // 課題キー → Backlog URL（{課題管理}/issues/ の同期mdの基本情報から抽出）
+    for (const file of await walkMd(path.join(repoRoot, issuesDir, "issues"))) {
         try {
             const head = (await node_fs_1.promises.readFile(file, "utf8")).slice(0, 4000);
             const keyMatch = head.match(/^- 課題キー: (\S+)$/m);
@@ -360,7 +415,7 @@ async function buildTargetUrlMap(repoRoot, repoBaseUrl, branch) {
         }
     }
     // material:{slug} → 共有資料の正本（Drive等のsource URL）。無ければ変換済みmdのGitHubリンク
-    for (const file of await walkMd(path.join(repoRoot, "共有資料"))) {
+    for (const file of await walkMd(path.join(repoRoot, materialsDir))) {
         if (path.basename(file).toLowerCase() === "link.md")
             continue;
         try {
@@ -390,22 +445,32 @@ async function buildTargetUrlMap(repoRoot, repoBaseUrl, branch) {
             // 読めないファイルはスキップ
         }
     }
-    // design:{fileKey}:{nodeId} → Figmaディープリンク（インベントリのfrontmatterから）
-    for (const file of await walkMd(path.join(repoRoot, "デザイン", "inventory"))) {
+    // design:{fileKey}:{nodeId} → Figmaディープリンク
+    //
+    // **frontmatter からは取れない。** インベントリは frontmatter を持たない
+    // （frontmatter を持つのは Gold層だけ＝オントロジー規約）。参照IDもFigmaのURLも**本文**にある:
+    //   - 参照ID: `design:xxx:103:1836`
+    //   - [Figmaで開く](https://www.figma.com/design/...)
+    // 以前は `data.type === "design"` を見ていたので、この解決は**一度も効いていなかった**。
+    for (const file of await walkMd(path.join(repoRoot, designDir, "inventory"))) {
         try {
-            const { data } = (0, gray_matter_1.default)(await node_fs_1.promises.readFile(file, "utf8"));
-            if (data.type === "design" && data.id && data.source) {
-                map.set(String(data.id), String(data.source));
-            }
+            const body = await node_fs_1.promises.readFile(file, "utf8");
+            const id = body.match(/^- 参照ID: `(design:[^`]+)`\s*$/m);
+            const url = body.match(/^- \[Figmaで開く\]\((https?:[^)]+)\)\s*$/m);
+            if (id && url)
+                map.set(id[1], url[1]);
         }
         catch {
             // 読めないファイルはスキップ
         }
     }
     // minute:{定例名}:{YYYYMMDD} → 議事録ファイルのGitHubリンク
-    // パス規約: ミーティング/{フェーズ}/{定例名}/{YYYYMMDD}/*minutes*.md
+    // パス規約: {会議}/{フェーズ}/{定例名}/{YYYYMMDD}/*minutes*.md
+    //
+    // **以前は存在しない `ミーティング/` を見ていた**（艦隊は全案件 `会議/`。改名案件は `MTG/`）。
+    // そのため minute: の解決は一度も効いていなかった。
     if (repoBaseUrl) {
-        for (const file of await walkMd(path.join(repoRoot, "ミーティング"))) {
+        for (const file of await walkMd(path.join(repoRoot, meetingDir))) {
             const name = path.basename(file).toLowerCase();
             if (!name.includes("minutes"))
                 continue;
@@ -560,7 +625,7 @@ async function scanContent(repoRoot, goldDirName, repoBaseUrl, branch) {
  * 参照先がリポジトリ内のレコードでないIDは、層を推定したファントムノードとして表示する
  * （minute:/material:→Silver、課題キー・外部URL→Bronze、Goldの未解決ID→phantom）。
  */
-function buildGraph(decisions, home, sections, content, targetUrls, repoBaseUrl, branch) {
+function buildGraph(decisions, home, sections, content, targetUrls, repoBaseUrl, branch, refs) {
     const nodes = new Map();
     const edges = [];
     const addNode = (node) => {
@@ -620,7 +685,12 @@ function buildGraph(decisions, home, sections, content, targetUrls, repoBaseUrl,
         else if (target.includes("/") || target.toLowerCase().endsWith(".md")) {
             group = "silver"; // リポジトリ内ファイルへのパス参照 → GitHubのコピーへ飛べるようにする
             label = target.split("/").pop() ?? target;
-            const blobUrl = repoBaseUrl ? `${repoBaseUrl}/blob/${branch}/${encodePath(target)}` : undefined;
+            // **実在するものだけリンクにする。** 詳細画面（resolveReference）と同じ判定を使う。
+            // ここだけ無条件に blob URL を作っていたため、同じ参照が「詳細ではテキスト・
+            // グラフでは404リンク」に分かれていた（グラフのノードは window.open される）。
+            const blobUrl = repoBaseUrl && refs.exists(target)
+                ? `${repoBaseUrl}/blob/${branch}/${encodePath(target)}`
+                : undefined;
             return addNode({ id: target, label, group, url: blobUrl });
         }
         return addNode({ id: target, label, group });
@@ -941,15 +1011,23 @@ async function build(opts) {
     // （Decisionsディレクトリで実行すると `-- Cortex/` に一致せず常に空になる）。
     // 浅いclone・git無しの環境では空集合が返り、「最新」ラベルが出ないだけで他の表示は壊れない。
     LATEST_PATHS = (0, repo_1.latestGeneratedPaths)(process.cwd());
+    // Cortex構成なら、ホームと「records/ を持つディレクトリ」をすべて読み込む（旧構成では空）
+    // ディレクトリ名はハードコードしない: 案件が新しいレコード種別のディレクトリを足せば、そのままタブとして表示される
+    const knowledgeRoot = resolveKnowledgeRoot(dirAbs);
+    // **参照の解決器は decisions のパースより前に作る。** parseDecision が references を
+    // 解決するので、あとから作ると空の表・空の実在判定が渡り、**実在するパスまで
+    // 「リンクにしない」に倒れて全滅する**（以前は順序が逆だった）。
+    const repoRoot = knowledgeRoot ? path.dirname(knowledgeRoot) : path.dirname(dirAbs);
+    const targetUrls = knowledgeRoot
+        ? await buildTargetUrlMap(repoRoot, repo.baseUrl, repo.branch)
+        : new Map();
+    const refs = createRefResolver({ repoRoot, targetUrls, repoBaseUrl: repo.baseUrl, branch: repo.branch });
     const decisions = [];
     for (const fileName of mdFiles) {
         const raw = await node_fs_1.promises.readFile(path.join(dirAbs, fileName), "utf8");
-        const rec = await parseDecision(raw, fileName, repoRelDir, repo.baseUrl, repo.branch);
+        const rec = await parseDecision(raw, fileName, repoRelDir, repo.baseUrl, repo.branch, refs);
         decisions.push(rec);
     }
-    // Cortex構成なら、ホームと「records/ を持つディレクトリ」をすべて読み込む（旧構成では空）
-    // ディレクトリ名はハードコードしない: 案件が新しいレコード種別のディレクトリを足せば、そのままタブとして表示される
-    const knowledgeRoot = resolveKnowledgeRoot(dirAbs);
     let home = null;
     const sections = [];
     if (knowledgeRoot) {
@@ -982,12 +1060,8 @@ async function build(opts) {
     const categories = uniqueSorted(decisions.map((d) => d.category).filter(Boolean));
     // 期間フィルタ用の月（YYYY-MM）を日付から導出し、降順（新しい順）にする
     const months = uniqueSorted(decisions.map((d) => d.date.slice(0, 7)).filter((m) => /^\d{4}-\d{2}$/.test(m))).reverse();
-    const repoRoot = knowledgeRoot ? path.dirname(knowledgeRoot) : path.dirname(dirAbs);
-    const targetUrls = knowledgeRoot
-        ? await buildTargetUrlMap(repoRoot, repo.baseUrl, repo.branch)
-        : new Map();
     const contentEntries = await scanContent(repoRoot, knowledgeRoot ? path.basename(knowledgeRoot) : null, repo.baseUrl, repo.branch);
-    const graph = buildGraph(decisions, home, sections, contentEntries, targetUrls, repo.baseUrl, repo.branch);
+    const graph = buildGraph(decisions, home, sections, contentEntries, targetUrls, repo.baseUrl, repo.branch, refs);
     computeLayout(graph); // 各ノードに x/y を書き込む（クライアントはこれを初期位置に使いウォームアップ省略）
     // データソースと自動化（fleet-status.json 由来）。無ければ undefined でセクション非表示。
     const fleet = await readFleetSources(repoRoot);
@@ -1048,7 +1122,7 @@ async function build(opts) {
     const sectionCount = sections.reduce((n, s) => n + s.records.length, 0);
     return { count: decisions.length + (home ? 1 : 0) + sectionCount, outDir: outAbs };
 }
-async function parseDecision(raw, fileName, repoRelDir, repoBaseUrl, branch) {
+async function parseDecision(raw, fileName, repoRelDir, repoBaseUrl, branch, refs) {
     const { data, content } = (0, gray_matter_1.default)(raw);
     const id = String(data.id ?? fileName.replace(/\.md$/i, ""));
     const title = String(data.title ?? stripIdPrefix(fileName));
@@ -1058,7 +1132,7 @@ async function parseDecision(raw, fileName, repoRelDir, repoBaseUrl, branch) {
     const summary = data.description != null ? String(data.description) : data.summary != null ? String(data.summary) : ""; // description優先（旧summaryは移行期互換）
     const deciders = toStringArray(data.deciders);
     const referencesRaw = toStringArray(data.references);
-    const references = referencesRaw.map((r) => resolveReference(r, repoBaseUrl, branch));
+    const references = referencesRaw.map((r) => refs.resolve(r));
     const relations = toRelations(data.relations);
     // 本文先頭の H1 はタイトルと重複するため除去（詳細画面ではタイトルを別途表示する）
     const bodyHtml = externalLinksToNewTab((await marked_1.marked.parse(content, { async: true })).replace(/^\s*<h1[^>]*>[\s\S]*?<\/h1>\s*/, ""));
@@ -1088,25 +1162,50 @@ async function parseDecision(raw, fileName, repoRelDir, repoBaseUrl, branch) {
         fmErrors: validateFm(data),
     };
 }
-function resolveReference(ref, repoBaseUrl, branch) {
-    const trimmed = ref.trim();
-    // markdownリンク [label](url)
-    const mdLink = trimmed.match(/^\[([^\]]+)\]\(([^)]+)\)$/);
-    if (mdLink) {
-        return { label: mdLink[1], url: mdLink[2] };
-    }
-    // 既にURL
-    if (/^https?:\/\//.test(trimmed)) {
-        return { label: trimmed, url: trimmed };
-    }
-    // リポジトリルート相対のローカルパスとみなす → GitHubのblobリンク
-    if (repoBaseUrl) {
-        return {
-            label: trimmed,
-            url: `${repoBaseUrl}/blob/${branch}/${encodePath(trimmed)}`,
-        };
-    }
-    return { label: trimmed, url: null };
+function createRefResolver(opts) {
+    const { repoRoot, targetUrls, repoBaseUrl, branch } = opts;
+    // **走査結果を流用しない。** scanContent は `.md` しか歩かず SCAN_SKIP_DIRS を除外するので、
+    // 実在する `.txt`（Geminiメモ）や `tmp/` 配下のファイルを「実在しない」と誤判定して
+    // **いま動いているリンクを消す**。参照は1リポジトリあたり数百件なので stat で十分。
+    const seen = new Map();
+    const exists = (relPath) => {
+        const hit = seen.get(relPath);
+        if (hit !== undefined)
+            return hit;
+        let ok = false;
+        try {
+            // パス要素に混ざる制御文字・絶対パス・親ディレクトリ参照は見に行かない
+            if (relPath && !path.isAbsolute(relPath) && !relPath.split("/").includes("..")) {
+                (0, node_fs_1.statSync)(path.join(repoRoot, relPath));
+                ok = true;
+            }
+        }
+        catch {
+            ok = false;
+        }
+        seen.set(relPath, ok);
+        return ok;
+    };
+    const resolve = (ref) => {
+        const trimmed = String(ref ?? "").trim();
+        if (!trimmed)
+            return { label: trimmed, url: null };
+        const mdLink = trimmed.match(/^\[([^\]]+)\]\(([^)]+)\)$/);
+        if (mdLink)
+            return { label: mdLink[1], url: mdLink[2] };
+        if (/^https?:\/\//.test(trimmed))
+            return { label: trimmed, url: trimmed };
+        // 正本のURL（課題キー・material:・design:・minute:）。**ここが「ツールへ飛ぶ」の本体**
+        const known = targetUrls.get(trimmed);
+        if (known)
+            return { label: trimmed, url: known };
+        // 実在するファイルだけ blob URL にする
+        if (repoBaseUrl && exists(trimmed)) {
+            return { label: trimmed, url: `${repoBaseUrl}/blob/${branch}/${encodePath(trimmed)}` };
+        }
+        return { label: trimmed, url: null };
+    };
+    return { resolve, exists };
 }
 /** dirをリポジトリルート相対（posix）に。cwd配下でなければ basename を返す。 */
 function toRepoRelative(dir) {

package/dist/render.js

@@ -781,6 +781,12 @@ a.ds-name:hover { text-decoration: underline; }
 .ds-addform .sec-input { margin-bottom: 6px; }
 .sec-input-inline { width: auto; max-width: 220px; margin-bottom: 0; }
 .sec-how { color: var(--muted); font-size: 12px; line-height: 1.7; margin: 0 0 10px; }
+.sec-how p { margin: 0; }
+/* 取り込みの条件は箇条書きで出す（ANDであることを読み飛ばせないように） */
+.sec-how-list { margin: 4px 0; padding-left: 1.3em; }
+.sec-how-list li { margin: 2px 0; }
+/* 0件を黙って見せない（追加フォームだけだと「未登録」か「読めていない」か区別できない） */
+.ds-empty { color: var(--muted); font-size: 12.5px; margin: 4px 0 10px; }
 .sec-input { width: 100%; box-sizing: border-box; font-size: 13px; padding: 7px 10px; margin-bottom: 8px; border: 1px solid var(--border); border-radius: 6px; background: #fff; color: var(--text); }
 .sec-input:focus { outline: none; border-color: var(--accent); }
 .ops-card {
@@ -1694,12 +1700,18 @@ const CLIENT_JS = `
         // （applyAddDriveFolder が enabled を立てる）ので「始める」を押す機会が無い。
         // 止めたいならフォルダを外す——登録そのものが意思表示、という会議と同じ考え方。
         var folders = s.driveFolderIds || [];
+        // 名前は任意。**無ければIDを出す**（既存の案件は名前を持っていない）
+        var fnames = s.driveFolderNames || {};
         if (folders.length) {
           var dlist = el("div", { class: "ds-subitems" });
           folders.forEach(function (id) {
+            var fname = fnames[id];
             var row = el("div", { class: "ds-subitem" }, [
-              dsNameEl(id, "https://drive.google.com/drive/folders/" + id),
+              dsNameEl(fname || id, "https://drive.google.com/drive/folders/" + id),
             ]);
+            // **名前を出したときはIDも併記する。** 同名のフォルダを見分けられなくなるのと、
+            // 「このIDを外す」という確認文と画面の表示が食い違うのを防ぐ
+            if (fname) row.appendChild(el("span", { class: "ds-sync", text: id }));
             row.appendChild(removeButton({
               label: "このフォルダを外す",
               confirmLines: function () {
@@ -1769,16 +1781,20 @@ const CLIENT_JS = `
           flist.appendChild(row);
         });
         box.appendChild(flist);
+      } else {
+        // **0件を黙って見せない。** 追加フォームだけだと「まだ登録していない」のか
+        // 「読めていない」のか区別できない（Driveは driveState で理由まで出している）
+        box.appendChild(el("p", { class: "ds-empty", text: "同期対象のFigmaファイルはまだ登録されていません。" }));
       }
       box.appendChild(addForm({
         title: "＋ Figmaファイルを追加",
         placeholder: "https://www.figma.com/design/…",
         namePlaceholder: "メモ（任意）",
         requiredNote: "FigmaのURLを入力してください。",
-        hint: "このファイルの画面インベントリ（画面名・サムネイル）が毎晩同期され、顧客も見るリポジトリに入ります。",
+        hint: "このファイルの画面インベントリ（画面名・Figmaへのリンク）が毎晩同期されます。画像は取り込まれません。",
         confirmLines: function (v) {
           return [
-            "このFigmaファイルの画面名とサムネイルを、顧客も見るリポジトリに取り込みます。",
+            "このFigmaファイルの画面名とFigmaへのリンクを、顧客も見るリポジトリに取り込みます（画像は取り込みません）。",
             "",
             "貼ったURLがこの案件のものか、もう一度確認してください。",
             "",
@@ -1805,13 +1821,17 @@ const CLIENT_JS = `
    * ②の材料を画面が出さないと、招待したのに届かない理由が分からない。
    */
   function meetingHowTo() {
-    // 会議名の条件は上の行（ds-keys）が出す。ここは「誰を招待するか」だけを言う
-    return el("p", {
-      class: "sec-how",
-      text:
-        "会議に cortex-notetaker Bot を招待すると、その会議の文字起こしが取り込まれ、議事録が作られます。" +
-        "招待しなければ取り込まれません（招待が取り込みの合図です）。上の会議名を含む会議が対象です。",
-    });
+    // **2つの条件がANDであることを、読み飛ばせない形で出す。** 散文で並べていたときに
+    // 「会議名は合っているのにBotを招待しておらず取り込まれなかった」事故が実際に起きた。
+    // 会議名の条件は上の行（ds-keys）が出すので、ここでは「上の語」と参照するに留める。
+    var wrap = el("div", { class: "sec-how" });
+    wrap.appendChild(el("p", { text: "次の2つが揃った会議だけが取り込まれます。" }));
+    var ul = el("ul", { class: "sec-how-list" });
+    ul.appendChild(el("li", { text: "cortex-notetaker Bot が招待されている（定例はシリーズに1回でOK）" }));
+    ul.appendChild(el("li", { text: "会議名に上の語のどれかが含まれている" }));
+    wrap.appendChild(ul);
+    wrap.appendChild(el("p", { text: "どちらか一方だけでは取り込まれません。" }));
+    return wrap;
   }
 
   /**

package/package.json

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