[compiler] Fix setState-in-effect for React.useEffect namespace calls (#35377) (#35419)

## Summary
Fix react-hooks/set-state-in-effect false negatives when Hooks are
called via a namespace import (e.g. `import * as React from 'react'` and
`React.useEffect(...))`. The validation now checks the MethodCall
property (the actual hook function) instead of the receiver object.

Issue: Bug: #35377

## How did you test this change?
Added a regression fixture;
Ran tests and verified it reports `EffectSetState` and matches the
expected output.

<img width="461" height="116" alt="Screenshot 2025-12-27 at 14 13 38"
src="https://github.com/user-attachments/assets/fff5aab9-0f2c-40e9-a6a5-b864c3fa6fbd"
/>
This commit is contained in:
Anton Chesnokov
2026-02-04 20:07:17 +04:00
committed by GitHub
parent 90c6d1b218
commit 22a20e1f2f
3 changed files with 53 additions and 1 deletions

View File

@@ -97,7 +97,7 @@ export function validateNoSetStateInEffects(
case 'CallExpression': { case 'CallExpression': {
const callee = const callee =
instr.value.kind === 'MethodCall' instr.value.kind === 'MethodCall'
? instr.value.receiver ? instr.value.property
: instr.value.callee; : instr.value.callee;
if (isUseEffectEventType(callee.identifier)) { if (isUseEffectEventType(callee.identifier)) {

View File

@@ -0,0 +1,42 @@
## Input
```javascript
// @loggerTestOnly @validateNoSetStateInEffects @outputMode:"lint"
import * as React from 'react';
function Component() {
const [state, setState] = React.useState(0);
React.useEffect(() => {
setState(s => s + 1);
});
return state;
}
```
## Code
```javascript
// @loggerTestOnly @validateNoSetStateInEffects @outputMode:"lint"
import * as React from "react";
function Component() {
const [state, setState] = React.useState(0);
React.useEffect(() => {
setState((s) => s + 1);
});
return state;
}
```
## Logs
```
{"kind":"CompileError","detail":{"options":{"category":"EffectSetState","reason":"Calling setState synchronously within an effect can trigger cascading renders","description":"Effects are intended to synchronize state between React and external systems such as manually updating the DOM, state management libraries, or other platform APIs. In general, the body of an effect should do one or both of the following:\n* Update external systems with the latest state from React.\n* Subscribe for updates from some external system, calling setState in a callback function when external state changes.\n\nCalling setState synchronously within an effect body causes cascading renders that can hurt performance, and is not recommended. (https://react.dev/learn/you-might-not-need-an-effect)","suggestions":null,"details":[{"kind":"error","loc":{"start":{"line":7,"column":4,"index":200},"end":{"line":7,"column":12,"index":208},"filename":"invalid-setState-in-useEffect-namespace.ts","identifierName":"setState"},"message":"Avoid calling setState() directly within an effect"}]}},"fnLoc":null}
{"kind":"CompileSuccess","fnLoc":{"start":{"line":4,"column":0,"index":100},"end":{"line":10,"column":1,"index":245},"filename":"invalid-setState-in-useEffect-namespace.ts"},"fnName":"Component","memoSlots":1,"memoBlocks":1,"memoValues":1,"prunedMemoBlocks":1,"prunedMemoValues":1}
```
### Eval output
(kind: exception) Fixture not implemented

View File

@@ -0,0 +1,10 @@
// @loggerTestOnly @validateNoSetStateInEffects @outputMode:"lint"
import * as React from 'react';
function Component() {
const [state, setState] = React.useState(0);
React.useEffect(() => {
setState(s => s + 1);
});
return state;
}