langgenius/dify

[Chore/Refactor] not call the 'set' function of 'useState' directly in 'useEffect'

Open

#25.194 aberto em 5 de set. de 2025

Ver no GitHub
 (2 comments) (1 reaction) (0 assignees)TypeScript (22.210 forks)batch import
good first issuerefactor

Métricas do repositório

Stars
 (141.412 stars)
Métricas de merge de PR
 (Mesclagem média 1d 21h) (579 fundiu PRs em 30d)

Description

Self Checks

  • I have read the Contributing Guide and Language Policy.
  • This is only for refactoring, if you would like to ask a question, please head to Discussions.
  • I have searched for existing issues search for existing issues, including closed ones.
  • I confirm that I am using English to submit this report, otherwise it will be closed.
  • 【中文用户 & Non English User】请使用英语提交，否则会被关闭 ：）
  • Please do not modify this template :) and fill in all the required fields.

Description

not call the 'set' function of 'useState' directly in 'useEffect'

Motivation

keep logic simpler

Additional Context

https://react.dev/learn/you-might-not-need-an-effect

https://eslint-react.xyz/docs/rules/hooks-extra-no-direct-set-state-in-use-effect

Guia do colaborador

Direção de pesquisa
Identifique instâncias onde setState é chamado diretamente dentro de useEffect. Refatore movendo a atualização de estado para um manipulador de eventos ou removendo o efeito se o estado puder ser derivado de props ou outro estado. Consulte a documentação do React 'You Might Not Need an Effect'.
Pilha de tecnologia
typescriptreact
Domain
frontend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ReactGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80