el heroe le come el dibujo al NPC justo a la distancia en la que se puede hablar

[leocagli]

El NPC ocupa 7 columnas en pantalla pero bloquea una sola celda. Resultado: para hablar hay que pararse adentro del cuerpo del NPC, y los dos sprites se pisan.

--- heroe a 1 celda de Alma (habla? true)
   |##|,,,,|(o,O),,
   ,,,,,,,,|/[/A\,,      <- el torso del heroe se comio el emblema [+]
--- heroe a 2 celdas (habla? true)
   |##|,,,,|(o,oO,,      <- la O del heroe reemplaza el ) de la cara
--- heroe a 3 celdas (habla? false)
   ,,,,,,,,|/[+]/A\      <- recien aca los dos se leen enteros

El NPC ocupa ax-3..ax+3 y el heroe hx-2..hx+1: hacen falta 6 celdas para no solaparse, pero se habla hasta 1 o 2.

Ademas se puede atravesar al NPC entero caminando, sin un solo mensaje de bloqueo.

Arreglo

fieldPane ya resuelve algo equivalente en lib/render.js:866-882: separa la presentacion del foe activo sin tocar la distancia logica. Aplicar lo mismo en mapPane. O que npcAt() bloquee toda la huella del sprite.

[adityawaghamare04]

🔀 Pull Request Created

I have opened a Pull Request with the verified solution patch: #33

Proposed Solution & Patch

🛠️ Proposed Solution (by Aditya Waghamare)

Analysis

The NPC occupies a visual footprint of 7 columns on screen (ax-3 to ax+3), but only blocks a single tile/cell in logic. This causes the hero sprite to overlap and visually overwrite the NPC art when standing at the interaction threshold distance, and allows the player to walk right through the NPC sprite.

Fix

Update mapPane (and npcAt / collision logic) to match the presentation pattern used in lib/render.js (like fieldPane), ensuring the NPC's full horizontal footprint blocks movement and defines the interaction bounds appropriately without breaking logical positioning.

Implementation

// lib/mapPane.js (or equivalent map & NPC collision handler)
function npcAt(x, y) {
  // Check if coordinates fall within the NPC's extended visual/collision footprint
  for (const npc of npcs) {
    if (y === npc.y && x >= npc.x - 3 && x <= npc.x + 3) {
      return npc;
    }
  }
  return null;
}

// Update collision checking so the entire footprint blocks movement
function isWalkable(x, y) {
  if (npcAt(x, y)) return false;
  // ... existing tile checks ...
}

Testing

1. Verify that trying to walk into any of the 7 columns occupied by the NPC correctly blocks movement.
2. Verify that talking to the NPC works smoothly at the adjusted interaction threshold without visual sprite overlapping.

---

Submitted by Aditya Waghamare
💰 Payout Address (Base L2 / EVM): 0xb61dBcdBc3407F71EaCb64D4CBFAcf9FFfe2415C

[blippip69]

PR abierto: #43
ВїCuГЎl es la recompensa asignada a este issue?
Mi wallet Stellar para el pago: GDWLC6C7YHSCGRRGQVDVSBWILP3PIV47Y7GCSCCJXGB5ZLNV6NETGOGZ