@zapier/kitcore 0.10.0 → 0.10.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @zapier/kitcore
 
+## 0.10.1
+
+### Patch Changes
+
+- 0e38143: Method-lifecycle observers (`onMethodStart` / `onMethodEnd`) are now isolated from the methods they observe. A throwing observer is logged and ignored instead of breaking the method call, and a throw in one observer no longer stops its siblings from running. Paginated methods also fire `onMethodEnd` in the same microtask the call settles, including when the first page fails, matching non-paginated methods.
+
 ## 0.10.0
 
 ### Minor Changes

package/dist/index.cjs

@@ -270,13 +270,34 @@ function buildRegistry({
 }
 
 // src/utils/build-hooks.ts
+var isolated = /* @__PURE__ */ new WeakSet();
+function isolate(observer) {
+  if (!observer) return void 0;
+  if (isolated.has(observer)) return observer;
+  const wrapped = (ctx) => {
+    try {
+      observer(ctx);
+    } catch (error) {
+      console.error(
+        "[core] A method-lifecycle observer threw and was ignored. Observers are fire-and-forget and must not throw.",
+        error
+      );
+    }
+  };
+  isolated.add(wrapped);
+  return wrapped;
+}
 function composeVoid(existing, added) {
-  if (!existing) return added;
-  if (!added) return existing;
-  return (ctx) => {
-    existing(ctx);
-    added(ctx);
+  const wrappedExisting = isolate(existing);
+  const wrappedAdded = isolate(added);
+  if (!wrappedExisting) return wrappedAdded;
+  if (!wrappedAdded) return wrappedExisting;
+  const composed = (ctx) => {
+    wrappedExisting(ctx);
+    wrappedAdded(ctx);
   };
+  isolated.add(composed);
+  return composed;
 }
 function buildHooks(existing, added) {
   const result = {};
@@ -1025,24 +1046,27 @@ function createPaginatedFunction(coreFn, options) {
             return result.value;
           });
           if (hooks?.onMethodEnd) {
-            firstPagePromise.then(() => {
-              hooks.onMethodEnd({
-                methodName: functionName,
-                args,
-                isPaginated: true,
-                depth,
-                durationMs: Date.now() - startTime
-              });
-            }).catch((error) => {
-              hooks.onMethodEnd({
-                methodName: functionName,
-                args,
-                isPaginated: true,
-                depth,
-                durationMs: Date.now() - startTime,
-                error: error instanceof Error ? error : new Error(String(error))
-              });
-            });
+            firstPagePromise.then(
+              () => {
+                hooks.onMethodEnd({
+                  methodName: functionName,
+                  args,
+                  isPaginated: true,
+                  depth,
+                  durationMs: Date.now() - startTime
+                });
+              },
+              (error) => {
+                hooks.onMethodEnd({
+                  methodName: functionName,
+                  args,
+                  isPaginated: true,
+                  depth,
+                  durationMs: Date.now() - startTime,
+                  error: error instanceof Error ? error : new Error(String(error))
+                });
+              }
+            );
           }
           const pageStream = async function* () {
             yield await firstPagePromise;