app: Workflow card on Overview, session titles in Sessions#784
Merged
Conversation
Two Phase-1 desktop additions over the renderer's existing card grammar. Overview workflow card: a panel below the daily chart with two stat tiles (correction rate with count, median time to first edit) plus a top-rework line and one coaching caption. The caption is derived locally with the CLI's buildCoachingNotes thresholds (corrections, then file churn, then time to first edit; first that fires). The card hides entirely when no real signal is present, never renders zeros, and shows a "N% priced" chip only when pricing coverage is a number below 1. Payload types gain optional workflow, topReworkedFiles, and pricingCoverage fields so older payloads render as before. Sessions view: the captured session title becomes the row headline with the id demoted to the mono secondary line; untitled rows keep the project as the headline (unchanged). Title is also searchable and added as an optional SessionRow field so older CLIs render unchanged.
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.
Phase 1 for the desktop app.
Workflow card below the daily-spend chart: correction-rate and time-to-first-edit tiles, a top-rework line (basename, sessions, edits), one coaching caption using the CLI's exact thresholds and copy, and an N% priced chip only when coverage is a number below 1 (never a fabricated 100%; sub-1 coverage clamps to 99% so it cannot round up to a lie). Hide-when-absent throughout; old payloads render today's Overview unchanged.
Sessions view: captured titles become the row headline with the id demoted to the mono secondary line; untitled rows keep the project headline, byte-identical to prior behavior; titles join the search filter.
Real-data check: 95% priced chip, 7 corrections at an honest 0%, first edit 6m, top rework sdk.py across 15 sessions with the churn coaching note; 608 of 1962 real session rows carry titles. App suite 420 green (7 new tests), both typechecks clean.
Part of the phased surfacing wave; PRs only, no release.