Bruno Miiller 46f86c0091 Tighten CI: coverage ratchet at 8% and ignore build artifacts

- package.json: test:coverage adds --check-coverage --lines=8 (current
  baseline is 8.32%; floor only goes up)
- biome.json: ignore *.main.js webpack chunks, coverage/, node_modules/
- docs/CONTRIBUTING.md: document ratchet policy and long-term 70% target

2026-05-19 23:07:36 -03:00

2.1 KiB

Raw Blame History

Contribuindo

Workflow

Branch a partir de master: git checkout -b <tipo>/<descrição>
- feat/ — feature nova
- fix/ — bugfix
- refactor/ — refactor sem mudança de comportamento
- docs/ — só documentação
- chore/ — config, build, CI
Commits pequenos, foco único
Antes de abrir PR: npm run format && npm test && npm run build
Abrir PR descrevendo o porquê (não o quê — o diff mostra o quê)

Mensagens de commit

Inglês, imperativo (alinhado com o upstream)
Primeira linha curta (~72 caracteres), inicia com letra maiúscula
Corpo opcional em lista com - detalhando o porquê

Exemplo:

Add WebDAV retry on 429/503

- new retryWithBackoff helper in src/fsWebdav.ts
- apply to walk/stat/writeFile methods
- based on upstream PR #1034

Cherry-pick de PRs do upstream

Upstream: https://github.com/remotely-save/remotely-save

git remote add upstream https://github.com/remotely-save/remotely-save.git
git fetch upstream pull/<N>/head:pr-<N>
git cherry-pick <commits-de-pr-N>

Citar o PR upstream na mensagem de commit (baseado no PR upstream #<N>).

Estilo de código

Biome formata e linta (npm run format)
TypeScript strict — evitar any
Funções idealmente 4–20 linhas; refatorar se passar
Nomes específicos, não data/handler/Manager
Sem comentários óbvios — explicar só o porquê quando não-trivial
Early returns (no máximo 2 níveis de aninhamento)

Testes

Toda feature ou bugfix inclui teste
Localização: tests/ para src/, pro/tests/ para pro/
Comando: npm run test:coverage

Política de cobertura (ratchet)

Cobertura mínima de linhas é um piso que só sobe. Atualmente em 8% (via c8 --check-coverage --lines=8 no script test:coverage).

Quando você adicionar testes que elevam a cobertura geral, suba o piso no script test:coverage para o novo valor (truncado). Nunca abaixar.

Meta de longo prazo: 70% (padrão code-standards). Áreas zero-cobertura prioritárias: src/main.ts, src/settings.ts, backends fs*.ts.

2.1 KiB Raw Blame History Unescape Escape