@arcjet/bun 1.0.0-beta.16 → 1.0.0-beta.18

package/index.d.ts

@@ -5,12 +5,22 @@ type Simplify<T> = {
     [KeyType in keyof T]: T[KeyType];
 } & {};
 declare const emptyObjectSymbol: unique symbol;
-type WithoutCustomProps = {
-    [emptyObjectSymbol]?: never;
-};
 type PlainObject = {
     [key: string]: unknown;
 };
+/**
+ * Dynamically generate whether zero or one `properties` object must or can be passed.
+ */
+type MaybeProperties<T> = {
+    [P in keyof T]?: T[P];
+} extends T ? T extends {
+    [emptyObjectSymbol]?: never;
+} ? [
+] : [
+    properties?: T
+] : [
+    properties: T
+];
 /**
  * Configuration for {@linkcode createRemoteClient}.
  */
@@ -80,7 +90,7 @@ export interface ArcjetBun<Props extends PlainObject> {
      *   Promise that resolves to an {@linkcode ArcjetDecision} indicating
      *   Arcjet’s decision about the request.
      */
-    protect(request: Request, ...props: Props extends WithoutCustomProps ? [] : [Props]): Promise<ArcjetDecision>;
+    protect(request: Request, ...properties: MaybeProperties<Props>): Promise<ArcjetDecision>;
     /**
      * Augment the client with another rule.
      *
@@ -94,7 +104,7 @@ export interface ArcjetBun<Props extends PlainObject> {
      * @returns
      *   Arcjet instance augmented with the given rule.
      */
-    withRule<Rule extends Primitive | Product>(rule: Rule): ArcjetBun<Simplify<Props & ExtraProps<Rule>>>;
+    withRule<ChildProperties extends PlainObject>(rule: Primitive<ChildProperties> | Product<ChildProperties>): ArcjetBun<Props & ChildProperties>;
     /**
      * Wrap the Bun `fetch` handler to provide additional details when calling
      * the `protect()` method.
@@ -118,4 +128,4 @@ export interface ArcjetBun<Props extends PlainObject> {
  * @returns
  *   Bun integration of Arcjet.
  */
-export default function arcjet<const Rules extends (Primitive | Product)[], const Characteristics extends readonly string[]>(options: ArcjetOptions<Rules, Characteristics>): ArcjetBun<Simplify<ExtraProps<Rules> & CharacteristicProps<Characteristics>>>;
+export default function arcjet<const Rules extends (Primitive | Product)[], const Characteristics extends readonly string[]>(options: ArcjetOptions<Rules, Characteristics>): ArcjetBun<ExtraProps<Rules> & CharacteristicProps<Characteristics>>;

package/index.js

@@ -4,26 +4,14 @@ export * from 'arcjet';
 import findIp, { parseProxy } from '@arcjet/ip';
 import { ArcjetHeaders } from '@arcjet/headers';
 import { env } from 'bun';
+import { readBodyWeb } from '@arcjet/body';
 import { logLevel, isDevelopment, baseUrl, platform } from '@arcjet/env';
 import { Logger } from '@arcjet/logger';
 import { createClient } from '@arcjet/protocol/client.js';
 import { createTransport } from '@arcjet/transport';
 
 /// <reference types="bun-types" />
-// 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";
-}
+let warnedForAutomaticBodyRead = false;
 /**
  * Create a remote client.
  *
@@ -38,7 +26,7 @@ function createRemoteClient(options) {
     // Transport is the HTTP client that the client uses to make requests.
     const transport = createTransport(url);
     const sdkStack = "BUN";
-    const sdkVersion = "1.0.0-beta.16";
+    const sdkVersion = "1.0.0-beta.18";
     return createClient({
         transport,
         baseUrl: url,
@@ -77,7 +65,7 @@ function arcjet(options) {
         log.warn("Arcjet will use 127.0.0.1 when missing public IP address in development mode");
     }
     function toArcjetRequest(request, props) {
-        const cookies = request.headers.get("cookie") ?? undefined;
+        const cookies = request.headers.get("cookie") ?? "";
         // We construct an ArcjetHeaders to normalize over Headers
         const headers = new ArcjetHeaders(request.headers);
         const url = new URL(request.url);
@@ -115,27 +103,33 @@ function arcjet(options) {
         };
     }
     function withClient(aj) {
-        return Object.freeze({
+        const client = {
             withRule(rule) {
                 const client = aj.withRule(rule);
                 return withClient(client);
             },
-            async protect(request, ...[props]) {
-                // TODO(#220): The generic manipulations get really mad here, so we cast
-                // Further investigation makes it seem like it has something to do with
-                // the definition of `props` in the signature but it's hard to track down
-                const req = toArcjetRequest(request, props ?? {});
+            async protect(request, props) {
+                // Cast of `{}` because here we switch from `undefined` to `Properties`.
+                const req = toArcjetRequest(request, props || {});
                 const getBody = async () => {
-                    try {
-                        const clonedRequest = request.clone();
-                        // Awaited to throw if it rejects and we'll just return undefined
-                        const body = await clonedRequest.text();
-                        return body;
+                    if (request.bodyUsed) {
+                        throw new Error("Cannot read body: already read");
                     }
-                    catch (e) {
-                        log.error("failed to get request body: %s", errorMessage(e));
-                        return;
+                    const clonedRequest = request.clone();
+                    let expectedLength;
+                    const expectedLengthString = request.headers.get("content-length");
+                    if (typeof expectedLengthString === "string") {
+                        expectedLength = parseInt(expectedLengthString, 10);
                     }
+                    // HEAD and GET requests do not have a body.
+                    if (!clonedRequest.body) {
+                        throw new Error("Cannot read body: body is missing");
+                    }
+                    if (!warnedForAutomaticBodyRead) {
+                        warnedForAutomaticBodyRead = true;
+                        log.warn("Automatically reading the request body is deprecated; please pass an explicit `sensitiveInfoValue` field. See <https://docs.arcjet.com/upgrading/sdk-migration>.");
+                    }
+                    return readBodyWeb(clonedRequest.body, { expectedLength });
                 };
                 return aj.protect({ getBody }, req);
             },
@@ -148,7 +142,8 @@ function arcjet(options) {
                     return fetch.call(server, request, server);
                 };
             },
-        });
+        };
+        return Object.freeze(client);
     }
     const aj = core__default({ ...options, client, log });
     return withClient(aj);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arcjet/bun",
-  "version": "1.0.0-beta.16",
+  "version": "1.0.0-beta.18",
   "description": "Arcjet SDK for Bun",
   "keywords": [
     "analyze",
@@ -45,22 +45,23 @@
     "test": "npm run build && npm run lint"
   },
   "dependencies": {
-    "@arcjet/env": "1.0.0-beta.16",
-    "@arcjet/headers": "1.0.0-beta.16",
-    "@arcjet/ip": "1.0.0-beta.16",
-    "@arcjet/logger": "1.0.0-beta.16",
-    "@arcjet/protocol": "1.0.0-beta.16",
-    "@arcjet/transport": "1.0.0-beta.16",
-    "arcjet": "1.0.0-beta.16"
+    "@arcjet/body": "1.0.0-beta.18",
+    "@arcjet/env": "1.0.0-beta.18",
+    "@arcjet/headers": "1.0.0-beta.18",
+    "@arcjet/ip": "1.0.0-beta.18",
+    "@arcjet/logger": "1.0.0-beta.18",
+    "@arcjet/protocol": "1.0.0-beta.18",
+    "@arcjet/transport": "1.0.0-beta.18",
+    "arcjet": "1.0.0-beta.18"
   },
   "devDependencies": {
-    "@arcjet/eslint-config": "1.0.0-beta.16",
-    "@arcjet/rollup-config": "1.0.0-beta.16",
-    "@rollup/wasm-node": "4.54.0",
-    "bun-types": "1.3.5",
+    "@arcjet/eslint-config": "1.0.0-beta.18",
+    "@arcjet/rollup-config": "1.0.0-beta.18",
+    "@rollup/wasm-node": "4.55.1",
+    "bun-types": "1.3.6",
     "eslint": "9.39.2",
     "typescript": "5.9.3",
-    "undici-types": "7.15.0"
+    "undici-types": "7.18.2"
   },
   "publishConfig": {
     "access": "public",