feat(app,menubar): surface parity — punchcard in the app, PR spend in the menubar - #962
Merged
Conversation
…requests strip in the menubar The release's new capabilities were unevenly surfaced: the Spend punchcard existed only on the web dashboard, and the menubar decoded nothing of the PR-attribution block the payload already carries. - Desktop app: new Punchcard component (hour-of-day x weekday spend matrix, ported from the dash and restyled to the app's tokens) on the Spend page. Fed by a dedicated getTimeline bridge channel that fetches the payload WITH history.timeline; every other fetch keeps the lean --no-timeline path, and the serve child makes the extra fetch cheap. Hides gracefully when the payload has no timeline (older CLI). - Menubar: PullRequestsSection renders the top three PRs by attributed spend under the Workflow strip; MenubarPayload now decodes the pullRequests block (decodeIfPresent, so older payloads are unchanged). Hidden when absent or empty. Deliberately NOT ported: codex Tok/s (reads rollout files per session - too heavy for payload cadence; stays a TUI/report analytics view) and the punchcard in the menubar (a 7x24 matrix has no legible place in a compact popover). App suite 468 green (bridge channel pinned, mocks extended), swift build + 156 tests green, CLI suite green.
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
1 participant
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.
Closes the surface gaps for the release's new capabilities: the desktop app gains the Spend punchcard (dedicated timeline fetch so all other payloads stay lean; cheap under the resident serve child), and the menubar gains a top-pull-requests strip fed by the pullRequests payload block it previously didn't decode. Both hide cleanly on older payloads. Deliberate non-ports documented in the commit (codex Tok/s stays TUI/report - per-session rollout file reads are too heavy for payload cadence; no punchcard in a compact popover). App suite 468, swift 156, CLI suite all green.