docs: drop the spec's reference to the auto-fill exemption

origin/master removed the dispatch panel's remembered-prosthesis auto-fill
(f52ad6b), so the exemption this branch carried for it went away in the
rebase. The spec named the mechanism and a verification step for a feature
that no longer exists.

The rule it existed to enforce still stands and is worth keeping written
down — confirm fills exactly what the sheet previewed — so it stays, marked
as a constraint on whatever gets added next rather than a description of
something in the code.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

.cursor/rules/treatment-workspace.mdc

@@ -10,16 +10,17 @@ alwaysApply: false
 - **Current draft** preview: omitted in live editing (form is the source). History browse still uses preview + **Load into workspace**.
 - **Entry:** Type dropdown + `TreatmentDetailAttachmentsStrip` on one row, bordered chart (Cases chrome), then full-width auto-growing Notes. No stepper unless prosthesis — then `WizardStepper` Treatment → Lab with Back/Next. Chip switches reset to the treatment form unless `pendingEntryStepRef` requests Lab (shipments rail open). Lab-dependent chips use colored sent/unsent text; sent date is on Lab dispatch. Lab dispatch keeps comments.
 - **Tooth hits:** unrotated full cell, `pointerdown` only (not `click` too — double-toggles). Glyph `pointer-events-none`; nest scale/hover inside the rotate wrapper. Groups: `toothSelectionGroups.ts` (never 1-tooth connected); prune lab `toothProsthesis` on change.
-- **Day strip:** `ScheduleDayPicker` `compact` centered in the header. Timed cards use purpose banners. Unscheduled cards use the same banner once typed (sync live draft onto the open card); untyped keep chip theming. Trash inherits banner ink. Strip-delete only when `areUnscheduledDetailsStripDeletable` (blank lines or `[]`); persist `[]` then `DELETE /treatments/:id`.
+- **Day strip:** `ScheduleDayPicker` `compact` centered in the header. Timed cards use purpose banners. Unscheduled cards use the same banner from the **first detail’s type only** (live draft for the open card); empty first line keeps chip theming. Trash inherits banner ink. Strip-delete only when `areUnscheduledDetailsStripDeletable` (blank lines or `[]`); persist `[]` then `DELETE /treatments/:id`.
 - **Lab dispatch UI:** due date end-aligned beside title (`sm:flex-row` + `justify-between`; stacks on mobile). Prosthesis type: stacked below `md`, 50/50 same-row from `md`. Content clinical field = **Notes** (not case comments).
 - **Lab comments:** shared `LabCaseCommentsPanel` (newest-first; sent=`justify-start`, received=`justify-end`; `viewerSide`) across Treatment / Cases / Tasks / share. Use logical `text-start`/`text-end`, not left/right.
-- **Detail chrome:** chips (type + teeth) + Add at top; **Remove** = trash on chip (unsent, including last line; disabled when day-locked / no edit / uploading). Empty details persist as `[]`. New treatment seeds one blank detail; load of empty stays `[]`. No delete in type/notes fields.
+- **Detail chrome:** chips (type + teeth) + Add at top; **Remove** = trash on chip (unsent, including last line; disabled when day-locked / no edit / uploading). Last-line confirm: plan will be empty until Add. Empty `[]` shows `noDetails` (not the type-first overlay). New treatment seeds one blank detail. No delete in type/notes fields.
+- **Patient search:** `PatientSearchCombobox` in the **page header** (workspace-wide). Opens today’s strip visit if any, else latest history. No visit/history → inline editor empty state pointing to **New treatment** in the rail (never a dialog, never auto-create). **New treatment** stays at the top of the left rail; selected-patient card sits under it. Picker: Walk-in first, then a matching full-width card for the current named patient (name + mobile/email, else hint), then search. Never auto-create from the appointment card.
 - **Lab dispatch attention:** `LabDispatchAttentionPanel` — unsent lab-dependent details; quick jump to dispatch.
 - **History API:** patient-scoped; non-owners filtered by provider on treatment or appointment; org owners see all.
 - **History filters (client-side):** `PastTreatmentsPanel` — “Not shipped to lab” + single date; helpers in `treatmentHistoryFilters.ts`.
-- **Lab shipment block:** lab-dependent detail with no teeth saves but cannot ship — `LabShipmentBlockedNotice`, inline banner, toast on add shipment.
+- **Lab shipment block:** lab-dependent detail with no teeth saves but cannot ship — same inline amber banner on Treatment and Lab (`labBlockedBannerClass`); toast on add shipment.
 - **Edit gating:** `canEditTreatmentForDay` = permission + appointment + not past day + `live` mode. Sent detail locks that line; Add still OK same day. Upload rejects sent (`TREATMENT_DETAIL_SENT`).
-- **Lab search:** `LinkedOrganizationSearchCombobox`; invite lab via `/organizations?action=invite-lab` when permitted.
+- **Lab search:** `LinkedOrganizationSearchCombobox`; invite lab via `/organizations?action=invite-lab` when permitted. New dispatch lines start empty (lab + apply-all / per-tooth prosthesis type). Chips = last 3 **sent** labs (`labDispatchDefaults.ts`) — never auto-select. Clear the search box when switching details.
 - **Scroll:** `scrollWithinMainScrollContainer`; shared `Checkbox` only.
 
 Full map: `.cursor/skills/treatment-workspace/SKILL.md`

.cursor/skills/treatment-workspace/SKILL.md

@@ -22,7 +22,7 @@ Thin route: `app/[locale]/(dashboard)/treatment/page.tsx` (supports `?appointmen
 
 
 
-1. **Day strip** — `AppointmentsStrip.tsx` renders `DayStripItem[]` (`appointment` | `unscheduled`) via `DayStripCard`. Header uses **`ScheduleDayPicker` `compact`**: date is centered in a 3-col grid; no “Schedule date” label; **Today** sits on the navigator (`CalendarDaySelect` when the label row is hidden). Timed appointments keep treatment-type pastel banners. Unscheduled cards use the same banner once a treatment type is selected (live draft for the open card; `draftHydratingRef` must be set **before** strip/appointment pick so overlay does not paint the previous card’s type). Until typed they keep chip theming. Trash inherits banner ink on typed cards. Strip trash only when `areUnscheduledDetailsStripDeletable` (no type/teeth/notes/attachments, including `[]`). Workspace fetches `GET /appointments` **and** `GET /treatments/day`. **New treatment** is one shared `Button`: it opens `NewTreatmentPatientPicker` (Walk-in always first, then search). Creating happens only after an explicit patient choice — never from the selected appointment card. New treatment seeds one blank detail so the type field is ready; a persisted empty plan hydrates as `[]` until Add.
+1. **Day strip** — `AppointmentsStrip.tsx` renders `DayStripItem[]` (`appointment` | `unscheduled`) via `DayStripCard`. Header uses **`ScheduleDayPicker` `compact`**: date is centered in a 3-col grid; no “Schedule date” label; **Today** sits on the navigator (`CalendarDaySelect` when the label row is hidden). Timed appointments keep treatment-type pastel banners. Unscheduled cards use the same banner from the **first detail’s type only** (`unscheduledStripColorCode`; live draft for the open card; `draftHydratingRef` must be set **before** strip/appointment pick so overlay does not paint the previous card’s type). Empty first line → chip theming even if later lines are typed. Trash inherits banner ink on typed cards. Strip trash only when `areUnscheduledDetailsStripDeletable` (no type/teeth/notes/attachments, including `[]`). Workspace fetches `GET /appointments` **and** `GET /treatments/day`. Patient search (`PatientSearchCombobox`) sits in the **page header** (workspace-wide). **New treatment** is one shared `Button` at the **top of the left rail**, with the selected-patient card under it: it opens `NewTreatmentPatientPicker` (Walk-in always first, then a matching full-width card for the current named patient with name + mobile/email or hint, then search). Creating happens only after an explicit patient choice — never from the selected appointment card. New treatment seeds one blank detail so the type field is ready; a persisted empty plan hydrates as `[]` until Add (`noDetails` copy — not the type-first overlay). Last-line chip delete confirms the plan will be empty until Add.
 
 2. **Treatment preview** — `TreatmentPreviewCard.tsx` (history browse only; omitted for the live draft)
 
@@ -41,11 +41,11 @@ Right-column entry is **not** a three-step wizard. Type dropdown + `TreatmentDet
 | Stage | UI | When |
 |-------|-----|------|
 | **Treatment** | Type dropdown + `TreatmentDetailAttachmentsStrip`, `FdiToothChart` (Cases scale), full-width Notes | Always |
-| **Lab** | `LabCasesDispatchPanel` | Only when active detail type is lab-dependent. Entering Lab auto-ensures a shipment draft. Last-used lab and prosthesis type are remembered. Comments stay on the dispatch panel. |
+| **Lab** | `LabCasesDispatchPanel` | Only when active detail type is lab-dependent. Entering Lab auto-ensures a shipment draft. **No default lab or prosthesis type** on a new detail (including siblings in the same plan). Last **3 sent** labs appear as chips under search — pick is explicit. Comments stay on the dispatch panel. |
 
 - Prosthesis uses `WizardStepper` (Treatment → Lab) with Back/Next. Lab dispatch keeps comments.
 - Detail chips show **type + teeth**, not “Detail N”. Lab-dependent chips use colored sent/unsent text (same size as the label); sent date stays on Lab dispatch.
-- Detail type may differ from appointment purpose (purpose only defaults new details).
+- Detail type may differ from appointment purpose. Purpose seeds the first line of an empty **appointment** draft (first open, and **Add detail** when the plan is `[]`). Later **Add detail** starts with an empty type. Unscheduled / New treatment still seeds a blank first line.
 - Switching `activeDetailId` resets to the treatment form, unless `pendingEntryStepRef` is set to `lab` first (lab shipments rail / “Go to dispatch” / load-with-focus).
 - Live draft is **not** duplicated in the left rail preview; preview is for history browse only.
 
@@ -120,7 +120,7 @@ Helpers: `frontend/src/components/treatment/treatmentHistoryFilters.ts`.
 
 Saved lab-dependent detail with **no teeth** can autosave but **cannot** create a lab shipment.
 
-- Inline banner in `TreatmentDetailsEditor` + `LabShipmentBlockedNotice` above dispatch when active detail qualifies (`isLabDependentDetailMissingTeeth`).
+- Inline amber banner in `TreatmentDetailsEditor` on **both** Treatment and Lab steps when the active detail qualifies (`isLabDependentDetailMissingTeeth`). Do not use a separate notice card.
 - `handleAddLabCase` shows toast with `labShipmentBlockedBody`.
 - Dispatch panel only appears when a detail passes `isDetailReadyForLabDispatch` (persisted + lab-dependent + teeth).
 
@@ -155,7 +155,7 @@ On today: in-progress slot first, else nearest start time to `now`. Other days:
 
 
 
-`LinkedOrganizationSearchCombobox` in `LabCasesDispatchPanel` — search-only results (no dropdown). No match + org tab access → **Invite a lab** navigates to `/organizations?action=invite-lab`. No org access → show permission message; dispatch stops.
+`LinkedOrganizationSearchCombobox` in `LabCasesDispatchPanel` — search-only results (no dropdown). Chips under the search are the last **3 labs this clinic sent a case to** (`rememberRecentLab` after successful send). They are shortcuts, not defaults: a new detail’s lab and prosthesis type (apply-all and per-tooth) stay empty until the user chooses. Switching details clears the search box. No match + org tab access → **Invite a lab** navigates to `/organizations?action=invite-lab`. No org access → show permission message; dispatch stops.
 
 
 
@@ -201,7 +201,7 @@ Use shared `Checkbox` (not native `<input type="checkbox">`) to avoid focus-driv
 
 
 
-Walk-in uses one sentinel `Patient` per clinic (`isWalkIn`, hidden from Patients/search/booking). Display via i18n, never the stored name. Patient search: same workspace patient → no-op; else load latest history into the editor; **no history → do not auto-create** (history rail empties; dentist uses **New treatment** and picks a patient, including Walk-in).
+Walk-in uses one sentinel `Patient` per clinic (`isWalkIn`, hidden from Patients/search/booking). Display via i18n, never the stored name. Patient search: same workspace patient with a live visit → no-op; else open today’s strip visit if any; else load latest history into the editor; **no history and no strip visit → do not auto-create**. Detach the previous visit, keep the searched patient, and show an inline editor empty state (`noTreatmentFoundTitle` / `noTreatmentFoundBody`) that points to **New treatment** in the rail (Walk-in, current named patient card, or search).
 
 Draft writes for appointments require provider match (`ensureAppointmentProvider`). Standalone requires `treatment.providerUserId === actor`.
 

AGENTS.md

@@ -44,8 +44,8 @@ frontend/src/
 **Treatment tab:** Preview and editable form are **separate** until the user clicks **Load into workspace** on a history item. See `.cursor/skills/treatment-workspace/SKILL.md` before changing that flow.
 
 **Treatment edit / details (quick ref):**
-- Day/mode gate: editable only for live draft on today/future (`canEditTreatmentForDay`). Past day / historical load → read-only form. **New treatment** opens a patient picker (Walk-in always visible); it does not copy the selected appointment’s patient. New treatment seeds one blank detail; a persisted empty plan loads as `[]` until Add.
-- **Unscheduled strip cards** use the same treatment-type banner as appointments once a type is selected (live draft for the open card). Strip trash only when every line is blank (no type/teeth/notes/attachments, including `[]`); typed cards need chip-delete first. Backend `DELETE /treatments/:id` is empty-only (`TREATMENT_HAS_DETAILS`).
+- Day/mode gate: editable only for live draft on today/future (`canEditTreatmentForDay`). Past day / historical load → read-only form. Patient search sits in the **page header** (workspace-wide). If the patient has a visit on the day strip, that visit opens; else the latest history plan loads. If neither exists, the editor shows an inline empty state (no dialog) pointing to **New treatment** in the rail — do not auto-create. **New treatment** is at the top of the left rail (selected-patient card below it) and opens a picker: Walk-in first, then a matching card for the current named patient (name + mobile/email), then search. It does not auto-copy the appointment card’s patient. New treatment seeds one blank detail; a persisted empty plan loads as `[]` until Add.
+- **Unscheduled strip cards** use the same treatment-type banner as appointments from the **first detail’s type only** (`unscheduledStripColorCode`; live draft for the open card). Empty first line → chip theming even if later lines are typed. Strip trash only when every line is blank (no type/teeth/notes/attachments, including `[]`); typed cards need chip-delete first. Backend `DELETE /treatments/:id` is empty-only (`TREATMENT_HAS_DETAILS`).
 - Sent-to-lab detail locks that line; **Add detail** still OK same day; **Remove detail** = trash on chip (unsent, including last line; day/edit gates apply). Empty details persist as `[]`. Attachment upload blocked when sent (`TREATMENT_DETAIL_SENT`).
 - **Entry:** Type dropdown + compact attachments strip (`TreatmentDetailAttachmentsStrip`) on one row, then bordered FDI chart (Cases chrome/scale), then full-width auto-growing **Notes**. No wizard for non-lab types. Prosthesis: `WizardStepper` Treatment → Lab with Back/Next. Detail chips show type + teeth; lab-dependent chips use colored sent/unsent text (sent date on Lab tab). Switching chips resets to the treatment form unless `pendingEntryStepRef` requests Lab (shipments rail). Lab dispatch keeps comments.
 - **Tooth selection:** Hit target is the unrotated cell (`pointerdown` only — do not also bind `click`). Glyph is `pointer-events-none`; nest hover/selected scale inside the rotate wrapper. Neighbor empty/filled circles between selected adjacent teeth connect/disconnect bridges; Shift+range selects only (empty circles; overlap absorbs as singles); midline 11–21 / 41–31 allowed. Plain click selects/deselects (deselect splits bridges). Never a 1-tooth connected. Helpers: `toothSelectionGroups.ts`. Connected label: `ConnectedSelectionBadge`. After send, Cases/Tasks merge teeth by prosthesis type.
@@ -53,8 +53,9 @@ frontend/src/
 - **Schedule date:** Treatment strip and Appointments page headers use `ScheduleDayPicker` `compact` (date centered in a 3-col header; no “Schedule date” label; Today on the navigator).
 
 **Treatment lab rules (quick ref):**
-- Lab-dependent details (e.g. prosthesis) **without teeth** can save but **cannot ship** — show `LabShipmentBlockedNotice` + inline banner; toast on dispatch add.
-- Detail treatment type need **not** match appointment purpose — purpose only pre-fills new details.
+- Lab-dependent details (e.g. prosthesis) **without teeth** can save but **cannot ship** — same inline amber banner (`labShipmentBlockedBody`) on Treatment **and** Lab steps; toast on dispatch add.
+- New prosthesis dispatch lines start **empty** (no default lab, no apply-all / per-tooth prosthesis type), even for siblings in the same plan. Recent-lab chips are the last **3 sent** destinations — pick is explicit, never auto-selected.
+- Detail treatment type need **not** match appointment purpose — purpose only seeds the **first** line of an empty **appointment** draft (first open, and **Add detail** when the plan is `[]`). Further **Add detail** starts with an empty type. Unscheduled / New treatment still seeds a blank first line.
 - **History filters** are client-side only (`treatmentHistoryFilters.ts`): “Not shipped to lab” + single date on already-fetched patient history; includes live current draft when filtering.
 - **Lab shipments rail**: unified list with scope toggle **This patient** vs **All updates** (unread across org for **this clinician's cases only**, includes patient name). Opening a case from the rail jumps to the **Lab** send sheet.
 - **Unread semantics**: Treatment tab badge = count of unread cases **for the user's own treatment plans** (per-case read cursor) and clears when a case is opened/marked read (not on tab visit).

backend/src/common/body-parsers.spec.ts

@@ -93,6 +93,9 @@ describe('createJsonBodyParser', () => {
       '/api/voice/extract/extra',
       '/api/voice',
       '/voice/extract',
+      // Express ignores one trailing slash, not two — this one never routes, so it must
+      // not get the large parser either.
+      '/api/voice/extract//',
     ]) {
       const res = await request(buildApp()).post(path).send(bodyOfKb(300));
       expect(res.status).toBe(413);

backend/src/common/body-parsers.ts

@@ -17,21 +17,19 @@ export const VOICE_BODY_LIMIT = '10mb';
  * of audio, so that one route needs a larger limit while every other endpoint keeps the
  * default — a large body should not become acceptable everywhere.
  *
- * Deliberately a single middleware that *chooses* a parser, rather than a path-mounted
- * parser stacked in front of a default one. That arrangement relied on Express's
- * mount-path stripping plus body-parser skipping an already-parsed request, and it
- * silently stopped applying when the surrounding middleware order shifted — at which point
- * the endpoint rejected every real recording with a 500. One explicit branch has no such
- * coupling, and is covered by body-parsers.spec.ts.
+ * Deliberately one middleware that *chooses* a parser, not a path-mounted parser stacked in
+ * front of a default one: that arrangement depended on Express's mount-path stripping and on
+ * body-parser skipping an already-parsed request, and silently stopped applying whenever the
+ * middleware order shifted. One explicit branch has no such coupling.
  */
 /**
- * Express routes case-insensitively and ignores a trailing slash unless configured
- * otherwise, so `/API/Voice/Extract/` reaches the same controller. Matching only the
- * canonical spelling would hand those requests the 100 kb parser and 413 every real
- * recording — a failure that looks like a broken microphone, not a routing detail.
+ * Express routes case-insensitively and ignores exactly one trailing slash, so
+ * `/API/Voice/Extract/` reaches the same controller and must get the same limit — otherwise
+ * it 413s every real recording, which reads as a broken microphone rather than a route.
+ * Two slashes never route, so they must not buy a 10 MB buffer either.
  */
 function isVoiceExtractPath(path: string): boolean {
-  return path.toLowerCase().replace(/\/+$/, '') === VOICE_EXTRACT_PATH;
+  return path.toLowerCase().replace(/\/$/, '') === VOICE_EXTRACT_PATH;
 }
 
 export function createJsonBodyParser(): RequestHandler {

backend/src/common/digits.ts

@@ -1,19 +1,11 @@
-/**
- * Persian (Extended Arabic-Indic, U+06F0–U+06F9) zero, and Arabic-Indic (U+0660–U+0669)
- * zero. ASR output can carry either block, sometimes mixed with ASCII in one transcript.
- */
 const PERSIAN_ZERO = 0x06f0;
 const ARABIC_INDIC_ZERO = 0x0660;
 
 /**
  * Normalise Persian and Arabic-Indic digits to ASCII. Non-digits pass through.
  *
- * Deliberately wider than the frontend original, which only handles the Persian block:
- * this parses model/ASR output rather than keystrokes, so both blocks must be accepted
- * or a spoken date or tooth number silently degrades to "unresolved".
- *
- * Lives on its own rather than inside jalali.ts because tooth codes need it too, and a
- * tooth module reaching into the calendar module would read as an accident.
+ * Both blocks, not just the Persian one the frontend handles: ASR output can carry either,
+ * sometimes mixed with ASCII in a single transcript.
  */
 export function toLatinDigits(value: string): string {
   return value.replace(/[۰-۹٠-٩]/g, (ch) => {

backend/src/common/fdi.ts

@@ -14,10 +14,9 @@ export type Arch = 'upper' | 'lower';
 export type PatientSide = 'patient_right' | 'patient_left';
 
 /**
- * Upper arch in chart order: patient's RIGHT (18) → midline → patient's LEFT (28).
- * That is the drawn left-to-right layout, which is the mirror of the patient's own sides.
- * Do not read a tooth position off this array by index — use `toFdi()`, which owns the
- * side convention.
+ * Upper arch in chart order: patient's RIGHT (18) → midline → patient's LEFT (28) — the drawn
+ * layout, which mirrors the patient's own sides. Never read a position off this array by
+ * index; use `toFdi()`, which owns the side convention.
  */
 export const FDI_UPPER_ARCH_ORDER = [
   '18',
@@ -68,12 +67,9 @@ export function isFdiTooth(value: unknown): value is string {
 }
 
 /**
- * Clean up a tooth code the extraction model echoed back, before it is matched.
- *
- * The model is transcribing Persian speech, so it can hand back "۲۶" in Persian digits or
- * "2 6" from a digit-by-digit dictation. Neither matches an FDI code literally, and a
- * near-miss here does not fail loudly — the tooth quietly turns into "not understood".
- * Returns '' for anything that is not a string.
+ * Clean up a tooth code the model echoed back. It is reading Persian speech, so it can hand
+ * back "۲۶" or "2 6" from digit-by-digit dictation; neither matches literally, and the
+ * near-miss does not fail loudly — the tooth just turns into "not understood".
  */
 export function normalizeFdiCode(value: unknown): string {
   if (typeof value !== 'string') return '';
@@ -114,9 +110,8 @@ export function teethBetweenInclusive(a: string, b: string): string[] | null {
 /**
  * Arch + patient side + position (1 = central incisor … 8 = third molar) → FDI code.
  *
- * This function is the single place the patient-right convention lives. Getting it
- * backwards mirrors every quadrant and produces a valid-looking code for the wrong tooth,
- * which no schema check can catch — hence the exhaustive test coverage.
+ * The single place the patient-right convention lives. Getting it backwards mirrors every
+ * quadrant into a valid-looking code for the wrong tooth, which no schema check can catch.
  */
 export function toFdi(
   arch: Arch,
@@ -135,8 +130,8 @@ export function toFdi(
 }
 
 /**
- * Sort teeth along the arch, not lexically — a bridge reads 16-15-14, and 11 sits beside
- * 21 across the midline. Teeth from another arch (or unknown) sort to the end, stably.
+ * Along the arch, not lexically — a bridge reads 16-15-14, and 11 sits beside 21 across the
+ * midline. Teeth from another arch sort to the end, stably.
  */
 export function sortInArchOrder(teeth: readonly string[]): string[] {
   if (teeth.length === 0) return [];

backend/src/common/jalali.ts

@@ -1,10 +1,8 @@
 /**
- * Jalali (Persian) calendar arithmetic.
- *
- * Ported from `frontend/src/lib/i18n/persianCalendar.ts` (itself from jalaali-js, MIT).
- * The backend needs this because voice extraction resolves spoken Jalali dates into ISO
- * dates server-side, where the resolvers are unit-tested — the frontend has no test
- * runner. Keep the two copies in step; the underlying calendar does not change.
+ * Jalali (Persian) calendar arithmetic, ported from
+ * `frontend/src/lib/i18n/persianCalendar.ts` (itself jalaali-js, MIT). The backend needs it
+ * because voice resolves spoken Jalali dates server-side, where the resolvers are tested.
+ * Keep the two copies in step; the underlying calendar does not change.
  */
 
 const BREAKS = [
@@ -144,11 +142,8 @@ export function isJalaliLeapYear(jy: number): boolean {
 }
 
 /**
- * Days in a Jalali month, or 0 when the year or month is not real.
- *
- * Zero rather than a throw: every export here is reachable from model-supplied values, so
- * the whole module degrades instead of raising. Zero also makes `isValidJalaliDate`'s
- * `jd <= jalaliDaysInMonth(...)` naturally false.
+ * Days in a Jalali month, or 0 when the year or month is not real. Zero rather than a throw:
+ * every export here is reachable from model-supplied values, so the module degrades.
  */
 export function jalaliDaysInMonth(jy: number, jm: number): number {
   if (!isSupportedJalaliYear(jy)) return 0;
@@ -170,10 +165,8 @@ export function isValidJalaliDate(jy: number, jm: number, jd: number): boolean {
 }
 
 /**
- * Jalali triple → `YYYY-MM-DD`, or null when the date is not real.
- *
- * Returns null rather than throwing: callers resolve model-supplied values, which may be
- * nonsense, and an invalid date must degrade to "unresolved" rather than a 500.
+ * Jalali triple → `YYYY-MM-DD`, or null when the date is not real. Null rather than a throw,
+ * for the same reason: callers resolve model-supplied values, which may be nonsense.
  */
 export function jalaliToIsoDate(
   jy: number,

backend/src/common/zoned-civil-time.ts

@@ -55,15 +55,12 @@ export function civilDateJsWeekday(isoDate: string): number {
 }
 
 /**
- * Today's civil date (`YYYY-MM-DD`) in an IANA zone.
- *
- * Lets the server derive "today" from a client-supplied time zone instead of trusting a
- * client-supplied date, which matters for relative deadlines like "by Thursday".
+ * Today's civil date (`YYYY-MM-DD`) in an IANA zone, so the server derives "today" from a
+ * client-supplied *zone* rather than trusting a client-supplied date.
  */
 export function civilDateInZone(date: Date, timeZone: string): string {
-  // Intl throws RangeError on an unknown zone, before any fallback below could help, and
-  // this receives a client-supplied string. Callers validate first; this is the backstop
-  // so a bad zone degrades to a date that is at most a day out rather than a 500.
+  // Intl throws RangeError on an unknown zone and this takes a client-supplied string;
+  // callers validate first, this is the backstop.
   const zone = isValidIanaTimeZone(timeZone) ? timeZone : 'UTC';
   const parts = new Intl.DateTimeFormat('en-CA', {
     timeZone: zone,

backend/src/configs/configurations.ts

@@ -186,10 +186,9 @@ function parseProviderId(
 }
 
 /**
- * Every enabled locale gets its own ASR and LLM provider+model, each overridable
- * independently. They all point at the same OpenRouter models today; the per-locale
- * indirection is kept because Persian ASR is the weakest link and swapping only `fa` must
- * not be a code change.
+ * Every enabled locale gets its own ASR and LLM provider+model, each independently
+ * overridable. They all point at the same OpenRouter models today; the per-locale
+ * indirection stays so a locale can diverge by configuration rather than by code.
  */
 function buildVoiceConfig(
   getEnvVarWithDefault: (key: string, defaultValue: string) => string,

backend/src/main.ts

@@ -25,9 +25,8 @@ console.log = (...args) => {
 };
 
 async function bootstrap() {
-  // bodyParser is disabled here so the JSON parsers can be registered in an explicit
-  // order below; Nest's built-in one is installed during create() and would otherwise
-  // reject a voice recording at its 100 kb default before any later middleware ran.
+  // bodyParser is disabled so the JSON parsers can be registered in an explicit order below;
+  // Nest's built-in one would otherwise reject a voice recording at 100 kb.
   const app = await NestFactory.create(AppModule, { bodyParser: false });
 
   // Voice needs a larger JSON limit than everything else; see body-parsers.ts.

backend/src/modules/treatments/treatments.service.ts

@@ -70,7 +70,7 @@ type SentLabCaseRow = Prisma.LabCaseGetPayload<{ include: typeof sentLabCaseIncl
 
 const treatmentInclude = {
   patient: {
-    select: { id: true, firstName: true, lastName: true, isWalkIn: true },
+    select: { id: true, firstName: true, lastName: true, isWalkIn: true, mobile: true, email: true },
   },
   details: {
     orderBy: [{ sortOrder: 'asc' as const }],
@@ -1366,6 +1366,8 @@ export class TreatmentsService {
       firstName: string;
       lastName: string;
       isWalkIn: boolean;
+      mobile?: string | null;
+      email?: string | null;
     };
     details: Array<{
       id: string;
@@ -1426,6 +1428,8 @@ export class TreatmentsService {
             firstName: treatment.patient.firstName,
             lastName: treatment.patient.lastName,
             isWalkIn: treatment.patient.isWalkIn,
+            mobile: treatment.patient.mobile ?? null,
+            email: treatment.patient.email ?? null,
           }
         : null,
       details: treatment.details.map((d) => this.mapDetail(d)),

backend/src/modules/voice/dto/voice.dto.ts

@@ -26,48 +26,32 @@ export type VoiceAudioFormat = (typeof VOICE_AUDIO_FORMATS)[number];
 export const VOICE_LOCALES = ['en', 'fa', 'nl'] as const;
 
 export class ExtractVoiceDto {
-  /**
-   * Base64 audio, no data: prefix. Capped well above a 2-minute opus clip (~400 KB) but
-   * far below OpenRouter's 25 MB ceiling, so an oversized upload is rejected before it
-   * costs a vendor call.
-   */
+  /** Base64 audio, no data: prefix. Well above a 2-minute opus clip (~400 KB). */
   @IsString()
   @IsBase64()
-  // Both constraints name their own code. Left to the default mapping, `maxLength` falls
-  // through to VALIDATION_FIELD_REQUIRED and `isIn` resolves to
-  // VALIDATION_LANGUAGE_INVALID — so an oversized recording told the clinician a field
-  // was missing, and an unsupported container told them their language was invalid.
+  // Both name their own code: the shared map sends `maxLength` to
+  // VALIDATION_FIELD_REQUIRED and `isIn` to VALIDATION_LANGUAGE_INVALID, neither of which
+  // is true here.
   @MaxLength(8_000_000, { message: ErrorCode.VOICE_CLIP_TOO_LONG })
   audio: string;
 
   @IsIn(VOICE_AUDIO_FORMATS, { message: ErrorCode.VOICE_UNSUPPORTED_FORMAT })
   format: VoiceAudioFormat;
 
-  /**
-   * The clinician's IANA zone. The server derives "today" from it rather than trusting a
-   * client-supplied date, which is what relative deadlines resolve against.
-   */
+  /** The clinician's IANA zone; "today" is derived from it, never sent by the client. */
   @IsString()
   @MaxLength(64)
   timeZone: string;
 
-  /**
-   * Recording length as measured by the client.
-   *
-   * Required, not optional: an optional value means omitting it bypasses
-   * VOICE_MAX_RECORDING_MS entirely, which would make the cap advisory.
-   */
+  /** Required, not optional — omitting it would bypass VOICE_MAX_RECORDING_MS entirely. */
   @IsInt()
   @Min(0)
   durationMs: number;
 
   /**
-   * The locale the clinician is actually speaking, as the UI offered the microphone.
-   *
-   * Sent explicitly rather than read from `user.language`: the two can diverge (a
-   * bookmarked /fa/ URL, a language toggle whose save failed), and a mismatch would
-   * transcribe Persian with an English hint and anchor "next Thursday" to the wrong
-   * week start. Gating the button and resolving the request must agree by construction.
+   * The locale the UI offered the microphone in. Sent explicitly because `user.language` can
+   * diverge from the URL locale, and a mismatch transcribes Persian with an English hint and
+   * anchors "next Thursday" to the wrong week start.
    */
   @IsIn(VOICE_LOCALES)
   locale: string;

backend/src/modules/voice/due-date.resolver.ts

@@ -15,7 +15,6 @@ import type { DueIntent, UnresolvedItem, Weekday } from './voice.types';
  * to reason about instants.
  */
 
-/** JS `getUTCDay()` numbering: Sunday = 0. */
 const WEEKDAY_TO_JS: Record<Weekday, number> = {
   saturday: 6,
   sunday: 0,
@@ -26,7 +25,6 @@ const WEEKDAY_TO_JS: Record<Weekday, number> = {
   friday: 5,
 };
 
-/** Refuse absurd deadlines however they were arrived at. */
 const MAX_DAYS_AHEAD = 365 * 5;
 
 export type DueResolution = {
@@ -78,12 +76,9 @@ function unresolved(spoken: string): DueResolution {
 }
 
 /**
- * What to quote back when a deadline could not be resolved.
- *
- * Every field here is nullable on the wire and `toVoiceIntent` casts rather than checks,
- * so a half-classified deadline arrives with nulls in it. The review sheet renders this
- * verbatim — `"null null" — not a usable date` in front of a clinician is worse than the
- * reason on its own, which the sheet already handles for a blank string.
+ * What to quote back when a deadline could not be resolved. Every field is nullable on the
+ * wire and `toVoiceIntent` casts rather than checks, and the sheet renders this verbatim —
+ * so a half-classified deadline must fall back to '', not to `"null null"`.
  */
 function describe(intent: DueIntent): string {
   const usable = (value: unknown): value is number =>
@@ -108,8 +103,7 @@ function describe(intent: DueIntent): string {
         ? `${intent.y}-${intent.m}-${intent.d}`
         : '';
     default: {
-      // Reaching here means an unrecognised `kind`, which resolveDueDate has already
-      // established is a string — echo it so the review row names what was heard.
+      // An unrecognised `kind`, already established as a string — echo what was heard.
       const kind = (intent as { kind?: unknown })?.kind;
       return typeof kind === 'string' ? kind : '';
     }
@@ -117,11 +111,9 @@ function describe(intent: DueIntent): string {
 }
 
 /**
- * Which weekday starts the week, per locale.
- *
  * "Next Thursday" is week-relative, so this changes the answer: the Iranian week starts
- * Saturday, the Dutch and (European) English week starts Monday. Hardcoding Saturday
- * would put an en/nl clinician's deadline a week out.
+ * Saturday, the Dutch and English week Monday. Hardcoding either puts the other locale's
+ * deadline a week out.
  */
 const WEEK_START_BY_LOCALE: Record<string, number> = {
   fa: WEEKDAY_TO_JS.saturday,
@@ -135,22 +127,18 @@ export function weekStartForLocale(locale: string): number {
   return WEEK_START_BY_LOCALE[locale] ?? DEFAULT_WEEK_START;
 }
 
-/** Most recent week-start day, counting today if today is that day. */
 function startOfWeek(iso: string, weekStartJs: number): string {
   const back = (civilDateJsWeekday(iso) - weekStartJs + 7) % 7;
   return addDays(iso, -back);
 }
 
 /**
- * `'this'` is occurrence-anchored: the soonest occurrence strictly after today, so "by
- * Thursday" said on a Thursday means the next one — a deadline of today is almost never
- * what was meant, and this can never resolve into the past.
+ * `'this'` is occurrence-anchored: the soonest occurrence strictly after today, so it can
+ * never resolve into the past.
  *
- * `'next'` is *week*-anchored, not "this plus seven". "Thursday next week" means the
- * Thursday of the Saturday-start week after this one; adding a week to `'this'` would
- * overshoot by seven days whenever `'this'` had already rolled into next week. The two
- * can legitimately coincide — said on a Thursday, "the coming Saturday" and "Saturday
- * next week" are the same day.
+ * `'next'` is *week*-anchored, not "this plus seven" — adding a week to `'this'` overshoots
+ * by seven days whenever `'this'` has already rolled into next week. The two legitimately
+ * coincide: said on a Thursday, "the coming Saturday" and "Saturday next week" are one day.
  */
 function resolveWeekday(
   intent: Extract<DueIntent, { kind: 'weekday' }>,
@@ -199,18 +187,15 @@ export function resolveDueDate(
   todayIso: string,
   weekStartJs: number = DEFAULT_WEEK_START,
 ): DueResolution {
-  // Absent is not an error — most utterances carry no deadline. Anything else that is not
-  // an intent object is a deadline we failed to understand, and must be flagged rather
-  // than silently dropped.
+  // Absent is not an error — most utterances carry no deadline.
   if (intent === null || intent === undefined) {
     return { dueDate: null, unresolved: null };
   }
   if (typeof intent !== 'object') {
     return unresolved(String(intent).slice(0, 120));
   }
-  // An object carrying no `kind` at all says nothing about a deadline; flagging it would
-  // put a blank "heard but lost" row in front of a clinician who never mentioned one. An
-  // object with an *unrecognised* kind did try to say something, and is flagged below.
+  // No `kind` at all says nothing about a deadline, so it is not "heard but lost". An
+  // *unrecognised* kind did try to say something, and is flagged below.
   if (typeof (intent as { kind?: unknown }).kind !== 'string') {
     return { dueDate: null, unresolved: null };
   }
@@ -240,8 +225,7 @@ export function resolveDueDate(
 
   if (!resolved) return unresolved(describe(intent));
 
-  // An absolute date the model invented can land anywhere; a deadline in the past or
-  // decades away is not a deadline.
+  // A date the model invented can land anywhere; past or decades away is not a deadline.
   const daysAhead = (utcMsOf(resolved) - utcMsOf(todayIso)) / 86_400_000;
   if (daysAhead < 0 || daysAhead > MAX_DAYS_AHEAD)
     return unresolved(describe(intent));

backend/src/modules/voice/extraction.resolver.ts

@@ -91,12 +91,9 @@ function mergeOverlapping(sets: string[][]): string[][] {
 }
 
 /**
- * Turn spoken bridge spans plus loose teeth into selection groups.
- *
- * Span teeth are added to the selection: saying "a bridge from 14 to 16" selects 15 even
- * though it was never named. A span whose endpoints are in different arches is impossible
- * and is reported rather than guessed at. A span that collapses to one tooth degrades to a
- * single — there is no such thing as a one-tooth bridge.
+ * Span teeth join the selection: "a bridge from 14 to 16" selects 15 though it was never
+ * named. A cross-arch span is reported rather than guessed at, and a span collapsing to one
+ * tooth degrades to a single — there is no one-tooth bridge.
  */
 export function resolveConnectedSpans(
   spans: readonly ConnectedSpanIntent[],
@@ -169,10 +166,8 @@ export function resolveConnectedSpans(
 }
 
 /**
- * Expand a default prosthesis type across the selection, then apply per-tooth overrides.
- *
- * "همه زیرکونیا، ۲۶ پی‌اف‌ام" is how clinicians actually speak, so the model names the type
- * once and overrides the exceptions.
+ * A default across the selection, then per-tooth overrides — "همه زیرکونیا، ۲۶ پی‌اف‌ام" is
+ * how clinicians actually speak.
  */
 export function resolveProsthesis(
   intent: ProsthesisIntent | null | undefined,
@@ -282,14 +277,12 @@ export function resolveVoiceIntent(
       ? intent.comment.trim()
       : null;
 
-  // A lab id the model invented is worse than none — it would ship a case to a lab the
-  // clinic never named. Only ids from the list we supplied survive, and a rejected one is
-  // reported: a hallucinated lab must not look identical to "no lab was spoken".
+  // An invented lab id would ship a case to a lab the clinic never named. A rejected one is
+  // reported, so it cannot look identical to "no lab was spoken".
   const labId = resolveCatalogCode(intent?.labId, ctx.linkedLabIds);
   if (intent?.labId != null && !labId) {
-    // `spoken` means "what the clinician said". A rejected lab id is an opaque
-    // identifier the model invented, so quoting it back would put a raw UUID in front
-    // of the user; the reason alone carries the meaning.
+    // `spoken` is what the clinician said — quoting an invented id back would put a raw
+    // UUID in front of the user.
     unresolved.push({ spoken: '', reason: 'unknown_catalog_code' });
   }
 

backend/src/modules/voice/extraction.wire.ts

@@ -10,13 +10,10 @@ import type {
 import { WEEKDAYS } from './voice.types';
 
 /**
- * The shape the model actually emits, and its JSON schema.
- *
  * Deliberately flat: strict `json_schema` mode has poor support for discriminated unions,
- * so every variant field is present and nullable on the wire. `toVoiceIntent` narrows the
- * flat shape into the internal union the resolvers consume, and is total — anything it
- * cannot classify becomes a shape the resolvers will report as unresolved rather than
- * something that throws here.
+ * so every variant field is present and nullable. `toVoiceIntent` narrows it into the
+ * internal union and is total — anything it cannot classify becomes a shape the resolvers
+ * report as unresolved rather than something that throws here.
  */
 
 export type WireToothIntent = {
@@ -183,9 +180,8 @@ const FDI_SHAPE = /^[1-8][1-8]$/;
 
 function toToothIntent(wire: WireToothIntent | undefined | null): ToothIntent {
   const spoken = typeof wire?.spoken === 'string' ? wire.spoken : '';
-  // Persian digits and digit-by-digit dictation ("۲۶", "2 6") are FDI codes that do not
-  // match literally; without normalising first they fall through to the positional branch
-  // with no quadrant and are reported as unresolved.
+  // "۲۶" and "2 6" are FDI codes that do not match literally; unnormalised they fall
+  // through to the positional branch with no quadrant and read as unresolved.
   const fdi = normalizeFdiCode(wire?.fdi);
   // Only take the explicit branch for something actually FDI-shaped. A model that emits
   // fdi:"6" alongside correct arch/side/position would otherwise lose the tooth entirely.

backend/src/modules/voice/openrouter.provider.ts

@@ -124,9 +124,8 @@ export class OpenRouterExtractionProvider implements ExtractionProvider {
       body: JSON.stringify({
         model: this.config.model,
         temperature: 0,
-        // Only route to endpoints that actually honour the JSON schema. Without this,
-        // OpenRouter may pick a provider that treats it as a hint and returns prose,
-        // which fails parsing intermittently and unreproducibly.
+        // Only route to endpoints that actually honour the JSON schema — otherwise OpenRouter may
+        // pick a provider that treats it as a hint and returns prose, failing intermittently.
         provider: { require_parameters: true },
         messages: buildExtractionPrompt(transcript, catalog, localeHint),
         response_format: {

backend/src/modules/voice/tooth-intent.resolver.ts

@@ -18,18 +18,14 @@ export type ToothResolution = {
 
 /** Everything here parses untrusted model output, so nothing may throw. */
 function normalizedFdi(intent: ToothIntent): string {
-  // Same normalisation the wire layer used to pick this branch, so the two cannot
-  // disagree: '14 ' is tooth 14 through the treatment API and '۲۶' is tooth 26, and
-  // neither may be reported as malformed here.
+  // The same normalisation the wire layer used to pick this branch, so the two agree.
   return normalizeFdiCode((intent as { fdi?: unknown }).fdi);
 }
 
 /**
- * Resolve one spoken tooth reference to an FDI code, or null.
- *
- * Never guesses and never clamps: a position of 9, a deciduous tooth, or a malformed
- * intent resolves to null so the caller can surface it as "not understood" rather than
- * silently selecting a neighbouring tooth.
+ * Never guesses and never clamps: position 9, a deciduous tooth or a malformed intent all
+ * resolve to null, so the caller surfaces "not understood" rather than silently selecting a
+ * neighbouring tooth.
  */
 export function resolveToothIntent(intent: ToothIntent): string | null {
   if (!intent || typeof intent !== 'object') return null;
@@ -65,9 +61,8 @@ function unresolvedReason(intent: ToothIntent): UnresolvedItem['reason'] {
       intent.position < 1 ||
       intent.position > 8;
     if (positionBad) return 'position_out_of_range';
-    // The position was understood, so the words were not the problem: the speaker never
-    // said which quadrant. "دندون دو" names four teeth at once, and telling the
-    // clinician it "could not be read" would send them looking for the wrong fault.
+    // The position was understood; the quadrant was never said. "دندون دو" names four
+    // teeth, so "could not be read" would send the clinician after the wrong fault.
     const archMissing = intent.arch !== 'upper' && intent.arch !== 'lower';
     const sideMissing =
       intent.side !== 'patient_right' && intent.side !== 'patient_left';
@@ -78,11 +73,8 @@ function unresolvedReason(intent: ToothIntent): UnresolvedItem['reason'] {
 }
 
 /**
- * The teeth still consistent with what *was* heard.
- *
- * Narrowed by whatever the clinician did say, so "دو" offers four and "دو بالا" offers
- * two. This is not a guess — it is the full set of readings, handed to the clinician to
- * choose from rather than picked on their behalf.
+ * The teeth still consistent with what *was* heard — "دو" leaves four, "دو بالا" two. Not
+ * a guess: the full set of readings, for the clinician to choose from.
  */
 function quadrantCandidates(intent: ToothIntent): string[] {
   if (intent.kind !== 'positional') return [];
@@ -109,11 +101,8 @@ function spokenOf(intent: ToothIntent): string {
 }
 
 /**
- * Resolve a list of spoken tooth references.
- *
- * Duplicates collapse — a clinician may name the same tooth twice in one sentence — and
- * anything unresolvable is reported rather than dropped, so the review sheet can show the
- * user exactly which words were not understood.
+ * Duplicates collapse; anything unresolvable is reported rather than dropped, so the sheet
+ * can show which words were not understood.
  */
 export function resolveToothIntents(
   intents: readonly ToothIntent[],
@@ -138,10 +127,8 @@ export function resolveToothIntents(
     const spoken = spokenOf(intent);
     const candidates =
       reason === 'tooth_missing_quadrant' ? quadrantCandidates(intent) : [];
-    // Only dedupe items we can actually tell apart. Without `spoken`, two distinct lost
-    // references would collapse into one blank review row and a tooth would vanish. The
-    // candidates are part of the identity: the same word with a different arch heard
-    // offers a different choice.
+    // Only dedupe what we can tell apart: without `spoken`, two lost references collapse
+    // into one blank row and a tooth vanishes. Candidates are part of the identity.
     if (spoken) {
       const key = `${spoken}::${reason}::${candidates.join(',')}`;
       if (seenUnresolved.has(key)) continue;

backend/src/modules/voice/voice-throttler.guard.ts

@@ -3,15 +3,9 @@ import { ThrottlerGuard } from '@nestjs/throttler';
 import { AppException, ErrorCode } from '../../common/errors';
 
 /**
- * Rate limits voice extraction per user rather than per IP.
- *
- * The default tracker keys on `req.ip`, which behind nginx means the whole deployment
- * shares one bucket unless `trust proxy` is set — and an abuser rotating IPs would bypass
- * it entirely. Since v1 ships with no plan gate, this is the only control on metered
- * vendor spend, so it has to key on something the client cannot change.
- *
- * Guard order matters: the controller's JwtAuthGuard runs before this method-level guard,
- * so `req.user` is populated by the time `getTracker` is called.
+ * Rate limits voice extraction per user, not per IP: the default tracker keys on `req.ip`,
+ * which behind nginx means the whole deployment shares one bucket unless `trust proxy` is
+ * set, and one clinic could then lock out every other.
  */
 @Injectable()
 export class VoiceThrottlerGuard extends ThrottlerGuard {

backend/src/modules/voice/voice.controller.ts

@@ -48,9 +48,8 @@ export class VoiceController {
     @Res({ passthrough: true }) res: Response,
     @Body() dto: ExtractVoiceDto,
   ) {
-    // Cancelling in the browser closes the connection; propagate that as an abort so the
-    // in-flight vendor call stops rather than settling and being discarded. It is metered
-    // per minute, so letting it run costs real money for a result nobody will see.
+    // Cancelling in the browser closes the connection; propagate it as an abort so the vendor
+    // call stops rather than settling unseen. It is metered per minute.
     const aborter = new AbortController();
     res.on('close', () => {
       if (!res.writableFinished) aborter.abort();

backend/src/modules/voice/voice.providers.ts

@@ -1,10 +1,8 @@
 import type { VoiceIntent } from './voice.types';
 
 /**
- * ASR and extraction are separate, independently swappable roles — they will not come
- * from the same vendor for every locale. Both are resolved per locale from
- * `config.voice.profiles`, so pointing `fa` at a Persian-specialist vendor while `en`
- * and `nl` keep OpenRouter is configuration, not code.
+ * ASR and extraction are separate, independently swappable roles — they will not come from
+ * the same vendor for every locale. Both resolve per locale from `config.voice.profiles`.
  */
 
 export type AudioInput = {

backend/src/modules/voice/voice.service.ts

@@ -55,10 +55,8 @@ export class VoiceService {
   }
 
   /**
-   * What the frontend needs to decide whether to render the microphone at all.
-   *
-   * v1 ships ungated beyond a configured locale profile — no plan check. The
-   * Plan.features design is deferred, not dropped.
+   * What the frontend needs to decide whether to render the microphone. v1 is ungated beyond
+   * a configured locale profile; the Plan.features design is deferred, not dropped.
    */
   getAvailability(): VoiceAvailability {
     const voice = this.voiceConfig;
@@ -107,10 +105,9 @@ export class VoiceService {
       throw this.toAppException(error, 'asr');
     }
 
-    // durationMs is client-reported and therefore not enforcement. usage.seconds is the
-    // vendor's own measurement of the audio it decoded, so a client under-reporting length
-    // to slip past the cap is caught here — after the ASR spend, but before the extraction
-    // call, and visibly in telemetry.
+    // durationMs is client-reported, so not enforcement. usage.seconds is the vendor's own
+    // measurement — a client under-reporting to slip past the cap is caught here, after the
+    // ASR spend but before the more expensive extraction call.
     if (asrSeconds != null) {
       this.assertWithinCap(asrSeconds * 1000);
     }
@@ -211,13 +208,9 @@ export class VoiceService {
   }
 
   /**
-   * Grace above the configured cap.
-   *
-   * The client auto-stops when elapsed >= maxMs, then measures the final length after the
-   * recorder has actually stopped — so a recording that runs to the cap always reports
-   * slightly over it. Without this tolerance the auto-stop would guarantee a rejection,
-   * discarding exactly the recording it was meant to save. The client still reports the
-   * true length, so telemetry stays honest.
+   * The client auto-stops at maxMs and only then measures, so a capped recording always
+   * reports slightly over. Without this tolerance every auto-stopped recording — the exact
+   * case the cap exists for — would be rejected as too long.
    */
   private static readonly CAP_TOLERANCE_MS = 2_000;
 
@@ -319,10 +312,7 @@ export class VoiceService {
     );
   }
 
-  /**
-   * Structured, patient-free. Never the transcript, never audio, never a patient id.
-   * Log lines are the interim sink until this repo has metrics infrastructure.
-   */
+  /** Structured and patient-free: never the transcript, never audio, never a patient id. */
   private logTelemetry(input: {
     locale: string;
     durationMs: number;

backend/src/modules/voice/voice.types.ts

@@ -78,10 +78,8 @@ export type UnresolvedItem = {
   spoken: string;
   reason: UnresolvedReason;
   /**
-   * FDI codes still consistent with what was heard, when a choice would settle it.
-   * Only `tooth_missing_quadrant` carries these: "دو" leaves four teeth on the table,
-   * "دو بالا" leaves two. The review sheet offers them so an under-specified tooth is one
-   * tap from resolved rather than a dead end.
+   * FDI codes still consistent with what was heard — "دو" leaves four, "دو بالا" two. Only
+   * `tooth_missing_quadrant` carries them; the sheet offers them as chips.
    */
   candidates?: string[];
 };

docs/specs/voice-treatment-entry/spec.md

@@ -145,9 +145,25 @@ path**. `onAddDetail` is not called and not changed.
 `setEntryStep('treatment')` matters: `showChrome` is always on, so the control is visible
 during the **Lab** wizard step too. Confirming there returns to the treatment step.
 
+**Confirm also saves.** The new detail is persisted immediately (`persistDraft({force:true})`),
+and when the result carries a lab, a due date or a prosthesis map the lab case is saved with it
+(`persistLabCases`). Not politeness — the autosave effect watches `details` only, so a lab draft
+left in component state alone loses the destination lab, the due date and the whole prosthesis
+map on the next reload. The detail survives, which is what makes that loss look like a
+successful save.
+
+One guard on it: `persistDraft` returns a **preview** treatment instead of saving when any
+detail still lacks a treatment type — the blank chip the workspace opens with is enough — and a
+preview's detail id falls back to the client id. Confirm therefore checks *what came back*, not
+the precondition, and skips the lab-case save when it did not get a real id; posting a lab case
+against an id the server has never seen fails the whole save. Checking the result rather than
+the condition keeps this true for every early return `persistDraft` has.
+
 > Accepted consequences:
 > - Tapping Add and then 🎤 leaves behind the blank chip that Add created. It carries the
 >   usual trash affordance.
+> - That same blank chip blocks confirm's immediate lab-case save until it is given a type
+>   or removed; the lab rows stay in local state until the ordinary Lab-step save.
 > - Dictating into an existing detail is not supported in v1 — voice always makes a new
 >   one.
 
@@ -482,6 +498,17 @@ that justified this whole design.
   - any row carrying an unresolved item or an incomplete prosthesis map.
 - Unresolved items are shown with what was heard ("دندان شیری — بازشناسی نشد"), so the
   clinician can see what the system did not understand.
+- **The sheet is a contract: confirm fills exactly what it previewed — no more.** Any
+  per-detail convenience that would top the case up afterwards has to be suppressed for a
+  voice-created case, because a default that quietly adds a prosthesis type to a tooth the
+  sheet never mentioned turns the confirmation step into a lie about what it was going to
+  do — which is the whole reason the step exists.
+
+  > This branch carried an exemption for one such default, the dispatch panel's
+  > remembered-prosthesis auto-fill. `origin/master` deleted that feature outright
+  > (`f52ad6b`), so the exemption went with it in the rebase and nothing enforces this rule
+  > in code today. It is a constraint on whatever gets added next, not a description of
+  > something that exists.
 - An item that carries `candidates` renders them as **tappable chips** — the one place the
   sheet is interactive. Picking one folds the tooth into the result (`withChosenTeeth`) and
   ticks the teeth row, so an under-specified tooth is one tap from resolved instead of a
@@ -516,8 +543,8 @@ who can edit treatments, in every configured locale. `Plan.features.voiceTreatme
 and the availability API stay documented here as the intended gate, deferred rather than
 dropped, so turning them on later is additive.
 
-Consequence to accept deliberately: with no plan gate and no duration cap (§2), the
-per-user throttle is the **only** control on metered vendor spend. See open item 14.
+Consequence to accept deliberately: with no plan gate, the per-user throttle and the 2-minute
+recording cap are the **only** controls on metered vendor spend. See open item 14.
 
 ---
 
@@ -529,15 +556,31 @@ English Nest exception for a user-facing failure.
 
 | Code | When |
 |---|---|
-| `VOICE_MIC_DENIED` | browser permission refused — **client-side only**: needs the `errors.X` key in all three message files, but no `ErrorCode` entry and no throw site |
-| `VOICE_CLIP_TOO_LONG` | over `maxMs` (server-side re-check), or over vendor limits |
-| `VOICE_UNSUPPORTED_FORMAT` | recorder produced a container the profile rejects |
+| `VOICE_MIC_DENIED` | microphone permission actually refused, or no input device — **client-side only**: needs the `errors.X` key in all three message files, but no `ErrorCode` entry and no throw site. Reserved for a real permission failure: see the note below |
+| `VOICE_CLIP_TOO_LONG` | over `maxMs` (server-side re-check), over vendor limits, or a request body past the DTO's size cap |
+| `VOICE_UNSUPPORTED_FORMAT` | **the browser cannot record at all** — no `MediaRecorder`, no container both it and the API accept, or a recorder that throws after permission was granted; and server-side, a `format` outside `VOICE_AUDIO_FORMATS` |
 | `VOICE_ASR_FAILED` | transcription stage failed |
 | `VOICE_EXTRACT_FAILED` | transcript obtained, structuring failed |
 | `VOICE_NOTHING_RECOGNIZED` | empty or unusable transcript |
 | `VOICE_NOT_AVAILABLE` | no profile for locale (v1); plan flag off, once enforced |
 | `VOICE_RATE_LIMITED` | throttle |
 
+**Two of these are raised by DTO validation, not by a throw site.**
+`validationExceptionFactory` returns a constraint's `message` verbatim when the message is
+itself a known `ErrorCode`, so the voice DTO names its own failures:
+`@MaxLength(…, { message: ErrorCode.VOICE_CLIP_TOO_LONG })` and
+`@IsIn(…, { message: ErrorCode.VOICE_UNSUPPORTED_FORMAT })`. Left to the shared constraint map
+they fall through to `VALIDATION_FIELD_REQUIRED` and `VALIDATION_LANGUAGE_INVALID` — an
+oversized recording telling the clinician a field is missing, and an unsupported container
+telling them their language is invalid. Any new voice constraint should name its code the same
+way.
+
+**`VOICE_MIC_DENIED` is only for a real permission failure.** Three client paths used to report
+it for something else entirely — no `MediaRecorder`, no acceptable container, and a recorder
+that throws after permission was already granted. All three are "this browser cannot record"
+and now report `VOICE_UNSUPPORTED_FORMAT`; blaming the microphone sends the clinician hunting
+in site settings for a permission nothing ever asked for.
+
 **Transcript salvage — specified, NOT built.** The backend half exists: `VOICE_EXTRACT_FAILED`
 carries `details.transcript` and `HttpExceptionFilter` forwards it. The client half was
 never written — `onError` only resolves a message through `getUserFacingError`, which never
@@ -695,7 +738,16 @@ enabling this for real clinics.
   - no layout shift in the header row on record start, stop, or the 2:00 auto-stop;
   - **hold past 2:00** → auto-stops and proceeds to processing, not an error;
   - **cancel during processing** → the vendor request is actually aborted;
-  - **review sheet on mobile** → full-screen overlay; closing it leaves the draft intact.
+  - **review sheet on mobile** → full-screen overlay; closing it leaves the draft intact;
+  - **confirm with a lab, a due date or a prosthesis map, then reload** → all three are still
+    there. They live on the lab case, which the autosave effect does not watch, so this is
+    the check that catches a lab draft left unsaved in component state;
+  - **record straight after opening a visit**, while the blank chip is still untyped, and
+    confirm with a lab ticked → no error toast: confirm detects the preview treatment and
+    skips the lab-case save rather than posting an id the server has never seen;
+  - **dictate two different prosthesis types** ("۱۲ روکش PFM، ۱۳ روکش PFZ") → the form shows
+    both, and the bulk «اعمال برای همه دندان‌ها» select stays on its placeholder. Nothing may
+    rewrite a per-tooth type the sheet already showed.
 
 ---
 
@@ -727,6 +779,14 @@ Settled in a grilling session on 2026-08-20.
 | 28 | Review sheet | Modal on desktop, full-screen overlay (not a route) on mobile; candidate chips are its only interactive part |
 | 29 | Cancel | Aborts the in-flight vendor call |
 | 30 | v1 gating | Open to everyone; `Plan.features` gate deferred, not dropped |
+
+Added while getting the first live recordings working (2026-08-21):
+
+| # | Question | Decision |
+|---|---|---|
+| 31 | Tooth numbering | A spoken number **is** its FDI code, in all three locales. A lone digit stays unresolved and offers its candidate teeth as chips (§6, §7) |
+| 32 | What confirm writes | Confirm persists the detail *and* its lab case, because autosave watches `details` only — but skips the lab-case save when it got a preview treatment back (§2) |
+| 33 | Preview as contract | Applying a voice result fills exactly what the sheet showed. Per-detail conveniences that would add more are suppressed for that case (§7) |
 | 15 | Gating | `Plan.features` flag — its first consumer |
 
 UI placement settled in a second grilling session on 2026-08-20.

frontend/messages/en.json

@@ -683,7 +683,9 @@
     "selectedPatient": "Selected patient",
     "purposeLabel": "Purpose:",
     "loadingAppointments": "Loading appointments…",
-    "selectDayWithAppointment": "Select a day with at least one appointment.",
+    "selectDayWithAppointment": "Search for a patient or select a visit from the day strip.",
+    "noTreatmentFoundTitle": "No treatment found",
+    "noTreatmentFoundBody": "There is no treatment or visit for {name} yet. You can add one with the {action} button in the rail.",
     "confirmDiscard": "You have unsaved changes. Discard them and continue?",
     "errorChooseOrg": "Choose at least one active organization to send this case.",
     "successCaseSent": "Case sent to selected organizations.",
@@ -707,6 +709,8 @@
     "walkIn": "Walk-in",
     "newTreatment": "New treatment",
     "newTreatmentPatientPrompt": "Who is this visit for?",
+    "newTreatmentUseCurrent": "Use {name}",
+    "newTreatmentUseCurrentHint": "Start a new visit for the patient already open on this page.",
     "walkInPickerHint": "No named patient — always available, no search needed.",
     "errorCreateTreatment": "Could not create treatment.",
     "deleteEmptyTreatment": "Delete empty treatment",
@@ -717,6 +721,7 @@
     "detailsTitle": "Treatment details",
     "addDetail": "Add detail",
     "confirmRemoveDetail": "Remove this treatment detail?",
+    "confirmRemoveLastDetail": "This is the last detail. Removing it leaves the plan empty until you add another. Continue?",
     "removeDetailAria": "Remove detail {n}",
     "detailLabel": "Detail {n}",
     "detailSentBadge": "sent",

frontend/messages/fa.json

@@ -684,7 +684,9 @@
     "selectedPatient": "بیمار انتخاب شده",
     "purposeLabel": "هدف:",
     "loadingAppointments": "در حال بارگذاری نوبت‌ها...",
-    "selectDayWithAppointment": "روزی را انتخاب کنید که حداقل یک نوبت داشته باشد.",
+    "selectDayWithAppointment": "برای بیمار جستجو کنید یا یک ویزیت را از نوار روز انتخاب کنید.",
+    "noTreatmentFoundTitle": "درمانی یافت نشد",
+    "noTreatmentFoundBody": "هنوز درمان یا ویزیتی برای {name} ثبت نشده است. می‌توانید با دکمه {action} در نوار کناری یکی اضافه کنید.",
     "confirmDiscard": "تغییرات ذخیره‌نشده دارید. آنها را کنار بگذارید و ادامه دهید؟",
     "errorChooseOrg": "حداقل یک سازمان فعال را برای ارسال این پرونده انتخاب کنید.",
     "successCaseSent": "پرونده به سازمان‌های انتخاب شده ارسال شد.",
@@ -708,6 +710,8 @@
     "walkIn": "بدون نوبت (مراجع)",
     "newTreatment": "درمان جدید",
     "newTreatmentPatientPrompt": "این ویزیت برای چه کسی است؟",
+    "newTreatmentUseCurrent": "استفاده از {name}",
+    "newTreatmentUseCurrentHint": "ویزیت جدیدی برای بیماری که همین حالا در این صفحه باز است شروع کنید.",
     "walkInPickerHint": "بیمار نام‌دار نیست — همیشه در دسترس است و نیازی به جستجو ندارد.",
     "errorCreateTreatment": "ایجاد درمان ممکن نشد.",
     "deleteEmptyTreatment": "حذف درمان خالی",
@@ -718,6 +722,7 @@
     "detailsTitle": "جزئیات درمان",
     "addDetail": "افزودن جزئیات",
     "confirmRemoveDetail": "این جزئیات درمان حذف شود؟",
+    "confirmRemoveLastDetail": "این آخرین جزئیات است. با حذف آن برنامه خالی می‌ماند تا جزئیات جدیدی اضافه کنید. ادامه می‌دهید؟",
     "removeDetailAria": "حذف جزئیات {n}",
     "detailLabel": "جزئیات {n}",
     "detailSentBadge": "ارسال‌شده",

frontend/messages/nl.json

@@ -683,7 +683,9 @@
     "selectedPatient": "Geselecteerde patiënt",
     "purposeLabel": "Doel:",
     "loadingAppointments": "Afspraken laden...",
-    "selectDayWithAppointment": "Selecteer een dag met ten minste één afspraak.",
+    "selectDayWithAppointment": "Zoek een patiënt of kies een bezoek uit de dagstrook.",
+    "noTreatmentFoundTitle": "Geen behandeling gevonden",
+    "noTreatmentFoundBody": "Er is nog geen behandeling of bezoek voor {name}. U kunt er een toevoegen met de knop {action} in de zijbalk.",
     "confirmDiscard": "U heeft niet-opgeslagen wijzigingen. Wilt u deze negeren en doorgaan?",
     "errorChooseOrg": "Kies ten minste één actieve organisatie om deze case te verzenden.",
     "successCaseSent": "Case verzonden naar geselecteerde organisaties.",
@@ -707,6 +709,8 @@
     "walkIn": "Inloop",
     "newTreatment": "Nieuwe behandeling",
     "newTreatmentPatientPrompt": "Voor wie is dit bezoek?",
+    "newTreatmentUseCurrent": "{name} gebruiken",
+    "newTreatmentUseCurrentHint": "Start een nieuw bezoek voor de patiënt die al op deze pagina openstaat.",
     "walkInPickerHint": "Geen benoemde patiënt — altijd beschikbaar, zonder zoeken.",
     "errorCreateTreatment": "Behandeling aanmaken is mislukt.",
     "deleteEmptyTreatment": "Lege behandeling verwijderen",
@@ -717,6 +721,7 @@
     "detailsTitle": "Behandeldetails",
     "addDetail": "Detail toevoegen",
     "confirmRemoveDetail": "Dit behandelingsdetail verwijderen?",
+    "confirmRemoveLastDetail": "Dit is het laatste detail. Als u het verwijdert, blijft het plan leeg tot u een nieuw detail toevoegt. Doorgaan?",
     "removeDetailAria": "Detail {n} verwijderen",
     "detailLabel": "Detail {n}",
     "detailSentBadge": "verzonden",

frontend/src/components/treatment/dayStrip.ts

@@ -12,3 +12,10 @@ export type DayStripItem = {
   subtitle: string;
   canDelete?: boolean;
 };
+
+/** Unscheduled card banner: first line’s type only. Empty / missing type → no color. */
+export function unscheduledStripColorCode(
+  details: readonly { treatmentType?: string | null }[],
+): string {
+  return details[0]?.treatmentType?.trim() ?? '';
+}

frontend/src/components/treatment/labDispatchDefaults.ts

@@ -1,73 +1,60 @@
 const STORAGE_PREFIX = 'dyolink.labDispatchDefaults.';
 
-export type LabDispatchDefaults = {
-  lastLabId: string | null;
-  lastProsthesisByLab: Record<string, string>;
-};
-
-const EMPTY: LabDispatchDefaults = {
-  lastLabId: null,
-  lastProsthesisByLab: {},
-};
+/** Chips under lab search — last destinations this clinic actually sent a case to. */
+export const MAX_RECENT_LABS = 3;
 
 function storageKey(clinicOrganizationId: string): string {
   return `${STORAGE_PREFIX}${clinicOrganizationId}`;
 }
 
-export function loadLabDispatchDefaults(clinicOrganizationId: string | null | undefined): LabDispatchDefaults {
-  if (!clinicOrganizationId || typeof window === 'undefined') return EMPTY;
+function uniqueIds(ids: string[]): string[] {
+  const out: string[] = [];
+  for (const id of ids) {
+    if (id && !out.includes(id)) out.push(id);
+  }
+  return out;
+}
+
+export function loadRecentLabIds(clinicOrganizationId: string | null | undefined): string[] {
+  if (!clinicOrganizationId || typeof window === 'undefined') return [];
   try {
     const raw = window.localStorage.getItem(storageKey(clinicOrganizationId));
-    if (!raw) return EMPTY;
-    const parsed = JSON.parse(raw) as Partial<LabDispatchDefaults>;
-    return {
-      lastLabId: typeof parsed.lastLabId === 'string' ? parsed.lastLabId : null,
-      lastProsthesisByLab:
-        parsed.lastProsthesisByLab && typeof parsed.lastProsthesisByLab === 'object'
-          ? parsed.lastProsthesisByLab
-          : {},
+    if (!raw) return [];
+    const parsed = JSON.parse(raw) as {
+      recentLabIds?: unknown;
+      lastLabId?: unknown;
     };
+    const fromList = Array.isArray(parsed.recentLabIds)
+      ? parsed.recentLabIds.filter((id): id is string => typeof id === 'string')
+      : [];
+    // Older builds stored a single lastLabId used as an auto-selected default.
+    const fromLegacy = typeof parsed.lastLabId === 'string' ? [parsed.lastLabId] : [];
+    return uniqueIds([...fromList, ...fromLegacy]).slice(0, MAX_RECENT_LABS);
   } catch {
-    return EMPTY;
+    return [];
   }
 }
 
-function writeDefaults(clinicOrganizationId: string, next: LabDispatchDefaults): void {
+function writeRecentLabIds(clinicOrganizationId: string, recentLabIds: string[]): void {
   if (typeof window === 'undefined') return;
   try {
-    window.localStorage.setItem(storageKey(clinicOrganizationId), JSON.stringify(next));
+    window.localStorage.setItem(
+      storageKey(clinicOrganizationId),
+      JSON.stringify({ recentLabIds }),
+    );
   } catch {
     // Ignore quota / private-mode failures.
   }
 }
 
-export function rememberLastLab(clinicOrganizationId: string | null | undefined, labId: string): void {
-  if (!clinicOrganizationId || !labId) return;
-  const current = loadLabDispatchDefaults(clinicOrganizationId);
-  writeDefaults(clinicOrganizationId, { ...current, lastLabId: labId });
-}
-
-export function rememberLastProsthesisType(
+/** Record a lab after a successful send — suggestion chips only, never a form default. */
+export function rememberRecentLab(
   clinicOrganizationId: string | null | undefined,
   labId: string,
-  prosthesisTypeCode: string,
 ): void {
-  if (!clinicOrganizationId || !labId || !prosthesisTypeCode) return;
-  const current = loadLabDispatchDefaults(clinicOrganizationId);
-  writeDefaults(clinicOrganizationId, {
-    ...current,
-    lastLabId: labId,
-    lastProsthesisByLab: {
-      ...current.lastProsthesisByLab,
-      [labId]: prosthesisTypeCode,
-    },
-  });
-}
-
-export function lastProsthesisTypeForLab(
-  clinicOrganizationId: string | null | undefined,
-  labId: string | null | undefined,
-): string | null {
-  if (!clinicOrganizationId || !labId) return null;
-  return loadLabDispatchDefaults(clinicOrganizationId).lastProsthesisByLab[labId] ?? null;
+  if (!clinicOrganizationId || !labId) return;
+  writeRecentLabIds(
+    clinicOrganizationId,
+    uniqueIds([labId, ...loadRecentLabIds(clinicOrganizationId)]).slice(0, MAX_RECENT_LABS),
+  );
 }

frontend/src/components/treatment/voiceReviewRows.ts

@@ -19,14 +19,9 @@ export function voiceRowAvailability(result: VoiceExtractionResult) {
 }
 
 /**
- * Which rows start ticked.
- *
- * Everything available ticks itself, with two deliberate exceptions:
- *
- * - **lab, when the name only approximately matched.** Shipping a case to a lab is the one
- *   extracted value whose error leaves the building, so it always requires a deliberate tick.
- * - **prosthesis, when the map is incomplete.** A prosthesis detail with an untyped tooth
- *   cannot ship at all, so applying it would just move the failure to dispatch.
+ * Everything available ticks itself, with two exceptions: an inexactly-matched lab, because
+ * it is the one extracted value whose error leaves the building; and an incomplete
+ * prosthesis map, which cannot ship at all and would just move the failure to dispatch.
  */
 export function initialVoiceSelection(result: VoiceExtractionResult): VoiceApplySelection {
   const available = voiceRowAvailability(result);
@@ -41,11 +36,8 @@ export function initialVoiceSelection(result: VoiceExtractionResult): VoiceApply
 }
 
 /**
- * How many rows will actually be applied — drives the confirm button's label.
- *
- * Intersected with availability rather than counting ticks: a row can be ticked and then
- * lose its content (the last candidate tooth un-picked), and "Apply 1 item" that applies
- * nothing is worse than a wrong number.
+ * Intersected with availability rather than counting ticks: a row can be ticked and then lose
+ * its content, and "Apply 1 item" that applies nothing is worse than a wrong number.
  */
 export function countSelected(
   selection: VoiceApplySelection,
@@ -66,16 +58,10 @@ function recheckProsthesis(
 }
 
 /**
- * Fold the clinician's candidate picks into the extracted result.
+ * Fold the candidate picks into the result, so nothing downstream has to know chips exist.
  *
- * Everything downstream reads a `VoiceExtractionResult` — row availability, the mini
- * chart, the prosthesis warning, `applyVoiceResult` — so resolving the picks into one here
- * means none of them has to know the chips exist.
- *
- * Union rather than toggle, for two reasons: a candidate can coincidentally be a tooth the
- * recording already produced ("۱۲ و دو"), where tapping it must not deselect that tooth;
- * and `groupsFromFlatTeeth` keeps the bridges intact while giving every remaining tooth a
- * single group, so no tooth can be lost on the way through.
+ * Union rather than toggle: a candidate can coincidentally be a tooth the recording already
+ * produced ("۱۲ و دو"), and tapping it must not deselect that one.
  */
 export function withChosenTeeth(
   result: VoiceExtractionResult,
@@ -103,12 +89,7 @@ export function connectedTeethFromResult(result: VoiceExtractionResult): Set<Fdi
   return connected;
 }
 
-/**
- * Whether the sheet has anything worth showing.
- *
- * A recording that produced nothing usable should say so plainly rather than present an
- * empty form of checkboxes.
- */
+/** A recording that produced nothing should say so, not show an empty form of checkboxes. */
 export function hasAnythingToApply(result: VoiceExtractionResult): boolean {
   return Object.values(voiceRowAvailability(result)).some(Boolean);
 }

frontend/src/components/ui/treatment/FdiToothChart.tsx

@@ -24,6 +24,12 @@ const TOOTH_NUMBER_GAP = 'mt-1';
 const REALISTIC_NUMBER_GAP = '2mm';
 /** Tight interproximal gap between tooth columns. */
 const TOOTH_GAP = 'gap-x-px';
+/**
+ * Centers a mark on the inline-end edge of a tooth column (between this tooth
+ * and the next in flex order). Logical `end` + 0-width flex stays correct in LTR and RTL.
+ */
+const EDGE_MARK_ANCHOR =
+  'absolute inset-y-0 end-0 z-10 w-0 flex items-center justify-center';
 
 function quadrantMirrored(fdi: FdiToothId): boolean {
   const q = fdi[0];
@@ -285,7 +291,8 @@ export function FdiToothChart({
                 />
               ) : null}
               {renderEdge ? (
-                linkInteractive ? (
+                <div className={EDGE_MARK_ANCHOR}>
+                  {linkInteractive ? (
                     <button
                       type="button"
                       disabled={isDisabled}
@@ -302,7 +309,7 @@ export function FdiToothChart({
                         onToggleLink?.(fdi, next!);
                       }}
                       className={`
-                      absolute right-0 z-10 translate-x-1/2 h-3.5 w-3.5 rounded-full border-2 transition-colors
+                        h-3.5 w-3.5 shrink-0 rounded-full border-2 transition-colors
                         focus:outline-none focus-visible:ring-2 focus-visible:ring-primary/50
                         ${
                           linked
@@ -314,11 +321,12 @@ export function FdiToothChart({
                     />
                   ) : (
                     <span
-                    className="absolute right-0 z-10 translate-x-1/2 h-2.5 w-2.5 rounded-full bg-primary shadow-sm"
+                      className="h-2.5 w-2.5 shrink-0 rounded-full bg-primary shadow-sm"
                       title={t('toothConnectedHint')}
                       aria-hidden
                     />
-                )
+                  )}
+                </div>
               ) : null}
             </div>
           );

frontend/src/components/ui/treatment/LabCasesDispatchPanel.tsx

@@ -1,6 +1,6 @@
 'use client';
 
-import { useEffect, useRef, useState } from 'react';
+import { useEffect, useState } from 'react';
 import { useTranslations } from 'next-intl';
 import { Button } from '@/components/ui/shared/Button';
 import { Checkbox } from '@/components/ui/shared/Checkbox';
@@ -16,12 +16,6 @@ import { LabCaseTrackerCard } from '@/components/ui/treatment/LabCaseTrackerCard
 import { treatmentTypeLabelFromCatalog } from '@/components/shared/treatmentTypeDisplay';
 import { treatmentsApi } from '@/lib/api/treatments';
 import { prosthesisCatalogApi } from '@/lib/api/prosthesis-catalog';
-import {
-  lastProsthesisTypeForLab,
-  loadLabDispatchDefaults,
-  rememberLastLab,
-  rememberLastProsthesisType,
-} from '@/components/treatment/labDispatchDefaults';
 import type { ProsthesisCatalogEntry, TreatmentCatalogEntry } from '@/types/treatment-catalog';
 import type { LabCaseDraft, LinkedOrganizationOption, TreatmentDetailDraft } from '@/types/treatment';
 import type { PatientLabCaseSummary } from '@/types/lab-case-activity';
@@ -34,7 +28,6 @@ interface LabCasesDispatchPanelProps {
   labCases: LabCaseDraft[];
   labDependentCodes: Set<string>;
   treatmentCatalog: TreatmentCatalogEntry[];
-  clinicOrganizationId?: string | null;
   labCaseSummary?: PatientLabCaseSummary | null;
   locale: string;
   onLabCaseSummaryChange?: (summary: PatientLabCaseSummary) => void;
@@ -48,7 +41,6 @@ interface LabCasesDispatchPanelProps {
   organizationSearch: string;
   onOrganizationSearchChange: (value: string) => void;
   recentOrganizationIds: string[];
-  onRecentOrganizationPick: (orgId: string) => void;
   canInviteLab?: boolean;
   onInviteLab?: () => void;
   sendBusyId: string | null;
@@ -124,7 +116,6 @@ export function LabCasesDispatchPanel({
   labCases,
   labDependentCodes,
   treatmentCatalog,
-  clinicOrganizationId,
   labCaseSummary,
   locale,
   onLabCaseSummaryChange,
@@ -138,7 +129,6 @@ export function LabCasesDispatchPanel({
   organizationSearch,
   onOrganizationSearchChange,
   recentOrganizationIds,
-  onRecentOrganizationPick,
   canInviteLab = false,
   onInviteLab,
   sendBusyId,
@@ -150,7 +140,6 @@ export function LabCasesDispatchPanel({
   const [prosthesisOptions, setProsthesisOptions] = useState<ProsthesisCatalogEntry[]>([]);
   const [applyAllProsthesis, setApplyAllProsthesis] = useState('');
   const [pendingComment, setPendingComment] = useState('');
-  const autoFilledCaseRef = useRef<string | null>(null);
   const hasTrackerSummary = Boolean(labCaseSummary && labCaseSummary.labCaseId);
 
   const activeLinkedOrganizations = orgs.filter((o) => o.active);
@@ -168,7 +157,14 @@ export function LabCasesDispatchPanel({
 
   const activeLabCase =
     labCaseForActiveDetail ??
-    (activeLabCaseId ? labCases.find((lc) => lc.clientId === activeLabCaseId) : null);
+    (activeLabCaseId
+      ? labCases.find(
+          (lc) =>
+            lc.clientId === activeLabCaseId &&
+            (lc.detailClientId == null || lc.detailClientId === activeDetailId),
+        )
+      : null) ??
+    null;
 
   const sent = Boolean(activeLabCase?.sentAt);
   const activeDetailNumber = details.findIndex((d) => d.clientId === activeDetailId) + 1;
@@ -208,6 +204,7 @@ export function LabCasesDispatchPanel({
 
   useEffect(() => {
     setPendingComment('');
+    setApplyAllProsthesis('');
   }, [activeLabCase?.clientId]);
 
   function updateActiveLabCase(patch: Partial<LabCaseDraft>) {
@@ -226,45 +223,6 @@ export function LabCasesDispatchPanel({
     // eslint-disable-next-line react-hooks/exhaustive-deps -- only sync newly uploaded files
   }, [activeDetail?.attachmentMetas, activeLabCase?.clientId, sent]);
 
-  useEffect(() => {
-    if (!activeLabCase || sent || activeLabCase.destinationOrganizationId) return;
-    const lastLabId = loadLabDispatchDefaults(clinicOrganizationId).lastLabId;
-    const lastLab = lastLabId
-      ? activeLinkedOrganizations.find((o) => o.id === lastLabId)
-      : undefined;
-    if (!lastLab) return;
-    updateActiveLabCase({ destinationOrganizationId: lastLab.id, toothProsthesis: [] });
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [activeLabCase?.clientId, clinicOrganizationId, sent]);
-
-  useEffect(() => {
-    if (!activeLabCase || sent) return;
-    if (!activeLabCase.destinationOrganizationId) return;
-    if (prosthesisOptions.length === 0 || prosthesisRows.length === 0) return;
-    const fillKey = `${activeLabCase.clientId}:${prosthesisRows.length}`;
-    if (autoFilledCaseRef.current === fillKey) return;
-    if (isProsthesisMapComplete(activeLabCase, prosthesisRows)) {
-      autoFilledCaseRef.current = fillKey;
-      return;
-    }
-    const lastCode = lastProsthesisTypeForLab(
-      clinicOrganizationId,
-      activeLabCase.destinationOrganizationId,
-    );
-    if (!lastCode || !prosthesisOptions.some((opt) => opt.code === lastCode)) return;
-    autoFilledCaseRef.current = fillKey;
-    setApplyAllProsthesis(lastCode);
-    updateActiveLabCase({ toothProsthesis: toothProsthesisForRows(prosthesisRows, lastCode) });
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [
-    activeLabCase?.clientId,
-    activeLabCase?.destinationOrganizationId,
-    clinicOrganizationId,
-    prosthesisOptions,
-    prosthesisRows.length,
-    sent,
-  ]);
-
   if (!activeDetail || !isLabDependentDetail) {
     return null;
   }
@@ -287,25 +245,11 @@ export function LabCasesDispatchPanel({
         ]
       : rest;
     updateActiveLabCase({ toothProsthesis: next });
-    if (prosthesisTypeCode && activeLabCase.destinationOrganizationId) {
-      rememberLastProsthesisType(
-        clinicOrganizationId,
-        activeLabCase.destinationOrganizationId,
-        prosthesisTypeCode,
-      );
-    }
   }
 
   function applyProsthesisToAll(code: string) {
     if (!activeLabCase || !code) return;
     updateActiveLabCase({ toothProsthesis: toothProsthesisForRows(prosthesisRows, code) });
-    if (activeLabCase.destinationOrganizationId) {
-      rememberLastProsthesisType(
-        clinicOrganizationId,
-        activeLabCase.destinationOrganizationId,
-        code,
-      );
-    }
   }
 
   function toggleAttachmentInActiveLabCase(attachmentId: string, checked: boolean) {
@@ -317,13 +261,11 @@ export function LabCasesDispatchPanel({
   }
 
   function handleSelectOrganization(org: LinkedOrganizationOption) {
-    autoFilledCaseRef.current = null;
     updateActiveLabCase({
       destinationOrganizationId: org.id,
       toothProsthesis: [],
     });
     setApplyAllProsthesis('');
-    rememberLastLab(clinicOrganizationId, org.id);
   }
 
   const caseFullyComplete = isLabCaseCompleted(activeLabCase?.taskProgress);
@@ -463,10 +405,7 @@ export function LabCasesDispatchPanel({
                         key={o.id}
                         type="button"
                         disabled={disabled}
-                        onClick={() => {
-                          handleSelectOrganization(o);
-                          onRecentOrganizationPick(o.id);
-                        }}
+                        onClick={() => handleSelectOrganization(o)}
                         className="text-xs rounded-[var(--radius-sm)] border border-border/70 px-2 py-1 text-text-secondary hover:text-text-primary hover:border-border focus:outline-none focus-visible:ring-2 focus-visible:ring-primary/35 disabled:opacity-50"
                       >
                         {o.name}
@@ -615,26 +554,7 @@ export function LabCasesDispatchPanel({
                     !prosthesisComplete
                   }
                   isLoading={sendBusyId === activeLabCase.clientId}
-                  onClick={() => {
-                    if (activeLabCase.destinationOrganizationId) {
-                      rememberLastLab(clinicOrganizationId, activeLabCase.destinationOrganizationId);
-                      const codes = [
-                        ...new Set(
-                          activeLabCase.toothProsthesis
-                            .map((tp) => tp.prosthesisTypeCode)
-                            .filter(Boolean),
-                        ),
-                      ];
-                      if (codes.length === 1) {
-                        rememberLastProsthesisType(
-                          clinicOrganizationId,
-                          activeLabCase.destinationOrganizationId,
-                          codes[0],
-                        );
-                      }
-                    }
-                    return onSendLabCase(activeLabCase, pendingComment.trim());
-                  }}
+                  onClick={() => onSendLabCase(activeLabCase, pendingComment.trim())}
                 >
                   {t('sendToLab')}
                 </Button>

frontend/src/components/ui/treatment/LabShipmentBlockedNotice.tsx

@@ -1,20 +0,0 @@
-'use client';
-
-import { AlertCircle } from 'lucide-react';
-import { useTranslations } from 'next-intl';
-
-export function LabShipmentBlockedNotice() {
-  const t = useTranslations('treatment');
-
-  return (
-    <div className="surface-card p-4 border border-amber-500/35 bg-amber-500/5">
-      <div className="flex items-start gap-2">
-        <AlertCircle className="h-4 w-4 text-amber-600 dark:text-amber-400 shrink-0 mt-0.5 icon-flat" />
-        <div className="min-w-0 space-y-1">
-          <h3 className="text-sm font-semibold text-text-primary">{t('labShipmentBlockedTitle')}</h3>
-          <p className="text-xs text-text-muted">{t('labShipmentBlockedBody')}</p>
-        </div>
-      </div>
-    </div>
-  );
-}

frontend/src/components/ui/treatment/NewTreatmentPatientPicker.tsx

@@ -4,19 +4,45 @@ import { useTranslations } from 'next-intl';
 import { Button } from '@/components/ui/shared/Button';
 import { PatientSearchCombobox } from '@/components/ui/patient/PatientSearchCombobox';
 import { usePatientSearchQuery } from '@/lib/hooks/usePatientSearchQuery';
+import { formatMobileForDisplay } from '@/lib/phone';
 import type { Patient } from '@/types/patient';
 
+const pickerChoiceClass =
+  'w-full rounded-[var(--radius-md)] border border-primary/40 bg-primary-soft px-3 py-2 text-start transition-colors hover:border-primary disabled:opacity-50 disabled:cursor-not-allowed focus:outline-none focus-visible:ring-2 focus-visible:ring-primary/45';
+
 interface NewTreatmentPatientPickerProps {
   creating?: boolean;
+  currentPatient?: {
+    id: string;
+    displayName: string;
+    mobile?: string | null;
+    email?: string | null;
+  } | null;
   onSelectWalkIn: () => void | Promise<void>;
   onSelectPatient: (patient: Patient) => void | Promise<void>;
+  onSelectCurrentPatient?: () => void | Promise<void>;
   onCancel: () => void;
 }
 
+function contactLine(
+  patient: { mobile?: string | null; email?: string | null },
+  fallback: string,
+): string {
+  const mobile = patient.mobile?.trim()
+    ? formatMobileForDisplay(patient.mobile.trim())
+    : '';
+  if (mobile) return mobile;
+  const email = patient.email?.trim();
+  if (email) return email;
+  return fallback;
+}
+
 export function NewTreatmentPatientPicker({
   creating = false,
+  currentPatient = null,
   onSelectWalkIn,
   onSelectPatient,
+  onSelectCurrentPatient,
   onCancel,
 }: NewTreatmentPatientPickerProps) {
   const t = useTranslations('treatment');
@@ -31,11 +57,25 @@ export function NewTreatmentPatientPicker({
         type="button"
         disabled={creating}
         onClick={() => void onSelectWalkIn()}
-        className="w-full rounded-[var(--radius-md)] border border-primary/40 bg-primary-soft px-3 py-2 text-start transition-colors hover:border-primary disabled:opacity-50 disabled:cursor-not-allowed focus:outline-none focus-visible:ring-2 focus-visible:ring-primary/45"
+        className={pickerChoiceClass}
       >
         <p className="text-sm font-medium text-text-primary">{t('walkIn')}</p>
         <p className="text-xs text-text-muted mt-0.5">{t('walkInPickerHint')}</p>
       </button>
+      {currentPatient && onSelectCurrentPatient ? (
+        <button
+          type="button"
+          disabled={creating}
+          onClick={() => void onSelectCurrentPatient()}
+          aria-label={t('newTreatmentUseCurrent', { name: currentPatient.displayName })}
+          className={pickerChoiceClass}
+        >
+          <p className="text-sm font-medium text-text-primary break-words">{currentPatient.displayName}</p>
+          <p className="text-xs text-text-muted mt-0.5 break-words">
+            {contactLine(currentPatient, t('newTreatmentUseCurrentHint'))}
+          </p>
+        </button>
+      ) : null}
       <PatientSearchCombobox
         search={search}
         onSearchChange={setSearch}

frontend/src/components/ui/treatment/TreatmentDetailsEditor.tsx

@@ -225,12 +225,12 @@ export function TreatmentDetailsEditor({
 
       {stepper && activeDetail ? <div className="pt-1">{stepper}</div> : null}
 
+      {activeDetail && showMissingTeethLabBlock ? (
+        <p className={labBlockedBannerClass}>{t('labShipmentBlockedBody')}</p>
+      ) : null}
+
       {showFields && activeDetail ? (
         <div className="space-y-3">
-          {showMissingTeethLabBlock && (
-            <p className={labBlockedBannerClass}>{t('labShipmentBlockedBody')}</p>
-          )}
-
           <div className="grid grid-cols-1 gap-3 sm:grid-cols-2 sm:items-end">
             <Dropdown
               label={t('treatmentType')}
@@ -290,6 +290,8 @@ export function TreatmentDetailsEditor({
 
           {footer ? <div className="pt-1">{footer}</div> : null}
         </div>
+      ) : showFields && !activeDetail ? (
+        <p className="text-sm text-text-muted">{t('noDetails')}</p>
       ) : null}
     </div>
   );
@@ -336,17 +338,12 @@ function NotesField({
 /**
  * "Add detail", split into two segments with the microphone at the logical end.
  *
- * Built like the detail chip's trash affordance in this same file — an
- * `inline-flex items-stretch overflow-hidden rounded` wrapper holding two raw `<button>`s
- * divided by `border-s` — rather than two shared `Button`s, which each hardcode their own
- * rounding and would fight a segmented control.
+ * Built like the detail chip's trash affordance in this same file — a wrapper holding two
+ * raw `<button>`s divided by `border-s` — rather than two shared `Button`s, which hardcode
+ * their own rounding and would fight a segmented control. `border-s` puts the microphone
+ * visually right in en/nl and left in fa, on the same side as the chip's trash in both.
  *
- * `border-s` puts the microphone at the *logical* end: visually right in en/nl, visually
- * left in fa, on the same side as the chip's trash in both directions.
- *
- * The two halves share a wrapper and nothing else. Add keeps its exact existing
- * behaviour; the microphone is an independent action that creates nothing until the
- * clinician confirms.
+ * Add keeps its exact existing behaviour; the microphone is an independent action.
  */
 function AddDetailWithVoice({
   addLabel,

frontend/src/components/ui/treatment/TreatmentWorkspace.tsx

@@ -15,7 +15,6 @@ import { AppointmentsStrip } from '@/components/ui/treatment/AppointmentsStrip';
 import { NewTreatmentPatientPicker } from '@/components/ui/treatment/NewTreatmentPatientPicker';
 import { FdiToothChart } from '@/components/ui/treatment/FdiToothChart';
 import { LabCasesDispatchPanel } from '@/components/ui/treatment/LabCasesDispatchPanel';
-import { LabShipmentBlockedNotice } from '@/components/ui/treatment/LabShipmentBlockedNotice';
 import { LabDispatchAttentionPanel } from '@/components/ui/treatment/LabDispatchAttentionPanel';
 import { PastTreatmentsPanel } from '@/components/ui/treatment/PastTreatmentsPanel';
 import { TreatmentDetailsEditor } from '@/components/ui/treatment/TreatmentDetailsEditor';
@@ -41,7 +40,7 @@ import type {
 import { treatmentsApi } from '@/lib/api/treatments';
 import { notificationsApi } from '@/lib/api/notifications';
 import { pickAutoAppointment } from '@/components/shared/treatmentSelection';
-import type { DayStripItem } from '@/components/treatment/dayStrip';
+import { unscheduledStripColorCode, type DayStripItem } from '@/components/treatment/dayStrip';
 import {
   areDetailsPersistable,
   defaultTreatmentTypeForAppointment,
@@ -65,7 +64,11 @@ import {
 } from '@/components/treatment/toothSelectionGroups';
 import type { LabDispatchAttentionItem } from '@/components/treatment/labDispatchAttention';
 import { collectLabDispatchAttention } from '@/components/treatment/labDispatchAttention';
-import { loadLabDispatchDefaults, rememberLastLab } from '@/components/treatment/labDispatchDefaults';
+import {
+  loadRecentLabIds,
+  MAX_RECENT_LABS,
+  rememberRecentLab,
+} from '@/components/treatment/labDispatchDefaults';
 import { canEditTreatment, canViewTreatment, canAccessDashboardRoute } from '@/components/shared/permissions';
 import { scrollWithinMainScrollContainer } from '@/components/shared/scrollWithinMain';
 import { useMarkTabReadOnVisit, useTabBadgeCounts } from '@/lib/hooks/useTabBadgeCounts';
@@ -211,6 +214,7 @@ function mapAppointment(record: AppointmentRecord): TreatmentAppointment {
     patientId: record.patientId,
     patientFirstName: record.patient.firstName,
     patientLastName: record.patient.lastName,
+    patientMobile: record.patient.mobile,
     providerUserId: record.providerUserId,
     startAt: record.startAt,
     endAt: record.endAt,
@@ -391,10 +395,12 @@ export function TreatmentWorkspace({
   const [unreadLabCasesLoading, setUnreadLabCasesLoading] = useState(false);
   const [labCasesScope, setLabCasesScope] = useState<TreatmentLabCasesScope>('patient');
   const [selectedRailLabCaseId, setSelectedRailLabCaseId] = useState<string | null>(null);
-  const [searchedPatient, setSearchedPatient] = useState<Pick<
-    Patient,
-    'id' | 'firstName' | 'lastName'
-  > | null>(null);
+  const [searchedPatient, setSearchedPatient] = useState<
+    (Pick<Patient, 'id' | 'firstName' | 'lastName'> & {
+      mobile?: string | null;
+      email?: string | null;
+    }) | null
+  >(null);
   const [patientSearchBusy, setPatientSearchBusy] = useState(false);
   const [newTreatmentPickerOpen, setNewTreatmentPickerOpen] = useState(false);
   const [creatingStandalone, setCreatingStandalone] = useState(false);
@@ -527,6 +533,8 @@ export function TreatmentWorkspace({
         id: selectedAppointment.patientId,
         firstName: selectedAppointment.patientFirstName,
         lastName: selectedAppointment.patientLastName,
+        mobile: selectedAppointment.patientMobile ?? null,
+        email: null,
         purpose: selectedAppointment.purpose,
         isWalkIn: false,
       };
@@ -537,6 +545,8 @@ export function TreatmentWorkspace({
         id: selectedStandalone.patientId,
         firstName: isWalkIn ? walkInLabel : (selectedStandalone.patient?.firstName ?? ''),
         lastName: isWalkIn ? '' : (selectedStandalone.patient?.lastName ?? ''),
+        mobile: isWalkIn ? null : (selectedStandalone.patient?.mobile ?? null),
+        email: isWalkIn ? null : (selectedStandalone.patient?.email ?? null),
         purpose: selectedStandalone.details[0]?.treatmentType,
         isWalkIn,
       };
@@ -546,7 +556,9 @@ export function TreatmentWorkspace({
         id: searchedPatient.id,
         firstName: searchedPatient.firstName,
         lastName: searchedPatient.lastName,
-        purpose: undefined as string | undefined,
+        mobile: searchedPatient.mobile ?? null,
+        email: searchedPatient.email ?? null,
+        purpose: undefined,
         isWalkIn: false,
       };
     }
@@ -557,6 +569,23 @@ export function TreatmentWorkspace({
   const activePatientName = activePatient
     ? `${activePatient.firstName} ${activePatient.lastName}`.trim()
     : null;
+  const namedActivePatient =
+    activePatient && !activePatient.isWalkIn && activePatient.id
+      ? {
+          id: activePatient.id,
+          displayName: activePatientName ?? '',
+          mobile: activePatient.mobile ?? null,
+          email: activePatient.email ?? null,
+        }
+      : null;
+  const searchedWithoutLiveVisit = Boolean(searchedPatient) && !hasLiveContext;
+  const showSearchedPatientLoading =
+    searchedWithoutLiveVisit && (patientSearchBusy || historyLoading);
+  const showNoTreatmentFound =
+    searchedWithoutLiveVisit &&
+    !patientSearchBusy &&
+    !historyLoading &&
+    history.length === 0;
 
   const unreadUpdatesCount = unreadLabCases.length;
   const otherPatientsUnreadCount = useMemo(
@@ -628,8 +657,8 @@ export function TreatmentWorkspace({
     setStandaloneTreatments((prev) => {
       const current = prev.find((row) => row.id === selectedStandaloneId);
       if (!current) return prev;
-      const prevColor = current.details.find((d) => d.treatmentType?.trim())?.treatmentType?.trim() ?? '';
-      const nextColor = nextDetails.find((d) => d.treatmentType?.trim())?.treatmentType?.trim() ?? '';
+      const prevColor = unscheduledStripColorCode(current.details);
+      const nextColor = unscheduledStripColorCode(nextDetails);
       if (
         prevColor === nextColor &&
         areUnscheduledDetailsStripDeletable(current.details) ===
@@ -660,8 +689,7 @@ export function TreatmentWorkspace({
       const sourceDetails = tr.id === selectedStandaloneId && !draftHydratingRef.current
         ? details
         : tr.details;
-      const typedDetail = sourceDetails.find((d) => Boolean(d.treatmentType?.trim()));
-      const colorCode = typedDetail?.treatmentType?.trim() ?? '';
+      const colorCode = unscheduledStripColorCode(sourceDetails);
       return {
         kind: 'unscheduled' as const,
         id: tr.id,
@@ -763,7 +791,7 @@ export function TreatmentWorkspace({
       mapped.length > 0
         ? mapped
         : options?.seedBlankIfEmpty
-          ? [newDetail(defaultTreatmentTypeForAppointment(undefined, treatmentCatalog))]
+          ? [newDetail()]
           : [];
     setDetails(nextDetails);
     setActiveDetailId((prev) => {
@@ -778,7 +806,7 @@ export function TreatmentWorkspace({
     setActiveLabCaseId(mappedLabCases[0]?.clientId ?? null);
     setOrganizationSearch('');
     setSaveStatus('idle');
-  }, [treatmentCatalog]);
+  }, []);
 
   const selectedTeethSet = useMemo(() => new Set(activeDetail?.teeth ?? []), [activeDetail?.teeth]);
   const connectedSelectedTeeth = useMemo(
@@ -831,6 +859,7 @@ export function TreatmentWorkspace({
   useEffect(() => {
     setShowWholeTreatmentPlan(false);
     rangeAnchorRef.current = null;
+    setOrganizationSearch('');
     const pending = pendingEntryStepRef.current;
     pendingEntryStepRef.current = null;
     setEntryStep(pending ?? 'treatment');
@@ -944,11 +973,11 @@ export function TreatmentWorkspace({
         setLabDependentCodes(
           new Set(catalogResponse.data.filter((entry) => entry.labDependent).map((entry) => entry.code)),
         );
-        const lastLabId = loadLabDispatchDefaults(currentOrganization?.id).lastLabId;
-        if (lastLabId && orgsResponse.data.some((o) => o.id === lastLabId && o.active)) {
-          setRecentOrganizationIds((prev) =>
-            prev.includes(lastLabId) ? prev : [lastLabId, ...prev].slice(0, 10),
+        const recentIds = loadRecentLabIds(currentOrganization?.id).filter((id) =>
+          orgsResponse.data.some((o) => o.id === id && o.active),
         );
+        if (recentIds.length > 0) {
+          setRecentOrganizationIds(recentIds);
         }
       } catch (error: unknown) {
         if (!cancelled) {
@@ -1583,24 +1612,51 @@ export function TreatmentWorkspace({
           return;
         }
 
+        const ok = await flushDraftSave();
+        if (!ok) return;
+
         setPatientSearchBusy(true);
-        setSearchedPatient({
-          id: patient.id,
-          firstName: patient.firstName,
-          lastName: patient.lastName,
-        });
         try {
-          const response = await treatmentsApi.listPatientHistory(patient.id, 1);
-          const latest = response.data[0];
-          if (latest) {
-            const ok = await loadTreatmentIntoWorkspace(latest);
-            if (!ok) {
+          const stripAppointment = appointments.find((row) => row.patientId === patient.id);
+          const stripStandalone = standaloneTreatments.find(
+            (row) => row.patientId === patient.id && !row.patient?.isWalkIn,
+          );
+          if (stripAppointment || stripStandalone) {
+            draftHydratingRef.current = true;
+            resetToLiveContext();
             setSearchedPatient(null);
+            setNewTreatmentPickerOpen(false);
+            setSelectionLocked(true);
+            if (stripAppointment) {
+              setSelectedAppointmentId(stripAppointment.id);
+              setSelectedStandaloneId(null);
+            } else if (stripStandalone) {
+              setSelectedAppointmentId(null);
+              setSelectedStandaloneId(stripStandalone.id);
             }
             return;
           }
 
-          await refreshHistory(patient.id);
+          setSearchedPatient({
+            id: patient.id,
+            firstName: patient.firstName,
+            lastName: patient.lastName,
+            mobile: patient.mobile,
+            email: patient.email,
+          });
+          resetToLiveContext();
+          setNewTreatmentPickerOpen(false);
+          setSelectedAppointmentId(null);
+          setSelectedStandaloneId(null);
+          setSelectionLocked(true);
+          setHistory([]);
+          setHistoryLoading(true);
+
+          const response = await treatmentsApi.listPatientHistory(patient.id, 1);
+          const latest = response.data[0];
+          if (latest) {
+            await loadTreatmentIntoWorkspace(latest);
+          }
         } catch (error: unknown) {
           showError(getUserFacingError(error, tErrors, t('errorLoadHistory')));
           setSearchedPatient(null);
@@ -1614,8 +1670,11 @@ export function TreatmentWorkspace({
       selectedStandalone?.patientId,
       searchedPatient?.id,
       hasLiveContext,
+      flushDraftSave,
+      appointments,
+      standaloneTreatments,
+      resetToLiveContext,
       loadTreatmentIntoWorkspace,
-      refreshHistory,
       showError,
       t,
       tErrors,
@@ -1941,8 +2000,7 @@ export function TreatmentWorkspace({
 
       const nextDetails = [...detailsRef.current, detail];
       setDetails(nextDetails);
-      // persistDraft reads detailsRef, and setDetails has not rendered yet. The codebase
-      // already writes this ref imperatively after a save for the same reason.
+      // persistDraft reads detailsRef, and setDetails has not rendered yet.
       detailsRef.current = nextDetails;
       setActiveDetailId(detail.clientId);
       setEntryStep('treatment');
@@ -1986,13 +2044,18 @@ export function TreatmentWorkspace({
         const updatedLabCases = [...labCaseDrafts, draft];
         setLabCaseDrafts(updatedLabCases);
 
-        // Every other path that creates a lab draft persists it immediately, and the
-        // autosave effect only watches `details`. Left in state alone, the destination
-        // lab, the due date and the whole prosthesis map vanish on the next reload —
-        // silently, because the detail itself does survive.
+        // Autosave only watches `details`, so a lab draft left in state alone loses the
+        // lab, the due date and the prosthesis map on reload — silently, because the
+        // detail itself survives.
         void (async () => {
           try {
             const saved = await persistDraft({ force: true });
+            // persistDraft returns a *preview* when the details are not persistable — one
+            // blank detail is enough — and a preview's detail id falls back to the client
+            // id. Check what came back, not the precondition, so this holds for every early
+            // return persistDraft has.
+            const savedDetail = saved.details.find((d) => d.clientId === detail.clientId);
+            if (!savedDetail?.id || savedDetail.id === detail.clientId) return;
             await persistLabCases(saved, updatedLabCases);
           } catch (error: unknown) {
             showError(getUserFacingError(error, tErrors, t('errorSaveLabShipments')));
@@ -2021,7 +2084,9 @@ export function TreatmentWorkspace({
       if (idx < 0) return;
       const target = details[idx];
       if (!target || isDetailLocked(target)) return;
-      if (!window.confirm(t('confirmRemoveDetail'))) return;
+      if (!window.confirm(
+        details.length <= 1 ? t('confirmRemoveLastDetail') : t('confirmRemoveDetail'),
+      )) return;
 
       const nextDetails = details.filter((d) => d.clientId !== detailClientId);
       const nextActive =
@@ -2097,14 +2162,9 @@ export function TreatmentWorkspace({
       return;
     }
 
-    const lastLabId = loadLabDispatchDefaults(currentOrganization?.id).lastLabId;
-    const lastLabStillActive = lastLabId
-      ? orgs.some((o) => o.id === lastLabId && o.active)
-      : false;
     const next: LabCaseDraft = {
       ...newLabCaseDraft(),
       detailClientId: shouldIncludeActive ? activeDetailId : null,
-      destinationOrganizationId: lastLabStillActive ? lastLabId! : null,
       attachmentIds: activeDetail?.attachmentMetas.map((a) => a.id) ?? [],
     };
     const updatedLabCases = [...cleaned, next];
@@ -2120,11 +2180,9 @@ export function TreatmentWorkspace({
   }, [
     activeDetailId,
     canEditTreatmentForDay,
-    currentOrganization?.id,
     details,
     labCaseDrafts,
     labDependentCodes,
-    orgs,
     persistDraft,
     persistLabCases,
     selectedAppointment,
@@ -2244,8 +2302,8 @@ export function TreatmentWorkspace({
 
         setRecentOrganizationIds((prev) => {
           const orgId = labCase.destinationOrganizationId!;
-          rememberLastLab(currentOrganization?.id, orgId);
-          return [orgId, ...prev.filter((id) => id !== orgId)].slice(0, 10);
+          rememberRecentLab(currentOrganization?.id, orgId);
+          return [orgId, ...prev.filter((id) => id !== orgId)].slice(0, MAX_RECENT_LABS);
         });
         showSuccess(t('successCaseSent'));
         notifyTabBadgesChanged();
@@ -2286,11 +2344,24 @@ export function TreatmentWorkspace({
 
   return (
     <div className="space-y-4">
-      <header className="space-y-1">
+      <header className="flex flex-col gap-3 sm:flex-row sm:items-center sm:gap-4">
+        <div className="min-w-0 shrink-0">
           <h1 className="text-xl sm:text-2xl font-semibold text-text-primary">{t('title')}</h1>
           {!canEdit ? (
-          <p className="text-sm text-text-secondary">{t('subtitleReadOnly')}</p>
+            <p className="text-sm text-text-secondary mt-1">{t('subtitleReadOnly')}</p>
           ) : null}
+        </div>
+        <div className="min-w-0 flex-1">
+          <PatientSearchCombobox
+            search={patientSearch}
+            onSearchChange={setPatientSearch}
+            patients={patientSearchResults}
+            loading={patientSearchLoading || patientSearchBusy}
+            onSelectPatient={handleSelectSearchedPatient}
+            placeholder={tPatients('searchPlaceholder')}
+            emptyResultsMessage={tPatients('noResults')}
+          />
+        </div>
       </header>
 
       <AppointmentsStrip
@@ -2321,15 +2392,6 @@ export function TreatmentWorkspace({
       <div className="treatment-layout-grid grid grid-cols-1 xl:grid-cols-[minmax(300px,380px)_minmax(0,1fr)] gap-4 items-start">
         <div className="space-y-3 min-w-0 xl:max-w-[380px]">
           <div className="surface-card p-3 space-y-3">
-            <PatientSearchCombobox
-              search={patientSearch}
-              onSearchChange={setPatientSearch}
-              patients={patientSearchResults}
-              loading={patientSearchLoading || patientSearchBusy}
-              onSelectPatient={handleSelectSearchedPatient}
-              placeholder={tPatients('searchPlaceholder')}
-              emptyResultsMessage={tPatients('noResults')}
-            />
             {canEdit && !isViewingPastDay ? (
               <div className="space-y-2">
                 <Button
@@ -2344,10 +2406,16 @@ export function TreatmentWorkspace({
                 {newTreatmentPickerOpen ? (
                   <NewTreatmentPatientPicker
                     creating={creatingStandalone}
+                    currentPatient={namedActivePatient}
                     onSelectWalkIn={() => createStandaloneTreatment({ walkIn: true })}
                     onSelectPatient={(patient) =>
                       createStandaloneTreatment({ patientId: patient.id })
                     }
+                    onSelectCurrentPatient={
+                      namedActivePatient
+                        ? () => createStandaloneTreatment({ patientId: namedActivePatient.id })
+                        : undefined
+                    }
                     onCancel={() => setNewTreatmentPickerOpen(false)}
                   />
                 ) : null}
@@ -2355,8 +2423,14 @@ export function TreatmentWorkspace({
             ) : null}
 
             {activePatient ? (
-              <div className="space-y-0.5 border-t border-border/60 pt-3">
-                <p className="text-[10px] uppercase tracking-wide text-text-muted">{t('selectedPatient')}</p>
+              <div
+                className={`space-y-0.5 ${
+                  canEdit && !isViewingPastDay ? 'border-t border-border/60 pt-3' : ''
+                }`}
+              >
+                <p className="text-[10px] uppercase tracking-wide text-text-muted">
+                  {t('selectedPatient')}
+                </p>
                 <p className="text-base font-semibold text-text-primary">{activePatientName}</p>
                 {activePatient.purpose ? (
                   <p className="text-[11px] text-text-secondary">
@@ -2368,7 +2442,11 @@ export function TreatmentWorkspace({
                 ) : null}
               </div>
             ) : (
-              <p className="text-sm text-text-muted border-t border-border/60 pt-3">
+              <p
+                className={`text-sm text-text-muted ${
+                  canEdit && !isViewingPastDay ? 'border-t border-border/60 pt-3' : ''
+                }`}
+              >
                 {apptsLoading ? t('loadingAppointments') : t('selectDayWithAppointment')}
               </p>
             )}
@@ -2471,6 +2549,24 @@ export function TreatmentWorkspace({
         </div>
 
         <div className="space-y-3 min-w-0 w-full">
+          {showSearchedPatientLoading ? (
+            <div className="surface-card p-6">
+              <p className="text-sm text-text-muted">{t('loading')}</p>
+            </div>
+          ) : showNoTreatmentFound ? (
+            <div className="surface-card w-full p-6 space-y-3">
+              <h2 className="text-lg font-semibold text-text-primary">
+                {t('noTreatmentFoundTitle')}
+              </h2>
+              <p className="text-sm text-text-secondary">
+                {t('noTreatmentFoundBody', {
+                  name: activePatientName ?? '',
+                  action: t('newTreatment'),
+                })}
+              </p>
+            </div>
+          ) : (
+            <>
           <TreatmentDetailsEditor
             details={details}
             activeDetailId={activeDetailId}
@@ -2484,9 +2580,14 @@ export function TreatmentWorkspace({
             saveStatus={saveStatus}
             uploadBusy={uploadBusyDetailId === activeDetailId}
             onAddDetail={() => {
-              const next = newDetail(
-                defaultTreatmentTypeForAppointment(selectedAppointment?.purpose, treatmentCatalog),
-              );
+              const seedFromAppointment =
+                details.length === 0 && selectedAppointment
+                  ? defaultTreatmentTypeForAppointment(
+                      selectedAppointment.purpose,
+                      treatmentCatalog,
+                    )
+                  : undefined;
+              const next = newDetail(seedFromAppointment);
               setDetails((prev) => [...prev, next]);
               setActiveDetailId(next.clientId);
               setEntryStep('treatment');
@@ -2726,7 +2827,6 @@ export function TreatmentWorkspace({
 
           {entryStep === 'lab' ? (
             <div ref={labPanelRef} className="space-y-3">
-              {showLabShipmentBlocked ? <LabShipmentBlockedNotice /> : null}
               {showLabDispatchPanel ? (
                 <LabCasesDispatchPanel
                   details={details}
@@ -2734,7 +2834,6 @@ export function TreatmentWorkspace({
                   labCases={labCaseDrafts}
                   labDependentCodes={labDependentCodes}
                   treatmentCatalog={treatmentCatalog}
-                  clinicOrganizationId={currentOrganization?.id}
                   labCaseSummary={activeLabCaseSummary}
                   locale={locale}
                   onLabCaseSummaryChange={handleLabCaseSummaryChange}
@@ -2752,33 +2851,21 @@ export function TreatmentWorkspace({
                   organizationSearch={organizationSearch}
                   onOrganizationSearchChange={setOrganizationSearch}
                   recentOrganizationIds={recentOrganizationIds}
-                  onRecentOrganizationPick={(orgId) => {
-                    setLabCaseDrafts((prev) => {
-                      const targetId =
-                        activeLabCaseId ??
-                        prev.find((lc) => !lc.sentAt && lc.detailClientId === activeDetailId)
-                          ?.clientId;
-                      if (!targetId) return prev;
-                      return prev.map((lc) =>
-                        lc.clientId === targetId && !lc.sentAt
-                          ? { ...lc, destinationOrganizationId: orgId }
-                          : lc,
-                      );
-                    });
-                  }}
                   sendBusyId={sendBusyId}
                   onSendLabCase={(lc, comment) => handleSendLabCase(lc, comment)}
                   onCommentError={showError}
                   canInviteLab={canAccessOrganizations}
                   onInviteLab={() => router.push('/organizations?action=invite-lab')}
                 />
-              ) : (
+              ) : showLabShipmentBlocked ? null : (
                 <p className="text-sm text-text-muted surface-card p-4">
                   {t('entryStepLabUnavailable')}
                 </p>
               )}
             </div>
           ) : null}
+            </>
+          )}
         </div>
       </div>
       {voiceResult ? (

frontend/src/components/ui/treatment/VoiceRecordingBar.tsx

@@ -16,9 +16,9 @@ function formatElapsed(ms: number): string {
 /**
  * Live recording / processing strip.
  *
- * Sits between the header row and the chip strip rather than inside the segmented
- * control: the header is `sm:justify-between`, so growing the button mid-recording would
- * shove the row on every start and every stop.
+ * Sits between the header row and the chip strip rather than inside the segmented control:
+ * the header is `sm:justify-between`, so growing the button mid-recording would shift the
+ * whole row.
  */
 export function VoiceRecordingBar({ voice }: { voice: VoiceCaptureState }) {
   const t = useTranslations('treatment');

frontend/src/components/ui/treatment/VoiceReviewSheet.tsx

@@ -38,9 +38,8 @@ interface VoiceReviewSheetProps {
 /**
  * Confirmation step between the model's output and the form.
  *
- * Modal on desktop, bottom sheet on mobile via ResponsiveDialog — deliberately an overlay
- * and not a route, because navigating would unmount TreatmentWorkspace and destroy the
- * in-progress draft.
+ * Modal on desktop, bottom sheet on mobile — an overlay and not a route, because navigating
+ * would unmount TreatmentWorkspace and destroy the in-progress draft.
  */
 export function VoiceReviewSheet({
   result,
@@ -78,10 +77,9 @@ export function VoiceReviewSheet({
       // The teeth row starts unticked whenever the recording produced no teeth of its own,
       // and a picked tooth that is not ticked applies nothing.
       teeth: true,
-      // The picked tooth has no prosthesis type, which makes the map unshippable. Leaving
-      // the row ticked would apply a map that `assertCompleteToothProsthesisMap` rejects
-      // at dispatch — the exact failure the never-auto-tick-incomplete rule exists to
-      // prevent. Only ever unticks: re-ticking is the clinician's call.
+      // A picked tooth has no prosthesis type, so the map is no longer shippable — leaving the
+      // row ticked would apply a map dispatch rejects. Only ever unticks; re-ticking is the
+      // clinician's call.
       prosthesis:
         prev.prosthesis &&
         withChosenTeeth(result, nextChosen).prosthesis?.complete !== false,
@@ -290,9 +288,8 @@ function Row({
 }
 
 /**
- * A bare `YYYY-MM-DD` is a *civil* date, but `new Date('2025-10-17')` parses it as UTC
- * midnight — which renders as the 16th for any viewer west of Greenwich. Build the date
- * from its parts so it means the same day everywhere.
+ * `new Date('2025-10-17')` parses a civil date as UTC midnight, which renders as the 16th
+ * west of Greenwich. Build it from its parts so it means the same day everywhere.
  */
 function civilDateToLocalDate(iso: string): Date {
   const [year, month, day] = iso.split('-').map(Number);

frontend/src/lib/api/voice.ts

@@ -22,8 +22,7 @@ export const voiceApi = {
     payload: ExtractVoicePayload,
     signal?: AbortSignal,
   ): Promise<{ success: boolean; data: VoiceExtractionResult }> => {
-    // The signal is forwarded so cancelling closes the connection, which aborts the
-    // metered vendor call server-side rather than letting it settle unseen.
+    // Forwarded so cancelling closes the connection; the controller turns that into an abort.
     const response = await apiClient.post('/voice/extract', payload, { signal });
     return response.data;
   },

frontend/src/lib/voice/audioFormat.ts

@@ -9,11 +9,9 @@ const PREFERRED_MIME_TYPES = [
 ] as const;
 
 /**
- * Pick a container this browser can record AND the backend accepts.
- *
- * Chrome and Android produce webm/opus; Safari and iPad produce mp4/aac. Both go to the
- * vendor unmodified, so there is no transcode step — but the choice still has to be made
- * at record time, and `isTypeSupported` is missing entirely on older Safari.
+ * Pick a container this browser can record AND the backend accepts. Chrome and Android give
+ * webm/opus, Safari and iPad mp4/aac; both go to the vendor unmodified, so there is no
+ * transcode step and the list is an intersection, not a preference.
  */
 export function pickRecordingMimeType(): string | null {
   if (typeof MediaRecorder === 'undefined') return null;

frontend/src/lib/voice/useVoiceCapture.ts

@@ -69,9 +69,8 @@ export function useVoiceCapture({
   /** getUserMedia is async; without this a permission granted after unmount leaks the mic. */
   const mountedRef = useRef(true);
   /**
-   * Set synchronously on click. `phase` does not become 'recording' until getUserMedia
-   * resolves, so without this a second click during the permission prompt would start a
-   * second stream and orphan the first — mic indicator lit, interval leaked.
+   * Set synchronously on click: `phase` only becomes 'recording' once getUserMedia resolves,
+   * so a second click during the permission prompt would orphan the first stream.
    */
   const startingRef = useRef(false);
 
@@ -139,8 +138,14 @@ export function useVoiceCapture({
   );
 
   const stop = useCallback(() => {
+    // No recorder means nothing will fire `onstop`, so nothing else moves the phase.
+    if (!recorderRef.current) {
+      teardown();
+      setPhase('idle');
+      return;
+    }
     try {
-      recorderRef.current?.stop();
+      recorderRef.current.stop();
     } catch {
       teardown();
       setPhase('idle');
@@ -150,7 +155,10 @@ export function useVoiceCapture({
   const onStart = useCallback(() => {
     if (phase !== 'idle' || startingRef.current) return;
     if (!isMediaRecorderSupported()) {
-      onError(clientError('VOICE_MIC_DENIED'));
+      // VOICE_UNSUPPORTED_FORMAT, not MIC_DENIED: nothing asked for a permission yet, and
+      // blaming the microphone sends the clinician into site settings for no reason. Same
+      // for the two paths below.
+      onError(clientError('VOICE_UNSUPPORTED_FORMAT'));
       return;
     }
 
@@ -178,7 +186,7 @@ export function useVoiceCapture({
         const mimeType = pickRecordingMimeType();
         if (mimeType === null) {
           stream.getTracks().forEach((track) => track.stop());
-          onError(clientError('VOICE_MIC_DENIED'));
+          onError(clientError('VOICE_UNSUPPORTED_FORMAT'));
           return;
         }
 
@@ -220,13 +228,11 @@ export function useVoiceCapture({
           if (maxMs != null && elapsed >= maxMs) stop();
         }, LEVEL_POLL_MS);
       } catch {
-        // `new MediaRecorder(...)` and `recorder.start()` both throw on some browsers,
-        // and by then the stream is already live. Without this the promise rejects
-        // unhandled, the UI sits at 'idle' with nothing shown, and the browser's
-        // recording indicator stays lit until the workspace unmounts.
+        // `new MediaRecorder()` and `recorder.start()` both throw on some browsers, and by
+        // then the stream is live — without this the mic indicator stays lit until unmount.
         teardown();
         setPhase('idle');
-        onError(clientError('VOICE_MIC_DENIED'));
+        onError(clientError('VOICE_UNSUPPORTED_FORMAT'));
       } finally {
         startingRef.current = false;
       }
@@ -235,8 +241,7 @@ export function useVoiceCapture({
 
   const onCancel = useCallback(() => {
     cancelledRef.current = true;
-    // Aborting closes the connection, which aborts the vendor call server-side. It is
-    // metered per minute, so letting it settle costs money for a result nobody sees.
+    // Aborting closes the connection, which aborts the vendor call server-side.
     abortRef.current?.abort();
     try {
       recorderRef.current?.stop();
@@ -270,10 +275,9 @@ function attachLevelMeter(
     source.connect(analyser);
 
     const data = new Uint8Array(analyser.frequencyBinCount);
-    // Sample every frame so a transient is not missed, but publish at LEVEL_POLL_MS.
-    // This hook lives in TreatmentWorkspace, so an unthrottled setLevel re-renders the
-    // details editor, the FDI chart and the lab panel on every animation frame — about
-    // 7,200 whole-tree renders across a two-minute recording.
+    // Sample every frame so a transient is not missed, publish at LEVEL_POLL_MS. The hook
+    // lives in TreatmentWorkspace, so an unthrottled setLevel is ~7,200 whole-tree renders
+    // across a two-minute recording.
     let peakSinceEmit = 0;
     let lastEmit = 0;
     const tick = (now: number) => {

frontend/src/types/treatment.ts

@@ -38,6 +38,7 @@ export interface TreatmentAppointment {
   patientId: string;
   patientFirstName: string;
   patientLastName: string;
+  patientMobile?: string | null;
   providerUserId: string;
   startAt: string;
   endAt: string;
@@ -139,6 +140,8 @@ export interface PastTreatment {
     firstName: string;
     lastName: string;
     isWalkIn: boolean;
+    mobile?: string | null;
+    email?: string | null;
   } | null;
   details: PastTreatmentDetail[];
   labCases: PastLabCase[];