Fixed option builder ignoring names, fixed positionals vanishing from…

… options, added subcommands

package.json

@@ -2,7 +2,7 @@
 	"name": "@drizzle-team/brocli",
 	"type": "module",
 	"author": "Drizzle Team",
-	"version": "0.3.0",
+	"version": "0.4.0",
 	"description": "Typed CLI command runner",
 	"license": "Apache-2.0",
 	"sideEffects": false,

src/command-core.ts

@@ -247,7 +247,11 @@ const validateOptions = <TOptionConfig extends Record<string, GenericBuilderInte
 	for (const [key, value] of cfgEntries) {
 		const cfg = value._.config;
 
-		if (cfg.type === 'positional') continue;
+		if (cfg.type === 'positional') {
+			entries.push([key, { config: cfg, $output: undefined as any }]);
+
+			continue;
+		}
 
 		const reservedNames = ['--help', '-h', '--version', '-v'];
 
@@ -483,6 +487,12 @@ const parseArg = (
 				return true;
 			}
 
+			if (!hasEq) {
+				data = true;
+				skipNext = false;
+				return true;
+			}
+
 			throw invalidBooleanSyntax(match);
 		} else {
 			const match = names.find((name) => name === namePart);

src/index.ts

@@ -3,9 +3,11 @@ export type {
 	AnyRawCommand,
 	BroCliConfig,
 	Command,
+	CommandCandidate,
 	CommandHandler,
 	GenericCommandHandler,
 	HelpHandler,
+	InnerCommandParseRes,
 	RawCommand,
 	RawCommandWithPositionals,
 	RawCommandWithSubcommands,

src/option-builder.ts

@@ -400,7 +400,7 @@ export function number(): Omit<
 	OptionType | 'enum'
 >;
 export function number<TName extends string>(name?: TName) {
-	return typeof name === 'number' ? new OptionBuilderBase().number(name) : new OptionBuilderBase().number();
+	return typeof name === 'string' ? new OptionBuilderBase().number(name) : new OptionBuilderBase().number();
 }
 
 export function boolean<TName extends string>(
@@ -442,7 +442,7 @@ export function positional(): Omit<
 	OptionType | 'min' | 'max' | 'int' | 'alias'
 >;
 export function positional(displayName?: string) {
-	return typeof displayName === 'number'
+	return typeof displayName === 'string'
 		? new OptionBuilderBase().positional(displayName)
 		: new OptionBuilderBase().positional();
 }