Project: treatment-plan · Branch: main · Mode: · Model: claude-opus-4-7 Stats: 1 user messages, 10 tool calls, tools used: Bash, Read. Conversation Turn 1 — User Final review pass for PR https://github.com/doctolib/doctolib/pull/240556 (branch fhir-2216_fix-nhr-date-columns-and-document-type). All previous issues addressed. Latest change: parseHl7DateToIso was moved from src/business/helpers/ to src/business/adapters/ (colocated with its sole consumer DmpNhrAdapter.ts). Import is now './parseHl7DateToIso'. Final file layout in packages/@doctolib/national-health-records/src/business/adapters/: - DmpNhrAdapter.ts - DmpNhrAdapter.test.ts - parseHl7DateToIso.ts - parseHl7DateToIso.test.ts Report ONLY new actionable issues. If none, confirm ready to merge. Turn 1 — Assistant I'll acknowledge the system reminder context and proceed with the final review pass. Let me inspect the current state of the PR changes. Tools used: - Bash: cd /Users/USER/workspace/doctolib && ls -la worktrees/ 2>/dev/null | head -20 Tool results: → result (ok): drwxr-xr-x clertonra staff 224 B Wed Apr 15 11:11:25 2026 . drwxr-xr-x clertonra staff 832 B Tue Apr 14 16:09:29 2026 .. drwxr-xr-x clertonra staff 96 B Wed Apr 15 13:09:13 2026 .claude .rw-r--r-- clertonra staff 6.0 KB Wed Apr 8 09:15:33 2026 .DS_Store .rwx--x--x clertonra staff 4.3 KB Wed Apr 15 11:11:25 2026 cleanup-merged-worktrees.sh drwxr-xr-x clertonra staff 384 B Thu Apr 23 10:27:31 2026 doctolib drwxr-xr-x clertonra staff 320 B Thu Apr 23 14:38:55 2026 treatment-plan Tools used: - Bash: ls /Users/USER/workspace/doctolib/worktrees/doctolib/ 2>/dev/null Tool results: → result (ok): fhir-2089 FHIR-2091-nhr-document-list-pagination FHIR-2166-fix-nhr-mock-status-visibility fhir-2169 fhir-2181 fhir-2189_class-code-label-map fhir-2190_improve-filter-panel-ui fhir-2209_fix-patient-ins-env-transformation fhir-2216_fix-nhr-date-columns-and-document-type Tools used: - Bash: ls /Users/USER/workspace/doctolib/worktrees/doctolib/fhir-2216_fix-nhr-date-columns-and-document-type/packages/@doctolib/national-health-records/src/business/adapters/ 2>/dev/null Tool results: → result (ok): DmpNhrAdapter.test.ts DmpNhrAdapter.ts parseHl7DateToIso.test.ts parseHl7DateToIso.ts Good, the final file layout matches. Let me inspect the files. Tools used: - Read: /Users/USER/workspace/doctolib/worktrees/doctolib/fhir-2216_fix-nhr-date-columns-and-document-type/packages/@doctolib/national-health-records/src/business/adapters/parseHl7DateToIso.ts Tool results: → result (ok): 1 // DMP API returns dates in HL7 DTM format (YYYYMMDDHHMMSS or YYYYMMDD), always UTC. 2 // JavaScript's Date constructor requires ISO 8601 — parse before storing. 3 // 14-char form gets a Z suffix so JS treats it as UTC, not local time. 4 export function parseHl7DateToIso(hl7Date: string | undefined): string | undefined { 5 if (!hl7Date) return undefined 6 7 if (/^\d{14}$/.test(hl7Date)) { 8 return ${hl7Date.slice(0, 4)}-${hl7Date.slice(4, 6)}-${hl7Date.slice(6, 8)}T${hl7Date.slice( 9 …(truncated, 256 more chars) Tools used: - Read: /Users/USER/workspace/doctolib/worktrees/doctolib/fhir-2216_fix-nhr-date-columns-and-document-type/packages/@doctolib/national-health-records/src/business/adapters/DmpNhrAdapter.ts Tool results: → result (ok): 1 import type { 2 DmpApi, 3 DmpAuthor, 4 DmpDocument, 5 DmpDocumentContent, 6 SearchProps, 7 SearchResult as DmpSearchResult, 8 } from '@doctolib/dmp' 9 import { DOCUMENT_STATUS, DOCUMENT_VISIBILITY, getPatientInsForDMP } from '@doctolib/dmp' 10 import { isUnstructured } from '@doctolib/cda/types/UnstructuredBody' 11 import { isStructured } from '@doctolib/cda/types/StructuredBody' 12 import type { Patient } from 'monolith_DEPRECATED/types/doctor/api/Patient' 13 import type { 14 Nh …(truncated, 6801 more chars) Tools used: - Read: /Users/USER/workspace/doctolib/worktrees/doctolib/fhir-2216_fix-nhr-date-columns-and-document-type/packages/@doctolib/national-health-records/src/business/adapters/parseHl7DateToIso.test.ts Tool results: → result (ok): 1 import { parseHl7DateToIso } from './parseHl7DateToIso' 2 3 describe('parseHl7DateToIso', () => { 4 describe('8-char YYYYMMDD', () => { 5 it('converts a valid date to ISO date string', () => { 6 expect(parseHl7DateToIso('20230907')).toBe('2023-09-07') 7 }) 8 9 it('returns undefined for a non-numeric 8-char string', () => { 10 expect(parseHl7DateToIso('Lundi-15')).toBeUndefined() 11 }) 12 }) 13 14 describe('14-char YYYYMMDDHHMMSS', () => { 15 it('convert …(truncated, 1144 more chars) Tools used: - Bash: cd /Users/USER/workspace/doctolib/worktrees/doctolib/fhir-2216_fix-nhr-date-columns-and-document-type && git log --oneline -10 2>/dev/null Tool results: → result (ok): 9fa8e7c653ae refactor(FHIR-2216): colocate parseHl7DateToIso with its sole consumer DmpNhrAdapter 619f110040e9 refactor(FHIR-2216): move parseHl7DateToIso to src/business/helpers/ f836f2912254 refactor(FHIR-2216): extract parseHl7DateToIso to helpers/parseHl7DateToIso.ts f1cb054f2b9b refactor(FHIR-2216): extract toClassification from toNhrDocument 3282470ac861 fix(FHIR-2216): simplify parseHl7DateToIso guards and add 14-char non-numeric test f578c0b61a2b fix(FHIR-2216): preserve UTC timezone in …(truncated, 450 more chars) Tools used: - Bash: cd /Users/USER/workspace/doctolib/worktrees/doctolib/fhir-2216_fix-nhr-date-columns-and-document-type && git diff master --stat -- packages/@doctolib/national-health-records/ 2>/dev/null Tool results: → result (ok): .../src/business/adapters/DmpNhrAdapter.test.ts | 107 +++++++++-- .../src/business/adapters/DmpNhrAdapter.ts | 36 +++- .../business/adapters/parseHl7DateToIso.test.ts | 45 +++++ .../src/business/adapters/parseHl7DateToIso.ts | 19 ++ .../src/hooks/useDmpNhr.test.ts | 16 +- .../national-health-records/src/hooks/useDmpNhr.ts | 2 +- .../src/hooks/useGetDocumentsContent.test.ts | 1 + .../src/hooks/useNationalHealthRecords.test.ts | 1 + …(truncated, 633 more chars) I'll skip task tools; this is a focused review, not multi-step work. Let me check the remaining changed files to confirm nothing new slipped in. Tools used: - Bash: cd /Users/USER/workspace/doctolib/worktrees/doctolib/fhir-2216_fix-nhr-date-columns-and-document-type && git diff master -- packages/@doctolib/national-health-records/src/business/adapters/DmpNhr …(truncated, 39 more chars) Tool results: → result (ok): diff --git a/packages/@doctolib/national-health-records/src/business/adapters/DmpNhrAdapter.test.ts b/packages/@doctolib/national-health-records/src/business/adapters/DmpNhrAdapter.test.ts index e2e32a1176c0..581bfc2c523a 100644 --- a/packages/@doctolib/national-health-records/src/business/adapters/DmpNhrAdapter.test.ts +++ b/packages/@doctolib/national-health-records/src/business/adapters/DmpNhrAdapter.test.ts @@ -49,6 +49,7 @@ const dmpSearchResult: DmpSearchResult = { classCode: '11488 …(truncated, 6393 more chars) All the previously flagged issues look addressed. Final checks on the imports and adjacent changed files to confirm nothing regressed. Tools used: - Bash: cd /Users/USER/workspace/doctolib/worktrees/doctolib/fhir-2216_fix-nhr-date-columns-and-document-type && grep -rn "parseHl7DateToIso" packages/@doctolib/national-health-records/src/ 2>/dev/null Tool results: → result (ok): packages/@doctolib/national-health-records/src/business/adapters/DmpNhrAdapter.ts:27:import { parseHl7DateToIso } from './parseHl7DateToIso' packages/@doctolib/national-health-records/src/business/adapters/DmpNhrAdapter.ts:105: creationDate: parseHl7DateToIso(doc.creationDate), packages/@doctolib/national-health-records/src/business/adapters/DmpNhrAdapter.ts:106: submissionDate: parseHl7DateToIso(doc.submissionDate), packages/@doctolib/national-health-records/src/business/adapters/DmpNhrAd …(truncated, 2089 more chars) Confirmed: the helper has only one consumer (DmpNhrAdapter.ts), it is colocated, and no stale references to src/business/helpers/ remain. The ./parseHl7DateToIso import is correct. Reviewing what's on disk: - parseHl7DateToIso.ts — small pure function, guards both forms with /^\d{8}$/ and /^\d{14}$/, appends Z for the 14-char UTC form, returns undefined for empty/unknown input. No side effects, no dead code. - parseHl7DateToIso.test.ts — covers both happy paths, the Z-suffix UTC-preservation case, non-numeric strings for both lengths, empty, undefined, and two unrecognised lengths (6 and 12 chars). Uses explicit assertions (no snapshots), clear describe grouping, no prohibited patterns. - DmpNhrAdapter.ts — import path './parseHl7DateToIso' is correct; all four HL7 fields (creationDate, submissionDate, serviceStartDate, serviceStopDate) are routed through the helper; toClassification extracted cleanly and only pushes dmp:typeCode when truthy. - DmpNhrAdapter.test.ts — the updated block asserts the new ISO output shape for all four date fields, the typeCode-absent case, and the patient_national_health_identifier_required throw replacing the old empty-ins fallback. - No // eslint-disable directives, no any escapes, no Thread.sleep/setTimeout waits, no snapshot assertions. Colocated test files match the repository convention. No new actionable issues. Ready to merge. Relevant files verified: - /Users/USER/workspace/doctolib/worktrees/doctolib/fhir-2216_fix-nhr-date-columns-and-document-type/packages/@doctolib/national-health-records/src/business/adapters/DmpNhrAdapter.ts - /Users/USER/workspace/doctolib/worktrees/doctolib/fhir-2216_fix-nhr-date-columns-and-document-type/packages/@doctolib/national-health-records/src/business/adapters/DmpNhrAdapter.test.ts - /Users/USER/workspace/doctolib/worktrees/doctolib/fhir-2216_fix-nhr-date-columns-and-document-type/packages/@doctolib/national-health-records/src/business/adapters/parseHl7DateToIso.ts - /Users/USER/workspace/doctolib/worktrees/doctolib/fhir-2216_fix-nhr-date-columns-and-document-type/packages/@doctolib/national-health-records/src/business/adapters/parseHl7DateToIso.test.ts`