From efbd1e451e8de2b117bd33502921a84f8ebebc0a Mon Sep 17 00:00:00 2001 From: Nexus Dev Date: Sat, 11 Apr 2026 10:56:22 +0000 Subject: [PATCH] refactor(nexus): remove redundant MemoryRouter re-export in router wrapper MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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) --- ui/src/lib/router.tsx | 4 ---- 1 file changed, 4 deletions(-) diff --git a/ui/src/lib/router.tsx b/ui/src/lib/router.tsx index a0243a18..5cf81c8d 100644 --- a/ui/src/lib/router.tsx +++ b/ui/src/lib/router.tsx @@ -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>( function CompanyLink({ to, ...props }, ref) { const companyPrefix = useActiveCompanyPrefix();