Skip to content

feat(reports): add backtest report library#224

Merged
warren618 merged 1 commit into
HKUDS:mainfrom
LemonCANDY42:feat/backtest-report-library
Jun 21, 2026
Merged

feat(reports): add backtest report library#224
warren618 merged 1 commit into
HKUDS:mainfrom
LemonCANDY42:feat/backtest-report-library

Conversation

@LemonCANDY42

@LemonCANDY42 LemonCANDY42 commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Add a dedicated /reports Backtest Report Library page.

This follows the product shape discussed in #216:

  • dedicated Reports route, not a Home section
  • lists recent runs with backtest metrics/report output
  • row fields include run id, status, created time, truncated prompt, symbols, date range, return, Sharpe, and links to Run Detail + Compare
  • search covers run id, prompt, symbol, and status
  • v1 filters cover status and created-date range
  • sorting defaults to newest first, with options for oldest, return, and Sharpe
  • page and sidebar copy are localized in English and zh-CN

This remains draft because #216 was explicitly sequenced after #214 and #215.

Validation

  • npm test -- --run src/pages/__tests__/Reports.test.tsx
  • npm run build
  • git diff --check

Refs #216.

@LemonCANDY42 LemonCANDY42 changed the title Add Backtest Report Library feat(reports): add backtest report library Jun 14, 2026
Signed-off-by: Kenny Zhou <l.w.r.f.42@icloud.com>
@LemonCANDY42 LemonCANDY42 force-pushed the feat/backtest-report-library branch from 6a8eb20 to 4ce72f2 Compare June 21, 2026 07:17
@LemonCANDY42 LemonCANDY42 marked this pull request as ready for review June 21, 2026 07:17
@LemonCANDY42

Copy link
Copy Markdown
Contributor Author

Ready for review now.

Updated this branch on top of current upstream/main and resolved the frontend/src/lib/api.ts conflict by keeping the opt-in Run Detail getRun(id, params) contract from #225 while preserving listRuns(limit) for the Reports Library scan.

I also tightened Reports.test.tsx so it explicitly covers the #216 acceptance points: listing report runs, Full Report links, and search filtering.

Validation:

  • npm test -- --run src/pages/__tests__/Reports.test.tsx
  • npm run build
  • git diff --check
  • NODE_OPTIONS="--localstorage-file=/tmp/vibe-vitest-localstorage.json" npm test -- --run

@warren618 warren618 merged commit a2284b8 into HKUDS:main Jun 21, 2026
1 check passed
@warren618

Copy link
Copy Markdown
Collaborator

Merged 🎉 Thanks @LemonCANDY42 — this matches the product shape we agreed in #216 (dedicated /reports Run Library page, the row fields + search/filter scope, deferring multi-select compare to a follow-up). Open-box verified before merge: npm run build is clean and the full vitest suite passes 209/209, including the new Reports.test.tsx covering listing, the Full Report link, and search filtering. The listRuns(limit?) change lines up with the existing GET /runs?limit backend contract. Closes #216.

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