Skip to content

Fold routes back into fetch-router#11388

Merged
mjackson merged 1 commit into
mainfrom
mjackson/fold-routes-into-fetch-router
May 12, 2026
Merged

Fold routes back into fetch-router#11388
mjackson merged 1 commit into
mainfrom
mjackson/fold-routes-into-fetch-router

Conversation

@mjackson

@mjackson mjackson commented May 12, 2026

Copy link
Copy Markdown
Member

This folds route definition helpers back into @remix-run/fetch-router under the existing routes package subpath while adding remix/routes as the canonical Remix facade import. This is part of #11384.

  • Moves the route map primitives, helpers, and tests into packages/fetch-router
  • Uses @remix-run/fetch-router/routes as the source for the generated remix/routes facade
  • Updates direct package imports and workspace dependencies to use the fetch-router-owned subpath
import { createRouter } from 'remix/fetch-router'
import { get, route } from 'remix/routes'

let routes = route({
  home: get('/'),
})

let router = createRouter()
router.get(routes.home, () => new Response('Home'))

@github-actions

Copy link
Copy Markdown
Contributor

Preview Build Available

A preview build has been created for this PR. You can install it using:

pnpm install "remix-run/remix#preview/pr-11388&path:packages/remix"

This preview build will be updated automatically as you push new commits.

@mjackson mjackson force-pushed the mjackson/fold-routes-into-fetch-router branch 2 times, most recently from b4ea6ed to 4ce35af Compare May 12, 2026 00:26
@mjackson mjackson force-pushed the mjackson/fold-routes-into-fetch-router branch from 4ce35af to 36600dc Compare May 12, 2026 00:30
@mjackson mjackson requested a review from brophdawg11 May 12, 2026 01:01
@mjackson mjackson merged commit 083ed08 into main May 12, 2026
27 checks passed
@mjackson mjackson deleted the mjackson/fold-routes-into-fetch-router branch May 12, 2026 01:03
@github-actions

Copy link
Copy Markdown
Contributor

The preview branch preview/pr-11388 has been deleted now that this PR is merged/closed.

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