[#202] Discovery tab queries with Phase 5 TODOs#61
Conversation
- Extract queryTab() with explicit switch/case for each tab - New: active storylines sorted by creation time desc - Completed: sunset storylines sorted by plot count desc - Trending: falls back to New ordering with TODO referencing P5-6a - Rising: falls back to New ordering with TODO referencing P5-6b - Each TODO includes specific ROADMAP.md cross-reference for Phase 5 Fixes #202 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: APPROVE
Summary
This refactor makes the discovery tab queries explicit and documents the Phase 5 fallback behavior clearly without changing the intended P4-2 semantics. The repository check is passing.
Findings
- None.
Decision
Approving because New, Completed, Trending, and Rising now have distinct query branches with the requested fallback/TODO guidance, and I did not find regressions in the discovery page behavior.
project7-interns
left a comment
There was a problem hiding this comment.
T2b Review: APPROVED
Clean refactor — inline query logic extracted into queryTab switch function with explicit per-tab cases. TODO comments reference P5-6a/P5-6b with clear descriptions of the future ranking signals. Trending/Rising correctly fall back to recency. Exhaustive switch (all 4 tabs covered). CI green.
Summary
queryTab()Test plan
tsc --noEmitpasseseslintpassesFixes #202
🤖 Generated with Claude Code