Skip to content

chore(arnês): serve injeta __SUPPORT e grafite preserva os outros mapas (BUG-60) - #369

Closed
rubenmarcus wants to merge 1 commit into
mainfrom
v2/arnes-grafite-serve
Closed

chore(arnês): serve injeta __SUPPORT e grafite preserva os outros mapas (BUG-60)#369
rubenmarcus wants to merge 1 commit into
mainfrom
v2/arnes-grafite-serve

Conversation

@rubenmarcus

Copy link
Copy Markdown
Member

O que

Dois consertos de infra do arnês + regen de artefatos stale, desimpedindo a rodada de fauna/grade que vem empilhada em cima:

  1. tools/eval/serve.mjs injeta __SUPPORT — o arnês serve o index.astro cru e o <script is:inline define:vars> vazava SUPPORT_URL_BR como identificador livre: pageerror em toda captura headless (23 erros numa corrida de screen-query). Injeta os defaults do site.ts.
  2. BUG-60 portado do /game (9cc84fe)gen-graffiti-layout.mjs parseava o layout anterior com lastIndexOf('}'), que caía no rodapé do GRAFITE_FP; o parse lançava e o catch zerava os outros mapas. Sintoma medido aqui: rodar npm run grafite ferro_velho deixava o check em 1 mapas · 894 peças com M1 vermelho para os outros 4. Fim do JSON por casamento de chaves com controle de string.
  3. Regen dos artefatos stale (foot-offsets, mat_check/scenes/shade) — o mat_check.json media só os 5 mapas antigos; o regen mede os 12 e expõe velho_oeste (mapa de 17/08) fora dos tetos de croma: Δab 2,28 (teto 1,5) e razão mão/chão 1,61× (faixa 0,7–1,45). Medido também numa árvore limpa (sem fauna) — não é regressão da rodada que vem. Registrado como dívida MAT1 no KNOWN-RED.json com a procedência.

Régua

  • eval:grafitelayout, feet:check, docs:check, arch:check: verdes
  • eval:invariants: 39/55, nenhuma falha nova (MAT1 = DÍVIDA registrada)
  • check:fast: vermelho único changelog:check (contagem de PRs da alpha.151) — já vermelho na main, pré-existente

@vercel

vercel Bot commented Aug 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
csbrasil Ready Ready Preview Aug 19, 2026 2:56am

Request Review

@csbrasil-BOT csbrasil-BOT added needs-staging Mudança precisa validação integrada em staging target:main PR apontando para a branch main labels Aug 18, 2026
@csbrasil-BOT

csbrasil-BOT commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

csbrasil-bot classification

Note

@rubenmarcus abriu 7 arquivo(s) · +18906 −462, base main.

O que mudou

área arquivos + amostra
🧪 arnês 6 +18895 −462 KNOWN-RED.json, mat_check.json, mat_scenes.json
📚 docs 1 +11 −0 KNOWN-BUGS.md

Labels

label por quê
target:main vai direto pra main, sem fila de integração

@estraga-codigo estraga-codigo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Palhaço Mal

Palhaço Mal · revisor da casa · 🤡

mapa da mudança

  • arnês (6): KNOWN-RED.json, mat_check.json, mat_scenes.json, mat_shade.json
  • docs (1): KNOWN-BUGS.md
  • assets (1): foot-offsets.json

Note

o que faz: Documentação de BUG-60 (grafite), atualização de KNOWN-RED com MAT1 e timestamp regredido em foot-offsets.json

2 leves

Tip

Risco BAIXO · nota 8.7/10 ▓▓▓▓▓▓▓▓▓░

Revisão de bot — fio não resolvido TRAVA o merge (a main exige conversa resolvida). Resolve ou refuta.

🟡 leves (2)

  • timestamp regredido dois diaspublic/models/anims/foot-offsets.json:2 (ação: leis-da-casa)
    Campo 'gerado' foi de 2026-08-17 para 2026-08-15 — timestamp retroativo sem explicação de roll-back no diff.
    💡 conserto: Verificar se a alteração é intencional (regeneração de dados mais antigos) ou erro de versionamento
  • entrada de mapa truncada no difftools/eval/mat_shade.json:14588 (ação: leis-da-casa)
    O diff mostra uma entrada incompleta 'vmLegado@atacadao_treta' seguida por 'chao@parque_treta', sem fechamento da chave. Isso sugere que o diff foi capturado no meio de uma edição ou há um erro estrutural no arquivo.

    mutação sugerida: A inclusão da entrada incompleta 'vmLegado@atacadao_treta' quebraria o parsing JSON válido.

impacto cruzado

  • arnês em tools/eval/harness.mjs que lê este JSON para carregar cenas de material
  • sistema de iluminação do jogo que aplica mat_scenes.json para renderizar viewmodels e ambiente

(grafo local de public/js, uma rodada atrás do head do PR.)


Resolve ou refuta nos fios — refutado com razão, eu caio fora.

maquiagem de verde por cima de vermelho eu desenmascaro no primeiro golpe de vista 🤡

…as (BUG-60)

O arnês local serve o index.astro cru: o <script is:inline define:vars>
vazava SUPPORT_URL_BR como identificador livre e o boot morria com
pageerror em toda captura. Injetamos os defaults do site.ts.

gen-graffiti-layout parseava o layout anterior com lastIndexOf('}'),
que caía no rodapé do GRAFITE_FP — o parse lançava e o catch zerava os
outros mapas: regen de um mapa apagava o layout dos demais (BUG-60,
mesmo patch do /game 9cc84fe: fim do JSON por casamento de chaves).

Junto: regen dos artefatos stale do portão (foot-offsets, mat_*). O
mat_check.json media só 5 mapas; o regen expõe 12 e revela velho_oeste
fora dos tetos de croma — registrado como dívida MAT1 no KNOWN-RED
(medido também sem a rodada de fauna; não é regressão dela).

Commit-grande: regen de artefato de régua (mat_check/mat_shade têm
dados por arma×mapa e passam de 10k linhas).

Signed-off-by: rubenmarcus <rubenmarcus.dev@gmail.com>
Agent: OpenCode

@estraga-codigo estraga-codigo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Palhaço Mal

Palhaço Mal · revisor da casa · 🤡

mapa da mudança

  • arnês (6): KNOWN-RED.json, mat_check.json, mat_scenes.json, mat_shade.json
  • docs (1): KNOWN-BUGS.md

Tip

Risco BAIXO · nota 10/10 ▓▓▓▓▓▓▓▓▓▓

limpo pelos critérios das minhas ações desta rodada

Revisão de bot — fio não resolvido TRAVA o merge (a main exige conversa resolvida). Resolve ou refuta.


maquiagem de verde por cima de vermelho eu desenmascaro no primeiro golpe de vista 🤡

@estraga-codigo

Copy link
Copy Markdown
Collaborator

Esbirro

Esbirro · plantão da produção · 🕵️

check vermelho parado neste PR

Caution

Não sei classificar este vermelho. Segue o que o CI cuspiu — palpite de bot sobre bug alheio só atrapalha.

o quê valor
head 8ec513431906
checks vermelhos 1
classe outra
hora do plantão 2026-08-19T03:17:19.708Z

Vermelho:

ratchet (https://github.com/corosolto/client/actions/runs/32210307832/job/95941544038)

Rabo do log:

- `eval:invariants`: 39/55, nenhuma falha nova (MAT1 = DÍVIDA registrada)
- `check:fast`: vermelho único `changelog:check` (contagem de PRs da alpha.151) — **já vermelho na main**, pré-existente
2026-08-19T02:56:18.4794691Z ##[endgroup]
2026-08-19T02:56:21.0740162Z RÉGUA DE RATCHET   base origin/main
2026-08-19T02:56:21.0740508Z
2026-08-19T02:56:21.0744600Z Dívidas na main:   16   no PR: 17
2026-08-19T02:56:21.0745042Z
2026-08-19T02:56:21.0745709Z R1 · entradas NOVAS no KNOWN-RED: 1
2026-08-19T02:56:21.0746206Z      MAT1
2026-08-19T02:56:21.0746740Z    · MAT1  REPROVA — precisa de ratchet: +ID porque <motivo>
2026-08-19T02:56:21.0747298Z      FALHA
2026-08-19T02:56:21.0747470Z
2026-08-19T02:56:21.0747729Z R2 · entradas REMOVIDAS (quitação): 0
2026-08-19T02:56:21.0751087Z      PASSA
2026-08-19T02:56:21.0751675Z
2026-08-19T02:56:21.0752077Z R3 · saldo: +1 novas, −0 quitadas → 1 líquido
2026-08-19T02:56:21.0752546Z
2026-08-19T02:56:21.0753339Z ✗ RATC1  dívida nova sem motivo escrito — escreva ratchet: +<ID> porque <motivo> no corpo do PR
2026-08-19T02:56:21.0844448Z ##[error]Process completed with exit code 1.


de plantão: vi, registrei, hora no relógio 🕵️

@rubenmarcus

Copy link
Copy Markdown
Member Author

Superado pela feat/times-e-mapas-completo. Conferi as três partes deste PR uma a uma, e as três já estão lá:

  • serve.mjs injetando __SUPPORT — a branch grande resolve o mesmo defeito de forma mais geral: em vez do replace literal das duas URLs, ela implementa define:vars de verdade (serve.mjs:66-77), lendo as constantes do site.ts. Cobre qualquer variável, não só SUPPORT_URL_*. Evidência de que funciona: 17 capturas de fauna + 5 de preload chegando em live com zero pageerror.
  • gen-graffiti-layout por casamento de chaves — já presente (tools/gen-graffiti-layout.mjs:57), mesmo patch.
  • Dívida MAT1 no KNOWN-RED — é do mat_check regenerado nesta linha (velho_oeste fora do croma). Na branch grande os portões de material não estão vermelhos, então importar a entrada registraria débito que o repo não tem.

Fechando junto com o #376 para a main não pegar duas versões divergentes do mesmo arquivo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-staging Mudança precisa validação integrada em staging target:main PR apontando para a branch main

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants