Skip to content

Conversation

@brc-dd
Copy link
Member

@brc-dd brc-dd commented Aug 3, 2025

No description provided.

@brc-dd brc-dd changed the title feat: new fx utils feat(utils): extended fetch - fx Aug 3, 2025
@brc-dd brc-dd marked this pull request as ready for review August 4, 2025 05:17
@brc-dd brc-dd requested a review from Copilot August 4, 2025 05:17
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces a new extended fetch utility called fx that provides advanced features like pooling, retries, timeouts, and schema validation. The implementation replaces and extends the existing fetch utilities with a more comprehensive API.

  • Introduces a new fx function with three main methods: fx() for single requests, fx.all() for concurrent batching, and fx.iter() for streaming results
  • Adds comprehensive error handling with custom FetchError and SchemaError classes
  • Updates Sentry integration to version 10.0.0 and improves documentation formatting

Reviewed Changes

Copilot reviewed 7 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/utils.ts Major refactor introducing the fx API with pooling, retries, and schema validation
tests/utils.test.ts Comprehensive test suite for the new fx functionality
src/sentry.ts Updates Sentry dependencies to v10.0.0 and improves documentation
src/router.ts Minor documentation formatting improvements
tests/router.test.ts Trivial test name capitalization fix
dev_deps.ts Adds testing dependencies for the new utils tests
deno.json Enables broadcast-channel unstable feature
Comments suppressed due to low confidence (4)

src/sentry.ts:67

  • Sentry version 10.0.0 may not exist. My knowledge cutoff is January 2025, and it's currently August 2025, so this version may have been released since then. However, please verify that version ^10.0.0 of @sentry/core is actually available before merging.
} from 'npm:@sentry/core@^10.0.0'

src/sentry.ts:84

  • Sentry version 10.0.0 may not exist. My knowledge cutoff is January 2025, and it's currently August 2025, so this version may have been released since then. However, please verify that version ^10.0.0 of @sentry/deno is actually available before merging.
} from 'npm:@sentry/deno@^10.0.0'

src/sentry.ts:86

  • Sentry version 10.0.0 may not exist. My knowledge cutoff is January 2025, and it's currently August 2025, so this version may have been released since then. However, please verify that version ^10.0.0 of @sentry/deno is actually available before merging.
export * from 'npm:@sentry/deno@^10.0.0'

dev_deps.ts:21

  • Zod version 4.0.14 may not exist. My knowledge cutoff is January 2025, and it's currently August 2025, so this version may have been released since then. However, please verify that version 4.0.14 of zod is actually available before merging.
export { z } from 'npm:zod@4.0.14'

@brc-dd brc-dd merged commit 79d55fa into main Aug 4, 2025
1 check passed
@brc-dd brc-dd deleted the fx branch August 4, 2025 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants