fix(status): render pipeline phase for legacy statuses (analyst_verified) #371

Merged
chaim merged 1 commits from worktree-status-legacy-phase-fix into main 2026-06-30 18:31:57 +00:00
Owner

למה

תיק על סטטוס שצומצם מ-PHASES אבל הסוכנים עדיין מציבים (analyst_verified מ-HEARTBEAT, research_complete מהחוקר) הוצג בלי שום שלב פעילphaseIndexOf החזיר ‎-1, ה-stepper התרוקן לגמרי, והיו"ר לא ראה איפה התיק עומד. דווח על 1043-02-26 (כעת analyst_verified).

מה

  • LEGACY_STATUS_PHASE ב-case-status.ts — ממפה כל סטטוס legacy/ביניים לשלב-התצוגה שלו (analyst_verified/research_complete → "ניתוח וכיוון").
  • phaseIndexOf() יחיד מיוצא מה-SSoT; status-hero.tsx ו-workflow-timeline.tsx הסירו את ה-phaseIndexOf המקומי הזהה (שנשא את הבאג) ומייבאים את המשותף.

גבולות

  • תיקון-תצוגה בלבד (מיפוי-נתון) — עיצוב ה-stepper ללא שינוי; בתוך חריג שער-העיצוב.
  • לא מטפל בשורש (הסוכנים עדיין מציבים analyst_verified) — יישור-הפרומפטים הוא follow-up נפרד שמצריך sync חוצה-חברות; ייעשה בזהירות.

Invariants

  • G2 — מסלול-פתרון-שלב יחיד; legacy מטופל ב-SSoT, בלי לוגיקה מקבילה חדשה.
  • ללא שינוי backend / מודל-סטטוס.

בדיקות

tsc --noEmit (build מלא ב-CI).

🤖 Generated with Claude Code

## למה תיק על סטטוס שצומצם מ-`PHASES` אבל הסוכנים עדיין מציבים (`analyst_verified` מ-HEARTBEAT, `research_complete` מהחוקר) **הוצג בלי שום שלב פעיל** — `phaseIndexOf` החזיר ‎-1, ה-stepper התרוקן לגמרי, והיו"ר לא ראה איפה התיק עומד. דווח על **1043-02-26** (כעת `analyst_verified`). ## מה - `LEGACY_STATUS_PHASE` ב-[`case-status.ts`](web-ui/src/lib/api/case-status.ts) — ממפה כל סטטוס legacy/ביניים לשלב-התצוגה שלו (`analyst_verified`/`research_complete` → "ניתוח וכיוון"). - `phaseIndexOf()` יחיד מיוצא מה-SSoT; [`status-hero.tsx`](web-ui/src/components/cases/status-hero.tsx) ו-[`workflow-timeline.tsx`](web-ui/src/components/cases/workflow-timeline.tsx) הסירו את ה-`phaseIndexOf` המקומי הזהה (שנשא את הבאג) ומייבאים את המשותף. ## גבולות - **תיקון-תצוגה בלבד** (מיפוי-נתון) — עיצוב ה-stepper ללא שינוי; בתוך חריג שער-העיצוב. - **לא** מטפל בשורש (הסוכנים עדיין מציבים `analyst_verified`) — יישור-הפרומפטים הוא follow-up נפרד שמצריך sync חוצה-חברות; ייעשה בזהירות. ## Invariants - **G2** — מסלול-פתרון-שלב יחיד; legacy מטופל ב-SSoT, בלי לוגיקה מקבילה חדשה. - ללא שינוי backend / מודל-סטטוס. ## בדיקות `tsc --noEmit` ✅ (build מלא ב-CI). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
chaim added 1 commit 2026-06-30 18:31:40 +00:00
fix(status): render pipeline phase for legacy statuses (analyst_verified)
All checks were successful
G12 Leak-Guard / leak-guard (pull_request) Successful in 4s
Lint — undefined names / undefined-names (pull_request) Successful in 10s
7c48c8753e
A case parked on a status the trimmed PHASES list dropped — but the agents
still set, e.g. `analyst_verified` (HEARTBEAT.md), `research_complete`
(legal-researcher) — rendered with NO active phase: phaseIndexOf returned -1,
so the pipeline stepper went fully blank and the chair couldn't see where the
case stood (reported on 1043-02-26, currently at analyst_verified).

Add LEGACY_STATUS_PHASE in case-status.ts mapping each legacy/intermediate
status to its display phase (analyst_verified/research_complete → "ניתוח וכיוון"),
and a single phaseIndexOf() exported from the SSoT. status-hero.tsx and
workflow-timeline.tsx both dropped their identical local phaseIndexOf (which had
the bug) and import the shared one — no parallel path (G2).

Display-only data-mapping fix; the stepper design is unchanged. Does NOT address
the root drift (agents still set analyst_verified) — that prompt-alignment is a
separate, carefully-sequenced follow-up needing cross-company agent sync.

Invariants: G2 (single phase-resolution path, legacy handled in the SSoT, no
new parallel logic). No backend / status-model change.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
chaim merged commit b79c7bf161 into main 2026-06-30 18:31:57 +00:00
chaim deleted branch worktree-status-legacy-phase-fix 2026-06-30 18:31:57 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ezer-mishpati/legal-ai#371