Skip to content

Remove unused internal tRPC routes - #1142

Open
jasonhedman wants to merge 1 commit into
json/remove-obsolete-public-routesfrom
json/remove-unused-internal-trpc
Open

Remove unused internal tRPC routes#1142
jasonhedman wants to merge 1 commit into
json/remove-obsolete-public-routesfrom
json/remove-unused-internal-trpc

Conversation

@jasonhedman

@jasonhedman jasonhedman commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • remove nine mounted user and admin tRPC procedures with no remaining application callers
  • delete service methods and composer callback UI owned only by those procedures
  • preserve non-tRPC API behavior, invite-code wallet transfers, partner find-or-create, and the connected-wallet onramp flow
  • remove the now-orphaned server-wallet onramp input while leaving the historical database column unchanged

Validation

  • pnpm --filter @x402scan/app types:generate
  • pnpm --filter @x402scan/app types:check
  • pnpm --filter @x402scan/app test (22 files, 239 tests)
  • pnpm --filter @x402scan/app ui:check
  • pnpm --filter @x402scan/app ui:integrity
  • pnpm knip
  • pnpm check
  • independent review: clean

Stack

Depends on #1141.

@vercel

vercel Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated
x402scan Ready Ready Preview Sep 4, 2026 5:46pm UTC

Request Review

Delete mounted admin and user procedures with no remaining callers. Remove exclusively owned service and callback UI code while preserving non-tRPC API and shared wallet behavior.
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.

1 participant