Skip to content

[5/11] test(rules): add private evaluator parity harness - #1487

Closed
aidenybai wants to merge 5 commits into
codex/internal-architecture-04-package-aware-rulesfrom
codex/internal-architecture-05-private-evaluator
Closed

[5/11] test(rules): add private evaluator parity harness#1487
aidenybai wants to merge 5 commits into
codex/internal-architecture-04-package-aware-rulesfrom
codex/internal-architecture-05-private-evaluator

Conversation

@aidenybai

@aidenybai aidenybai commented Jul 29, 2026

Copy link
Copy Markdown
Member

Part 5 of 11. Review this PR against codex/internal-architecture-04-package-aware-rules.

What changed

  • Adds the private rule evaluator and project/source testkit.
  • Adds differential fixtures comparing evaluator results with oxlint.
  • Extends fuzz variants to preserve evaluator verdicts.

Why

The package-aware rule work needs a deterministic in-process harness that detects semantic drift from the production oxlint path.

Impact

Private test/evaluation infrastructure; no published API or CLI changes.

Validation

  • Full workspace typecheck at this exact commit
  • Plugin and fuzz typechecks
  • 76 evaluator, testkit, and oxlint-parity tests

Stack

Review only each PR's diff against its base branch. Merge in order.

  1. #1463 — Git service decomposition
  2. #1478 — Core execution pipeline
  3. #1479 — Workspace package graph
  4. #1480 — Package-aware rules
  5. #1487 — Private evaluator parity
  6. #1482 — API/LSP adapter facades
  7. #1483 — CLI workflow decomposition
  8. #1481 — CLI facade import migration
  9. #1484 — Architecture and compatibility gates
  10. #1485 — Shared deslop cycle traversal
  11. #1486 — Architecture docs and skill sync

@pkg-pr-new

pkg-pr-new Bot commented Jul 29, 2026

Copy link
Copy Markdown

Open in StackBlitz

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

commit: 811beba

@aidenybai
aidenybai force-pushed the codex/internal-architecture-04-package-aware-rules branch from 7754279 to 2bbfb30 Compare July 30, 2026 03:14
@aidenybai

Copy link
Copy Markdown
Member Author

Closing this private evaluator harness slice as part of the regression-risk reduction. The original architecture stack has been replaced by four independent correctness/performance PRs that each target main. The full rewrite remains preserved on codex/internal-architecture-rewrite-full if we want to revisit it separately.

@aidenybai aidenybai closed this Jul 30, 2026
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