@dereekb/dbx-components-mcp 13.12.8 → 13.12.9

package/dbx-components-mcp.js

@@ -5,7 +5,7 @@ const require = __createRequire(import.meta.url);
 // packages/dbx-components-mcp/package.json
 var package_default = {
   name: "@dereekb/dbx-components-mcp",
-  version: "13.12.8",
+  version: "13.12.9",
   type: "module",
   bin: {
     "dbx-components-mcp": "dbx-components-mcp.js"
@@ -16,8 +16,8 @@ var package_default = {
     "ts-morph": "^21.0.0"
   },
   peerDependencies: {
-    "@dereekb/dbx-cli": "13.12.8",
-    "@dereekb/util": "13.12.8"
+    "@dereekb/dbx-cli": "13.12.9",
+    "@dereekb/util": "13.12.9"
   }
 };
 
@@ -120,18 +120,18 @@ var SPEC_SUFFIX = ".spec.ts";
 function classifySpecFile(config) {
   const { filename, parentFolderName } = config;
   let result;
-  if (!filename.endsWith(SPEC_SUFFIX)) {
-    result = { filename, group: "", kind: "non-spec", subgroups: [], isCanonical: false };
-  } else {
+  if (filename.endsWith(SPEC_SUFFIX)) {
     const stem = filename.slice(0, -SPEC_SUFFIX.length);
     const parts = stem.split(".");
     const group = parts[0] ?? "";
-    if (group !== parentFolderName) {
-      result = { filename, group, kind: "non-group", subgroups: [], isCanonical: false };
-    } else {
+    if (group === parentFolderName) {
       const rest = parts.slice(1);
       result = classifyRemainingSegments({ filename, group, rest });
+    } else {
+      result = { filename, group, kind: "non-group", subgroups: [], isCanonical: false };
     }
+  } else {
+    result = { filename, group: "", kind: "non-spec", subgroups: [], isCanonical: false };
   }
   return result;
 }
@@ -7425,19 +7425,24 @@ function collectLocalRoleConsts(project) {
   const arrays = /* @__PURE__ */ new Map();
   for (const sourceFile of project.getSourceFiles()) {
     for (const stmt of sourceFile.getVariableStatements()) {
-      if (!stmt.isExported()) continue;
-      for (const decl of stmt.getDeclarations()) {
-        const initializer = decl.getInitializer();
-        const name = decl.getName();
-        if (initializer === void 0) continue;
-        if (Node11.isStringLiteral(initializer) || Node11.isNoSubstitutionTemplateLiteral(initializer)) {
-          if (!scalars.has(name)) scalars.set(name, initializer.getLiteralText());
-        } else if (Node11.isArrayLiteralExpression(initializer) && !arrays.has(name)) arrays.set(name, initializer);
-      }
+      collectRoleConstsFromVariableStatement(stmt, scalars, arrays);
     }
   }
   return { scalars, arrays };
 }
+function collectRoleConstsFromVariableStatement(stmt, scalars, arrays) {
+  if (!stmt.isExported()) return;
+  for (const decl of stmt.getDeclarations()) {
+    const initializer = decl.getInitializer();
+    if (initializer === void 0) continue;
+    const name = decl.getName();
+    if (Node11.isStringLiteral(initializer) || Node11.isNoSubstitutionTemplateLiteral(initializer)) {
+      if (!scalars.has(name)) scalars.set(name, initializer.getLiteralText());
+    } else if (Node11.isArrayLiteralExpression(initializer) && !arrays.has(name)) {
+      arrays.set(name, initializer);
+    }
+  }
+}
 function mergeKnownRoles(builtins, localScalars) {
   const out = /* @__PURE__ */ new Map();
   for (const [name, role] of localScalars) out.set(name, role);
@@ -31443,38 +31448,57 @@ async function run5(rawArgs) {
     return toolError("Provide either `model` (PascalCase model name) or `identity` (the `<camelName>Identity` const string).");
   }
   const cwd = process.cwd();
+  const ensureError = runEnsurePathInsideCwd(parsed.apiDir, cwd);
   let result;
-  let ensureError;
-  try {
-    ensurePathInsideCwd(parsed.apiDir, cwd);
-  } catch (err) {
-    ensureError = err instanceof Error ? err.message : String(err);
-  }
-  if (ensureError === void 0) {
+  if (ensureError !== void 0) {
+    result = toolError(ensureError);
+  } else {
     const apiAbs = resolve39(cwd, parsed.apiDir);
-    let extraction;
-    let inspectError;
-    try {
-      extraction = await inspectAppFixtures(apiAbs, parsed.apiDir);
-    } catch (err) {
-      inspectError = `Failed to read fixture file: ${err instanceof Error ? err.message : String(err)}`;
-    }
-    if (inspectError === void 0 && extraction !== void 0) {
-      const lookupModel = parsed.model ?? identityToModel4(parsed.identity);
-      const entry = extraction.entries.find((e) => e.model === lookupModel);
-      if (entry) {
-        const text = parsed.format === "json" ? formatLookupAsJson2(extraction, entry) : formatLookupAsMarkdown2(extraction, entry);
-        result = { content: [{ type: "text", text }] };
-      } else {
-        const known = extraction.entries.map((e) => e.model).join(", ") || "(none)";
-        const usedIdentity = parsed.identity && !parsed.model ? ` (resolved \`identity="${parsed.identity}"\` \u2192 \`${lookupModel}\`)` : "";
-        result = toolError(`Model \`${lookupModel}\` not found in \`${extraction.fixturePath}\`${usedIdentity}. Known: ${known}.`);
-      }
+    const inspection = await inspectFixturesSafe(apiAbs, parsed.apiDir);
+    if (inspection.kind === "error") {
+      result = toolError(inspection.message);
     } else {
-      result = toolError(inspectError ?? "Failed to inspect fixtures.");
+      result = formatFixtureLookupResult({
+        extraction: inspection.extraction,
+        model: parsed.model,
+        identity: parsed.identity,
+        format: parsed.format
+      });
     }
+  }
+  return result;
+}
+function runEnsurePathInsideCwd(apiDir, cwd) {
+  let error;
+  try {
+    ensurePathInsideCwd(apiDir, cwd);
+  } catch (err) {
+    error = err instanceof Error ? err.message : String(err);
+  }
+  return error;
+}
+async function inspectFixturesSafe(apiAbs, apiDir) {
+  let inspection;
+  try {
+    const extraction = await inspectAppFixtures(apiAbs, apiDir);
+    inspection = { kind: "ok", extraction };
+  } catch (err) {
+    inspection = { kind: "error", message: `Failed to read fixture file: ${err instanceof Error ? err.message : String(err)}` };
+  }
+  return inspection;
+}
+function formatFixtureLookupResult(input) {
+  const { extraction, model, identity, format } = input;
+  const lookupModel = model ?? identityToModel4(identity);
+  const entry = extraction.entries.find((e) => e.model === lookupModel);
+  let result;
+  if (entry) {
+    const text = format === "json" ? formatLookupAsJson2(extraction, entry) : formatLookupAsMarkdown2(extraction, entry);
+    result = { content: [{ type: "text", text }] };
   } else {
-    result = toolError(ensureError);
+    const known = extraction.entries.map((e) => e.model).join(", ") || "(none)";
+    const usedIdentity = identity && !model ? ` (resolved \`identity="${identity}"\` \u2192 \`${lookupModel}\`)` : "";
+    result = toolError(`Model \`${lookupModel}\` not found in \`${extraction.fixturePath}\`${usedIdentity}. Known: ${known}.`);
   }
   return result;
 }
@@ -32927,9 +32951,7 @@ async function run8(rawArgs) {
     ensureError = err instanceof Error ? err.message : String(err);
   }
   let result;
-  if (ensureError !== void 0) {
-    result = toolError(ensureError);
-  } else {
+  if (ensureError === void 0) {
     const apiAbs = resolve43(cwd, parsed.apiDir);
     let catalog;
     let discoveryError;
@@ -32938,13 +32960,15 @@ async function run8(rawArgs) {
     } catch (err) {
       discoveryError = `Failed to walk \`${parsed.apiDir}/src/app/function\`: ${err instanceof Error ? err.message : String(err)}`;
     }
-    if (catalog === void 0 || discoveryError !== void 0) {
-      result = toolError(discoveryError ?? "Failed to discover spec files.");
-    } else {
+    if (catalog !== void 0 && discoveryError === void 0) {
       const filtered = parsed.group === void 0 ? catalog : filterCatalogByGroup(catalog, parsed.group);
       const text = parsed.format === "json" ? formatListAppAsJson(filtered) : formatListAppAsMarkdown(filtered, { group: parsed.group });
       result = { content: [{ type: "text", text }] };
+    } else {
+      result = toolError(discoveryError ?? "Failed to discover spec files.");
     }
+  } else {
+    result = toolError(ensureError);
   }
   return result;
 }
@@ -33237,21 +33261,15 @@ function bucketSummary(group, bucket) {
 var DRIFT_RULES = ["`<group>.<sub>.crud.spec.ts` \u2192 rename to `<group>.crud.<sub>.spec.ts` (`crud` belongs directly after the group name).", "`<group>.<sub>.scenario.spec.ts` \u2192 rename to `<group>.scenario.<sub>.spec.ts` (`scenario` belongs directly after the group name).", "`<group>.<rest>.spec.ts` with no `crud` / `scenario` segment \u2192 add one (default to `scenario` when in doubt)."];
 function formatMarkdown(report) {
   const lines = [];
-  lines.push(`# Spec convention for \`${report.group}\``, "");
-  lines.push(`Tests live under \`${report.apiDir}/src/app/function/${report.group}/\`. Canonical name segments:`, "");
+  lines.push(`# Spec convention for \`${report.group}\``, "", `Tests live under \`${report.apiDir}/src/app/function/${report.group}/\`. Canonical name segments:`, "");
   for (const rec of report.recommendations) {
-    lines.push(`## ${rec.label} bucket`);
-    lines.push("", `- **Path:** \`${rec.canonicalPath}\``);
-    lines.push(`- **Filename:** \`${rec.canonicalFilename}\``);
-    lines.push(`- ${rec.summary}`);
-    lines.push("");
+    lines.push(`## ${rec.label} bucket`, "", `- **Path:** \`${rec.canonicalPath}\``, `- **Filename:** \`${rec.canonicalFilename}\``, `- ${rec.summary}`, "");
   }
   lines.push("## Drift rules", "");
   for (const rule of report.driftRules) {
     lines.push(`- ${rule}`);
   }
-  lines.push("");
-  lines.push("For an inventory + drift audit across an existing API app, run `dbx_model_test_list_app`. For naming-convention enforcement (errors / warnings), run `dbx_model_test_validate_app`.");
+  lines.push("", "For an inventory + drift audit across an existing API app, run `dbx_model_test_list_app`. For naming-convention enforcement (errors / warnings), run `dbx_model_test_validate_app`.");
   return lines.join("\n").trimEnd();
 }
 function formatJson(report) {
@@ -49770,22 +49788,9 @@ function parseArgs15(rawArgs, defaultCwd) {
   if (parsed instanceof type94.errors) {
     return `Invalid arguments: ${parsed.summary}`;
   }
-  let workspaceRoot = defaultCwd;
-  if (parsed.cwd !== void 0 && parsed.cwd.length > 0) {
-    workspaceRoot = resolve62(defaultCwd, parsed.cwd);
-  }
+  const workspaceRoot = parsed.cwd !== void 0 && parsed.cwd.length > 0 ? resolve62(defaultCwd, parsed.cwd) : defaultCwd;
+  const invalidDirError = validateExplicitDirs(parsed.explicitDirs, workspaceRoot);
   let result;
-  let invalidDirError;
-  if (parsed.explicitDirs !== void 0) {
-    for (const dir of parsed.explicitDirs) {
-      if (invalidDirError !== void 0) break;
-      try {
-        ensurePathInsideCwd(dir, workspaceRoot);
-      } catch (err) {
-        invalidDirError = err instanceof Error ? err.message : String(err);
-      }
-    }
-  }
   if (invalidDirError === void 0) {
     result = {
       op: parsed.op,
@@ -49798,6 +49803,20 @@ function parseArgs15(rawArgs, defaultCwd) {
   }
   return result;
 }
+function validateExplicitDirs(explicitDirs, workspaceRoot) {
+  let invalidDirError;
+  if (explicitDirs !== void 0) {
+    for (const dir of explicitDirs) {
+      if (invalidDirError !== void 0) break;
+      try {
+        ensurePathInsideCwd(dir, workspaceRoot);
+      } catch (err) {
+        invalidDirError = err instanceof Error ? err.message : String(err);
+      }
+    }
+  }
+  return invalidDirError;
+}
 async function run20(rawArgs) {
   const parsed = parseArgs15(rawArgs, process.cwd());
   if (typeof parsed === "string") return toolError(parsed);
@@ -49873,15 +49892,7 @@ async function discoverLogs(input) {
   } else {
     const primaryAbs = join24(input.basePath, input.project);
     const primaryExists = await pathIsDir(primaryAbs);
-    if (!primaryExists) {
-      if (input.includeSiblings) {
-        const siblings = await listChildDirs(input.basePath);
-        const logs = await collectFromProjects({ basePath: input.basePath, projects: siblings, since: input.since });
-        result = { logs, scannedProjects: siblings, missingBase: false, missingProject: true, fellBackToSiblings: true };
-      } else {
-        result = { logs: [], scannedProjects: [], missingBase: false, missingProject: true, fellBackToSiblings: false };
-      }
-    } else {
+    if (primaryExists) {
       const primaryLogs = await collectFromProjects({ basePath: input.basePath, projects: [input.project], since: input.since });
       if (primaryLogs.length === 0 && input.includeSiblings) {
         const siblings = (await listChildDirs(input.basePath)).filter((p) => p !== input.project);
@@ -49890,6 +49901,12 @@ async function discoverLogs(input) {
       } else {
         result = { logs: primaryLogs, scannedProjects: [input.project], missingBase: false, missingProject: false, fellBackToSiblings: false };
       }
+    } else if (input.includeSiblings) {
+      const siblings = await listChildDirs(input.basePath);
+      const logs = await collectFromProjects({ basePath: input.basePath, projects: siblings, since: input.since });
+      result = { logs, scannedProjects: siblings, missingBase: false, missingProject: true, fellBackToSiblings: true };
+    } else {
+      result = { logs: [], scannedProjects: [], missingBase: false, missingProject: true, fellBackToSiblings: false };
     }
   }
   return result;
@@ -49917,40 +49934,55 @@ async function listChildDirs(absPath) {
 async function collectFromProjects(input) {
   const collected = [];
   for (const project of input.projects) {
-    const projectDir = join24(input.basePath, project);
-    let entries = [];
-    try {
-      const dirents = await readdir15(projectDir, { withFileTypes: true });
-      entries = dirents.map((d) => ({ name: d.name, isFile: d.isFile() || d.isSymbolicLink() }));
-    } catch {
-      entries = [];
-    }
-    for (const entry of entries) {
-      if (entry.isFile && entry.name.endsWith(".md")) {
-        const absolutePath = join24(projectDir, entry.name);
-        let info;
-        try {
-          info = await stat22(absolutePath);
-        } catch {
-          info = void 0;
-        }
-        if (info !== void 0 && info.isFile() && info.mtimeMs >= input.since) {
-          const ref = {
-            absolutePath,
-            relativePath: join24(project, entry.name),
-            project,
-            fileName: basename9(entry.name),
-            mtimeMs: info.mtimeMs,
-            sizeBytes: info.size
-          };
-          collected.push(ref);
-        }
-      }
-    }
+    const refs = await collectProjectLogs({ basePath: input.basePath, project, since: input.since });
+    collected.push(...refs);
   }
   collected.sort((a, b) => b.mtimeMs - a.mtimeMs);
   return collected;
 }
+async function collectProjectLogs(input) {
+  const projectDir = join24(input.basePath, input.project);
+  const entries = await readMarkdownEntries(projectDir);
+  const refs = [];
+  for (const entry of entries) {
+    const ref = await buildLogRefIfRecent({ projectDir, project: input.project, fileName: entry, since: input.since });
+    if (ref !== void 0) {
+      refs.push(ref);
+    }
+  }
+  return refs;
+}
+async function readMarkdownEntries(projectDir) {
+  let entries = [];
+  try {
+    const dirents = await readdir15(projectDir, { withFileTypes: true });
+    entries = dirents.filter((d) => (d.isFile() || d.isSymbolicLink()) && d.name.endsWith(".md")).map((d) => d.name);
+  } catch {
+    entries = [];
+  }
+  return entries;
+}
+async function buildLogRefIfRecent(input) {
+  const absolutePath = join24(input.projectDir, input.fileName);
+  let info;
+  try {
+    info = await stat22(absolutePath);
+  } catch {
+    info = void 0;
+  }
+  let ref;
+  if (info !== void 0 && info.isFile() && info.mtimeMs >= input.since) {
+    ref = {
+      absolutePath,
+      relativePath: join24(input.project, input.fileName),
+      project: input.project,
+      fileName: basename9(input.fileName),
+      mtimeMs: info.mtimeMs,
+      sizeBytes: info.size
+    };
+  }
+  return ref;
+}
 
 // packages/dbx-components-mcp/src/tools/log-search/format-results.ts
 var MAX_LIST_SNIPPET_LENGTH = 240;
@@ -49991,13 +50023,7 @@ function formatKeywordResults(header, hits) {
     lines.push(`${hits.length} match(es) of ${header.totalCandidates} log(s) in window.`, "");
     for (const hit of hits) {
       const entry = hit.entry;
-      lines.push(`### ${entry.title}`);
-      lines.push(`_${formatMeta(entry)}_`);
-      lines.push("");
-      lines.push("```");
-      lines.push(sanitizeFence(hit.snippet));
-      lines.push("```");
-      lines.push("");
+      lines.push(`### ${entry.title}`, `_${formatMeta(entry)}_`, "", "```", sanitizeFence(hit.snippet), "```", "");
     }
   }
   return lines.join("\n").trimEnd();
@@ -50009,14 +50035,10 @@ function buildHeading(header) {
 function renderEntry(entry, matchedSuffix, maxSnippetLen) {
   const lines = [];
   const suffix = matchedSuffix ?? "";
-  lines.push(`### ${entry.title}${suffix}`);
-  lines.push(`_${formatMeta(entry)}_`);
+  lines.push(`### ${entry.title}${suffix}`, `_${formatMeta(entry)}_`);
   const snippet = pickSnippet(entry, maxSnippetLen);
   if (snippet !== void 0) {
-    lines.push("");
-    for (const line of snippet.split("\n")) {
-      lines.push(`> ${line}`);
-    }
+    lines.push("", ...snippet.split("\n").map((line) => `> ${line}`));
   }
   lines.push("");
   return lines;
@@ -50042,7 +50064,7 @@ function pickSnippet(entry, maxLen) {
   return snippet === void 0 ? void 0 : sanitizeFence(snippet);
 }
 function sanitizeFence(text) {
-  return text.replace(/```/g, "`\u200B``");
+  return text.replaceAll("```", "`\u200B``");
 }
 
 // packages/dbx-components-mcp/src/tools/log-search/keyword-match.ts
@@ -50143,34 +50165,35 @@ function extractSectionBody(lines, headingRegex) {
   return body;
 }
 function extractFirstFencedBlock(lines) {
-  let openIdx = -1;
-  for (const [i, line] of lines.entries()) {
-    if (line.startsWith("```")) {
-      openIdx = i;
-      break;
-    }
-  }
+  const openIdx = findLineStarting(lines, "```", 0);
   let result = { subject: void 0, body: void 0 };
   if (openIdx >= 0) {
-    let closeIdx = -1;
-    for (let i = openIdx + 1; i < lines.length; i += 1) {
-      if (lines[i].startsWith("```")) {
-        closeIdx = i;
-        break;
-      }
-    }
+    const closeIdx = findLineStarting(lines, "```", openIdx + 1);
     if (closeIdx > openIdx) {
-      const inner = lines.slice(openIdx + 1, closeIdx);
-      const subject = inner[0]?.trim();
-      const body = inner.slice(2).join("\n").trim();
-      result = {
-        subject: subject !== void 0 && subject.length > 0 ? subject : void 0,
-        body: body.length > 0 ? body : void 0
-      };
+      result = sliceFencedBlock(lines, openIdx, closeIdx);
     }
   }
   return result;
 }
+function findLineStarting(lines, prefix, fromIndex) {
+  let found = -1;
+  for (let i = fromIndex; i < lines.length; i += 1) {
+    if (lines[i].startsWith(prefix)) {
+      found = i;
+      break;
+    }
+  }
+  return found;
+}
+function sliceFencedBlock(lines, openIdx, closeIdx) {
+  const inner = lines.slice(openIdx + 1, closeIdx);
+  const subject = inner[0]?.trim();
+  const body = inner.slice(2).join("\n").trim();
+  return {
+    subject: subject !== void 0 && subject.length > 0 ? subject : void 0,
+    body: body.length > 0 ? body : void 0
+  };
+}
 function isIsoDate(value) {
   return value !== void 0 && /^\d{4}-\d{2}-\d{2}$/.test(value);
 }
@@ -53234,26 +53257,7 @@ function createAuthRoleLookupTool(input) {
       if (parsed instanceof type103.errors) {
         result = toolError(`Invalid arguments: ${parsed.summary}`);
       } else {
-        const { topic, tag, model, verb, depth = "full" } = parsed;
-        let text;
-        if (model !== void 0 || verb !== void 0) {
-          text = formatReverseLookup({ registry, model, verb });
-        } else if (tag !== void 0 && tag.length > 0) {
-          text = formatTagLookup(registry, tag, depth);
-        } else if (topic !== void 0 && topic.length > 0) {
-          const normalized = topic.trim();
-          if (isCatalogTopic3(normalized)) {
-            text = formatCatalog11(registry.roles);
-          } else {
-            const role = registry.findRole(normalized);
-            if (role === void 0) {
-              text = formatNotFound19(normalized, registry.roles);
-            } else {
-              const claims = registry.findClaimsForRole(role.role);
-              text = formatRole(role, claims, depth);
-            }
-          }
-        }
+        const text = dispatchRoleLookup(registry, parsed);
         result = text === void 0 ? toolError("Provide one of `topic`, `tag`, or `model` + `verb`.") : { content: [{ type: "text", text }] };
       }
       return result;
@@ -53261,6 +53265,33 @@ function createAuthRoleLookupTool(input) {
   };
   return tool;
 }
+function dispatchRoleLookup(registry, args) {
+  const { topic, tag, model, verb, depth = "full" } = args;
+  let text;
+  if (model !== void 0 || verb !== void 0) {
+    text = formatReverseLookup({ registry, model, verb });
+  } else if (tag !== void 0 && tag.length > 0) {
+    text = formatTagLookup(registry, tag, depth);
+  } else if (topic !== void 0 && topic.length > 0) {
+    text = formatTopicLookup(registry, topic.trim(), depth);
+  }
+  return text;
+}
+function formatTopicLookup(registry, normalized, depth) {
+  let text;
+  if (isCatalogTopic3(normalized)) {
+    text = formatCatalog11(registry.roles);
+  } else {
+    const role = registry.findRole(normalized);
+    if (role === void 0) {
+      text = formatNotFound19(normalized, registry.roles);
+    } else {
+      const claims = registry.findClaimsForRole(role.role);
+      text = formatRole(role, claims, depth);
+    }
+  }
+  return text;
+}
 function isCatalogTopic3(topic) {
   const lowered = topic.toLowerCase();
   return lowered === "list" || lowered === "all" || lowered === "catalog";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dereekb/dbx-components-mcp",
-  "version": "13.12.8",
+  "version": "13.12.9",
   "type": "module",
   "bin": {
     "dbx-components-mcp": "dbx-components-mcp.js"
@@ -11,7 +11,7 @@
     "ts-morph": "^21.0.0"
   },
   "peerDependencies": {
-    "@dereekb/dbx-cli": "13.12.8",
-    "@dereekb/util": "13.12.8"
+    "@dereekb/dbx-cli": "13.12.9",
+    "@dereekb/util": "13.12.9"
   }
 }