@swmansion/argent 0.18.1-next.13 → 0.18.1-next.15

package/dist/cli-cmds.mjs

@@ -1304,15 +1304,20 @@ function setAtPath(obj, dottedKey, value) {
 }
 function deleteAtPath(obj, dottedKey) {
   const parts = splitKey(dottedKey);
-  let cur = obj;
+  const chain = [obj];
   for (let i2 = 0; i2 < parts.length - 1; i2++) {
-    const next = cur[parts[i2]];
+    const next = chain[i2][parts[i2]];
     if (!isPlainObject(next)) return false;
-    cur = next;
+    chain.push(next);
   }
+  const parent = chain[parts.length - 1];
   const leaf = parts[parts.length - 1];
-  if (!Object.hasOwn(cur, leaf)) return false;
-  delete cur[leaf];
+  if (!Object.hasOwn(parent, leaf)) return false;
+  delete parent[leaf];
+  for (let i2 = chain.length - 1; i2 >= 1; i2--) {
+    if (Object.keys(chain[i2]).length > 0) break;
+    delete chain[i2 - 1][parts[i2 - 1]];
+  }
   return true;
 }
 var LOCK_STALE_MS2 = 1e4;
@@ -1447,6 +1452,9 @@ function asString(raw) {
   const trimmed = raw.trim();
   return trimmed === "" ? void 0 : trimmed;
 }
+function asNumber(raw) {
+  return typeof raw === "number" && Number.isFinite(raw) ? raw : void 0;
+}
 function asStringArray(raw) {
   if (!Array.isArray(raw)) return void 0;
   const out = [];
@@ -1455,6 +1463,15 @@ function asStringArray(raw) {
   }
   return out;
 }
+var PARSER_EXPECTATIONS = /* @__PURE__ */ new Map([
+  [asBoolean, "a boolean (true or false)"],
+  [asString, "a non-empty string"],
+  [asNumber, "a number"],
+  [asStringArray, "an array of strings"]
+]);
+function describeExpectedValue(def) {
+  return def.expected ?? PARSER_EXPECTATIONS.get(def.parse);
+}
 var CONFIG_SCHEMA = [
   {
     key: "telemetry.enabled",
@@ -1560,12 +1577,18 @@ var ConfigScopeError = class extends Error {
   allowed;
 };
 var ConfigValidationError = class extends Error {
-  constructor(key) {
-    super(`Invalid value for config key "${key}".`);
+  constructor(key, expected, example) {
+    super(
+      expected ? `Invalid value for config key "${key}": expected ${expected}.` : `Invalid value for config key "${key}".`
+    );
     this.key = key;
+    this.expected = expected;
+    this.example = example;
     this.name = "ConfigValidationError";
   }
   key;
+  expected;
+  example;
 };
 var ConfigManagedElsewhereError = class extends Error {
   constructor(key, command) {
@@ -1582,7 +1605,8 @@ function setConfigValue(key, rawValue, scope = "global", options = {}, registry
   if (def.manageCommand) throw new ConfigManagedElsewhereError(key, def.manageCommand);
   if (!def.scopes.includes(scope)) throw new ConfigScopeError(key, scope, def.scopes);
   const parsed = def.parse(rawValue);
-  if (parsed === void 0) throw new ConfigValidationError(key);
+  if (parsed === void 0)
+    throw new ConfigValidationError(def.key, describeExpectedValue(def), def.example);
   updateConfig((config2) => setAtPath(config2, key, parsed), scope, options);
   return parsed;
 }
@@ -1607,6 +1631,8 @@ function listConfig(options = {}, registry = CONFIG_SCHEMA) {
     description: def.description,
     scopes: def.scopes,
     ...def.manageCommand ? { manageCommand: def.manageCommand } : {},
+    ...describeExpectedValue(def) ? { expected: describeExpectedValue(def) } : {},
+    ...def.example ? { example: def.example } : {},
     effective: getConfigValue(def, options),
     project: readScopeValue(def, "project", options),
     global: readScopeValue(def, "global", options)
@@ -8465,6 +8491,12 @@ function isJsonField(prop) {
 function flagNameFor(name, prop) {
   return isJsonField(prop) ? `--${name}-json` : `--${name}`;
 }
+function booleanLiteral(raw) {
+  const value = raw.trim().toLowerCase();
+  if (value === "true" || value === "1") return true;
+  if (value === "false" || value === "0") return false;
+  return void 0;
+}
 function coerceScalar(raw, type, field) {
   if (type === "number") {
     if (raw.trim() === "")
@@ -8482,9 +8514,9 @@ function coerceScalar(raw, type, field) {
     return n2;
   }
   if (type === "boolean") {
-    if (raw === "true" || raw === "1") return true;
-    if (raw === "false" || raw === "0") return false;
-    throw new FlagParseException(`--${field} expected true/false, got "${raw}"`);
+    const value = booleanLiteral(raw);
+    if (value !== void 0) return value;
+    throw new FlagParseException(`--${field} expected true/false (or 1/0), got "${raw}"`);
   }
   return raw;
 }
@@ -8561,6 +8593,12 @@ function parseFlags(argv, schema) {
         if (inlineValue !== void 0) {
           throw new FlagParseException(`--no-${fieldName} does not take a value`);
         }
+        const following = i2 + 1 < argv.length ? booleanLiteral(argv[i2 + 1]) : void 0;
+        if (following !== void 0) {
+          throw new FlagParseException(
+            `--no-${fieldName} does not take a value; use --${fieldName} ${following}`
+          );
+        }
         args[fieldName] = false;
         continue;
       }
@@ -8569,6 +8607,12 @@ function parseFlags(argv, schema) {
     if (propSchema?.type === "boolean") {
       if (inlineValue !== void 0) {
         args[flag] = coerceScalar(inlineValue, "boolean", flag);
+        continue;
+      }
+      const next = i2 + 1 < argv.length ? booleanLiteral(argv[i2 + 1]) : void 0;
+      if (next !== void 0) {
+        args[flag] = next;
+        i2 += 1;
       } else {
         args[flag] = true;
       }
@@ -8626,6 +8670,12 @@ function formatSchemaUsage(schema) {
     const desc = prop.description ? `  ${prop.description}` : "";
     lines.push(`  ${flag}  ${typeLabel}${req}${desc}`);
   }
+  if (entries.some(([, prop]) => prop.type === "boolean")) {
+    lines.push(
+      "",
+      "  Booleans: --flag, --flag true, or --flag 1 sets true; --flag false, --flag 0, --flag=false, or --no-flag sets false."
+    );
+  }
   return lines.join("\n");
 }
 function renderFlagName(name, prop) {
@@ -8924,6 +8974,11 @@ Examples:
     printToolHelp(meta);
     return;
   }
+  if (parsed.positional.length > 0) {
+    console.error(
+      `Note: ignoring unused argument(s): ${parsed.positional.join(", ")}. Pass values as --flag <value> or --flag=<value>.`
+    );
+  }
   let payload = {};
   if (parsed.rawArgs !== null) {
     let rawJson = parsed.rawArgs;
@@ -9081,6 +9136,12 @@ function parseRunArgs(argv) {
     };
     const noValue = (name) => {
       if (inline !== void 0) throw new FlagParseException(`${name} does not take a value`);
+      const next = argv[i2 + 1]?.trim().toLowerCase();
+      if (next === "true" || next === "false") {
+        throw new FlagParseException(
+          `${name} does not take a value \u2014 it is a switch; omit it to leave the option off`
+        );
+      }
     };
     if (flag === "--update-baselines") {
       noValue("--update-baselines");
@@ -10451,8 +10512,15 @@ function parseArgs(argv) {
   for (let i2 = 0; i2 < argv.length; i2++) {
     const tok = argv[i2];
     if (tok === "--help" || tok === "-h") help = true;
-    else if (tok === "--forget") forget = true;
-    else if (tok === "--terminal" || tok === "-t") {
+    else if (tok === "--forget") {
+      forget = true;
+      const next = argv[i2 + 1]?.trim().toLowerCase();
+      if (next === "true" || next === "false") {
+        process.stderr.write(`lens: --forget does not take a value; omit it to keep the state
+`);
+        process.exit(2);
+      }
+    } else if (tok === "--terminal" || tok === "-t") {
       const v = argv[++i2];
       if (v === "iterm" || v === "terminal") terminal = v;
       else {
@@ -11013,6 +11081,7 @@ the raw value stored at each scope.`);
 }
 function scopeDetail(e) {
   const parts = [`scopes: ${e.scopes.join(", ")}`];
+  if (e.expected) parts.push(`value: ${e.expected}${e.example ? `, e.g. ${e.example}` : ""}`);
   if (e.project !== void 0) parts.push(`project=${formatValuePlain(e.project)}`);
   if (e.global !== void 0) parts.push(`global=${formatValuePlain(e.global)}`);
   return parts.join("  \xB7  ");
@@ -11077,7 +11146,7 @@ parsed (e.g. \`true\`, \`42\`, \`["a","b"]\`); anything else is stored as a stri
     if (warning) console.error(import_picocolors2.default.yellow(warning));
     console.log(`Set ${import_picocolors2.default.bold(key)} = ${formatValuePlain(stored)} (${scopeLabel(targetScope)}).`);
   } catch (err) {
-    reportError(err);
+    reportError(err, () => suggestCorrectedSet(err, key, rawValue, scope));
   }
 }
 function cmdUnset(argv) {
@@ -11170,7 +11239,20 @@ function formatValue(value) {
   }
   return formatValuePlain(value);
 }
-function reportError(err) {
+function quoteForShell(value) {
+  if (/^[A-Za-z0-9._/@:+-]+$/.test(value)) return value;
+  return `'${value.replace(/'/g, `'\\''`)}'`;
+}
+function suggestCorrectedSet(err, key, rawValue, scope) {
+  if (!(err instanceof ConfigValidationError)) return null;
+  const def = getConfigDefinition(key);
+  if (!def) return null;
+  const wrapped = def.parse([rawValue]);
+  if (wrapped === void 0) return null;
+  const scopeFlag = scope ? ` --scope ${scope}` : "";
+  return `argent config set ${key} ${quoteForShell(JSON.stringify([rawValue]))}${scopeFlag}`;
+}
+function reportError(err, suggest) {
   if (err instanceof ConfigManagedElsewhereError) {
     console.error(`Error: ${err.message} Use \`${err.command}\` instead.`);
   } else if (err instanceof UnknownConfigKeyError || err instanceof ConfigScopeError || err instanceof ConfigValidationError) {
@@ -11178,6 +11260,15 @@ function reportError(err) {
     if (err instanceof UnknownConfigKeyError) {
       console.error(`Run \`argent config list\` to see available keys.`);
     }
+    if (err instanceof ConfigValidationError) {
+      const corrected = suggest?.() ?? null;
+      if (corrected) {
+        console.error(`Did you mean: ${corrected}`);
+      } else if (err.example) {
+        console.error(`Example: argent config set ${err.key} ${quoteForShell(err.example)}`);
+      }
+      console.error(`Run \`argent config list\` to see each key's expected value.`);
+    }
   } else {
     console.error(`Error: ${err instanceof Error ? err.message : String(err)}`);
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@swmansion/argent",
-  "version": "0.18.1-next.13",
+  "version": "0.18.1-next.15",
   "mcpName": "io.github.software-mansion/argent",
   "description": "MCP server for iOS Simulator and Android Emulator control",
   "license": "Apache-2.0",