ghostfolio/ghostfolio

Enable strictTemplates for the client app

Aberta

#7.393 aberto em 22 de jul. de 2026

Ver no GitHub
 (3 comentários) (0 reação) (0 responsável)TypeScript (1.234 forks)auto 404
Angularenhancementhelp wanted

Métricas do repositório

Stars
 (8.983 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

The apps/client project has Angular's strict template checking currently disabled in apps/client/tsconfig.json:

"angularCompilerOptions": {
  "strictInjectionParameters": true,
  // TODO: Enable stricter rules for this project
  "strictInputAccessModifiers": false,
  "strictTemplates": false
}

libs/ui already runs with strictTemplates: true, so the client is the last project holding this back. Enabling it catches template type errors (nullability, mismatched input types, misspelled bindings) at build time.

Scope

A trial run flipping both flags surfaces ~420 template type errors across ~54 components, whilte the majority is nullability-driven (strictNullChecks is already on but not yet enforced in templates), plus a handful of genuine bugs (bindings to properties that don't exist). Note that libs/ui template errors also surface here, since its strict setting is only enforced when the client compiles it.

Suggested approach (4 phases, to keep PRs reviewable)

  1. Fix the genuine bugs surfaced (missing / misspelled bindings)
  2. Clean up libs/ui template errors
  3. Fix apps/client nullability errors, batched by page/component
  4. Flip strictTemplates + strictInputAccessModifiers to true and remove the TODO

Guia do colaborador

Direção de pesquisa
Ative strictTemplates no tsconfig.json e compile para listar todos os erros. Classifique os erros em bugs reais, problemas de libs/ui e problemas de nulabilidade. Corrija na ordem: bugs reais, erros de libs/ui, erros de nulabilidade por página, depois ative as flags.
Pilha de tecnologia
typescriptangular
Domain
frontend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
4
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
3-5 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
AngularTypeScriptGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
45