Repository navigation
UI revamp Phase 2: new Home (brief, spend chart, Lumen found, calendar, bills, budget, activity) - #92
Open
coderconnoisseur wants to merge 7 commits into
Open
coderconnoisseur wants to merge 7 commits into
coderconnoisseur wants to merge 7 commits into
Conversation
…and light theme-color
…ard components; docs
…ithout LLM never blocks; empty chart state; backlog
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
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.
UI revamp Phase 2: the new Home. Stacked on #91 (Phase 1), which is stacked on #89 (Phase 0). Merge them in that order and retarget each PR to
refactoras its parent lands.Spec:
docs/superpowers/specs/2026-10-20-ui-revamp-design.md§4. Plan:docs/superpowers/plans/2026-10-20-ui-revamp-phase2-home-ui.md.What you get on /home
/receipts?tab=review.theme-color.(app)/layoutis a tiny server file; the client shell moved tocomponents/app-shell.tsx.Honest by design: cards without a backend yet show empty states, never made-up numbers. Phase 3 replaces the interim brief, Lumen found and Activity, and wires up Budget. Every item is listed in spec §12.
Backend (small)
GET /api/analytics/forecast?insights=0returns the forecast numbers without the LLM-written insight text, so Home costs no LLM call. The default is unchanged. Test:test_forecast_route_can_skip_llm_insights. The full backend suite passes: 903 passed, 15 skipped.Checks
npm run test:e2e: 71/71 (Chromium).lib/home-math.ts, covering periods, cumulative spend, the same-day comparison clamped to month length, the brief and UTC timestamps.tsc,lintandbuildare clean./homeis static at 16.1 kB of route JS, with no chart library on Home.Review fixes (whole-branch review, Opus)
?insights=0), caches them and loads them separately.color-mixinstead of relative colour syntax, for older browsers.Deferred minors are in spec §12.
Manual check