Fix build

Author: joshwooding

packages/mcp/package.json

@@ -5,7 +5,7 @@
   "repository": {
     "type": "git",
     "url": "git+https://github.com/jpmorganchase/salt-ds.git",
-    "directory": "packages/salt-mcp"
+    "directory": "packages/mcp"
   },
   "bugs": "https://github.com/jpmorganchase/salt-ds/issues",
   "main": "src/index.ts",

packages/mcp/scripts/buildRegistry.mjs

@@ -2,6 +2,7 @@ import fs from "node:fs/promises";
 import { createRequire } from "node:module";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
+import { ensureSiteBuildInputs } from "./ensureSiteBuildInputs.mjs";
 
 const ALLOWED_FLAGS = new Set(["source-root", "output-dir"]);
 
@@ -46,6 +47,13 @@ const distIndexPath = path.join(
   "index.js",
 );
 const require = createRequire(import.meta.url);
+const flags = parseFlags(process.argv.slice(2));
+const sourceRoot = flags["source-root"]
+  ? path.resolve(flags["source-root"])
+  : repoRoot;
+const outputDir = flags["output-dir"]
+  ? path.resolve(flags["output-dir"])
+  : path.join(packageRoot, "generated");
 
 try {
   await fs.access(distIndexPath);
@@ -56,13 +64,7 @@ try {
 }
 
 const { buildRegistry } = require(distIndexPath);
-const flags = parseFlags(process.argv.slice(2));
-const sourceRoot = flags["source-root"]
-  ? path.resolve(flags["source-root"])
-  : repoRoot;
-const outputDir = flags["output-dir"]
-  ? path.resolve(flags["output-dir"])
-  : path.join(packageRoot, "generated");
+await ensureSiteBuildInputs(sourceRoot);
 
 const registry = await buildRegistry({
   sourceRoot,

packages/mcp/scripts/ensureSiteBuildInputs.mjs

@@ -0,0 +1,102 @@
+import { spawnSync } from "node:child_process";
+import fs from "node:fs/promises";
+import path from "node:path";
+
+const REQUIRED_PROP_FILES = [
+  "ag-grid-theme-props.json",
+  "core-props.json",
+  "countries-props.json",
+  "data-grid-props.json",
+  "embla-carousel-props.json",
+  "icons-props.json",
+  "lab-props.json",
+  "react-resizable-panel-theme-props.json",
+];
+
+async function pathExists(targetPath) {
+  try {
+    await fs.access(targetPath);
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+async function getMissingInputs(repoRoot) {
+  const missing = [];
+  const searchDataPath = path.join(
+    repoRoot,
+    "site",
+    "public",
+    "search-data.json",
+  );
+  const snapshotRoot = path.join(
+    repoRoot,
+    "site",
+    "snapshots",
+    "latest",
+    "salt",
+  );
+  const propsDir = path.join(repoRoot, "site", "src", "props");
+
+  if (!(await pathExists(searchDataPath))) {
+    missing.push(searchDataPath);
+  }
+  if (!(await pathExists(snapshotRoot))) {
+    missing.push(snapshotRoot);
+  }
+
+  for (const fileName of REQUIRED_PROP_FILES) {
+    const filePath = path.join(propsDir, fileName);
+    if (!(await pathExists(filePath))) {
+      missing.push(filePath);
+    }
+  }
+
+  return missing;
+}
+
+function runYarnGenSnapshot(repoRoot) {
+  const command = process.platform === "win32" ? "yarn.cmd" : "yarn";
+  const result = spawnSync(
+    command,
+    ["workspace", "@salt-ds/site", "gen:snapshot"],
+    {
+      cwd: repoRoot,
+      stdio: "inherit",
+      env: process.env,
+    },
+  );
+
+  if (result.status !== 0) {
+    throw new Error("Failed to generate required Salt site artifacts for MCP.");
+  }
+}
+
+export async function ensureSiteBuildInputs(repoRoot) {
+  const missingBefore = await getMissingInputs(repoRoot);
+  if (missingBefore.length === 0) {
+    return;
+  }
+
+  console.error(
+    "Missing Salt site artifacts required for MCP registry build. Regenerating with `yarn workspace @salt-ds/site gen:snapshot`.",
+  );
+
+  await fs.rm(path.join(repoRoot, "site", "snapshots", "latest"), {
+    recursive: true,
+    force: true,
+  });
+  await fs.rm(path.join(repoRoot, "site", "public", "search-data.json"), {
+    force: true,
+  });
+
+  runYarnGenSnapshot(repoRoot);
+
+  const missingAfter = await getMissingInputs(repoRoot);
+  if (missingAfter.length > 0) {
+    throw new Error(
+      `Salt site artifact generation completed, but required MCP inputs are still missing:\n${missingAfter.join("\n")}`,
+    );
+  }
+}

scripts/build.mjs

@@ -13,6 +13,10 @@ import { transformWorkspaceDeps } from "./transformWorkspaceDeps.mjs";
 import { distinct } from "./utils.mjs";
 
 const cwd = process.cwd();
+const repoRoot = path.resolve(
+  path.dirname(url.fileURLToPath(import.meta.url)),
+  "..",
+);
 
 const packageJson = (
   await import(url.pathToFileURL(path.join(cwd, "package.json")), {
@@ -128,7 +132,10 @@ await fs.writeJSON(
 );
 
 for (const file of FILES_TO_COPY) {
-  const filePath = path.join(cwd, file);
+  let filePath = path.join(cwd, file);
+  if (file === "LICENSE" && !(await fs.pathExists(filePath))) {
+    filePath = path.join(repoRoot, file);
+  }
   try {
     await fs.copy(filePath, path.join(outputDir, file));
   } catch (error) {