just-bash-util 0.1.2 → 0.1.4

package/README.md

@@ -28,9 +28,9 @@ const cli = command("mycli", {
 const serve = cli.command("serve", {
   description: "Start the dev server",
   options: {
-    port: o.number().default(3000).short("p").describe("Port to listen on"),
+    port: o.number().default(3000).alias("p").describe("Port to listen on"),
     host: o.string().describe("Host to bind to"),
-    open: f().short("o").describe("Open browser"),
+    open: f().alias("o").describe("Open browser"),
   },
   args: [a.string().name("entry").describe("Entry file")],
   handler: (args, ctx) => {
@@ -67,6 +67,7 @@ await serve.invoke({ port: 8080, entry: "app.ts" }, ctx);
 - `--` passthrough separator
 - Environment variable fallbacks for options
 - Levenshtein-based "did you mean?" suggestions for typos
+- Automatic error handling — thrown errors in handlers are caught and returned as clean `ExecResult` with `exitCode: 1`
 
 ### `just-bash-util/config` — Config file discovery
 
@@ -75,7 +76,7 @@ Cosmiconfig-style config search that walks up the directory tree, trying convent
 ```ts
 import { searchConfig } from "just-bash-util/config";
 
-// Walks up from cwd trying: package.json#myapp, .myapprc, .myapprc.json, myapp.config.json
+// Walks up from cwd trying: .myapprc, .myapprc.json, myapp.config.json
 const result = await searchConfig(ctx, { name: "myapp" });
 if (result) {
   result.config; // parsed config object
@@ -83,11 +84,14 @@ if (result) {
   result.isEmpty; // true if config is null/undefined/empty object
 }
 
-// Customize search places, starting directory, or add custom loaders
+// Find nearest package.json and return its full contents
+const pkg = await searchConfig(ctx, { name: "package", searchPlaces: ["package.json"] });
+
+// Extract a tool-specific property from package.json
 const result2 = await searchConfig(ctx, {
   name: "myapp",
-  from: "/specific/start/dir",
-  searchPlaces: [".myapprc.json", "myapp.config.json"],
+  searchPlaces: ["package.json", ".myapprc", ".myapprc.json"],
+  packageJsonProp: "myapp",
 });
 ```
 
@@ -126,12 +130,16 @@ import {
   relative,
   parse,
   normalize,
+  parsePackageSpecifier,
 } from "just-bash-util/path";
 
 join("src", "utils", "index.ts"); // "src/utils/index.ts"
 dirname("/project/src/index.ts"); // "/project/src"
 basename("src/index.ts", ".ts"); // "index"
 relative("/a/b/c", "/a/d"); // "../../d"
+
+parsePackageSpecifier("@vue/shared/dist"); // { name: "@vue/shared", subpath: "./dist" }
+parsePackageSpecifier("lodash/merge"); // { name: "lodash", subpath: "./merge" }
 ```
 
 ## Peer dependencies

package/dist/{chunk-OLDCCFR6.js → chunk-35QZZQ4A.js}

@@ -9,9 +9,9 @@ var OptionBuilder = class _OptionBuilder {
   describe(text) {
     return new _OptionBuilder({ ...this._def, description: text });
   }
-  /** Set a short alias (single character) */
-  short(alias) {
-    return new _OptionBuilder({ ...this._def, short: alias });
+  /** Set a short alias (single character, e.g. "p" for -p) */
+  alias(short) {
+    return new _OptionBuilder({ ...this._def, short });
   }
   /** Set an environment variable fallback */
   env(name) {
@@ -56,9 +56,9 @@ var FlagBuilder = class _FlagBuilder {
   describe(text) {
     return new _FlagBuilder({ ...this._def, description: text });
   }
-  /** Set a short alias (single character) */
-  short(alias) {
-    return new _FlagBuilder({ ...this._def, short: alias });
+  /** Set a short alias (single character, e.g. "v" for -v) */
+  alias(short) {
+    return new _FlagBuilder({ ...this._def, short });
   }
   /** Set a default value */
   default(value) {
@@ -91,7 +91,9 @@ var ArgBuilder = class _ArgBuilder {
       required: false
     });
   }
-  /** Mark as variadic — collects all remaining positionals into an array */
+  /** Mark as variadic — collects all remaining positionals into an array.
+   *  If the arg is already optional, element-level undefined is stripped
+   *  (the optionality means "zero or more", not "elements can be undefined"). */
   variadic() {
     return new _ArgBuilder({
       ...this._def,
@@ -307,6 +309,8 @@ function parseArgs(options, argDefs, tokens, env) {
         errors.push({ type: "missing_required", name: argName, kind: "arg" });
       } else if (argDef.default !== void 0) {
         result[argName] = argDef.default;
+      } else {
+        result[argName] = [];
       }
       posIdx = positionals.length;
     } else {
@@ -527,12 +531,8 @@ var Command = class _Command {
   handler;
   children = /* @__PURE__ */ new Map();
   parent;
-  /** @internal — accumulated builder types for generic inference */
-  _accOpts;
-  /** @internal — args builder types for generic inference */
-  _accArgs;
   /** @internal */
-  constructor(name, description, options, args, examples, omitInherited, handler, accOpts, accArgs) {
+  constructor(name, description, options, args, examples, omitInherited, handler) {
     this.name = name;
     this.description = description;
     this.options = options;
@@ -540,8 +540,6 @@ var Command = class _Command {
     this.examples = examples;
     this.omitInherited = omitInherited;
     this.handler = handler;
-    this._accOpts = accOpts;
-    this._accArgs = accArgs;
   }
   // --------------------------------------------------------------------------
   // Tree building
@@ -549,9 +547,6 @@ var Command = class _Command {
   /** Add a subcommand. Returns the child command for further nesting. */
   command(name, config) {
     const omitSet = new Set(config.omitInherited ?? []);
-    const parentAcc = { ...this._accOpts };
-    for (const key of omitSet) delete parentAcc[key];
-    const accOpts = { ...parentAcc, ...config.options ?? {} };
     const child = new _Command(
       name,
       config.description,
@@ -559,9 +554,7 @@ var Command = class _Command {
       resolveArgsInput(config.args),
       config.examples ?? [],
       omitSet,
-      config.handler,
-      accOpts,
-      config.args ?? []
+      config.handler
     );
     child.parent = this;
     this.children.set(name, child);
@@ -720,7 +713,12 @@ var Command = class _Command {
         }
       }
     }
-    return this.handler(resolved, ctx, { passthrough: [] });
+    try {
+      return await this.handler(resolved, ctx, { passthrough: [] });
+    } catch (err) {
+      const message = err instanceof Error ? err.message : String(err);
+      return { stdout: "", stderr: message, exitCode: 1 };
+    }
   }
   // --------------------------------------------------------------------------
   // Execution
@@ -746,7 +744,12 @@ var Command = class _Command {
       if (!parsed.ok) {
         return { stdout: "", stderr: formatErrors(parsed.errors), exitCode: 1 };
       }
-      return this.handler(parsed.args, ctx, { passthrough: parsed.passthrough });
+      try {
+        return await this.handler(parsed.args, ctx, { passthrough: parsed.passthrough });
+      } catch (err) {
+        const message = err instanceof Error ? err.message : String(err);
+        return { stdout: "", stderr: message, exitCode: 1 };
+      }
     }
     if (firstToken && !firstToken.startsWith("-")) {
       const suggestions = findSuggestions(firstToken, [...this.children.keys()]);
@@ -771,9 +774,7 @@ function command(name, config) {
     resolveArgsInput(config.args),
     config.examples ?? [],
     /* @__PURE__ */ new Set(),
-    config.handler,
-    config.options ?? {},
-    config.args ?? []
+    config.handler
   );
 }
 function hasHelpFlag(tokens) {

package/dist/{chunk-DLIJNNDI.js → chunk-FUVD54LW.js}

@@ -3,7 +3,7 @@ import {
   dirname,
   extname,
   join
-} from "./chunk-DAO5RF73.js";
+} from "./chunk-HJ3ZU6QO.js";
 
 // src/config/find-up.ts
 async function findUp(ctx, name, options) {
@@ -88,7 +88,6 @@ function deepMerge(base, override) {
 var defaultLoader = (content) => parseJsonc(content);
 function defaultSearchPlaces(name) {
   return [
-    "package.json",
     `.${name}rc`,
     `.${name}rc.json`,
     `${name}.config.json`
@@ -133,7 +132,7 @@ async function collectAll(ctx, options) {
     from = ctx.cwd,
     searchPlaces = defaultSearchPlaces(name),
     loaders: customLoaders,
-    packageJsonProp = name,
+    packageJsonProp = false,
     stopAt = "/",
     stopWhen
   } = options;
@@ -184,7 +183,7 @@ async function searchConfig(ctx, options) {
     from = ctx.cwd,
     searchPlaces = defaultSearchPlaces(name),
     loaders: customLoaders,
-    packageJsonProp = name,
+    packageJsonProp = false,
     stopAt = "/"
   } = options;
   let dir = from;

package/dist/{chunk-DAO5RF73.js → chunk-HJ3ZU6QO.js}

@@ -115,6 +115,30 @@ function format(pathObject) {
   }
   return `${root}${resolvedBase}`;
 }
+function parsePackageSpecifier(specifier) {
+  if (specifier.startsWith("@")) {
+    const firstSlash2 = specifier.indexOf("/");
+    if (firstSlash2 === -1) {
+      return { name: specifier, subpath: "." };
+    }
+    const secondSlash = specifier.indexOf("/", firstSlash2 + 1);
+    if (secondSlash === -1) {
+      return { name: specifier, subpath: "." };
+    }
+    return {
+      name: specifier.slice(0, secondSlash),
+      subpath: `.${specifier.slice(secondSlash)}`
+    };
+  }
+  const firstSlash = specifier.indexOf("/");
+  if (firstSlash === -1) {
+    return { name: specifier, subpath: "." };
+  }
+  return {
+    name: specifier.slice(0, firstSlash),
+    subpath: `.${specifier.slice(firstSlash)}`
+  };
+}
 function relative(from, to) {
   const fromNorm = normalize(from);
   const toNorm = normalize(to);
@@ -149,5 +173,6 @@ export {
   extname,
   parse,
   format,
+  parsePackageSpecifier,
   relative
 };

package/dist/command/index.d.ts

@@ -74,8 +74,8 @@ declare class OptionBuilder<TOut, THasDefault extends boolean = false> {
     constructor(def: OptionDef<TOut>);
     /** Add a description */
     describe(text: string): OptionBuilder<TOut, THasDefault>;
-    /** Set a short alias (single character) */
-    short(alias: string): OptionBuilder<TOut, THasDefault>;
+    /** Set a short alias (single character, e.g. "p" for -p) */
+    alias(short: string): OptionBuilder<TOut, THasDefault>;
     /** Set an environment variable fallback */
     env(name: string): OptionBuilder<TOut, THasDefault>;
     /** Mark as required — removes undefined from TOut */
@@ -94,8 +94,8 @@ declare class FlagBuilder {
     constructor(def?: FlagDef);
     /** Add a description */
     describe(text: string): FlagBuilder;
-    /** Set a short alias (single character) */
-    short(alias: string): FlagBuilder;
+    /** Set a short alias (single character, e.g. "v" for -v) */
+    alias(short: string): FlagBuilder;
     /** Set a default value */
     default(value: boolean): FlagBuilder;
 }
@@ -110,8 +110,10 @@ declare class ArgBuilder<TOut, TName extends string = never, THasDefault extends
     describe(text: string): ArgBuilder<TOut, TName, THasDefault>;
     /** Mark as optional — adds undefined to TOut */
     optional(): ArgBuilder<TOut | undefined, TName, THasDefault>;
-    /** Mark as variadic — collects all remaining positionals into an array */
-    variadic(): ArgBuilder<TOut[], TName, THasDefault>;
+    /** Mark as variadic — collects all remaining positionals into an array.
+     *  If the arg is already optional, element-level undefined is stripped
+     *  (the optionality means "zero or more", not "elements can be undefined"). */
+    variadic(): ArgBuilder<NonNullable<TOut>[], TName, THasDefault>;
     /** Set a default value (also makes the arg optional at parse time) */
     default(value: TOut): ArgBuilder<TOut, TName, true>;
 }
@@ -161,7 +163,7 @@ type InferInvokeArgs<T extends ArgsInput> = {
 } & {
     [I in keyof T & `${number}` as T[I] extends ArgBuilder<infer _V, infer N extends string, infer D> ? [D] extends [true] ? N : T[I] extends ArgBuilder<infer V, any, any> ? undefined extends V ? N : never : never : never]?: T[I] extends ArgBuilder<infer V, any, any> ? V : never;
 };
-declare class Command<TAccOpts extends OptionsInput = {}, TAccArgs extends ArgsInput = []> {
+declare class Command<THandlerArgs extends object = {}, TInvokeArgs extends object = {}> {
     readonly name: string;
     readonly description: string;
     readonly options: OptionsSchema;
@@ -171,12 +173,12 @@ declare class Command<TAccOpts extends OptionsInput = {}, TAccArgs extends ArgsI
     readonly handler?: Handler<any>;
     readonly children: Map<string, Command<any, any>>;
     parent?: Command<any, any>;
-    /** @internal — accumulated builder types for generic inference */
-    readonly _accOpts: TAccOpts;
-    /** @internal — args builder types for generic inference */
-    readonly _accArgs: TAccArgs;
+    /** @internal — phantom type carrying the resolved handler args */
+    readonly _handlerArgs: THandlerArgs;
+    /** @internal — phantom type carrying the resolved invoke args */
+    readonly _invokeArgs: TInvokeArgs;
     /** @internal */
-    constructor(name: string, description: string, options: OptionsSchema, args: ArgsSchema, examples: readonly string[], omitInherited: ReadonlySet<string>, handler: Handler<any> | undefined, accOpts: TAccOpts, accArgs: TAccArgs);
+    constructor(name: string, description: string, options: OptionsSchema, args: ArgsSchema, examples: readonly string[], omitInherited: ReadonlySet<string>, handler: Handler<any> | undefined);
     /** Add a subcommand. Returns the child command for further nesting. */
     command<TOpts extends OptionsInput = {}, const TArgs extends ArgsInput = [], const TOmit extends string[] = []>(name: string, config: {
         readonly description: string;
@@ -184,8 +186,8 @@ declare class Command<TAccOpts extends OptionsInput = {}, TAccArgs extends ArgsI
         readonly args?: TArgs;
         readonly examples?: readonly string[];
         readonly omitInherited?: TOmit;
-        readonly handler?: Handler<Prettify<Omit<InferOptionsFromInput<TAccOpts>, TOmit[number]> & InferOptionsFromInput<TOpts> & InferArgsFromInput<TArgs>>>;
-    }): Command<Omit<TAccOpts, TOmit[number]> & TOpts, TArgs>;
+        readonly handler?: Handler<Prettify<Omit<THandlerArgs, TOmit[number]> & InferOptionsFromInput<TOpts> & InferArgsFromInput<TArgs>>>;
+    }): Command<Prettify<Omit<THandlerArgs, TOmit[number]> & InferOptionsFromInput<TOpts> & InferArgsFromInput<TArgs>>, Prettify<Omit<TInvokeArgs, TOmit[number]> & InferInvokeOptions<TOpts> & InferInvokeArgs<TArgs>>>;
     /** Full path from root (e.g. "mycli db migrate") */
     get fullPath(): string;
     /**
@@ -221,7 +223,7 @@ declare class Command<TAccOpts extends OptionsInput = {}, TAccArgs extends ArgsI
      * await cli.execute(["serve", ...tokens], ctx);
      * ```
      */
-    toTokens(args: Partial<Prettify<InferOptionsFromInput<TAccOpts> & InferArgsFromInput<TAccArgs>>>): string[];
+    toTokens(args: Partial<THandlerArgs>): string[];
     /**
      * Call this command's handler directly with typed args.
      *
@@ -234,7 +236,7 @@ declare class Command<TAccOpts extends OptionsInput = {}, TAccArgs extends ArgsI
      * const result = await serve.invoke({ port: 8080, entry: "app.ts" }, ctx);
      * ```
      */
-    invoke(args: Prettify<InferInvokeOptions<TAccOpts> & InferInvokeArgs<TAccArgs>>, ctx: CommandContext): Promise<ExecResult>;
+    invoke(args: TInvokeArgs, ctx: CommandContext): Promise<ExecResult>;
     /**
      * Execute this command tree with the given tokens.
      *
@@ -251,7 +253,7 @@ declare function command<TOpts extends OptionsInput = {}, const TArgs extends Ar
     readonly args?: TArgs;
     readonly examples?: readonly string[];
     readonly handler?: Handler<Prettify<InferOptionsFromInput<TOpts> & InferArgsFromInput<TArgs>>>;
-}): Command<TOpts, TArgs>;
+}): Command<Prettify<InferOptionsFromInput<TOpts> & InferArgsFromInput<TArgs>>, Prettify<InferInvokeOptions<TOpts> & InferInvokeArgs<TArgs>>>;
 /**
  * Infer the handler args type from a Command instance.
  *
@@ -270,7 +272,7 @@ declare function command<TOpts extends OptionsInput = {}, const TArgs extends Ar
  * //   ^? { port: number; entry: string }
  * ```
  */
-type Infer<T extends Command<any, any>> = Prettify<InferOptionsFromInput<T["_accOpts"]> & InferArgsFromInput<T["_accArgs"]>>;
+type Infer<T extends Command<any, any>> = T["_handlerArgs"];
 
 type ParseArgsResult = {
     ok: true;

package/dist/command/index.js

@@ -8,7 +8,7 @@ import {
   generateHelp,
   o,
   parseArgs
-} from "../chunk-OLDCCFR6.js";
+} from "../chunk-35QZZQ4A.js";
 export {
   Command,
   a,

package/dist/config/index.d.ts

@@ -40,7 +40,6 @@ interface SearchConfigOptions {
      * Filenames to look for at each directory level, tried in order.
      *
      * Defaults:
-     * - `"package.json"` (extracts the `name` property)
      * - `".{name}rc"`
      * - `".{name}rc.json"`
      * - `"{name}.config.json"`
@@ -54,7 +53,7 @@ interface SearchConfigOptions {
     /**
      * Property to extract from `package.json`.
      * Set to `false` to disable property extraction (load full object).
-     * Default: the `name` option.
+     * Default: `false`.
      */
     packageJsonProp?: string | false;
     /** Directory to stop searching at (default: `"/"`) */
@@ -105,6 +104,16 @@ interface LoadConfigOptions {
  * const result = await searchConfig(ctx, { name: "myapp" });
  * if (result) console.log(result.config, result.filepath);
  *
+ * // Find nearest package.json and return its full contents
+ * const pkg = await searchConfig(ctx, { name: "package", searchPlaces: ["package.json"] });
+ *
+ * // Extract a specific property from package.json
+ * const result = await searchConfig(ctx, {
+ *   name: "myapp",
+ *   searchPlaces: ["package.json", ".myapprc", ".myapprc.json"],
+ *   packageJsonProp: "myapp",
+ * });
+ *
  * // Layered / cascading config
  * const merged = await searchConfig(ctx, { name: "myapp", merge: true });
  * ```

package/dist/config/index.js

@@ -2,8 +2,8 @@ import {
   findUp,
   loadConfig,
   searchConfig
-} from "../chunk-DLIJNNDI.js";
-import "../chunk-DAO5RF73.js";
+} from "../chunk-FUVD54LW.js";
+import "../chunk-HJ3ZU6QO.js";
 export {
   findUp,
   loadConfig,

package/dist/index.d.ts

@@ -1,4 +1,4 @@
 export { Command, Handler, Infer, ParseArgsResult, ParseError, a, command, f, formatError, formatErrors, generateHelp, o, parseArgs } from './command/index.js';
 export { ConfigResult, FindUpOptions, LoadConfigOptions, Loader, SearchConfigOptions, findUp, loadConfig, searchConfig } from './config/index.js';
-export { ParsedPath, basename, delimiter, dirname, extname, format, isAbsolute, join, normalize, parse, relative, resolve, sep } from './path/index.js';
+export { PackageSpecifier, ParsedPath, basename, delimiter, dirname, extname, format, isAbsolute, join, normalize, parse, parsePackageSpecifier, relative, resolve, sep } from './path/index.js';
 import 'just-bash';

package/dist/index.js

@@ -8,12 +8,12 @@ import {
   generateHelp,
   o,
   parseArgs
-} from "./chunk-OLDCCFR6.js";
+} from "./chunk-35QZZQ4A.js";
 import {
   findUp,
   loadConfig,
   searchConfig
-} from "./chunk-DLIJNNDI.js";
+} from "./chunk-FUVD54LW.js";
 import {
   basename,
   delimiter,
@@ -24,10 +24,11 @@ import {
   join,
   normalize,
   parse,
+  parsePackageSpecifier,
   relative,
   resolve,
   sep
-} from "./chunk-DAO5RF73.js";
+} from "./chunk-HJ3ZU6QO.js";
 export {
   Command,
   a,
@@ -49,6 +50,7 @@ export {
   o,
   parse,
   parseArgs,
+  parsePackageSpecifier,
   relative,
   resolve,
   searchConfig,

package/dist/path/index.d.ts

@@ -33,7 +33,26 @@ interface ParsedPath {
 declare function parse(path: string): ParsedPath;
 /** Build a path string from components (inverse of `parse`) */
 declare function format(pathObject: Partial<ParsedPath>): string;
+interface PackageSpecifier {
+    /** Package name (e.g. `"pkg"` or `"@scope/pkg"`) */
+    name: string;
+    /** Subpath within the package (e.g. `"./sub/path"`), or `"."` for the root */
+    subpath: string;
+}
+/**
+ * Parse a bare package specifier into its package name and subpath.
+ *
+ * Handles both scoped (`@scope/pkg/sub`) and unscoped (`pkg/sub`) specifiers.
+ * The subpath is normalized to start with `"./"` (or `"."` for the root).
+ *
+ * ```ts
+ * parsePackageSpecifier("lodash/merge")     // { name: "lodash", subpath: "./merge" }
+ * parsePackageSpecifier("@vue/shared/dist") // { name: "@vue/shared", subpath: "./dist" }
+ * parsePackageSpecifier("react")            // { name: "react", subpath: "." }
+ * ```
+ */
+declare function parsePackageSpecifier(specifier: string): PackageSpecifier;
 /** Compute the relative path from `from` to `to` */
 declare function relative(from: string, to: string): string;
 
-export { type ParsedPath, basename, delimiter, dirname, extname, format, isAbsolute, join, normalize, parse, relative, resolve, sep };
+export { type PackageSpecifier, type ParsedPath, basename, delimiter, dirname, extname, format, isAbsolute, join, normalize, parse, parsePackageSpecifier, relative, resolve, sep };

package/dist/path/index.js

@@ -8,10 +8,11 @@ import {
   join,
   normalize,
   parse,
+  parsePackageSpecifier,
   relative,
   resolve,
   sep
-} from "../chunk-DAO5RF73.js";
+} from "../chunk-HJ3ZU6QO.js";
 export {
   basename,
   delimiter,
@@ -22,6 +23,7 @@ export {
   join,
   normalize,
   parse,
+  parsePackageSpecifier,
   relative,
   resolve,
   sep

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "just-bash-util",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "CLI command framework, config file discovery, and path utilities for just-bash",
   "type": "module",
   "license": "MIT",