argsbarg 0.1.0 → 1.0.1

package/src/validate.ts

@@ -28,7 +28,7 @@ export function cliValidateRoot(root: CliCommand): void {
   }
 
   // Check for reserved command names at root
-  for (const child of root.children ?? []) {
+  for (const child of root.commands ?? []) {
     if (reservedCommandNames.includes(child.key)) {
       throw new CliSchemaValidationError(`Reserved command name: ${child.key}`);
     }
@@ -38,6 +38,7 @@ export function cliValidateRoot(root: CliCommand): void {
   walkCommand(root, true);
 }
 
+/** Recursively validates a command node: handlers vs children, options, and positionals. */
 function walkCommand(cmd: CliCommand, isRoot: boolean = false): void {
   // Fallback only on root
   if (!isRoot && cmd.fallbackCommand !== undefined) {
@@ -55,7 +56,7 @@ function walkCommand(cmd: CliCommand, isRoot: boolean = false): void {
     );
   }
 
-  if ((cmd.children ?? []).length > 0) {
+  if ((cmd.commands ?? []).length > 0) {
     if (cmd.handler !== undefined) {
       throw new CliSchemaValidationError(`Routing command must not set handler: ${cmd.key}`);
     }
@@ -67,7 +68,7 @@ function walkCommand(cmd: CliCommand, isRoot: boolean = false): void {
 
   // Check for duplicate child names
   const seenNames = new Set<string>();
-  for (const child of cmd.children ?? []) {
+  for (const child of cmd.commands ?? []) {
     if (seenNames.has(child.key)) {
       throw new CliSchemaValidationError(`Duplicate command name: ${child.key}`);
     }
@@ -93,17 +94,18 @@ function walkCommand(cmd: CliCommand, isRoot: boolean = false): void {
   }
 
   // Validate positionals
-  const positionals = (cmd.positionals ?? []).filter((p) => p.positional);
+  const positionals = cmd.positionals ?? [];
   for (const p of positionals) {
-    if (p.argMin < 0) {
+    if (p.argMin !== undefined && p.argMin < 0) {
       throw new CliSchemaValidationError(`argMin must be >= 0 for positional ${cmd.key}/${p.name}`);
     }
-    if (p.argMax < 0) {
+    if (p.argMax !== undefined && p.argMax < 0) {
       throw new CliSchemaValidationError(
         `argMax must be >= 0 (use 0 for unlimited) for positional ${cmd.key}/${p.name}`,
       );
     }
-    if (p.argMax > 0 && p.argMin > p.argMax) {
+    const { argMin = 1, argMax = 1 } = p;
+    if (argMax > 0 && argMin > argMax) {
       throw new CliSchemaValidationError(
         `argMin must not exceed argMax for positional ${cmd.key}/${p.name}`,
       );
@@ -113,7 +115,8 @@ function walkCommand(cmd: CliCommand, isRoot: boolean = false): void {
   // Check positional ordering: required before optional
   let sawOptional = false;
   for (const p of positionals) {
-    if (p.argMin === 0) {
+    const { argMin = 1 } = p;
+    if (argMin === 0) {
       sawOptional = true;
     } else if (sawOptional) {
       throw new CliSchemaValidationError(`Required positional after optional in scope ${cmd.key}`);
@@ -122,15 +125,16 @@ function walkCommand(cmd: CliCommand, isRoot: boolean = false): void {
 
   // Check unlimited positional must be last
   for (let idx = 0; idx < positionals.length; idx++) {
-    if (positionals[idx].argMax === 0 && idx + 1 < positionals.length) {
+    const { argMax = 1 } = positionals[idx]!;
+    if (argMax === 0 && idx + 1 < positionals.length) {
       throw new CliSchemaValidationError(
         `Unlimited positional (argMax == 0) must be last in scope ${cmd.key}`,
       );
     }
   }
 
-  // Recurse into children
-  for (const child of cmd.children ?? []) {
+  // Recurse into nested commands
+  for (const child of cmd.commands ?? []) {
     walkCommand(child, false);
   }
 }

package/bun.lock

@@ -1,21 +0,0 @@
-{
-  "lockfileVersion": 1,
-  "configVersion": 1,
-  "workspaces": {
-    "": {
-      "name": "bun-argsbarg",
-      "devDependencies": {
-        "@types/bun": "^1.3.12",
-      },
-    },
-  },
-  "packages": {
-    "@types/bun": ["@types/bun@1.3.12", "", { "dependencies": { "bun-types": "1.3.12" } }, "sha512-DBv81elK+/VSwXHDlnH3Qduw+KxkTIWi7TXkAeh24zpi5l0B2kUg9Ga3tb4nJaPcOFswflgi/yAvMVBPrxMB+A=="],
-
-    "@types/node": ["@types/node@25.6.0", "", { "dependencies": { "undici-types": "~7.19.0" } }, "sha512-+qIYRKdNYJwY3vRCZMdJbPLJAtGjQBudzZzdzwQYkEPQd+PJGixUL5QfvCLDaULoLv+RhT3LDkwEfKaAkgSmNQ=="],
-
-    "bun-types": ["bun-types@1.3.12", "", { "dependencies": { "@types/node": "*" } }, "sha512-HqOLj5PoFajAQciOMRiIZGNoKxDJSr6qigAttOX40vJuSp6DN/CxWp9s3C1Xwm4oH7ybueITwiaOcWXoYVoRkA=="],
-
-    "undici-types": ["undici-types@7.19.2", "", {}, "sha512-qYVnV5OEm2AW8cJMCpdV20CDyaN3g0AjDlOGf1OW4iaDEx8MwdtChUp4zu4H0VP3nDRF/8RKWH+IPp9uW0YGZg=="],
-  }
-}