el cartel ofrece 'e hablar' a dos celdas pero la tecla e no alcanza

[leocagli]

view() usa this.nearbyNpc(2) (lib/game.js:1218) y enter() usa this.nearbyNpc(), que es rango 1 (lib/game.js:828, firma en :436).

x=113 dist=2 | cabecera: | Roan, guardia | e hablar | -> e: ["aca no hay nada"]
x=114 dist=1 | cabecera: | Roan, guardia | e hablar | -> e: ["Roan, guardia: custodio el castillo..."]

Pasa con los nueve NPC: es el mismo par de llamadas.

Arreglo

Usar el mismo rango en los dos lados.

[adityawaghamare04]

🔀 Pull Request Created

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

Proposed Solution & Patch

🛠️ Proposed Solution (by Aditya Waghamare)

Analysis

The bug occurs because view() (lib/game.js:1218) checks for nearby NPCs using a radius of 2 (this.nearbyNpc(2)), whereas enter() (lib/game.js:828) uses the default radius of 1 (this.nearbyNpc()). This causes the UI to display "e hablar" (via view) when the player is 2 tiles away from an NPC, but pressing E fails ("aca no hay nada") because enter() requires distance 1.

Fix

Update enter() (or wherever NPC interaction is triggered) to use the same range/radius check as view(), or standardize the range across both functions to 2 so that interaction matches the visual prompt (or vice versa depending on intended design, but matching view()'s range of 2 or unifying via a constant/parameter makes the prompt and action consistent).

Looking at the issue details:
view() uses this.nearbyNpc(2) and enter() uses this.nearbyNpc(). To make the E key reach NPCs at distance 2 where the sign/prompt says e hablar, enter() should be updated to this.nearbyNpc(2).

Implementation

// In lib/game.js
// Update enter() to match view()'s range of 2:
enter() {
  // ...
  const npc = this.nearbyNpc(2);
  // ...
}

Testing

Verify that standing at distance 2 (dist=2) from an NPC correctly triggers the interaction when pressing E, eliminating the "aca no hay nada" false negative when the UI shows "e hablar".

---

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

[leocagli]

Sigue viva, verificado hoy

Contra main 066b280:

nearbyNpc se llama con tres formas distintas: range = 1 (la firma), el default, y 2. El cartel usa 2 y la tecla usa el default.

Como comprobarlo:

grep -n nearbyNpc( lib/game.js

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]

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