From 2bc659f48f97837b240bc2d453ac07053ddf53c0 Mon Sep 17 00:00:00 2001 From: Ryan Bahan Date: Thu, 2 Apr 2026 17:43:19 -0600 Subject: [PATCH] Remove react-router-dom, render Extensions directly The Routes component used a catch-all route (path="*") that always rendered . No actual routing was happening. Render the component directly in App.tsx and remove the Routes wrapper, its directory, and the react-router-dom dependency. Co-Authored-By: Claude Opus 4.6 (1M context) --- .../ui-extensions-dev-console/package.json | 1 - .../ui-extensions-dev-console/src/App.tsx | 4 +-- .../src/foundation/Routes/Routes.tsx | 13 ------- .../src/foundation/Routes/index.ts | 1 - pnpm-lock.yaml | 34 ------------------- 5 files changed, 2 insertions(+), 51 deletions(-) delete mode 100644 packages/ui-extensions-dev-console/src/foundation/Routes/Routes.tsx delete mode 100644 packages/ui-extensions-dev-console/src/foundation/Routes/index.ts diff --git a/packages/ui-extensions-dev-console/package.json b/packages/ui-extensions-dev-console/package.json index 46ed8f0b4f2..6c67cd419ba 100644 --- a/packages/ui-extensions-dev-console/package.json +++ b/packages/ui-extensions-dev-console/package.json @@ -18,7 +18,6 @@ "qrcode.react": "^4.2.0", "react": "^18.2.0", "react-dom": "^18.2.0", - "react-router-dom": "^6.14.2", "react-toastify": "^9.1.3", "react-transition-group": "^4.4.5" }, diff --git a/packages/ui-extensions-dev-console/src/App.tsx b/packages/ui-extensions-dev-console/src/App.tsx index d94d2667338..a2b346a1c16 100644 --- a/packages/ui-extensions-dev-console/src/App.tsx +++ b/packages/ui-extensions-dev-console/src/App.tsx @@ -1,5 +1,5 @@ import {Layout} from '@/foundation/Layout' -import {Routes} from '@/foundation/Routes' +import {Extensions} from '@/sections/Extensions' import {Toast} from '@/foundation/Toast' import {Theme} from '@/foundation/Theme' import {ModalContainer} from '@/foundation/ModalContainer' @@ -29,7 +29,7 @@ function App() { - + diff --git a/packages/ui-extensions-dev-console/src/foundation/Routes/Routes.tsx b/packages/ui-extensions-dev-console/src/foundation/Routes/Routes.tsx deleted file mode 100644 index 6dd59db7199..00000000000 --- a/packages/ui-extensions-dev-console/src/foundation/Routes/Routes.tsx +++ /dev/null @@ -1,13 +0,0 @@ -import {Extensions} from '@/sections/Extensions' -import React from 'react' -import {BrowserRouter, Routes as ReactRouterRoutes, Route} from 'react-router-dom' - -export function Routes() { - return ( - - - } /> - - - ) -} diff --git a/packages/ui-extensions-dev-console/src/foundation/Routes/index.ts b/packages/ui-extensions-dev-console/src/foundation/Routes/index.ts deleted file mode 100644 index 254d69e7060..00000000000 --- a/packages/ui-extensions-dev-console/src/foundation/Routes/index.ts +++ /dev/null @@ -1 +0,0 @@ -export * from './Routes' diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml index adceb575232..6b0c7b1cf40 100644 --- a/pnpm-lock.yaml +++ b/pnpm-lock.yaml @@ -693,9 +693,6 @@ importers: react-dom: specifier: ^18.2.0 version: 18.3.1(react@18.3.1) - react-router-dom: - specifier: ^6.14.2 - version: 6.30.3(react-dom@18.3.1(react@18.3.1))(react@18.3.1) react-toastify: specifier: ^9.1.3 version: 9.1.3(react-dom@18.3.1(react@18.3.1))(react@18.3.1) @@ -3346,10 +3343,6 @@ packages: '@protobufjs/utf8@1.1.0': resolution: {integrity: sha512-Vvn3zZrhQZkkBE8LSuW3em98c0FwgO4nxzv6OdSxPKJIEKY2bGbHn+mhGIPerzI4twdxaP8/0+06HBpwf345Lw==} - '@remix-run/router@1.23.2': - resolution: {integrity: sha512-Ic6m2U/rMjTkhERIa/0ZtXJP17QUi2CbWE7cqx4J58M8aA3QTfW+2UlQ4psvTX9IO1RfNVhK3pcpdjej7L+t2w==} - engines: {node: '>=14.0.0'} - '@repeaterjs/repeater@3.0.6': resolution: {integrity: sha512-Javneu5lsuhwNCryN+pXH93VPQ8g0dBX7wItHFgYiwQmzE1sVdg5tWHiOgHywzL2W21XQopa7IwIEnNbmeUJYA==} @@ -7645,19 +7638,6 @@ packages: resolution: {integrity: sha512-QgT5//D3jfjJb6Gsjxv0Slpj23ip+HtOpnNgnb2S5zU3CB26G/IDPGoy4RJB42wzFE46DRsstbW6tKHoKbhAxw==} engines: {node: '>=0.10.0'} - react-router-dom@6.30.3: - resolution: {integrity: sha512-pxPcv1AczD4vso7G4Z3TKcvlxK7g7TNt3/FNGMhfqyntocvYKj+GCatfigGDjbLozC4baguJ0ReCigoDJXb0ag==} - engines: {node: '>=14.0.0'} - peerDependencies: - react: '>=16.8' - react-dom: '>=16.8' - - react-router@6.30.3: - resolution: {integrity: sha512-XRnlbKMTmktBkjCLE8/XcZFlnHvr2Ltdr1eJX4idL55/9BbORzyZEaIkBFDhFGCEWBBItsVrDxwx3gnisMitdw==} - engines: {node: '>=14.0.0'} - peerDependencies: - react: '>=16.8' - react-toastify@9.1.3: resolution: {integrity: sha512-fPfb8ghtn/XMxw3LkxQBk3IyagNpF/LIKjOBflbexr2AWxAH1MJgvnESwEwBn9liLFXgTKWgBSdZpw9m4OTHTg==} peerDependencies: @@ -12478,8 +12458,6 @@ snapshots: '@protobufjs/utf8@1.1.0': {} - '@remix-run/router@1.23.2': {} - '@repeaterjs/repeater@3.0.6': {} '@rolldown/pluginutils@1.0.0-rc.3': {} @@ -17455,18 +17433,6 @@ snapshots: react-refresh@0.18.0: {} - react-router-dom@6.30.3(react-dom@18.3.1(react@18.3.1))(react@18.3.1): - dependencies: - '@remix-run/router': 1.23.2 - react: 18.3.1 - react-dom: 18.3.1(react@18.3.1) - react-router: 6.30.3(react@18.3.1) - - react-router@6.30.3(react@18.3.1): - dependencies: - '@remix-run/router': 1.23.2 - react: 18.3.1 - react-toastify@9.1.3(react-dom@18.3.1(react@18.3.1))(react@18.3.1): dependencies: clsx: 1.2.1