@bagelink/sdk 1.6.2 → 1.6.7

package/dist/index.cjs

@@ -496,6 +496,21 @@ function generateFunctionForOperation(method, path, operation) {
   if (!operation) {
     return "";
   }
+  const methodLower = method.toLowerCase();
+  if (["get", "delete"].includes(methodLower) && operation.requestBody) {
+    const requestBodyDeref = dereference(operation.requestBody);
+    const content = requestBodyDeref?.content;
+    if (content && Object.keys(content).length > 0) {
+      const hasSchema = Object.values(content).some(
+        (mediaType) => mediaType && "schema" in mediaType && mediaType.schema !== void 0 && mediaType.schema !== null
+      );
+      if (hasSchema) {
+        throw new Error(
+          `Invalid OpenAPI spec: ${method.toUpperCase()} request at path "${path}" has a request body. GET and DELETE requests should not have request bodies according to HTTP standards. Please move the body parameters to query parameters or use a different HTTP method.`
+        );
+      }
+    }
+  }
   const multiPartSchema = dereference(operation.requestBody)?.content?.["multipart/form-data"]?.schema;
   const isFileUpload = isReferenceObject(multiPartSchema) && multiPartSchema?.$ref?.includes("Body_upload_files");
   const parameters = generateFunctionParameters(

package/dist/index.mjs

@@ -490,6 +490,21 @@ function generateFunctionForOperation(method, path, operation) {
   if (!operation) {
     return "";
   }
+  const methodLower = method.toLowerCase();
+  if (["get", "delete"].includes(methodLower) && operation.requestBody) {
+    const requestBodyDeref = dereference(operation.requestBody);
+    const content = requestBodyDeref?.content;
+    if (content && Object.keys(content).length > 0) {
+      const hasSchema = Object.values(content).some(
+        (mediaType) => mediaType && "schema" in mediaType && mediaType.schema !== void 0 && mediaType.schema !== null
+      );
+      if (hasSchema) {
+        throw new Error(
+          `Invalid OpenAPI spec: ${method.toUpperCase()} request at path "${path}" has a request body. GET and DELETE requests should not have request bodies according to HTTP standards. Please move the body parameters to query parameters or use a different HTTP method.`
+        );
+      }
+    }
+  }
   const multiPartSchema = dereference(operation.requestBody)?.content?.["multipart/form-data"]?.schema;
   const isFileUpload = isReferenceObject(multiPartSchema) && multiPartSchema?.$ref?.includes("Body_upload_files");
   const parameters = generateFunctionParameters(

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@bagelink/sdk",
   "type": "module",
-  "version": "1.6.2",
+  "version": "1.6.7",
   "description": "Bagel core sdk packages",
   "author": {
     "name": "Neveh Allon",

package/src/openAPITools/functionGenerator.ts

@@ -384,8 +384,6 @@ function generateAxiosFunction(
 	} else {
 		const configParams = paramStr ? `, { ${paramStr} }` : ''
 		const bodyVar = requestBodyPayload || '{}'
-
-		// Handle different HTTP methods appropriately
 		axiosFunction += `return axios.${method}(${formattedPath}${
 			['get', 'delete'].includes(method)
 				? configParams
@@ -445,6 +443,27 @@ function generateFunctionForOperation(
 ): string {
 	if (!operation) { return '' }
 
+	// Validate: GET and DELETE requests should not have request bodies
+	const methodLower = method.toLowerCase()
+	if (['get', 'delete'].includes(methodLower) && operation.requestBody) {
+		const requestBodyDeref = dereference(operation.requestBody)
+		const content = requestBodyDeref?.content
+		// Check if request body has actual content with schemas
+		if (content && Object.keys(content).length > 0) {
+			// Check if any content type has a schema (indicating actual body data)
+			const hasSchema = Object.values(content).some(
+				mediaType => mediaType && 'schema' in mediaType && mediaType.schema !== undefined && mediaType.schema !== null
+			)
+			if (hasSchema) {
+				throw new Error(
+					`Invalid OpenAPI spec: ${method.toUpperCase()} request at path "${path}" has a request body. `
+					+ `GET and DELETE requests should not have request bodies according to HTTP standards. `
+					+ `Please move the body parameters to query parameters or use a different HTTP method.`
+				)
+			}
+		}
+	}
+
 	// Check if this is a file upload
 	const multiPartSchema = dereference(operation.requestBody)?.content?.[
 		'multipart/form-data'