revert: revert #245 (#247)

This reverts commit 281a09c aka #245 so
we can re-publish it as a breaking change.

Author: kanadgupta

.github/workflows/ci.yml

@@ -13,9 +13,8 @@ jobs:
     strategy:
       matrix:
         node-version:
-          - lts/-1
-          - lts/*
-          - latest
+          - 18
+          - 20
 
     steps:
       - uses: actions/checkout@v4

.vscode/settings.json

@@ -1,15 +1,10 @@
 {
-  "editor.defaultFormatter": "esbenp.prettier-vscode",
   "editor.codeActionsOnSave": {
     "source.fixAll": "explicit"
   },
-  "editor.formatOnSave": true,
+  "editor.defaultFormatter": "esbenp.prettier-vscode",
 
   // controlled by the .editorconfig at root since we can't map vscode settings directly to files
   // https://github.com/microsoft/vscode/issues/35350
-  "files.insertFinalNewline": false,
-
-  "search.exclude": {
-    "coverage": true
-  }
+  "files.insertFinalNewline": false
 }

README.md

@@ -109,8 +109,8 @@ console.log(
   }),
 );
 
-// generate Node.js: Axios output
-console.log(snippet.convert('node', 'axios'));
+// generate Node.js: Unirest output
+console.log(snippet.convert('node', 'unirest'));
 ```
 
 ### addTarget(target)

integrations/node.Dockerfile

@@ -13,7 +13,10 @@ WORKDIR /src
 ADD package.json /src/
 
 # https://www.npmjs.com/package/axios
-RUN npm install axios && \
+# https://www.npmjs.com/package/request
+# Installing node-fetch@2 because as of 3.0 is't now an ESM-only package.
+# https://www.npmjs.com/package/node-fetch
+RUN npm install axios request node-fetch@2 && \
   npm install
 
 ADD . /src

package.json

@@ -53,12 +53,14 @@
     "ocaml",
     "php",
     "python",
+    "request",
     "requests",
     "ruby",
     "shell",
     "snippet",
     "swift",
     "swift",
+    "unirest",
     "xhr",
     "xmlhttprequest"
   ],

src/fixtures/customTarget.ts

@@ -1,15 +1,15 @@
 import type { Target } from '../targets/index.js';
 
-import { axios } from '../targets/node/axios/client.js';
+import { request } from '../targets/node/request/client.js';
 
 export const customTarget = {
   info: {
-    key: 'node-variant',
-    title: 'Node Variant',
+    key: 'js-variant',
+    title: 'JavaScript Variant',
     extname: '.js',
-    default: 'axios',
+    default: 'request',
   },
   clientsById: {
-    axios,
+    request,
   },
 } as unknown as Target;

src/helpers/__snapshots__/utils.test.ts.snap

@@ -150,7 +150,7 @@ exports[`availableTargets > returns all available targets 1`] = `
         "title": "jQuery",
       },
     ],
-    "default": "fetch",
+    "default": "xhr",
     "key": "javascript",
     "title": "JavaScript",
   },
@@ -192,23 +192,39 @@ exports[`availableTargets > returns all available targets 1`] = `
         "link": "http://nodejs.org/api/http.html#http_http_request_options_callback",
         "title": "HTTP",
       },
+      {
+        "description": "Simplified HTTP request client",
+        "extname": ".cjs",
+        "installation": "npm install request --save",
+        "key": "request",
+        "link": "https://github.com/request/request",
+        "title": "Request",
+      },
+      {
+        "description": "Lightweight HTTP Request Client Library",
+        "extname": ".cjs",
+        "key": "unirest",
+        "link": "http://unirest.io/nodejs.html",
+        "title": "Unirest",
+      },
       {
         "description": "Promise based HTTP client for the browser and node.js",
-        "extname": ".js",
+        "extname": ".cjs",
         "installation": "npm install axios --save",
         "key": "axios",
         "link": "https://github.com/axios/axios",
         "title": "Axios",
       },
       {
-        "description": "Perform asynchronous HTTP requests with the Fetch API",
-        "extname": ".js",
+        "description": "Simplified HTTP node-fetch client",
+        "extname": ".cjs",
+        "installation": "npm install node-fetch@2 --save",
         "key": "fetch",
-        "link": "https://nodejs.org/docs/latest/api/globals.html#fetch",
-        "title": "fetch",
+        "link": "https://github.com/bitinn/node-fetch",
+        "title": "Fetch",
       },
     ],
-    "default": "fetch",
+    "default": "native",
     "key": "node",
     "title": "Node.js",
   },

src/helpers/utils.test.ts

@@ -22,7 +22,7 @@ describe('extname', () => {
     expect(extname('c', 'libcurl')).toBe('.c');
     expect(extname('clojure', 'clj_http')).toBe('.clj');
     expect(extname('javascript', 'axios')).toBe('.js');
-    expect(extname('node', 'axios')).toBe('.js');
+    expect(extname('node', 'axios')).toBe('.cjs');
   });
 
   it('returns empty string if the extension is not found', () => {

src/integration.test.ts

@@ -21,7 +21,7 @@ const ENVIRONMENT_CONFIG = {
     c: ['libcurl'],
     csharp: ['httpclient', 'restsharp'],
     go: ['native'],
-    node: ['axios', 'fetch'],
+    node: ['axios', 'fetch', 'native', 'request'],
     php: ['curl', 'guzzle'],
     python: ['requests'],
     shell: ['curl'],
@@ -30,7 +30,7 @@ const ENVIRONMENT_CONFIG = {
     // When running tests locally, or within a CI environment, we shold limit the targets that
     // we're testing so as to not require a mess of dependency requirements that would be better
     // served within a container.
-    node: ['fetch'],
+    node: ['native'],
     php: ['curl'],
     python: ['requests'],
     shell: ['curl'],

src/targets/index.test.ts

@@ -31,7 +31,7 @@ const targetFilter: TargetId[] = [
 /** useful for debuggin, only run a particular set of targets */
 const clientFilter: ClientId[] = [
   // put your clientId here:
-  // 'axios',
+  // 'unirest',
 ];
 
 /** useful for debuggin, only run a particular set of fixtures */