Skip to content

FAR-03 · seguimiento: una receta sin fin de vigencia no vence en el simulador - #1695

Merged
Leonardolv21 merged 2 commits into
devfrom
ender/far-03-simulador-vigencia-dev
Oct 30, 2026
Merged

Leonardolv21 merged 2 commits into
devfrom
ender/far-03-simulador-vigencia-dev

Conversation

@Jsaldias39

Copy link
Copy Markdown
Collaborator

Qué cambia

Es el seguimiento de mdavila-2001/mantra-core-health#1686, que se mergeó en 1d05fa29. Con eso quedó abierto el último hallazgo de Codex en ese PR: el hilo del simulador sobre validTo.

  1. 7fdfd34d · fix(mock): una receta sin fin de vigencia no vence en el simulador.
    • En el simulador, al crear una receta sin validTo se le ponía ahora(), así que, ya emitida, «Dónde comprar» la daba por vencida.
    • La API deja la columna vacía (medications.service.ts:402) y la serializa como null. Ahora el simulador hace lo mismo, y SimulatedPrescription.validTo admite null.
    • orderable-prescription.ts no cambia.
  2. f266aa9d · style(mock): el simulador de recetas no crece por encima de su techo.
    • check-max-lines congela el tamaño de concepts.ts, clinic.ts y clinical.handlers.ts. Los comentarios agregados en FAR-03 · «Dónde comprar» pide solo la receta elegida y vigente #1686 y en el commit anterior los hacían crecer, y clinical.handlers.ts quedaba por encima de su techo.
    • Los comentarios pasan a la misma línea del código que explican, y los tres archivos vuelven a su tamaño de dev. No cambia el comportamiento.

Regresión

  • clinical.handlers.spec.ts: una receta creada sin validTo, firmada y emitida, queda MR_ISSUED con validTo === null. Una que trae validTo lo conserva.
  • orderable-prescription.spec.ts: el recorrido completo por el router del simulador (crear sin validTo, firmar, emitir), leído como lo lee el cliente, da isOrderableRow === true.

Verificación

Gate Árbol Resultado
yarn lint, tsc -p tsconfig.app.json --noEmit, yarn build f266aa9d sobre dev 4f7529b4, y también junto a los seguimientos de FAR-01 y FAR-02 exit 0
tsc -p tsconfig.spec.json --noEmit f266aa9d 0 errores
ng test --include src/app/core/mock --include …/where-to-buy f266aa9d 113 archivos, 1421/1421 en verde
Suite completa, en 5 tandas porque entera no entra en memoria f266aa9d más los seguimientos de FAR-01 y FAR-02 1047 de 1049 archivos en verde. Los 2 en rojo, follow-up-block y pdf-theme, fallan igual sobre dev limpio
check-max-lines ídem 96 archivos, la misma lista que en dev; ninguno de este PR
check-architecture, check-form-pages, check-colores-sueltos, check-admin-portal-contract ídem exit 1, la misma salida que en dev

git merge-tree contra dev 6c38386e, que ya trae la reportería y el cierre de farmacia (#1691) → limpio.

BLOCKED: la evidencia de navegador en start:demo no se corrió en esta sesión.

Push: con SALTAR_BUILD=1, porque el hook construye el árbol de trabajo local, que tiene otros carriles. El build de este árbol exacto está arriba.

Orden: primero este PR. Encima van apilados los seguimientos de FAR-01 y FAR-02, que usan validTo: null.


Generated by Claude Code

El alta simulada de una receta le ponía `validTo = ahora()` cuando no venía, y
ya emitida salía vencida en «Dónde comprar». La API deja la columna vacía
(`medications.service.ts`) y la serializa como `null`: ahora el simulador hace
lo mismo, y `SimulatedPrescription.validTo` admite `null`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015MMaS8qHQHpXEXggSAcCJT
`check-max-lines` congela el tamaño de `concepts.ts`, `clinic.ts` y
`clinical.handlers.ts`, y los dos commits anteriores los hacían crecer con
comentarios. Los comentarios quedan en la misma línea del código que explican
y los tres archivos vuelven a su tamaño de `dev`. No cambia el comportamiento.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015MMaS8qHQHpXEXggSAcCJT

Copy link
Copy Markdown
Collaborator Author

@codex review

Cabeza a revisar: f266aa9d.


Generated by Claude Code

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-30T13:16:32.547162Z f266aa9 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit: f266aa9dc9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Leonardolv21
Leonardolv21 merged commit 254db29 into dev Oct 30, 2026
Jsaldias39 pushed a commit that referenced this pull request Oct 30, 2026
Sin cambios de contenido: esta rama ya traía esos commits. Así el diff del PR
queda solo en lo de FAR-01.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015MMaS8qHQHpXEXggSAcCJT
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants