[TRI-1333] : Upgrade all packages to use Node18 and use fetch instead node-fetch (#581)

* [TRI-1333] : Upgrade all packages to use Node18 and use fetch instead of node-fetch

* Changeset Added

* Update pnpm-lock to compatible pnpm v7.18.1

* Restored pnpm-lock to the last known good point compatible with pnpm v7.18.1

* Update chilled-plants-exercise.md

---------

Co-authored-by: Eric Allam <[email protected]>

Author: Rutam21

.changeset/chilled-plants-exercise.md

@@ -0,0 +1,14 @@
+---
+"@trigger.dev/integration-kit": minor
+"@trigger.dev/eslint-plugin": minor
+"@trigger.dev/sdk": minor
+"@trigger.dev/sveltekit": minor
+"@trigger.dev/express": minor
+"@trigger.dev/nestjs": minor
+"@trigger.dev/nextjs": minor
+"@trigger.dev/astro": minor
+"@trigger.dev/core": minor
+"@trigger.dev/cli": minor
+---
+
+Drop support for Node v16, require Node >= 18. This allows us to use native fetch in our SDK which paves the way for multi-platform support.

packages/astro/package.json

@@ -33,7 +33,7 @@
     "typescript": "^4.8.4"
   },
   "engines": {
-    "node": ">=16.8.0"
+    "node": ">=18.0.0"
   },
   "license": "MIT",
   "publishConfig": {

packages/cli/package.json

@@ -84,6 +84,6 @@
     "zod": "3.22.3"
   },
   "engines": {
-    "node": ">=16.8.0"
+    "node": ">=18.0.0"
   }
 }

packages/core/package.json

@@ -42,6 +42,6 @@
     "typescript": "^4.9.4"
   },
   "engines": {
-    "node": ">=16.8.0"
+    "node": ">=18.0.0"
   }
 }

packages/eslint-plugin/package.json

@@ -29,7 +29,7 @@
     "npm-run-all": "^4.1.5"
   },
   "engines": {
-    "node": "^14.17.0 || ^16.0.0 || >= 18.0.0"
+    "node": ">=18.0.0"
   },
   "peerDependencies": {
     "eslint": ">=7"

packages/express/package.json

@@ -41,6 +41,6 @@
     "express": "^4.18.2"
   },
   "engines": {
-    "node": ">=16.8.0"
+    "node": ">=18.0.0"
   }
 }

packages/integration-kit/package.json

@@ -21,7 +21,6 @@
   "devDependencies": {
     "@trigger.dev/tsconfig": "workspace:*",
     "@types/node": "18",
-    "@types/node-fetch": "2.6.x",
     "@types/uuid": "^9.0.0",
     "rimraf": "^3.0.2",
     "tsup": "^6.5.0",
@@ -35,10 +34,9 @@
     "typecheck": "tsup --dts-resolve --no-dts"
   },
   "dependencies": {
-    "node-fetch": "2.6.x",
     "uuid": "^9.0.0"
   },
   "engines": {
-    "node": ">=16.8.0"
+    "node": ">=18.0.0"
   }
 }

packages/integration-kit/src/file.ts

@@ -1,7 +1,6 @@
 import fs, { promises } from "fs";
 import path from "path";
 import { v4 as uuidv4 } from "uuid";
-import fetch from "node-fetch";
 
 export async function fileFromString(contents: string | Buffer, fileName: string): Promise<File> {
   const directory = path.join("tmp", uuidv4());
@@ -13,7 +12,8 @@ export async function fileFromString(contents: string | Buffer, fileName: string
 
 export async function fileFromUrl(url: string) {
   const response = await fetch(url);
-  const content = await response.buffer();
+  const arrayBuffer = await response.arrayBuffer();
+  const content = Buffer.from(arrayBuffer);
   const fileName = path.basename(url);
 
   return fileFromString(content, fileName);

packages/nestjs/package.json

@@ -43,6 +43,6 @@
     "debug": "^4.3.4"
   },
   "engines": {
-    "node": ">=16.8.0"
+    "node": ">=18.0.0"
   }
 }

packages/nextjs/package.json

@@ -41,6 +41,6 @@
     "debug": "^4.3.4"
   },
   "engines": {
-    "node": ">=16.8.0"
+    "node": ">=18.0.0"
   }
 }