Skip to content

FPFNFIXAPEX avoid empty-default warnings for imported plain class projections - #2218

Draft
aidenybai wants to merge 1 commit into
fix/current-context-record-resetfrom
fix/imported-plain-class-empty-default
Draft

aidenybai wants to merge 1 commit into
fix/current-context-record-resetfrom
fix/imported-plain-class-empty-default

Conversation

@aidenybai

Copy link
Copy Markdown
Member

An empty object default passed to an imported plain Component triggered memo advice even when its render method only reads the object’s members. Add a bounded proof for direct default imports. Identity comparisons, whole-object forwarding, exposed instances, mutations, unsupported providers, and independent Hook dependencies retain their warnings.

Re-evaluate these candidates when consumer files or configuration change, since a file-only dependency fingerprint cannot represent the project-wide proof.

Validation:

  • 36,133 tests passed, including 75 new project tests and the physical cross-file fuzz regression; 1,118 skipped.
  • Source and native comparisons across 65 synthetic projects plus a local reproduction; all expected results matched.
  • Strict fuzz: seed 42, 500 iterations, 2,101 executions, 85 diagnostic fires, no findings.
  • Type checking, lint, formatting, JSON-report smoke, and changed-file React Doctor scans passed.

Draft stacked on #2217. Remote parity was not run.

@pkg-pr-new

pkg-pr-new Bot commented Oct 24, 2026

Copy link
Copy Markdown

Open in StackBlitz

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

commit: 1958428

This branch has not been deployed

No deployments
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