codigo de dibujo escrito, exportado y que no llega nunca a la pantalla

[leocagli]

Hay bastante arte y generacion que nadie invoca, en varios casos mantenido vivo con void para que el lint no proteste.

Que	Donde	Tamano
makeCityRows	lib/map.js:370, unico uso void en :583	~213 lineas
market()	lib/map.js:942, unico uso void en :956	dibuja un mercado que la ciudad no tiene
LEGACY_NPC_*_SPRITES	lib/map.js:138-236, void en :243-244	~100 lineas
makePersonSprite (25x25)	lib/sprites.js:119	~95 lineas
makeMapPersonSprite (9x7)	lib/sprites.js:243	~70 lineas
arenaPane / combatScreen	lib/render.js:957 y :1269	~115 lineas
Runa.arena()	lib/game.js:1067	~50 lineas
NPC_MASTER_SPRITES	lib/map.js:246	exportado, sin usar

Entre makePersonSprite y makeMapPersonSprite son ~250 de las 452 lineas de sprites.js que no llegan a la pantalla.

test/index.js:377 afirma "the faithful high-resolution knight is preserved", pero ese caballero no se dibuja en ninguna pantalla del juego.

Ademas

lib/sage.js esta huerfano: 1494 lineas, se presenta como "El sabio. Un NPC que traduce lenguaje natural", y el unico require esta en su test. El modulo anda: Sage.ask("usa la ballesta si esta lejos") devuelve ?foe.dist >= 5\n equip crossbow.

Y el juego manda al jugador a editar script.txt a mano con la tecla ?, que es exactamente el problema que el sabio resuelve.

O se cablea (un NPC con action { kind: 'sage' }) o se saca. Hoy son 1494 lineas testeadas que el jugador no puede alcanzar.

Y en lib/game.js:933-941 quedo un bloque JSDoc huerfano describiendo una funcion que ya no esta ahi.

[leocagli]

Sumo una mas a la lista, del recorrido completo:

LEGEND esta definida y exportada y no la consume nadie (lib/map.js:125).

Va junto al resto: o se usa (seria util para una pantalla de ayuda que explique
que significa cada glifo del mapa, que hoy el juego no tiene) o se borra.

[leocagli]

Sigue viva, verificado hoy

Contra main 066b280:

LEGEND sigue exportada desde lib/map.js y no la importa nadie: cero apariciones fuera de ese archivo

Como comprobarlo:

grep -rn LEGEND lib/

Nota para quien la tome: main avanzo bastante (sistema de guardado con ranuras,
mapa del Coliseo, jefe mundial), asi que conviene partir del main de hoy y no de
un fork viejo. La base de pruebas es 65 tests / 515 aserciones en verde, y un
arreglo no deberia bajar de ahi.

El trabajo se reclama comentando la issue con un plan concreto (que archivo, que
funcion, y como lo vas a verificar). Se asigna por la calidad de ese plan.

[blippip69]

Tomo esta issue. Plan concreto:

1. Cablear lib/sage.js (no borrarlo) — es el mayor activo huérfano (1494 líneas testeadas):

• Nuevo NPC en la ciudad con action { kind: 'sage' } (mismo patrón que los NPC existentes con action)
• Al interactuar (e), abre el prompt de script existente pero routéado a Sage.ask():
  lenguaje natural → reglas; muestra el resultado traducido y lo guarda como script activo
• Reemplaza el flujo actual "editar script.txt a mano" de la tecla ? (el sabio queda como
  entrada principal y ? sigue funcionando como fallback manual)

2. Consumir LEGEND — pantalla de ayuda con glifos del mapa:

• Nueva tecla l (o sección dentro del panel de ayuda existente) que renderiza
  LEGEND en un box (lib/render.js:box) titulada "glifos del mapa"

3. Conectar el arte muerto:

• makeCityRows → reemplazar las filas hardcodeadas actuales de la ciudad por su salida,
  verificando tile por tile que no cambien los sólidos (comparación de grid antes/después)
• market() → colocar el mercado en una manzana libre del centro
• makePersonSprite/makeMapPersonSprite → sustituir a los sprites actuales del héroe/NPCs si
  son pixel-compatibles; si no, se borra y queda documentado en el PR
• arenaPane/combatScreen/Runa.arena() → cablear la arena al menú del título o borrar

4. Limpieza: JSDoc huérfano en lib/game.js:933-941, LEGACY_NPC_*_SPRITES,
NPC_MASTER_SPRITES (borrar o usar, decidir por tamaño del diff)

Verificación: base 67 tests / 517 asserts en verde desde main de hoy (ya estoy sobre ella),
ningún arreglo baja de eso; cada conexión de arte con comparación de grid antes/después;
Salida de Godot... n/a (juego de terminal). PR con capturas del título/ciudad/arena.

Mi wallet Stellar para el pago: GDWLC6C7YHSCGRRGQVDVSBWILP3PIV47Y7GCSCCJXGB5ZLNV6NETGOGZ

[blippip69]

PR abierto: #50

Mi wallet Stellar para el pago: GDWLC6C7YHSCGRRGQVDVSBWILP3PIV47Y7GCSCCJXGB5ZLNV6NETGOGZ

[Joaco2603]

Hi @leocagli, I'd like to take this issue.

I'll work from the current main and first verify each reported dead-code path before removing or wiring anything.

My plan:

• lib/map.js: trace makeCityRows, market(), LEGACY_NPC_*_SPRITES, NPC_MASTER_SPRITES, and LEGEND, confirming their references with grep/static search. I'll remove code that is genuinely unreachable rather than keeping it alive through void.
• lib/sprites.js: verify whether makePersonSprite and makeMapPersonSprite have any runtime consumers. If they are fully superseded by the current sprite path, remove them and update the related tests, including the high-resolution knight assertion.
• lib/render.js / lib/game.js: trace arenaPane, combatScreen, and Runa.arena() against the current Coliseum/combat implementation before deciding what can safely be removed.
• lib/game.js: remove the orphaned JSDoc block around lines 933-941 if it still exists on current main.
• lib/sage.js: I don't want to delete this blindly since it solves an existing player-facing problem. I'll inspect the current action/NPC flow and, if it fits cleanly, wire Sage through an action { kind: 'sage' } path so the feature becomes reachable from the game. If the current architecture makes that inappropriate, I'll document that before removing it.
• LEGEND: verify again that it has no consumers. I'll either connect it to an existing help flow if there is a natural integration point, or remove it rather than introducing a new screen solely to preserve dead code.

Verification:

• Start from current main.
• Run the existing test suite before making changes and establish the current baseline (65 tests / 515 assertions as reported).
• Run the full suite after the changes and ensure there are no regressions.
• Search again for the removed symbols and stale void references.
• Manually exercise any user-facing path I wire, particularly Sage, rather than relying only on its unit tests.

If this approach looks good, I'd be happy to take it.