Grouping exports breaks the devtools injection

[jozefmery]

Hi,

firstly, thank You for maintaining various react-router/remix related utilities.

I noticed that the devtools injection seems to assume a direct export style. Instead of direct exports, I group them at the end of the file for consistency reasons, but that seems to break the devtools plugin injection with the following error:

Identifier 'loader' has already been declared.

Here is a minimal stackblitz reproduction.

[AlemTuzlak]

Thank you, will look into this asap!

[AlemTuzlak]

#199 @jozefmery mind checking if this PR fixes it, you can install it via the pkg.pr.new command

[jozefmery]

Sure, I updated the reproduction, but unfortunately the behaviour remains unchanged.

[AlemTuzlak]

@jozefmery sorry, the fool that is me forgot that he had a minimum repro, fixed

[AlemTuzlak]

https://github.com/forge-42/react-router-devtools/releases/tag/v1.1.8

[jozefmery]

No worries! Unfortunately, the fix seems to have introduced another problem. If the exports are grouped in the root.tsx, the devtools are not injected at all. I updated the repro.

[AlemTuzlak]

Oof that one is a bit trickier, will see if I can fix it in a sensible way 🫡

[AlemTuzlak]

Fixed, will also either make a PR or help the react-router team also add this into react-router as it breaks the { loaderData } access in components