Support for Integrating Routes from Packages in React Router route manifest

[fernandojbf]

Hi,

I'm working on a project that includes shared routes/pages via packages. To achieve this, we made a solution that adds these routes dynamically via a routes.ts file.

Since React Router does not natively allow adding routes from outside the project, we implemented a workaround that matches the node_modules file via a relative path:

defineSharedRoute(route, '/x', '../node_modules');

This translates to:

route(
    `/x`,
    `node_modules/path-to-dist-file.js`,
    { id: 'id' }
);

This setup worked well with Yarn since the file paths resolved correctly.

The Problem

We are migrating to pnpm, and due to how viteManifest is created, the chunk names now point to the .pnpm/ folder instead of the node_modules package path. This breaks the build since our workaround depends on the node_modules structure and the chunk is search by let entryChunk = viteManifest[rootRelativeFilePath];.

My Question

Is there (or will be) a way to officially integrate routes from external packages into the React Router routes manifest? I understand that our current approach relies on implementation details that can change and may not be ideal. However, having built-in support for sharing routes across packages could be valuable.

We have the options to create local files and import/export the package. But if we had a way to use the manifest, the routes could be integrated from the package directly (multiple routes, etc)

Thanks!

[pawelblaszczyk5]

It’s not a direct solution to your problem but you can export routes definition from the package instead of this approach. It’ll be more bulletproof, package will be self contained and there’re some helpers like “prefix” and “relative” for this.

[fernandojbf]

@pawelblaszczyk5 The route definition needs the route file path to work right? The route path continues to be the problem

[pawelblaszczyk5]

I’m not fully sure whether I get what you mean here, could you elaborate? Take a look at example from this PR - #11966