-
Notifications
You must be signed in to change notification settings - Fork 302
feat(react-doctor): interactive code-health TUI as a tui subcommand
#173
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
aidenybai
wants to merge
23
commits into
main
Choose a base branch
from
cursor/react-doctor-tui-50e9
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 20 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
646f844
feat(react-doctor): add ScanReporter event interface
b89fcc5
feat(react-doctor-tui): interactive Ink-based code-health TUI
6a1a6bc
feat(react-doctor): wire watch / review subcommands to lazy-loaded TUI
641a953
refactor(react-doctor): merge TUI into the main package + redesign th…
d60277e
refactor(react-doctor): replace console.error with logger for improve…
NisargIO 2d98d6c
refactor(react-doctor-tui): simplify imports and enhance diagnostics …
NisargIO fb3de8a
refactor(tui): focus the dashboard on what users can act on
d96776f
fix(react-doctor): reverse sorted rule groups for improved diagnostic…
NisargIO 239933b
Merge branch 'cursor/react-doctor-tui-50e9' of https://github.com/mil…
cursoragent ebc6722
Revert "refactor(react-doctor-tui): simplify imports and enhance diag…
cursoragent 8a31e39
Revert "refactor(react-doctor): replace console.error with logger for…
cursoragent 200c2cb
Merge branch 'cursor/react-doctor-tui-50e9' of https://github.com/mil…
cursoragent 6caeccc
feat(tui): add an Ink-native project picker for monorepos
d2fc800
fix(tui): keep the review and dashboard readable at narrow terminal w…
ef19fa2
test(tui): add snapshots at 91 and 100 cols to lock in narrow-screen …
077c63f
fix(tui): refuse to start in agent / CI environments, name the trigger
f4816e9
feat(tui): add a 'By category' overview chart above the focused issue
abeee57
feat(tui): copy diagnostic to clipboard, esc closes help, nail down a…
c3746ee
feat(tui): surface react.doctor home URL, share URL, and diagnostics …
49819bb
fix(tui): drop the 'Last scan {elapsed}' prefix from the summary footer
56f108c
rename(tui): filter -> search across user-facing strings and internals
a61df72
fix(tui): render in alternate screen buffer to stop frame doubling
dc6cc91
feat(tui): always-on watch, copy-all-on-c, narrow-screen tightening, …
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Documented
--watchflag missing from CLI command definitionHigh Severity
The README and PR description document
react-doctor tui . --watchas a valid command, but thetuisubcommand only registers--reviewand--projectoptions. TheTuiSubcommandOptionsinterface also lacks awatchfield. Since Commander rejects unknown options by default, runningreact-doctor tui . --watchwill produce a CLI error like "unknown option '--watch'". Watch mode is always-on in the app (per the comment inapp.tsx), so either the--watchoption needs to be registered (even if it's a no-op), or the documentation needs updating.Additional Locations (1)
packages/react-doctor/README.md#L35-L36Reviewed by Cursor Bugbot for commit dc6cc91. Configure here.