Repository navigation
Research Markets: Top movers read the overview's live heat grid (D533, #1080) - #1305
Merged
Merged
Conversation
…#1080) The card waited on #1092, which closed unmerged; the Research overview reached main through #1151 (D536). The card now ranks the four largest rises and falls on last month across the live index cells, each linked to its sector page, with the period caption. A cell with no change recorded is left out, never ranked as 0; a failed read is Unreadable and an index not yet computed is Not recorded with the Worker's reason. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016Q3DcWsMTSxPxJa7tMeDAs
|
Preview: https://studioos-pr-1305.guillaumelauzier.workers.dev (built from 26a7620) The pull request's SPA build on a Worker with no bindings: pages and deep links work, |
guillaumelauzier
marked this pull request as ready for review
October 6, 2026 21:58
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Objective
On
/research/markets, the "Top movers this month" card read "Not recorded", waiting on the Research overview in #1092. #1092 closed unmerged, but the overview reachedmainthrough #1151 (D536). This PR wires the card to it, to the Markets canvas. It was the one part of S1's 2026-10-05 list that #1150 left open.Part of #1080
Implementation
frontend/src/pages/research/MarketDirectoryZone.jsx:GET /api/research/overviewthroughresearchOverviewApi.get(1).top=1because it needs none of the overview's top-N lists.useRead, so a failure stays on this card.topMovers(heat, n = 4)ranks the heat grid's cells:delta_pct, across the 12 sectors × 6 dimensions;previousPeriod('YYYY-MM')gives the month before, for the caption:market_intel_indexes · global · 2026-10 against 2026-09./research/markets/sector/<slug>, which uses the same slug asresearch_sectors.ts. A ▲ or ▼ shows the change's sign.unreadable.Docs: a "Top movers" section in
decisions/D533.md, and the ROUTE_MAP Markets row. Both used to say the movers waited on #1092.Files changed
frontend/src/pages/research/MarketDirectoryZone.jsxfrontend/test/research_top_movers_d533.test.mjs(new, 6 tests)frontend/test/research_markets_pages_d533.test.mjs: drops the old pin that the movers read Not recorded and name D536: GET /api/research/overview, its tests, and the founder desk's counts (#1082, route) #1092.documentation/architecture/decisions/D533.md,documentation/architecture/ROUTE_MAP.mdTesting
npm run build, thennpm run test:drift > drift.log 2>&1; echo EXIT=$?, gives EXIT=0.research_top_movers_d533.test.mjsfails on main. It covers:research_markets_pages_d533(18 tests) andresearch_markets_d593(14) pass.Risks
wrangler.tomlchange.Dependencies
#1151 (D536, the overview on main). #1150 (the Markets page).
Agent
S08 · Claude Code
Review requested
S1.
🤖 Generated with Claude Code
https://claude.ai/code/session_016Q3DcWsMTSxPxJa7tMeDAs
Generated by Claude Code