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

Métricas do repositório

Stars

5.022 stars (5.022 stars)

Métricas de merge de PR

Mesclagem média 1d 8h (Mesclagem média 1d 8h)6 fundiu PRs em 30d (6 fundiu PRs em 30d)

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'

Guia do colaborador

Direção de pesquisa

Examine a PR existente #2957 para entender a abordagem de implementação. Revise regras semelhantes como 'no unassigned import' e 'no new' para lidar com efeitos colaterais. A regra precisa detectar expressões de nível superior como atribuições, chamadas de função e importações com efeitos colaterais. Decida sobre exceções opcionais (por exemplo, declarações const, nomes de arquivos de polyfill). Investigue os tipos de nós AST para instruções de expressão e escopo do módulo.

Pilha de tecnologia

javascriptnodejs

Domain

toolingdeveloper experience

Tipo Issue

Funcionalidade

DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.

3

Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.

1-2 dias

Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.

Bloqueado

ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.

Principalmente claro

Prerequisites

JavaScriptESLint rule developmentAST

Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.

60