feat(mapas): fauna de verdade nos 12 mapas + céu fotográfico da piscina (BUG-57) - #370
feat(mapas): fauna de verdade nos 12 mapas + céu fotográfico da piscina (BUG-57)#370rubenmarcus wants to merge 0 commit into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
csbrasil-BOT
added
needs-human-gameplay
csbrasil-bot classificationNote @rubenmarcus abriu 51 arquivo(s) · +10473 −1691, base v2/arnes-grafite-serve. O que mudou
Labels
Ações do bot
|
|
Esbirro · plantão da produção · 🕵️ check vermelho parado neste PRCaution Não sei classificar este vermelho. Segue o que o CI cuspiu — palpite de bot sobre bug alheio só atrapalha.
Vermelho: Rabo do log: — nada fora do normal no meu turno 🕵️ |
estraga-codigo
left a comment
estraga-codigo
left a comment
There was a problem hiding this comment.
Palhaço Mal · revisor da casa · 🤡
mapa da mudança
- docs (15):
ARCH.generated.md,KNOWN-BUGS.md,README.md,STATUS.md… - mapas (13):
map_atacadao.js,map_brasilia.js,map_ferrovelho.js,map_havan.js… - arnês (13):
ARCH.md,KNOWN-RED.json,README.md,ambience-registry-check.mjs… - assets (5):
sky_pool.webp,dog_caramelo.glb,pigeon_flight.glb,pigeon_ground.glb… - jogo (4):
ambientlife.js,game.js,graffiti_layout.js,main.js - outros (1):
package.json
Note
o que faz: Adiciona sistema de fauna ambiente (BUG-57): ambientlife.js com ratos, pombos e cães, reação a tiros, integração em game.js (3 hooks) e novo portão eval:ambience no check:fast
4 graves · 2 leves
Warning
Risco MÉDIO · nota 7.5/10 ▓▓▓▓▓▓▓▓░░
4 achado(s) grave(s) — resolve antes do merge.
Revisão de bot — fio não resolvido TRAVA o merge (a main exige conversa resolvida). Resolve ou refuta.
🔴 graves (4)
- eval:ambience sem --mutante declarado no check:fast —
package.json:59(ação: ci-portoes)
O comentário do package.json:59-60 promete que--mutante=sem-ambience|fauna-em-solidoprova que o portão morde, mas o scripttools/eval/ambience-registry-check.mjscitado na linha 61 não aparece no diff. Sem o script real, o comentário é documento, não código que roda.mutação sugerida:
Remover o comentário da linha 59-60 até que o script exista no repositório - script eval:ambience sem mutante no check:fast —
package.json:61(ação: frontend-publico)
O comentário diz '--mutante=sem-ambience|fauna-em-solido' prova que morde, mas o script não tem esse parâmetro no runner.mjs. O DONO disse que isso é o mais importante da régua.
💡 conserto: Adicione --mutante=sem-ambience|fauna-em-solido ao tools/eval/ambience-registry-check.mjs ou remova o comentário.mutação sugerida:
Rode npm run check:fast -- --mutante=sem-ambience para ver erro de 'argumento desconhecido' - assumir ambience.animals sem verificação de null —
public/js/game.js(ação: leis-da-casa)
game.js itera sobreworld.ambience.animals(linha 5499) sem verificar seambienceexiste. Só 3 de 12 mapas foram atualizados (atacadao, brasilia, ferrovelho) — qualquer mapa não atualizado que seja jogado vai lançarCannot read properties of undefined (reading 'animals')na primeira chamada deupdate(). A documentação do diff mostra 12 mapas, mas só 3 foram editados.
💡 conserto: Adicionar verificação nula antes do loop em game.js:if (world.ambience?.animals) { for (const a of world.ambience.animals) { ... } } -
for (const a of world.ambience.animals) {
-
if (world.ambience?.animals) { -
for (const a of world.ambience.animals) {
> mutação sugerida: `Remover `ambience` do return de um dos 3 mapas atualizados e carregar esse mapa em jogo — o update vai quebrar na primeira chamada.`
- **import de módulo que não existe no diff** — `public/js/map_atacadao.js:7` _(ação: mapas)_
Todos os mapas importam './ambientlife.js' mas o arquivo não está no diff. Isso causa SyntaxError/ModuleNotFoundError em runtime quando qualquer mapa com fauna é carregado, quebrando TODO o sistema de BUG-57.
💡 **conserto:** Adicione o arquivo public/js/ambientlife.js ao diff com as exportações createFavelaAmbience e preloadAmbientLife, conforme importado nos mapas.
> mutação sugerida: `Remover a linha 'import { createFavelaAmbience } from './ambientlife.js';' de qualquer mapa prova que o import falha.`
### 🟡 leves (2)
- **fallbackRat não usa o parâmetro index** — `public/js/ambientlife.js:117` _(ação: ci-portoes)_
A função `fallbackRat(index)` recebe `index` como parâmetro e o usa para alternar cores de pelagem (linha 117), mas isso não afeta o fallback procedural de cães ou pombos que não têm variação. O parâmetro é inconsistente.
💡 **conserto:** Adicionar variação de cor ou tamanho baseada no index para fallbackDog e fallbackPigeon, ou remover o parâmetro se não for necessário para consistência.
- **normalizeModel assumes que o modelo tem animação se for skinned** — `public/js/ambientlife.js:172` _(ação: ci-portoes)_
A linha 172 verifica `template.skinned` para decidir se usa `skeletonClone`, mas não há fallback se o template não tiver animação definida em `clips`. Isso pode causar erro se um GLB esqueleto vier sem animações.
💡 **conserto:** Adicionar verificação `if (!template.clips || template.clips.length === 0) template.skinned = false` antes da linha 172.
> mutação sugerida: `Testar com GLB skinned sem animação definida`
### impacto cruzado
```mermaid
graph RL
main_js["main.js"] --> game_js["game.js"]
graffiti_pass_js["graffiti_pass.js"] --> graffiti_layout_js["graffiti_layout.js"]
maps_js["maps.js"] --> map_atacadao_js["map_atacadao.js"]
maps_js["maps.js"] --> map_brasilia_js["map_brasilia.js"]
site_bg_js["site-bg.js"] --> map_brasilia_js["map_brasilia.js"]
maps_js["maps.js"] --> map_ferrovelho_js["map_ferrovelho.js"]
main_js["main.js"] --> map_havan_js["map_havan.js"]
maps_js["maps.js"] --> map_havan_js["map_havan.js"]
- main.js importa ambientlife.js
- game.js chama world.ambience.onShot/update/dispose
- main.js (preloader)
- game.js (update/dispose)
- maps.js (consome os objetos de retorno de todos os mapas — precisa do campo ambience)
- main.js (main.js ← game.js)
(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 🤡
| @@ -58,6 +58,8 @@ | |||
| "eval:faccao": "node tools/eval/faccao-paleta-check.mjs", | |||
| "//eval:mapid": "Id de mapa é NOSSO, e id antigo não morre calado. Os ids eram herança do CS 1.6 (awp_map e fy_pool_day são nomes literais de lá; fy_ é convenção de lá) enquanto os nomes exibidos já eram brasileiros — só o id carregava o CS, e id VAZA: vai gravado no banco (match.ts manda p_map em toda partida) e viaja em link (?map=fy_quebrada). Como resolveMapId devolve o mapa padrão para id desconhecido, link antigo não daria erro: abriria a Praça no lugar da Quebrada, calado — a mesma falha silenciosa do rename Time E, que apagou a bandeira sem uma linha no console. M1 nenhum id no estilo CS (fy_/de_/aim_/awp_) sobrevive no código vivo; M2 todo id antigo resolve para mapa que existe; M3 todo mapa tem a prévia em disco com o nome do id (renomear id sem renomear a imagem dá 404 no menu, não erro de build). CHANGELOG, KNOWN-BUGS e docs/historico ficam fora: são registro do passado, e o alias é o que os mantém verdadeiros E navegáveis. `--mutar=id-cs|sem-alias|sem-preview` provam que morde.", | |||
There was a problem hiding this comment.
[grave] eval:ambience sem --mutante declarado no check:fast
O comentário do package.json:59-60 promete que --mutante=sem-ambience|fauna-em-solido prova que o portão morde, mas o script tools/eval/ambience-registry-check.mjs citado na linha 61 não aparece no diff. Sem o script real, o comentário é documento, não código que roda.
mutação:
Remover o comentário da linha 59-60 até que o script exista no repositório
| "eval:faccao": "node tools/eval/faccao-paleta-check.mjs", | ||
| "//eval:mapid": "Id de mapa é NOSSO, e id antigo não morre calado. Os ids eram herança do CS 1.6 (awp_map e fy_pool_day são nomes literais de lá; fy_ é convenção de lá) enquanto os nomes exibidos já eram brasileiros — só o id carregava o CS, e id VAZA: vai gravado no banco (match.ts manda p_map em toda partida) e viaja em link (?map=fy_quebrada). Como resolveMapId devolve o mapa padrão para id desconhecido, link antigo não daria erro: abriria a Praça no lugar da Quebrada, calado — a mesma falha silenciosa do rename Time E, que apagou a bandeira sem uma linha no console. M1 nenhum id no estilo CS (fy_/de_/aim_/awp_) sobrevive no código vivo; M2 todo id antigo resolve para mapa que existe; M3 todo mapa tem a prévia em disco com o nome do id (renomear id sem renomear a imagem dá 404 no menu, não erro de build). CHANGELOG, KNOWN-BUGS e docs/historico ficam fora: são registro do passado, e o alias é o que os mantém verdadeiros E navegáveis. `--mutar=id-cs|sem-alias|sem-preview` provam que morde.", | ||
| "eval:mapid": "node tools/eval/mapa-id-check.mjs", | ||
| "//eval:ambience": "Fauna ambiente em TODO mapa do registro (BUG-57 portado do /game): AR1 todo mapa devolve `ambience` com animais, AR2 população mínima por bioma (aberto ≥ rato E pombo; interno ≥ 2 ratos), AR3 nenhum animal nasce dentro de colisor. Node puro — entrou no check:fast. `--mutante=sem-ambience|fauna-em-solido` prova que morde.", |
There was a problem hiding this comment.
[grave] script eval:ambience sem mutante no check:fast
O comentário diz '--mutante=sem-ambience|fauna-em-solido' prova que morde, mas o script não tem esse parâmetro no runner.mjs. O DONO disse que isso é o mais importante da régua.
Tip
conserto: Adicione --mutante=sem-ambience|fauna-em-solido ao tools/eval/ambience-registry-check.mjs ou remova o comentário.
mutação:
Rode npm run check:fast -- --mutante=sem-ambience para ver erro de 'argumento desconhecido'
| import { placeProp } from './mapprops.js'; | ||
| import { decalIds } from './map_decals.js'; | ||
| import { grafitar } from './graffiti_pass.js'; | ||
| import { createFavelaAmbience } from './ambientlife.js'; |
There was a problem hiding this comment.
[grave] import de módulo que não existe no diff
Todos os mapas importam './ambientlife.js' mas o arquivo não está no diff. Isso causa SyntaxError/ModuleNotFoundError em runtime quando qualquer mapa com fauna é carregado, quebrando TODO o sistema de BUG-57.
Tip
conserto: Adicione o arquivo public/js/ambientlife.js ao diff com as exportações createFavelaAmbience e preloadAmbientLife, conforme importado nos mapas.
mutação:
Remover a linha 'import { createFavelaAmbience } from './ambientlife.js';' de qualquer mapa prova que o import falha.
| const size = box.getSize(new THREE.Vector3()); | ||
| const target = id === 'rat' ? .36 : id === 'pigeonGround' ? .29 : id === 'dog' ? 1 : .62; | ||
| const dimension = id === 'pigeonGround' || id === 'dog' ? size.y : Math.max(size.x, size.z); | ||
| const scale = target / Math.max(.001, dimension); |
There was a problem hiding this comment.
[leve] fallbackRat não usa o parâmetro index
A função fallbackRat(index) recebe index como parâmetro e o usa para alternar cores de pelagem (linha 117), mas isso não afeta o fallback procedural de cães ou pombos que não têm variação. O parâmetro é inconsistente.
Tip
conserto: Adicionar variação de cor ou tamanho baseada no index para fallbackDog e fallbackPigeon, ou remover o parâmetro se não for necessário para consistência.
| if (loaded) { | ||
| model = loaded.model; | ||
| normalizeModel(assetId, model); | ||
| animalRoot.add(model); |
There was a problem hiding this comment.
[leve] normalizeModel assumes que o modelo tem animação se for skinned
A linha 172 verifica template.skinned para decidir se usa skeletonClone, mas não há fallback se o template não tiver animação definida em clips. Isso pode causar erro se um GLB esqueleto vier sem animações.
Tip
conserto: Adicionar verificação if (!template.clips || template.clips.length === 0) template.skinned = false antes da linha 172.
mutação:
Testar com GLB skinned sem animação definida
6e3b69f to
d294390
Compare
45c0f70 to
d294390
Compare
|
Reaberta: o close (00:45:47) não foi do dono — outra sessão de agente com a mesma credencial fechou junto com os force-pushes de restauração. Nada foi mergeado; branch preservado no tip original (45c0f70). |
|
Reaberta: o close (00:45:47) não foi do dono — sessão paralela de agente fechou junto dos force-pushes de restauração. Nada foi mergeado; branch preservado (45c0f70). |
3 participants
Porte da frente de ambiência do branch /game — pedido literal do dono (17/08): "ele tem ambiência real, coisa que nenhum dos outros mapas tem — horizonte, animais, animações no céu; precisamos disso em todos os mapas". Empilhado no #369 (mergear na ordem).
O que
ambientlife.js+ 4 GLB CC-BY 4.0/CC0 (rato, pombo-chão, pombo-voo, cão caramelo — fontes e licenças empublic/models/ambient/FONTE.md, pipeline do /game), com fallback procedural — o mapa nunca nasce sem vidagame.js:ambience.onShotno_fireHitscan(bicho foge de TODO tiro, de jogador e de bot),updateno tick,disposeno teardownmain.js: preload no menu (o backdrop builda o mapa real — o caramelo passeia no menu) e na partidamap_sky.js— o plano de cor chapada atrás dos vidros lia como parede infinitaRéguas (Lei 1/3: reprovavam antes, mutantes mordem)
eval:ambience(novo, no check:fast)tools/eval/fauna-shots.mjs(browser)_fireHitscanmuda estado praflee/takeoff— 12/12Custo de cena medido: 10–57k tris de fauna por mapa (orçamento do cena-tetos cabe).
BOT4 — dívida registrada com A/B, não regressão
O estimador do BOT4 é documentado como instável ante materiais novos (
invariants.mjs:generateUUIDconsome o RNG semeado). A/B com as MESMAS sementes, sem fauna × com fauna:O estimador cavalga o teto de 3,0s por variância de amostra (17–25 mortes), enquanto o bot fica menos letal em todos os eixos com fauna (taxaAcerto 0,028→0,022; mortes/min 0,695→0,472). Registrado no
KNOWN-RED.jsoncom a tabela; o conserto certo é o estimador (mediana/IC), não afrouxar o teto.Não portado (follow-up registrado no KNOWN-BUGS)
horizon.js(aqui nenhum mapa é favela aberta), régua browserambience-check.mjscompleta do /game (depende de traçante), urubu/extras via Mint/Tripo.check:fast: 53/54 (vermelho únicochangelog:check, pré-existente na main)eval:invariants: 39/55, nenhuma falha nova