@happyvertical/smrt-cli 0.36.3 → 0.36.4

package/dist/{index-psX--9zT.js → index-CoHJ40KM.js}

@@ -176,7 +176,10 @@ const configExportCommand = {
         format: outputFormat,
         includeSecrets
       });
-      const outputPath = path.resolve(process.cwd(), options.output);
+      const outputPath = path.resolve(
+        process.cwd(),
+        options.output ?? "smrt.exported.json"
+      );
       const outputDir = path.dirname(outputPath);
       if (!fs.existsSync(outputDir)) {
         fs.mkdirSync(outputDir, { recursive: true });
@@ -488,13 +491,7 @@ const dbDiffCommand = {
   handler: async (_args, options) => {
     let db;
     try {
-      const unsupportedFileOptions = [
-        "generate",
-        "name",
-        "format",
-        "with-down",
-        "output"
-      ].filter(
+      const unsupportedFileOptions = ["generate", "name", "format", "with-down", "output"].filter(
         (option) => options[option] !== void 0 && options[option] !== false
       );
       if (unsupportedFileOptions.length > 0) {
@@ -581,15 +578,9 @@ const dbDiffCommand = {
         }
         console.log();
       }
-      const columnChanges = diff.changes.filter(
-        (c) => c.type === "add_column"
-      );
-      const indexChanges = diff.changes.filter(
-        (c) => c.type === "add_index"
-      );
-      const indexDrops = diff.changes.filter(
-        (c) => c.type === "drop_index"
-      );
+      const columnChanges = diff.changes.filter((c) => c.type === "add_column");
+      const indexChanges = diff.changes.filter((c) => c.type === "add_index");
+      const indexDrops = diff.changes.filter((c) => c.type === "drop_index");
       const typeUpgrades = diff.changes.filter(
         (c) => c.type === "type_upgrade"
       );
@@ -623,7 +614,7 @@ const dbDiffCommand = {
         );
       }
       const orphanDrops = indexDrops.filter(
-        (c) => !recreateNames.has(c.name)
+        (c) => !(c.name && recreateNames.has(c.name))
       );
       if (orphanDrops.length > 0) {
         console.log(`  🗑️  Indexes to drop (${orphanDrops.length}):`);
@@ -633,7 +624,7 @@ const dbDiffCommand = {
         console.log();
       }
       const newIndexes = indexChanges.filter(
-        (c) => !recreateNames.has(c.name)
+        (c) => !(c.name && recreateNames.has(c.name))
       );
       if (newIndexes.length > 0) {
         console.log(`  🗂️  New indexes (${newIndexes.length}):`);
@@ -3213,7 +3204,8 @@ async function getExportableFields(typeName, fileConfig, fieldExportDefault) {
     );
   }
   const exportableFields = [];
-  for (const [fieldName, fieldDef] of fields) {
+  for (const [fieldName, rawFieldDef] of fields) {
+    const fieldDef = rawFieldDef;
     if (fieldDef._meta?.__smrtSystemField === true) continue;
     if (fieldDef.transient) continue;
     if (fieldDef.type === "oneToMany" || fieldDef.type === "manyToMany")
@@ -4110,7 +4102,9 @@ function mergeArraysByKey(base, ours, theirs, keyField = "id") {
   const keylessObjects = [];
   const getKey = (obj) => {
     if (!obj || typeof obj !== "object") return null;
-    return obj[keyField] ?? obj.slug ?? null;
+    const record = obj;
+    const key = record[keyField] ?? record.slug ?? null;
+    return key;
   };
   const getTimestamp = (obj) => {
     if (obj.updated_at) {
@@ -4123,26 +4117,29 @@ function mergeArraysByKey(base, ours, theirs, keyField = "id") {
     }
     return 0;
   };
-  for (const obj of base) {
-    const key = getKey(obj);
+  for (const item of base) {
+    const key = getKey(item);
     if (key) {
+      const obj = item;
       merged.set(key, { ...obj, _source: "base" });
     }
   }
-  for (const obj of ours) {
-    const key = getKey(obj);
+  for (const item of ours) {
+    const key = getKey(item);
     if (key) {
+      const obj = item;
       const existing = merged.get(key);
       if (!existing || existing._source === "base") {
         merged.set(key, { ...obj, _source: "ours" });
       }
-    } else if (obj && typeof obj === "object") {
-      keylessObjects.push(obj);
+    } else if (item && typeof item === "object") {
+      keylessObjects.push(item);
     }
   }
-  for (const obj of theirs) {
-    const key = getKey(obj);
+  for (const item of theirs) {
+    const key = getKey(item);
     if (key) {
+      const obj = item;
       const existing = merged.get(key);
       if (!existing) {
         merged.set(key, { ...obj, _source: "theirs" });
@@ -4183,12 +4180,13 @@ function sanitizeMergeValue(value) {
   if (value === null || typeof value !== "object") {
     return value;
   }
+  const source = value;
   const clean = {};
-  for (const key of Object.keys(value)) {
+  for (const key of Object.keys(source)) {
     if (DANGEROUS_KEYS.has(key)) {
       continue;
     }
-    clean[key] = sanitizeMergeValue(value[key]);
+    clean[key] = sanitizeMergeValue(source[key]);
   }
   return clean;
 }
@@ -4203,19 +4201,22 @@ function mergeObjects(base, ours, theirs) {
   if (typeof theirs !== "object" || theirs === null) {
     return sanitizeMergeValue(ours);
   }
+  const baseObj = base ?? {};
+  const oursObj = ours;
+  const theirsObj = theirs;
   const result = {};
   const allKeys = /* @__PURE__ */ new Set([
-    ...Object.keys(base || {}),
-    ...Object.keys(ours),
-    ...Object.keys(theirs)
+    ...Object.keys(baseObj),
+    ...Object.keys(oursObj),
+    ...Object.keys(theirsObj)
   ]);
   for (const key of allKeys) {
     if (DANGEROUS_KEYS.has(key)) {
       continue;
     }
-    const baseVal = base?.[key];
-    const oursVal = ours[key];
-    const theirsVal = theirs[key];
+    const baseVal = baseObj[key];
+    const oursVal = oursObj[key];
+    const theirsVal = theirsObj[key];
     if (Array.isArray(oursVal) || Array.isArray(theirsVal)) {
       const baseArray = Array.isArray(baseVal) ? baseVal : [];
       const oursArray = Array.isArray(oursVal) ? oursVal : [];
@@ -4495,10 +4496,12 @@ function countObjects(data) {
     return data.length;
   }
   if (typeof data === "object" && data !== null) {
+    const record = data;
     let count = 0;
-    for (const key of Object.keys(data)) {
-      if (Array.isArray(data[key])) {
-        count += data[key].length;
+    for (const key of Object.keys(record)) {
+      const value = record[key];
+      if (Array.isArray(value)) {
+        count += value.length;
       } else {
         count += 1;
       }
@@ -4770,20 +4773,21 @@ async function cleanupGitTemplate(config) {
   }
 }
 function validateTemplateConfig$2(config, source) {
+  const record = config && typeof config === "object" ? config : {};
   const required = ["name", "description", "dependencies"];
   for (const field of required) {
-    if (!config[field]) {
+    if (!record[field]) {
       throw new Error(
         `Invalid template config at ${source}: missing required field '${field}'`
       );
     }
   }
-  if (typeof config.dependencies !== "object") {
+  if (typeof record.dependencies !== "object") {
     throw new Error(
       `Invalid template config at ${source}: 'dependencies' must be an object`
     );
   }
-  if (config.devDependencies && typeof config.devDependencies !== "object") {
+  if (record.devDependencies && typeof record.devDependencies !== "object") {
     throw new Error(
       `Invalid template config at ${source}: 'devDependencies' must be an object`
     );
@@ -4881,20 +4885,21 @@ async function loadLocalTemplate(resolvedPath) {
   }
 }
 function validateTemplateConfig$1(config, source) {
+  const record = config && typeof config === "object" ? config : {};
   const required = ["name", "description", "dependencies"];
   for (const field of required) {
-    if (!config[field]) {
+    if (!record[field]) {
       throw new Error(
         `Invalid template config at ${source}: missing required field '${field}'`
       );
     }
   }
-  if (typeof config.dependencies !== "object") {
+  if (typeof record.dependencies !== "object") {
     throw new Error(
       `Invalid template config at ${source}: 'dependencies' must be an object`
     );
   }
-  if (config.devDependencies && typeof config.devDependencies !== "object") {
+  if (record.devDependencies && typeof record.devDependencies !== "object") {
     throw new Error(
       `Invalid template config at ${source}: 'devDependencies' must be an object`
     );
@@ -5007,20 +5012,21 @@ async function discoverInstalledTemplates() {
   return templates;
 }
 function validateTemplateConfig(config, source) {
+  const record = config && typeof config === "object" ? config : {};
   const required = ["name", "description", "dependencies"];
   for (const field of required) {
-    if (!config[field]) {
+    if (!record[field]) {
       throw new Error(
         `Invalid template config at ${source}: missing required field '${field}'`
       );
     }
   }
-  if (typeof config.dependencies !== "object") {
+  if (typeof record.dependencies !== "object") {
     throw new Error(
       `Invalid template config at ${source}: 'dependencies' must be an object`
     );
   }
-  if (config.devDependencies && typeof config.devDependencies !== "object") {
+  if (record.devDependencies && typeof record.devDependencies !== "object") {
     throw new Error(
       `Invalid template config at ${source}: 'devDependencies' must be an object`
     );
@@ -5204,7 +5210,8 @@ async function overlayTemplate(source, config, outputDir) {
       baseDir = dirname(source.resolved);
       break;
     case "git": {
-      const baseGitDir = config.__templateRoot ?? config.__tempDir;
+      const gitConfig = config;
+      const baseGitDir = gitConfig.__templateRoot ?? gitConfig.__tempDir;
       if (!baseGitDir) {
         throw new Error("Git template temp directory not found");
       }
@@ -5264,6 +5271,9 @@ async function mergePackageJson(outputDir, config, projectName) {
       devDependencies: {}
     };
   }
+  pkg.scripts ??= {};
+  pkg.dependencies ??= {};
+  pkg.devDependencies ??= {};
   pkg.name = projectName;
   pkg.dependencies = {
     ...pkg.dependencies,
@@ -5950,7 +5960,9 @@ const playgroundCommands = {
         const packageResult = writeFileIfAllowed(
           packagePlaygroundPath,
           (await loadPlaygroundRuntime()).createPackagePlaygroundTemplate(
-            packageJson.name
+            // The package target is only reached for a named SMRT package;
+            // String() preserves the previous (untyped) pass-through exactly.
+            String(packageJson.name)
           ),
           force
         );
@@ -7381,7 +7393,7 @@ function rowsFromResult(result) {
   return Array.isArray(result?.rows) ? result.rows : [];
 }
 function getRowCount(result) {
-  return typeof result?.rowCount === "number" ? result.rowCount : void 0;
+  return !Array.isArray(result) && typeof result?.rowCount === "number" ? result.rowCount : void 0;
 }
 function buildNullSafeIdentityPredicate(columns) {
   return columns.map((column) => {
@@ -7389,6 +7401,10 @@ function buildNullSafeIdentityPredicate(columns) {
     return `(${quoted} = ? OR (${quoted} IS NULL AND ? IS NULL))`;
   }).join(" AND ");
 }
+function getRowId(row) {
+  const id = row.id;
+  return id == null ? null : id;
+}
 function getIdentityParams(row, columns) {
   const params = [];
   for (const column of columns) {
@@ -7447,8 +7463,8 @@ async function repairStiDiscriminatorRows(options) {
         tableName,
         legacyMetaType,
         qualifiedMetaType,
-        legacyId: row.id ?? null,
-        qualifiedId: duplicateRows[0].id ?? null,
+        legacyId: getRowId(row),
+        qualifiedId: getRowId(duplicateRows[0]),
         conflictIdentity
       });
       continue;
@@ -7521,6 +7537,15 @@ function formatStiConflictIdentity(conflict) {
   const ids = conflict.legacyId || conflict.qualifiedId ? ` (legacy id: ${conflict.legacyId ?? "unknown"}, qualified id: ${conflict.qualifiedId ?? "unknown"})` : "";
   return `${identity}${ids}`;
 }
+function getErrorContext(error) {
+  if (error && typeof error === "object" && "context" in error) {
+    const context = error.context;
+    if (context && typeof context === "object") {
+      return context;
+    }
+  }
+  return void 0;
+}
 const utilityCommands = {
   introspect: {
     name: "introspect",
@@ -8575,9 +8600,9 @@ ${error.stack}
             errorCount++;
             const errorMsg = error instanceof Error ? error.message : String(error);
             console.error(`  ✗ atomic schema migration failed: ${errorMsg}`);
-            if (error instanceof Error && "context" in error && error.context?.originalError) {
+            if (error instanceof Error && getErrorContext(error)?.originalError) {
               console.error(
-                `     Cause: ${error.context.originalError}`
+                `     Cause: ${getErrorContext(error)?.originalError}`
               );
             }
             if (options.verbose && error instanceof Error && error.stack) {
@@ -8691,7 +8716,7 @@ ${error.stack}
                 } catch (error) {
                   const qualifiedName = resolution.currentQualifiedName;
                   const errorMsg = error instanceof Error ? error.message : String(error);
-                  const originalError = error?.context?.originalError;
+                  const originalError = getErrorContext(error)?.originalError;
                   console.error(
                     `  ✗ ${tableName}: "${metaType}" → "${qualifiedName}" failed: ${originalError || errorMsg}`
                   );
@@ -8783,7 +8808,7 @@ ${error.stack}
         );
         if (error instanceof Error) {
           console.error(`   ${error.message}`);
-          const ctx = error.context;
+          const ctx = getErrorContext(error);
           if (ctx) {
             if (ctx.originalError) {
               console.error(`   Database error: ${ctx.originalError}`);
@@ -8852,7 +8877,9 @@ ${error.stack}
           check("package.json is valid JSON", false, "Invalid JSON format");
         }
       }
-      const hasSmrtCore = packageJson.dependencies?.["@happyvertical/smrt-core"] || packageJson.devDependencies?.["@happyvertical/smrt-core"];
+      const hasSmrtCore = Boolean(
+        packageJson.dependencies?.["@happyvertical/smrt-core"] || packageJson.devDependencies?.["@happyvertical/smrt-core"]
+      );
       check(
         "@happyvertical/smrt-core installed",
         hasSmrtCore,

package/dist/index.js

@@ -25,56 +25,56 @@ let _docsCommands = null;
 let _playgroundCommands = null;
 async function getGnodeCommands() {
   if (!_gnodeCommands) {
-    const { gnodeCommands } = await import("./index-psX--9zT.js");
+    const { gnodeCommands } = await import("./index-CoHJ40KM.js");
     _gnodeCommands = gnodeCommands;
   }
   return _gnodeCommands;
 }
 async function getGitCommands() {
   if (!_gitCommands) {
-    const { gitCommands } = await import("./index-psX--9zT.js");
+    const { gitCommands } = await import("./index-CoHJ40KM.js");
     _gitCommands = gitCommands;
   }
   return _gitCommands;
 }
 async function getGenerateCommands() {
   if (!_generateCommands) {
-    const { generateCommands } = await import("./index-psX--9zT.js");
+    const { generateCommands } = await import("./index-CoHJ40KM.js");
     _generateCommands = generateCommands;
   }
   return _generateCommands;
 }
 async function getInitCommands() {
   if (!_initCommands) {
-    const { initCommands } = await import("./index-psX--9zT.js");
+    const { initCommands } = await import("./index-CoHJ40KM.js");
     _initCommands = initCommands;
   }
   return _initCommands;
 }
 async function getUtilityCommands() {
   if (!_utilityCommands) {
-    const { utilityCommands } = await import("./index-psX--9zT.js");
+    const { utilityCommands } = await import("./index-CoHJ40KM.js");
     _utilityCommands = utilityCommands;
   }
   return _utilityCommands;
 }
 async function getDispatchCommands() {
   if (!_dispatchCommands) {
-    const { dispatchCommands } = await import("./index-psX--9zT.js");
+    const { dispatchCommands } = await import("./index-CoHJ40KM.js");
     _dispatchCommands = dispatchCommands;
   }
   return _dispatchCommands;
 }
 async function getDocsCommands() {
   if (!_docsCommands) {
-    const { docsCommands } = await import("./index-psX--9zT.js");
+    const { docsCommands } = await import("./index-CoHJ40KM.js");
     _docsCommands = docsCommands;
   }
   return _docsCommands;
 }
 async function getPlaygroundCommands() {
   if (!_playgroundCommands) {
-    const { playgroundCommands } = await import("./index-psX--9zT.js");
+    const { playgroundCommands } = await import("./index-CoHJ40KM.js");
     _playgroundCommands = playgroundCommands;
   }
   return _playgroundCommands;
@@ -105,7 +105,8 @@ class CLIGenerator {
    * Check if running in test environment
    */
   isTestMode() {
-    return process.env.NODE_ENV === "test" || process.env.VITEST === "true" || typeof global.it === "function" || typeof global.describe === "function";
+    const testGlobals = global;
+    return process.env.NODE_ENV === "test" || process.env.VITEST === "true" || typeof testGlobals.it === "function" || typeof testGlobals.describe === "function";
   }
   /**
    * Check if a type string represents an inline object type parameter
@@ -208,7 +209,10 @@ class CLIGenerator {
           const tableName = itemClass.SMRT_TABLE_NAME || itemClass.name.toLowerCase();
           const existing = ObjectRegistry.getClass(tableName);
           if (existing && !existing.collectionConstructor) {
-            ObjectRegistry.registerCollection(tableName, exportValue);
+            ObjectRegistry.registerCollection(
+              tableName,
+              exportValue
+            );
             if (config.verbose) {
               console.log(`[CLI] Registered local collection ${exportName}`);
             }
@@ -1173,14 +1177,14 @@ ${objectName}`);
     try {
       const collection = await this.getCollection(objectName);
       const listOptions = {
-        limit: Number.parseInt(options.limit, 10),
-        offset: Number.parseInt(options.offset, 10)
+        limit: Number.parseInt(String(options.limit), 10),
+        offset: Number.parseInt(String(options.offset), 10)
       };
       const orderBy = options["order-by"] ?? options.orderBy;
-      if (orderBy) {
+      if (typeof orderBy === "string" && orderBy) {
         listOptions.orderBy = orderBy;
       }
-      if (options.where) {
+      if (typeof options.where === "string" && options.where) {
         listOptions.where = JSON.parse(options.where);
       }
       const results = await collection.list(listOptions);
@@ -1230,7 +1234,7 @@ ${objectName}`);
     try {
       let data = {};
       const fromFile = options["from-file"] ?? options.fromFile;
-      if (fromFile) {
+      if (typeof fromFile === "string" && fromFile) {
         const fs = await import("node:fs/promises");
         const content = await fs.readFile(fromFile, "utf-8");
         data = JSON.parse(content);
@@ -1241,7 +1245,7 @@ ${objectName}`);
         for (const [fieldName] of fields) {
           const optionName = fieldName.replace(/_/g, "-");
           if (options[optionName] !== void 0) {
-            data[fieldName] = this.parseFieldValue(options[optionName]);
+            data[fieldName] = this.parseFieldValue(String(options[optionName]));
           }
         }
       }
@@ -1272,18 +1276,21 @@ ${objectName}`);
       }
       let data = {};
       const fromFile = options["from-file"] ?? options.fromFile;
-      if (fromFile) {
+      if (typeof fromFile === "string" && fromFile) {
         const fs = await import("node:fs/promises");
         const content = await fs.readFile(fromFile, "utf-8");
         data = JSON.parse(content);
       } else if (options.interactive && this.config.prompt) {
-        data = await this.promptForFields(objectName, existing);
+        data = await this.promptForFields(
+          objectName,
+          existing
+        );
       } else {
         const fields = ObjectRegistry.getFields(objectName);
         for (const [fieldName] of fields) {
           const optionName = fieldName.replace(/_/g, "-");
           if (options[optionName] !== void 0) {
-            data[fieldName] = this.parseFieldValue(options[optionName]);
+            data[fieldName] = this.parseFieldValue(String(options[optionName]));
           }
         }
       }
@@ -1312,8 +1319,9 @@ ${objectName}`);
         return;
       }
       if (!options.force && this.config.prompt) {
+        const label = existing.name || existing.slug || existing.id;
         const confirmed = await this.confirm(
-          `Are you sure you want to delete ${objectName} "${existing.name || existing.slug || existing.id}"?`
+          `Are you sure you want to delete ${objectName} "${label}"?`
         );
         if (!confirmed) {
           console.log("Cancelled");
@@ -1406,7 +1414,10 @@ ${objectName}`);
         this.exitWithError(`Method ${methodName} is not callable`);
         return;
       }
-      const result = await method.call(obj, ...methodCallArgs);
+      const result = await method.call(
+        obj,
+        ...methodCallArgs
+      );
       spinner.succeed(`Executed ${methodName}`);
       console.log(JSON.stringify(result, null, 2));
     } catch (error) {
@@ -1500,8 +1511,8 @@ Check that the package exports the class and .smrt/register.js imports it.`
       const instanceConfig = {
         ...smrtConfig,
         ...moduleConfig,
-        ...useCliDb && { db },
-        ...this.context.ai && { ai: this.context.ai },
+        ...useCliDb ? { db } : {},
+        ...this.context.ai ? { ai: this.context.ai } : {},
         // In JSON mode, silence all log output to ensure clean JSON
         ...jsonMode && { silent: true }
       };
@@ -1559,7 +1570,10 @@ Check that the package exports the class and .smrt/register.js imports it.`
           }
         }
       }
-      const result = await method.call(obj, ...methodCallArgs);
+      const result = await method.call(
+        obj,
+        ...methodCallArgs
+      );
       spinner.succeed(`Executed ${methodName}`);
       if (result !== void 0) {
         console.log(JSON.stringify(result, null, 2));
@@ -1686,9 +1700,10 @@ Troubleshooting:
       return;
     }
     const keys = ["id", "name", "slug", "created_at"];
-    const rows = results.map(
-      (item) => keys.map((key) => String(item[key] || "").substring(0, 30))
-    );
+    const rows = results.map((item) => {
+      const record = item;
+      return keys.map((key) => String(record[key] || "").substring(0, 30));
+    });
     console.log();
     console.log(keys.join("	"));
     console.log("-".repeat(80));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happyvertical/smrt-cli",
-  "version": "0.36.3",
+  "version": "0.36.4",
   "description": "Developer CLI for SMRT framework - introspection, testing, and project management",
   "type": "module",
   "main": "./dist/index.js",
@@ -32,12 +32,12 @@
     "acorn": "^8.15.0",
     "fast-glob": "3.3.3",
     "tar": "^7.5.2",
-    "@happyvertical/smrt-agents": "0.36.3",
-    "@happyvertical/smrt-config": "0.36.3",
-    "@happyvertical/smrt-playground": "0.36.3",
-    "@happyvertical/smrt-core": "0.36.3",
-    "@happyvertical/smrt-dev-mcp": "0.36.3",
-    "@happyvertical/smrt-types": "0.36.3"
+    "@happyvertical/smrt-config": "0.36.4",
+    "@happyvertical/smrt-agents": "0.36.4",
+    "@happyvertical/smrt-core": "0.36.4",
+    "@happyvertical/smrt-dev-mcp": "0.36.4",
+    "@happyvertical/smrt-playground": "0.36.4",
+    "@happyvertical/smrt-types": "0.36.4"
   },
   "devDependencies": {
     "@types/node": "25.0.9",