agent-relay 4.0.32 → 4.0.34

package/node_modules/@smithy/node-http-handler/dist-es/node-http2-handler.js

@@ -36,12 +36,14 @@ export class NodeHttp2Handler {
     async handle(request, { abortSignal, requestTimeout, isEventStream } = {}) {
         if (!this.config) {
             this.config = await this.configProvider;
-            this.connectionManager.setDisableConcurrentStreams(this.config.disableConcurrentStreams ?? false);
-            if (this.config.maxConcurrentStreams) {
-                this.connectionManager.setMaxConcurrentStreams(this.config.maxConcurrentStreams);
+            const { disableConcurrentStreams, maxConcurrentStreams } = this.config;
+            this.connectionManager.setDisableConcurrentStreams(disableConcurrentStreams ?? false);
+            if (maxConcurrentStreams) {
+                this.connectionManager.setMaxConcurrentStreams(maxConcurrentStreams);
             }
         }
         const { requestTimeout: configRequestTimeout, disableConcurrentStreams } = this.config;
+        const useIsolatedSession = disableConcurrentStreams || isEventStream;
         const effectiveRequestTimeout = requestTimeout ?? configRequestTimeout;
         return new Promise((_resolve, _reject) => {
             let fulfilled = false;
@@ -69,13 +71,17 @@ export class NodeHttp2Handler {
             }
             const authority = `${protocol}//${auth}${hostname}${port ? `:${port}` : ""}`;
             const requestContext = { destination: new URL(authority) };
-            const session = this.connectionManager.lease(requestContext, {
+            const connectConfig = {
                 requestTimeout: this.config?.sessionTimeout,
                 isEventStream,
-            });
+            };
+            const ref = useIsolatedSession
+                ? this.connectionManager.createIsolatedSession(requestContext, connectConfig)
+                : this.connectionManager.lease(requestContext, connectConfig);
+            const session = ref.deref();
             const rejectWithDestroy = (err) => {
-                if (disableConcurrentStreams) {
-                    this.destroySession(session);
+                if (useIsolatedSession) {
+                    ref.destroy();
                 }
                 fulfilled = true;
                 reject(err);
@@ -88,28 +94,14 @@ export class NodeHttp2Handler {
             if (request.fragment) {
                 path += `#${request.fragment}`;
             }
-            const req = session.request({
+            const clientHttp2Stream = session.request({
                 ...request.headers,
                 [constants.HTTP2_HEADER_PATH]: path,
                 [constants.HTTP2_HEADER_METHOD]: method,
             });
-            session.ref();
-            req.on("response", (headers) => {
-                const httpResponse = new HttpResponse({
-                    statusCode: headers[":status"] ?? -1,
-                    headers: getTransformedHeaders(headers),
-                    body: req,
-                });
-                fulfilled = true;
-                resolve({ response: httpResponse });
-                if (disableConcurrentStreams) {
-                    session.close();
-                    this.connectionManager.deleteSession(authority, session);
-                }
-            });
             if (effectiveRequestTimeout) {
-                req.setTimeout(effectiveRequestTimeout, () => {
-                    req.close();
+                clientHttp2Stream.setTimeout(effectiveRequestTimeout, () => {
+                    clientHttp2Stream.close();
                     const timeoutError = new Error(`Stream timed out because of no activity for ${effectiveRequestTimeout} ms`);
                     timeoutError.name = "TimeoutError";
                     rejectWithDestroy(timeoutError);
@@ -117,36 +109,50 @@ export class NodeHttp2Handler {
             }
             if (abortSignal) {
                 const onAbort = () => {
-                    req.close();
+                    clientHttp2Stream.close();
                     const abortError = buildAbortError(abortSignal);
                     rejectWithDestroy(abortError);
                 };
                 if (typeof abortSignal.addEventListener === "function") {
                     const signal = abortSignal;
                     signal.addEventListener("abort", onAbort, { once: true });
-                    req.once("close", () => signal.removeEventListener("abort", onAbort));
+                    clientHttp2Stream.once("close", () => signal.removeEventListener("abort", onAbort));
                 }
                 else {
                     abortSignal.onabort = onAbort;
                 }
             }
-            req.on("frameError", (type, code, id) => {
+            clientHttp2Stream.on("frameError", (type, code, id) => {
                 rejectWithDestroy(new Error(`Frame type id ${type} in stream id ${id} has failed with code ${code}.`));
             });
-            req.on("error", rejectWithDestroy);
-            req.on("aborted", () => {
-                rejectWithDestroy(new Error(`HTTP/2 stream is abnormally aborted in mid-communication with result code ${req.rstCode}.`));
+            clientHttp2Stream.on("error", rejectWithDestroy);
+            clientHttp2Stream.on("aborted", () => {
+                rejectWithDestroy(new Error(`HTTP/2 stream is abnormally aborted in mid-communication with result code ${clientHttp2Stream.rstCode}.`));
+            });
+            clientHttp2Stream.on("response", (headers) => {
+                const httpResponse = new HttpResponse({
+                    statusCode: headers[":status"] ?? -1,
+                    headers: getTransformedHeaders(headers),
+                    body: clientHttp2Stream,
+                });
+                fulfilled = true;
+                resolve({ response: httpResponse });
+                if (useIsolatedSession) {
+                    session.close();
+                }
             });
-            req.on("close", () => {
-                session.unref();
-                if (disableConcurrentStreams) {
-                    session.destroy();
+            clientHttp2Stream.on("close", () => {
+                if (useIsolatedSession) {
+                    ref.destroy();
+                }
+                else {
+                    this.connectionManager.release(requestContext, ref);
                 }
                 if (!fulfilled) {
                     rejectWithDestroy(new Error("Unexpected error: http2 request did not get a response"));
                 }
             });
-            writeRequestBodyPromise = writeRequestBody(req, request, effectiveRequestTimeout);
+            writeRequestBodyPromise = writeRequestBody(clientHttp2Stream, request, effectiveRequestTimeout);
         });
     }
     updateHttpClientConfig(key, value) {
@@ -161,9 +167,4 @@ export class NodeHttp2Handler {
     httpHandlerConfigs() {
         return this.config ?? {};
     }
-    destroySession(session) {
-        if (!session.destroyed) {
-            session.destroy();
-        }
-    }
 }

package/node_modules/@smithy/node-http-handler/dist-types/http2/ClientHttp2SessionRef.d.ts

@@ -0,0 +1,42 @@
+import type { ClientHttp2Session } from "node:http2";
+/**
+ * Shared access ref counter for ClientHttp2Session, where owners are
+ * in-flight requests.
+ *
+ * @internal
+ * @since 4.6.0
+ */
+export declare class ClientHttp2SessionRef {
+    readonly id: number;
+    /**
+     * Total calls to retain for this session.
+     */
+    total: number;
+    /**
+     * Max ref count observed.
+     */
+    max: number;
+    private readonly session;
+    private refs;
+    constructor(session: ClientHttp2Session);
+    /**
+     * Signal that the session is entering a request span and has an additional owning request.
+     * This must be called when beginning a request using the session.
+     */
+    retain(): void;
+    /**
+     * Release reference to session, to be called when it exits request span, indicating one fewer owning request.
+     * When reaching zero, the session is unref'd.
+     * This must be called when concluding a request using the session.
+     */
+    free(): void;
+    /**
+     * Access the session (don't call ref/unref on it).
+     */
+    deref(): ClientHttp2Session;
+    destroy(): void;
+    /**
+     * @returns the current number of active references (in-flight requests).
+     */
+    useCount(): number;
+}

package/node_modules/@smithy/node-http-handler/dist-types/node-http2-connection-manager.d.ts

@@ -1,24 +1,44 @@
-import type { RequestContext } from "@smithy/types";
-import type { ConnectConfiguration } from "@smithy/types";
-import type { ConnectionManager, ConnectionManagerConfiguration } from "@smithy/types";
-import type { ClientHttp2Session } from "node:http2";
+import type { ConnectConfiguration, ConnectionManager, ConnectionManagerConfiguration, RequestContext } from "@smithy/types";
+import { ClientHttp2SessionRef } from "./http2/ClientHttp2SessionRef";
 /**
- * @public
+ * This class previously implemented the ConnectionManager<ClientHttp2Session> interface,
+ * but this class isn't exported from this package, except as a private property of NodeHttp2Handler.
+ *
+ * @since 4.6.0
+ * @internal
  */
-export declare class NodeHttp2ConnectionManager implements ConnectionManager<ClientHttp2Session> {
-    constructor(config: ConnectionManagerConfiguration);
+export declare class NodeHttp2ConnectionManager implements ConnectionManager<ClientHttp2SessionRef> {
     private config;
-    private readonly sessionCache;
-    lease(requestContext: RequestContext, connectionConfiguration: ConnectConfiguration): ClientHttp2Session;
+    private readonly connectionPools;
+    constructor(config: ConnectionManagerConfiguration);
+    /**
+     * Acquire a session for making a request.
+     */
+    lease(requestContext: RequestContext, connectionConfiguration: ConnectConfiguration): ClientHttp2SessionRef;
+    /**
+     * Signal that a request using this session has completed.
+     *
+     * The session remains in its pool for reuse.
+     * This method is not called for isolated sessions.
+     */
+    release(_requestContext: RequestContext, ref: ClientHttp2SessionRef): void;
     /**
-     * Delete a session from the connection pool.
-     * @param authority The authority of the session to delete.
-     * @param session The session to delete.
+     * Create an isolated session that isn't part of the connection pools.
+     * For use in event-streams or when concurrency is turned off.
      */
-    deleteSession(authority: string, session: ClientHttp2Session): void;
-    release(requestContext: RequestContext, session: ClientHttp2Session): void;
+    createIsolatedSession(requestContext: RequestContext, connectionConfiguration: ConnectConfiguration): ClientHttp2SessionRef;
     destroy(): void;
     setMaxConcurrentStreams(maxConcurrentStreams: number): void;
     setDisableConcurrentStreams(disableConcurrentStreams: boolean): void;
+    /**
+     * @internal
+     * @returns a snapshot of the state of all connection pools and their sessions.
+     */
+    debug(): Record<string, any>;
+    /**
+     * Remove a session from the pools. Does not destroy it.
+     */
+    private removeFromPool;
+    private getPool;
     private getUrlString;
 }

package/node_modules/@smithy/node-http-handler/dist-types/node-http2-connection-pool.d.ts

@@ -1,12 +1,36 @@
 import type { ConnectionPool } from "@smithy/types";
 import type { ClientHttp2Session } from "node:http2";
-export declare class NodeHttp2ConnectionPool implements ConnectionPool<ClientHttp2Session> {
-    private sessions;
+import { ClientHttp2SessionRef } from "./http2/ClientHttp2SessionRef";
+/**
+ * These are keyed by URL, therefore all sessions within this class' state
+ * are for the same URL.
+ *
+ * Sessions remain in the pool for their entire lifetime (until destroyed or
+ * removed). The pool tracks capacity via each session's ref count.
+ *
+ * Interface implementation changed from ConnectionPool<ClientHttp2Session>.
+ * @since 4.6.0
+ * @internal
+ */
+export declare class NodeHttp2ConnectionPool implements ConnectionPool<ClientHttp2SessionRef> {
+    private readonly sessions;
+    private maxConcurrency;
     constructor(sessions?: ClientHttp2Session[]);
-    poll(): ClientHttp2Session | void;
-    offerLast(session: ClientHttp2Session): void;
-    contains(session: ClientHttp2Session): boolean;
-    remove(session: ClientHttp2Session): void;
-    [Symbol.iterator](): ArrayIterator<ClientHttp2Session>;
-    destroy(connection: ClientHttp2Session): void;
+    /**
+     * Find a session with available capacity (refs < maxConcurrency).
+     * Returns undefined if all sessions are at capacity or the pool is empty.
+     */
+    poll(): ClientHttp2SessionRef | undefined;
+    /**
+     * Add a session to the pool.
+     */
+    offerLast(ref: ClientHttp2SessionRef): void;
+    remove(ref: ClientHttp2SessionRef): void;
+    [Symbol.iterator](): ArrayIterator<ClientHttp2SessionRef>;
+    setMaxConcurrency(maxConcurrency: number): void;
+    /**
+     * This is unused, but part of the interface.
+     * @deprecated
+     */
+    destroy(ref: ClientHttp2SessionRef): void;
 }

package/node_modules/@smithy/node-http-handler/dist-types/node-http2-handler.d.ts

@@ -67,10 +67,5 @@ export declare class NodeHttp2Handler implements HttpHandler<NodeHttp2HandlerOpt
     }>;
     updateHttpClientConfig(key: keyof NodeHttp2HandlerOptions, value: NodeHttp2HandlerOptions[typeof key]): void;
     httpHandlerConfigs(): NodeHttp2HandlerOptions;
-    /**
-     * Destroys a session.
-     * @param session - the session to destroy.
-     */
-    private destroySession;
 }
 export {};

package/node_modules/@smithy/node-http-handler/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smithy/node-http-handler",
-  "version": "4.5.3",
+  "version": "4.6.0",
   "description": "Provides a way to make requests",
   "scripts": {
     "build": "concurrently 'yarn:build:types' 'yarn:build:es:cjs'",

package/node_modules/@smithy/service-error-classification/dist-cjs/index.js

@@ -56,6 +56,7 @@ const isTransientError = (error, depth = 0) => isRetryableByTrait(error) ||
     NODEJS_NETWORK_ERROR_CODES.includes(error?.code || "") ||
     TRANSIENT_ERROR_STATUS_CODES.includes(error.$metadata?.httpStatusCode || 0) ||
     isBrowserNetworkError(error) ||
+    isNodeJsHttp2TransientError(error) ||
     (error.cause !== undefined && depth <= 10 && isTransientError(error.cause, depth + 1));
 const isServerError = (error) => {
     if (error.$metadata?.httpStatusCode !== undefined) {
@@ -67,10 +68,14 @@ const isServerError = (error) => {
     }
     return false;
 };
+function isNodeJsHttp2TransientError(error) {
+    return error.code === "ERR_HTTP2_STREAM_ERROR" && error.message.includes("NGHTTP2_REFUSED_STREAM");
+}
 
 exports.isBrowserNetworkError = isBrowserNetworkError;
 exports.isClockSkewCorrectedError = isClockSkewCorrectedError;
 exports.isClockSkewError = isClockSkewError;
+exports.isNodeJsHttp2TransientError = isNodeJsHttp2TransientError;
 exports.isRetryableByTrait = isRetryableByTrait;
 exports.isServerError = isServerError;
 exports.isThrottlingError = isThrottlingError;

package/node_modules/@smithy/service-error-classification/dist-es/index.js

@@ -26,6 +26,7 @@ export const isTransientError = (error, depth = 0) => isRetryableByTrait(error)
     NODEJS_NETWORK_ERROR_CODES.includes(error?.code || "") ||
     TRANSIENT_ERROR_STATUS_CODES.includes(error.$metadata?.httpStatusCode || 0) ||
     isBrowserNetworkError(error) ||
+    isNodeJsHttp2TransientError(error) ||
     (error.cause !== undefined && depth <= 10 && isTransientError(error.cause, depth + 1));
 export const isServerError = (error) => {
     if (error.$metadata?.httpStatusCode !== undefined) {
@@ -37,3 +38,6 @@ export const isServerError = (error) => {
     }
     return false;
 };
+export function isNodeJsHttp2TransientError(error) {
+    return error.code === "ERR_HTTP2_STREAM_ERROR" && error.message.includes("NGHTTP2_REFUSED_STREAM");
+}

package/node_modules/@smithy/service-error-classification/dist-types/index.d.ts

@@ -22,3 +22,9 @@ export declare const isThrottlingError: (error: SdkError) => boolean;
  */
 export declare const isTransientError: (error: SdkError, depth?: number) => boolean;
 export declare const isServerError: (error: SdkError) => boolean;
+/**
+ * @internal
+ */
+export declare function isNodeJsHttp2TransientError(error: Error & {
+    code?: string;
+}): boolean;

package/node_modules/@smithy/service-error-classification/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smithy/service-error-classification",
-  "version": "4.2.14",
+  "version": "4.3.0",
   "scripts": {
     "build": "concurrently 'yarn:build:types' 'yarn:build:es:cjs'",
     "build:es:cjs": "yarn g:tsc -p tsconfig.es.json && node ../../scripts/inline service-error-classification",

package/node_modules/@smithy/smithy-client/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smithy/smithy-client",
-  "version": "4.12.11",
+  "version": "4.12.12",
   "scripts": {
     "build": "concurrently 'yarn:build:types' 'yarn:build:es:cjs'",
     "build:es:cjs": "yarn g:tsc -p tsconfig.es.json && node ../../scripts/inline smithy-client",
@@ -26,12 +26,12 @@
   "license": "Apache-2.0",
   "sideEffects": false,
   "dependencies": {
-    "@smithy/core": "^3.23.15",
-    "@smithy/middleware-endpoint": "^4.4.30",
+    "@smithy/core": "^3.23.16",
+    "@smithy/middleware-endpoint": "^4.4.31",
     "@smithy/middleware-stack": "^4.2.14",
     "@smithy/protocol-http": "^5.3.14",
     "@smithy/types": "^4.14.1",
-    "@smithy/util-stream": "^4.5.23",
+    "@smithy/util-stream": "^4.5.24",
     "tslib": "^2.6.2"
   },
   "engines": {

package/node_modules/@smithy/util-defaults-mode-browser/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smithy/util-defaults-mode-browser",
-  "version": "4.3.47",
+  "version": "4.3.48",
   "scripts": {
     "build": "concurrently 'yarn:build:types' 'yarn:build:es:cjs'",
     "build:es:cjs": "yarn g:tsc -p tsconfig.es.json && node ../../scripts/inline util-defaults-mode-browser",
@@ -24,7 +24,7 @@
   "sideEffects": false,
   "dependencies": {
     "@smithy/property-provider": "^4.2.14",
-    "@smithy/smithy-client": "^4.12.11",
+    "@smithy/smithy-client": "^4.12.12",
     "@smithy/types": "^4.14.1",
     "tslib": "^2.6.2"
   },

package/node_modules/@smithy/util-defaults-mode-node/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smithy/util-defaults-mode-node",
-  "version": "4.2.52",
+  "version": "4.2.53",
   "scripts": {
     "build": "concurrently 'yarn:build:types' 'yarn:build:es:cjs'",
     "build:es:cjs": "yarn g:tsc -p tsconfig.es.json && node ../../scripts/inline util-defaults-mode-node",
@@ -23,11 +23,11 @@
   "license": "Apache-2.0",
   "sideEffects": false,
   "dependencies": {
-    "@smithy/config-resolver": "^4.4.16",
+    "@smithy/config-resolver": "^4.4.17",
     "@smithy/credential-provider-imds": "^4.2.14",
     "@smithy/node-config-provider": "^4.3.14",
     "@smithy/property-provider": "^4.2.14",
-    "@smithy/smithy-client": "^4.12.11",
+    "@smithy/smithy-client": "^4.12.12",
     "@smithy/types": "^4.14.1",
     "tslib": "^2.6.2"
   },

package/node_modules/@smithy/util-endpoints/dist-cjs/index.js

@@ -318,50 +318,54 @@ const callFunction = ({ fn, argv }, options) => {
             evaluatedArgs[i] = group$2.evaluateExpression(arg, "arg", options);
         }
     }
-    if (fn.includes(".")) {
-        const fnSegments = fn.split(".");
-        if (fnSegments[0] in customEndpointFunctions && fnSegments[1] != null) {
-            return customEndpointFunctions[fnSegments[0]][fnSegments[1]](...evaluatedArgs);
+    const namespaceSeparatorIndex = fn.indexOf(".");
+    if (namespaceSeparatorIndex !== -1) {
+        const namespaceFunctions = customEndpointFunctions[fn.slice(0, namespaceSeparatorIndex)];
+        const customFunction = namespaceFunctions?.[fn.slice(namespaceSeparatorIndex + 1)];
+        if (typeof customFunction === "function") {
+            return customFunction(...evaluatedArgs);
         }
     }
-    if (typeof endpointFunctions[fn] !== "function") {
-        throw new Error(`function ${fn} not loaded in endpointFunctions.`);
-    }
     const callable = endpointFunctions[fn];
-    return callable(...evaluatedArgs);
+    if (typeof callable === "function") {
+        return callable(...evaluatedArgs);
+    }
+    throw new Error(`function ${fn} not loaded in endpointFunctions.`);
 };
 const group$2 = {
     evaluateExpression,
     callFunction,
 };
 
-const evaluateCondition = ({ assign, ...fnArgs }, options) => {
+const evaluateCondition = (condition, options) => {
+    const { assign } = condition;
     if (assign && assign in options.referenceRecord) {
         throw new EndpointError(`'${assign}' is already defined in Reference Record.`);
     }
-    const value = callFunction(fnArgs, options);
-    options.logger?.debug?.(`${debugId} evaluateCondition: ${toDebugString(fnArgs)} = ${toDebugString(value)}`);
-    return {
-        result: value === "" ? true : !!value,
-        ...(assign != null && { toAssign: { name: assign, value } }),
-    };
+    const value = callFunction(condition, options);
+    options.logger?.debug?.(`${debugId} evaluateCondition: ${toDebugString(condition)} = ${toDebugString(value)}`);
+    const result = value === "" ? true : !!value;
+    if (assign != null) {
+        return { result, toAssign: { name: assign, value } };
+    }
+    return { result };
 };
 
-const getEndpointHeaders = (headers, options) => Object.entries(headers ?? {}).reduce((acc, [headerKey, headerVal]) => ({
-    ...acc,
-    [headerKey]: headerVal.map((headerValEntry) => {
+const getEndpointHeaders = (headers, options) => Object.entries(headers ?? {}).reduce((acc, [headerKey, headerVal]) => {
+    acc[headerKey] = headerVal.map((headerValEntry) => {
         const processedExpr = evaluateExpression(headerValEntry, "Header value entry", options);
         if (typeof processedExpr !== "string") {
             throw new EndpointError(`Header '${headerKey}' value '${processedExpr}' is not a string`);
         }
         return processedExpr;
-    }),
-}), {});
+    });
+    return acc;
+}, {});
 
-const getEndpointProperties = (properties, options) => Object.entries(properties).reduce((acc, [propertyKey, propertyVal]) => ({
-    ...acc,
-    [propertyKey]: group$1.getEndpointProperty(propertyVal, options),
-}), {});
+const getEndpointProperties = (properties, options) => Object.entries(properties).reduce((acc, [propertyKey, propertyVal]) => {
+    acc[propertyKey] = group$1.getEndpointProperty(propertyVal, options);
+    return acc;
+}, {});
 const getEndpointProperty = (property, options) => {
     if (Array.isArray(property)) {
         return property.map((propertyEntry) => getEndpointProperty(propertyEntry, options));
@@ -438,23 +442,27 @@ const decideEndpoint = (bdd, options) => {
 
 const evaluateConditions = (conditions = [], options) => {
     const conditionsReferenceRecord = {};
+    const conditionOptions = {
+        ...options,
+        referenceRecord: { ...options.referenceRecord },
+    };
+    let didAssign = false;
     for (const condition of conditions) {
-        const { result, toAssign } = evaluateCondition(condition, {
-            ...options,
-            referenceRecord: {
-                ...options.referenceRecord,
-                ...conditionsReferenceRecord,
-            },
-        });
+        const { result, toAssign } = evaluateCondition(condition, conditionOptions);
         if (!result) {
             return { result };
         }
         if (toAssign) {
+            didAssign = true;
             conditionsReferenceRecord[toAssign.name] = toAssign.value;
+            conditionOptions.referenceRecord[toAssign.name] = toAssign.value;
             options.logger?.debug?.(`${debugId} assign: ${toAssign.name} := ${toDebugString(toAssign.value)}`);
         }
     }
-    return { result: true, referenceRecord: conditionsReferenceRecord };
+    if (didAssign) {
+        return { result: true, referenceRecord: conditionsReferenceRecord };
+    }
+    return { result: true };
 };
 
 const evaluateEndpointRule = (endpointRule, options) => {
@@ -463,21 +471,22 @@ const evaluateEndpointRule = (endpointRule, options) => {
     if (!result) {
         return;
     }
-    const endpointRuleOptions = {
-        ...options,
-        referenceRecord: { ...options.referenceRecord, ...referenceRecord },
-    };
+    const endpointRuleOptions = referenceRecord
+        ? {
+            ...options,
+            referenceRecord: { ...options.referenceRecord, ...referenceRecord },
+        }
+        : options;
     const { url, properties, headers } = endpoint;
     options.logger?.debug?.(`${debugId} Resolving endpoint from template: ${toDebugString(endpoint)}`);
-    return {
-        ...(headers != undefined && {
-            headers: getEndpointHeaders(headers, endpointRuleOptions),
-        }),
-        ...(properties != undefined && {
-            properties: getEndpointProperties(properties, endpointRuleOptions),
-        }),
-        url: getEndpointUrl(url, endpointRuleOptions),
-    };
+    const endpointToReturn = { url: getEndpointUrl(url, endpointRuleOptions) };
+    if (headers != null) {
+        endpointToReturn.headers = getEndpointHeaders(headers, endpointRuleOptions);
+    }
+    if (properties != null) {
+        endpointToReturn.properties = getEndpointProperties(properties, endpointRuleOptions);
+    }
+    return endpointToReturn;
 };
 
 const evaluateErrorRule = (errorRule, options) => {
@@ -486,10 +495,13 @@ const evaluateErrorRule = (errorRule, options) => {
     if (!result) {
         return;
     }
-    throw new EndpointError(evaluateExpression(error, "Error", {
-        ...options,
-        referenceRecord: { ...options.referenceRecord, ...referenceRecord },
-    }));
+    const errorRuleOptions = referenceRecord
+        ? {
+            ...options,
+            referenceRecord: { ...options.referenceRecord, ...referenceRecord },
+        }
+        : options;
+    throw new EndpointError(evaluateExpression(error, "Error", errorRuleOptions));
 };
 
 const evaluateRules = (rules, options) => {
@@ -521,10 +533,10 @@ const evaluateTreeRule = (treeRule, options) => {
     if (!result) {
         return;
     }
-    return group.evaluateRules(rules, {
-        ...options,
-        referenceRecord: { ...options.referenceRecord, ...referenceRecord },
-    });
+    const treeRuleOptions = referenceRecord
+        ? { ...options, referenceRecord: { ...options.referenceRecord, ...referenceRecord } }
+        : options;
+    return group.evaluateRules(rules, treeRuleOptions);
 };
 const group = {
     evaluateRules,

package/node_modules/@smithy/util-endpoints/dist-es/utils/evaluateCondition.js

@@ -1,14 +1,16 @@
 import { debugId, toDebugString } from "../debug";
 import { EndpointError } from "../types";
 import { callFunction } from "./callFunction";
-export const evaluateCondition = ({ assign, ...fnArgs }, options) => {
+export const evaluateCondition = (condition, options) => {
+    const { assign } = condition;
     if (assign && assign in options.referenceRecord) {
         throw new EndpointError(`'${assign}' is already defined in Reference Record.`);
     }
-    const value = callFunction(fnArgs, options);
-    options.logger?.debug?.(`${debugId} evaluateCondition: ${toDebugString(fnArgs)} = ${toDebugString(value)}`);
-    return {
-        result: value === "" ? true : !!value,
-        ...(assign != null && { toAssign: { name: assign, value } }),
-    };
+    const value = callFunction(condition, options);
+    options.logger?.debug?.(`${debugId} evaluateCondition: ${toDebugString(condition)} = ${toDebugString(value)}`);
+    const result = value === "" ? true : !!value;
+    if (assign != null) {
+        return { result, toAssign: { name: assign, value } };
+    }
+    return { result };
 };