mikkel/nexus
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

refactor(nexus): remove redundant MemoryRouter re-export in router wrapper

Browse source 
Spec compliance review for Task 1 flagged that the explicit
`export { MemoryRouter } from "react-router-dom"` added in 332ed47b
is redundant — the line immediately above already does `export *`,
which includes MemoryRouter. The explicit line was cleanup debt from
defensive "just in case" reasoning, not an actual fix.

No behavioral change. IconRail.test.tsx still resolves MemoryRouter
via the export * and all 7 tests pass.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Nexus Dev 2026-04-11 10:56:22 +00:00
parent 332ed47bc0
commit efbd1e451e
1 changed files with 0 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
ui/src/lib/router.tsx
View file

@ -40,10 +40,6 @@ function useActiveCompanyPrefix(): string | null {
export * from "react-router-dom";
// Re-exported for tests only. Production code should not need MemoryRouter —
// the application wires BrowserRouter in main.tsx.
export { MemoryRouter } from "react-router-dom";
export const Link = React.forwardRef<HTMLAnchorElement, React.ComponentProps<typeof RouterDom.Link>>(
function CompanyLink({ to, ...props }, ref) {
const companyPrefix = useActiveCompanyPrefix();