Skip to content

refactor: simplify scan orchestration cleanup#968

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/codebase-architecture-elegance-915b
Draft

refactor: simplify scan orchestration cleanup#968
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/codebase-architecture-elegance-915b

Conversation

@cursor

@cursor cursor Bot commented Jun 25, 2026

Copy link
Copy Markdown

Summary

  • Remove dead project-detection render path and skip hidden dead-code/render work in explain scans.
  • Fix LSP bundle version injection for the published experimental-lsp entry.
  • Clean small core/deslop no-op/stale docs paths and tighten focused test assertions.

Verification

  • pnpm test
  • pnpm typecheck
  • PATH="/home/ubuntu/.nvm/versions/node/v22.22.2/bin:$PATH" pnpm lint
  • PATH="/home/ubuntu/.nvm/versions/node/v22.22.2/bin:$PATH" pnpm format:check
  • pnpm smoke:json-report
  • Focused react-doctor/deslop/core tests during iteration
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
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 Jun 25, 2026

Copy link
Copy Markdown

Open in StackBlitz

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

commit: 533023a

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Cleanup pass completed.

Implemented:

  • Removed the dead project-detection render call chain and made explain scans skip hidden dead-code/render work.
  • Injected package VERSION into the shipped experimental-lsp bundle.
  • Removed a deslop no-op empty script glob path and fixed stale semantic/duplicate-block docs/tests.
  • Tightened run-oxlint smoke assertions and fixed order-dependent namespace-hooks setup.

Verification:

  • pnpm test
  • pnpm typecheck
  • PATH="/home/ubuntu/.nvm/versions/node/v22.22.2/bin:$PATH" pnpm lint
  • PATH="/home/ubuntu/.nvm/versions/node/v22.22.2/bin:$PATH" pnpm format:check
  • pnpm smoke:json-report

PR follow-up:

  • Posted /rde parity as requested.
  • Polled PR automation; all visible checks are green.
  • No Bugbot comments or RDE parity response were visible during the polling window.

View PR

Open in Web View Automation 

Sent by Cursor Automation: React Doctor Cleanup

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