agentmail 0.1.6 → 0.1.8

package/dist/esm/api/resources/webhooks/types/CreateWebhookRequest.d.mts

@@ -1,7 +1,7 @@
 import type * as AgentMail from "../../../index.mjs";
 export interface CreateWebhookRequest {
     url: AgentMail.webhooks.Url;
-    eventTypes: AgentMail.webhooks.EventTypes;
+    eventTypes: AgentMail.EventTypes;
     inboxIds?: AgentMail.webhooks.InboxIds;
     clientId?: AgentMail.webhooks.ClientId;
 }

package/dist/esm/api/resources/webhooks/types/Webhook.d.mts

@@ -2,7 +2,7 @@ import type * as AgentMail from "../../../index.mjs";
 export interface Webhook {
     webhookId: AgentMail.webhooks.WebhookId;
     url: AgentMail.webhooks.Url;
-    eventTypes?: AgentMail.webhooks.EventTypes;
+    eventTypes?: AgentMail.EventTypes;
     inboxIds?: AgentMail.webhooks.InboxIds;
     /** Secret for webhook signature verification. */
     secret: string;

package/dist/esm/api/resources/webhooks/types/index.d.mts

@@ -1,7 +1,5 @@
 export * from "./ClientId.mjs";
 export * from "./CreateWebhookRequest.mjs";
-export * from "./EventType.mjs";
-export * from "./EventTypes.mjs";
 export * from "./InboxIds.mjs";
 export * from "./ListWebhooksResponse.mjs";
 export * from "./Url.mjs";

package/dist/esm/api/resources/webhooks/types/index.mjs

@@ -1,7 +1,5 @@
 export * from "./ClientId.mjs";
 export * from "./CreateWebhookRequest.mjs";
-export * from "./EventType.mjs";
-export * from "./EventTypes.mjs";
 export * from "./InboxIds.mjs";
 export * from "./ListWebhooksResponse.mjs";
 export * from "./Url.mjs";

package/dist/esm/api/resources/websockets/client/Client.d.mts

@@ -1,6 +1,7 @@
 import type { BaseClientOptions } from "../../../../BaseClient.mjs";
+import { type NormalizedClientOptionsWithAuth } from "../../../../BaseClient.mjs";
 import { WebsocketsSocket } from "./Socket.mjs";
-export declare namespace Websockets {
+export declare namespace WebsocketsClient {
     interface Options extends BaseClientOptions {
     }
     interface ConnectArgs {
@@ -13,9 +14,8 @@ export declare namespace Websockets {
         reconnectAttempts?: number;
     }
 }
-export declare class Websockets {
-    protected readonly _options: Websockets.Options;
-    constructor(_options?: Websockets.Options);
-    connect(args?: Websockets.ConnectArgs): Promise<WebsocketsSocket>;
-    protected _getAuthorizationHeader(): Promise<string>;
+export declare class WebsocketsClient {
+    protected readonly _options: NormalizedClientOptionsWithAuth<WebsocketsClient.Options>;
+    constructor(options?: WebsocketsClient.Options);
+    connect(args?: WebsocketsClient.ConnectArgs): Promise<WebsocketsSocket>;
 }

package/dist/esm/api/resources/websockets/client/Client.mjs

@@ -8,14 +8,14 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
-import { mergeHeaders, mergeOnlyDefinedHeaders } from "../../../../core/headers.mjs";
+import { normalizeClientOptionsWithAuth } from "../../../../BaseClient.mjs";
+import { mergeHeaders } from "../../../../core/headers.mjs";
 import * as core from "../../../../core/index.mjs";
 import * as environments from "../../../../environments.mjs";
-import * as errors from "../../../../errors/index.mjs";
 import { WebsocketsSocket } from "./Socket.mjs";
-export class Websockets {
-    constructor(_options = {}) {
-        this._options = _options;
+export class WebsocketsClient {
+    constructor(options = {}) {
+        this._options = normalizeClientOptionsWithAuth(options);
     }
     connect() {
         return __awaiter(this, arguments, void 0, function* (args = {}) {
@@ -25,7 +25,8 @@ export class Websockets {
             if (authToken != null) {
                 _queryParams.auth_token = authToken;
             }
-            const _headers = mergeHeaders(mergeOnlyDefinedHeaders({ Authorization: yield this._getAuthorizationHeader() }), headers);
+            const _authRequest = yield this._options.authProvider.getAuthRequest();
+            const _headers = mergeHeaders(_authRequest.headers, headers);
             const socket = new core.ReconnectingWebSocket({
                 url: core.url.join((_a = (yield core.Supplier.get(this._options.baseUrl))) !== null && _a !== void 0 ? _a : ((_b = (yield core.Supplier.get(this._options.environment))) !== null && _b !== void 0 ? _b : environments.AgentMailEnvironment.Production).websockets, "/v0"),
                 protocols: [],
@@ -36,16 +37,4 @@ export class Websockets {
             return new WebsocketsSocket({ socket });
         });
     }
-    _getAuthorizationHeader() {
-        return __awaiter(this, void 0, void 0, function* () {
-            var _a;
-            const bearer = (_a = (yield core.Supplier.get(this._options.apiKey))) !== null && _a !== void 0 ? _a : process === null || process === void 0 ? void 0 : process.env.AGENTMAIL_API_KEY;
-            if (bearer == null) {
-                throw new errors.AgentMailError({
-                    message: "Please specify a bearer by either passing it in to the constructor or initializing a AGENTMAIL_API_KEY environment variable",
-                });
-            }
-            return `Bearer ${bearer}`;
-        });
-    }
 }

package/dist/esm/api/resources/websockets/client/Socket.d.mts

@@ -4,7 +4,7 @@ export declare namespace WebsocketsSocket {
     interface Args {
         socket: core.ReconnectingWebSocket;
     }
-    type Response = AgentMail.Subscribed | AgentMail.MessageReceivedEvent | AgentMail.MessageSentEvent | AgentMail.MessageDeliveredEvent | AgentMail.MessageBouncedEvent | AgentMail.MessageComplainedEvent | AgentMail.MessageRejectedEvent;
+    type Response = AgentMail.Subscribed | AgentMail.MessageReceivedEvent | AgentMail.MessageSentEvent | AgentMail.MessageDeliveredEvent | AgentMail.MessageBouncedEvent | AgentMail.MessageComplainedEvent | AgentMail.MessageRejectedEvent | AgentMail.DomainVerifiedEvent;
     type EventHandlers = {
         open?: () => void;
         message?: (message: Response) => void;

package/dist/esm/api/resources/websockets/types/Subscribe.d.mts

@@ -1,5 +1,9 @@
+import type * as AgentMail from "../../../index.mjs";
 export interface Subscribe {
     type: "subscribe";
+    eventTypes?: AgentMail.EventTypes;
     /** IDs of the inboxes to subscribe to. */
-    inboxIds: string[];
+    inboxIds?: string[];
+    /** IDs of the pods to subscribe to. */
+    podIds?: string[];
 }

package/dist/esm/api/resources/websockets/types/Subscribed.d.mts

@@ -1,5 +1,9 @@
+import type * as AgentMail from "../../../index.mjs";
 export interface Subscribed {
     type: "subscribed";
+    eventTypes?: AgentMail.EventTypes;
     /** IDs of the inboxes that have been subscribed to. */
-    inboxIds: string[];
+    inboxIds?: string[];
+    /** IDs of the pods that have been subscribed to. */
+    podIds?: string[];
 }

package/dist/esm/auth/BearerAuthProvider.d.mts

@@ -0,0 +1,14 @@
+import * as core from "../core/index.mjs";
+export declare namespace BearerAuthProvider {
+    interface Options {
+        apiKey?: core.Supplier<core.BearerToken | undefined>;
+    }
+}
+export declare class BearerAuthProvider implements core.AuthProvider {
+    private readonly token;
+    constructor(options: BearerAuthProvider.Options);
+    static canCreate(options: BearerAuthProvider.Options): boolean;
+    getAuthRequest(_arg?: {
+        endpointMetadata?: core.EndpointMetadata;
+    }): Promise<core.AuthRequest>;
+}

package/dist/esm/auth/BearerAuthProvider.mjs

@@ -0,0 +1,35 @@
+// This file was auto-generated by Fern from our API Definition.
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
+import * as core from "../core/index.mjs";
+import * as errors from "../errors/index.mjs";
+export class BearerAuthProvider {
+    constructor(options) {
+        this.token = options.apiKey;
+    }
+    static canCreate(options) {
+        var _a;
+        return options.apiKey != null || ((_a = process.env) === null || _a === void 0 ? void 0 : _a.AGENTMAIL_API_KEY) != null;
+    }
+    getAuthRequest(_arg) {
+        return __awaiter(this, void 0, void 0, function* () {
+            var _a, _b;
+            const apiKey = (_a = (yield core.Supplier.get(this.token))) !== null && _a !== void 0 ? _a : (_b = process.env) === null || _b === void 0 ? void 0 : _b.AGENTMAIL_API_KEY;
+            if (apiKey == null) {
+                throw new errors.AgentMailError({
+                    message: "Please specify a apiKey by either passing it in to the constructor or initializing a AGENTMAIL_API_KEY environment variable",
+                });
+            }
+            return {
+                headers: { Authorization: `Bearer ${apiKey}` },
+            };
+        });
+    }
+}

package/dist/esm/auth/index.d.mts

@@ -0,0 +1 @@
+export { BearerAuthProvider } from "./BearerAuthProvider.mjs";

package/dist/esm/auth/index.mjs

@@ -0,0 +1 @@
+export { BearerAuthProvider } from "./BearerAuthProvider.mjs";

package/dist/esm/core/auth/AuthProvider.d.mts

@@ -1,4 +1,7 @@
+import type { EndpointMetadata } from "../fetcher/EndpointMetadata.mjs";
 import type { AuthRequest } from "./AuthRequest.mjs";
 export interface AuthProvider {
-    getAuthRequest(): Promise<AuthRequest>;
+    getAuthRequest(arg?: {
+        endpointMetadata?: EndpointMetadata;
+    }): Promise<AuthRequest>;
 }

package/dist/esm/core/auth/BearerToken.d.mts

@@ -1,5 +1,7 @@
 export type BearerToken = string;
+declare function toAuthorizationHeader(token: string | undefined): string | undefined;
 export declare const BearerToken: {
-    toAuthorizationHeader: (token: BearerToken | undefined) => string | undefined;
+    toAuthorizationHeader: typeof toAuthorizationHeader;
     fromAuthorizationHeader: (header: string) => BearerToken;
 };
+export {};

package/dist/esm/core/auth/BearerToken.mjs

@@ -1,11 +1,12 @@
 const BEARER_AUTH_HEADER_PREFIX = /^Bearer /i;
+function toAuthorizationHeader(token) {
+    if (token == null) {
+        return undefined;
+    }
+    return `Bearer ${token}`;
+}
 export const BearerToken = {
-    toAuthorizationHeader: (token) => {
-        if (token == null) {
-            return undefined;
-        }
-        return `Bearer ${token}`;
-    },
+    toAuthorizationHeader: toAuthorizationHeader,
     fromAuthorizationHeader: (header) => {
         return header.replace(BEARER_AUTH_HEADER_PREFIX, "").trim();
     },

package/dist/esm/core/auth/NoOpAuthProvider.d.mts

@@ -0,0 +1,5 @@
+import type { AuthProvider } from "./AuthProvider.mjs";
+import type { AuthRequest } from "./AuthRequest.mjs";
+export declare class NoOpAuthProvider implements AuthProvider {
+    getAuthRequest(): Promise<AuthRequest>;
+}

package/dist/esm/core/auth/NoOpAuthProvider.mjs

@@ -0,0 +1,5 @@
+export class NoOpAuthProvider {
+    getAuthRequest() {
+        return Promise.resolve({ headers: {} });
+    }
+}

package/dist/esm/core/auth/index.d.mts

@@ -2,3 +2,4 @@ export type { AuthProvider } from "./AuthProvider.mjs";
 export type { AuthRequest } from "./AuthRequest.mjs";
 export { BasicAuth } from "./BasicAuth.mjs";
 export { BearerToken } from "./BearerToken.mjs";
+export { NoOpAuthProvider } from "./NoOpAuthProvider.mjs";

package/dist/esm/core/auth/index.mjs

@@ -1,2 +1,3 @@
 export { BasicAuth } from "./BasicAuth.mjs";
 export { BearerToken } from "./BearerToken.mjs";
+export { NoOpAuthProvider } from "./NoOpAuthProvider.mjs";

package/dist/esm/core/exports.d.mts

@@ -1 +1,2 @@
+export * from "./logging/exports.mjs";
 export * from "./websocket/exports.mjs";

package/dist/esm/core/exports.mjs

@@ -1 +1,2 @@
+export * from "./logging/exports.mjs";
 export * from "./websocket/exports.mjs";

package/dist/esm/core/fetcher/Fetcher.d.mts

@@ -1,3 +1,4 @@
+import { type LogConfig, type Logger } from "../logging/logger.mjs";
 import type { APIResponse } from "./APIResponse.mjs";
 import type { EndpointMetadata } from "./EndpointMetadata.mjs";
 import { EndpointSupplier } from "./EndpointSupplier.mjs";
@@ -14,10 +15,12 @@ export declare namespace Fetcher {
         maxRetries?: number;
         withCredentials?: boolean;
         abortSignal?: AbortSignal;
-        requestType?: "json" | "file" | "bytes";
+        requestType?: "json" | "file" | "bytes" | "form" | "other";
         responseType?: "json" | "blob" | "sse" | "streaming" | "text" | "arrayBuffer" | "binary-response";
         duplex?: "half";
         endpointMetadata?: EndpointMetadata;
+        fetchFn?: typeof fetch;
+        logging?: LogConfig | Logger;
     }
     type Error = FailedStatusCodeError | NonJsonError | TimeoutError | UnknownError;
     interface FailedStatusCodeError {

package/dist/esm/core/fetcher/Fetcher.mjs

@@ -8,21 +8,153 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
     });
 };
 import { toJson } from "../json.mjs";
+import { createLogger } from "../logging/logger.mjs";
 import { createRequestUrl } from "./createRequestUrl.mjs";
 import { EndpointSupplier } from "./EndpointSupplier.mjs";
 import { getErrorResponseBody } from "./getErrorResponseBody.mjs";
 import { getFetchFn } from "./getFetchFn.mjs";
 import { getRequestBody } from "./getRequestBody.mjs";
 import { getResponseBody } from "./getResponseBody.mjs";
+import { Headers } from "./Headers.mjs";
 import { makeRequest } from "./makeRequest.mjs";
 import { abortRawResponse, toRawResponse, unknownRawResponse } from "./RawResponse.mjs";
 import { requestWithRetries } from "./requestWithRetries.mjs";
+const SENSITIVE_HEADERS = new Set([
+    "authorization",
+    "www-authenticate",
+    "x-api-key",
+    "api-key",
+    "apikey",
+    "x-api-token",
+    "x-auth-token",
+    "auth-token",
+    "cookie",
+    "set-cookie",
+    "proxy-authorization",
+    "proxy-authenticate",
+    "x-csrf-token",
+    "x-xsrf-token",
+    "x-session-token",
+    "x-access-token",
+]);
+function redactHeaders(headers) {
+    const filtered = {};
+    for (const [key, value] of headers instanceof Headers ? headers.entries() : Object.entries(headers)) {
+        if (SENSITIVE_HEADERS.has(key.toLowerCase())) {
+            filtered[key] = "[REDACTED]";
+        }
+        else {
+            filtered[key] = value;
+        }
+    }
+    return filtered;
+}
+const SENSITIVE_QUERY_PARAMS = new Set([
+    "api_key",
+    "api-key",
+    "apikey",
+    "token",
+    "access_token",
+    "access-token",
+    "auth_token",
+    "auth-token",
+    "password",
+    "passwd",
+    "secret",
+    "api_secret",
+    "api-secret",
+    "apisecret",
+    "key",
+    "session",
+    "session_id",
+    "session-id",
+]);
+function redactQueryParameters(queryParameters) {
+    if (queryParameters == null) {
+        return queryParameters;
+    }
+    const redacted = {};
+    for (const [key, value] of Object.entries(queryParameters)) {
+        if (SENSITIVE_QUERY_PARAMS.has(key.toLowerCase())) {
+            redacted[key] = "[REDACTED]";
+        }
+        else {
+            redacted[key] = value;
+        }
+    }
+    return redacted;
+}
+function redactUrl(url) {
+    const protocolIndex = url.indexOf("://");
+    if (protocolIndex === -1)
+        return url;
+    const afterProtocol = protocolIndex + 3;
+    // Find the first delimiter that marks the end of the authority section
+    const pathStart = url.indexOf("/", afterProtocol);
+    let queryStart = url.indexOf("?", afterProtocol);
+    let fragmentStart = url.indexOf("#", afterProtocol);
+    const firstDelimiter = Math.min(pathStart === -1 ? url.length : pathStart, queryStart === -1 ? url.length : queryStart, fragmentStart === -1 ? url.length : fragmentStart);
+    // Find the LAST @ before the delimiter (handles multiple @ in credentials)
+    let atIndex = -1;
+    for (let i = afterProtocol; i < firstDelimiter; i++) {
+        if (url[i] === "@") {
+            atIndex = i;
+        }
+    }
+    if (atIndex !== -1) {
+        url = `${url.slice(0, afterProtocol)}[REDACTED]@${url.slice(atIndex + 1)}`;
+    }
+    // Recalculate queryStart since url might have changed
+    queryStart = url.indexOf("?");
+    if (queryStart === -1)
+        return url;
+    fragmentStart = url.indexOf("#", queryStart);
+    const queryEnd = fragmentStart !== -1 ? fragmentStart : url.length;
+    const queryString = url.slice(queryStart + 1, queryEnd);
+    if (queryString.length === 0)
+        return url;
+    // FAST PATH: Quick check if any sensitive keywords present
+    // Using indexOf is faster than regex for simple substring matching
+    const lower = queryString.toLowerCase();
+    const hasSensitive = lower.includes("token") ||
+        lower.includes("key") ||
+        lower.includes("password") ||
+        lower.includes("passwd") ||
+        lower.includes("secret") ||
+        lower.includes("session") ||
+        lower.includes("auth");
+    if (!hasSensitive) {
+        return url;
+    }
+    // SLOW PATH: Parse and redact
+    const redactedParams = [];
+    const params = queryString.split("&");
+    for (const param of params) {
+        const equalIndex = param.indexOf("=");
+        if (equalIndex === -1) {
+            redactedParams.push(param);
+            continue;
+        }
+        const key = param.slice(0, equalIndex);
+        let shouldRedact = SENSITIVE_QUERY_PARAMS.has(key.toLowerCase());
+        if (!shouldRedact && key.includes("%")) {
+            try {
+                const decodedKey = decodeURIComponent(key);
+                shouldRedact = SENSITIVE_QUERY_PARAMS.has(decodedKey.toLowerCase());
+            }
+            catch (_a) { }
+        }
+        redactedParams.push(shouldRedact ? `${key}=[REDACTED]` : param);
+    }
+    return url.slice(0, queryStart + 1) + redactedParams.join("&") + url.slice(queryEnd);
+}
 function getHeaders(args) {
     return __awaiter(this, void 0, void 0, function* () {
         var _a;
-        const newHeaders = {};
+        const newHeaders = new Headers();
+        newHeaders.set("Accept", args.responseType === "json" ? "application/json" : args.responseType === "text" ? "text/plain" : "*/*");
         if (args.body !== undefined && args.contentType != null) {
-            newHeaders["Content-Type"] = args.contentType;
+            newHeaders.set("Content-Type", args.contentType);
         }
         if (args.headers == null) {
             return newHeaders;
@@ -30,31 +162,52 @@ function getHeaders(args) {
         for (const [key, value] of Object.entries(args.headers)) {
             const result = yield EndpointSupplier.get(value, { endpointMetadata: (_a = args.endpointMetadata) !== null && _a !== void 0 ? _a : {} });
             if (typeof result === "string") {
-                newHeaders[key] = result;
+                newHeaders.set(key, result);
                 continue;
             }
             if (result == null) {
                 continue;
             }
-            newHeaders[key] = `${result}`;
+            newHeaders.set(key, `${result}`);
         }
         return newHeaders;
     });
 }
 export function fetcherImpl(args) {
     return __awaiter(this, void 0, void 0, function* () {
-        var _a;
+        var _a, _b, _c;
         const url = createRequestUrl(args.url, args.queryParameters);
         const requestBody = yield getRequestBody({
             body: args.body,
-            type: args.requestType === "json" ? "json" : "other",
+            type: (_a = args.requestType) !== null && _a !== void 0 ? _a : "other",
         });
-        const fetchFn = yield getFetchFn();
+        const fetchFn = (_b = args.fetchFn) !== null && _b !== void 0 ? _b : (yield getFetchFn());
+        const headers = yield getHeaders(args);
+        const logger = createLogger(args.logging);
+        if (logger.isDebug()) {
+            const metadata = {
+                method: args.method,
+                url: redactUrl(url),
+                headers: redactHeaders(headers),
+                queryParameters: redactQueryParameters(args.queryParameters),
+                hasBody: requestBody != null,
+            };
+            logger.debug("Making HTTP request", metadata);
+        }
         try {
             const response = yield requestWithRetries(() => __awaiter(this, void 0, void 0, function* () {
-                return makeRequest(fetchFn, url, args.method, yield getHeaders(args), requestBody, args.timeoutMs, args.abortSignal, args.withCredentials, args.duplex);
+                return makeRequest(fetchFn, url, args.method, headers, requestBody, args.timeoutMs, args.abortSignal, args.withCredentials, args.duplex);
             }), args.maxRetries);
             if (response.status >= 200 && response.status < 400) {
+                if (logger.isDebug()) {
+                    const metadata = {
+                        method: args.method,
+                        url: redactUrl(url),
+                        statusCode: response.status,
+                        responseHeaders: redactHeaders(response.headers),
+                    };
+                    logger.debug("HTTP request succeeded", metadata);
+                }
                 return {
                     ok: true,
                     body: (yield getResponseBody(response, args.responseType)),
@@ -63,6 +216,15 @@ export function fetcherImpl(args) {
                 };
             }
             else {
+                if (logger.isError()) {
+                    const metadata = {
+                        method: args.method,
+                        url: redactUrl(url),
+                        statusCode: response.status,
+                        responseHeaders: redactHeaders(Object.fromEntries(response.headers.entries())),
+                    };
+                    logger.error("HTTP request failed with error status", metadata);
+                }
                 return {
                     ok: false,
                     error: {
@@ -75,7 +237,14 @@ export function fetcherImpl(args) {
             }
         }
         catch (error) {
-            if ((_a = args.abortSignal) === null || _a === void 0 ? void 0 : _a.aborted) {
+            if ((_c = args.abortSignal) === null || _c === void 0 ? void 0 : _c.aborted) {
+                if (logger.isError()) {
+                    const metadata = {
+                        method: args.method,
+                        url: redactUrl(url),
+                    };
+                    logger.error("HTTP request was aborted", metadata);
+                }
                 return {
                     ok: false,
                     error: {
@@ -86,6 +255,14 @@ export function fetcherImpl(args) {
                 };
             }
             else if (error instanceof Error && error.name === "AbortError") {
+                if (logger.isError()) {
+                    const metadata = {
+                        method: args.method,
+                        url: redactUrl(url),
+                        timeoutMs: args.timeoutMs,
+                    };
+                    logger.error("HTTP request timed out", metadata);
+                }
                 return {
                     ok: false,
                     error: {
@@ -95,6 +272,14 @@ export function fetcherImpl(args) {
                 };
             }
             else if (error instanceof Error) {
+                if (logger.isError()) {
+                    const metadata = {
+                        method: args.method,
+                        url: redactUrl(url),
+                        errorMessage: error.message,
+                    };
+                    logger.error("HTTP request failed with error", metadata);
+                }
                 return {
                     ok: false,
                     error: {
@@ -104,6 +289,14 @@ export function fetcherImpl(args) {
                     rawResponse: unknownRawResponse,
                 };
             }
+            if (logger.isError()) {
+                const metadata = {
+                    method: args.method,
+                    url: redactUrl(url),
+                    error: toJson(error),
+                };
+                logger.error("HTTP request failed with unknown error", metadata);
+            }
             return {
                 ok: false,
                 error: {

package/dist/esm/core/fetcher/getRequestBody.d.mts

@@ -1,7 +1,7 @@
 export declare namespace GetRequestBody {
     interface Args {
         body: unknown;
-        type: "json" | "file" | "bytes" | "other";
+        type: "json" | "file" | "bytes" | "form" | "other";
     }
 }
 export declare function getRequestBody({ body, type }: GetRequestBody.Args): Promise<BodyInit | undefined>;

package/dist/esm/core/fetcher/getRequestBody.mjs

@@ -8,8 +8,12 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
     });
 };
 import { toJson } from "../json.mjs";
+import { toQueryString } from "../url/qs.mjs";
 export function getRequestBody(_a) {
     return __awaiter(this, arguments, void 0, function* ({ body, type }) {
+        if (type === "form") {
+            return toQueryString(body, { arrayFormat: "repeat", encode: true });
+        }
         if (type.includes("json")) {
             return toJson(body);
         }

package/dist/esm/core/fetcher/makeRequest.d.mts

@@ -1 +1 @@
-export declare const makeRequest: (fetchFn: (url: string, init: RequestInit) => Promise<Response>, url: string, method: string, headers: Record<string, string>, requestBody: BodyInit | undefined, timeoutMs?: number, abortSignal?: AbortSignal, withCredentials?: boolean, duplex?: "half") => Promise<Response>;
+export declare const makeRequest: (fetchFn: (url: string, init: RequestInit) => Promise<Response>, url: string, method: string, headers: Headers | Record<string, string>, requestBody: BodyInit | undefined, timeoutMs?: number, abortSignal?: AbortSignal, withCredentials?: boolean, duplex?: "half") => Promise<Response>;

package/dist/esm/core/fetcher/makeRequest.mjs

@@ -10,14 +10,12 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 import { anySignal, getTimeoutSignal } from "./signals.mjs";
 export const makeRequest = (fetchFn, url, method, headers, requestBody, timeoutMs, abortSignal, withCredentials, duplex) => __awaiter(void 0, void 0, void 0, function* () {
     const signals = [];
-    // Add timeout signal
     let timeoutAbortId;
     if (timeoutMs != null) {
         const { signal, abortId } = getTimeoutSignal(timeoutMs);
         timeoutAbortId = abortId;
         signals.push(signal);
     }
-    // Add arbitrary signal
     if (abortSignal != null) {
         signals.push(abortSignal);
     }