[Bug?]: File system routes bundle local paths in prod builds

[n-p-e]

Duplicates

• I have searched the existing issues

Latest version

• I have tested the latest version

Current behavior 😯

Currently the prod js builds created by vinxi build would include full absolute paths of routes like so

path: '/',
filePath: '/path/to/project/src/routes/index.tsx'
...
path: '/*404',
filePath: '/path/to/project/src/routes/[...404].tsx'
...

These don't affect functionality but might be a minor security risk if the path where you build the app somehow contains sensitive information.

The RPCs created by 'use server' also have similar issues. They produce a /path/to/src/routes/index.tsx?tsr-directive-use-server= string.

Expected behavior 🤔

Ideally the full paths don't get bundled in prod builds. filePath doesn't mean anything in browsers/ssr so changing the output to filePath: '/src/routes/[...404].tsx' shouldn't be harmful.

Steps to reproduce 🕹

Steps:

1. Create a starter project git clone git@github.com:solidjs/solid-start && cd solid-start/examples/basic && pnpm install
2. pnpm run build
3. grep -Horn "$PWD" .vinxi/build/client/_build

You should see some js files include your PWD. They get served in the final build.

Context 🔦

No response

Your environment 🌎

[brenelz]

I am guessing this will be addressed when we move on from vinxi

[brenelz]

This looks like a duplicate of this
#1896

[m1rn]

This looks like a duplicate of this #1896

@brenelz Hey! Sorry, just a little confused — the comment says this is a duplicate of #1896, but #1896 is this exact issue. Was there another issue you meant to link instead?

[brenelz]

Oops good catch. I think i meant to link #1858