pmndrs/react-three-fiber

RFC: @react-three/eslint-plugin rules

Open

#2701 aperta il 10 gen 2023

Vedi su GitHub
 (15 commenti) (10 reazioni) (0 assegnatari)TypeScript (1464 fork)batch import
help wantedrequest for commentsv10

Metriche repository

Star
 (25.748 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Aligning with our docs for general performance pitfalls and API usage around context (use R3F hooks within Canvas):

  • no-clone-in-frame-loop: Prefer creating temporary objects in global space and copy rather than clone in hot paths. This should be restricted to three.js classes to avoid collisions. https://github.com/pmndrs/react-three-fiber/pull/2710
  • no-fast-state: Don't set state within loops or continuous events (startTransition can be used if you must, this can be disabled for specific polling cases)
  • prefer-useloader: Prefer useLoader for suspense and caching rather than calling Loader.load or Loader.loadAsync in an effect. This will de-dup resources on both the CPU and GPU and avoid later expensive runtime compilation.

Guida contributor

Direzione di ricerca
Studia la documentazione esistente di eslint plugin react three fiber o la documentazione ufficiale delle regole personalizzate di ESLint. Guarda le tre regole proposte: no clone in frame loop (già completata) richiede di rilevare chiamate a .clone() all'interno dei loop di rendering; no fast state richiede di rilevare aggiornamenti di stato all'interno di loop o eventi continui; prefer useloader richiede di rilevare chiamate dirette a Loader.load all'interno di effetti. Scrivi una regola che utilizzi selettori AST per identificare questi pattern.
Tech stack
javascripttypescriptreact
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
JavaScriptESLintReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65