perf(frontend): split Recharts into a lazy-loaded chart chunk

[Flickwire-Agent]

Summary

Lazy-load the sale history chart so Recharts is not required before the item page shell renders.

Evidence

• SaleHistoryChart imports Recharts statically at apps/web/src/components/SaleHistoryChart.tsx:2.
• SaleHistoryView imports the chart statically at apps/web/src/components/SaleHistoryView.tsx:5.

Expected Impact

Medium for cold item-detail loads.

Implementation Notes

Use React.lazy for SaleHistoryChart or configure Vite manual chunks for Recharts.

Acceptance Criteria

• Item detail page shell can render before the chart library chunk loads.
• Chart still renders behind a loading fallback.

[Flickwire-Agent]

Already implemented in commit 3d357dd (perf(web): optimize lighthouse performance). lazy-imports and renders it inside a boundary with the accessible "Loading chart..." status fallback. This keeps the item-detail shell renderable while the Recharts chunk loads. Verified on current main (74b8fee) and the live API health endpoint is healthy.

[Flickwire-Agent]

Correction to the closure evidence: apps/web/src/components/SaleHistoryView.tsx lazy-imports SaleHistoryChart with React.lazy, then renders it in a Suspense boundary whose fallback is the accessible Loading chart... status message. Both lines trace to commit 3d357dd (June 18). This satisfies the issue criteria without a new code change.