toolcraft-openapi 0.0.60 → 0.0.61

package/dist/generate.d.ts

@@ -126,6 +126,7 @@ export interface GeneratedCommand {
     idempotencyHeader?: string;
     rawResponse?: boolean;
     confirm: boolean;
+    positional: string[];
     params: GeneratedParam[];
     paramsSchemaOptions?: GeneratedObjectSchemaOptions;
     preflightBlocks: GeneratedPreflightBlock[];

package/dist/generate.js

@@ -21,16 +21,6 @@ const NULL_HELPER_SUPPORT = {
     query: { array: false, scalar: false }
 };
 const TRANSPORT_PARAMS = [
-    {
-        paramName: "dryRun",
-        sourceName: "dryRun",
-        location: "transport",
-        description: "Print the HTTP request and exit without sending it.",
-        scope: ["cli", "sdk"],
-        global: true,
-        optional: true,
-        definition: { kind: "boolean" }
-    },
     {
         paramName: "verbose",
         sourceName: "verbose",
@@ -279,6 +269,7 @@ function createGeneratedCommand(document, entry) {
     const noun = createSafeGeneratedNoun(deriveNoun(operation, entry.path, operationId));
     const verb = deriveVerb(entry.method, entry.path, operation, operationId, noun);
     const collected = collectParams(document, entry, operation, operationId, auth, response.mode);
+    const positional = collectPathPositionals(entry.path, collected.params, operationId);
     const methodDefaults = METHOD_DEFAULTS[entry.method];
     const exportName = `${toCamelCase(noun)}${toPascalCase(verb)}Command`;
     const filePath = `${noun}/${verb}.ts`;
@@ -299,6 +290,7 @@ function createGeneratedCommand(document, entry) {
         contentType: collected.contentType,
         multipartBinaryFields: collected.multipartBinaryFields,
         confirm: methodDefaults?.confirm === true,
+        positional,
         params: collected.params,
         paramsSchemaOptions: collected.paramsSchemaOptions,
         preflightBlocks: collected.preflightBlocks,
@@ -307,6 +299,28 @@ function createGeneratedCommand(document, entry) {
         optionalSections: collected.optionalSections
     };
 }
+function collectPathPositionals(pathTemplate, params, operationId) {
+    const pathParamNames = new Set(params.filter((param) => param.location === "path").map((param) => param.sourceName));
+    const positionals = [];
+    for (const segment of pathTemplate.split("/")) {
+        if (!segment.startsWith("{") || !segment.endsWith("}")) {
+            continue;
+        }
+        const sourceName = segment.slice(1, -1);
+        if (!pathParamNames.has(sourceName)) {
+            throw new UserError(`Operation ${JSON.stringify(operationId)} path parameter ${JSON.stringify(sourceName)} is missing from generated params.`);
+        }
+        const param = params.find((candidate) => candidate.location === "path" && candidate.sourceName === sourceName);
+        if (param === undefined) {
+            continue;
+        }
+        if (param.definition.kind === "array" && positionals.length < pathParamNames.size - 1) {
+            throw new UserError(`Operation ${JSON.stringify(operationId)} path parameter ${JSON.stringify(sourceName)} is an array and can only be the final positional argument.`);
+        }
+        positionals.push(param.paramName);
+    }
+    return positionals;
+}
 function resolveOperationBaseUrl(operation, operationId) {
     if (operation.servers === undefined) {
         return undefined;
@@ -1489,6 +1503,9 @@ function createCommandFile(options) {
     if (options.confirm) {
         lines.push("  confirm: true,");
     }
+    if (options.positional.length > 0) {
+        lines.push(`  positional: ${JSON.stringify(options.positional)},`);
+    }
     lines.push("  params: S.Object({");
     lines.push(...renderParamLines(options.params));
     lines.push(options.paramsSchemaOptions?.additionalProperties === false
@@ -1550,7 +1567,6 @@ function createCommandFile(options) {
     }
     lines.push("      tokenSource,");
     lines.push("      fetch,");
-    lines.push("      dryRun: params.dryRun,");
     lines.push("      verbose: params.verbose,");
     if (options.rawResponse === true) {
         lines.push("      rawResponse: params.rawResponse,");
@@ -1806,7 +1822,7 @@ function createIndexFile(commands) {
     if (groups.length === 0) {
         return {
             path: "index.ts",
-            contents: createGeneratedTypeScriptFile(["export {};", ""])
+            contents: createGeneratedTypeScriptFile(["export const generatedCommands = [] as const;", ""])
         };
     }
     const lines = createGeneratedTypeScriptFileLines();
@@ -1826,6 +1842,8 @@ function createIndexFile(commands) {
         lines.push("});");
         lines.push("");
     }
+    lines.push(`export const generatedCommands = [${groups.map(({ noun }) => toCamelCase(noun)).join(", ")}] as const;`);
+    lines.push("");
     return {
         path: "index.ts",
         contents: lines.join("\n")
@@ -1838,11 +1856,11 @@ function createCliFile(theme) {
             "#!/usr/bin/env node",
             ...createGeneratedTypeScriptFileLines(),
             'import { configureTheme, runCLI } from "toolcraft/cli";',
-            'import * as groups from "./index.js";',
+            'import { generatedCommands } from "./index.js";',
             "",
             `configureTheme({ brand: ${JSON.stringify(theme.brand)}, label: ${JSON.stringify(theme.label)} });`,
             "",
-            "await runCLI(Object.values(groups));",
+            "await runCLI([...generatedCommands]);",
             ""
         ].join("\n")
     };

package/dist/http.d.ts

@@ -24,7 +24,6 @@ export interface HttpRequestOptions {
     multipartBinaryFields?: readonly string[];
     responseMode?: "json" | "text" | "binary";
     accept?: string;
-    dryRun?: boolean;
     verbose?: boolean;
     signal?: AbortSignal;
     rawResponse?: boolean;
@@ -41,7 +40,6 @@ export interface HttpRequestOptions {
         key?: string;
         createKey?: () => string;
     };
-    writeStdout?: (chunk: string) => void;
     writeStderr?: (chunk: string) => void;
 }
 export interface BinaryHttpResponse {

package/dist/http.js

@@ -28,13 +28,7 @@ export async function requestJson(options) {
         }
         : {};
     const headers = createHeaders(token, hasBody, { ...options.headers, ...idempotencyHeader }, options.accept, options.bodyMode, options.contentType);
-    const writeStdout = options.writeStdout ?? process.stdout.write.bind(process.stdout);
     const writeStderr = options.writeStderr ?? process.stderr.write.bind(process.stderr);
-    const requestLine = `${method} ${redactSensitiveQueryValues(url)}`;
-    if (options.dryRun) {
-        writeStdout(formatDryRunOutput(requestLine, headers, options.body));
-        return undefined;
-    }
     if (options.verbose) {
         writeStderr(formatTranscriptLines(formatVerboseRequestTranscript(method, url, headers, options.body)));
     }
@@ -453,20 +447,6 @@ function createHttpErrorRequest(method, url, headers, body) {
 function serializeHeaders(headers) {
     return Object.fromEntries(headers.entries());
 }
-function formatDryRunOutput(requestLine, headers, body) {
-    const lines = [
-        requestLine,
-        ...Object.entries(headers).map(([key, value]) => {
-            const headerValue = redactHeaderValue(key, value);
-            return `${key}: ${headerValue}`;
-        }),
-        ""
-    ];
-    if (body !== undefined) {
-        lines.push(JSON.stringify(body));
-    }
-    return `${lines.join("\n")}\n`;
-}
 function formatVerboseRequestTranscript(method, url, headers, body) {
     const lines = [
         `→ ${method} ${redactSensitiveQueryValues(url)}`,

package/dist/runtime.js

@@ -62,6 +62,7 @@ function createRuntimeCommand(command) {
         ...(command.description === undefined ? {} : { description: command.description }),
         scope: RUNTIME_COMMAND_SCOPE,
         ...(command.confirm ? { confirm: true } : {}),
+        ...(command.positional.length > 0 ? { positional: command.positional } : {}),
         params: paramsSchema,
         handler: createRuntimeHandler(command)
     });
@@ -88,7 +89,6 @@ function createRuntimeHandler(command) {
             multipartBinaryFields: command.multipartBinaryFields,
             tokenSource,
             fetch,
-            dryRun: params.dryRun,
             verbose: params.verbose,
             ...preparedRequestShape
         });

package/node_modules/toolcraft-design/dist/components/help-formatter-plain.js

@@ -100,7 +100,7 @@ export function formatColumns(opts) {
             return [`${firstIndent}${row.left}`];
         }
         const rightLines = wrapWords(row.right, rightWidth);
-        if (row.left.length > leftWidth) {
+        if (row.left.length >= leftWidth) {
             return [
                 `${firstIndent}${row.left}`,
                 ...rightLines.map((line) => `${continuationIndent}${line}`)

package/node_modules/toolcraft-design/dist/components/help-formatter.js

@@ -159,7 +159,7 @@ export function formatColumns(opts) {
             return [`${firstIndent}${row.left}`];
         }
         const rightLines = wrapWords(row.right, rightWidth);
-        if (visibleWidth(row.left) > leftWidth) {
+        if (visibleWidth(row.left) >= leftWidth) {
             return [
                 `${firstIndent}${row.left}`,
                 ...rightLines.map((line) => `${continuationIndent}${line}`)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "toolcraft-openapi",
-  "version": "0.0.60",
+  "version": "0.0.61",
   "type": "module",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -28,7 +28,7 @@
     "toolcraft-openapi-generate": "dist/bin/generate.js"
   },
   "dependencies": {
-    "toolcraft": "0.0.60",
+    "toolcraft": "0.0.61",
     "auth-store": "^0.0.1",
     "fast-string-width": "^3.0.2",
     "fast-wrap-ansi": "^0.2.0",