Skip to content

chore: remove unreachable code and dependencies - #229

Merged
Makyl merged 2 commits into
mainfrom
chore/remove-unreachable-code
Jul 30, 2026
Merged

chore: remove unreachable code and dependencies#229
Makyl merged 2 commits into
mainfrom
chore/remove-unreachable-code

Conversation

@Makyl

@Makyl Makyl commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

This PR removes unreachable SDK, example, tooling, analytics, and test-only code identified by the ponytail audit.

The cleanup deletes 2,167 lines across 76 files and removes 11 direct dependency entries without changing the supported public SDK surface or runtime behavior.

Changes

  • Remove the unreachable sponsored swap-bridge preparation path, its dedicated test, duplicate hook coverage, and stale architecture documentation.
  • Remove the unused EIP-5792 EOA executor, Ankr balance client, analytics aggregation layer, and synthetic performance-baseline subsystem with their tests.
  • Remove six unreferenced browser components and the orphaned tools ReportChains component.
  • Remove pass-through SDK, swap, ABI, constants, and utility barrels, redirecting live imports to their concrete modules.
  • Remove zero-reference helpers, constants, ABIs, internal types, and test-only wallet capability code.
  • Drop unused dependencies from the root, browser example, and tools manifests, then refresh their lockfiles.

Testing

  • npm run lint
  • npm run lint:deps
  • npm run typecheck
  • npm run typecheck:tests
  • npm test

Risk / Impact

  • No supported public SDK method, package-root export, or runtime flow changes.
  • Internal deep imports of deleted files would break, but repository-wide reachability and stale-reference scans found no remaining consumers.
  • The browser example and tools packages install fewer unused dependencies.
  • Tests tied exclusively to unreachable implementations were removed with those implementations; the remaining suite passes in full.

@vercel

vercel Bot commented Jul 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nexus-sdk Ready Ready Preview Jul 30, 2026 2:21pm

Request Review

shrinathprabhu
shrinathprabhu previously approved these changes Jul 30, 2026
@Makyl
Makyl merged commit 98c02ad into main Jul 30, 2026
3 checks passed
@Makyl
Makyl deleted the chore/remove-unreachable-code branch July 30, 2026 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants