Skip to content

feat: support web router#1

Merged
aui merged 31 commits into
mainfrom
feature/web-router
Jul 7, 2025
Merged

feat: support web router#1
aui merged 31 commits into
mainfrom
feature/web-router

Conversation

@aui

@aui aui commented Jun 25, 2025

Copy link
Copy Markdown

No description provided.

@changeset-bot

changeset-bot Bot commented Jun 25, 2025

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 30ed4dd

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
flags Minor
web-router-example Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

aui added 28 commits July 1, 2025 15:32
…dapters

- Add optional error message parameters to ReadonlyHeadersError and ReadonlyRequestCookiesError classes
- Update HeadersAdapter.seal() and RequestCookiesAdapter.seal() to accept custom error messages
- Implement framework-specific error messages in web-router and sveltekit
- Next.js continues to use original error messages with documentation links
- Web Router shows web-router-specific error messages
- SvelteKit shows sveltekit-specific error messages with SvelteKit documentation links
- Maintains backward compatibility and all existing tests pass

This resolves the issue where web-router and sveltekit users were seeing confusing Next.js documentation links in error messages.
- Remove default Next.js error messages from shared adapters
- Make error message parameter required in HeadersAdapter.seal() and RequestCookiesAdapter.seal()
- Each framework now explicitly provides its own error messages
- Web Router and SvelteKit builds will no longer contain Next.js documentation links
- Ensures complete separation of framework-specific error messaging

This addresses the concern about Next.js documentation links appearing in web-router build artifacts.
@aui aui force-pushed the feature/web-router branch from 5c821ae to afe4ef8 Compare July 7, 2025 02:32
@aui aui added this pull request to the merge queue Jul 7, 2025
Merged via the queue into main with commit b5bf551 Jul 7, 2025
7 checks passed
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.

1 participant