chore: remove unused liquidations code and storage migration

[starksama]

Summary

Cleanup of unused code after migrating to Monarch API's everLiquidated field.

Removed Files (-1,049 lines)

Liquidations (no longer needed - using everLiquidated from market metrics)

• src/hooks/queries/useMonarchLiquidationsQuery.ts — unused hook
• src/hooks/queries/useLiquidationsQuery.ts — deprecated fallback
• src/data-sources/morpho-api/liquidations.ts — unused data source
• src/data-sources/subgraph/liquidations.ts — unused data source
• app/api/monarch/liquidations/route.ts — unused API proxy

Storage Migration (migration period complete)

• src/components/StorageMigrator.tsx — one-time migration component
• src/utils/storage-migration.ts — migration utility

Updated Files

• app/layout.tsx — removed StorageMigrator component
• src/hooks/queries/useMarketMetricsQuery.ts — cleaned up deprecated comments

Verification

pnpm typecheck ✓
pnpm lint ✓

Summary by CodeRabbit

• Refactor

  • Removed the liquidations data query stack and its server proxy — liquidations data will no longer be fetched via the removed flows.
  • Removed the legacy one-time localStorage migration system and related migration utilities.
  • Adjusted data refresh cadence: market metrics and market data now refresh less frequently (longer stale times / reduced polling).

• Documentation

  • Updated documentation to reflect the revised data sources and polling cadence.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[vercel]

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

Project	Deployment	Review	Updated (UTC)
monarch	Ready	Preview, Comment	Jan 29, 2026 7:36am

[coderabbitai]

📝 Walkthrough

Walkthrough

Deletes liquidation-related data pipeline and storage migration code: removes a Next.js API route, multiple liquidations data-source modules and hooks, the StorageMigrator component, and the storage-migration utilities; documentation and a few query polling intervals updated.

Changes

Cohort / File(s)	Summary
Monarch API Route app/api/monarch/liquidations/route.ts	Removed the GET route handler that proxied /v1/liquidations to Monarch (API key validation, chain_id handling, fetch, error handling, and revalidation).
Liquidations Data Sources src/data-sources/morpho-api/liquidations.ts, src/data-sources/subgraph/liquidations.ts	Deleted Morpho GraphQL fetcher and Subgraph fetcher for liquidated market keys (pagination, types, aggregation, and error handling removed).
Liquidations Hooks src/hooks/queries/useLiquidationsQuery.ts, src/hooks/queries/useMonarchLiquidationsQuery.ts	Removed React Query hooks that aggregated liquidations across networks and the monarch-liquidations hook with its response type.
Storage Migration src/utils/storage-migration.ts, src/components/StorageMigrator.tsx	Removed the storage migration utilities, types, execution/orchestration logic, expiry handling, and the StorageMigrator React component that triggered migrations.
Layout & Minor Hook Docs/Timing app/layout.tsx, src/hooks/queries/useMarketMetricsQuery.ts, src/hooks/useMarketData.ts, docs/TECHNICAL_OVERVIEW.md	Removed StorageMigrator from layout; adjusted market metrics docs and useMarketMetricsQuery docs; changed stale/refetch timings (useMarketMetricsQuery refetch from 1m→5m; useMarketData stale/refetch 2m→30s).

Sequence Diagram(s)

(omitted)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• misc: simplify monarch API consumption #337: Overlaps removal of the /api/monarch/liquidations route and useMonarchLiquidationsQuery hook.
• refactor: liquidation context #263: Previously refactored or introduced useLiquidationsQuery which this PR deletes.
• fix: stale state #191: Modifies the same market-data hook area (timing/logging) that this PR adjusts.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main changes: removal of unused liquidations code and completed storage migration.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore/cleanup-unused-code

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/hooks/queries/useMarketMetricsQuery.ts (1)

128-130: Docstring says 15 minutes, code says 5 minutes.

The JSDoc claims "cached for 15 minutes" but staleTime is actually 5 minutes. Consider updating the comment to match.

Proposed fix

 /**
  * Fetches enhanced market metrics from the Monarch monitoring API.
- * Pre-fetched and cached for 15 minutes.
+ * Pre-fetched and cached for 5 minutes.