feat: enhance content type handling and add post-bump check

This commit updates the Axios and React Query generators to prioritize `application/json` over `application/ld+json` for request and response bodies. Additionally, it introduces a new hook in the release-it configuration to run checks after version bumping, ensuring code quality is maintained.

Author: owinter86

.release-it.json

@@ -14,6 +14,7 @@
 	},
 	"hooks": {
 		"before:init": ["npm run build", "npm run check"],
+		"after:bump": ["npm run check"],
 		"after:release": "echo Successfully released ${name} v${version} to npm."
 	}
 }

package.json

@@ -4,10 +4,7 @@
 	"license": "MIT",
 	"description": "Generate Axios API clients and React Query options from OpenAPI specifications",
 	"exports": "./dist/index.js",
-	"files": [
-		"src",
-		"dist"
-	],
+	"files": ["src", "dist"],
 	"author": {
 		"name": "Oliver Winter",
 		"email": "[email protected]"

src/generator/clientGenerator.ts

@@ -51,8 +51,8 @@ function generateAxiosMethod(operation: OperationInfo, spec: OpenAPIV3.Document)
 		const responseObj = response as OpenAPIV3.ResponseObject;
 		const desc = "description" in responseObj ? responseObj.description : "";
 		const contentType =
-			responseObj.content?.["application/json"]?.schema ??
 			responseObj.content?.["application/ld+json"]?.schema ??
+			responseObj.content?.["application/json"]?.schema ??
 			responseObj.content?.["application/octet-stream"]?.schema;
 
 		const typeName = pascalCase(`${operationId}Response${code}`);
@@ -80,8 +80,8 @@ function generateAxiosMethod(operation: OperationInfo, spec: OpenAPIV3.Document)
 
 	const content =
 		requestBody && "content" in requestBody
-			? (requestBody.content?.["application/json"]?.schema ??
-				requestBody.content?.["application/ld+json"]?.schema ??
+			? (requestBody.content?.["application/ld+json"]?.schema ??
+				requestBody.content?.["application/json"]?.schema ??
 				requestBody.content?.["application/octet-stream"]?.schema)
 			: undefined;
 
@@ -138,7 +138,7 @@ function generateAxiosMethod(operation: OperationInfo, spec: OpenAPIV3.Document)
 		requestBodySchema?.properties
 			? `const bodyData = {
 				${Object.entries(requestBodySchema.properties)
-					.map(([key]) => `${key}: data.${key}`)
+					.map(([key]) => `["${key}"]: data["${key}"]`)
 					.join(",\n				")}
 			};`
 			: "",

src/generator/reactQueryGenerator.ts

@@ -22,8 +22,8 @@ function generateQueryOptions(operation: OperationInfo, spec: OpenAPIV3.Document
 
 	const content =
 		requestBody && "content" in requestBody
-			? (requestBody.content?.["application/json"]?.schema ??
-				requestBody.content?.["application/ld+json"]?.schema ??
+			? (requestBody.content?.["application/ld+json"]?.schema ??
+				requestBody.content?.["application/json"]?.schema ??
 				requestBody.content?.["application/octet-stream"]?.schema)
 			: undefined;
 	// Get required parameter names from both parameters and request body

src/generator/schemaGenerator.ts

@@ -19,26 +19,28 @@ function getTypeFromSchema(
 		const refType = schema.$ref.split("/").pop();
 		return sanitizeTypeName(refType as string);
 	}
+	const nullable = schema.nullable ? " | null" : "";
 
 	// Handle enum types properly
 	if (schema.enum) {
-		return schema.enum.map((e) => (typeof e === "string" ? `'${e}'` : e)).join(" | ");
+		return schema.enum.map((e) => (typeof e === "string" ? `'${e}'` : e)).join(" | ") + nullable;
 	}
 
 	switch (schema.type) {
 		case "string":
 			if ("format" in schema && schema.format === "binary") {
-				return "string | { name?: string; type?: string; uri: string }";
+				return `string | { name?: string; type?: string; uri: string }${nullable}`;
 			}
-			return "string";
+
+			return `string${nullable}`;
 		case "number":
 		case "integer":
-			return "number";
+			return `number${nullable}`;
 		case "boolean":
-			return "boolean";
+			return `boolean${nullable}`;
 		case "array": {
 			const itemType = getTypeFromSchema(schema.items, context);
-			return `Array<${itemType}>`;
+			return `Array<${itemType}>${nullable}`;
 		}
 		case "object":
 			if (schema.properties) {
@@ -50,18 +52,18 @@ function getTypeFromSchema(
 						return `  ${safeName}${isRequired ? "" : "?"}: ${propertyType};`;
 					})
 					.join("\n");
-				return `{\n${properties}\n}`;
+				return `{${properties}\n}${nullable}`;
 			}
 			if (schema.additionalProperties) {
 				const valueType =
 					typeof schema.additionalProperties === "boolean"
 						? "any"
 						: getTypeFromSchema(schema.additionalProperties, context);
-				return `Record<string, ${valueType}>`;
+				return `Record<string, ${valueType}>${nullable}`;
 			}
-			return "Record<string, any>";
+			return `Record<string, any>${nullable}`;
 		default:
-			return "any";
+			return `any${nullable}`;
 	}
 }
 
@@ -114,9 +116,9 @@ export function generateTypeDefinitions(spec: OpenAPIV3.Document): string {
 				if (operationObject.requestBody) {
 					const content = (operationObject.requestBody as OpenAPIV3.RequestBodyObject).content;
 					const jsonContent =
+						content["application/ld+json"] ??
 						content["application/json"] ??
 						content["multipart/form-data"] ??
-						content["application/ld+json"] ??
 						content["application/octet-stream"];
 					if (jsonContent?.schema) {
 						const typeName = `${operationId}Request`;
@@ -129,8 +131,8 @@ export function generateTypeDefinitions(spec: OpenAPIV3.Document): string {
 					for (const [code, response] of Object.entries(operationObject.responses)) {
 						const responseObj = response as OpenAPIV3.ResponseObject;
 						const content =
-							responseObj.content?.["application/json"] ??
 							responseObj.content?.["application/ld+json"] ??
+							responseObj.content?.["application/json"] ??
 							responseObj.content?.["application/octet-stream"];
 						if (content?.schema) {
 							const typeName = `${operationId}Response${code}`;