@kubb/plugin-ts 5.0.0-beta.98 → 5.0.0-beta.99

package/dist/index.d.ts

@@ -845,10 +845,6 @@ type CreateOperationParamsOptions = {
    * - `'inlineSpread'` emits a single rest parameter `...pathParams: PathParams`
    */
   pathParamsType: 'object' | 'inline' | 'inlineSpread';
-  /**
-   * Converts parameter names to camelCase before output.
-   */
-  paramsCasing?: 'camelcase';
   /**
    * Resolver for parameter and request body type names.
    * Pass `ResolverTs` from `@kubb/plugin-ts` directly.

package/dist/index.js

@@ -912,39 +912,19 @@ function Enum({ node, enum: enumOptions, resolver }) {
 }
 //#endregion
 //#region ../../internals/shared/src/params.ts
-const caseParamsCache = /* @__PURE__ */ new WeakMap();
 /**
-* Applies camelCase to parameter names and returns a new array without mutating the input.
+* Drops parameters that share the same name, keeping the first.
 *
-* Run it before handing parameters to schema builders so output property keys get the right casing
-* while `OperationNode.parameters` stays intact for other consumers. When `casing` is unset, the
-* original array is returned unchanged. Results are cached per input array.
+* A malformed spec can declare the same parameter name twice within one `in` location. Both would
+* resolve to the same output property, so emitting both would yield an object type with a duplicate
+* member, which TypeScript rejects. This is a defensive guard against that case, not a casing guard:
+* parameter names flow through unchanged, so no two distinct names ever collide here anymore.
 */
-function caseParams(params, casing) {
-	if (!casing) return params;
-	const cached = caseParamsCache.get(params);
-	if (cached) return cached;
-	const result = params.map((param) => ({
-		...param,
-		name: camelCase(param.name)
-	}));
-	caseParamsCache.set(params, result);
-	return result;
-}
-/**
-* Drops parameters that collapse to the same property identity once camelCased, keeping the first.
-*
-* Some specs declare the same parameter twice under different casings (for example AWS S3 lists both
-* `max-uploads` and `MaxUploads`). Both resolve to one output property, so emitting both would yield
-* an object type with a duplicate member, which TypeScript rejects. De-duplicate by the camelCased
-* identity so the resulting group is collision-free regardless of the names each caller carries.
-*/
-function dedupeByCasedName(params) {
+function dedupeParams(params) {
 	const seen = /* @__PURE__ */ new Set();
 	return params.filter((param) => {
-		const key = camelCase(param.name);
-		if (seen.has(key)) return false;
-		seen.add(key);
+		if (seen.has(param.name)) return false;
+		seen.add(param.name);
 		return true;
 	});
 }
@@ -998,13 +978,15 @@ function resolveContentTypeVariants(entries, baseName) {
 		};
 	});
 }
-function getOperationParameters(node, options = {}) {
-	const params = caseParams(node.parameters, options.paramsCasing === "original" ? void 0 : "camelcase");
+function getOperationParameters(node) {
 	return {
-		path: dedupeByCasedName(params.filter((param) => param.in === "path")),
-		query: dedupeByCasedName(params.filter((param) => param.in === "query")),
-		header: dedupeByCasedName(params.filter((param) => param.in === "header")),
-		cookie: dedupeByCasedName(params.filter((param) => param.in === "cookie"))
+		path: dedupeParams(node.parameters.filter((param) => param.in === "path").map((param) => isValidVarName(param.name) ? param : {
+			...param,
+			name: camelCase(param.name)
+		})),
+		query: dedupeParams(node.parameters.filter((param) => param.in === "query")),
+		header: dedupeParams(node.parameters.filter((param) => param.in === "header")),
+		cookie: dedupeParams(node.parameters.filter((param) => param.in === "cookie"))
 	};
 }
 //#endregion
@@ -1128,6 +1110,35 @@ function createGroupConfig(group) {
 	};
 }
 //#endregion
+//#region ../../internals/shared/src/schemaTraversal.ts
+/**
+* Maps each property of an object schema to its transformed output. Pairs every result with the
+* original property so the printer keeps full control over modifiers, getters, and key syntax.
+*
+* @example
+* ```ts
+* const entries = mapSchemaProperties(node, (schema) => this.transform(schema))
+* // entries: [{ name: 'id', property, output: 'z.number()' }, ...]
+* ```
+*/
+function mapSchemaProperties(node, transform) {
+	return node.properties.map((property) => ({
+		name: property.name,
+		property,
+		output: transform(property.schema)
+	}));
+}
+/**
+* Maps each item of an array or tuple schema to its transformed output, pairing every result with
+* the original item.
+*/
+function mapSchemaItems(node, transform) {
+	return (node.items ?? []).map((schema) => ({
+		schema,
+		output: transform(schema)
+	}));
+}
+//#endregion
 //#region src/utils.ts
 /**
 * Tells whether a `const` (single-value enum) should render as a bare literal type (`'active'`)
@@ -1192,7 +1203,7 @@ function buildParams({ params }) {
 	});
 }
 function buildOptions(node, { resolver }) {
-	const { path: pathParams, query: queryParams, header: headerParams } = getOperationParameters(node, { paramsCasing: "original" });
+	const { path: pathParams, query: queryParams, header: headerParams } = getOperationParameters(node);
 	const hasBody = Boolean(node.requestBody?.content?.[0]?.schema);
 	const hasRequiredPath = pathParams.some((param) => param.required);
 	const hasRequiredQuery = queryParams.some((param) => param.required);
@@ -1443,7 +1454,7 @@ const printerTs = ast.createPrinter((options) => {
 			},
 			array(node) {
 				return createArrayDeclaration({
-					nodes: ast.mapSchemaItems(node, (item) => this.transform(item)).map(({ output }) => output).filter(isNonNullable),
+					nodes: mapSchemaItems(node, (item) => this.transform(item)).map(({ output }) => output).filter(isNonNullable),
 					arrayType: this.options.arrayType
 				}) ?? null;
 			},
@@ -1453,7 +1464,7 @@ const printerTs = ast.createPrinter((options) => {
 			object(node) {
 				const { transform, options } = this;
 				const addsQuestionToken = OPTIONAL_ADDS_QUESTION_TOKEN.has(options.optionalType);
-				const propertyNodes = ast.mapSchemaProperties(node, (schema) => transform(schema)).map(({ name, property, output }) => {
+				const propertyNodes = mapSchemaProperties(node, (schema) => transform(schema)).map(({ name, property, output }) => {
 					const baseType = output ?? keywordTypeNodes.unknown;
 					const optional = !property.required || !!property.schema.optional || !!property.schema.nullish;
 					const type = buildPropertyType(property.schema, baseType, options.optionalType, optional);
@@ -1609,7 +1620,6 @@ const typeGenerator = defineGenerator({
 	operation(node, ctx) {
 		const { enum: enumOptions, optionalType, arrayType, syntaxType, group, output, printer } = ctx.options;
 		const { config, resolver, root } = ctx;
-		const params = caseParams(node.parameters, "camelcase");
 		const meta = { file: resolver.file({
 			name: node.operationId,
 			extname: ".ts",
@@ -1688,10 +1698,7 @@ const typeGenerator = defineGenerator({
 				name: baseName
 			})] });
 		}
-		const { path: pathParams, query: queryParams, header: headerParams } = getOperationParameters({
-			...node,
-			parameters: params
-		}, { paramsCasing: "original" });
+		const { path: pathParams, query: queryParams, header: headerParams } = getOperationParameters(node);
 		const paramGroupTypes = [
 			pathParams.length > 0 && renderSchemaType({
 				schema: buildParams({ params: pathParams }),
@@ -1738,10 +1745,7 @@ const typeGenerator = defineGenerator({
 			});
 		});
 		const optionsType = renderSchemaType({
-			schema: buildOptions({
-				...node,
-				parameters: params
-			}, { resolver }),
+			schema: buildOptions(node, { resolver }),
 			name: resolver.response.options(node)
 		});
 		const responsesType = renderSchemaType({
@@ -2191,17 +2195,14 @@ function resolveGroupType({ node, params, group, resolver }) {
 * name resolution and `extraParams` for plugin-specific trailing parameters such as an `options` object.
 */
 function createOperationParams(node, options) {
-	const { paramsType, pathParamsType, paramsCasing, resolver, pathParamsDefault, extraParams = [], paramNames, typeWrapper } = options;
+	const { paramsType, pathParamsType, resolver, pathParamsDefault, extraParams = [], paramNames, typeWrapper } = options;
 	const dataName = paramNames?.data ?? "data";
 	const paramsName = paramNames?.params ?? "params";
 	const headersName = paramNames?.headers ?? "headers";
 	const pathName = paramNames?.path ?? "pathParams";
 	const wrapType = (type) => typeWrapper ? typeWrapper(type) : type;
 	const wrapTypeExpression = (type) => typeof type === "string" ? wrapType(type) : type;
-	const casedParams = caseParams(node.parameters, paramsCasing);
-	const pathParams = casedParams.filter((p) => p.in === "path");
-	const queryParams = casedParams.filter((p) => p.in === "query");
-	const headerParams = casedParams.filter((p) => p.in === "header");
+	const { path: pathParams, query: queryParams, header: headerParams } = getOperationParameters(node);
 	const toProperty = (param) => ({
 		name: param.name,
 		type: wrapTypeExpression(resolveParamType({