Skip to content

RD CLEANUP: simplify workspace and eval internals - #1807

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
aiden/code-architecture-cleanup-1c56
Draft

cursor[bot] wants to merge 1 commit into
mainfrom
aiden/code-architecture-cleanup-1c56

Conversation

@cursor

@cursor cursor Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Share pnpm workspace package parsing between project-info and project-analysis so flow-style YAML workspaces are discovered consistently
  • Ensure entry resolution only returns files, allowing directory entries to fall through to index resolution
  • Simplify eval argument and queue internals, remove verified dead code, and reuse existing AST/filesystem helpers

Verification

  • npx -p @antfu/ni nr test --filter=@react-doctor/core --filter=@react-doctor/evals --filter=oxlint-plugin-react-doctor
  • npx -p @antfu/ni nr lint
  • npx -p @antfu/ni nr typecheck
  • PATH="$HOME/.nvm/versions/node/v22.22.2/bin:$PATH" npx -p @antfu/ni nr format:check
  • PATH="$HOME/.nvm/versions/node/v22.22.2/bin:$PATH" npx -p @antfu/ni nr build && PATH="$HOME/.nvm/versions/node/v22.22.2/bin:$PATH" npx -p @antfu/ni nr smoke:json-report
Open in Web View Automation 

Co-authored-by: Aiden Bai <aidenybai@users.noreply.github.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/rde parity

View PR

Open in Web View Automation 

Sent by Cursor Automation: React Doctor Cleanup

@pkg-pr-new

pkg-pr-new Bot commented Sep 14, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/eslint-plugin-react-doctor@1807
npm i https://pkg.pr.new/oxlint-plugin-react-doctor@1807
npm i https://pkg.pr.new/react-doctor@1807

commit: 2c15f8f

@github-actions

Copy link
Copy Markdown
Contributor

Interactive terminal E2E

Terminal Control verified the built CLI at 2c15f8f in a real PTY:

  • selected a project interactively and observed Scanning... before the three-second Git delay completed
  • waited for the clean result and exercised the compact report
  • opened copy context and the GitHub Actions confirmation, then cancelled safely

Download the edited MP4 and PNG evidence

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant