@cardelli/ambit 0.3.3 → 0.3.4

package/esm/cli/commands/create/index.js

@@ -245,7 +245,7 @@ const create = async (argv) => {
         alias: { y: "yes" },
     };
     const args = parseArgs(argv, opts);
-    checkArgs(args, opts, "ambit create");
+    checkArgs(args, opts, "ambit create", 1);
     if (args.help) {
         console.log(`
 ${bold("ambit create")} - Create Tailscale Subnet Router

package/esm/cli/commands/deploy/index.js

@@ -153,7 +153,7 @@ const deploy = async (argv) => {
         default: { "main-port": "80" },
     };
     const args = parseArgs(argv, opts);
-    checkArgs(args, opts, "ambit deploy");
+    checkArgs(args, opts, "ambit deploy", 1);
     if (args.help) {
         console.log(`
 ${bold("ambit deploy")} - Deploy an App Safely on a Custom Private Network

package/esm/cli/commands/destroy/app.js

@@ -107,7 +107,7 @@ export const destroyApp = async (argv) => {
         alias: { y: "yes" },
     };
     const args = parseArgs(argv, opts);
-    checkArgs(args, opts, "ambit destroy app");
+    checkArgs(args, opts, "ambit destroy app", 1);
     if (args.help) {
         console.log(`
 ${bold("ambit destroy app")} - Destroy a Workload App

package/esm/cli/commands/destroy/index.js

@@ -4,6 +4,7 @@
 import * as dntShim from "../../../_dnt.shims.js";
 import { parseArgs } from "../../../deps/jsr.io/@std/cli/1.0.28/mod.js";
 import { bold } from "../../../lib/cli.js";
+import { checkArgs } from "../../../lib/args.js";
 import { registerCommand } from "../../mod.js";
 import { destroyNetwork } from "./network.js";
 import { destroyApp } from "./app.js";
@@ -46,7 +47,9 @@ const destroy = async (argv) => {
     if (subcommand === "app") {
         return destroyApp(argv.slice(1));
     }
-    const args = parseArgs(argv, { boolean: ["help"] });
+    const opts = { boolean: ["help"] };
+    const args = parseArgs(argv, opts);
+    checkArgs(args, opts, "ambit destroy", 0);
     if (args.help) {
         showDestroyHelp();
         return;

package/esm/cli/commands/destroy/network.js

@@ -225,7 +225,7 @@ export const destroyNetwork = async (argv) => {
         alias: { y: "yes" },
     };
     const args = parseArgs(argv, opts);
-    checkArgs(args, opts, "ambit destroy network");
+    checkArgs(args, opts, "ambit destroy network", 1);
     if (args.help) {
         console.log(`
 ${bold("ambit destroy network")} - Tear Down Router

package/esm/cli/commands/doctor.js

@@ -145,7 +145,7 @@ const doctorNetwork = async (argv) => {
         boolean: ["help", "json"],
     };
     const args = parseArgs(argv, opts);
-    checkArgs(args, opts, "ambit doctor network");
+    checkArgs(args, opts, "ambit doctor network", 1);
     if (args.help) {
         console.log(`
 ${bold("ambit doctor network")} - Check Router Health
@@ -192,7 +192,7 @@ const doctorApp = async (argv) => {
         boolean: ["help", "json"],
     };
     const args = parseArgs(argv, opts);
-    checkArgs(args, opts, "ambit doctor app");
+    checkArgs(args, opts, "ambit doctor app", 1);
     if (args.help) {
         console.log(`
 ${bold("ambit doctor app")} - Check App Health
@@ -286,7 +286,9 @@ const doctor = async (argv) => {
         return doctorNetwork(argv.slice(1));
     if (subcommand === "app")
         return doctorApp(argv.slice(1));
-    const args = parseArgs(argv, { boolean: ["help"] });
+    const opts = { string: ["org"], boolean: ["help", "json"] };
+    const args = parseArgs(argv, opts);
+    checkArgs(args, opts, "ambit doctor", 0);
     if (args.help) {
         showDoctorHelp();
         return;

package/esm/cli/commands/list/apps.js

@@ -14,7 +14,7 @@ import { initSession } from "../../../util/session.js";
 export const listApps = async (argv) => {
     const opts = { string: ["org"], boolean: ["help", "json"] };
     const args = parseArgs(argv, opts);
-    checkArgs(args, opts, "ambit list apps");
+    checkArgs(args, opts, "ambit list apps", 1);
     if (args.help) {
         console.log(`
 ${bold("ambit list apps")} - List Apps on a Network

package/esm/cli/commands/list/index.js

@@ -4,6 +4,7 @@
 import * as dntShim from "../../../_dnt.shims.js";
 import { parseArgs } from "../../../deps/jsr.io/@std/cli/1.0.28/mod.js";
 import { bold } from "../../../lib/cli.js";
+import { checkArgs } from "../../../lib/args.js";
 import { registerCommand } from "../../mod.js";
 import { listNetworks } from "./networks.js";
 import { listApps } from "./apps.js";
@@ -43,7 +44,9 @@ const list = async (argv) => {
         return listNetworks(argv.slice(1));
     if (subcommand === "apps")
         return listApps(argv.slice(1));
-    const args = parseArgs(argv, { boolean: ["help"] });
+    const opts = { boolean: ["help"] };
+    const args = parseArgs(argv, opts);
+    checkArgs(args, opts, "ambit list", 0);
     if (args.help) {
         showListHelp();
         return;

package/esm/cli/commands/list/networks.js

@@ -52,7 +52,7 @@ const stageRender = (out, routers) => {
 export const listNetworks = async (argv) => {
     const opts = { string: ["org"], boolean: ["help", "json"] };
     const args = parseArgs(argv, opts);
-    checkArgs(args, opts, "ambit list networks");
+    checkArgs(args, opts, "ambit list networks", 0);
     if (args.help) {
         console.log(`
 ${bold("ambit list networks")} - List All Networks

package/esm/cli/commands/share.js

@@ -168,7 +168,7 @@ const share = async (argv) => {
         boolean: ["help", "json"],
     };
     const args = parseArgs(argv, opts);
-    checkArgs(args, opts, "ambit share");
+    checkArgs(args, opts, "ambit share", Infinity);
     if (args.help) {
         console.log(`
 ${bold("ambit share")} - Grant Members Access to a Network

package/esm/cli/commands/status/app.js

@@ -100,7 +100,7 @@ export const statusApp = async (argv) => {
         boolean: ["help", "json"],
     };
     const args = parseArgs(argv, opts);
-    checkArgs(args, opts, "ambit status app");
+    checkArgs(args, opts, "ambit status app", 1);
     if (args.help) {
         console.log(`
 ${bold("ambit status app")} - Show App Status

package/esm/cli/commands/status/index.js

@@ -3,6 +3,7 @@
 // =============================================================================
 import { parseArgs } from "../../../deps/jsr.io/@std/cli/1.0.28/mod.js";
 import { bold } from "../../../lib/cli.js";
+import { checkArgs } from "../../../lib/args.js";
 import { registerCommand } from "../../mod.js";
 import { statusNetworks } from "./networks.js";
 import { statusNetwork } from "./network.js";
@@ -50,7 +51,9 @@ const status = async (argv) => {
         return statusNetwork(argv.slice(1));
     if (subcommand === "app")
         return statusApp(argv.slice(1));
-    const args = parseArgs(argv, { boolean: ["help"] });
+    const opts = { string: ["org"], boolean: ["help", "json"] };
+    const args = parseArgs(argv, opts);
+    checkArgs(args, opts, "ambit status", 0);
     if (args.help) {
         showStatusHelp();
         return;

package/esm/cli/commands/status/network.js

@@ -73,7 +73,7 @@ const stageNetworkStatus = async (fly, tailscale, network, org, json) => {
 export const statusNetwork = async (argv) => {
     const opts = { string: ["org"], boolean: ["help", "json"] };
     const args = parseArgs(argv, opts);
-    checkArgs(args, opts, "ambit status network");
+    checkArgs(args, opts, "ambit status network", 1);
     if (args.help) {
         console.log(`
 ${bold("ambit status network")} - Show Detailed Status for a Network

package/esm/cli/commands/status/networks.js

@@ -14,7 +14,7 @@ import { initSession } from "../../../util/session.js";
 export const statusNetworks = async (argv) => {
     const opts = { string: ["org"], boolean: ["help", "json"] };
     const args = parseArgs(argv, opts);
-    checkArgs(args, opts, "ambit status networks");
+    checkArgs(args, opts, "ambit status networks", 0);
     if (args.help) {
         console.log(`
 ${bold("ambit status networks")} - Show Status of All Networks

package/esm/deno.js

@@ -1,6 +1,6 @@
 export default {
     "name": "@cardelli/ambit",
-    "version": "0.3.3",
+    "version": "0.3.4",
     "description": "Deploy apps to the cloud that only you and your AI agents can reach",
     "license": "MIT",
     "tasks": {

package/esm/lib/args.d.ts

@@ -1,11 +1,14 @@
 /**
  * Validates parsed args against the declared options spec.
- * Dies with a Title Case error if unknown flags are found.
- * Pass the same options object you gave to parseArgs.
+ * Dies with a Title Case error if unknown flags or unexpected positional
+ * arguments are found.  Pass the same options object you gave to parseArgs.
+ *
+ * `maxPositional` (default 0) limits how many bare positional args are
+ * allowed.  Set it to the number the command actually expects.
  */
 export declare const checkArgs: (args: Record<string, unknown>, opts: {
     string?: readonly string[] | string[];
     boolean?: readonly string[] | string[];
     alias?: Record<string, string | string[]>;
-}, command: string) => void;
+}, command: string, maxPositional?: number) => void;
 //# sourceMappingURL=args.d.ts.map

package/esm/lib/args.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"args.d.ts","sourceRoot":"","sources":["../../src/lib/args.ts"],"names":[],"mappings":"AAMA;;;;GAIG;AACH,eAAO,MAAM,SAAS,GACpB,MAAM,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,MAAM;IACJ,MAAM,CAAC,EAAE,SAAS,MAAM,EAAE,GAAG,MAAM,EAAE,CAAC;IACtC,OAAO,CAAC,EAAE,SAAS,MAAM,EAAE,GAAG,MAAM,EAAE,CAAC;IACvC,KAAK,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,EAAE,CAAC,CAAC;CAC3C,EACD,SAAS,MAAM,KACd,IAmBF,CAAC"}
+{"version":3,"file":"args.d.ts","sourceRoot":"","sources":["../../src/lib/args.ts"],"names":[],"mappings":"AAMA;;;;;;;GAOG;AACH,eAAO,MAAM,SAAS,GACpB,MAAM,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,MAAM;IACJ,MAAM,CAAC,EAAE,SAAS,MAAM,EAAE,GAAG,MAAM,EAAE,CAAC;IACtC,OAAO,CAAC,EAAE,SAAS,MAAM,EAAE,GAAG,MAAM,EAAE,CAAC;IACvC,KAAK,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,EAAE,CAAC,CAAC;CAC3C,EACD,SAAS,MAAM,EACf,sBAAiB,KAChB,IA4BF,CAAC"}

package/esm/lib/args.js

@@ -4,10 +4,13 @@
 import { createOutput } from "./output.js";
 /**
  * Validates parsed args against the declared options spec.
- * Dies with a Title Case error if unknown flags are found.
- * Pass the same options object you gave to parseArgs.
+ * Dies with a Title Case error if unknown flags or unexpected positional
+ * arguments are found.  Pass the same options object you gave to parseArgs.
+ *
+ * `maxPositional` (default 0) limits how many bare positional args are
+ * allowed.  Set it to the number the command actually expects.
  */
-export const checkArgs = (args, opts, command) => {
+export const checkArgs = (args, opts, command, maxPositional = 0) => {
     const known = new Set(["_"]);
     for (const k of opts.string ?? [])
         known.add(k);
@@ -25,4 +28,10 @@ export const checkArgs = (args, opts, command) => {
         const out = createOutput(!!args.json);
         out.die(`Unknown Flag(s): ${bad.join(", ")}. Run '${command} --help' for Usage.`);
     }
+    const positional = Array.isArray(args._) ? args._ : [];
+    if (positional.length > maxPositional) {
+        const extra = positional.slice(maxPositional).join(", ");
+        const out = createOutput(!!args.json);
+        out.die(`Unexpected Argument(s): ${extra}. Run '${command} --help' for Usage.`);
+    }
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cardelli/ambit",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "description": "Deploy apps to the cloud that only you and your AI agents can reach",
   "license": "MIT",
   "scripts": {},