@lumerahq/cli 0.35.1 → 0.35.2

package/dist/index.js

@@ -243,25 +243,25 @@ async function main() {
     switch (command) {
       // Resource commands
       case "plan":
-        await import("./resources-X7K436TK.js").then((m) => m.plan(args.slice(1)));
+        await import("./resources-E32DYKLL.js").then((m) => m.plan(args.slice(1)));
         break;
       case "apply":
-        await import("./resources-X7K436TK.js").then((m) => m.apply(args.slice(1)));
+        await import("./resources-E32DYKLL.js").then((m) => m.apply(args.slice(1)));
         break;
       case "pull":
-        await import("./resources-X7K436TK.js").then((m) => m.pull(args.slice(1)));
+        await import("./resources-E32DYKLL.js").then((m) => m.pull(args.slice(1)));
         break;
       case "destroy":
-        await import("./resources-X7K436TK.js").then((m) => m.destroy(args.slice(1)));
+        await import("./resources-E32DYKLL.js").then((m) => m.destroy(args.slice(1)));
         break;
       case "list":
-        await import("./resources-X7K436TK.js").then((m) => m.list(args.slice(1)));
+        await import("./resources-E32DYKLL.js").then((m) => m.list(args.slice(1)));
         break;
       case "show":
-        await import("./resources-X7K436TK.js").then((m) => m.show(args.slice(1)));
+        await import("./resources-E32DYKLL.js").then((m) => m.show(args.slice(1)));
         break;
       case "diff":
-        await import("./resources-X7K436TK.js").then((m) => m.diff(args.slice(1)));
+        await import("./resources-E32DYKLL.js").then((m) => m.diff(args.slice(1)));
         break;
       // Development
       case "dev":

package/dist/{resources-X7K436TK.js → resources-E32DYKLL.js}

@@ -382,7 +382,7 @@ var automationMainGuardRule = {
     if (entrypoint === null) return [];
     const guard = findTopLevelMainGuard(target.source);
     if (guard === null) return [];
-    const warning2 = {
+    const warning3 = {
       ruleId: "automation-main-guard",
       target,
       severity: "warning",
@@ -390,7 +390,7 @@ var automationMainGuardRule = {
       snippet: guard.snippet,
       message: `Function automations must not contain a top-level __main__ guard. Lumera invokes "${entrypoint}" automatically; remove the guard.`
     };
-    return [warning2];
+    return [warning3];
   }
 };
 
@@ -407,6 +407,14 @@ function error(target, message, snippet) {
     snippet
   };
 }
+function warning(target, message) {
+  return {
+    ruleId: "collection-schema",
+    target,
+    severity: "warning",
+    message
+  };
+}
 var collectionSchemaRule = {
   id: "collection-schema",
   description: "Validates local collection JSON files before planning or applying resources.",
@@ -466,6 +474,19 @@ var collectionSchemaRule = {
         if (typeof field.type !== "string" || field.type.trim() === "") {
           issues.push(error(target, `Field ${typeof field.name === "string" ? `"${field.name}"` : `at index ${i}`} is missing required string field "type".`));
         }
+        if (field.type === "lumera_file" || field.type === "file") {
+          const label = typeof field.name === "string" ? `"${field.name}"` : `at index ${i}`;
+          const ignored = [
+            ...isRecord3(field.options) && field.options.maxSelect !== void 0 ? ['"options.maxSelect"'] : [],
+            ...field.multiple !== void 0 ? ['"multiple"'] : []
+          ];
+          if (ignored.length > 0) {
+            issues.push(warning(
+              target,
+              `File field ${label} sets ${ignored.join(" and ")}, which collection files ignore; it deploys with maxSelect ${field.maxSelect ?? 1}. Set top-level "maxSelect" instead. A file field's maxSelect cannot change after it is first deployed.`
+            ));
+          }
+        }
       }
     }
     if (audit !== void 0 && typeof audit !== "boolean") {
@@ -817,7 +838,7 @@ function error2(target, message, line) {
     ...line !== void 0 ? { line } : {}
   };
 }
-function warning(target, message, line) {
+function warning2(target, message, line) {
   return {
     ruleId: "hook-verify",
     target,
@@ -897,7 +918,7 @@ var hookVerifyRule = {
     if (errors.length > 0) return errors;
     const advisories = [];
     if (finding.legacyHandlerPattern) {
-      advisories.push(warning(target, LEGACY_HANDLER_MIGRATION_HINT));
+      advisories.push(warning2(target, LEGACY_HANDLER_MIGRATION_HINT));
     }
     let script = finding.scriptBody ?? "";
     if (ctx.appName) {
@@ -1193,16 +1214,19 @@ async function safeLintPass(label, build) {
     return null;
   }
 }
-async function lintCollectionFiles(projectRoot, platformDir, filterName) {
+async function lintCollectionFiles(projectRoot, platformDir, filterName, printWarnings = true) {
   const result = await safeLintPass(
     "collection",
     async () => runLint({ projectRoot, targets: buildCollectionTargets(platformDir, filterName) })
   );
-  if (!result || result.issues.length === 0) return;
-  safePrintLint(result.issues, projectRoot);
+  if (!result || result.issues.length === 0) return [];
+  if (printWarnings || result.errors.length > 0) {
+    safePrintLint(result.issues, projectRoot);
+  }
   if (result.errors.length > 0) {
     throw new Error(`Found ${result.errors.length} collection lint error(s)`);
   }
+  return result.issues;
 }
 async function lintHookFiles(api, projectRoot, platformDir, appName, filterName) {
   const result = await safeLintPass("hook", async () => {
@@ -2111,6 +2135,20 @@ function fieldsDiffer(local, remote) {
   }
   return false;
 }
+function lumeraFileMaxSelectErrors(local, remote) {
+  const remoteFields = new Map(remote.schema.map((f) => [f.name, f]));
+  const errors = [];
+  for (const field of local.fields) {
+    const deployed = remoteFields.get(field.name);
+    if (mapFieldType(field.type) !== "lumera_file" || deployed?.type !== "lumera_file") continue;
+    const deployedMaxSelect = Math.max(Number(normalizeFieldOption("lumera_file", "maxSelect", deployed.options?.maxSelect)), 1);
+    const requestedMaxSelect = Math.max(Number(normalizeFieldOption("lumera_file", "maxSelect", field.maxSelect)), 1);
+    if (deployedMaxSelect === requestedMaxSelect) continue;
+    const requested = field.maxSelect === void 0 ? "omitting maxSelect (defaults to 1)" : `maxSelect ${requestedMaxSelect}`;
+    errors.push(`File field "${field.name}" is deployed with maxSelect ${deployedMaxSelect}; ${requested} would change it, which is not supported. Keep maxSelect ${deployedMaxSelect}.`);
+  }
+  return errors;
+}
 function localIndexesToSql(local, appName) {
   if (!local.indexes || local.indexes.length === 0) return [];
   return local.indexes.map((idx) => buildIndexSql(local, idx, appName)).sort();
@@ -2224,13 +2262,15 @@ async function planCollections(api, localCollections, appName, projectId) {
           const f = remoteFieldMap.get(name);
           fieldDetails.push({ action: "-", name: f.name, type: f.type, required: f.required });
         }
+        const errors = lumeraFileMaxSelectErrors(local, remote);
         changes.push({
           type: "update",
           resource: "collection",
           id: local.id,
           name: local.name,
           details: details.join(", "),
-          fieldDetails
+          fieldDetails,
+          ...errors.length > 0 ? { errors } : {}
         });
       }
     }
@@ -4422,8 +4462,9 @@ async function plan(args) {
   console.log(pc2.cyan(pc2.bold("  Plan")));
   console.log(pc2.dim("  Comparing local files to remote state..."));
   console.log();
+  let collectionLintWarnings = [];
   if (!type || type === "collections") {
-    await lintCollectionFiles(projectRoot, platformDir, name || void 0);
+    collectionLintWarnings = await lintCollectionFiles(projectRoot, platformDir, name || void 0, false);
   }
   if (!type || type === "hooks") {
     await lintHookFiles(api, projectRoot, platformDir, appName, name || void 0);
@@ -4498,6 +4539,7 @@ async function plan(args) {
     }
   }
   const lintWarnings = [
+    ...collectionLintWarnings,
     ...agentLintWarnings,
     ...await safeLint(projectRoot, localAutomations)
   ];
@@ -4518,6 +4560,7 @@ async function plan(args) {
     safePrintLint(lintWarnings, projectRoot);
     return;
   }
+  const collectionErrorCount = allChanges.reduce((count, change) => count + (change.errors?.length ?? 0), 0);
   if (jsonOutput) {
     const warnings = [];
     for (const change of allChanges) {
@@ -4528,8 +4571,9 @@ async function plan(args) {
         }
       }
     }
-    console.log(JSON.stringify({ changes: allChanges, warnings, validationErrors: planValidationErrors, lintWarnings: serializeLintWarnings(lintWarnings, projectRoot) }));
-    if (planValidationErrors > 0) process.exitCode = 1;
+    const validationErrors = planValidationErrors + collectionErrorCount;
+    console.log(JSON.stringify({ changes: allChanges, warnings, validationErrors, lintWarnings: serializeLintWarnings(lintWarnings, projectRoot) }));
+    if (validationErrors > 0) process.exitCode = 1;
     return;
   }
   console.log(pc2.bold("  Changes:"));
@@ -4547,6 +4591,10 @@ async function plan(args) {
       }
       console.log();
     }
+    if (change.errors && change.errors.length > 0) {
+      for (const message of change.errors) console.log(pc2.red(`      \u2717 ${message}`));
+      console.log();
+    }
     if (change.textDiffs && change.textDiffs.length > 0) {
       const maxDiffLines = 20;
       for (const td of change.textDiffs) {
@@ -4572,6 +4620,11 @@ async function plan(args) {
   safePrintLint(lintWarnings, projectRoot);
   if (planValidationErrors > 0) {
     console.log(pc2.red(`  \u2717 Fix ${planValidationErrors} agent skill validation error${planValidationErrors === 1 ? "" : "s"} before applying.`));
+  }
+  if (collectionErrorCount > 0) {
+    console.log(pc2.red(`  \u2717 Fix ${collectionErrorCount} collection error${collectionErrorCount === 1 ? "" : "s"} before applying.`));
+  }
+  if (planValidationErrors > 0 || collectionErrorCount > 0) {
     process.exitCode = 1;
   } else {
     console.log(pc2.dim(`  Run 'lumera apply' to apply these changes.`));
@@ -4679,6 +4732,15 @@ async function apply(args) {
     console.log();
     process.exit(1);
   }
+  const collectionErrors = allChanges.flatMap((change) => (change.errors ?? []).map((message) => `${change.resource} ${change.name}: ${message}`));
+  if (collectionErrors.length > 0) {
+    console.log();
+    for (const message of collectionErrors) console.log(pc2.red(`  \u2717 ${message}`));
+    console.log();
+    console.log(pc2.red(`  \u2717 Apply aborted: fix ${collectionErrors.length} collection error${collectionErrors.length === 1 ? "" : "s"} first. Nothing was applied.`));
+    console.log();
+    process.exit(1);
+  }
   let willDeployApp = false;
   if (!type && !skipApp) {
     try {
@@ -5297,6 +5359,7 @@ export {
   loadLocalCollections,
   loadLocalSkills,
   localProjectSkillSlug,
+  lumeraFileMaxSelectErrors,
   plan,
   planAgents,
   planCollections,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lumerahq/cli",
-  "version": "0.35.1",
+  "version": "0.35.2",
   "description": "CLI for building and deploying Lumera apps",
   "type": "module",
   "engines": {