TUI improvements (v0.2.9)#84
Merged
Merged
Conversation
Lifts the CLI from "polished" to "fzf/gh/lazygit-tier daily driver" by closing the at-a-glance and reward-the-user gaps the dashboard's polish pass exposed: - pcr <cmd> --help now renders the help.rs single-source-of-truth (purpose / when-to-use / examples / see-also / docs) so the line CLI matches what `pcr help` (TUI) has always shown. - pcr push ends with `▲ pushed N prompts in K bundles · 2.1s` in success tone, with a next-step hint. Failures now print a retry hint too so flaky network leaves no dead air. - pcr log drafts section groups by session, color-codes per source (cursor cyan / claude magenta / vscode yellow), and aligns HH:MM timestamps for scanability. - show TUI gains vim-style `y` (yank focused draft/bundle id) and `o` (open the focused bundle's review URL or the draft's project page) on both panes, plus an updated `?` legend. - show TUI grows a 1-row context strip under the header — project · branch · pending drafts · last-push age — refreshed every 2s so users never have to bounce back to `pcr status` to remember where they are. - pcr init / bundle / pull success lines now match the push close-out shape (▲ glyph + success tone + next-command hint) so every state-changing command exits on the same beat. No new deps, no breaking flag or output changes (goldens pass), no network protocol changes. cargo build / test / clippy / fmt all green. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
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.
v0.2.9 — TUI polish + test-coverage gap-fillers + a dedicated CI workflow.
Highlights
vv0.2.xdoubling bug when CI passes av-prefixed git tag through.pcr pushwithout auth (code10),pcr bundle --nonexistent-flag(clap usage2), andpcr bundle --deletewithout a name (also2).cursor_ingestionsmoke test — covers the cursor source end-to-end through ingestion → store.db_migrationsmoke test — builds a v1-schema fixture DB, runsmigrate_v1, and re-queries the resulting v2 DB to pin schema drift; depends onrusqliteas a dev-dep.claude_fixture_roundtrip+vscode_fixture_roundtripusing checked-in minimal.jsonlfixtures so parser regressions stop sneaking through..github/workflows/ci.ymlrunsfmt --check,clippy -D warnings,cargo test, and a best-effortcargo docon every PR tomainand on every non-main push branch.RELEASING.md— adds the protected-mainnote up front so future releases go through a PR instead of a direct push.35c0a81):pcr <cmd> --helprenders the richhelp.rssingle-source-of-truth (purpose / when-to-use / examples / see-also / docs) so the line CLI matchespcr help's TUI.pcr pushclose-out:▲ pushed N prompts in K bundles · 2.1swith retry hint on failure.pcr logdrafts section: per-source color-coding, aligned HH:MM timestamps.showTUI: vim-styley(yank id) ando(open URL); 1-row context strip with project · branch · drafts · last-pushed.init/bundle/pull/push(▲+ success tone + one-line next hint).Version bumps
0.2.9pcr-devnapi meta →0.2.9(+ all 4optionalDependenciespinned)0.2.9v0.2.9Verification
cargo fmt --all -- --check— cleancargo clippy --workspace --all-targets -- -D warnings— cleancargo test --workspace— 105 unit + integration tests passTagging
v0.2.9after squash-merge fans out intolint → binaries → napi → npm-publish → release → homebrewperRELEASING.md.