fix(mapa): abrir accesos peatonales a los jardines de la ciudad

[s6pa1rta3n-lab]

Los jardines de la ciudad dibujaban un perimetro continuo con border(), dejando 4946 celdas caminables completamente selladas sin entrada para el jugador.

Se agregaron cuatro accesos abiertos en las direcciones cardinales siguiendo el mismo patron que civicSquare(), permitiendo que el jugador camine hacia el interior del jardin y recorra los senderos y el estanque central.

Tambien se agrego un test en test/index.js que valida la alcanzabilidad de los jardines desde el spawn.

Closes #5

[leocagli]

This one no longer merges into main.

main moved a lot since it was opened: the save slot system, the coliseum map, the
world boss, and several render fixes all landed. The branch conflicts, so I cannot
test what the merged result would do.

If you rebase it on today's main I will re-run it and report the numbers here. The
current baseline is 65 tests and 515 assertions in green.

One thing worth checking while rebasing: another contributor opened a pull request
for the same issue, so it is worth a look before spending time on the conflict, in
case the work is already covered.

[s6pa1rta3n-lab]

Rebased onto latest main, resolved conflicts, and verified npm test and npm run lint are green.

[gitar-bot]

Code Review ✅ Approved 1 resolved / 1 findings

Opens pedestrian access points to the city gardens and adds a reachability test, addressing the unmapped tile crash finding. No remaining issues found.

✅ 1 resolved

✅ Bug: Reachability test crashes on unmapped ' ' tile before assertions

📄 test/index.js:1061
The new BFS reads TILES[city.rows[y][x]].solid without the || NOWHERE guard that production tileAt uses. Building/plaza interiors and the fountain surround are painted with space characters (' ') that are walkable-adjacent, and ' ' is not a key in TILES. I confirmed the walk reaches a space at (147,89) — before ever touching the gardens — so TILES[' '] is undefined and .solid throws a TypeError, making the test error out instead of validating anything (the four t.ok assertions never run). Guard the lookup so unknown glyphs count as solid, matching tileAt. (The map.js fix itself is correct: with the guard the four garden points are reachable.)

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply	Compact
gitar auto-apply:on	gitar display:verbose

---

Important

Your trial ends in 7 days — upgrade now to keep code review, CI analysis, auto-apply, custom automations, and more.

_{Was this helpful? React with 👍 / 👎 | Gitar}

[s6pa1rta3n-lab]

Rebased onto current main, resolved any conflicts, and verified all tests pass cleanly.

[s6pa1rta3n-lab]

Rebased on latest main, resolved any conflicts, and ensured all tests and linting pass locally.