Repository navigation
FAR-03 · seguimiento: una receta sin fin de vigencia no vence en el simulador - #1695
Merged
Merged
Conversation
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
Collaborator
Author
|
@codex review Cabeza a revisar: Generated by Claude Code |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. More of your lovely PRs please. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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 sobrevalidTo.7fdfd34d·fix(mock): una receta sin fin de vigencia no vence en el simulador.validTose le poníaahora(), así que, ya emitida, «Dónde comprar» la daba por vencida.medications.service.ts:402) y la serializa comonull. Ahora el simulador hace lo mismo, ySimulatedPrescription.validToadmitenull.orderable-prescription.tsno cambia.f266aa9d·style(mock): el simulador de recetas no crece por encima de su techo.check-max-linescongela el tamaño deconcepts.ts,clinic.tsyclinical.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, yclinical.handlers.tsquedaba por encima de su techo.dev. No cambia el comportamiento.Regresión
clinical.handlers.spec.ts: una receta creada sinvalidTo, firmada y emitida, quedaMR_ISSUEDconvalidTo === null. Una que traevalidTolo conserva.orderable-prescription.spec.ts: el recorrido completo por el router del simulador (crear sinvalidTo, firmar, emitir), leído como lo lee el cliente, daisOrderableRow === true.Verificación
yarn lint,tsc -p tsconfig.app.json --noEmit,yarn buildf266aa9dsobredev4f7529b4, y también junto a los seguimientos de FAR-01 y FAR-02tsc -p tsconfig.spec.json --noEmitf266aa9dng test --include src/app/core/mock --include …/where-to-buyf266aa9df266aa9dmás los seguimientos de FAR-01 y FAR-02follow-up-blockypdf-theme, fallan igual sobredevlimpiocheck-max-linesdev; ninguno de este PRcheck-architecture,check-form-pages,check-colores-sueltos,check-admin-portal-contractdevgit merge-treecontradev6c38386e, que ya trae la reportería y el cierre de farmacia (#1691) → limpio.BLOCKED: la evidencia de navegador en
start:demono 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