@zapier/kitcore 0.18.0 → 0.20.0

package/dist/index.mjs

@@ -643,6 +643,11 @@ function toIterable(source) {
   return { [Symbol.asyncIterator]: () => source[Symbol.asyncIterator]() };
 }
 
+// src/utils/promise-utils.ts
+function isPromiseLike(value) {
+  return value !== null && typeof value === "object" && typeof value.then === "function";
+}
+
 // src/utils/validation.ts
 var parseOrThrow = (schema, input, { adaptError } = {}) => {
   const result = schema.safeParse(input);
@@ -1111,7 +1116,7 @@ function createRawFunction(coreFn, options) {
       try {
         const parsed = schema ? validateOptions(schema, input, { adaptError }) : input;
         const result = coreFn(parsed, context);
-        if (result !== null && typeof result === "object" && typeof result.then === "function") {
+        if (isPromiseLike(result)) {
           return result.then(
             (value) => {
               fireEnd();
@@ -1893,8 +1898,9 @@ function declareOptionalMethod(config) {
         `Plugin "${id}" is an optional stand-in (declareOptionalMethod) with no implementation. Its binding is \`undefined\` unless a real plugin is registered under this id.`
       );
     }
-    // Requires nothing (phantom carrier `<never, never>`): a consumer that
-    // imports it still passes `createSdk`'s completeness check unprovided. The
+    // Requires nothing: a consumer that imports it still passes `createSdk`'s
+    // completeness check unprovided. The contract is still carried, so a
+    // provider that DOES appear under the id is checked against it. The
     // `optional: true` literal drives `PluginSurface` to type the binding
     // `| undefined`.
   };
@@ -1944,9 +1950,12 @@ function declareOptionalProperty(config) {
     optional: true,
     imports: [],
     importBindings: []
-    // Requires nothing (phantom carrier `<never, never>`): a consumer that
-    // imports it still passes `createSdk`'s completeness check unprovided. The
-    // import binding is still typed `TValue | undefined` from the descriptor.
+    // Requires nothing: a consumer that imports it still passes `createSdk`'s
+    // completeness check unprovided. The contract is the binding a consumer
+    // sees, `TValue | undefined`, which is also what a by-reference provider
+    // (`defineProperty(ref, { value })`) is allowed to pass. A consumer of an
+    // optional reference has to handle the absent case either way, so an
+    // explicit `undefined` breaks nothing a narrower contract would protect.
   };
 }
 function declareDefault({
@@ -2443,9 +2452,6 @@ function applyListOutputPolicy(page, policy) {
 var FRAMEWORK_CONFIGURATION_IDS = /* @__PURE__ */ new Set([
   CORE_OPTIONS_ID
 ]);
-function isPromiseLike(value) {
-  return value !== null && typeof value === "object" && typeof value.then === "function";
-}
 function normalizeOutput(output) {
   if (output === void 0) return { type: "raw" };
   if (typeof output === "string") return { type: output };
@@ -4932,6 +4938,7 @@ var DEFAULT_IDEMPOTENT_METHODS = [
   "TRACE"
 ];
 var BASE_BACKOFF_MILLISECONDS = 1e3;
+var MIN_BACKOFF_MILLISECONDS = 100;
 var JITTER_FACTOR = 0.5;
 var EPOCH_THRESHOLD_SECONDS = 1e9;
 function directedDelayMilliseconds(response) {
@@ -4956,6 +4963,12 @@ function backoffMilliseconds(attemptNumber) {
   const base = BASE_BACKOFF_MILLISECONDS * 2 ** (attemptNumber - 1);
   return base + Math.random() * JITTER_FACTOR * base;
 }
+function clampBackoffMilliseconds(attemptNumber, maxDelayMilliseconds) {
+  return Math.max(
+    Math.min(backoffMilliseconds(attemptNumber), maxDelayMilliseconds),
+    MIN_BACKOFF_MILLISECONDS
+  );
+}
 function abortReason(signal) {
   const reason = signal?.reason;
   return reason ?? new Error("The request was aborted.");
@@ -4990,6 +5003,20 @@ var retryHttpRequestPlugin = defineHook({
       const maxDelayMilliseconds = options.maxDelayMilliseconds ?? DEFAULT_MAX_DELAY_MILLISECONDS;
       const idempotentMethods = options.idempotentMethods ?? DEFAULT_IDEMPOTENT_METHODS;
       const statuses = isIdempotent(input.request, idempotentMethods) ? options.retryStatuses ?? DEFAULT_RETRY_STATUSES : options.nonIdempotentRetryStatuses ?? DEFAULT_NON_IDEMPOTENT_RETRY_STATUSES;
+      const notifyRetry = (notice) => {
+        try {
+          const observed = options.onRetry?.({
+            ...notice,
+            request: input.attempt.operation.request,
+            operationId: input.attempt.operation.operationId
+          });
+          if (isPromiseLike(observed)) {
+            void observed.then(void 0, () => {
+            });
+          }
+        } catch {
+        }
+      };
       for (let attemptNumber = 1; ; attemptNumber++) {
         const attempt = {
           ...input.attempt,
@@ -5011,10 +5038,16 @@ var retryHttpRequestPlugin = defineHook({
           )) {
             throw error;
           }
-          await sleep(
-            Math.min(backoffMilliseconds(attemptNumber), maxDelayMilliseconds),
-            attempt.signal
+          const errorDelay = clampBackoffMilliseconds(
+            attemptNumber,
+            maxDelayMilliseconds
           );
+          notifyRetry({
+            attemptNumber,
+            delayMilliseconds: errorDelay,
+            error
+          });
+          await sleep(errorDelay, attempt.signal);
           continue;
         }
         if (!statuses.includes(response.status) || !canRetry(
@@ -5027,10 +5060,8 @@ var retryHttpRequestPlugin = defineHook({
         const directed = directedDelayMilliseconds(response);
         if (directed != null && directed > maxDelayMilliseconds)
           return response;
-        const delay = Math.min(
-          directed ?? backoffMilliseconds(attemptNumber),
-          maxDelayMilliseconds
-        );
+        const delay = directed ?? clampBackoffMilliseconds(attemptNumber, maxDelayMilliseconds);
+        notifyRetry({ attemptNumber, delayMilliseconds: delay, response });
         await response.body?.cancel().catch(() => {
         });
         await sleep(delay, attempt.signal);