chore(extension): remove old code from extension (#613)

* remove edit route

* remove obsolete e2e tests

* clean up dependencies

* remove usage of fromLegacyConnection helper

* remove legacy route migration from extension

* increase type-safety

deployables/app/app/components/ModSelect.tsx

@@ -13,7 +13,7 @@ export interface Option {
 
 interface Props<Option = unknown, Multi extends boolean = boolean>
   extends SelectProps<false, Option, Multi> {
-  avatarAddress: string
+  avatarAddress: HexAddress
 }
 
 export function ModSelect<Multi extends boolean = boolean>({
@@ -47,7 +47,7 @@ export function ModSelect<Multi extends boolean = boolean>({
 
 type ValueProps = PropsWithChildren<{
   label: string
-  address: string
+  address: HexAddress
 }>
 
 const Value = ({ label, address, children }: ValueProps) => (

deployables/app/app/components/ZodiacMod.tsx

@@ -117,7 +117,7 @@ export const ZodiacMod = ({
         }
         isDisabled={disabled || isLoading}
         placeholder={isLoading ? 'Loading modules...' : 'Select a module'}
-        avatarAddress={avatar}
+        avatarAddress={parsePrefixedAddress(avatar)}
       />
 
       {selectedModule?.type === SupportedZodiacModuleType.ROLES_V1 && (

deployables/app/app/components/wallet/Account.tsx

@@ -8,7 +8,7 @@ type AccountProps = {
 }
 
 export const Account = ({ children, type }: AccountProps) => {
-  const address = validateAddress(children) ?? ''
+  const address = validateAddress(children)
 
   return (
     <AddressInput

deployables/extension/package.json

@@ -28,14 +28,9 @@
   },
   "devDependencies": {
     "@depay/web3-mock": "^14.19.1",
-    "@gnosis.pm/zodiac": "^4.0.3",
-    "@noble/hashes": "^1.4.0",
     "@playwright/test": "^1.48.1",
-    "@safe-global/api-kit": "2.5.7",
-    "@safe-global/protocol-kit": "5.2.0",
-    "@safe-global/safe-apps-sdk": "9.1.0",
     "@safe-global/safe-gateway-typescript-sdk": "3.22.7",
-    "@shazow/whatsabi": "^0.19.0",
+    "@sentry/esbuild-plugin": "^3.0.0",
     "@testing-library/dom": "10.4.0",
     "@testing-library/jest-dom": "6.6.3",
     "@testing-library/react": "16.2.0",
@@ -46,53 +41,41 @@
     "@types/react": "19.0.7",
     "@types/react-dom": "19.0.3",
     "@vitest/coverage-v8": "2.1.8",
-    "@walletconnect/core": "2.17.3",
-    "@walletconnect/ethereum-provider": "2.17.3",
-    "@walletconnect/keyvaluestorage": "^1.1.1",
-    "@walletconnect/modal": "2.7.0",
-    "@walletconnect/sign-client": "2.17.3",
-    "@walletconnect/universal-provider": "2.17.3",
     "@zodiac/eslint-config": "workspace:*",
     "@zodiac/tailwind-config": "workspace:*",
     "@zodiac/test-utils": "workspace:*",
     "@zodiac/typescript-config": "workspace:*",
     "autoprefixer": "^10.4.20",
     "chalk": "^5.4.1",
-    "classnames": "^2.3.1",
-    "copy-to-clipboard": "^3.3.1",
     "dotenv": "^16.0.1",
     "esbuild": "^0.24.0",
     "esbuild-style-plugin": "^1.6.3",
     "eslint": "^9.7.0",
-    "ethereum-blockies-base64": "^1.0.2",
     "ethers": "6.13.5",
     "events": "^3.3.0",
-    "evm-proxy-detection": "^2.0.1",
     "globals": "^15.8.0",
     "happy-dom": "^16.0.0",
-    "isomorphic-fetch": "^3.0.0",
     "jsdom-testing-mocks": "^1.13.1",
     "madge": "^8.0.0",
-    "nanoid": "^5.0.0",
-    "node-stdlib-browser": "^1.2.0",
     "postcss": "^8.4.47",
-    "react": "^19.0.0",
-    "react-dom": "^19.0.0",
-    "react-select": "5.9.0",
     "rimraf": "6.0.1",
-    "ser-kit": "1.1.0",
     "tailwindcss": "^3.4.14",
     "typescript": "^5.5.4",
     "vitest": "2.1.8",
-    "vitest-chrome": "0.1.0",
-    "zodiac-roles-deployments": "^2.2.2"
+    "vitest-chrome": "0.1.0"
   },
   "packageManager": "[email protected]",
   "dependencies": {
     "@epic-web/invariant": "^1.0.0",
+    "@gnosis.pm/zodiac": "^4.0.3",
     "@headlessui/react": "^2.2.0",
-    "@sentry/esbuild-plugin": "^3.0.0",
     "@sentry/browser": "^8.50.0",
+    "@walletconnect/core": "2.17.3",
+    "@walletconnect/ethereum-provider": "2.17.3",
+    "@walletconnect/keyvaluestorage": "^1.1.1",
+    "@walletconnect/modal": "2.7.0",
+    "@walletconnect/sign-client": "2.17.3",
+    "@walletconnect/universal-provider": "2.17.3",
     "@zodiac/chains": "workspace:*",
     "@zodiac/env": "workspace:*",
     "@zodiac/form-data": "workspace:*",
@@ -101,11 +84,16 @@
     "@zodiac/safe": "workspace:*",
     "@zodiac/schema": "workspace:*",
     "@zodiac/ui": "workspace:*",
+    "classnames": "^2.3.1",
     "date-fns": "^4.1.0",
     "lucide-react": "^0.470.0",
+    "nanoid": "^5.0.0",
+    "node-stdlib-browser": "^1.2.0",
+    "react": "^19.0.0",
+    "react-dom": "^19.0.0",
     "react-router": "7.1.3",
     "react-stick": "^5.0.6",
-    "react-toastify": "11.0.3",
+    "ser-kit": "1.1.0",
     "zod": "^3.23.8"
   }
 }

deployables/extension/src/panel/app.tsx

@@ -3,10 +3,10 @@
 import { ProvideConnectProvider, ProvideInjectedWallet } from '@/providers'
 import { sentry } from '@/sentry'
 import { invariant } from '@epic-web/invariant'
+import { ToastContainer } from '@zodiac/ui'
 import { StrictMode, useEffect } from 'react'
 import { createRoot } from 'react-dom/client'
 import { createHashRouter, RouterProvider } from 'react-router'
-import { ToastContainer } from 'react-toastify'
 import '../global.css'
 import { pages } from './pages'
 import { ProvidePort } from './port-handling'

deployables/extension/src/panel/integrations/zodiac/index.ts

@@ -2,8 +2,5 @@ export {
   MULTISEND,
   MULTISEND_CALL_ONLY,
   SupportedZodiacModuleType,
-  queryRolesV2MultiSend,
 } from '@zodiac/modules'
 export type { ZodiacModule } from '@zodiac/modules'
-export { fetchZodiacModules } from './fetchZodiacModules'
-export { queryRolesV1MultiSend } from './rolesMultisend'

deployables/extension/src/panel/pages/$activeRouteId/ActiveRoute.tsx

@@ -11,6 +11,8 @@ import {
 } from '@/providers'
 import { ProvideProvider } from '@/providers-ui'
 import { formData, getString } from '@/utils'
+import { ZERO_ADDRESS } from '@zodiac/chains'
+import { updatePilotAddress } from '@zodiac/modules'
 import {
   Outlet,
   redirect,
@@ -20,10 +22,6 @@ import {
   type LoaderFunctionArgs,
 } from 'react-router'
 import { saveStorageEntry } from '../../utils/saveStorageEntry'
-import {
-  asLegacyConnection,
-  fromLegacyConnection,
-} from '../legacyConnectionMigrations'
 import { getActiveRouteId } from './getActiveRouteId'
 import { Intent } from './intents'
 
@@ -51,12 +49,7 @@ export const action = async ({ request, params }: ActionFunctionArgs) => {
       const routeId = getActiveRouteId(params)
       const route = await getRoute(routeId)
 
-      await saveRoute(
-        fromLegacyConnection({
-          ...asLegacyConnection(route),
-          pilotAddress: '',
-        }),
-      )
+      await saveRoute(updatePilotAddress(route, ZERO_ADDRESS))
 
       return null
     }