fix(loader-website): an empty error message is shown when /manifest fails

Aberta Para iniciantes
  • Algo errado?
#140 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
82/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
bun, typescript
Domínio
frontend

Direção de pesquisa

Comece em apps/website/src/manifest.ts, no fetch para /manifest, e compare o tratamento de erros com apps/website/src/cache/cache.ts e o formato da resposta em apps/client/src/server.ts. Leia o erro e os detalhes de uma resposta com falha, usando o fallback sugerido para o status HTTP. O trabalho estará concluído quando uma requisição de manifesto com falha exibir uma mensagem útil no painel de erro.

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

Descrição

app:website bug good first issue
Which app is this bug report for?

website

Issue description

When /manifest returns a non-2xx status, the website throws new Error() with no message (manifest.ts#L7-L9). setError then shows an empty error panel, and the JSON body the client sends with its 500 ({ error, details }, client server.ts#L97-L103) is ignored.

Expected: the error panel shows why the manifest failed.

Suggested fix: read error/details from the body, as cache.ts#L94-L107 already does for game files, with HTTP Error <status> as a fallback.

Found during a read-only audit of main @ 044c766 (2026-11-11).

Code sample
const res = await fetch(`/manifest`);
if (!res.ok) {
  throw new Error(); // empty message shown to the user
}
Versions
  • @nanoforge-dev/loader-client / @nanoforge-dev/loader-server / @nanoforge-dev/loader-website 1.7.0 (main @ 044c766)
  • Bun 1.3.14
  • Linux x86_64
Issue priority

Low (slightly annoying)

I have tested this issue on a development release

044c766

Linguagem predominante
TypeScript
Estrelas
1
Forks
0
Merge médio
17h 25min
PRs com merge (30d)
11

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/Loader

Todas as issues de NanoForge-dev/Loader

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.