Skip to content

chore: cleanup — remove dead code, indexer stack, stale docs and assets - #311

Merged
thisisouvik merged 1 commit into
mainfrom
chore/phase-0-cleanup
Sep 12, 2026
Merged

chore: cleanup — remove dead code, indexer stack, stale docs and assets#311
thisisouvik merged 1 commit into
mainfrom
chore/phase-0-cleanup

Conversation

@thisisouvik

Copy link
Copy Markdown
Owner

Remove everything that was tracked but not used, and tidy the repo root ahead of the Neon migration and UI redesign.

Removed

  • 8 unreferenced components (RoleDashboardScreen, AvailablePools, FinanceChart, WalletSetupCard, SorobanProfileCard, AuthAccessButton, SectionTitle, ui/table) and the duplicate root-level ErrorFallback/GlobalErrorBoundary (dashboard/ copies are the live ones)
  • 4 unreferenced lib modules (level5-feedback, pool-security-config and the two lib/config/*.config.ts duplicates)
  • SubQuery indexer stack: subquery/, schema.graphql, project.yaml, docker-compose.subquery.yml, lib/indexer, its test, the horizon_sync SQL and TRUSTLEND_INDEXER_* env vars. Dashboards now read straight from the database instead of trying the indexer first.
  • Dockerfile.dev / docker-compose.yml (Supabase-specific, unused on Vercel)
  • assets/ screenshots (22 PNGs; all stale once the UI is redesigned)
  • Obsolete docs: OPTIMIZATION_SUMMARY, POOL_OPTIMIZATION, QUICK_START_POOL_QUERIES, SOROBAN_INDEXER_MIGRATION
  • Unused deps: ethers, @albedo-link/intent, @subql/types-stellar

Moved

  • Technical docs from the repo root into docs/ (kebab-case), and the four contracts/*.md into docs/contracts/. Only README, CONTRIBUTING, CODE_OF_CONDUCT, SECURITY, CHANGELOG and LICENSE stay at the root. All references (README, scripts, env example, workflow, Rust doc comment) updated.

Fixed

  • .gitignore ignored /docs while docs/api.md was tracked
  • app/actions/admin-pools.ts had four duplicated lines from a bad merge, which broke tsc --noEmit on main
  • admin pools page did not pass borrow_cap through to the client (tsc error)
  • vitest was collecting the Playwright spec in e2e/

…and assets

Remove everything that was tracked but not used, and tidy the repo root
ahead of the Neon migration and UI redesign.

Removed
- 8 unreferenced components (RoleDashboardScreen, AvailablePools,
  FinanceChart, WalletSetupCard, SorobanProfileCard, AuthAccessButton,
  SectionTitle, ui/table) and the duplicate root-level
  ErrorFallback/GlobalErrorBoundary (dashboard/ copies are the live ones)
- 4 unreferenced lib modules (level5-feedback, pool-security-config and
  the two lib/config/*.config.ts duplicates)
- SubQuery indexer stack: subquery/, schema.graphql, project.yaml,
  docker-compose.subquery.yml, lib/indexer, its test, the
  horizon_sync SQL and TRUSTLEND_INDEXER_* env vars. Dashboards now
  read straight from the database instead of trying the indexer first.
- Dockerfile.dev / docker-compose.yml (Supabase-specific, unused on Vercel)
- assets/ screenshots (22 PNGs; all stale once the UI is redesigned)
- Obsolete docs: OPTIMIZATION_SUMMARY, POOL_OPTIMIZATION,
  QUICK_START_POOL_QUERIES, SOROBAN_INDEXER_MIGRATION
- Unused deps: ethers, @albedo-link/intent, @subql/types-stellar

Moved
- Technical docs from the repo root into docs/ (kebab-case), and the four
  contracts/*.md into docs/contracts/. Only README, CONTRIBUTING,
  CODE_OF_CONDUCT, SECURITY, CHANGELOG and LICENSE stay at the root.
  All references (README, scripts, env example, workflow, Rust doc
  comment) updated.

Fixed
- .gitignore ignored /docs while docs/api.md was tracked
- app/actions/admin-pools.ts had four duplicated lines from a bad merge,
  which broke `tsc --noEmit` on main
- admin pools page did not pass borrow_cap through to the client (tsc error)
- vitest was collecting the Playwright spec in e2e/

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 12, 2026

Copy link
Copy Markdown

Deployment failed for project trustlend-stellar with the following error:

Hobby accounts are limited to daily cron jobs. This cron expression (* * * * *) would run more than once per day. Upgrade to the Pro plan to unlock all Cron Jobs features on Vercel.

Learn More: https://vercel.link/3Fpeeb1

@thisisouvik
thisisouvik marked this pull request as ready for review September 12, 2026 11:32
@coderabbitai

coderabbitai Bot commented Sep 12, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@thisisouvik
thisisouvik merged commit 812fce9 into main Sep 12, 2026
7 of 11 checks passed
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