Enable typed linting (typescript-eslint recommendedTypeChecked)

Aberta
  • Algo errado?
#411 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 2 dias

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
4/5
Tempo estimado
3-5 dias
Facilidade para iniciantes
43/100
Tipo de issue
Funcionalidade
Clareza
Razoavelmente clara
Status de atividade
Ativa
Stack de tecnologia
eslint, typescript
Domínio
testing, tooling

Direção de pesquisa

Start by reviewing the ESLint configuration and the TypeScript project configuration, especially coverage for e2e/** and drizzle.config.ts; the issue does not name their paths. Then measure the findings for the proposed rules and split the rollout into the listed steps. Done means an agreed rule set is enabled with project service configured for .ts files and the required paths covered.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

Follow-up from #344. Running with recommendedTypeChecked produced about 1,000 findings, too many for one change.

Measured on main (2026-10-26), after removing noise from the trial config:

Rule Findings
no-unsafe-member-access / -assignment / -call / -return / -argument ~580
require-await 174
no-unnecessary-type-assertion 113
only-throw-error 98 (nearly all SvelteKit throw redirect() / error(), fixed by its allow option)
unbound-method 51
no-floating-promises 30
await-thenable 19
no-misused-promises 17
no-base-to-string 9

Prerequisites:

  • parserOptions.projectService for .ts files, not only Svelte files.
  • A tsconfig that covers e2e/** and drizzle.config.ts. Today the project service can't find them.
  • Lint time goes from about 36 s to about 55 s.

Proposed rollout, one PR per step:

  1. Turn on the rules that catch real bugs: no-floating-promises, no-misused-promises, await-thenable, and only-throw-error with a SvelteKit allow entry.
  2. no-unnecessary-type-assertion and require-await (mostly autofixable or mechanical).
  3. Decide whether to adopt the no-unsafe-* family at all, and how much of it, given how many hits come from untyped JSON and test code.
Linguagem predominante
TypeScript
Estrelas
1
Forks
0
Merge médio
3d 22h
PRs com merge (30d)
34

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de stackabletech/cockpit

Todas as issues de stackabletech/cockpit

Issues semelhantes

Mais issues de TypeScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.