genoc 0.4.1 → 0.4.2

package/dist/generator/method-generator.js

@@ -1,34 +1,50 @@
 import { sanitizeJsDocText } from '../utils/generator-helpers.js';
 import { getSuccessType, operationEmissions } from '../utils/operation-naming.js';
+/**
+ * A `?:` parameter may only be followed by other optional parameters — a
+ * required parameter after an optional one is a TS1016 compile error
+ * ("A required parameter cannot follow an optional parameter"). Optional
+ * parameters that precede a required one are therefore rendered as
+ * `name: T | undefined` instead of `name?: T`.
+ */
+function renderParams(specs) {
+    return specs
+        .map((spec, index) => {
+        if (!spec.isOptional) {
+            return `${spec.name}: ${spec.type}`;
+        }
+        const hasRequiredAfter = specs.slice(index + 1).some((s) => !s.isOptional);
+        if (hasRequiredAfter) {
+            return `${spec.name}: ${spec.type} | undefined`;
+        }
+        return `${spec.name}?: ${spec.type}`;
+    })
+        .join(', ');
+}
 function buildParameters(op) {
-    const params = [];
     const emissions = operationEmissions(op);
+    const specs = [];
     for (const param of op.pathParams) {
-        params.push(`${param.name}: string`);
+        specs.push({ name: param.name, type: 'string', isOptional: false });
     }
     if (emissions.query !== undefined) {
         const allOptional = op.queryParams.every((p) => !p.required);
-        const hasRequiredAfter = !!op.requestBody?.required || op.headerParams.some((p) => p.required);
-        if (allOptional && hasRequiredAfter) {
-            // All optional query params + required param after: use explicit undefined to avoid "required param cannot follow optional" error
-            params.push(`query: ${emissions.query} | undefined`);
-        }
-        else {
-            // Normal case: use optional notation
-            const optional = allOptional ? '?' : '';
-            params.push(`query${optional}: ${emissions.query}`);
-        }
+        specs.push({ name: 'query', type: emissions.query, isOptional: allOptional });
     }
     if (op.requestBody) {
-        const optional = op.requestBody.required ? '' : '?';
-        params.push(`body${optional}: ${emissions.body}`);
+        // A schema-less body has no named type; the literal keeps the rendered
+        // signature byte-identical to the previous template-literal fallback.
+        specs.push({
+            name: 'body',
+            type: emissions.body ?? 'undefined',
+            isOptional: !op.requestBody.required,
+        });
     }
     if (emissions.headers !== undefined) {
         const allOptional = op.headerParams.every((p) => !p.required);
-        const optional = allOptional ? '?' : '';
-        params.push(`headers${optional}: ${emissions.headers}`);
+        specs.push({ name: 'headers', type: emissions.headers, isOptional: allOptional });
     }
-    return params.join(', ');
+    return renderParams(specs);
 }
 function buildJsDoc(op) {
     const lines = [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "genoc",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "description": "Generate typed HTTP clients from OpenAPI 3.0 / 3.1 specifications",
   "keywords": [
     "client",
@@ -47,7 +47,7 @@
     "zod": "^4.0.0"
   },
   "devDependencies": {
-    "@changesets/cli": "^3.0.3",
+    "@changesets/cli": "^2.31.0",
     "@types/node": "^26.6.2",
     "oxfmt": "^0.70.0",
     "oxlint": "^1.85.0",