El protocolo binario esta duplicado entre cliente y servidor

[leocagli]

Problema

El protocolo binario del juego está implementado dos veces, sin nada que garantice que las dos versiones coincidan:

Archivo	Líneas
frontend/lib/aowProtocol.ts	2.275
server/src/protocol.ts	4.430

Son dos definiciones separadas del mismo formato de bytes. Si alguien agrega un paquete o cambia el orden de un campo en un lado y se olvida del otro, el juego no falla al compilar: falla en runtime, con datos corruptos y comportamiento raro que es muy difícil de rastrear hasta la causa.

Es el tipo de bug que le va a pasar a cualquier contribuidor nuevo que toque el protocolo.

Referencia

El proyecto elpatoenlasolas/argentum-web (MIT) resolvió esto con un packages/protocol compartido dentro de un monorepo pnpm: los tipos y la definición del protocolo viven en un solo lugar y los importan tanto el cliente como el servidor.

Alcance

1. Extraer a un paquete compartido lo que hoy está duplicado:
   • identificadores de paquete (opcodes)
   • forma de cada mensaje
   • constantes compartidas (rangos de visión, tamaños)
2. Que cliente y servidor importen de ahí, en vez de tener cada uno su copia.
3. Que un cambio incompatible rompa la compilación, que es exactamente lo que hoy no pasa.
4. Tests de ida y vuelta: serializar y deserializar cada tipo de paquete y verificar que se recupera lo mismo.

Lo difícil

No es un copiar y pegar. Las dos implementaciones probablemente divergieron en detalles, y hay que reconciliarlas antes de unificar. Conviene:

• Empezar por los opcodes y las constantes, que es lo más fácil y lo que más se desincroniza
• Migrar los paquetes de a uno, no todos juntos
• Dejar tests de ida y vuelta antes de tocar nada, para tener red

Criterios de aceptación

• Existe un paquete compartido con la definición del protocolo
• Cliente y servidor lo importan, sin copias locales de lo migrado
• Cambiar un opcode en el paquete compartido rompe la compilación de ambos lados
• Hay tests de serialización y deserialización
• El juego sigue funcionando igual (verificar entrando y jugando)

Nota

Se puede hacer por partes y cada parte deja el juego funcionando. No hace falta migrar los 6.700 renglones de una vez.

[ceshez]

Hi, I would like to work on this issue.

I have reviewed the scope and acceptance criteria. I can first add round-trip serialization/deserialization tests, then reconcile the duplicated protocol definitions and extract the shared opcodes, message types, and constants into a common package.

I will migrate the client and server incrementally, keep compatibility during the transition, and verify that the game continues working correctly.

I will keep the PR focused and include clear documentation for the migration.

[imrichkocai-sys]

Me crucé con este repo buscando proyectos que definen bien sus tareas — y la verdad, estos issues están escritos como specs de bounty listos para usar: alcance, criterios de aceptación, hasta notas de dificultad. No es común.

Soy Imrich, fundador de Archimedes Market (https://archimedes.market). Hacemos exactamente esto: tomás una tarea así de bien definida, le ponés un monto, y un ingeniero externo la entrega — con verificación independiente del trabajo, pago por escrow (solo se libera si cumple los criterios), y transferencia de IP. Si algún día querés ponerle presupuesto a un issue como este para sacarlo adelante, me encantaría ayudarte a armarlo — el primero te lo acompaño personalmente.

Y si no, igual felicitaciones por cómo documentan las tareas. Suerte con OpenAO 🎮