fix:removed node-fetch and updated node version dependencies

Signed-off-by: Amitkanswal <[email protected]>

Author: Amitkanswal

.eslintrc

@@ -12,11 +12,6 @@
         "plugin:@typescript-eslint/eslint-recommended",
         "plugin:@typescript-eslint/recommended"
     ],
-    "env": {
-        "browser": true,
-        "node": true
-    },
-
     "rules": {
         "@typescript-eslint/no-explicit-any": 0,
         "@typescript-eslint/prefer-const": 0,

.github/workflows/node.js.yml

@@ -16,7 +16,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [12.x]
+        node-version: [18.x]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
     steps:

.github/workflows/npm-publish-github-packages.yml

@@ -14,7 +14,7 @@ jobs:
             - uses: actions/checkout@v3
             - uses: actions/setup-node@v3
               with:
-                  node-version: 14
+                  node-version: 18
             - run: npm ci
             - run: npm test
 
@@ -28,7 +28,7 @@ jobs:
             - uses: actions/checkout@v3
             - uses: actions/setup-node@v3
               with:
-                  node-version: 14
+                  node-version: 18
                   registry-url: https://npm.pkg.github.com/
             - run: npm ci
             - run: npm publish

.github/workflows/npm-publish.yml

@@ -14,7 +14,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v2
         with:
-          node-version: 16
+          node-version: 18
       - run: npm ci
       - run: npm test
 
@@ -25,7 +25,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v2
         with:
-          node-version: 16
+          node-version: 18
           registry-url: https://registry.npmjs.org/
       - run: npm ci
       - run: npm publish

.github/workflows/unit-test.yml

@@ -16,18 +16,18 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [14.x]
+        node-version: [18.x]
 
     steps:
       - name: Checkout repository
         uses: actions/checkout@v2
         with:
-          node-version: '14.x'
+          node-version: '18.x'
 
       - name: Use Node.js
         uses: actions/setup-node@v2
         with:
-          node-version: '14.x'
+          node-version: '18.x'
 
       - name: Install deps
         run: npm ci

package-lock.json

@@ -73,5 +73,8 @@
             "npm run prettier:fix ",
             "npm run lint:fix"
         ]
+    },
+    "engines": {
+        "node": ">=18.x"
     }
-}
+}

package.json

@@ -1,5 +1,4 @@
 import PostRobot from "post-robot";
-import { Response } from "node-fetch";
 import {
     AxiosError,
     AxiosHeaders,
@@ -50,11 +49,6 @@ export const dispatchAdapter =
         });
     };
 /**
-GitHub Copilot
-To handle errors generically and robustly, we can refactor the code to ensure that all errors are properly processed and converted into a standardized format. Here's how you can fix and improve the error handling in the provided code:
-
-Refactored Code
-
  * Dispatches an API request using axios and PostRobot.
  * @param url - The URL of the API endpoint.
  * @param options - Optional request options.
@@ -73,23 +67,21 @@ export const dispatchApiRequest = async (
         return new Response(response?.data, {
             status: response.status,
             statusText: response.statusText,
-            url: response.config.url,
-            headers: new Headers(Object.entries(response.headers ?? {})),
+            headers: response.config.headers,
         });
     } catch (err: any) {
         if (err.response) {
             return new Response(err.response?.data, {
-                status: err.status,
-                statusText: err.statusText,
-                headers: new Headers(
-                    Object.entries(err.response.headers ?? {})
-                ),
+                status: err.response.status,
+                statusText: err.response.statusText,
+                headers: err.response.headers
+             
             });
         }
         return new Response(err.stack, {
             status: err.status || 500,
             statusText: err.message || "Internal Server Error",
-            headers: new Headers(Object.entries(err.headers ?? {})),
+            headers: err.config.headers,
         });
     }
 };

src/utils/adapter.ts

@@ -13,14 +13,6 @@ export function onError(error: Error) {
     return Promise.reject(error);
 }
 
-export function sanitizeResponseHeader(axiosHeaders) {
-    const fetchHeaders = new Headers();
-    for (const key in axiosHeaders) {
-        fetchHeaders.append(key, axiosHeaders[key]);
-    }
-    return fetchHeaders;
-}
-
 export function formatAppRegion(region: string): RegionType {
     return region ?? Region.UNKNOWN;
 }