@wix/himalaya-cli 0.863.0 → 0.864.0

package/dist/cli.mjs

@@ -62248,8 +62248,97 @@ function writeAgentContext(targetDir) {
 // src/cli.ts
 init_wix_placeholders();
 var defaultIO = { log: (l) => console.log(l), error: (l) => console.error(l), isTTY: process.stdout.isTTY === true };
-var BOOLEAN_FLAGS = /* @__PURE__ */ new Set(["browser", "no-open", "yes", "json", "deep", "quiet", "force", "replace", "watch", "no-claim", "new", "dev", "draft", "live", "accept-regressions", "accept-partial", "no-rebind"]);
+var BOOLEAN_FLAGS = /* @__PURE__ */ new Set([
+  "browser",
+  "no-open",
+  "yes",
+  "json",
+  "deep",
+  "quiet",
+  "force",
+  "replace",
+  "watch",
+  "no-claim",
+  "new",
+  "dev",
+  "draft",
+  "live",
+  "accept-regressions",
+  "accept-partial",
+  "no-rebind",
+  "help",
+  "static",
+  "strict",
+  "apply",
+  "apply-permissions",
+  "allow-destructive",
+  "allow-draft",
+  "all",
+  "all-screens",
+  "ai",
+  "binary",
+  "crashes",
+  "crashes-only",
+  "crashesOnly",
+  "dpx",
+  "dry-run",
+  "dump-state",
+  "follow",
+  "invoke",
+  "licenses",
+  "measure-widgets",
+  "no-events",
+  "no-hostile",
+  "no-logs",
+  "no-oauth-app",
+  "no-shots",
+  "no-source",
+  "no-wait",
+  "offline",
+  "publish",
+  "record",
+  "remote",
+  "rtl",
+  "skip-version-check",
+  "snapshot",
+  "summary",
+  "templates",
+  "tree",
+  "visitor",
+  "wait",
+  "wix",
+  "silent",
+  "include-disabled",
+  "skip-build"
+]);
 var SHELL_ENUM = SHELL_NAMES2.join("|");
+var SWITCH_FLAGS = new Set([...BOOLEAN_FLAGS].filter((name) => name !== "deep"));
+var SWITCH_CONSEQUENCE = {
+  yes: "silently ignored, it would turn a real mutation into a dry run",
+  new: "silently ignored, it would reuse a client instead of minting a new one",
+  force: "silently ignored, it would stop at the first refusal it was meant to override",
+  record: "silently ignored, it would leave you believing the fixture was refreshed",
+  "no-oauth-app": "silently ignored, it would mint an OAuth client you asked to skip",
+  "dry-run": "silently ignored, it would perform the real writes you asked to rehearse",
+  "allow-destructive": "silently ignored, it would refuse the destructive step you opted into",
+  static: "silently ignored, it would render HYDRATED when you asked for static",
+  // `--wait=false` is the one spelling where "pass it bare" is the WRONG advice: bare `--wait`
+  // means wait. The flag that turns waiting off is a different one, so name it.
+  wait: "if you meant to stop waiting, that is `--no-wait` \u2014 a bare `--wait` means the opposite"
+};
+function switchFlagErrors(argv) {
+  const out = [];
+  for (const a of argv) {
+    if (!a.startsWith("--")) continue;
+    const eq = a.indexOf("=");
+    if (eq < 0) continue;
+    const name = a.slice(2, eq);
+    if (!SWITCH_FLAGS.has(name)) continue;
+    const because = SWITCH_CONSEQUENCE[name];
+    out.push(`--${name} takes no value (got "${a}") \u2014 pass it bare${because ? `; ${because}` : ", it is an on/off flag"}`);
+  }
+  return out;
+}
 function parseFlags(argv) {
   const _ = [];
   const flags = {};
@@ -63789,9 +63878,9 @@ async function producedWidgetFamilies(dir) {
 }
 async function run(argv, io = defaultIO) {
   const { _: pos, flags } = parseFlags(argv);
+  const flagErrors = switchFlagErrors(argv);
   const command = pos[0];
   const print = (o) => io.log(JSON.stringify(o, null, 2));
-  const flagInArgv = (name) => argv.some((a) => a === `--${name}` || a.startsWith(`--${name}=`));
   if (flags.version === true || command === "version" || pos.includes("-v")) {
     const v = cliVersion();
     io.log(flags.json ? JSON.stringify({ himi: v }, null, 2) : `himi v${v}`);
@@ -63805,6 +63894,10 @@ himi v${cliVersion()} \u2014 run \`himi --version\`. If authoring commands fail
     io.log(flags.json ? JSON.stringify(helpJson(), null, 2) : USAGE + footer);
     return command === "help" || flags.help ? 0 : 2;
   }
+  if (flagErrors.length > 0 && flags.help !== true) {
+    io.error(JSON.stringify({ ok: false, error: "invalid_flag_value", problems: flagErrors, hint: "run `himi help` for the flag grammar." }));
+    return 2;
+  }
   const contextualDefaultTarget = defaultTargetForCommand(command, contentDir(flags));
   const targetName = () => resolveTargetName({ target: str5(flags.target), defaultTarget: contextualDefaultTarget });
   const client2 = () => new ServeCliClient(
@@ -65520,7 +65613,7 @@ default (no --glyph): ${DEFAULT_GLYPH2} \u2014 a mark that claims nothing, so th
               return 2;
             }
           }
-          if (flagInArgv("from-release") && !str5(flags["from-release"])) {
+          if (flags["from-release"] !== void 0 && !str5(flags["from-release"])) {
             io.error(JSON.stringify({ ok: false, error: "from_release_needs_a_value", message: "--from-release takes a release id to branch from. Omit the flag entirely to patch the live release.", usage: ONLY_USAGE }));
             return 2;
           }
@@ -66767,7 +66860,7 @@ default (no --glyph): ${DEFAULT_GLYPH2} \u2014 a mark that claims nothing, so th
           }));
           return 2;
         }
-        if (flagInArgv("from-release") && !str5(flags["from-release"])) {
+        if (flags["from-release"] !== void 0 && !str5(flags["from-release"])) {
           io.error(JSON.stringify({ ok: false, error: "from_release_needs_a_value", message: "--from-release takes a release id to branch from. Omit the flag entirely to patch the live release.", usage: PATCH_USAGE }));
           return 2;
         }
@@ -66864,15 +66957,15 @@ default (no --glyph): ${DEFAULT_GLYPH2} \u2014 a mark that claims nothing, so th
         const bytes2 = readFileSync64(file);
         const sha = createHash24("sha256").update(bytes2).digest("hex");
         const forScreens = (str5(flags["for-screen"]) ?? "").split(",").map((s) => s.trim()).filter(Boolean);
-        if (flagInArgv("no-rebind") && flags["no-rebind"] !== true) {
+        if (flags["no-rebind"] !== void 0 && flags["no-rebind"] !== true) {
           io.error(JSON.stringify({ ok: false, error: "no_rebind_takes_no_value", message: "--no-rebind is a switch and takes no value; pass it bare.", usage: PATCH_USAGE }));
           return 2;
         }
-        if (flagInArgv("for-screen") && forScreens.length === 0) {
+        if (flags["for-screen"] !== void 0 && forScreens.length === 0) {
           io.error(JSON.stringify({ ok: false, error: "for_screen_needs_a_value", message: "--for-screen takes one or more screen ids to repoint at the new bytes, comma-separated.", usage: PATCH_USAGE }));
           return 2;
         }
-        if (flagInArgv("for-screen") && flagInArgv("no-rebind")) {
+        if (flags["for-screen"] !== void 0 && flags["no-rebind"] !== void 0) {
           io.error(JSON.stringify({
             ok: false,
             error: "conflicting_rebind",
@@ -68042,14 +68135,6 @@ ${report.ok ? "OK \u2014 data + serve plane reachable." : typeof report.hint ===
         if (sub === "client") {
           const { runSiteClientEnsure: runSiteClientEnsure2, runSiteClientList: runSiteClientList2, SITE_CLIENT_USAGE: SITE_CLIENT_USAGE2 } = await Promise.resolve().then(() => (init_site_oauth_client(), site_oauth_client_exports));
           const op = pos[2];
-          for (const bare of ["yes", "new"]) {
-            if (flags[bare] !== void 0 && flags[bare] !== true) {
-              io.error(JSON.stringify({
-                error: `--${bare} takes no value (got "${String(flags[bare])}") \u2014 pass it bare; ` + (bare === "yes" ? "silently ignored, it would turn a real create into a dry run" : "silently ignored, it would reuse a client instead of minting a new one")
-              }));
-              return 2;
-            }
-          }
           if (op !== "ensure" && op !== "list") {
             io.error(JSON.stringify({
               error: op ? `unknown site client op "${op}" \u2014 did you mean \`ensure\`? (there is no bare \`create\`: the command reconciles an existing client rather than minting a second one every run \u2014 pass --new to force a fresh client)` : "site client needs an op: ensure | list",
@@ -68234,6 +68319,7 @@ export {
   servePlaneAnswers,
   serveUnreachableError,
   serveUnreachableHint,
+  switchFlagErrors,
   templatePreviewAuthError,
   tokenCoversApp,
   wixPlayerTestflightTarget,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wix/himalaya-cli",
-  "version": "0.863.0",
+  "version": "0.864.0",
   "type": "module",
   "description": "himi — the Himalaya authoring CLI. Build a Tier-5 content package on your own machine, push it to the production serve as a draft, preview it on web + Himi Player, and read worker logs back — the remote dev loop. Wraps @himalaya/serve-cli for transport.",
   "keywords": [
@@ -63,5 +63,5 @@
       "artifactId": "himalaya-cli"
     }
   },
-  "falconPackageHash": "c5d242935309a310e8916c1fc607f5bfc7aebc217e81fa9fbaa08ca7"
+  "falconPackageHash": "613d8c3ecd710dff1527de5fb1e8139bf446eca2891d93eb6934d5a4"
 }