Revert "feat(action): React Doctor Review composite action (#307)"#443
Merged
Conversation
|
✅ No new issues Reviewed by reactreview for commit ec7461f. Configure here. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This reverts commit 30f05ef.
aidenybai
force-pushed
the
revert/pr-307-doctor-review-action
branch
from
26834b3 to
ec7461f
Compare
aidenybai
added a commit
that referenced
this pull request
May 23, 2026
README (594 → 394 lines, -34%): - collapse 3 redundant YAML examples in `PR blocking` to one explanation - shrink Husky/lint-staged walkthrough to one config block + one warning - compress `Custom rules` from 60 lines of numbered tutorial to a single example - merge `Agent and CI integration` bullet list (duplicated CLI ref + PR blocking + Node API) - collapse 5 prose paragraphs after the config table into a 5-bullet list - cut implementation-detail bullet list of `rn-no-raw-text` Platform.OS forms - drop NickvanDyke "we ported these rules" trivia paragraph - replace duplicated install block in `Resources & Contributing Back` with one line - inline single-row `Optional companion plugins` table CONTEXT.md + .specs/inspect-pipeline.md: - drop `actions/review` / `@react-doctor/review-action` section (reverted in #443) - replace `@react-doctor/project-info` / `@react-doctor/types` refs with `core/src/project-info` / `core/src/types` (folded into core in #440) - fix "13 leaf reasons today" miscount → 12 - update service count 9 → 11 Deletes: - TODOS.md (442-line working backlog — belongs in issues, not the repo) - .agents/skills/thermo-nuclear-code-quality-review/ (200-line skill) - packages/react-doctor/CHANGELOG.md.bak (68KB tracked backup file) Validated: `pnpm format`, `pnpm lint`, `pnpm typecheck`, `pnpm test` (1449 pass, 3 skip). 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
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.
Reverts #307.
Cleanly reverts the squash-merge commit
30f05ef5(no conflicts — the two commits landed onmainafter #307 don't touch any of the reverted files).Removed:
.github/workflows/doctor-action.ymlactions/inspect/action.ymlactions/review/**action.yml,packages/react-doctor/README.md,packages/react-doctor/tests/github-action.test.ts,pnpm-lock.yaml,pnpm-workspace.yamlto their pre-feat(action): React Doctor Review composite action #307 state.Made with Cursor
Note
Low Risk
Low risk revert that primarily deletes action/workflow code and dependencies; main impact is behavior change back to the simpler
npx react-doctor-based composite action for GitHub feedback.Overview
Reverts the earlier PR-review-style GitHub Action: removes the
.github/workflows/doctor-action.ymlworkflow and deletes the entireactions/reviewimplementation (Octokit/GQL, inline thread reconciliation, check runs) along with the legacyactions/inspectaction.Restores the root
action.ymlas the only composite action, which runsnpx react-doctor@latestwith inputs likediff,fail-on,annotations,offline, and optionally posts/updates a sticky PR comment plus ascoreoutput.Updates docs/tests to reference
uses: millionco/react-doctor@mainat the repo root (and the newaction.ymlpath), and trims workspace/lockfile entries by removing theactions/*package and its dependencies.Reviewed by Cursor Bugbot for commit ec7461f. Bugbot is set up for automated code reviews on this repo. Configure here.