httpsnippet-client-api 4.0.0 → 4.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "httpsnippet-client-api",
-  "version": "4.0.0",
+  "version": "4.1.3",
   "description": "An HTTP Snippet client for generating snippets for the api module.",
   "main": "src/index.js",
   "scripts": {
@@ -29,12 +29,12 @@
   },
   "peerDependencies": {
     "@readme/httpsnippet": "^3.0.0",
-    "oas": "^16.0.3"
+    "oas": "^17.1.0"
   },
   "devDependencies": {
-    "@readme/eslint-config": "^7.2.2",
+    "@readme/eslint-config": "^8.0.2",
     "@readme/oas-examples": "^4.3.2",
-    "eslint": "^7.32.0",
+    "eslint": "^8.3.0",
     "jest": "^27.3.1",
     "prettier": "^2.4.1"
   },
@@ -43,6 +43,5 @@
     "testPathIgnorePatterns": [
       "__tests__/__fixtures__/"
     ]
-  },
-  "gitHead": "f1d2c251f1468bceb78e39731e20f30c6745c419"
+  }
 }

package/src/index.js

@@ -2,7 +2,7 @@ const { match } = require('path-to-regexp');
 const stringifyObject = require('stringify-object');
 const CodeBuilder = require('@readme/httpsnippet/src/helpers/code-builder');
 const contentType = require('content-type');
-const Oas = require('oas');
+const Oas = require('oas').default;
 
 function stringify(obj, opts = {}) {
   return stringifyObject(obj, { indent: '  ', ...opts });
@@ -90,6 +90,7 @@ module.exports = function (source, options) {
 
   const method = source.method.toLowerCase();
   const oas = new Oas(opts.apiDefinition);
+  const apiDefinition = oas.getDefinition();
   const foundOperation = oas.findOperation(source.url, method);
   if (!foundOperation) {
     throw new Error(
@@ -112,7 +113,7 @@ module.exports = function (source, options) {
   // `oas` library then the URL either has server variables contained in it (that don't match the defaults), or the
   // OAS offers alternate server URLs and we should expose that in the generated snippet.
   const configData = [];
-  if ((oas.servers || []).length > 1) {
+  if ((apiDefinition.servers || []).length > 1) {
     const stockUrl = oas.url();
     const baseUrl = source.url.replace(path, '');
     if (baseUrl !== stockUrl) {
@@ -212,22 +213,24 @@ module.exports = function (source, options) {
       break;
 
     case 'multipart/form-data':
-      body = {};
+      if (source.postData.params) {
+        body = {};
+
+        // If there's a `Content-Type` header present in the metadata, but it's for the form-data
+        // request then dump it off the snippet. We shouldn't offload that unnecessary bloat to the
+        // user, instead letting the SDK handle it automatically.
+        if ('content-type' in metadata && metadata['content-type'].indexOf('multipart/form-data') === 0) {
+          delete metadata['content-type'];
+        }
 
-      // If there's a `Content-Type` header present in the metadata, but it's for the form-data
-      // request then dump it off the snippet. We shouldn't offload that unnecessary bloat to the
-      // user, instead letting the SDK handle it automatically.
-      if ('content-type' in metadata && metadata['content-type'].indexOf('multipart/form-data') === 0) {
-        delete metadata['content-type'];
+        source.postData.params.forEach(function (param) {
+          if (param.fileName) {
+            body[param.name] = param.fileName;
+          } else {
+            body[param.name] = param.value;
+          }
+        });
       }
-
-      source.postData.params.forEach(function (param) {
-        if (param.fileName) {
-          body[param.name] = param.fileName;
-        } else {
-          body[param.name] = param.value;
-        }
-      });
       break;
 
     default: