Microsoft/TypeScript

Computing buildInfoTime even when !isIncremental

Open

#60.500 aberto em 14 de nov. de 2024

Ver no GitHub
 (3 comments) (1 reaction) (0 assignees)TypeScript (6.726 forks)batch import
Domain: tsc: --incrementalHelp WantedPossible Improvement

Métricas do repositório

Stars
 (48.455 stars)
Métricas de merge de PR
 (Mesclagem média 6d 17h) (9 fundiu PRs em 30d)

Description

@sheetalkamat @johnnyreilly I'm out of my depth, but I think there's maybe a bug introduced here.

I'm getting TypeError: Cannot read properties of undefined (reading 'includes') in fork-ts-checker-webpack-plugin when my project is NOT incremental: true.

Prior to this revision buildInfoTime was not computed if buildInfoPath was undefined but after this change, even though buildInfoPath is undefined, it is still attempting to compute ts_getModifiedTime(host, buildInfoPath)

Eventually, this calls to isArtifact(undefined) in fork-ts-checker-webpack-plugin\lib\typescript\worker\lib\system.js

[function isArtifact(path) {

return ((artifacts.dirs.some((dir) => path.includes(dir)) ||
    artifacts.files.some((file) => path === file)) &&
    artifacts.extensions.some((extension) => path.endsWith(extension)));

}](https://github.com/TypeStrong/fork-ts-checker-webpack-plugin/blob/0fab463b21c6edc4d94834568a3f440241d57887/src/typescript/worker/lib/system.ts#L284C1-L290C2)

Perhaps this needs to be handled in fork-ts-checker-webpack-plugin? But it seems to me that perhaps buildInfoTime could be skipped when buildInfoPath is unavailable.

Originally posted by @JasonKleban in dca9182

Guia do colaborador

Direção de pesquisa
Investigue o commit dca9182 que introduziu a regressão. O problema é que `buildInfoTime` é calculado via `ts getModifiedTime(host, buildInfoPath)` mesmo quando `buildInfoPath` é indefinido. A correção deve proteger essa chamada com uma verificação para `isIncremental` ou que `buildInfoPath` esteja definido. Examine a função `computeBuildInfoTime` no código fonte do compilador TypeScript (provavelmente em src/compiler/emitter.ts ou similar). Certifique se de que a condição esteja alinhada com o comportamento anterior, onde era ignorada se `buildInfoPath` fosse indefinido.
Pilha de tecnologia
typescript
Domain
build systemtooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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
TypeScriptGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75