close
Skip to content

[eslint] Fix useEffectEvent checks in component syntax#35041

Merged
jbrown215 merged 1 commit intomainfrom
pr35041
Nov 4, 2025
Merged

[eslint] Fix useEffectEvent checks in component syntax#35041
jbrown215 merged 1 commit intomainfrom
pr35041

Conversation

@jbrown215
Copy link
Copy Markdown
Contributor

@jbrown215 jbrown215 commented Nov 4, 2025

We were not recording uEE calls in component/hook syntax. Easy fix.

Added tests matching function component syntax for component syntax + added one for hooks

@meta-cla meta-cla Bot added the CLA Signed label Nov 4, 2025
@react-sizebot
Copy link
Copy Markdown

react-sizebot commented Nov 4, 2025

Comparing: d000261...b8b2373

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.js = 6.68 kB 6.68 kB +0.11% 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 606.23 kB 606.23 kB = 107.38 kB 107.38 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB +0.05% 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 665.20 kB 665.20 kB = 117.26 kB 117.26 kB
facebook-www/ReactDOM-prod.classic.js = 690.02 kB 690.02 kB = 121.46 kB 121.46 kB
facebook-www/ReactDOM-prod.modern.js = 680.45 kB 680.45 kB = 119.84 kB 119.85 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against b8b2373

// Component syntax version
// Should error because it's being passed down to JSX, although it's been referenced once
// in an effect
function MyComponent({ theme }) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this isn't using component syntax

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.

F

Copy link
Copy Markdown
Member

@josephsavona josephsavona left a comment

Choose a reason for hiding this comment

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

Ahh that would explain it! Looks good, just see one comment before landing

@jbrown215 jbrown215 merged commit 8f8b336 into main Nov 4, 2025
240 of 267 checks passed
github-actions Bot pushed a commit that referenced this pull request Nov 4, 2025
We were not recording uEE calls in component/hook syntax. Easy fix.

Added tests matching function component syntax for component syntax +
added one for hooks

DiffTrain build for [8f8b336](8f8b336)
github-actions Bot pushed a commit that referenced this pull request Nov 4, 2025
We were not recording uEE calls in component/hook syntax. Easy fix.

Added tests matching function component syntax for component syntax +
added one for hooks

DiffTrain build for [8f8b336](8f8b336)
github-actions Bot pushed a commit to code/lib-react that referenced this pull request Nov 8, 2025
We were not recording uEE calls in component/hook syntax. Easy fix.

Added tests matching function component syntax for component syntax +
added one for hooks

DiffTrain build for [8f8b336](facebook@8f8b336)
github-actions Bot pushed a commit to code/lib-react that referenced this pull request Nov 8, 2025
We were not recording uEE calls in component/hook syntax. Easy fix.

Added tests matching function component syntax for component syntax +
added one for hooks

DiffTrain build for [8f8b336](facebook@8f8b336)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants