Rule proposal: `no-top-level-side-effects`

Métriques du dépôt

Stars

5 022 stars (5 022 stars)

Métriques de merge PR

Merge moyen 1j 8h (Merge moyen 1j 8h)6 PRs mergées en 30 j (6 PRs mergées en 30 j)

Description

document.title = 'gone'

export function init () {
  document.title = 'gone'
}

function init () {
  document.title = 'gone'
}

// Optional? Still a side effect, but at least it's wrapped
// and not lost between many function declarations.
// Maybe a single, unassigned function call per file could be allowed
init(); 

// Optional? Assignments could be considered side-effect free for the sake of usability
const details = new Map();
const response = fetch(); // even if they're not, really
const memoizedFetch = mem(fetch);

// in ./polyfill.js

// Optional? Some exceptions could be made by filename 
document.title = 'gone'

Guide contributeur

Direction de recherche

Examinez la PR existante #2957 pour comprendre l'approche d'implémentation. Passez en revue des règles similaires comme 'no unassigned import' et 'no new' pour gérer les effets secondaires. La règle doit détecter les expressions de niveau supérieur comme les affectations, les appels de fonction et les imports avec effets secondaires. Décidez des exceptions optionnelles (par exemple, déclarations const, noms de fichiers de polyfill). Étudiez les types de nœuds AST pour les instructions d'expression et la portée du module.

Stack technique

javascriptnodejs

Domaine

toolingdeveloper experience

Type d'issue

Fonctionnalité

DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.

3

Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.

1-2 jours

Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.

Bloquée

ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.

Plutôt claire

Prérequis

JavaScriptESLint rule developmentAST

Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.

60