<!-- /autoplan restore point: /home/claude/.gstack/projects/romfast-rar-autopass/main-autoplan-restore-20260707-194818.md -->
# PRD 5.23 — Editor prestatii unificat in modalul "Editare rand" (un select, un rand de chips)

**Stare**: inchis (2026-07-08; executie echipa agenti Sonnet TDD, VERIFY context curat PASS,
code-review CLOSE = 1 blocker de siguranta prins + reparat. Vezi Anexa /autoplan, GSTACK REVIEW
REPORT, Raport VERIFY; mockup final aprobat: select cu placeholder-tinta)

> Proces: `docs/ROADMAP.md` §5. Contract RAR: `docs/api-rar-contract.md`.
> Context vizual: screenshot `2026-07-07-003.jpg` (modal "Editare rand 4", badge "Cod RAR lipsa").

## 1. Obiectiv

Modalul "Editare rand" (import preview + corectie Trimiteri) are azi TREI controale pentru aceeasi
sarcina — asocierea codurilor RAR la prestatii: (1) rand per operatie cu select "— alege cod RAR —"
+ buton "Adauga", (2) container separat de chips pentru coduri libere (ex. `OE-1 ×`), (3) select
"+ Adauga alta operatie / cod RAR" cu buton "+". Utilizatorul nu stie care control face ce, iar
chips-urile apar in containere diferite, pe randuri diferite.

Livrabila: **un singur control** — un rand unic de chips (toate codurile, legate de operatie sau
libere) + **un singur select** care adauga chip instant la selectie (fara butoane "Adauga"/"+").
Selectul primeste sugestii fuzzy/k-NN pentru operatia nemapata si optiunea "Nu se declara la RAR".

## 2. Non-Goals (anti scope-creep)

- NU se schimba panoul de mapare (`_mapari.html`, `_preview_import.html`). EXCEPTIE confirmata in
  review (eng, 2026-07-07): `resolve_prestatii` primeste O SINGURA treapta noua de precedenta —
  `exclus=1` explicit pe item (din override/corectie) supravietuieste re-rezolvarii (azi
  `it.pop("exclus", None)` la `mapping.py:298` il sterge, ceea ce face US-004 imposibil altfel).
  Restul precedentei ramane neatins.
- NU se schimba contractul API `POST /v1/prezentari`. `override_json["prestatii"]` castiga campul
  `exclus` pe item; itemii exclusi RAMAN in `payload_json` (adnotati), nu mai sunt eliminati la
  salvare — `split_prestatii_excluse` ii scoate doar din payload-ul trimis la RAR.
- NU se schimba validarea codului fata de nomenclator la salvare (invariant ORA-12899) — ramane.
- NU se adauga cautare/autocomplete text in select (ramane `<select>` nativ; imbunatatire separata).
- NU se atinge butonul "salveaza ca regula" de la Trimiteri — ramane, doar mutat pe chip-ul nou.

## 3. Comportamentul controlului unificat (spec)

```
Prestatii — coduri RAR
┌───────────────────────────────────────────────────────────────┐
│ [⚠ OP-ITP · fara cod]  [OE-1 ×]        [— alege cod RAR — ▾] │
└───────────────────────────────────────────────────────────────┘
```

- **Un singur container** `.chips` cu toate elementele: chips-uri cod (× sterge), chips-uri warning
  pentru operatii nemapate, si selectul unic la capat. Wrap natural doar la lipsa de spatiu —
  niciodata containere/chenare separate pe zone.
- **Chip cod legat de operatie**: eticheta `COD`, titlu/tooltip cu denumirea operatiei sursa
  (ex. `OE-8 — Inlocuire anvelope`). `×` scoate codul; daca operatia ramane fara cod, reapare
  chip-ul warning al operatiei.
- **Chip cod liber** (fara operatie sursa): identic vizual, tooltip "cod adaugat manual".
- **Chip warning** (`⚠ OP-ITP · fara cod`): reprezinta operatia din import nemapata. Click pe el
  il face **tinta activa** (evidentiat); implicit tinta = prima operatie nemapata.
- **Selectul unic**, `hx-trigger="change"` → `POST /form-chips` cu `chips_action=add`:
  - daca exista tinta activa (operatie nemapata) → codul se leaga de acea operatie
    (chip warning → chip cod); tinta trece automat la urmatoarea operatie nemapata;
  - **placeholder-ul selectului numeste tinta** (decizie gate D2, 2026-07-07): cu tinta activa →
    `Cod pentru OP-X — alege`; fara tinta → `— adauga cod RAR —`. Legatura cod↔operatie e
    explicita la momentul alegerii, fara text suplimentar sub control;
  - fara operatii nemapate → codul se adauga ca chip liber;
  - dupa adaugare selectul revine la placeholder; chip-ul nou primeste evidentierea `.chip-nou`.
- **Continutul selectului**, cand tinta e o operatie nemapata:
  - `optgroup "Sugestii"` sus: sugestia principala + fuzzy/k-NN top-3 pentru denumirea operatiei
    (aceleasi surse ca in `_preview_import.html:89-121`);
  - optiunea `"Nu se declara la RAR (exclude operatia)"` = sentinel `__NEDECLARAT__`
    (`EXCLUDE_SENTINEL`, `app/mapping.py:44`) → operatia primeste chip distinct `Nedeclarat`
    (stil mut, ×-ul readuce chip-ul warning), item anotat `exclus` in starea chips;
  - restul nomenclatorului, ca azi.
  Fara tinta (doar cod liber): fara optgroup Sugestii si fara `__NEDECLARAT__`.
- **Dedup**: acelasi `(operatie, cod)` nu se adauga de doua ori (comportament existent pastrat).
- **Fara text explicativ permanent** sub control (decizie utilizator la aprobarea mockup-ului,
  2026-07-07): starea tintei se comunica DOAR vizual (stilul chip-ului warning evidentiat);
  mesajele `chips_ok`/`chips_error` raman exclusiv ca feedback tranzitoriu la actiuni.
- Mockup aprobat: artifact "Mockup — Editor prestatii unificat (PRD 5.23)".
- **Modul plat** (corectii fara operatii din import) foloseste ACELASI control: zero chips warning,
  selectul adauga doar coduri libere. Logica `has_r_odo`/`data-has-r-odo` (reveal odometru initial
  pe R-ODO/I-ODO) si stilul `chip-warn` pe aceste coduri se pastreaza neschimbate.

### 3.1 Specificatie vizuala si de stari (din review-ul de design, 2026-07-07)

**Scara de dominanta vizuala — UN SINGUR tratament puternic simultan:**

| Element | Fundal / chenar / text | Nota |
|---|---|---|
| Chip tinta activa | fundal warn-soft, chenar SOLID 1px `--warn` + inel 2px warn-transparent | singurul element "tare" din rand |
| Chip warning (ne-tinta) | fundal warn-soft, chenar DASHED 1px `--warn` | click → devine tinta |
| Chip cod | fundal `--card2`-albastru (`--accent` 15%), text `--accent`, chenar subtil | eticheta `COD` mono |
| Chip Nedeclarat | fundal gri mut, text taiat (line-through), chenar dashed gri | NU discret pana la invizibil |
| `.chip-nou` | animatie de emfaza >=1.5s (fade), tranzitorie | nu concureaza cu tinta |
| Chip R-ODO/I-ODO | `chip-warn` existent, neschimbat | reveal odometru pastrat |

**Invariante de interactiune:**
- Ordinea chips e STABILA: operatiile in ordinea din import, codurile libere la coada; o rezolvare
  nu re-ordoneaza randul (warning → cod in acelasi slot).
- Cat exista operatii nemapate, EXISTA tinta (implicit prima nemapata); click pe tinta activa =
  no-op (nu se poate "deselecta" — adaugarile libere sunt posibile doar fara warnings).
- Click pe chip warning = comutare de tinta CLIENT-SIDE (toggle clasa + hidden `chips_target_index`),
  fara round-trip; indexul tintei pleaca la server la urmatorul add.
- Selectul are id stabil (`chips-picker`); dupa swap `outerHTML`, focusul revine pe select.
- Optiunile selectului: `COD — denumire nomenclator`; in optgroup "Sugestii" acelasi format
  (optgroup-ul e singura marca de sugestie). Eticheta chip warning: max ~18ch + ellipsis,
  text complet in `title`.

**Tabel de stari (ce VEDE utilizatorul):**

| Stare | Ce se vede |
|---|---|
| Loading (add in zbor) | selectul disabled (`hx-disabled-elt="this"`) + stil `.htmx-request` pe sectiune; dedup face inofensiv un dublu-fire |
| Eroare add (400/500/retea) | mesaj `chips_error` SUB control (pattern 5.22, persista pana la urmatoarea actiune); selectul re-activat; tinta neschimbata; `hx-on::response-error` acopera esecul de retea |
| Gol (mod plat, zero chips) | doar selectul cu placeholder "— adauga cod RAR —"; container cu min-height (modalul nu salta la primul chip) |
| Succes partial | chip-ul nou cu emfaza >=1.5s; tinta avanseaza (vezi decizia de la gate) |
| Complet (zero nemapate) | mesaj tranzitoriu "Toate operatiile au cod" (`chips_ok`); optgroup Sugestii si `__NEDECLARAT__` dispar din select |
| Nedeclarat | mesajul numeste operatia: "OP-X nu se va declara la RAR" |
| Nomenclator gol | empty state existent (pastrat) |

## 4. Stories atomice

### US-001: Stare chips extinsa cu `exclus` + actiuni add/remove unificate in backend
**Ca** dezvoltator **vreau** o singura pereche de actiuni `add`/`remove` pe starea chips (cu tinta
optionala si suport sentinel `__NEDECLARAT__`) **pentru ca** azi exista 4 actiuni suprapuse
(`add`, `add_flat`, `add_extra`, `remove_flat`) care dubleaza logica.

- **Depinde de**: —
- **Fisiere**: `app/web/routes.py` (`post_form_chips` :2402-2521, `_chips_state_from_form` :3862-3918), `tests/test_web_chips.py` (~2 fisiere)
- **Test intai (RED)**: `tests/test_web_chips.py` — `test_add_cu_tinta_leaga_de_operatie`,
  `test_add_fara_tinta_adauga_cod_liber`, `test_add_nedeclarat_marcheaza_exclus`,
  `test_remove_cod_readuce_operatia_nemapata`, `test_dedup_operatie_cod`
- **Acceptance criteria**:
  - [x] Starea chips serializata intr-UN SINGUR hidden `chips_state` = JSON versionat
        (`{"v":1,"items":[{"cod":"","op":"","den":"","exclus":0},...]}`) — decizie gate D4:
        elimina structural clasa de bug-uri de aliniere pe index (2 regresii in 2 release-uri).
        `_chips_state_from_form` parseaza JSON-ul; JSON invalid → eroare vizibila, nu stare goala.
  - [x] Compat tranzitorie la deploy: daca `chips_state` lipseste dar exista listele vechi
        (`cod_prestatie`/`chip_op_service`/`chip_denumire`), parserul cade inapoi pe ele cu
        `exclus=0` (form-uri deschise peste deploy); fallback-ul se sterge in release-ul urmator.
  - [x] `chips_state` prezent mereu (lista goala = `{"v":1,"items":[]}`): golirea intentionata a
        chips se persista la salvare (inlocuieste santinela `chips_present`).
  - [x] `chips_action=add` + `chips_target_index=i` leaga codul de operatia i; fara target →
        chip liber; cod `__NEDECLARAT__` + target → item cu `exclus=1` si fara `cod_prestatie`.
  - [x] `chips_action=remove` (index unic) acopera si fostul `remove_flat`; actiunile
        `add_flat`/`add_extra`/`chips_add_cod_{i}`/`chips_add_cod_flat` sunt eliminate.
  - [x] Tinta implicita dupa un add reusit = urmatoarea operatie nemapata (sau nici una).
  - [x] Validare tinta SEMANTICA, nu doar in-range: `chips_target_index` trebuie sa arate spre o
        operatie nemapata si ne-exclusa; altfel `chips_error` (acopera POST fabricat si index
        stale dupa un remove).
  - [x] `chip_exclus` parsat strict ("1"→1, orice altceva→0); `__NEDECLARAT__` fara tinta valida →
        `chips_error` explicit.
  - [x] Testele existente pe actiunile vechi (`tests/test_web_corectie_prestatii.py`,
        `tests/test_web_form_editare_slim.py`) sunt ACTUALIZATE in acelasi val (suita nu ramane
        rosie intre valuri).
  - [x] Emfaza chip nou pe INDEX (`chips_added_index`), nu pe valoarea codului (doua operatii pot
        avea legitim acelasi cod).
  - [x] `python3 -m pytest -q` trece.
- **Verificare E2E**: acoperit de US-005.

### US-002: Template unificat — un container, un select, fara butoane Adauga/+
**Ca** operator **vreau** un singur rand de chips si un singur select care adauga instant
**pentru ca** azi nu stiu care din cele 3 controale trebuie folosit.

- **Depinde de**: US-001
- **Fisiere**: `app/web/templates/_chips_prestatii.html` (rescriere), `app/web/templates/base.html`
  (CSS `.chips`/`.chip*` :815-843, eliminare `.op-row`), `tests/test_web_chips.py` (~3 fisiere)
- **Test intai (RED)**: `tests/test_web_chips.py` — `test_render_un_singur_container_chips`,
  `test_render_fara_op_row_si_fara_butoane_adauga`, `test_chip_warning_pentru_operatie_nemapata`
- **Acceptance criteria**:
  - [x] Un singur container `.chips` contine: chips cod (cu tooltip operatia sursa), chips warning
        operatii nemapate, chip `Nedeclarat` pentru itemi exclusi, selectul unic la capat.
  - [x] Selectul (nume camp: `chips_pick`, id stabil `chips-picker`) are `hx-trigger="change"`,
        `hx-post="/form-chips"`, `hx-target="#chips-section"`, `hx-swap="outerHTML"`,
        `hx-include="closest form"` (transporta CSRF + starea hidden — omiterea = 403) si
        `hx-disabled-elt="this"` (capcana htmx din 5.22 — NU mosteni de la form).
  - [x] Selectul revine la placeholder dupa ORICE raspuns (succes SI eroare); valoarea ramasa in
        `chips_pick` NU e aplicata la Salveaza pe niciuna din cai (regresia c6e29aa nu reapare
        sub alt nume — codul se adauga DOAR prin actiunea add).
  - [x] Pe durata unui request /form-chips, INTREAGA sectiune e inerta (`pointer-events:none` pe
        `.htmx-request` sau echivalent) — click pe × / chip warning mid-flight nu pierde actiuni.
  - [x] Click pe chip warning comuta tinta CLIENT-SIDE (clasa + hidden `chips_target_index`),
        fara round-trip; click pe tinta activa = no-op.
  - [x] Focus restaurat pe `#chips-picker` dupa swap.
  - [x] Zero aparitii `op-row`, `Adauga`, `add_flat`, `chips_add_cod_` in template.
  - [x] `chips_submission_id` e emis ca hidden in sectiune si ecouat de `/form-chips`, astfel
        incat "salveaza ca regula" ramane disponibil si DUPA re-randari (azi dispare la prima
        interactiune — bug perpetuat altfel).
- **Verificare E2E**: acoperit de US-005.

### US-003: Sugestii fuzzy/k-NN + optiunea "Nu se declara la RAR" in selectul unic
**Ca** operator **vreau** sugestiile de mapare direct in selectul din modal **pentru ca** azi ele
exista doar in panoul de mapare, iar in modal caut manual prin tot nomenclatorul.

- **Depinde de**: US-001, US-002
- **Fisiere**: `app/web/routes.py` (`post_form_chips` — construire sugestii pentru operatia tinta,
  refolosind sursele din preview), `app/web/templates/_chips_prestatii.html`, `tests/test_web_chips.py` (~3 fisiere)
- **Test intai (RED)**: `tests/test_web_chips.py` — `test_select_are_optgroup_sugestii_pt_tinta`,
  `test_select_are_nedeclarat_doar_cu_tinta`, `test_select_fara_tinta_fara_sugestii`
- **Acceptance criteria**:
  - [x] Cu tinta activa: `optgroup "Sugestii"` cu sugestia principala + top-3 fuzzy/k-NN pentru
        `denumire`-a operatiei; sub el optiunea `__NEDECLARAT__`; apoi nomenclatorul complet.
  - [x] Fara tinta: selectul listeaza doar nomenclatorul (fara Sugestii, fara `__NEDECLARAT__`).
  - [x] Sugestiile folosesc aceleasi functii ca `_preview_import.html` (fara drum nou de cod).
  - [x] Degradare gratioasa: motor de sugestii rece/dezactivat → select DOAR cu nomenclatorul
        (fara optgroup), fara eroare si fara blocare pe warmup (lock non-blocking, PRD 5.21).
  - [x] `python3 -m pytest -q` trece.
- **Verificare E2E**: acoperit de US-005.

### US-004: Persistarea `exclus` prin salvare (import + Trimiteri) — cu round-trip complet
**Ca** operator **vreau** ca excluderea aleasa in modal sa se pastreze la salvare, sa produca
starea corecta si sa fie REVERSIBILA la redeschidere **pentru ca** altfel optiunea din select ar fi
cosmetica sau, mai rau, operatia ar fi declarata contra intentiei mele (RAR e ireversibil).

> Corectie din review (eng, CONFIRMATA in cod): pipeline-ul actual sterge `exclus` per-item
> (`resolve_prestatii` face `it.pop("exclus", None)`, `mapping.py:298`) si elimina itemii exclusi
> din `payload_json` la corectie (`routes.py:1744`). Fara fix-urile de mai jos, un item exclus din
> modal ar reveni `needs_mapping` sau — daca operatia are mapare pe cont — ar fi DECLARAT la RAR.

- **Depinde de**: US-001
- **Fisiere**: `app/mapping.py` (`resolve_prestatii` — treapta noua: item-exclus explicit
  supravietuieste), `app/web/routes.py` (`web_editeaza_rand`, `post_corectie_trimitere` :1619,
  `_prestatii_chips_from_payload` :1336 + toti constructorii de context chips),
  `app/api/v1/import_router.py` (`apply_row_override` :319-366), `tests/test_web_editare_rand.py` (~4 fisiere)
- **Test intai (RED)**: `tests/test_web_editare_rand.py` — `test_salvare_pastreaza_exclus_in_override`,
  `test_exclus_supravietuieste_resolve_chiar_cu_mapare_pe_cont`,
  `test_redeschidere_modal_arata_chip_nedeclarat_si_x_readuce_warning`,
  `test_rand_toate_operatiile_excluse_devine_excluded`, `test_salvare_trimitere_cu_exclus`
- **Acceptance criteria**:
  - [x] `resolve_prestatii` pastreaza `exclus=1` venit explicit pe item (precedenta: item-exclus >
        cod explicit > regula exclus cont > mapare > reguli text); restul treptelor neschimbate.
  - [x] Itemii exclusi RAMAN in `payload_json`/`override_json` adnotati cu `exclus=1`;
        `split_prestatii_excluse` ii scoate DOAR din payload-ul trimis la RAR si din cheia idempotenta.
  - [x] Toti constructorii de chips (`_prestatii_chips_from_payload`, preview, re-render eroare)
        propaga `exclus` → chip `Nedeclarat` reapare la redeschidere; `×` il readuce la warning.
  - [x] Rand cu toate operatiile excluse → stare preview `excluded` ("Nedeclarat"); la Trimiteri →
        `needs_data` cu motiv explicit (invariantele existente).
  - [x] Validarea cod-vs-nomenclator la salvare ramane; sentinelul `__NEDECLARAT__` e RESPINS
        explicit daca apare ca `cod_prestatie` in starea hidden (nu doar in select), pe ambele cai.
  - [x] `python3 -m pytest -q` trece.
- **Verificare E2E**: acoperit de US-005.

### US-005: Verificare E2E browser pe ambele fluxuri
**Ca** lead **vreau** dovada in browser ca editarea e acum un singur control coerent
**pentru ca** bug-ul raportat e de UX vizual, nu doar de logica.

- **Depinde de**: US-002, US-003, US-004
- **Fisiere**: — (doar verificare; eventuale fix-uri in fisierele de mai sus)
- **Test intai (RED)**: —
- **Acceptance criteria**:
  - [x] Import preview: rand cu operatie nemapata + cod liber → modalul arata UN rand de chips
        (warning + cod + select), fara chenare/zone separate; screenshot atasat la raport.
  - [x] Selectarea unui cod din Sugestii transforma chip-ul warning in chip cod, instant, fara "+".
  - [x] Alegerea "Nu se declara la RAR" produce chip `Nedeclarat`; salvare → rand "Nedeclarat".
  - [x] Trimiteri (`/trimitere/{id}` corectie): acelasi control, "salveaza ca regula" functional.
  - [x] Salvarea persista chips-urile (regresie c6e29aa) si nu reapar controale duble la re-deschidere.
- **Verificare E2E**: browser HTMX pe http://localhost:8010/ (import xlsx test + o trimitere
  `needs_mapping`), conform pasilor de mai sus.

## 5. Riscuri

- **`hx-disabled-elt` mostenit de la form** a mai rupt butoanele chips o data (5.22) — selectul unic
  primeste explicit `hx-disabled-elt="this"`; test de template pentru atribut.
- **`change` pe select nativ** se declanseaza si la navigarea cu tastatura pe unele browsere vechi;
  acceptat (comportamentul modern e la confirmare), iar chip-ul gresit se sterge cu un click.
- **Compatibilitate stare veche**: submission-uri/override-uri existente fara `chip_exclus` trebuie
  citite ca `exclus=0` (default la deserializare) — altfel salvarea de pe randuri vechi crapa.
- **Doua endpoint-uri de save** (import + Trimiteri) cu acelasi contract de campuri — orice
  schimbare de nume de camp se face simultan in ambele, plus `_chips_state_from_form` unic partajat.

## 6. Intrebari deschise

> Rezolvate cu utilizatorul la aprobare (2026-07-07):

- `×` pe un chip `Nedeclarat` → **readuce chip-ul warning** al operatiei (operatia NU se sterge).
- Selectul unic ramane `<select>` nativ cu tot nomenclatorul — **acceptat**; autocomplete ramane
  non-goal.

## 7. Valuri de executie (graful de dependente)

```
Val 1: [US-001]                      ← fara dependente
Val 2: [US-002] [US-004]             ← deblocate de US-001, fisiere distincte → paralel
Val 3: [US-003]                      ← deblocat de US-002
Val 4: [US-005]                      ← verificare finala E2E
```

---

## Anexa /autoplan — Review CEO (Faza 1, 2026-07-07, mod SELECTIVE EXPANSION, voci: subagent-only)

### NOT in scope (decizii din review)
- Autocomplete/cautare in picker — non-goal confirmat de utilizator.
- Schimbarea panoului de mapare / precedentei `resolve_prestatii` — non-goal PRD.
- Badge-uri sursa sugestie (GOLD/SILVER) in select — respins (P5): optgroup "Sugestii" e suficient.
- "Bulk: aplica maparea la N randuri identice" in preview — deferat TODOS (in afara blast radius).
- Metrici de baza (deschideri modal / % randuri mapate manual) — deferat TODOS.

### What already exists (refolosit, nu reconstruit)
- Stare chips in hidden inputs + `/form-chips` stateless (`_chips_prestatii.html`, `routes.py:2402`).
- Validare cod vs nomenclator la add si la save (c6e29aa); mesaje `chips_error`/`chips_ok` (5.22).
- Sugestii: `suggest_codes` + `sugestie_principala` (`mapping.py:117,868`) — aceleasi surse ca preview.
- `EXCLUDE_SENTINEL`/`split_prestatii_excluse` (0bc6187); `apply_row_override` (c6e29aa).
- Empty state nomenclator lipsa; `has_r_odo`/reveal odometru; buton "salveaza ca regula" (Trimiteri).

### Dream state delta
Planul muta editorul de la 3 controale/4 actiuni la 1 control/2 actiuni si aduce sugestiile in
modal — pas direct spre idealul 12 luni (o singura componenta de mapare peste tot). Ramase pe
traiectorie: autocomplete (daca nomenclatorul creste), componenta partajata cu panoul de mapare.

### Arhitectura (Sectiunea 1)
```
[modal Editare rand]──include──▶[_form_editare]──include──▶[_chips_prestatii (REWRITE)]
        │                                                        │ hx-post change/×
        ▼                                                        ▼
POST editeaza (import) ─┐                              POST /form-chips (add|remove)
POST corecteaza (Trim.) ─┴─▶ _chips_state_from_form ──▶ apply_row_override(prestatii+exclus)
                                                              │ (commit)
                              suggest_codes/mapping.py ◀──────┘ resolve + split_prestatii_excluse
```
Cuplaj NOU: /form-chips → motorul de sugestii (pana acum doar preview-ul) — justificat (reuse),
cu conditia caii NE-blocante pe request (lock embeddings non-blocking, PRD 5.21). Fara migrare DB
(`exclus` traieste in override_json). Rollback: git revert simplu. Fereastra de deploy: un modal
deschis peste deploy trimite campuri vechi (`chips_add_cod_{i}`) pe care handlerul nou le ignora
— utilizatorul redeschide editarea; acceptat (sesiune de editare scurta).

### Registru erori & rescue (Sectiunea 2)
```
CODEPATH                  | CE POATE ESUA                      | TRATARE                        | USER VEDE
--------------------------|------------------------------------|--------------------------------|-----------
/form-chips add           | cod gol / necunoscut               | chips_error (exista)           | mesaj rosu
/form-chips add           | index tinta iesit din forma        | chips_error (exista)           | mesaj rosu
/form-chips add NED       | __NEDECLARAT__ fara tinta          | GAP -> chips_error nou         | mesaj rosu
/form-chips render        | motor sugestii rece/dezactivat     | GAP -> degradare la nomenclator| select fara optgroup
/form-chips               | nomenclator gol                    | empty state (exista)           | avertisment
save (ambele)             | cod necunoscut in stare            | re-render cu eroare (exista)   | modal + eroare
save (ambele)             | stare veche fara chip_exclus       | default exclus=0 (PRD risc)    | nimic (compat)
save (ambele)             | CSRF invalid/expirat               | 403 (existent)                 | eroare form
enqueue                   | toate operatiile excluse           | excluded / needs_data (exista) | stare explicita
```
GAP-uri de inchis in implementare: NED-fara-tinta si degradarea sugestiilor (ambele = criterii noi).

### Securitate (Sectiunea 3)
Fara endpoint nou. Input nou `chip_exclus`: parsare stricta ("1" -> 1, orice altceva -> 0).
Sentinelul `__NEDECLARAT__` nu ajunge NICIODATA in payload RAR (split inainte de enqueue —
invariant existent; test explicit). XSS: `denumire` in options/title — autoescape Jinja (exista).
Sugestii cross-account: politica 5.14 (suggestion-only) neschimbata. CSRF pe /form-chips: exista.
Nicio amenintare High nemitigata.

### Edge cases date/interactiune (Sectiunea 4)
- Dublu `change` in zbor: `hx-disabled-elt="this"` pe select (criteriu US-002).
- Focus pierdut dupa swap `outerHTML`: focusul revine pe select dupa add (criteriu nou US-002).
- Navigare cu tastatura prin select declanseaza change pe unele browsere: acceptat (PRD risc),
  chip-ul gresit se scoate cu un click.
- Iesire din modal mid-edit: stare pierduta (comportament existent, form-local) — neschimbat.
- Dedup (operatie,cod) si cod liber duplicat: exista, se pastreaza.
- Mod plat + `has_r_odo`: pastrate explicit (spec actualizat).

### Calitate (Sectiunea 5)
4 actiuni -> 2 reduce ramificarea din `post_form_chips` (~120 linii azi). `_chips_state_from_form`
ramane UNICUL parser, partajat de ambele endpoint-uri de save (risc PRD; test comun).
Alternativa "un singur camp hidden JSON" in loc de 4 liste paralele — decizie de gust (vezi gate).

### Teste (Sectiunea 6) — completari la RED-urile din PRD
- `test_add_nedeclarat_fara_tinta_da_eroare` (US-001)
- `test_sugestii_indisponibile_degradare_fara_optgroup` (US-003)
- `test_mod_plat_fara_operatii_select_doar_coduri_libere` (US-002)
- `test_has_r_odo_pastrat_dupa_actiuni` (US-002)
- `test_sentinel_nedeclarat_nu_ajunge_in_payload` (US-004)
- Template: `test_select_are_hx_disabled_elt_this` (capcana 5.22)

### Performanta (Sectiunea 7)
Sugestiile se calculeaza la FIECARE re-render /form-chips pentru operatia tinta — bounded
(matvec numpy 5.21, corpus 17k, <100ms tipic); calea request NU asteapta warmup (lock non-blocking).
18 optiuni in select — trivial. Fara probleme.

### Observabilitate (Sectiunea 8)
Starea randurilor e deja vizibila in dashboard (needs_mapping/excluded). Gap acceptat si deferat
TODOS: metrica "deschideri modal / sugestie acceptata" (leaga de decizia de produs, nu de 5.23).

### Deploy (Sectiunea 9)
Fara migrare, fara feature flag (UI inlocuit atomic in acelasi template). Worker neatins.
Post-deploy: smoke = deschide un rand needs_mapping, adauga un cod, salveaza.

### Traiectorie (Sectiunea 10)
Reversibilitate 4/5 (revert curat; starile cu exclus persistate raman valide — split exista
independent). Datorie REDUSA (actiuni duplicate eliminate). Comentariul de header din
`_chips_prestatii.html` se rescrie (diagrama noua a starii — parte din US-002).

### Design/UX la nivel CEO (Sectiunea 11)
Ierarhie: chips (starea) inaintea selectului (actiunea) — corect. Stari acoperite: gol (hint),
nomenclator lipsa, eroare, succes, exclus. Mobil: rand cu wrap; latimea selectului pe 390px =
criteriu in faza Design. Accesibilitate: aria-labels existente se pastreaza; focus dupa swap.
Riscul de mis-binding (tinta implicita + fara text permanent) — ridicat de vocea externa,
decizie la gate. Detaliu complet in Faza 2 (plan-design-review).

### Decision Audit Trail (/autoplan)

| # | Faza | Decizie | Clasificare | Principiu | Rationament | Respins |
|---|------|---------|-------------|-----------|-------------|---------|
| 1 | CEO | Abordare A (rescriere template + unificare actiuni) | Mecanica | P1,P5 | B pastreaza confuzia; C incalca non-goal aprobat | B, C |
| 2 | CEO | A11y tastatura + focus dupa swap → criterii US-002 | Mecanica | P1 | in blast radius, 2 fisiere | — |
| 3 | CEO | Badge sursa sugestie in select → respins | Mecanica | P5 | optgroup "Sugestii" spune deja asta | badge-uri |
| 4 | CEO | Bulk-apply mapare pe randuri identice → TODOS | Mecanica | P2,P3 | in afara blast radius (panou preview = non-goal) | — |
| 5 | CEO | Metrici deschideri-modal/sugestii → TODOS | Mecanica | P3 | produs, nu 5.23 | — |
| 6 | CEO | GAP-uri erori (NED-fara-tinta, degradare sugestii) → criterii noi | Mecanica | P1 | zero silent failures | — |
| 7 | CEO | Auto-advance tinta + fara eticheta pe select | TASTE (gate) | — | voce externa: risc mis-binding pe declaratii ireversibile | — |
| 8 | CEO | Paleta click 18 coduri vs select nativ | TASTE (gate) | — | premisa "tot nomenclatorul" e falsa (18 coduri fixe) | — |
| 9 | CEO | Stare chips: 4 liste hidden vs un camp JSON | TASTE (gate) | — | a 3-a regresie in zona; JSON = sursa unica | — |
| 10 | CEO | Test cu operator real in US-005 | Mecanica | P1 | verifica direct riscul de mis-binding; cost ~0 | — |
| 11 | Design | Scara de dominanta vizuala (tabel §3.1) | Mecanica | P1,P5 | 3 tratamente concurente fara ierarhie = confuzia initiala reambalata | — |
| 12 | Design | Stari loading/eroare/gol/complet specificate (tabel §3.1) | Mecanica | P1 | eroarea de retea lasa selectul disabled permanent altfel | — |
| 13 | Design | Tinta invarianta (exista cat sunt nemapate; click pe tinta = no-op) | Mecanica | P5 | interzice starea "cod liber cu warnings inca pe rand" tacuta | — |
| 14 | Design | Comutare tinta CLIENT-SIDE (fara round-trip) | Mecanica | P3 | round-trip pe click de highlight = lent + race cu add-ul in zbor | server-side |
| 15 | Design | Ordine chips stabila + focus pe select dupa swap | Mecanica | P1 | reflow-ul + focusul pierdut rup fluxul de tastatura pe 10 randuri | — |
| 16 | Design | Optiuni `COD — denumire`; chip warning ellipsis 18ch | Mecanica | P5 | 18 coduri criptice fara denumire = memorare fortata | — |
| 17 | Design | Nedeclarat vizibil (line-through, nu mut) + mesaj cu numele operatiei | Mecanica | P1 | cea mai riscanta actiune legala nu poate fi cea mai discreta | stil mut |
| 18 | Design | Mesaje chips_ok/err raman pattern 5.22 (persista pana la actiune) | Mecanica | P4 | timer de auto-dismiss = JS nou fara valoare | timer 4s |
| 19 | Design | Auto-advance tinta: pastrat cu emfaza vs eliminat | TASTE (gate) | — | CEO-voice: elimina (mis-binding); Design-voice: pastreaza (momentum) + emfaza | — |
| 20 | Design | Touch targets × pe chips sub 44px (UI compact 11px existent) | Mecanica | P4 | consecvent cu design-ul compact existent; nota in TODOS a11y | 44px |
| 21 | Eng | `exclus` supravietuieste `resolve_prestatii` (treapta noua, non-goal amendat) | Mecanica | P1 | P1 confirmat: `it.pop("exclus")` mapping.py:298 face US-004 imposibil; alternativa (regula pe cont) schimba semantica aprobata | regula per-cont; taierea US-004 |
| 22 | Eng | Itemii exclusi raman in payload_json + `exclus` in toti constructorii de chips | Mecanica | P1 | P1 confirmat: routes.py:1744 ii elimina → × pe Nedeclarat imposibil dupa salvare | — |
| 23 | Eng | `chips_submission_id` hidden + ecou in /form-chips ("salveaza ca regula" persistent) | Mecanica | P1,P3 | butonul dispare azi la prima re-randare; fluxul nou trece TOT prin /form-chips | — |
| 24 | Eng | `chips_pick` reset dupa orice raspuns; valoarea NEaplicata la Salveaza | Mecanica | P5 | clasa de bug c6e29aa; sentinel NED pozitional pe calea corectie | aplicare implicita |
| 25 | Eng | Validare semantica tinta + chip_exclus strict + sectiune inerta in request | Mecanica | P1 | POST fabricat / index stale / race mid-flight leaga codul de alt item | — |
| 26 | Eng | `chips_present=1` — golirea listei se persista (nu no-op) | Mecanica | P1 | `_chips_state_from_form` intoarce None pe form fara chips → golire imposibila | no-op actual |
| 27 | Eng | Actualizarea testelor existente pe actiunile vechi in acelasi val | Mecanica | P6 | altfel suita e rosie intre Val 1 si Val 2 | — |
| 28 | Eng | Emfaza chip nou pe index, nu pe cod | Mecanica | P5 | doua operatii cu acelasi cod = dubla evidentiere falsa | — |
| 29 | Eng | Fara memoizare sugestii per re-render | Mecanica | P3 | rapidfuzz pe 18 randuri + k-NN gardat = bounded; memoizarea = complexitate fara castig | memoizare |
| D2 | Gate | Auto-advance pastrat + placeholder-ul selectului numeste tinta | USER | — | leaga explicit codul de operatie la momentul alegerii, fara text sub control | doar emfaza; fara auto-advance |
| D3 | Gate | Select nativ pastrat (mockup v1); paleta v2 vazuta si RESPINSA de utilizator | USER | — | directia utilizatorului; paleta ramane alternativa daca selectul se dovedeste lent (US-005 cu operator) | paleta 18 coduri |
| D4 | Gate | Stare chips = UN hidden `chips_state` JSON versionat + fallback tranzitoriu liste vechi | USER | — | elimina clasa de regresii de aliniere pe index | 4 liste paralele |
| D5 | Gate | Plan revizuit APROBAT de utilizator | USER | — | — | — |

### Anexa Eng (Faza 3) — diagrama de acoperire teste

```
CODE PATHS                                              USER FLOWS
[+] post_form_chips (rescris, 2 actiuni)                [+] Corectie rand cu N operatii nemapate
  ├── add cu tinta valida        [RED planificat]         ├── sugestie→chip instant   [RED + E2E US-005]
  ├── add fara tinta (liber)     [RED planificat]         ├── comutare tinta client   [E2E US-005]
  ├── add NED cu/fara tinta      [RED planificat]         ├── × pe cod → warning      [RED planificat]
  ├── tinta semantica invalida   [RED planificat]         ├── NED → Nedeclarat → ×    [RED planificat]
  ├── remove index/liber         [RED planificat]         ├── eroare retea la add     [→E2E] [GAP inchis §3.1]
  └── dedup + cod necunoscut     [exista, se muta]        └── golire totala + salvare [RED chips_present]
[+] resolve_prestatii treapta item-exclus [RED planificat — REGRESIUNE potentiala: IRON RULE,
    test obligatoriu ca exclus NU e sters si ca restul precedentei ramane identic]
[+] _chips_state_from_form (4 liste + compat vechi + liste scurte) [RED planificat]
[+] save x2 (import/corectie): exclus round-trip, NED respins, chips_pick ignorat [RED planificat]
[+] template: hx-atribute, focus, sectiune inerta, mod plat, has_r_odo [RED planificat]
COVERAGE planificata: 100% din ramurile noi au RED numit in stories. Fara eval-uri LLM (nu se
ating prompturi). Regresiune-cheie: precedenta resolve_prestatii → test dedicat blocant.
```

ENG DUAL VOICES — CONSENS (subagent-only):
```
  Dimensiune                     Claude   Codex  Consens
  1. Arhitectura solida?         DA*      N/A    OK (directia 4→2 actiuni corecta; * dupa fix P1-uri)
  2. Acoperire teste suficienta? NU→DA    N/A    FLAGGED→INCHIS (6 teste noi adaugate in stories)
  3. Riscuri de performanta?     NU       N/A    OK (18 coduri, k-NN gardat, bounded)
  4. Securitate acoperita?       PARTIAL→DA N/A  INCHIS (validare semantica tinta, chip_exclus strict, NED respins la save)
  5. Drumuri de eroare tratate?  PARTIAL→DA N/A  INCHIS (§3.1 + criterii noi)
  6. Risc de deploy gestionabil? DA       N/A    OK (fara migrare; fereastra modal-peste-deploy acceptata)
```

---

## Raport VERIFY

Verificator independent (subagent context curat, doar PRD + §5.6), 2026-07-08. **VERDICT: PASS**
pe toate criteriile testabile; regresia de aur RAR live = NEPROBAT (contul QA fara creds RAR).

- **Suita**: `python3 -m pytest -q` → verde. La VERIFY: 1672 passed, 1 skipped. Dupa fix-urile
  de code-review CLOSE: **1680 passed, 1 skipped** (+45 teste noi fata de baseline 1635).
- **US-001..US-004** PASS cu dovezi de cod + teste (chips_state JSON unic, 2 actiuni add/remove,
  exclus supravietuieste resolve_prestatii cu test de regresie blocant pe precedenta, sentinel
  respins pe ambele cai).
- **US-005 E2E browser** (import preview + Trimiteri corectie, Playwright): PASS pe (a) un singur
  rand de chips cu tinta evidentiata + placeholder-tinta + optgroup Sugestii + optiune "Nu se
  declara la RAR" la deschiderea initiala; (b) sugestie→chip cod instant fara "+"; (c) Nedeclarat
  → salvare → rand "Nedeclarat", redeschidere arata chipul, × readuce warning; (d) acelasi control
  la Trimiteri, "salveaza ca regula" persistent; (e) fara controale duble, zero erori JS de flux.
- **Gap prins in executie** (dupa US-003, reparat inainte de VERIFY): tinta implicita + sugestiile
  nu se calculau la randarea initiala a modalului — helper partajat `_chips_target_si_sugestii`
  in toate caile de randare.

### Code-review CLOSE (high, 8 unghiuri + verify) — 1 BLOCKER de siguranta + 2 findings, toate reparate

- **BLOCKER (CONFIRMED, reparat)**: itemii exclusi (`exclus=1`, cod null) ramaneau in `payload_json`
  pe calea de corectie, iar worker-ul ii trimitea la RAR ca `{codPrestatie: null}` → risc
  ORA-12899 / FINALIZATA partial ireversibil. Nimic nu facea `split_prestatii_excluse` la MOMENTUL
  trimiterii. Fix la chokepoint unic (worker, inainte de `build_rar_payload`) — acopera toate
  canalele (API/import/corectie) — + filtru defensiv in `build_rar_payload` (sare itemii exclusi /
  fara cod) + teste care dovedesc ca itemul exclus NU apare in payload-ul RAR.
- **Consistenta round-trip (reparat)**: `commit_import`/`web_confirma_import` pastreaza acum
  prestatiile complete in `payload_json` (cheia idempotentei = doar declarabile, identica cu
  preview); bonus: bug latent de idempotenta preview↔commit in `_web_compute_preview` reparat.
- **Conventii CLAUDE.md (reparat)**: referinte PRD/US-xxx/decizii scoase din comentariile noi.
- **Robustete tinta (reparat)**: un item fara `cod_op_service` nu mai poate deveni tinta
  invizibila; sugestiile intorc gol pe denumire goala.

## GSTACK REVIEW REPORT

| Review | Trigger | Why | Runs | Status | Findings |
|--------|---------|-----|------|--------|----------|
| CEO Review | `/plan-ceo-review` | Scope & strategy | 1 | CLEAR (PLAN via /autoplan) | 5 proposals, 2 accepted, 3 deferred; 8 constatari voce externa |
| Codex Review | `/codex review` | Independent 2nd opinion | 0 | UNAVAILABLE (usage limit pana 2026-07-18) | voci = subagent-only |
| Eng Review | `/plan-eng-review` | Architecture & tests (required) | 1 | CLEAR (PLAN via /autoplan) | 8 issues (4×P1 confirmate in cod), 0 critical gaps ramase |
| Design Review | `/plan-design-review` | UI/UX gaps | 1 | CLEAR (FULL via /autoplan) | score: 6/10 → 9/10, 10 decizii in §3.1 |
| DX Review | `/plan-devex-review` | Developer experience gaps | 0 | SKIPPED | fara scope developer-facing |

- **CROSS-MODEL:** indisponibil (Codex la limita de utilizare); toate vocile externe = subagenti
  Claude independenti (context curat, fara review-ul anterior).
- **VERDICT:** CEO + ENG + DESIGN CLEARED — gata de implementare. Decizii gate aplicate:
  D2 placeholder-tinta, D3 select (paleta respinsa), D4 stare JSON, D5 aprobat.

NO UNRESOLVED DECISIONS

