chore: nightly.sh points at removed packages and skips modules

Aberta Para iniciantes
#511 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
77/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
bash, node.js, typescript
Domínio
build-system, testing

Direção de pesquisa

Comece pelas duas seções citadas em scripts/nightly.sh e leia como o relatório noturno de cobertura descobre os pacotes e trata o comando de build. Confira a estrutura atual de packages/ e modules/ e, em seguida, execute o script noturno para confirmar que o relatório inclui os módulos e deixa as falhas de build visíveis. O trabalho estará concluído quando as referências obsoletas a pacotes forem removidas e o relatório cobrir ambos os diretórios, tratando as falhas de build de forma visível.

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

Descrição

bug chore tests
Which package is this bug report for?

other

Issue description

The setup copies prebuilt WASM between packages/ecs-lib, ecs-client and ecs-server, which no longer exist. discover_packages scans only packages/*, so the nightly coverage report misses modules/*, the riskiest code (network, ecs). pnpm run build || true hides build failures.

Where:

Evidence:

if [[ -d "packages/ecs-lib/lib" ]]; then
...
for p in packages/*; do

Suggested fix: Delete the ecs-lib block, scan packages/* modules/*, and fail (or report) when the build fails.

Found during a code audit of main at be72076.

Code sample

No response

Versions
  • Engine main at be72076
  • pnpm 12.0.0, Node.js 26, Bun 1.3.14
Issue priority

Low (slightly annoying)

I have tested this issue on a development release

be72076

Linguagem predominante
TypeScript
Estrelas
7
Forks
2
Merge médio
1d 17h
PRs com merge (30d)
17

Preparar o ambiente

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 NanoForge-dev/Engine

Todas as issues de NanoForge-dev/Engine

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.