fix: throwThroughErrorTypes skips Sentry capture (HI-02)

Errors matching Try.throwThroughErrorTypes are now skipped from
Reporter.report() (Sentry.captureException) in addition to being
thrown unwrapped. Breadcrumbs still recorded.

Tests in Try.test.ts assert captureException not called for both
sync and async throw-through paths.

Author: w01fgang

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

@@ -0,0 +1,5 @@
+---
+'@power-rent/try-catch': patch
+---
+
+Fix HI-02: errors matching `Try.throwThroughErrorTypes` are no longer reported to Sentry when `.report()` is set. Previously the error was sent to `Sentry.captureException` before being re-thrown unwrapped, contradicting the "throw-through" name. Breadcrumbs configured via `.breadcrumbs()` are still recorded.

src/__tests__/Try.test.ts

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

src/core/Try.ts

@@ -185,20 +185,22 @@ export class Try<
   }
 
   /**
-   * 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.
+   * 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.
    *
    * @param ignoreErrorTypes Array of error type names (error.name) to throw through
    *
    * @example
    * ```typescript
-   * // Configure to throw ValidationError and AuthError as-is
+   * // Configure to throw ValidationError and AuthError as-is (and skip Sentry)
    * Try.throwThroughErrorTypes(['ValidationError', 'AuthError']);
    *
-   * // Now these errors won't be wrapped:
+   * // Now these errors won't be wrapped or sent to Sentry:
    * await new Try(validateUser, userData)
-   *   .report('User validation failed') // ValidationError will be thrown as-is
+   *   .report('User validation failed') // ValidationError throws as-is, no captureException
    *   .unwrap();
    * ```
    */
@@ -487,18 +489,18 @@ export class Try<
     if (isPromiseLike<TryResult<TReturn>>(result)) {
       return result.then((resolved) => {
         if (!resolved.success) {
-          const shouldCapture = this.config.message;
+          const isThrowThrough = Try.ignoreErrorTypes.includes(
+            resolved.error.name,
+          );
+          const shouldCapture = this.config.message && !isThrowThrough;
 
           if (shouldCapture) {
             this.reportError(resolved.error);
           } else if (this.config.breadcrumbConfig) {
             this.addBreadcrumbsIfConfigured();
           }
 
-          if (
-            this.config.message &&
-            !Try.ignoreErrorTypes.includes(resolved.error.name)
-          ) {
+          if (this.config.message && !isThrowThrough) {
             const wrappedError = Try.defaultReporter.createWrappedError(
               resolved.error,
               this.config.message,
@@ -513,18 +515,16 @@ export class Try<
     }
 
     if (!result.success) {
-      const shouldCapture = this.config.message;
+      const isThrowThrough = Try.ignoreErrorTypes.includes(result.error.name);
+      const shouldCapture = this.config.message && !isThrowThrough;
 
       if (shouldCapture) {
         this.reportError(result.error);
       } else if (this.config.breadcrumbConfig) {
         this.addBreadcrumbsIfConfigured();
       }
 
-      if (
-        this.config.message &&
-        !Try.ignoreErrorTypes.includes(result.error.name)
-      ) {
+      if (this.config.message && !isThrowThrough) {
         const wrappedError = Try.defaultReporter.createWrappedError(
           result.error,
           this.config.message,