feat(tui): complete terminal experience roadmap - #12
Open
echoVic wants to merge 158 commits into
Open
Conversation
Define cached fenced-code and diff highlighting, strict safety limits, and verified background full-file refinement.
Break cached code and diff highlighting, background refinement, and completion auditing into TDD checkpoints.
Initialize shared syntax data, enforce bounded highlighting, and prefer complete filenames during path grammar lookup.
Buffer fenced Markdown source through closing events so syntax state and blank lines are preserved. Keep fallback boundaries and proposed-plan spans intact.
Parse unified diffs with independent old/new hunk state while preserving Orca prefixes, metadata order, guardrails, and truncation.
Cache highlighted wrapped lines by the explicit syntax theme revision and pass message indices to selective rebuilds.
Walk post-edit source once and retain only matching visible new-side lines.
Coalesce live edit jobs and compute bounded full-file styles outside the draw thread.
Submit only live safe edit targets and reject stale worker results before cache invalidation.
Poll verified worker results and rebuild only the affected cached transcript message.
Pin selection foreground preservation and clean up syntax highlight test lint diagnostics.
Count exact synthetic parser bytes, reject malformed hunk bodies with a full legacy fallback, and group transcript render cache options behind a context object.
Move hunk coordinate, ordering, overlap, overflow, and /dev/null validation into the unified diff parser so first paint, AppState, and the highlight worker share one fail-closed structural contract.
Define display-width-aware native cursor placement for the custom composer and masked setup input while preserving the software cursor.
Break display-aware textarea layout, native composer cursor integration, setup masking, visibility tests, and delivery into TDD checkpoints.
Derive software and native cursor coordinates from one wrapped textarea layout, including masked and wide-character input.
Position the native terminal cursor on the custom composer layout while modal frames keep it hidden.
Reuse the display-aware textarea surface for masked API-key input and verify cursor visibility transitions.
Snap internal grapheme cursors to their rendered lead cell and hide the native cursor when the styled grapheme cannot fit. Preserve logical click positions, exact-width synthetic rows, and Vim cursor colors with completed-frame coverage.
Define semantic Markdown palette fields, renderer mappings, and cache invalidation for every named theme.
Break semantic palette, renderer migration, and cache invalidation into reviewed TDD tasks.
Define document-specific heading and inline-code accents for every named palette.
Replace fixed ANSI document colors with named palette semantics without changing Markdown layout.
Rebuild wrapped transcript rows exactly once when document palette semantics change.
Define auto background selection and independent truecolor, ANSI-256, ANSI-16, and monochrome rendering profiles.
Decompose automatic background selection, palette quantization, syntax adaptation, monochrome selection, and startup probing into reviewed TDD tasks.
Make auto the default while preserving every explicit named theme.
Model independent background and color-depth capabilities and quantize every named palette once.
Generate fenced-code and diff refinement styles in the terminal's supported color space.
Use reversible style overlays when background colors are unavailable.
Add exact doctor command parsing, bounded safe reports, session-only FPS controls, and keybinding reload projections. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Track authentication independently from UI status and parse built-in slash commands before any cwd discovery. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Propagate cursor candidates from textarea surfaces and apply the single hardware cursor position after all overlays render. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Render a display-width-aware session HUD after overlays while preserving final hardware cursor placement and default-off layout parity. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Render the composer unconditionally while keeping the transcript-search cursor as the selected hardware cursor candidate. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Reserve built-in slash names even when their arguments are invalid so dynamic alias discovery never reads cwd for malformed doctor commands. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Install startup facts from one terminal identity, synchronize Vim projection centrally, record successful draw and event metrics, and reset rolling samples on suspend. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Cover saturated frame coordinates and prove the disabled HUD returns before metric snapshots or widget rendering. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Describe the safe doctor report, excluded data, no-reprobe behavior, and default-off session-only FPS HUD in English and Chinese. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Track reload rejection explicitly so repeated invalid JSON stays rejected until a valid or missing observation is accepted. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Keep snapshot inspection helpers test-only so production all-target builds do not add dead-code warnings. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Document the privacy-preserving location labels emitted by the implemented diagnostics report. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Keep the terminal report field complete by rendering an unknown version without duplicating a fully unknown identity. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Define a typed seven-step wizard, safe user preference persistence, production provider precedence, and capability-preserving theme preview. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Name every provider entry point and clarify transactional Review behavior when shared configuration is unavailable. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Clarify transactional credential staging, typed persistence errors, exact provider entry points, and safe theme preview projections. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Split provider precedence, atomic persistence, typed wizard state, transactional actions, capability-safe preview, bounded UI, audit, push, and PR creation into strict TDD tasks. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Resolve provider precedence from file defaults through environment and CLI overrides while keeping worker handoffs explicit and fully resolved. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Patch only provider, model, and theme while preserving valid user TOML and rejecting unsafe existing paths. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Preserve valid auth entries and reject unsafe or malformed files without exposing credential text. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Add closed provider, credential, model, theme, review, and completion state before migrating setup consumers. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Stage credentials until Review, apply current-session choices atomically, and project independent persistence outcomes. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Resolve setup previews from captured terminal capabilities and synchronize syntax and diagnostic projections. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Add bounded provider, credential, model, theme, review, and completion surfaces with API-key-only cursor ownership. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Describe provider, model, theme, credential, review, persistence, and current-session fallback behavior. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Document the reviewed owner-only persistence policy and the zero-write Review escape boundary. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Consume staged credentials only after validation and surface safe API-key validation errors. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Keep pasted API-key input consistent with typed input validation feedback. Co-authored-by: TRAE CLI <noreply@bytedance.com>
Map every P0 and P2 requirement to production artifacts, direct tests, review evidence, and remote-ready commits. Co-authored-by: TRAE CLI <noreply@bytedance.com>
|
Important Review skippedToo many files! This PR contains 114 files, which is 14 over the limit of 100. To get a review, narrow the scope: Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (114)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Summary
Verification
cargo test -p orca-core— 228 passedcargo test -p orca-tui— 1095 passedcargo test --workspace --all-targets— remaining suite passed after three intermittent failures in unchanged source files were baseline-checked and each exact test passed on rerun; CI or another recorded merge gate should rerun the unfiltered suitecargo check --workspacecargo fmt --all -- --checkgit diff --checkandgit diff --cached --checkSafety
docs/superpowers/audits/2026-07-29-tui-roadmap-completion-audit.md