fix(opapi): ensure we never send a body with GET requests (#545)

Author: pascal-botpress

opapi/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bpinternal/opapi",
-  "version": "0.12.0",
+  "version": "0.12.1",
   "description": "",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",

opapi/src/generators/client-node.ts

@@ -87,7 +87,7 @@ export type ParsedRequest = {
 const isDefined = <T>(pair: [string, T | undefined]): pair is [string, T] => pair[1] !== undefined
 
 export const toAxiosRequest = (req: ParsedRequest): AxiosRequestConfig => {
-  const { method, path, query, headers: headerParams, body: data } = req
+  const { method, path, query, headers: headerParams, body } = req
 
   // prepare headers
   const headerEntries: [string, string][] = Object.entries(headerParams).filter(isDefined)
@@ -97,6 +97,10 @@ export const toAxiosRequest = (req: ParsedRequest): AxiosRequestConfig => {
   const queryString = qs.stringify(query, { encode: true, arrayFormat: 'repeat', allowDots: true })
 
   const url = queryString ? [path, queryString].join('?') : path
+  const data =
+    ['put', 'post', 'delete', 'patch'].includes(method.toLowerCase())
+      ? body
+      : undefined
 
   return {
     method,