Introducir un error lógico visible

Vamos a introducir un error pequeño para tener algo concreto que revisar. Hacelo vos desde el editor y la terminal del host, en lab, sobre development, no desde la sesión del agente revisor. Antes de editar, necesitás el hook nativo configurado y un PR de tu propio repo abierto hacia main, no draft y sin trabajo ajeno pendiente.

Comprobá el contexto y actualizá la rama antes de tocar el archivo:

git status
git switch development
git pull --ff-only

En el editor, abrí app/capacidad.ts. La función provista es:

export function lugaresLibres(capacidad: number, ocupados: number): number {
  return capacidad - ocupados;
}

Cambiá deliberadamente sólo la resta por suma y guardá:

return capacidad + ocupados;

Con capacidad 20 y 5 ocupados deberían quedar 15, pero la suma devuelve 25. TypeScript acepta sumar dos números y devolver otro número, por eso este cambio puede pasar typecheck aunque el cálculo esté mal. Las pruebas de comportamiento, en cambio, comparan el resultado con lo esperado. Este fallo es sólo para practicar y no debe llegar a Production.

Revisá el diff para confirmar que cambiaste sólo esa operación y publicá únicamente ese archivo:

git diff -- app/capacidad.ts
git add app/capacidad.ts
git commit -m "Practicar revision del calculo de lugares"
git push
gh pr list --base main --head development

Conservá el PR abierto. Si ya existe, el push lo actualiza. Sólo si no hay un PR abierto de esa rama, crealo:

gh pr create --base main --head development \
  --title "Revisar calculo de lugares libres" \
  --body "Practica de review sobre un error logico; no integrar sin corregir."

Ahora reemplazá NUMERO por el número real del PR y anotá ambos SHA. Los vamos a usar para identificar la versión que recibe el revisor:

gh pr view NUMERO --json number,state,isDraft,baseRefName,headRefName,baseRefOid,headRefOid
gh pr view NUMERO --web

En la respuesta y en la web, comprobá que el PR esté Open, no draft, con base main y head development del mismo repo, y que contenga el commit de la suma. Corroborá en la web el repo de ambas ramas. El push por sí solo no solicita review en esta ruta, aunque la etiqueta estuviera presente.

No sigas si falló pull o push, hay cambios no previstos, el PR viene de un fork, apunta a otra base o sigue draft. No integres el fallo. Vamos a pedir que la review analice el diff como datos, sin ejecutar código del PR.