Etapa 2: API de colocacion y movimiento de NPCs en mapa

[leocagli]

Parte de #2 (modo construcción). Depende de la capa de persistencia.

Problema

api/src/lib/mapNpcStorage.ts lee la colocación de NPCs de cada mapa y valida el modelo, pero no ofrece ninguna forma de modificarla. Poner un NPC nuevo hoy requiere editar npcs.json a mano y reiniciar.

Modelo de datos actual

type MapNpcPlacement = {
    mapNum: number;
    x: number;
    y: number;
    npcIndex: number;
    movement?: number;
}

Validación existente: enteros, x > 0, y > 0, npcIndex > 0.

Alcance

1. Poner un NPC en (mapa, x, y) con su npcIndex y patrón de movimiento.
2. Mover un NPC ya colocado a otra coordenada.
3. Sacar un NPC de un mapa.
4. Listar los NPCs de un mapa.

Validaciones necesarias

• El npcIndex existe en el catálogo de NPCs (hay 340 plantillas cargadas en DB)
• La coordenada está dentro de la grilla y no es un tile bloqueado: un NPC dentro de una pared no puede moverse ni ser atacado
• No apilar dos NPCs en el mismo tile, salvo que el juego lo permita explícitamente
• Límite de NPCs por mapa, para no poder tirar abajo el server llenando un mapa de criaturas

Criterios de aceptación

• Poner, mover y sacar NPCs funciona y persiste
• Un npcIndex inexistente devuelve error de validación
• Colocar sobre un tile bloqueado se rechaza
• Se respeta el límite por mapa
• Los NPCs colocados aparecen en el juego tras recargar el mapa
• Tests de integración

Archivos relevantes

• api/src/lib/mapNpcStorage.ts (lectura y validación existentes)
• api/src/mapas_source/mapa_<n>/npcs.json
• api/src/repositories/gameNpcs.ts (catálogo de plantillas de NPC)

[chfr19820610-cell]

I have strong experience building REST APIs and working with game-related data models, including NPC placement and map persistence. I am confident I can implement the required endpoints to add, remove, and update NPCs on the map without requiring manual JSON edits or restarts. I am already familiar with the project structure and the existing mapNpcStorage.ts logic. I can complete this task within 3–4 days after being assigned.

[chfr19820610-cell]

I have strong experience building REST APIs and working with game-related data models, including NPC placement and map persistence. I am confident I can implement the required endpoints to add, remove, and update NPCs on the map without requiring manual JSON edits or restarts. I am already familiar with the project structure and the existing mapNpcStorage.ts logic. I can complete this task within 3–4 days after being assigned.

[chfr19820610-cell]

I have strong experience building REST APIs for game systems and real-time entity management, including NPC placement and movement logic. I am fully comfortable working with TypeScript, JSON-based storage, and persistence layers similar to those used in OpenAO. I can implement the required API endpoints to add, update, and move NPCs on the map without requiring manual file edits or restarts. I expect to complete this task within 3–5 days, including testing and validation. Please assign me this bounty.

[chfr19820610-cell]

I have strong experience building REST APIs for game systems and real-time entity management, including NPC placement and movement logic. I am fully comfortable working with TypeScript, JSON-based storage, and persistence layers similar to those used in OpenAO. I can implement the required API endpoints to add, update, and move NPCs on the map without requiring manual file edits or restarts. I expect to complete this task within 3–5 days, including testing and validation. Please assign me this bounty.

[chfr19820610-cell]

I have strong experience building REST APIs and working with game backend logic, including NPC placement and movement systems. I am comfortable extending existing storage layers like mapNpcStorage.ts and adding mutation endpoints without breaking current validation. I can implement the required API for placing and moving NPCs on the map within 3–4 days after starting. Please assign this bounty to me — I’m confident I can deliver a clean, testable solution.

[Rodrigoue9]

¡Hola @leocagli! Me postulo formalmente para tomar la Issue #8 (Etapa 2: API de colocación y movimiento de NPCs en mapa).

Plan de Implementación Concreto:

1. Archivos a modificar / crear:

   • api/src/lib/mapNpcStorage.ts: Extender con funciones CRUD placeMapNpc, moveMapNpc, removeMapNpc y listMapNpcs con soporte para persistencia en api/src/mapas_source/mapa_<n>/npcs.json y DB.
   • api/src/repositories/worldBuilder.ts: Integrar validación de NPCs contra el catálogo gameNpcs (340 templates) y verificación de collision con tiles bloqueados (blocked: true / isTileBlocked).
   • api/src/server.ts: Exponer endpoints REST:
     • GET /api/editor/maps/:mapNum/npcs -> Lista NPCs colocados.
     • POST /api/editor/maps/:mapNum/npcs -> Colocar NPC con validaciones.
     • PUT /api/editor/maps/:mapNum/npcs/move -> Mover NPC verificando destino libre y no bloqueado.
     • DELETE /api/editor/maps/:mapNum/npcs -> Quitar NPC del mapa.
   • api/src/tests/mapNpcPlacement.integration.test.ts: Nueva suite completa de integration tests.

2. Validaciones que incluiré:

   • Verificación de que npcIndex exista en el catálogo de NPCs.
   • Verificación de límites de coordenadas (grilla 1..100).
   • Rechazo estricto si el tile de destino está bloqueado (blocked: true).
   • Rechazo de apilamiento en el mismo tile.
   • Límite máximo de seguridad de NPCs por mapa (máx. 50 NPCs).

3. Verificación:

   • Tests de integración automáticos cubriendo colocación, movimiento, borrado, listado y todos los casos de error (tile bloqueado, duplicado, NPC inexistente).

¿Me la podrías asignar para empezar? ¡Muchas gracias! 🚀