Revert "fix: throwThroughErrorTypes skips Sentry capture (HI-02)"

This reverts commit 0273820.

Author: w01fgang

.changeset/throw-through-skips-sentry.md

@@ -406,7 +406,6 @@ describe('Try', () => {
         .unwrap();
 
       await expect(exec).rejects.toThrow('validation error');
-      expect(Sentry.captureException).not.toHaveBeenCalled();
     });
 
     it('should not give typescript error', async () => {
@@ -1243,7 +1242,6 @@ describe('Try', () => {
           .report('failed')
           .unwrap();
       }).toThrow('validation error');
-      expect(Sentry.captureException).not.toHaveBeenCalled();
     });
 
     it('should not give typescript error', () => {

src/__tests__/Try.test.ts

@@ -185,22 +185,20 @@ export class Try<
   }
 
   /**
-   * Configure error types that should be thrown through without being wrapped
-   * AND without being reported to the configured Reporter (Sentry).
-   * When using `.report()`, errors matching these types are re-thrown as-is
-   * and `Reporter.report()` is NOT called for them. Breadcrumbs configured via
-   * `.breadcrumbs()` are still recorded.
+   * Configure error types that should be thrown through without being wrapped.
+   * When using `.report()`, errors matching these types will be re-thrown as-is
+   * instead of being wrapped with the custom message.
    *
    * @param ignoreErrorTypes Array of error type names (error.name) to throw through
    *
    * @example
    * ```typescript
-   * // Configure to throw ValidationError and AuthError as-is (and skip Sentry)
+   * // Configure to throw ValidationError and AuthError as-is
    * Try.throwThroughErrorTypes(['ValidationError', 'AuthError']);
    *
-   * // Now these errors won't be wrapped or sent to Sentry:
+   * // Now these errors won't be wrapped:
    * await new Try(validateUser, userData)
-   *   .report('User validation failed') // ValidationError throws as-is, no captureException
+   *   .report('User validation failed') // ValidationError will be thrown as-is
    *   .unwrap();
    * ```
    */
@@ -489,18 +487,18 @@ export class Try<
     if (isPromiseLike<TryResult<TReturn>>(result)) {
       return result.then((resolved) => {
         if (!resolved.success) {
-          const isThrowThrough = Try.ignoreErrorTypes.includes(
-            resolved.error.name,
-          );
-          const shouldCapture = this.config.message && !isThrowThrough;
+          const shouldCapture = this.config.message;
 
           if (shouldCapture) {
             this.reportError(resolved.error);
           } else if (this.config.breadcrumbConfig) {
             this.addBreadcrumbsIfConfigured();
           }
 
-          if (this.config.message && !isThrowThrough) {
+          if (
+            this.config.message &&
+            !Try.ignoreErrorTypes.includes(resolved.error.name)
+          ) {
             const wrappedError = Try.defaultReporter.createWrappedError(
               resolved.error,
               this.config.message,
@@ -515,16 +513,18 @@ export class Try<
     }
 
     if (!result.success) {
-      const isThrowThrough = Try.ignoreErrorTypes.includes(result.error.name);
-      const shouldCapture = this.config.message && !isThrowThrough;
+      const shouldCapture = this.config.message;
 
       if (shouldCapture) {
         this.reportError(result.error);
       } else if (this.config.breadcrumbConfig) {
         this.addBreadcrumbsIfConfigured();
       }
 
-      if (this.config.message && !isThrowThrough) {
+      if (
+        this.config.message &&
+        !Try.ignoreErrorTypes.includes(result.error.name)
+      ) {
         const wrappedError = Try.defaultReporter.createWrappedError(
           result.error,
           this.config.message,