refactor: cull porto fork to only modules used by splits-connect and splits-teams#3
Draft
refactor: cull porto fork to only modules used by splits-connect and splits-teams#3
Conversation
…splits-teams Remove all unused code from the forked porto wallet library to minimize maintenance surface area. The upstream (ithacaxyz/porto) is abandoned and we are the only consumers. Removed: - All 16 apps (dialog, id, service, playground, etc.) - All 17 examples - Solidity contracts, Docker services, CI workflows, changesets - src/cli/, src/server/, src/react-native/, src/wagmi/, src/register/ - Unused viem leaf files (AccountActions, WalletActions, CapabilitiesSchema) - Dialog.ts dead code: authSession() (~215 lines), experimental_inline() (~100 lines) - remote/Hooks.ts (unused by either consumer) - ~32 unused generated contract ABIs - Unused deps: hono, expo-*, react-native, changeset, tsdown, wagmi CLI Kept (transitively required by core): - src/viem/ core files (Account, Key, RelayClient, RelayActions, ContractActions) - src/theme/Theme.ts (type-imported by Dialog, Messenger) - src/trusted-hosts.ts (runtime import in remote/Porto.ts) 993 files changed, ~142K lines deleted. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Cull the forked porto wallet library to only the modules used by splits-connect and splits-teams, reducing maintenance surface area for an abandoned upstream.
Details
src/cli/,src/server/,src/react-native/,src/wagmi/,src/register/authSession()(~215 lines),experimental_inline()(~100 lines), React Native importsremote/Hooks.ts(unused by either consumer)src/viem/core files (Account, Key, RelayClient, RelayActions, ContractActions) — transitively required by core internalssrc/theme/Theme.ts(type-imported by Dialog, Messenger)src/trusted-hosts.ts(runtime import in remote/Porto.ts)pnpm-workspace.yamlto single packagepackage.jsonscripts, exports, dependencies, and peer dependenciestsconfig.jsonreferencesporto/servermodule993 files changed, ~142K lines deleted.
Areas Touched
portoLibrary (src/)apps/) — removed entirelyexamples/) — removed entirelycontracts/) — removed entirelydocker/) — removed entirelypackage.json,tsconfig.json,pnpm-workspace.yaml,biome.json)Verification
pnpm build✅pnpm check:types✅pnpm build✅pnpm typecheck✅🤖 Generated with Claude Code