@arcjet/node 1.0.0-beta.15 → 1.0.0-beta.17

package/index.js

@@ -1,3 +1,4 @@
+import * as core from 'arcjet';
 import core__default from 'arcjet';
 export * from 'arcjet';
 import findIp, { parseProxy } from '@arcjet/ip';
@@ -27,9 +28,6 @@ const env = {
     get NODE_ENV() {
         return process.env.NODE_ENV;
     },
-    get ARCJET_KEY() {
-        return process.env.ARCJET_KEY;
-    },
     get ARCJET_ENV() {
         return process.env.ARCJET_ENV;
     },
@@ -39,21 +37,10 @@ const env = {
     get ARCJET_BASE_URL() {
         return process.env.ARCJET_BASE_URL;
     },
+    get FIREBASE_CONFIG() {
+        return process.env.FIREBASE_CONFIG;
+    },
 };
-// TODO: Deduplicate with other packages
-function errorMessage(err) {
-    if (err) {
-        if (typeof err === "string") {
-            return err;
-        }
-        if (typeof err === "object" &&
-            "message" in err &&
-            typeof err.message === "string") {
-            return err.message;
-        }
-    }
-    return "Unknown problem";
-}
 /**
  * Create a remote client.
  *
@@ -68,7 +55,7 @@ function createRemoteClient(options) {
     // Transport is the HTTP client that the client uses to make requests.
     const transport = createTransport(url);
     const sdkStack = "NODEJS";
-    const sdkVersion = "1.0.0-beta.15";
+    const sdkVersion = "1.0.0-beta.17";
     return createClient({
         transport,
         baseUrl: url,
@@ -196,47 +183,23 @@ function arcjet(options) {
                 // the definition of `props` in the signature but it's hard to track down
                 const req = toArcjetRequest(request, props ?? {});
                 const getBody = async () => {
-                    try {
-                        // If request.body is present then the body was likely read by a package like express' `body-parser`.
-                        // If it's not present then we attempt to read the bytes from the IncomingMessage ourselves.
-                        if (typeof request.body === "string") {
-                            return request.body;
-                        }
-                        else if (typeof request.body !== "undefined" &&
-                            // BigInt cannot be serialized with JSON.stringify
-                            typeof request.body !== "bigint") {
-                            return JSON.stringify(request.body);
-                        }
-                        if (typeof request.on === "function" &&
-                            typeof request.removeListener === "function") {
-                            let expectedLength;
-                            // TODO: This shouldn't need to build headers again but the type
-                            // for `req` above is overly relaxed
-                            const headers = new ArcjetHeaders(request.headers);
-                            const expectedLengthStr = headers.get("content-length");
-                            if (typeof expectedLengthStr === "string") {
-                                try {
-                                    expectedLength = parseInt(expectedLengthStr, 10);
-                                }
-                                catch {
-                                    // If the expected length couldn't be parsed we'll just not set one.
-                                }
-                            }
-                            // Awaited to throw if it rejects and we'll just return undefined
-                            const body = await readBody(request, {
-                                // We will process 1mb bodies
-                                limit: 1048576,
-                                expectedLength,
-                            });
-                            return body;
+                    // Read the stream if the body is not present.
+                    if (request.body === null || request.body === undefined) {
+                        let expectedLength;
+                        // TODO: This shouldn't need to build headers again but the type
+                        // for `req` above is overly relaxed
+                        const headers = new ArcjetHeaders(request.headers);
+                        const expectedLengthStr = headers.get("content-length");
+                        if (typeof expectedLengthStr === "string") {
+                            expectedLength = parseInt(expectedLengthStr, 10);
                         }
-                        log.warn("no body available");
-                        return;
+                        return readBody(request, { expectedLength });
                     }
-                    catch (e) {
-                        log.error("failed to get request body: %s", errorMessage(e));
-                        return;
+                    // A package like `body-parser` was used to read the stream.
+                    if (typeof request.body === "string") {
+                        return request.body;
                     }
+                    return JSON.stringify(request.body);
                 };
                 return aj.protect({ getBody }, req);
             },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arcjet/node",
-  "version": "1.0.0-beta.15",
+  "version": "1.0.0-beta.17",
   "description": "Arcjet SDK for Node.js",
   "keywords": [
     "analyze",
@@ -44,24 +44,27 @@
     "build": "rollup --config rollup.config.js",
     "lint": "eslint .",
     "prepublishOnly": "npm run build",
-    "test": "npm run build && npm run lint"
+    "test-api": "node --test -- test/*.test.js",
+    "test-coverage#": "TODO: after node 20, use: ` --test-coverage-branches=100 --test-coverage-exclude \"../{analyze-wasm,analyze,arcjet,cache,duration,env,headers,ip,logger,protocol,runtime,sprintf,stable-hash,transport}/**/*.{js,ts}\" --test-coverage-exclude \"test/**/*.{js,ts}\" --test-coverage-functions=100 --test-coverage-lines=100`",
+    "test-coverage": "node --experimental-test-coverage --test -- test/*.test.js",
+    "test": "npm run build && npm run lint && npm run test-coverage"
   },
   "dependencies": {
-    "@arcjet/env": "1.0.0-beta.15",
-    "@arcjet/headers": "1.0.0-beta.15",
-    "@arcjet/ip": "1.0.0-beta.15",
-    "@arcjet/logger": "1.0.0-beta.15",
-    "@arcjet/protocol": "1.0.0-beta.15",
-    "@arcjet/transport": "1.0.0-beta.15",
-    "@arcjet/body": "1.0.0-beta.15",
-    "arcjet": "1.0.0-beta.15"
+    "@arcjet/env": "1.0.0-beta.17",
+    "@arcjet/headers": "1.0.0-beta.17",
+    "@arcjet/ip": "1.0.0-beta.17",
+    "@arcjet/logger": "1.0.0-beta.17",
+    "@arcjet/protocol": "1.0.0-beta.17",
+    "@arcjet/transport": "1.0.0-beta.17",
+    "@arcjet/body": "1.0.0-beta.17",
+    "arcjet": "1.0.0-beta.17"
   },
   "devDependencies": {
-    "@arcjet/eslint-config": "1.0.0-beta.15",
-    "@arcjet/rollup-config": "1.0.0-beta.15",
-    "@types/node": "24.9.1",
-    "@rollup/wasm-node": "4.52.5",
-    "eslint": "9.38.0",
+    "@arcjet/eslint-config": "1.0.0-beta.17",
+    "@arcjet/rollup-config": "1.0.0-beta.17",
+    "@types/node": "25.0.3",
+    "@rollup/wasm-node": "4.55.1",
+    "eslint": "9.39.2",
     "typescript": "5.9.3"
   },
   "publishConfig": {