docs(design): add two-accent system, selective mono, and dark mode decisions

Decisions from plan-design-review and plan-eng-review:
- Two-accent system: amber = state (nav, pills), blue = action (buttons)
- JetBrains Mono selective: codes/numbers only, text uses DM Sans
- Dark mode now in scope for Commit 0.5
- Add TODOS.md with deferred P2 items

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

CLAUDE.md

@@ -106,6 +106,13 @@ python3 scripts/sync_vending_to_mariusm.py --apply --yes
 Sincronizează via SSH din VENDING (prod Windows) în MARIUSM_AUTO (dev ROA_CENTRAL):
 nom_articole (noi by codmat, codmat updatat) + articole_terti (noi, modificate, soft-delete).
 
+## Design System
+
+Always read DESIGN.md before making any visual or UI decisions.
+All font choices, colors, spacing, and aesthetic direction are defined there.
+Do not deviate without explicit user approval.
+In QA mode, flag any code that doesn't match DESIGN.md.
+
 ## Deploy Windows
 
 Vezi [README.md](README.md#deploy-windows)

DESIGN.md

@@ -0,0 +1,324 @@
+# Design System — GoMag Vending
+
+## Product Context
+- **What this is:** Internal admin dashboard for importing web orders from GoMag e-commerce into ROA Oracle ERP
+- **Who it's for:** Ops/admin team who monitor order sync daily, fix SKU mappings, check import errors
+- **Space/industry:** Internal tools, B2B operations, ERP integration
+- **Project type:** Data-heavy admin dashboard (tables, status indicators, sync controls)
+
+## Aesthetic Direction
+- **Direction:** Industrial/Utilitarian — function-first, data-dense, quietly confident
+- **Decoration level:** Minimal — typography and color do the work. No illustrations, gradients, or decorative elements. The data IS the decoration.
+- **Mood:** Command console. This tool says "built by someone who respects the operator." Serious, efficient, warm.
+- **Anti-patterns:** No purple gradients, no 3-column icon grids, no centered-everything layouts, no decorative blobs, no stock-photo heroes
+
+## Typography
+
+### Font Stack
+- **Display/Headings:** Space Grotesk — geometric, slightly techy, distinctive `a` and `g`. Says "engineered."
+- **Body/UI:** DM Sans — clean, excellent readability, good tabular-nums for inline numbers
+- **Data/Tables:** JetBrains Mono — order IDs, CODMATs, status codes align perfectly. Tables become scannable.
+- **Code:** JetBrains Mono
+
+### Loading
+```html
+<link rel="preconnect" href="https://fonts.googleapis.com">
+<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
+<link href="https://fonts.googleapis.com/css2?family=DM+Sans:ital,opsz,wght@0,9..40,300;0,9..40,400;0,9..40,500;0,9..40,600;0,9..40,700;1,9..40,400&family=JetBrains+Mono:wght@400;500;600&family=Space+Grotesk:wght@400;500;600;700&display=swap" rel="stylesheet">
+```
+
+### CSS Variables
+```css
+--font-display: 'Space Grotesk', sans-serif;
+--font-body: 'DM Sans', sans-serif;
+--font-data: 'JetBrains Mono', monospace;
+```
+
+### Type Scale
+| Level | Size | Weight | Font | Usage |
+|-------|------|--------|------|-------|
+| Page title | 18px | 600 | Display | "Panou de Comanda" |
+| Section title | 16px | 600 | Display | Card headers |
+| Label/uppercase | 12px | 500 | Display | Column headers, section labels (letter-spacing: 0.04em) |
+| Body | 14px | 400 | Body | Paragraphs, descriptions |
+| UI/Button | 13px | 500 | Body | Buttons, nav links, form labels |
+| Data cell | 13px | 400 | Data | Codes, IDs, numbers, sums, dates (NOT text names — those use Body font) |
+| Data small | 12px | 400 | Data | Timestamps, secondary data |
+| Code/mono | 11px | 400 | Data | Inline code, debug info |
+
+## Color
+
+### Approach: Two-accent system (amber state + blue action)
+Every admin tool is blue. This one uses amber — reads as "operational" and "attention-worthy."
+- **Amber (--accent):** Navigation active state, filter pill active, accent backgrounds. "Where you are."
+- **Blue (--info):** Primary buttons, CTAs, actionable links. "What you can do."
+- Primary buttons (`btn-primary`) stay blue for clear action hierarchy.
+
+### Light Mode (default)
+```css
+:root {
+  /* Surfaces */
+  --bg: #F8F7F5;              /* warm off-white, not clinical gray */
+  --surface: #FFFFFF;
+  --surface-raised: #F3F2EF;  /* hover states, table headers */
+  --card-shadow: 0 1px 3px rgba(28,25,23,0.1), 0 1px 2px rgba(28,25,23,0.06);
+
+  /* Text */
+  --text-primary: #1C1917;    /* warm black */
+  --text-secondary: #57534E;  /* warm gray */
+  --text-muted: #78716C;      /* labels, timestamps */
+
+  /* Borders */
+  --border: #E7E5E4;
+  --border-subtle: #F0EFED;
+
+  /* Accent — amber */
+  --accent: #D97706;
+  --accent-hover: #B45309;
+  --accent-light: #FEF3C7;    /* amber backgrounds */
+  --accent-text: #92400E;     /* text on amber bg */
+
+  /* Semantic */
+  --success: #16A34A;
+  --success-light: #DCFCE7;
+  --success-text: #166534;
+
+  --warning: #CA8A04;
+  --warning-light: #FEF9C3;
+  --warning-text: #854D0E;
+
+  --error: #DC2626;
+  --error-light: #FEE2E2;
+  --error-text: #991B1B;
+
+  --info: #2563EB;
+  --info-light: #DBEAFE;
+  --info-text: #1E40AF;
+
+  --cancelled: #78716C;
+  --cancelled-light: #F5F5F4;
+}
+```
+
+### Dark Mode
+Strategy: invert surfaces, reduce accent saturation ~15%, keep semantic colors recognizable.
+
+```css
+[data-theme="dark"] {
+  --bg: #121212;
+  --surface: #1E1E1E;
+  --surface-raised: #2A2A2A;
+  --card-shadow: 0 1px 3px rgba(0,0,0,0.4), 0 1px 2px rgba(0,0,0,0.3);
+
+  --text-primary: #E8E4DD;     /* warm bone white */
+  --text-secondary: #A8A29E;
+  --text-muted: #78716C;
+
+  --border: #333333;
+  --border-subtle: #262626;
+
+  --accent: #F59E0B;
+  --accent-hover: #D97706;
+  --accent-light: rgba(245,158,11,0.12);
+  --accent-text: #FCD34D;
+
+  --success: #16A34A;
+  --success-light: rgba(22,163,74,0.15);
+  --success-text: #4ADE80;
+
+  --warning: #CA8A04;
+  --warning-light: rgba(202,138,4,0.15);
+  --warning-text: #FACC15;
+
+  --error: #DC2626;
+  --error-light: rgba(220,38,38,0.15);
+  --error-text: #FCA5A5;
+
+  --info: #2563EB;
+  --info-light: rgba(37,99,235,0.15);
+  --info-text: #93C5FD;
+
+  --cancelled: #78716C;
+  --cancelled-light: rgba(120,113,108,0.15);
+}
+```
+
+### Status Color Mapping
+| Status | Dot Color | Badge BG | Glow |
+|--------|-----------|----------|------|
+| IMPORTED | `--success` | `--success-light` | none (quiet when healthy) |
+| ERROR | `--error` | `--error-light` | `0 0 8px 2px rgba(220,38,38,0.35)` |
+| SKIPPED | `--warning` | `--warning-light` | `0 0 6px 2px rgba(202,138,4,0.3)` |
+| ALREADY_IMPORTED | `--info` | `--info-light` | none |
+| CANCELLED | `--cancelled` | `--cancelled-light` | none |
+| DELETED_IN_ROA | `--cancelled` | `--cancelled-light` | none |
+
+**Design rule:** Problems glow, success is calm. The operator's eye is pulled to rows that need action.
+
+## Spacing
+- **Base unit:** 4px
+- **Density:** Comfortable — not cramped, not wasteful
+- **Scale:**
+
+| Token | Value | Usage |
+|-------|-------|-------|
+| 2xs | 2px | Tight internal gaps |
+| xs | 4px | Icon-text gap, badge padding |
+| sm | 8px | Compact card padding, table cell padding |
+| md | 16px | Standard card padding, section gaps |
+| lg | 24px | Section spacing |
+| xl | 32px | Major section gaps |
+| 2xl | 48px | Page-level spacing |
+| 3xl | 64px | Hero spacing (rarely used) |
+
+## Layout
+
+### Approach: Grid-disciplined, full-width
+Tables with 8+ columns and hundreds of rows need every pixel of width.
+
+- **Nav:** Horizontal top bar, fixed, 48px height. Active tab has amber underline (2px).
+- **Content max-width:** None on desktop (full-width for tables), 1200px for non-table content
+- **Grid:** Single-column layout, cards stack vertically
+- **Breakpoints:**
+
+| Name | Width | Columns | Behavior |
+|------|-------|---------|----------|
+| Desktop | >= 1024px | Full width | All features visible |
+| Tablet | 768-1023px | Full width | Nav labels abbreviated, tables scroll horizontally |
+| Mobile | < 768px | Single column | Bottom nav, cards stack, condensed views |
+
+### Border Radius
+| Token | Value | Usage |
+|-------|-------|-------|
+| sm | 4px | Buttons, inputs, badges, status dots |
+| md | 8px | Cards, dropdowns, modals |
+| lg | 12px | Large containers, mockup frames |
+| full | 9999px | Pills, avatar circles |
+
+## Motion
+- **Approach:** Minimal-functional — only transitions that aid comprehension
+- **Easing:** enter(ease-out) exit(ease-in) move(ease-in-out)
+- **Duration:**
+
+| Token | Value | Usage |
+|-------|-------|-------|
+| micro | 50-100ms | Button hover, focus ring |
+| short | 150-250ms | Dropdown open, tab switch, color transitions |
+| medium | 250-400ms | Modal open/close, page transitions |
+| long | 400-700ms | Only for sync pulse animation |
+
+- **Sync pulse:** The live sync dot uses a 2s infinite pulse (opacity 1 → 0.4 → 1)
+- **No:** entrance animations, scroll effects, decorative motion
+
+## Mobile Design
+
+### Navigation
+- **Bottom tab bar** replaces top horizontal nav on screens < 768px
+- 5 tabs: Dashboard, Mapari, Lipsa, Jurnale, Setari
+- Each tab: icon (Bootstrap Icons) + short label below
+- Active tab: amber accent color, inactive: `--text-muted`
+- Height: 56px, safe-area padding for notched devices
+- Fixed position bottom, with `padding-bottom: env(safe-area-inset-bottom)`
+
+```css
+@media (max-width: 767px) {
+  .top-navbar { display: none; }
+  .bottom-nav {
+    position: fixed;
+    bottom: 0;
+    left: 0;
+    right: 0;
+    height: 56px;
+    padding-bottom: env(safe-area-inset-bottom);
+    background: var(--surface);
+    border-top: 1px solid var(--border);
+    display: flex;
+    justify-content: space-around;
+    align-items: center;
+    z-index: 1000;
+  }
+  .main-content {
+    padding-bottom: 72px; /* clear bottom nav */
+    padding-top: 8px;     /* no top navbar */
+  }
+}
+```
+
+### Dashboard — Mobile
+- **Sync card:** Full width, stacked vertically
+  - Status + controls row wraps to 2 lines
+  - Sync button full-width at bottom of card
+  - Last sync info wraps naturally
+- **Orders table:** Condensed card view instead of horizontal table
+  - Each order = a compact card showing: status dot + ID + client name + total
+  - Tap to expand: shows date, factura, full details
+  - Swipe left on card: quick action (view error details)
+- **Filter bar:** Horizontal scrollable chips instead of dropdowns
+  - Period selector: pill chips (1zi, 7zi, 30zi, Toate)
+  - Status filter: colored chips matching status colors
+- **Touch targets:** Minimum 44x44px for all interactive elements
+
+### Orders Mobile Card Layout
+```
+┌────────────────────────────────┐
+│ ● CMD-47832        2,450.00 RON│
+│   SC Automate Express SRL      │
+│   27.03.2026 · FCT-2026-1847  │
+└────────────────────────────────┘
+```
+- Status dot (8px, left-aligned with glow for errors)
+- Order ID in JetBrains Mono, amount right-aligned
+- Client name in DM Sans
+- Date + factura in muted data font
+
+### SKU Mappings — Mobile
+- Each mapping = expandable card
+- Collapsed: SKU + product name + type badge (KIT/SIMPLU)
+- Expanded: Full CODMAT list with quantities
+- Search: Full-width sticky search bar at top
+- Filter: Horizontal scrollable type chips
+
+### Logs — Mobile
+- Timeline view instead of table
+- Each log entry = timestamp + status icon + summary
+- Tap to expand full log details
+- Infinite scroll with date separators
+
+### Settings — Mobile
+- Standard stacked form layout
+- Full-width inputs
+- Toggle switches for boolean settings (min 44px touch target)
+- Save button sticky at bottom
+
+### Gestures
+- **Pull to refresh** on Dashboard: triggers sync status check
+- **Swipe left** on order card: reveal quick actions
+- **Long press** on SKU mapping: copy CODMAT to clipboard
+- **No swipe navigation** between pages (use bottom tabs)
+
+### Mobile Typography Adjustments
+| Level | Desktop | Mobile |
+|-------|---------|--------|
+| Page title | 18px | 16px |
+| Body | 14px | 14px (no change) |
+| Data cell | 13px | 13px (no change) |
+| Data small | 12px | 12px (no change) |
+| Table header | 12px | 11px |
+
+### Responsive Images & Icons
+- Use Bootstrap Icons throughout (already loaded via CDN)
+- Icon size: 16px desktop, 20px mobile (larger touch targets)
+- No images in the admin interface (data-only)
+
+## Decisions Log
+| Date | Decision | Rationale |
+|------|----------|-----------|
+| 2026-03-27 | Initial design system created | Created by /design-consultation. Industrial/utilitarian aesthetic with amber accent, Space Grotesk + DM Sans + JetBrains Mono. |
+| 2026-03-27 | Amber accent over blue | Every admin tool is blue. Amber reads as "operational" and gives the tool its own identity. Confirmed by Claude subagent ("Control Room Noir" also converged on amber). |
+| 2026-03-27 | JetBrains Mono for data tables | Both primary analysis and subagent independently recommended monospace for data tables. Scannability win outweighs the ~15% wider columns. |
+| 2026-03-27 | Warm tones throughout | Off-white (#F8F7F5) instead of clinical gray. Warm black text instead of blue-gray. Makes the tool feel handcrafted. |
+| 2026-03-27 | Glowing status dots for errors | Problems glow (box-shadow), success is calm. Operator's eye is pulled to rows that need action. Inspired by subagent's "LED indicator" concept. |
+| 2026-03-27 | Full mobile design | Bottom nav, card-based order views, touch-optimized gestures. Supports quick-glance usage from phone. |
+| 2026-03-27 | Two-accent system | Blue = action (buttons, CTAs), amber = state (nav active, filter active). Clear hierarchy. |
+| 2026-03-27 | JetBrains Mono selective | Mono font only for codes, IDs, numbers, sums, dates. Text names use DM Sans for readability. |
+| 2026-03-27 | Dark mode in scope | CSS variables + toggle + localStorage. All DESIGN.md dark tokens implemented in Commit 0.5. |

README-ORACLE-MODES.md

@@ -1,150 +0,0 @@
-# Oracle Modes Configuration Guide - UNIFIED
-
-## 🎯 Un Singur Dockerfile + Docker Compose
-
-| Oracle Version | Configurație .env | Comandă Build | Port |
-|---------------|-------------------|---------------|------|
-| 10g (test)    | `INSTANTCLIENTPATH=...` | `docker-compose up --build` | 5003 |
-| 11g (prod)    | `INSTANTCLIENTPATH=...` | `docker-compose up --build` | 5003 |
-| 12.1+ (nou)   | `FORCE_THIN_MODE=true` | `ORACLE_MODE=thin docker-compose up --build` | 5003 |
-
----
-
-## 🔧 THICK MODE (Oracle 10g/11g) - DEFAULT
-
-### Configurare .env:
-```env
-# Uncomment această linie pentru thick mode:
-INSTANTCLIENTPATH=/opt/oracle/instantclient_23_9
-
-# Comment această linie:
-# FORCE_THIN_MODE=true
-```
-
-### Rulare:
-```bash
-docker-compose up --build -d
-curl http://localhost:5003/health
-```
-
----
-
-## 🚀 THIN MODE (Oracle 12.1+) 
-
-### Varianta 1 - Prin .env (Recomandat):
-```env
-# Comment această linie pentru thin mode:
-# INSTANTCLIENTPATH=/opt/oracle/instantclient_23_9  
-
-# Uncomment această linie:
-FORCE_THIN_MODE=true
-```
-
-### Varianta 2 - Prin build argument:
-```bash
-ORACLE_MODE=thin docker-compose up --build -d
-```
-
-### Test:
-```bash
-curl http://localhost:5003/health
-```
-
----
-
-## 🔄 LOGICA AUTO-DETECT
-
-Container-ul detectează automat modul:
-
-1. **FORCE_THIN_MODE=true** → **Thin Mode**
-2. **INSTANTCLIENTPATH** există → **Thick Mode**
-3. Build cu **ORACLE_MODE=thin** → **Thin Mode**  
-4. Default → **Thick Mode**
-
----
-
-## 🛠️ COMENZI SIMPLE
-
-### Pentru Oracle 10g/11g (setup-ul tău actual):
-```bash
-# Verifică .env să aibă:
-grep INSTANTCLIENTPATH ./api/.env
-
-# Start
-docker-compose up --build -d
-curl http://localhost:5003/test-db
-```
-
-### Pentru Oracle 12.1+ (viitor):
-```bash
-# Editează .env: decomentează FORCE_THIN_MODE=true
-# SAU rulează direct:
-ORACLE_MODE=thin docker-compose up --build -d
-curl http://localhost:5003/test-db
-```
-
-### Switch rapid:
-```bash
-# Stop
-docker-compose down
-
-# Edit .env (change INSTANTCLIENTPATH ↔ FORCE_THIN_MODE)
-# Start
-docker-compose up --build -d
-```
-
----
-
-## ⚠️ TROUBLESHOOTING
-
-### Eroare DPY-3010 în Thin Mode:
-```
-DPY-3010: connections to this database server version are not supported
-```
-**Soluție:** Oracle este 11g sau mai vechi → folosește thick mode
-
-### Eroare libaio în Thick Mode:
-```
-Cannot locate a 64-bit Oracle Client library: libaio.so.1
-```
-**Soluție:** Rebuild container (fix automat în Dockerfile.thick)
-
-### Container nu pornește:
-```bash
-docker-compose logs
-docker-compose down && docker-compose up --build
-```
-
----
-
-## 📊 COMPARAȚIE PERFORMANȚĂ
-
-| Aspect | Thick Mode | Thin Mode |
-|--------|------------|-----------|
-| Container Size | ~200MB | ~50MB |
-| Startup Time | 10-15s | 3-5s |
-| Memory Usage | ~100MB | ~30MB |
-| Oracle Support | 10g+ | 12.1+ |
-| Dependencies | Instant Client | None |
-
----
-
-## 🔧 DEZVOLTARE
-
-### Pentru dezvoltatori:
-1. **Thick mode** pentru compatibilitate maximă
-2. **Thin mode** pentru development rapid pe Oracle nou
-3. **Auto-detect** în producție pentru flexibilitate
-
-### Testare ambele moduri:
-```bash
-# Thick pe port 5003
-docker-compose -f docker-compose.thick.yaml up -d
-
-# Thin pe port 5004  
-docker-compose -f docker-compose.thin.yaml up -d
-
-# Test ambele
-curl http://localhost:5003/health
-curl http://localhost:5004/health
-```

README.md

@@ -110,7 +110,8 @@ gomag-vending/
 │   │   │   ├── gomag_client.py      # Download comenzi GoMag API
 │   │   │   ├── sync_service.py      # Orchestrare: download→validate→import
 │   │   │   ├── import_service.py    # Import comanda in Oracle ROA
-│   │   │   ├── mapping_service.py   # CRUD ARTICOLE_TERTI + pct_total
+│   │   │   ├── mapping_service.py   # CRUD ARTICOLE_TERTI + cantitate_roa
+│   │   │   ├── price_sync_service.py # Sync preturi GoMag → Oracle politici
 │   │   │   ├── sqlite_service.py    # Tracking runs/orders/missing SKUs
 │   │   │   ├── order_reader.py      # Citire gomag_orders_page*.json
 │   │   │   ├── validation_service.py
@@ -127,7 +128,8 @@ gomag-vending/
 │   ├── test_integration.py      # Test C - cu Oracle
 │   └── requirements.txt
 ├── logs/                        # Log-uri aplicatie (sync_comenzi_*.log)
-├── docs/                        # Documentatie (PRD, stories)
+├── docs/                        # Documentatie (Oracle schema, facturare analysis)
+├── scripts/                     # Utilitare (sync_vending_to_mariusm, create_inventory_notes)
 ├── screenshots/                 # Before/preview/after pentru UI changes
 ├── start.sh                     # Script pornire (Linux/WSL)
 └── CLAUDE.md                    # Instructiuni pentru AI assistants
@@ -447,6 +449,16 @@ curl -X POST http://localhost:5003/api/dashboard/refresh-invoices
 
 ---
 
+## Documentatie Tehnica
+
+| Fisier | Subiect |
+|--------|---------|
+| [docs/oracle-schema-notes.md](docs/oracle-schema-notes.md) | Schema Oracle: tabele comenzi, facturi, preturi, proceduri cheie |
+| [docs/pack_facturare_analysis.md](docs/pack_facturare_analysis.md) | Analiza flow facturare: call chain, parametri, STOC lookup, FACT-008 |
+| [scripts/HANDOFF_MAPPING.md](scripts/HANDOFF_MAPPING.md) | Matching GoMag SKU → ROA articole (strategie si rezultate) |
+
+---
+
 ## WSL2 Note
 
 - `uvicorn --reload` **nu functioneaza** pe `/mnt/e/` (WSL2 limitation) — restarta manual

TODOS.md

@@ -0,0 +1,15 @@
+# TODOS
+
+## P2: Refactor sync_service.py in module separate
+**What:** Split sync_service.py (870 linii) in: download_service, parse_service, sync_orchestrator.
+**Why:** Faciliteza debugging si testare. Un bug in price sync nu ar trebui sa afecteze import flow.
+**Effort:** M (human: ~1 sapt / CC: ~1-2h)
+**Context:** Dupa implementarea planului Command Center (retry_service deja extras). sync_service face download + parse + validate + import + price sync + invoice check — prea multe responsabilitati.
+**Depends on:** Finalizarea planului Command Center.
+
+## P2: Email/webhook alert pe sync esuat
+**What:** Cand sync-ul gaseste >5 erori sau esueaza complet, trimite un email/webhook.
+**Why:** Post-lansare, cand app-ul ruleaza automat, nimeni nu sta sa verifice constant.
+**Effort:** M (human: ~1 sapt / CC: ~1h)
+**Context:** Depinde de infrastructura email/webhook disponibila la client. Implementare: SMTP simplu sau webhook URL configurabil in Settings.
+**Depends on:** Lansare in productie + infrastructura email la client.

api/README.md

@@ -26,6 +26,8 @@ Admin interface si orchestrator pentru importul comenzilor GoMag in Oracle ROA.
 | article_service | Cautare in NOM_ARTICOLE (Oracle) |
 | import_service | Port din VFP: partner/address/order creation |
 | sync_service | Orchestrare: read JSONs → validate → import → log |
+| price_sync_service | Sync preturi GoMag → Oracle politici de pret |
+| invoice_service | Verificare facturi ROA + cache SQLite |
 | validation_service | Batch-validare SKU-uri (chunks of 500) |
 | order_reader | Citire gomag_orders_page*.json din vfp/output/ |
 | sqlite_service | CRUD pe SQLite (sync_runs, import_orders, missing_skus) |
@@ -35,17 +37,19 @@ Admin interface si orchestrator pentru importul comenzilor GoMag in Oracle ROA.
 
 ```bash
 pip install -r requirements.txt
-uvicorn app.main:app --host 0.0.0.0 --port 5003 --reload
+# INTOTDEAUNA via start.sh din project root (seteaza Oracle env vars)
+cd .. && ./start.sh
 ```
 
 ## Testare
 
 ```bash
-# Test A - fara Oracle (verifica importuri + rute)
+# Din project root:
-python test_app_basic.py
+./test.sh ci       # Teste rapide (unit + e2e, ~30s, fara Oracle)
-
+./test.sh full     # Teste complete (inclusiv Oracle, ~2-3 min)
-# Test C - cu Oracle (integrare completa)
+./test.sh unit     # Doar unit tests
-python test_integration.py
+./test.sh e2e      # Doar browser tests (Playwright)
+./test.sh oracle   # Doar Oracle integration
 ```
 
 ## Dual Database

api/admin.py

@@ -1,250 +0,0 @@
-"""
-Flask Admin Interface pentru Import Comenzi Web → ROA
-Gestionează mapările SKU în tabelul ARTICOLE_TERTI
-"""
-
-from flask import Flask, jsonify, request, render_template_string
-from flask_cors import CORS
-from dotenv import load_dotenv
-import oracledb
-import os
-import logging
-from datetime import datetime
-
-# Configurare environment
-load_dotenv()
-
-# Configurare logging
-logging.basicConfig(
-    level=logging.DEBUG,
-    format='%(asctime)s | %(levelname)s | %(message)s',
-    handlers=[
-        logging.FileHandler('/app/logs/admin.log'),
-        logging.StreamHandler()
-    ]
-)
-logger = logging.getLogger(__name__)
-
-# Environment Variables pentru Oracle
-user = os.environ['ORACLE_USER']
-password = os.environ['ORACLE_PASSWORD'] 
-dsn = os.environ['ORACLE_DSN']
-
-# Oracle client - AUTO-DETECT: thick mode pentru 10g/11g, thin mode pentru 12.1+
-force_thin_mode = os.environ.get('FORCE_THIN_MODE', 'false').lower() == 'true'
-instantclient_path = os.environ.get('INSTANTCLIENTPATH')
-
-if force_thin_mode:
-    logger.info(f"FORCE_THIN_MODE=true: Folosind thin mode pentru {dsn} (Oracle 12.1+ required)")
-elif instantclient_path:
-    try:
-        oracledb.init_oracle_client(lib_dir=instantclient_path)
-        logger.info(f"Thick mode activat pentru {dsn} (compatibil Oracle 10g/11g/12.1+)")
-    except Exception as e:
-        logger.error(f"Eroare thick mode: {e}")
-        logger.info("Fallback la thin mode - verifică că Oracle DB este 12.1+")
-else:
-    logger.info(f"Thin mode (default) pentru {dsn} - Oracle 12.1+ required")
-
-app = Flask(__name__)
-CORS(app)
-
-def start_pool():
-    """Inițializează connection pool Oracle"""
-    try:        
-        pool = oracledb.create_pool(
-            user=user,
-            password=password, 
-            dsn=dsn,
-            min=2,
-            max=4,
-            increment=1
-        )
-        logger.info(f"Oracle pool creat cu succes pentru {dsn}")
-        return pool
-    except Exception as e:
-        logger.error(f"Eroare creare pool Oracle: {e}")
-        raise
-
-@app.route('/health')
-def health():
-    """Health check pentru Docker"""
-    return jsonify({"status": "ok", "timestamp": datetime.now().isoformat()})
-
-@app.route('/')
-def home():
-    """Pagina principală admin interface"""
-    html_template = """
-    <!DOCTYPE html>
-    <html>
-    <head>
-        <title>GoMag Admin - Mapări SKU</title>
-        <meta charset="utf-8">
-        <style>
-            body { font-family: Arial, sans-serif; margin: 40px; background-color: #f5f5f5; }
-            .container { max-width: 1200px; margin: 0 auto; background: white; padding: 20px; border-radius: 8px; box-shadow: 0 2px 4px rgba(0,0,0,0.1); }
-            h1 { color: #333; border-bottom: 3px solid #007bff; padding-bottom: 10px; }
-            .status { padding: 10px; border-radius: 4px; margin: 10px 0; }
-            .success { background-color: #d4edda; color: #155724; border: 1px solid #c3e6cb; }
-            .error { background-color: #f8d7da; color: #721c24; border: 1px solid #f5c6cb; }
-            .btn { background: #007bff; color: white; padding: 10px 20px; border: none; border-radius: 4px; cursor: pointer; margin: 5px; }
-            .btn:hover { background: #0056b3; }
-            .table-container { margin-top: 20px; }
-            table { width: 100%; border-collapse: collapse; margin-top: 10px; }
-            th, td { padding: 8px 12px; text-align: left; border-bottom: 1px solid #ddd; }
-            th { background-color: #f8f9fa; font-weight: bold; }
-            tr:hover { background-color: #f5f5f5; }
-        </style>
-    </head>
-    <body>
-        <div class="container">
-            <h1>🛍️ GoMag Admin - Import Comenzi Web → ROA</h1>
-            
-            <div id="status-area">
-                <div class="success">✅ Container Docker activ pe port 5003</div>
-                <div id="db-status">🔄 Verificare conexiune Oracle...</div>
-            </div>
-            
-            <div class="table-container">
-                <h2>📋 Mapări SKU Active</h2>
-                <button class="btn" onclick="loadMappings()">🔄 Reîmprospătează</button>
-                <button class="btn" onclick="testConnection()">🔍 Test Conexiune DB</button>
-                
-                <div id="mappings-container">
-                    <p>Loading...</p>
-                </div>
-            </div>
-        </div>
-        
-        <script>
-            // Test conexiune la load
-            window.onload = function() {
-                testConnection();
-                loadMappings();
-            }
-            
-            function testConnection() {
-                fetch('/test-db')
-                    .then(response => response.json())
-                    .then(data => {
-                        const statusDiv = document.getElementById('db-status');
-                        if (data.success) {
-                            statusDiv.className = 'status success';
-                            statusDiv.innerHTML = '✅ Oracle conectat: ' + data.message;
-                        } else {
-                            statusDiv.className = 'status error';  
-                            statusDiv.innerHTML = '❌ Eroare Oracle: ' + data.error;
-                        }
-                    })
-                    .catch(error => {
-                        document.getElementById('db-status').innerHTML = '❌ Eroare fetch: ' + error;
-                    });
-            }
-            
-            function loadMappings() {
-                fetch('/api/mappings')
-                    .then(response => response.json())
-                    .then(data => {
-                        let html = '<table>';
-                        html += '<tr><th>SKU</th><th>CODMAT</th><th>Cantitate ROA</th><th>Procent Preț</th><th>Activ</th><th>Data Creare</th></tr>';
-                        
-                        if (data.mappings && data.mappings.length > 0) {
-                            data.mappings.forEach(row => {
-                                const activIcon = row[4] === 1 ? '✅' : '❌';
-                                html += `<tr>
-                                    <td><strong>${row[0]}</strong></td>
-                                    <td>${row[1]}</td>  
-                                    <td>${row[2]}</td>
-                                    <td>${row[3]}%</td>
-                                    <td>${activIcon}</td>
-                                    <td>${new Date(row[5]).toLocaleDateString()}</td>
-                                </tr>`;
-                            });
-                        } else {
-                            html += '<tr><td colspan="6">Nu există mapări configurate</td></tr>';
-                        }
-                        html += '</table>';
-                        
-                        document.getElementById('mappings-container').innerHTML = html;
-                    })
-                    .catch(error => {
-                        document.getElementById('mappings-container').innerHTML = '❌ Eroare: ' + error;
-                    });
-            }
-        </script>
-    </body>
-    </html>
-    """
-    return render_template_string(html_template)
-
-@app.route('/test-db')
-def test_db():
-    """Test conexiune Oracle și verificare tabel"""
-    try:
-        with pool.acquire() as con:
-            with con.cursor() as cur:
-                # Test conexiune de bază
-                cur.execute("SELECT SYSDATE FROM DUAL")
-                db_date = cur.fetchone()[0]
-                
-                # Verificare existență tabel ARTICOLE_TERTI
-                cur.execute("""
-                    SELECT COUNT(*) FROM USER_TABLES 
-                    WHERE TABLE_NAME = 'ARTICOLE_TERTI'
-                """)
-                table_exists = cur.fetchone()[0] > 0
-                
-                if not table_exists:
-                    return jsonify({
-                        "success": False, 
-                        "error": "Tabelul ARTICOLE_TERTI nu există. Rulează 01_create_table.sql"
-                    })
-                
-                # Count records
-                cur.execute("SELECT COUNT(*) FROM ARTICOLE_TERTI")
-                record_count = cur.fetchone()[0]
-                
-                return jsonify({
-                    "success": True,
-                    "message": f"DB Time: {db_date}, Records: {record_count}",
-                    "table_exists": table_exists,
-                    "record_count": record_count
-                })
-                
-    except Exception as e:
-        logger.error(f"Test DB failed: {e}")
-        return jsonify({"success": False, "error": str(e)})
-
-@app.route('/api/mappings')
-def get_mappings():
-    """Returnează toate mapările SKU active"""
-    try:
-        with pool.acquire() as con:
-            with con.cursor() as cur:
-                cur.execute("""
-                    SELECT sku, codmat, cantitate_roa, procent_pret, activ, data_creare
-                    FROM ARTICOLE_TERTI 
-                    ORDER BY sku, codmat
-                """)
-                mappings = cur.fetchall()
-                
-                return jsonify({
-                    "success": True, 
-                    "mappings": mappings,
-                    "count": len(mappings)
-                })
-                
-    except Exception as e:
-        logger.error(f"Get mappings failed: {e}")
-        return jsonify({"success": False, "error": str(e)})
-
-# Inițializare pool la startup
-try:
-    pool = start_pool()
-    logger.info("Admin interface started successfully")
-except Exception as e:
-    logger.error(f"Failed to start admin interface: {e}")
-    pool = None
-
-if __name__ == '__main__':
-    app.run(host='0.0.0.0', port=5000, debug=True)

api/app/services/mapping_service.py

@@ -125,16 +125,6 @@ def create_mapping(sku: str, codmat: str, cantitate_roa: float = 1, auto_restore
             if cur.fetchone()[0] == 0:
                 raise HTTPException(status_code=400, detail="CODMAT-ul nu exista in nomenclator")
 
-            # Warn if SKU is already a direct CODMAT in NOM_ARTICOLE
-            if sku == codmat:
-                cur.execute("""
-                    SELECT COUNT(*) FROM NOM_ARTICOLE
-                    WHERE codmat = :sku AND sters = 0 AND inactiv = 0
-                """, {"sku": sku})
-                if cur.fetchone()[0] > 0:
-                    raise HTTPException(status_code=409,
-                                        detail="SKU-ul exista direct in nomenclator ca CODMAT, nu necesita mapare")
-
             # Check for active duplicate
             cur.execute("""
                 SELECT COUNT(*) FROM ARTICOLE_TERTI

api/app/services/price_sync_service.py

@@ -154,6 +154,12 @@ async def run_catalog_price_sync(run_id: str):
                         if is_kit:
                             for comp in mapped_data[sku]:
                                 comp_codmat = comp["codmat"]
+
+                                # Skip components that have their own ARTICOLE_TERTI mapping
+                                # (they'll be synced with correct cantitate_roa in individual path)
+                                if comp_codmat in mapped_data:
+                                    continue
+
                                 comp_product = products_by_sku.get(comp_codmat)
                                 if not comp_product:
                                     continue  # Component not in GoMag as standalone product

api/app/services/sqlite_service.py

@@ -240,6 +240,23 @@ async def track_missing_sku(sku: str, product_name: str = "",
         await db.close()
 
 
+async def resolve_missing_skus_batch(skus: set):
+    """Mark multiple missing SKUs as resolved (they now have mappings)."""
+    if not skus:
+        return 0
+    db = await get_sqlite()
+    try:
+        placeholders = ",".join("?" for _ in skus)
+        cursor = await db.execute(f"""
+            UPDATE missing_skus SET resolved = 1, resolved_at = datetime('now')
+            WHERE sku IN ({placeholders}) AND resolved = 0
+        """, list(skus))
+        await db.commit()
+        return cursor.rowcount
+    finally:
+        await db.close()
+
+
 async def resolve_missing_sku(sku: str):
     """Mark a missing SKU as resolved."""
     db = await get_sqlite()

api/app/services/sync_service.py

@@ -410,6 +410,13 @@ async def run_sync(id_pol: int = None, id_sectie: int = None, run_id: str = None
                         customers=json.dumps(ctx.get("customers", [])) if ctx.get("customers") else None,
                     )
 
+                # Auto-resolve missing SKUs that now have mappings
+                resolved_skus = validation["mapped"] | validation["direct"]
+                if resolved_skus:
+                    resolved_count = await sqlite_service.resolve_missing_skus_batch(resolved_skus)
+                    if resolved_count:
+                        _log_line(run_id, f"Auto-resolved {resolved_count} previously missing SKUs")
+
                 # Step 2d: Pre-validate prices for importable articles
                 if id_pol and (truly_importable or already_in_roa):
                     _update_progress("validation", "Validating prices...", 0, len(truly_importable))

api/app/services/validation_service.py

@@ -542,8 +542,9 @@ def sync_prices_from_order(orders, mapped_codmat_data: dict, direct_id_map: dict
     kit_discount_codmat = (settings or {}).get("kit_discount_codmat", "")
     skip_codmats = {transport_codmat, discount_codmat, kit_discount_codmat} - {""}
 
-    # Build set of kit SKUs (>1 component)
+    # Build set of kit/bax SKUs (>1 component, or single component with cantitate_roa > 1)
-    kit_skus = {sku for sku, comps in mapped_codmat_data.items() if len(comps) > 1}
+    kit_skus = {sku for sku, comps in mapped_codmat_data.items()
+                if len(comps) > 1 or (len(comps) == 1 and (comps[0].get("cantitate_roa") or 1) > 1)}
 
     updated = []
     own_conn = conn is None

api/app/static/js/dashboard.js

@@ -747,10 +747,24 @@ function renderReceipt(items, order) {
         return;
     }
 
+    const articole = items.reduce((s, i) => s + Number(i.price || 0) * Number(i.quantity || 0), 0);
+    const discount = Number(order.discount_total || 0);
+    const transport = Number(order.delivery_cost || 0);
     const total = order.order_total != null ? fmtNum(order.order_total) : '-';
-    const html = `<span><strong>Total: ${total} lei</strong></span>`;
+
-    desktop.innerHTML = html;
+    // Desktop: full labels
-    mobile.innerHTML = html;
+    let dHtml = `<span class="text-muted">Articole: <strong class="text-body">${fmtNum(articole)}</strong></span>`;
+    if (discount > 0) dHtml += `<span class="text-muted">Discount: <strong class="text-danger">\u2013${fmtNum(discount)}</strong></span>`;
+    if (transport > 0) dHtml += `<span class="text-muted">Transport: <strong class="text-body">${fmtNum(transport)}</strong></span>`;
+    dHtml += `<span>Total: <strong>${total} lei</strong></span>`;
+    desktop.innerHTML = dHtml;
+
+    // Mobile: shorter labels
+    let mHtml = `<span class="text-muted">Art: <strong class="text-body">${fmtNum(articole)}</strong></span>`;
+    if (discount > 0) mHtml += `<span class="text-muted">Disc: <strong class="text-danger">\u2013${fmtNum(discount)}</strong></span>`;
+    if (transport > 0) mHtml += `<span class="text-muted">Transp: <strong class="text-body">${fmtNum(transport)}</strong></span>`;
+    mHtml += `<span>Total: <strong>${total} lei</strong></span>`;
+    mobile.innerHTML = mHtml;
 }
 
 // ── Quick Map Modal (uses shared openQuickMap) ───

api/app/templates/dashboard.html

@@ -168,5 +168,5 @@
 {% endblock %}
 
 {% block scripts %}
-<script src="{{ request.scope.get('root_path', '') }}/static/js/dashboard.js?v=24"></script>
+<script src="{{ request.scope.get('root_path', '') }}/static/js/dashboard.js?v=25"></script>
 {% endblock %}

api/database-scripts/06_pack_import_comenzi.pck

@@ -1,72 +1,3 @@
--- ====================================================================
--- PACK_IMPORT_COMENZI
--- Package pentru importul comenzilor din platforme web (GoMag, etc.)
--- in sistemul ROA Oracle.
---
--- Dependinte:
---   Packages: PACK_COMENZI (adauga_comanda, adauga_articol_comanda)
---             pljson (pljson_list, pljson) - instalat in CONTAFIN_ORACLE,
---               accesat prin PUBLIC SYNONYM
---   Tabele:   ARTICOLE_TERTI (mapari SKU -> CODMAT)
---             NOM_ARTICOLE (nomenclator articole ROA)
---             COMENZI (verificare duplicat comanda_externa)
---             CRM_POLITICI_PRETURI (flag PRETURI_CU_TVA per politica)
---             CRM_POLITICI_PRET_ART (preturi componente kituri)
---
--- Proceduri publice:
---
---   importa_comanda(...)
---     Importa o comanda completa: creeaza comanda + adauga articolele.
---     p_json_articole accepta:
---       - array JSON:  [{"sku":"X","quantity":"1","price":"10","vat":"19"}, ...]
---       - obiect JSON:  {"sku":"X","quantity":"1","price":"10","vat":"19"}
---     Optional per articol: "id_pol":"5" — politica de pret specifica
---       (pentru transport/discount cu politica separata de cea a comenzii)
---     Valorile sku, quantity, price, vat sunt extrase ca STRING si convertite.
---     Daca comanda exista deja (comanda_externa), nu se dubleaza.
---     La eroare ridica RAISE_APPLICATION_ERROR(-20001, mesaj).
---     Returneaza v_id_comanda (OUT) = ID-ul comenzii create.
---
---     Parametri kit pricing:
---       p_kit_mode            — 'distributed' | 'separate_line' | NULL
---         distributed: discountul fata de suma componentelor se distribuie
---           proportional in pretul fiecarei componente
---         separate_line: componentele se insereaza la pret plin +
---           linii discount separate grupate pe cota TVA
---       p_id_pol_productie    — politica de pret pentru articole de productie
---         (cont in 341/345); NULL = nu se foloseste
---       p_kit_discount_codmat — CODMAT-ul articolului discount (Mode separate_line)
---       p_kit_discount_id_pol — id_pol pentru liniile discount (Mode separate_line)
---
---     Logica cautare articol per SKU:
---       1. Mapari speciale din ARTICOLE_TERTI (reimpachetare, seturi compuse)
---          - daca SKU are >1 rand si p_kit_mode IS NOT NULL: kit pricing logic
---          - altfel (1 rand sau kit_mode NULL): pret web / cantitate_roa direct
---       2. Fallback: cautare directa in NOM_ARTICOLE dupa CODMAT = SKU
---
---   get_last_error / clear_error
---     Management erori pentru orchestratorul VFP.
---
--- Exemplu utilizare:
---   DECLARE
---     v_id NUMBER;
---   BEGIN
---     PACK_IMPORT_COMENZI.importa_comanda(
---       p_nr_comanda_ext => '479317993',
---       p_data_comanda   => SYSDATE,
---       p_id_partener    => 1424,
---       p_json_articole  => '[{"sku":"5941623003366","quantity":"1.00","price":"40.99","vat":"21"}]',
---       p_id_pol         => 39,
---       v_id_comanda     => v_id);
---     DBMS_OUTPUT.PUT_LINE('ID comanda: ' || v_id);
---   END;
--- 20.03.2026 - dual policy vanzare/productie, kit pricing distributed/separate_line, SKU→CODMAT via ARTICOLE_TERTI
--- 20.03.2026 - kit discount deferred cross-kit (separate_line, merge-on-collision)
--- 20.03.2026 - merge_or_insert_articol: merge cantitati cand kit+individual au acelasi articol/pret
--- 20.03.2026 - kit pricing extins pt reambalari single-component (cantitate_roa > 1)
--- 21.03.2026 - diagnostic detaliat discount kit (id_pol, id_art, codmat in eroare)
--- 21.03.2026 - fix discount amount: v_disc_amt e per-kit, nu se imparte la v_cantitate_web
--- ====================================================================
 CREATE OR REPLACE PACKAGE PACK_IMPORT_COMENZI AS
 
   -- Variabila package pentru ultima eroare (pentru orchestrator VFP)
@@ -96,6 +27,31 @@ END PACK_IMPORT_COMENZI;
 /
 CREATE OR REPLACE PACKAGE BODY PACK_IMPORT_COMENZI AS
 
+  -- ====================================================================
+  -- PACK_IMPORT_COMENZI
+  -- Package pentru importul comenzilor din platforme web (GoMag, etc.)
+  -- in sistemul ROA Oracle.
+  --
+  -- Dependinte:
+  --   Packages: PACK_COMENZI (adauga_comanda, adauga_articol_comanda)
+  --             pljson (pljson_list, pljson) - instalat in CONTAFIN_ORACLE,
+  --               accesat prin PUBLIC SYNONYM
+  --   Tabele:   ARTICOLE_TERTI (mapari SKU -> CODMAT)
+  --             NOM_ARTICOLE (nomenclator articole ROA)
+  --             COMENZI (verificare duplicat comanda_externa)
+  --             CRM_POLITICI_PRETURI (flag PRETURI_CU_TVA per politica)
+  --             CRM_POLITICI_PRET_ART (preturi componente kituri)
+
+  -- 20.03.2026 - dual policy vanzare/productie, kit pricing distributed/separate_line, SKU→CODMAT via ARTICOLE_TERTI
+  -- 20.03.2026 - kit discount deferred cross-kit (separate_line, merge-on-collision)
+  -- 20.03.2026 - merge_or_insert_articol: merge cantitati cand kit+individual au acelasi articol/pret
+  -- 20.03.2026 - kit pricing extins pt reambalari single-component (cantitate_roa > 1)
+  -- 21.03.2026 - diagnostic detaliat discount kit (id_pol, id_art, codmat in eroare)
+  -- 21.03.2026 - fix discount amount: v_disc_amt e per-kit, nu se imparte la v_cantitate_web
+  -- 25.03.2026 - skip negative kit discount (markup), ROUND prices to nzecimale_pretv
+  -- 25.03.2026 - kit discount inserat per-kit sub componente (nu deferred cross-kit)
+  -- ====================================================================
+
   -- Constante pentru configurare
   c_id_util   CONSTANT NUMBER := -3; -- Sistem
   c_interna   CONSTANT NUMBER := 2; -- Comenzi de la client (web)
@@ -273,16 +229,8 @@ CREATE OR REPLACE PACKAGE BODY PACK_IMPORT_COMENZI AS
     v_pret_ajustat       NUMBER;
     v_discount_allocated NUMBER;
 
-    -- Acumulare discount-uri kit cross-kit (separate_line, deferred insertion)
+    -- Zecimale pret vanzare (din optiuni firma, default 2)
-    TYPE t_kit_disc_entry IS RECORD (
+    v_nzec_pretv PLS_INTEGER := NVL(TO_NUMBER(pack_sesiune.getoptiunefirma(USER, 'PPRETV')), 2);
-      ptva  NUMBER,
-      pret  NUMBER,   -- pret unitar (disc_amt / cantitate_web)
-      qty   NUMBER    -- cantitate negativa acumulata
-    );
-    TYPE t_kit_disc_list IS TABLE OF t_kit_disc_entry INDEX BY PLS_INTEGER;
-    v_kit_disc_list  t_kit_disc_list;
-    v_kit_disc_count PLS_INTEGER := 0;
-    v_kit_disc_found BOOLEAN;
 
     -- pljson
     l_json_articole      CLOB := p_json_articole;
@@ -492,8 +440,10 @@ CREATE OR REPLACE PACKAGE BODY PACK_IMPORT_COMENZI AS
                 END IF;
 
                 -- pret_ajustat = pret_cu_tva - discount_share / cantitate_roa
-                v_pret_ajustat := v_kit_comps(i_comp).pret_cu_tva -
+                v_pret_ajustat := ROUND(
-                  (v_discount_share / v_kit_comps(i_comp).cantitate_roa);
+                  v_kit_comps(i_comp).pret_cu_tva -
+                    (v_discount_share / v_kit_comps(i_comp).cantitate_roa),
+                  v_nzec_pretv);
 
                 BEGIN
                   merge_or_insert_articol(
@@ -517,14 +467,13 @@ CREATE OR REPLACE PACKAGE BODY PACK_IMPORT_COMENZI AS
             END LOOP;
 
           ELSIF p_kit_mode = 'separate_line' THEN
-            -- Mode B: componente la pret plin, discount deferred cross-kit
+            -- Mode B: componente la pret plin, discount per-kit imediat sub componente
             DECLARE
               TYPE t_vat_discount IS TABLE OF NUMBER INDEX BY PLS_INTEGER;
               v_vat_disc       t_vat_discount;
               v_vat_key        PLS_INTEGER;
               v_vat_disc_alloc NUMBER;
               v_disc_amt       NUMBER;
-              v_unit_pret      NUMBER;
             BEGIN
               -- Inserare componente la pret plin + acumulare discount pe cota TVA (per kit)
               FOR i_comp IN 1 .. v_kit_comps.COUNT LOOP
@@ -566,7 +515,13 @@ CREATE OR REPLACE PACKAGE BODY PACK_IMPORT_COMENZI AS
                 END IF;
               END LOOP;
 
-              -- Merge per-kit discounts into cross-kit list (v_kit_disc_list)
+              -- Inserare imediata discount per kit (sub componentele kitului)
+              IF v_discount_total > 0 AND p_kit_discount_codmat IS NOT NULL THEN
+                DECLARE
+                  v_disc_artid NUMBER;
+                BEGIN
+                  v_disc_artid := resolve_id_articol(p_kit_discount_codmat, p_id_gestiune);
+                  IF v_disc_artid IS NOT NULL THEN
                     v_vat_disc_alloc := 0;
                     v_vat_key := v_vat_disc.FIRST;
                     WHILE v_vat_key IS NOT NULL LOOP
@@ -578,30 +533,32 @@ CREATE OR REPLACE PACKAGE BODY PACK_IMPORT_COMENZI AS
                         v_vat_disc_alloc := v_vat_disc_alloc + v_disc_amt;
                       END IF;
 
-                IF v_disc_amt != 0 THEN
+                      IF v_disc_amt > 0 THEN
-                  v_unit_pret := v_disc_amt;
+                        BEGIN
-
+                          PACK_COMENZI.adauga_articol_comanda(
-                  -- Search for existing entry with same (ptva, pret) to merge qty
+                            V_ID_COMANDA => v_id_comanda,
-                  v_kit_disc_found := FALSE;
+                            V_ID_ARTICOL => v_disc_artid,
-                  FOR j IN 1 .. v_kit_disc_count LOOP
+                            V_ID_POL     => NVL(p_kit_discount_id_pol, p_id_pol),
-                    IF v_kit_disc_list(j).ptva = v_vat_key
+                            V_CANTITATE  => -1 * v_cantitate_web,
-                       AND v_kit_disc_list(j).pret = v_unit_pret THEN
+                            V_PRET       => ROUND(v_disc_amt, v_nzec_pretv),
-                      v_kit_disc_list(j).qty := v_kit_disc_list(j).qty + (-1 * v_cantitate_web);
+                            V_ID_UTIL    => c_id_util,
-                      v_kit_disc_found := TRUE;
+                            V_ID_SECTIE  => p_id_sectie,
-                      EXIT;
+                            V_PTVA       => v_vat_key);
-                    END IF;
+                          v_articole_procesate := v_articole_procesate + 1;
-                  END LOOP;
+                        EXCEPTION
-
+                          WHEN OTHERS THEN
-                  IF NOT v_kit_disc_found THEN
+                            v_articole_eroare := v_articole_eroare + 1;
-                    v_kit_disc_count := v_kit_disc_count + 1;
+                            g_last_error := g_last_error || CHR(10) ||
-                    v_kit_disc_list(v_kit_disc_count).ptva := v_vat_key;
+                              'Eroare linie discount kit TVA=' || v_vat_key ||
-                    v_kit_disc_list(v_kit_disc_count).pret := v_unit_pret;
+                              '% codmat=' || p_kit_discount_codmat || ': ' || SQLERRM;
-                    v_kit_disc_list(v_kit_disc_count).qty  := -1 * v_cantitate_web;
+                        END;
-                  END IF;
                       END IF;
 
                       v_vat_key := v_vat_disc.NEXT(v_vat_key);
                     END LOOP;
+                  END IF;
+                END;
+              END IF;
             END; -- end mode B per-kit block
           END IF; -- end kit mode branching
 
@@ -686,43 +643,6 @@ CREATE OR REPLACE PACKAGE BODY PACK_IMPORT_COMENZI AS
 
     END LOOP;
 
-    -- ============================================================
-    -- INSERARE DISCOUNT-URI KIT DEFERRED (separate_line)
-    -- Linii cu preturi diferite raman separate, coliziuni merged pe qty
-    -- ============================================================
-    IF p_kit_mode = 'separate_line' AND v_kit_disc_count > 0 THEN
-      DECLARE
-        v_disc_artid NUMBER;
-      BEGIN
-        v_disc_artid := resolve_id_articol(p_kit_discount_codmat, p_id_gestiune);
-
-        IF v_disc_artid IS NOT NULL THEN
-          FOR j IN 1 .. v_kit_disc_count LOOP
-            BEGIN
-              PACK_COMENZI.adauga_articol_comanda(
-                V_ID_COMANDA => v_id_comanda,
-                V_ID_ARTICOL => v_disc_artid,
-                V_ID_POL     => NVL(p_kit_discount_id_pol, p_id_pol),
-                V_CANTITATE  => v_kit_disc_list(j).qty,
-                V_PRET       => v_kit_disc_list(j).pret,
-                V_ID_UTIL    => c_id_util,
-                V_ID_SECTIE  => p_id_sectie,
-                V_PTVA       => v_kit_disc_list(j).ptva);
-              v_articole_procesate := v_articole_procesate + 1;
-            EXCEPTION
-              WHEN OTHERS THEN
-                v_articole_eroare := v_articole_eroare + 1;
-                g_last_error := g_last_error || CHR(10) ||
-                  'Eroare linie discount kit TVA=' || v_kit_disc_list(j).ptva ||
-                  '% id_pol=' || NVL(p_kit_discount_id_pol, p_id_pol) ||
-                  ' id_art=' || v_disc_artid ||
-                  ' codmat=' || p_kit_discount_codmat || ': ' || SQLERRM;
-            END;
-          END LOOP;
-        END IF;
-      END;
-    END IF;
-
     -- Verifica daca s-au procesat articole cu succes
     IF v_articole_procesate = 0 THEN
       g_last_error := g_last_error || CHR(10) || 'IMPORTA_COMANDA ' ||

api/database-scripts/08_merge_kituri.sql

@@ -1,69 +0,0 @@
--- ====================================================================
--- Import mapari kituri (seturi cu componente multiple) in ARTICOLE_TERTI
--- Sursa: kituri site.csv
--- Data: 2026-03-20
--- Schema: VENDING (productie)
--- ====================================================================
-
--- Kit revizie grup Wittenborg 7100
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5002' sku, '2517572' codmat, 3 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5002' sku, '252538' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5002' sku, '094594' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-
--- Kit revizie garnituri grup Wittenborg 7100 originale
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5002-2331' sku, '251757' codmat, 3 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5002-2331' sku, '252538' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5002-2331' sku, '094594' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-
--- Kit revizie boiler Necta 300cc
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5003' sku, '098701' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5003' sku, '099059' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5003' sku, '254711' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5003' sku, '252538' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5003' sku, '095624' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-
--- Kit revizie garnituri boiler Necta Astro Spazio 600cc
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5004-5988' sku, 'DV099748' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5004-5988' sku, '252538' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5004-5988' sku, '254711' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5004-5988' sku, '095624' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-
--- Kit revizie grup Necta 7gr
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5000' sku, '093167' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5000' sku, '252538' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5000' sku, '094611' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-
--- Kit revizie grup Necta 9gr
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5001' sku, '2517572' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5001' sku, '252538' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5001' sku, '094611' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-
--- Kit butoane selectie zahar Necta Astro Zenith
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '0V2071/250159/250158' sku, '0V2071' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '0V2071/250159/250158' sku, '250158' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '0V2071/250159/250158' sku, '250159' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-
--- Kit revizie grup Necta Opera/9100 D38
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5008890' sku, '252538' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5008890' sku, '094611' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5008890' sku, '093167' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-
--- Kit revizie grup Necta Opera/9100 D46
-MERGE INTO ARTICOLE_TERTI t USING (SELECT 'VM515' sku, '251757' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT 'VM515' sku, '252538' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT 'VM515' sku, '094611' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT 'VM515' sku, '0V0782' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT 'VM515' sku, '254650' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-
--- Kit revizie boiler Necta/Wittenborg 600cc
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5004' sku, '099059' codmat, 3 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5004' sku, 'DV099748' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5004' sku, '254711' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5004' sku, '095624' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5004' sku, '252538' codmat, 1 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-
--- Kit revizie rasnita Necta
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5006' sku, '095840' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-MERGE INTO ARTICOLE_TERTI t USING (SELECT '5006' sku, '0V3229' codmat, 2 cantitate_roa FROM DUAL) s ON (t.sku = s.sku AND t.codmat = s.codmat) WHEN MATCHED THEN UPDATE SET t.cantitate_roa = s.cantitate_roa, t.activ = 1, t.sters = 0, t.data_modif = SYSDATE WHEN NOT MATCHED THEN INSERT (sku, codmat, cantitate_roa, activ, sters, data_creare, data_modif, id_util_creare) VALUES (s.sku, s.codmat, s.cantitate_roa, 1, 0, SYSDATE, SYSDATE, -3);
-
-COMMIT;

api/database-scripts/09_articole_terti_050.sql

@@ -0,0 +1,54 @@
+-- ====================================================================
+-- 09_articole_terti_050.sql
+-- Mapări ARTICOLE_TERTI cu cantitate_roa = 0.5 pentru articole
+-- unde unitatea web (50 buc/set) ≠ unitatea ROA (100 buc/set).
+--
+-- Efect: price sync va calcula pret_crm = pret_web / 0.5,
+--   iar kit pricing va folosi prețul corect per set ROA.
+--
+-- 25.03.2026 - creat pentru fix discount negativ kit pahare
+-- ====================================================================
+
+-- Pahar 6oz Coffee Coffee SIBA 50buc (GoMag) → 100buc/set (ROA)
+INSERT INTO articole_terti (sku, codmat, cantitate_roa, activ, sters, data_creare, id_util_creare)
+SELECT '1708828', '1708828', 0.5, 1, 0, SYSDATE, -3 FROM dual
+WHERE NOT EXISTS (
+  SELECT 1 FROM articole_terti WHERE sku = '1708828' AND codmat = '1708828' AND sters = 0
+);
+
+-- Pahar 8oz Coffee Coffee SIBA 50buc → 100buc/set
+INSERT INTO articole_terti (sku, codmat, cantitate_roa, activ, sters, data_creare, id_util_creare)
+SELECT '528795', '528795', 0.5, 1, 0, SYSDATE, -3 FROM dual
+WHERE NOT EXISTS (
+  SELECT 1 FROM articole_terti WHERE sku = '528795' AND codmat = '528795' AND sters = 0
+);
+
+-- Pahar 8oz Tchibo 50buc → 100buc/set
+INSERT INTO articole_terti (sku, codmat, cantitate_roa, activ, sters, data_creare, id_util_creare)
+SELECT '58', '58', 0.5, 1, 0, SYSDATE, -3 FROM dual
+WHERE NOT EXISTS (
+  SELECT 1 FROM articole_terti WHERE sku = '58' AND codmat = '58' AND sters = 0
+);
+
+-- Pahar 7oz Lavazza SIBA 50buc → 100buc/set
+INSERT INTO articole_terti (sku, codmat, cantitate_roa, activ, sters, data_creare, id_util_creare)
+SELECT '51', '51', 0.5, 1, 0, SYSDATE, -3 FROM dual
+WHERE NOT EXISTS (
+  SELECT 1 FROM articole_terti WHERE sku = '51' AND codmat = '51' AND sters = 0
+);
+
+-- Pahar 8oz Albastru JND 50buc → 100buc/set
+INSERT INTO articole_terti (sku, codmat, cantitate_roa, activ, sters, data_creare, id_util_creare)
+SELECT '105712338826', '105712338826', 0.5, 1, 0, SYSDATE, -3 FROM dual
+WHERE NOT EXISTS (
+  SELECT 1 FROM articole_terti WHERE sku = '105712338826' AND codmat = '105712338826' AND sters = 0
+);
+
+-- Pahar 8oz Paris JND 50buc → 100buc/set
+INSERT INTO articole_terti (sku, codmat, cantitate_roa, activ, sters, data_creare, id_util_creare)
+SELECT '10573080', '10573080', 0.5, 1, 0, SYSDATE, -3 FROM dual
+WHERE NOT EXISTS (
+  SELECT 1 FROM articole_terti WHERE sku = '10573080' AND codmat = '10573080' AND sters = 0
+);
+
+COMMIT;

api/database-scripts/mapari_sql.sql

@@ -1,79 +0,0 @@
--- =============================================================================
--- Script mapari articole GoMag → ROA
--- Generat: 2026-03-19
--- Baza: vending | Server: vending
--- =============================================================================
-
--- =============================================
--- PARTEA 1: Update CODMAT in NOM_ARTICOLE
--- =============================================
-
--- id=2020 LAVAZZA BBE EXPERT GUSTO FORTE — CODMAT lipseste (NULL)
-UPDATE nom_articole SET codmat = '8000070028685' WHERE id_articol = 2020 AND codmat IS NULL;
-
--- id=4345 MY POS SIGMA — lowercase ca sa fie identic cu SKU GoMag
-UPDATE nom_articole SET codmat = 'mypossigma' WHERE id_articol = 4345 AND codmat = 'MYPOSSIGMA';
-
--- =============================================
--- PARTEA 2: Mapari ARTICOLE_TERTI (sku != codmat)
--- =============================================
-
--- Fresso — EAN-uri diferite de codmat intern
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('5940031026295', 'FRSBRZ1000', 1, 1, 0);    -- Fresso Brazilia 1kg
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('5940031062538', 'FRSEVK1000', 1, 1, 0);    -- Fresso Evoke blend 1kg
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('5940031026325', 'FRSCLB1000', 1, 1, 0);    -- Fresso Columbia Caldas 1kg
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('5940031026356', 'FRSCRA1000', 1, 1, 0);    -- Fresso Costa Rica Tarrazu 1kg
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('5940031026462', 'FRSETP250', 1, 1, 0);     -- Fresso Etiopia 250g
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('5940031026479', 'FRSETP500', 1, 1, 0);     -- Fresso Etiopia 500g
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('5940031026486', 'FRSETP1000', 1, 1, 0);    -- Fresso Etiopia 1kg
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('5940031044138', 'FRSEVK250', 1, 1, 0);     -- Fresso Evoke blend 250g
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('59400310625381000MI', 'FRSEVK1000', 1, 1, 0); -- Fresso Evoke macinata 1kg
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('FBS500PE', 'FRSBRZ500', 1, 1, 0);          -- Fresso Brazilia 500g macinata
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('FEY250PI', 'FRSETP250', 1, 1, 0);          -- Fresso Etiopia 250g macinata
-
--- Tchibo / Lavazza / alte branduri — EAN-uri diferite
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('4006067176463', 'SUISSE500', 1, 1, 0);     -- Tchibo Cafe Creme Suisse 500g
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('69891863', '8000070038493', 1, 1, 0);      -- Lavazza Crema e Gusto Forte 1Kg
-
--- Piese / accesorii — coduri diferite
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('65221', '33.7006.5221', 1, 1, 0);          -- Pastile curatare Schaerer
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('C7774', 'COL100', 1, 1, 0);                -- Eticheta colant cu pret
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('MEICF7900', 'MEICF560', 1, 1, 0);          -- Restiera MEI Cashflow CF 7900
-
--- =============================================
--- PARTEA 3: Mapari ARTICOLE_TERTI — impachetari diferite (cantitate != 1)
--- =============================================
-
--- Prolait/Regilait/Ristora 500g — ROA tine in KG sau BUC, 500g = 0.5
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('8004990125530', '8004990125530', 0.5, 1, 0);  -- Prolait Topping Blue 500g (UM=KG)
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('3043937103250', '3043937103250', 0.5, 1, 0);  -- Regilait Topping 2 Green 500g (UM=KG)
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('8004990123680', '8004990123680', 0.5, 1, 0);  -- Ristora Top Lapte Granulat 500g
-
--- Pahare — baxuri mari (1 bax web = N seturi ROA de 100buc)
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('10008ozparis', '10573080', 10, 1, 0);         -- Pahar 8oz Paris bax 1000 = 10 seturi
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('100012ozlvzJND', '58912326634', 10, 1, 0);    -- Pahar 12oz Lavazza JND bax 1000 = 10 seturi
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('589123214745675', '8OZLRLP', 10, 1, 0);       -- Pahar 8oz Lavazza RLP bax 1000 = 10 seturi
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('10008ozTchibo', '58', 10, 1, 0);              -- Pahar 8oz Tchibo bax 1000 = 10 seturi
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('10008ozBlueJND', '105712338826', 10, 1, 0);   -- Pahar 8oz Albastru JND bax 1000 = 10 seturi
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('30006ozLavazza', '169', 30, 1, 0);            -- Pahar 6oz Lavazza RLP bax 3000 = 30 seturi
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('30007ozLavazza', '1655455', 30, 1, 0);        -- Pahar 7oz Lavazza RLP bax 3000 = 30 seturi
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('22507ozLavazza', '51', 22.5, 1, 0);           -- Pahar 7oz Lavazza SIBA bax 2250 = 22.5 seturi
-
--- Pahare — ambalaje mici (50buc = 0.5 set de 100)
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('5891232122239', '8OZLRLP', 0.5, 1, 0);        -- Pahar 8oz Albastru RLP 50buc = 0.5 set
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('87872376', '87872376', 0.5, 1, 0);            -- Pahar 7oz Lavazza JND 50buc = 0.5 set
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('6ozFloMAZ', '6OZFLOMAZ', 0.5, 1, 0);          -- Pahar 6oz Floral MAZ 50buc = 0.5 set
-
--- Pachet cafea
-INSERT INTO ARTICOLE_TERTI (sku, codmat, cantitate_roa, activ, sters) VALUES ('6ktcs', 'SUISSE500', 10, 1, 0);              -- Pachet 5kg Tchibo Suisse = 10x500g
-
-COMMIT;
-
--- =============================================
--- VERIFICARE
--- =============================================
--- SELECT at.sku, at.codmat, na.denumire, na.um, at.cantitate_roa, at.activ
--- FROM ARTICOLE_TERTI at
--- LEFT JOIN nom_articole na ON na.codmat = at.codmat AND na.sters = 0
--- WHERE at.sters = 0
--- ORDER BY at.sku;

api/tests/qa/test_qa_plsql.py

@@ -47,34 +47,39 @@ def test_order_id(oracle_connection):
     try:
         with conn.cursor() as cur:
             cur.execute(
-                "SELECT MIN(id_partener) FROM parteneri WHERE id_partener > 0"
+                "SELECT MIN(id_part) FROM nom_parteneri WHERE id_part > 0"
             )
             row = cur.fetchone()
             if not row or row[0] is None:
                 pytest.skip("No partners found in Oracle — cannot create test order")
             partner_id = int(row[0])
     except Exception as exc:
-        pytest.skip(f"Cannot query parteneri table: {exc}")
+        pytest.skip(f"Cannot query nom_parteneri table: {exc}")
 
-    # Build minimal JSON articles — use a SKU known from NOM_ARTICOLE if possible
+    # Find an article that has a price in some policy (required for import)
     with conn.cursor() as cur:
-        cur.execute(
+        cur.execute("""
-            "SELECT codmat FROM nom_articole WHERE rownum = 1"
+            SELECT na.codmat, cp.id_pol, cp.pret
-        )
+            FROM nom_articole na
+            JOIN crm_politici_pret_art cp ON cp.id_articol = na.id_articol
+            WHERE cp.pret > 0 AND na.codmat IS NOT NULL AND rownum = 1
+        """)
         row = cur.fetchone()
-        test_sku = row[0] if row else "CAFE100"
+        if not row:
+            pytest.skip("No articles with prices found in Oracle — cannot create test order")
+        test_sku, id_pol, test_price = row[0], int(row[1]), float(row[2])
 
     nr_comanda_ext = f"PYTEST-{int(time.time())}"
+    # Values must be strings — Oracle's JSON_OBJECT_T.get_string() returns NULL for numbers
     articles = json.dumps([{
         "sku": test_sku,
-        "cantitate": 1,
+        "quantity": "1",
-        "pret": 50.0,
+        "price": str(test_price),
-        "denumire": "Test article (pytest)",
+        "vat": "19",
-        "tva": 19,
-        "discount": 0,
     }])
 
     try:
+        from datetime import datetime
         with conn.cursor() as cur:
             clob_var = cur.var(oracledb.DB_TYPE_CLOB)
             clob_var.setvalue(0, articles)
@@ -82,12 +87,12 @@ def test_order_id(oracle_connection):
 
             cur.callproc("PACK_IMPORT_COMENZI.importa_comanda", [
                 nr_comanda_ext,      # p_nr_comanda_ext
-                None,            # p_data_comanda  (NULL = SYSDATE in pkg)
+                datetime.now(),      # p_data_comanda
                 partner_id,          # p_id_partener
                 clob_var,            # p_json_articole
                 None,                # p_id_adresa_livrare
                 None,                # p_id_adresa_facturare
-                None,            # p_id_pol
+                id_pol,              # p_id_pol
                 None,                # p_id_sectie
                 None,                # p_id_gestiune
                 None,                # p_kit_mode
@@ -122,11 +127,11 @@ def test_order_id(oracle_connection):
         try:
             with conn.cursor() as cur:
                 cur.execute(
-                    "DELETE FROM comenzi_articole WHERE id_comanda = :id",
+                    "DELETE FROM comenzi_elemente WHERE id_comanda = :id",
                     {"id": order_id}
                 )
                 cur.execute(
-                    "DELETE FROM com_antet WHERE id_comanda = :id",
+                    "DELETE FROM comenzi WHERE id_comanda = :id",
                     {"id": order_id}
                 )
             conn.commit()
@@ -193,7 +198,7 @@ def test_cleanup_test_order(oracle_connection, test_order_id):
 
     with oracle_connection.cursor() as cur:
         cur.execute(
-            "SELECT COUNT(*) FROM com_antet WHERE id_comanda = :id",
+            "SELECT COUNT(*) FROM comenzi WHERE id_comanda = :id",
             {"id": test_order_id}
         )
         row = cur.fetchone()

api/tests/qa/test_qa_responsive.py

@@ -119,7 +119,8 @@ def test_mobile_table_responsive(pw_browser, base_url: str, page_path: str):
 
         tables = page.locator("table").all()
         if not tables:
-            pytest.skip(f"No tables on {page_path} (empty state)")
+            # No tables means nothing to check — pass (no non-responsive tables exist)
+            return
 
         # Check each table has an ancestor with overflow-x scroll or .table-responsive class
         for table in tables:

api/tests/test_business_rules.py

@@ -0,0 +1,494 @@
+"""
+Business Rule Regression Tests
+==============================
+Regression tests for historical bug fixes in kit pricing, discount calculation,
+duplicate CODMAT resolution, price sync, and VAT normalization.
+
+Run:
+    cd api && python -m pytest tests/test_business_rules.py -v
+"""
+
+import json
+import os
+import sys
+import tempfile
+
+import pytest
+
+pytestmark = pytest.mark.unit
+
+# --- Set env vars BEFORE any app import ---
+_tmpdir = tempfile.mkdtemp()
+_sqlite_path = os.path.join(_tmpdir, "test_biz.db")
+
+os.environ["FORCE_THIN_MODE"] = "true"
+os.environ["SQLITE_DB_PATH"] = _sqlite_path
+os.environ["ORACLE_DSN"] = "dummy"
+os.environ["ORACLE_USER"] = "dummy"
+os.environ["ORACLE_PASSWORD"] = "dummy"
+os.environ["JSON_OUTPUT_DIR"] = _tmpdir
+
+_api_dir = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
+if _api_dir not in sys.path:
+    sys.path.insert(0, _api_dir)
+
+from unittest.mock import MagicMock, patch
+
+from app.services.import_service import build_articles_json, compute_discount_split
+from app.services.order_reader import OrderData, OrderItem
+
+
+# ---------------------------------------------------------------------------
+# Helpers
+# ---------------------------------------------------------------------------
+
+def make_item(sku="SKU1", price=100.0, quantity=1, vat=19):
+    return OrderItem(sku=sku, name=f"Product {sku}", price=price, quantity=quantity, vat=vat)
+
+
+def make_order(items, discount_total=0.0, delivery_cost=0.0, discount_vat=None):
+    order = OrderData(
+        id="1", number="TEST-001", date="2026-01-01",
+        items=items, discount_total=discount_total,
+        delivery_cost=delivery_cost,
+    )
+    if discount_vat is not None:
+        order.discount_vat = discount_vat
+    return order
+
+
+def is_kit(comps):
+    """Kit detection pattern used in validation_service and price_sync_service."""
+    return len(comps) > 1 or (
+        len(comps) == 1 and (comps[0].get("cantitate_roa") or 1) > 1
+    )
+
+
+# ===========================================================================
+# Group 1: compute_discount_split()
+# ===========================================================================
+
+class TestDiscountSplit:
+    """Regression: discount split by VAT rate (import_service.py:63)."""
+
+    def test_single_vat_rate(self):
+        order = make_order([make_item(vat=19), make_item("SKU2", vat=19)], discount_total=10.0)
+        result = compute_discount_split(order, {"split_discount_vat": "1"})
+        assert result == {"19": 10.0}
+
+    def test_multiple_vat_proportional(self):
+        items = [make_item("A", price=100, quantity=1, vat=19),
+                 make_item("B", price=50, quantity=1, vat=9)]
+        order = make_order(items, discount_total=15.0)
+        result = compute_discount_split(order, {"split_discount_vat": "1"})
+        assert result == {"9": 5.0, "19": 10.0}
+
+    def test_zero_returns_none(self):
+        order = make_order([make_item()], discount_total=0)
+        assert compute_discount_split(order, {"split_discount_vat": "1"}) is None
+
+    def test_zero_price_items_excluded(self):
+        items = [make_item("A", price=0, quantity=1, vat=19),
+                 make_item("B", price=100, quantity=2, vat=9)]
+        order = make_order(items, discount_total=5.0)
+        result = compute_discount_split(order, {"split_discount_vat": "1"})
+        assert result == {"9": 5.0}
+
+    def test_disabled_multiple_rates(self):
+        items = [make_item("A", vat=19), make_item("B", vat=9)]
+        order = make_order(items, discount_total=10.0)
+        result = compute_discount_split(order, {"split_discount_vat": "0"})
+        assert result is None
+
+    def test_rounding_remainder(self):
+        items = [make_item("A", price=33.33, quantity=1, vat=19),
+                 make_item("B", price=33.33, quantity=1, vat=9),
+                 make_item("C", price=33.34, quantity=1, vat=5)]
+        order = make_order(items, discount_total=10.0)
+        result = compute_discount_split(order, {"split_discount_vat": "1"})
+        assert result is not None
+        assert abs(sum(result.values()) - 10.0) < 0.001
+
+
+# ===========================================================================
+# Group 2: build_articles_json()
+# ===========================================================================
+
+class TestBuildArticlesJson:
+    """Regression: discount lines, policy bridge, transport (import_service.py:117)."""
+
+    def test_discount_line_negative_quantity(self):
+        items = [make_item()]
+        order = make_order(items, discount_total=5.0)
+        settings = {"discount_codmat": "DISC01", "split_discount_vat": "0"}
+        result = json.loads(build_articles_json(items, order, settings))
+        disc_lines = [a for a in result if a["sku"] == "DISC01"]
+        assert len(disc_lines) == 1
+        assert disc_lines[0]["quantity"] == "-1"
+        assert disc_lines[0]["price"] == "5.0"
+
+    def test_discount_uses_actual_vat_not_21(self):
+        items = [make_item("A", vat=9), make_item("B", vat=9)]
+        order = make_order(items, discount_total=3.0)
+        settings = {"discount_codmat": "DISC01", "split_discount_vat": "1"}
+        result = json.loads(build_articles_json(items, order, settings))
+        disc_lines = [a for a in result if a["sku"] == "DISC01"]
+        assert len(disc_lines) == 1
+        assert disc_lines[0]["vat"] == "9"
+
+    def test_discount_multi_vat_creates_multiple_lines(self):
+        items = [make_item("A", price=100, vat=19), make_item("B", price=50, vat=9)]
+        order = make_order(items, discount_total=15.0)
+        settings = {"discount_codmat": "DISC01", "split_discount_vat": "1"}
+        result = json.loads(build_articles_json(items, order, settings))
+        disc_lines = [a for a in result if a["sku"] == "DISC01"]
+        assert len(disc_lines) == 2
+        vats = {d["vat"] for d in disc_lines}
+        assert "9" in vats
+        assert "19" in vats
+
+    def test_discount_fallback_uses_gomag_vat(self):
+        items = [make_item("A", vat=19), make_item("B", vat=9)]
+        order = make_order(items, discount_total=5.0, discount_vat="9")
+        settings = {"discount_codmat": "DISC01", "split_discount_vat": "0"}
+        result = json.loads(build_articles_json(items, order, settings))
+        disc_lines = [a for a in result if a["sku"] == "DISC01"]
+        assert len(disc_lines) == 1
+        assert disc_lines[0]["vat"] == "9"
+
+    def test_per_article_policy_bridge(self):
+        items = [make_item("SKU1")]
+        settings = {"_codmat_policy_map": {"SKU1": 42}, "id_pol": "1"}
+        result = json.loads(build_articles_json(items, settings=settings))
+        assert result[0]["id_pol"] == "42"
+
+    def test_policy_same_as_default_omitted(self):
+        items = [make_item("SKU1")]
+        settings = {"_codmat_policy_map": {"SKU1": 1}, "id_pol": "1"}
+        result = json.loads(build_articles_json(items, settings=settings))
+        assert "id_pol" not in result[0]
+
+    def test_transport_line_added(self):
+        items = [make_item()]
+        order = make_order(items, delivery_cost=15.0)
+        settings = {"transport_codmat": "TR", "transport_vat": "19"}
+        result = json.loads(build_articles_json(items, order, settings))
+        tr_lines = [a for a in result if a["sku"] == "TR"]
+        assert len(tr_lines) == 1
+        assert tr_lines[0]["quantity"] == "1"
+        assert tr_lines[0]["price"] == "15.0"
+
+
+# ===========================================================================
+# Group 3: Kit Detection Pattern
+# ===========================================================================
+
+class TestKitDetection:
+    """Regression: kit detection for single-component repackaging (multiple code locations)."""
+
+    def test_multi_component(self):
+        comps = [{"codmat": "A", "cantitate_roa": 1}, {"codmat": "B", "cantitate_roa": 1}]
+        assert is_kit(comps) is True
+
+    def test_single_component_repackaging(self):
+        comps = [{"codmat": "CAF01", "cantitate_roa": 10}]
+        assert is_kit(comps) is True
+
+    def test_true_1to1_not_kit(self):
+        comps = [{"codmat": "X", "cantitate_roa": 1}]
+        assert is_kit(comps) is False
+
+    def test_none_cantitate_treated_as_1(self):
+        comps = [{"codmat": "X", "cantitate_roa": None}]
+        assert is_kit(comps) is False
+
+    def test_empty_components(self):
+        assert is_kit([]) is False
+
+
+# ===========================================================================
+# Group 4: sync_prices_from_order() — Kit Skip Logic
+# ===========================================================================
+
+class TestSyncPricesKitSkip:
+    """Regression: kit SKUs must be skipped in order-based price sync."""
+
+    def _make_mock_order(self, sku, price=50.0):
+        mock_order = MagicMock()
+        mock_item = MagicMock()
+        mock_item.sku = sku
+        mock_item.price = price
+        mock_order.items = [mock_item]
+        return mock_order
+
+    @patch("app.services.validation_service.compare_and_update_price")
+    def test_skips_multi_component_kit(self, mock_compare):
+        from app.services.validation_service import sync_prices_from_order
+        orders = [self._make_mock_order("KIT01")]
+        mapped = {"KIT01": [
+            {"codmat": "A", "id_articol": 1, "cantitate_roa": 1},
+            {"codmat": "B", "id_articol": 2, "cantitate_roa": 1},
+        ]}
+        mock_conn = MagicMock()
+        sync_prices_from_order(orders, mapped, {}, {}, 1, conn=mock_conn,
+                               settings={"price_sync_enabled": "1"})
+        mock_compare.assert_not_called()
+
+    @patch("app.services.validation_service.compare_and_update_price")
+    def test_skips_repackaging_kit(self, mock_compare):
+        from app.services.validation_service import sync_prices_from_order
+        orders = [self._make_mock_order("CAFE100")]
+        mapped = {"CAFE100": [
+            {"codmat": "CAF01", "id_articol": 1, "cantitate_roa": 10},
+        ]}
+        mock_conn = MagicMock()
+        sync_prices_from_order(orders, mapped, {}, {}, 1, conn=mock_conn,
+                               settings={"price_sync_enabled": "1"})
+        mock_compare.assert_not_called()
+
+    @patch("app.services.validation_service.compare_and_update_price")
+    def test_processes_1to1_mapping(self, mock_compare):
+        from app.services.validation_service import sync_prices_from_order
+        mock_compare.return_value = {"updated": False, "old_price": 50.0, "new_price": 50.0, "codmat": "X"}
+        orders = [self._make_mock_order("SKU1", price=50.0)]
+        mapped = {"SKU1": [
+            {"codmat": "X", "id_articol": 100, "cantitate_roa": 1, "cont": "371"},
+        ]}
+        mock_conn = MagicMock()
+        sync_prices_from_order(orders, mapped, {}, {"SKU1": 1}, 1, conn=mock_conn,
+                               settings={"price_sync_enabled": "1"})
+        mock_compare.assert_called_once()
+        call_args = mock_compare.call_args
+        assert call_args[0][0] == 100  # id_articol
+        assert call_args[0][2] == 50.0  # price
+
+    @patch("app.services.validation_service.compare_and_update_price")
+    def test_skips_transport_discount_codmats(self, mock_compare):
+        from app.services.validation_service import sync_prices_from_order
+        orders = [self._make_mock_order("TRANSP", price=15.0)]
+        mock_conn = MagicMock()
+        sync_prices_from_order(orders, {}, {"TRANSP": {"id_articol": 99}}, {}, 1,
+                               conn=mock_conn,
+                               settings={"price_sync_enabled": "1",
+                                         "transport_codmat": "TRANSP",
+                                         "discount_codmat": "DISC"})
+        mock_compare.assert_not_called()
+
+
+# ===========================================================================
+# Group 5: Kit Component with Own Mapping
+# ===========================================================================
+
+class TestKitComponentOwnMapping:
+    """Regression: price_sync_service skips kit components that have their own ARTICOLE_TERTI mapping."""
+
+    def test_component_with_own_mapping_skipped(self):
+        """If comp_codmat is itself a key in mapped_data, it's skipped."""
+        mapped_data = {
+            "PACK-A": [{"codmat": "COMP-X", "id_articol": 1, "cantitate_roa": 1, "cont": "371"}],
+            "COMP-X": [{"codmat": "COMP-X", "id_articol": 1, "cantitate_roa": 1, "cont": "371"}],
+        }
+        # The check is: if comp_codmat in mapped_data: continue
+        comp_codmat = "COMP-X"
+        assert comp_codmat in mapped_data  # Should be skipped
+
+    def test_component_without_own_mapping_processed(self):
+        """If comp_codmat is NOT in mapped_data, it should be processed."""
+        mapped_data = {
+            "PACK-A": [{"codmat": "COMP-Y", "id_articol": 2, "cantitate_roa": 1, "cont": "371"}],
+        }
+        comp_codmat = "COMP-Y"
+        assert comp_codmat not in mapped_data  # Should be processed
+
+
+# ===========================================================================
+# Group 6: VAT Included Type Normalization
+# ===========================================================================
+
+class TestVatIncludedNormalization:
+    """Regression: GoMag returns vat_included as int 1 or string '1' (price_sync_service.py:144)."""
+
+    def _compute_price_cu_tva(self, product):
+        price = float(product.get("price", "0"))
+        vat = float(product.get("vat", "19"))
+        if str(product.get("vat_included", "1")) == "1":
+            return price
+        else:
+            return price * (1 + vat / 100)
+
+    def test_vat_included_int_1(self):
+        result = self._compute_price_cu_tva({"price": "100", "vat": "19", "vat_included": 1})
+        assert result == 100.0
+
+    def test_vat_included_str_1(self):
+        result = self._compute_price_cu_tva({"price": "100", "vat": "19", "vat_included": "1"})
+        assert result == 100.0
+
+    def test_vat_included_int_0(self):
+        result = self._compute_price_cu_tva({"price": "100", "vat": "19", "vat_included": 0})
+        assert result == 119.0
+
+    def test_vat_included_str_0(self):
+        result = self._compute_price_cu_tva({"price": "100", "vat": "19", "vat_included": "0"})
+        assert result == 119.0
+
+
+# ===========================================================================
+# Group 7: validate_kit_component_prices — pret=0 allowed
+# ===========================================================================
+
+class TestKitComponentPriceValidation:
+    """Regression: pret=0 in CRM is valid for kit components (validation_service.py:469)."""
+
+    def _call_validate(self, fetchone_returns):
+        from app.services.validation_service import validate_kit_component_prices
+
+        mock_conn = MagicMock()
+        mock_cursor = MagicMock()
+        mock_conn.cursor.return_value.__enter__ = MagicMock(return_value=mock_cursor)
+        mock_conn.cursor.return_value.__exit__ = MagicMock(return_value=False)
+        mock_cursor.fetchone.return_value = fetchone_returns
+
+        mapped = {"KIT-SKU": [
+            {"codmat": "COMP1", "id_articol": 100, "cont": "371", "cantitate_roa": 5},
+        ]}
+        return validate_kit_component_prices(mapped, id_pol=1, conn=mock_conn)
+
+    def test_price_zero_not_rejected(self):
+        result = self._call_validate((0,))
+        assert result == {}
+
+    def test_missing_entry_rejected(self):
+        result = self._call_validate(None)
+        assert "KIT-SKU" in result
+        assert "COMP1" in result["KIT-SKU"]
+
+    def test_skips_true_1to1(self):
+        from app.services.validation_service import validate_kit_component_prices
+        mock_conn = MagicMock()
+        mapped = {"SKU1": [
+            {"codmat": "X", "id_articol": 1, "cont": "371", "cantitate_roa": 1},
+        ]}
+        result = validate_kit_component_prices(mapped, id_pol=1, conn=mock_conn)
+        assert result == {}
+
+    def test_checks_repackaging(self):
+        """Single component with cantitate_roa > 1 should be checked."""
+        from app.services.validation_service import validate_kit_component_prices
+
+        mock_conn = MagicMock()
+        mock_cursor = MagicMock()
+        mock_conn.cursor.return_value.__enter__ = MagicMock(return_value=mock_cursor)
+        mock_conn.cursor.return_value.__exit__ = MagicMock(return_value=False)
+        mock_cursor.fetchone.return_value = (51.50,)
+
+        mapped = {"CAFE100": [
+            {"codmat": "CAF01", "id_articol": 100, "cont": "371", "cantitate_roa": 10},
+        ]}
+        result = validate_kit_component_prices(mapped, id_pol=1, conn=mock_conn)
+        assert result == {}
+        mock_cursor.execute.assert_called_once()
+
+
+# ===========================================================================
+# Group 8: Dual Policy Assignment
+# ===========================================================================
+
+class TestDualPolicyAssignment:
+    """Regression: cont 341/345 → production policy, others → sales (validation_service.py:282)."""
+
+    def _call_dual(self, codmats, direct_id_map, cursor_rows):
+        from app.services.validation_service import validate_and_ensure_prices_dual
+
+        mock_conn = MagicMock()
+        mock_cursor = MagicMock()
+        mock_conn.cursor.return_value.__enter__ = MagicMock(return_value=mock_cursor)
+        mock_conn.cursor.return_value.__exit__ = MagicMock(return_value=False)
+        # The code uses `for row in cur:` to iterate, not fetchall
+        mock_cursor.__iter__ = MagicMock(return_value=iter(cursor_rows))
+        # Mock ensure_prices to do nothing
+        with patch("app.services.validation_service.ensure_prices"):
+            return validate_and_ensure_prices_dual(
+                codmats, id_pol_vanzare=1, id_pol_productie=2,
+                conn=mock_conn, direct_id_map=direct_id_map
+            )
+
+    def test_cont_341_production(self):
+        result = self._call_dual(
+            {"COD1"},
+            {"COD1": {"id_articol": 100, "cont": "341"}},
+            []  # no existing prices
+        )
+        assert result["COD1"] == 2  # id_pol_productie
+
+    def test_cont_345_production(self):
+        result = self._call_dual(
+            {"COD1"},
+            {"COD1": {"id_articol": 100, "cont": "345"}},
+            []
+        )
+        assert result["COD1"] == 2
+
+    def test_other_cont_sales(self):
+        result = self._call_dual(
+            {"COD1"},
+            {"COD1": {"id_articol": 100, "cont": "371"}},
+            []
+        )
+        assert result["COD1"] == 1  # id_pol_vanzare
+
+    def test_existing_sales_preferred(self):
+        result = self._call_dual(
+            {"COD1"},
+            {"COD1": {"id_articol": 100, "cont": "345"}},
+            [(100, 1), (100, 2)]  # price exists in BOTH policies
+        )
+        assert result["COD1"] == 1  # sales preferred when both exist
+
+
+# ===========================================================================
+# Group 9: Duplicate CODMAT — resolve_codmat_ids
+# ===========================================================================
+
+class TestResolveCodmatIds:
+    """Regression: ROW_NUMBER dedup returns exactly 1 id_articol per CODMAT."""
+
+    @patch("app.services.validation_service.database")
+    def test_returns_one_per_codmat(self, mock_db):
+        from app.services.validation_service import resolve_codmat_ids
+
+        mock_conn = MagicMock()
+        mock_cursor = MagicMock()
+        mock_conn.cursor.return_value.__enter__ = MagicMock(return_value=mock_cursor)
+        mock_conn.cursor.return_value.__exit__ = MagicMock(return_value=False)
+        # Simulate ROW_NUMBER already deduped: 1 row per codmat
+        mock_cursor.__iter__ = MagicMock(return_value=iter([
+            ("COD1", 100, "345"),
+            ("COD2", 200, "341"),
+        ]))
+
+        result = resolve_codmat_ids({"COD1", "COD2"}, conn=mock_conn)
+        assert len(result) == 2
+        assert result["COD1"]["id_articol"] == 100
+        assert result["COD2"]["id_articol"] == 200
+
+    @patch("app.services.validation_service.database")
+    def test_resolve_mapped_one_per_sku_codmat(self, mock_db):
+        from app.services.validation_service import resolve_mapped_codmats
+
+        mock_conn = MagicMock()
+        mock_cursor = MagicMock()
+        mock_conn.cursor.return_value.__enter__ = MagicMock(return_value=mock_cursor)
+        mock_conn.cursor.return_value.__exit__ = MagicMock(return_value=False)
+        # 1 row per (sku, codmat) pair
+        mock_cursor.__iter__ = MagicMock(return_value=iter([
+            ("SKU1", "COD1", 100, "345", 10),
+            ("SKU1", "COD2", 200, "341", 1),
+        ]))
+
+        result = resolve_mapped_codmats({"SKU1"}, mock_conn)
+        assert "SKU1" in result
+        assert len(result["SKU1"]) == 2
+        codmats = [c["codmat"] for c in result["SKU1"]]
+        assert "COD1" in codmats
+        assert "COD2" in codmats

api/tests/test_complete_import.py

@@ -528,6 +528,379 @@ def test_repackaging_kit_pricing():
         return False
 
 
+# ===========================================================================
+# Group 10: Business Rule Regression Tests (Oracle integration)
+# ===========================================================================
+
+def _create_test_partner(cur, suffix):
+    """Helper: create a test partner and return its ID."""
+    partner_var = cur.var(oracledb.NUMBER)
+    name = f'Test BizRule {suffix}'
+    cur.execute("""
+        DECLARE v_id NUMBER;
+        BEGIN
+            v_id := PACK_IMPORT_PARTENERI.cauta_sau_creeaza_partener(
+                NULL, :name, 'JUD:Bucuresti;BUCURESTI;Str Test;1',
+                '0720000000', 'bizrule@test.com');
+            :result := v_id;
+        END;
+    """, {'name': name, 'result': partner_var})
+    return partner_var.getvalue()
+
+
+def _import_order(cur, order_number, partner_id, articles_json, kit_mode='separate_line', id_pol=1):
+    """Helper: call importa_comanda and return order ID."""
+    result_var = cur.var(oracledb.NUMBER)
+    cur.execute("""
+        DECLARE v_id NUMBER;
+        BEGIN
+            PACK_IMPORT_COMENZI.importa_comanda(
+                :order_number, SYSDATE, :partner_id,
+                :articles_json,
+                NULL, NULL,
+                :id_pol, NULL, NULL,
+                :kit_mode,
+                NULL, NULL, NULL,
+                v_id);
+            :result := v_id;
+        END;
+    """, {
+        'order_number': order_number,
+        'partner_id': partner_id,
+        'articles_json': articles_json,
+        'id_pol': id_pol,
+        'kit_mode': kit_mode,
+        'result': result_var
+    })
+    return result_var.getvalue()
+
+
+def _get_order_lines(cur, order_id):
+    """Helper: fetch COMENZI_ELEMENTE rows for an order."""
+    cur.execute("""
+        SELECT ce.CANTITATE, ce.PRET, na.CODMAT, ce.PTVA
+        FROM COMENZI_ELEMENTE ce
+        JOIN NOM_ARTICOLE na ON ce.ID_ARTICOL = na.ID_ARTICOL
+        WHERE ce.ID_COMANDA = :oid
+        ORDER BY ce.CANTITATE DESC, ce.PRET DESC
+    """, {'oid': order_id})
+    return cur.fetchall()
+
+
+def test_multi_kit_discount_merge():
+    """Regression (0666d6b): 2 identical kits at same VAT must merge discount lines,
+    not crash on duplicate check collision."""
+    print("\n" + "=" * 60)
+    print("TEST: Multi-kit discount merge (separate_line)")
+    print("=" * 60)
+
+    try:
+        with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
+            with conn.cursor() as cur:
+                suffix = f'{datetime.now().strftime("%H%M%S")}-{random.randint(1000, 9999)}'
+                setup_test_data(cur)
+                partner_id = _create_test_partner(cur, suffix)
+
+                # 2 identical CAFE100 kits: total web = 2 * 450 = 900
+                articles_json = '[{"sku": "CAFE100", "cantitate": 2, "pret": 450}]'
+                order_id = _import_order(cur, f'TEST-BIZ-MERGE-{suffix}', partner_id, articles_json)
+
+                assert order_id and order_id > 0, "Order import failed"
+                rows = _get_order_lines(cur, order_id)
+
+                art_lines = [r for r in rows if r[0] > 0]
+                disc_lines = [r for r in rows if r[0] < 0]
+                assert len(art_lines) >= 1, f"Expected article line(s), got {len(art_lines)}"
+                assert len(disc_lines) >= 1, f"Expected discount line(s), got {len(disc_lines)}"
+
+                total = sum(r[0] * r[1] for r in rows)
+                expected = 900.0
+                print(f"   Total: {total:.2f} (expected: {expected:.2f})")
+                assert abs(total - expected) < 0.02, f"Total {total:.2f} != expected {expected:.2f}"
+                print("   PASS")
+
+                conn.commit()
+                teardown_test_data(cur)
+                conn.commit()
+                return True
+
+    except Exception as e:
+        print(f"   FAIL: {e}")
+        import traceback
+        traceback.print_exc()
+        try:
+            with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
+                with conn.cursor() as cur:
+                    teardown_test_data(cur)
+                    conn.commit()
+        except:
+            pass
+        return False
+
+
+def test_kit_discount_per_kit_placement():
+    """Regression (580ca59): discount lines must appear after article lines (both present)."""
+    print("\n" + "=" * 60)
+    print("TEST: Kit discount per-kit placement")
+    print("=" * 60)
+
+    try:
+        with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
+            with conn.cursor() as cur:
+                suffix = f'{datetime.now().strftime("%H%M%S")}-{random.randint(1000, 9999)}'
+                setup_test_data(cur)
+                partner_id = _create_test_partner(cur, suffix)
+
+                articles_json = '[{"sku": "CAFE100", "cantitate": 1, "pret": 450}]'
+                order_id = _import_order(cur, f'TEST-BIZ-PLACE-{suffix}', partner_id, articles_json)
+
+                assert order_id and order_id > 0, "Order import failed"
+                rows = _get_order_lines(cur, order_id)
+
+                art_lines = [r for r in rows if r[0] > 0]
+                disc_lines = [r for r in rows if r[0] < 0]
+                print(f"   Article lines: {len(art_lines)}, Discount lines: {len(disc_lines)}")
+                assert len(art_lines) >= 1, "No article line found"
+                assert len(disc_lines) >= 1, "No discount line found — kit pricing did not activate"
+                print("   PASS")
+
+                conn.commit()
+                teardown_test_data(cur)
+                conn.commit()
+                return True
+
+    except Exception as e:
+        print(f"   FAIL: {e}")
+        import traceback
+        traceback.print_exc()
+        try:
+            with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
+                with conn.cursor() as cur:
+                    teardown_test_data(cur)
+                    conn.commit()
+        except:
+            pass
+        return False
+
+
+def test_repackaging_distributed_total_matches_web():
+    """Regression (61ae58e): distributed mode total must match web price exactly."""
+    print("\n" + "=" * 60)
+    print("TEST: Repackaging distributed total matches web")
+    print("=" * 60)
+
+    try:
+        with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
+            with conn.cursor() as cur:
+                suffix = f'{datetime.now().strftime("%H%M%S")}-{random.randint(1000, 9999)}'
+                setup_test_data(cur)
+                partner_id = _create_test_partner(cur, suffix)
+
+                # 3 packs @ 400 lei => total web = 1200
+                articles_json = '[{"sku": "CAFE100", "cantitate": 3, "pret": 400}]'
+                order_id = _import_order(cur, f'TEST-BIZ-DIST-{suffix}', partner_id,
+                                         articles_json, kit_mode='distributed')
+
+                assert order_id and order_id > 0, "Order import failed"
+                rows = _get_order_lines(cur, order_id)
+
+                # Distributed: single line with adjusted price
+                positive_lines = [r for r in rows if r[0] > 0]
+                assert len(positive_lines) == 1, f"Expected 1 line in distributed mode, got {len(positive_lines)}"
+
+                total = positive_lines[0][0] * positive_lines[0][1]
+                expected = 1200.0
+                print(f"   Line: qty={positive_lines[0][0]}, price={positive_lines[0][1]:.2f}")
+                print(f"   Total: {total:.2f} (expected: {expected:.2f})")
+                assert abs(total - expected) < 0.02, f"Total {total:.2f} != expected {expected:.2f}"
+                print("   PASS")
+
+                conn.commit()
+                teardown_test_data(cur)
+                conn.commit()
+                return True
+
+    except Exception as e:
+        print(f"   FAIL: {e}")
+        import traceback
+        traceback.print_exc()
+        try:
+            with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
+                with conn.cursor() as cur:
+                    teardown_test_data(cur)
+                    conn.commit()
+        except:
+            pass
+        return False
+
+
+def test_kit_markup_no_negative_discount():
+    """Regression (47b5723): when web price > list price (markup), no discount line should be inserted."""
+    print("\n" + "=" * 60)
+    print("TEST: Kit markup — no negative discount")
+    print("=" * 60)
+
+    try:
+        with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
+            with conn.cursor() as cur:
+                suffix = f'{datetime.now().strftime("%H%M%S")}-{random.randint(1000, 9999)}'
+                setup_test_data(cur)
+                partner_id = _create_test_partner(cur, suffix)
+
+                # CAF01 list price = 51.50/unit, 10 units = 515
+                # Web price 600 > 515 => markup, no discount line
+                articles_json = '[{"sku": "CAFE100", "cantitate": 1, "pret": 600}]'
+                order_id = _import_order(cur, f'TEST-BIZ-MARKUP-{suffix}', partner_id, articles_json)
+
+                assert order_id and order_id > 0, "Order import failed"
+                rows = _get_order_lines(cur, order_id)
+
+                disc_lines = [r for r in rows if r[0] < 0]
+                print(f"   Total lines: {len(rows)}, Discount lines: {len(disc_lines)}")
+                assert len(disc_lines) == 0, f"Expected 0 discount lines for markup, got {len(disc_lines)}"
+                print("   PASS")
+
+                conn.commit()
+                teardown_test_data(cur)
+                conn.commit()
+                return True
+
+    except Exception as e:
+        print(f"   FAIL: {e}")
+        import traceback
+        traceback.print_exc()
+        try:
+            with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
+                with conn.cursor() as cur:
+                    teardown_test_data(cur)
+                    conn.commit()
+        except:
+            pass
+        return False
+
+
+def test_kit_component_price_zero_import():
+    """Regression (1703232): kit components with pret=0 should import successfully."""
+    print("\n" + "=" * 60)
+    print("TEST: Kit component price=0 import")
+    print("=" * 60)
+
+    try:
+        with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
+            with conn.cursor() as cur:
+                suffix = f'{datetime.now().strftime("%H%M%S")}-{random.randint(1000, 9999)}'
+                setup_test_data(cur)
+                partner_id = _create_test_partner(cur, suffix)
+
+                # Temporarily set CAF01 price to 0
+                cur.execute("""
+                    UPDATE crm_politici_pret_art SET PRET = 0
+                    WHERE id_articol = 9999001 AND id_pol = 1
+                """)
+                conn.commit()
+
+                try:
+                    # Import with pret=0 — should succeed (discount = full web price)
+                    articles_json = '[{"sku": "CAFE100", "cantitate": 1, "pret": 100}]'
+                    order_id = _import_order(cur, f'TEST-BIZ-PRET0-{suffix}', partner_id, articles_json)
+
+                    print(f"   Order ID: {order_id}")
+                    assert order_id and order_id > 0, "Order import failed with pret=0"
+                    print("   PASS: Order imported successfully with pret=0")
+
+                    conn.commit()
+                finally:
+                    # Restore original price
+                    cur.execute("""
+                        UPDATE crm_politici_pret_art SET PRET = 51.50
+                        WHERE id_articol = 9999001 AND id_pol = 1
+                    """)
+                    conn.commit()
+
+                teardown_test_data(cur)
+                conn.commit()
+                return True
+
+    except Exception as e:
+        print(f"   FAIL: {e}")
+        import traceback
+        traceback.print_exc()
+        try:
+            with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
+                with conn.cursor() as cur:
+                    # Restore price on error
+                    cur.execute("""
+                        UPDATE crm_politici_pret_art SET PRET = 51.50
+                        WHERE id_articol = 9999001 AND id_pol = 1
+                    """)
+                    conn.commit()
+                    teardown_test_data(cur)
+                    conn.commit()
+        except:
+            pass
+        return False
+
+
+def test_duplicate_codmat_different_prices():
+    """Regression (95565af): same CODMAT at different prices should create separate lines,
+    discriminated by PRET + SIGN(CANTITATE)."""
+    print("\n" + "=" * 60)
+    print("TEST: Duplicate CODMAT different prices")
+    print("=" * 60)
+
+    try:
+        with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
+            with conn.cursor() as cur:
+                suffix = f'{datetime.now().strftime("%H%M%S")}-{random.randint(1000, 9999)}'
+                setup_test_data(cur)
+                partner_id = _create_test_partner(cur, suffix)
+
+                # Two articles both mapping to CAF01 but at different prices
+                # CAFE100 -> CAF01 via ARTICOLE_TERTI (kit pricing)
+                # We use separate_line mode so article gets list price 51.50
+                # Then a second article at a different price on the same CODMAT
+                # For this test, we import 2 separate orders to same CODMAT with different prices
+                # The real scenario: kit article line + discount line on same id_articol
+
+                articles_json = '[{"sku": "CAFE100", "cantitate": 1, "pret": 450}]'
+                order_id = _import_order(cur, f'TEST-BIZ-DUP-{suffix}', partner_id, articles_json)
+
+                assert order_id and order_id > 0, "Order import failed"
+                rows = _get_order_lines(cur, order_id)
+
+                # separate_line mode: article at list price + discount at negative qty
+                # Both reference same CODMAT (CAF01) but different PRET and SIGN(CANTITATE)
+                codmats = [r[2] for r in rows]
+                print(f"   Lines: {len(rows)}")
+                for r in rows:
+                    print(f"     qty={r[0]}, pret={r[1]:.2f}, codmat={r[2]}")
+
+                # Should have at least 2 lines with same CODMAT but different qty sign
+                caf_lines = [r for r in rows if r[2] == 'CAF01']
+                assert len(caf_lines) >= 2, f"Expected 2+ CAF01 lines (article + discount), got {len(caf_lines)}"
+                signs = {1 if r[0] > 0 else -1 for r in caf_lines}
+                assert len(signs) == 2, "Expected both positive and negative quantity lines for same CODMAT"
+                print("   PASS: Same CODMAT with different PRET/SIGN coexist")
+
+                conn.commit()
+                teardown_test_data(cur)
+                conn.commit()
+                return True
+
+    except Exception as e:
+        print(f"   FAIL: {e}")
+        import traceback
+        traceback.print_exc()
+        try:
+            with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
+                with conn.cursor() as cur:
+                    teardown_test_data(cur)
+                    conn.commit()
+        except:
+            pass
+        return False
+
+
 if __name__ == "__main__":
     print("Starting complete order import test...")
     print(f"Timestamp: {datetime.now()}")
@@ -536,16 +909,32 @@ if __name__ == "__main__":
 
     print(f"\nTest completed at: {datetime.now()}")
     if success:
-        print("🎯 PHASE 1 VALIDATION: SUCCESSFUL")
+        print("PHASE 1 VALIDATION: SUCCESSFUL")
     else:
-        print("🔧 PHASE 1 VALIDATION: NEEDS ATTENTION")
+        print("PHASE 1 VALIDATION: NEEDS ATTENTION")
 
     # Run repackaging kit pricing test
     print("\n")
     repack_success = test_repackaging_kit_pricing()
     if repack_success:
-        print("🎯 REPACKAGING KIT PRICING: SUCCESSFUL")
+        print("REPACKAGING KIT PRICING: SUCCESSFUL")
     else:
-        print("🔧 REPACKAGING KIT PRICING: NEEDS ATTENTION")
+        print("REPACKAGING KIT PRICING: NEEDS ATTENTION")
+
+    # Run business rule regression tests
+    print("\n")
+    biz_tests = [
+        ("Multi-kit discount merge", test_multi_kit_discount_merge),
+        ("Kit discount per-kit placement", test_kit_discount_per_kit_placement),
+        ("Distributed total matches web", test_repackaging_distributed_total_matches_web),
+        ("Markup no negative discount", test_kit_markup_no_negative_discount),
+        ("Component price=0 import", test_kit_component_price_zero_import),
+        ("Duplicate CODMAT different prices", test_duplicate_codmat_different_prices),
+    ]
+    biz_passed = 0
+    for name, test_fn in biz_tests:
+        if test_fn():
+            biz_passed += 1
+    print(f"\nBusiness rule tests: {biz_passed}/{len(biz_tests)} passed")
 
     exit(0 if success else 1)

api/tests/test_integration.py

@@ -82,46 +82,51 @@ def test_health_oracle_connected(client):
 # ---------------------------------------------------------------------------
 # Test B: Mappings CRUD cycle (uses real CODMAT from Oracle nomenclator)
 # ---------------------------------------------------------------------------
-TEST_SKU = "PYTEST_INTEG_SKU_001"
+@pytest.fixture(scope="module")
+def test_sku():
+    """Generate a unique test SKU per run to avoid conflicts with prior soft-deleted entries."""
+    import time
+    return f"PYTEST_SKU_{int(time.time())}"
 
 
 @pytest.fixture(scope="module")
 def real_codmat(client):
     """Find a real CODMAT from Oracle nomenclator to use in mappings tests."""
-    resp = client.get("/api/articles/search", params={"q": "A"})
+    # min_length=2 on the endpoint, so use 2+ char search terms
-    if resp.status_code != 200:
+    for term in ["01", "PH", "CA"]:
-        pytest.skip("Articles search unavailable")
+        resp = client.get("/api/articles/search", params={"q": term})
+        if resp.status_code == 200:
             results = resp.json().get("results", [])
-    if not results:
+            if results:
-        pytest.skip("No articles found in Oracle for CRUD test")
                 return results[0]["codmat"]
+    pytest.skip("No articles found in Oracle for CRUD test")
 
 
-def test_mappings_create(client, real_codmat):
+def test_mappings_create(client, real_codmat, test_sku):
     resp = client.post("/api/mappings", json={
-        "sku": TEST_SKU,
+        "sku": test_sku,
         "codmat": real_codmat,
         "cantitate_roa": 2.5,
     })
-    assert resp.status_code == 200
+    assert resp.status_code == 200, f"create returned {resp.status_code}: {resp.json()}"
     body = resp.json()
     assert body.get("success") is True, f"create returned: {body}"
 
 
-def test_mappings_list_after_create(client, real_codmat):
+def test_mappings_list_after_create(client, real_codmat, test_sku):
-    resp = client.get("/api/mappings", params={"search": TEST_SKU})
+    resp = client.get("/api/mappings", params={"search": test_sku})
     assert resp.status_code == 200
     body = resp.json()
     mappings = body.get("mappings", [])
     found = any(
-        m["sku"] == TEST_SKU and m["codmat"] == real_codmat
+        m["sku"] == test_sku and m["codmat"] == real_codmat
         for m in mappings
     )
     assert found, f"mapping not found in list; got {mappings}"
 
 
-def test_mappings_update(client, real_codmat):
+def test_mappings_update(client, real_codmat, test_sku):
-    resp = client.put(f"/api/mappings/{TEST_SKU}/{real_codmat}", json={
+    resp = client.put(f"/api/mappings/{test_sku}/{real_codmat}", json={
         "cantitate_roa": 3.0,
     })
     assert resp.status_code == 200
@@ -129,25 +134,25 @@ def test_mappings_update(client, real_codmat):
     assert body.get("success") is True, f"update returned: {body}"
 
 
-def test_mappings_delete(client, real_codmat):
+def test_mappings_delete(client, real_codmat, test_sku):
-    resp = client.delete(f"/api/mappings/{TEST_SKU}/{real_codmat}")
+    resp = client.delete(f"/api/mappings/{test_sku}/{real_codmat}")
     assert resp.status_code == 200
     body = resp.json()
     assert body.get("success") is True, f"delete returned: {body}"
 
 
-def test_mappings_verify_soft_deleted(client, real_codmat):
+def test_mappings_verify_soft_deleted(client, real_codmat, test_sku):
-    resp = client.get("/api/mappings", params={"search": TEST_SKU, "show_deleted": "true"})
+    resp = client.get("/api/mappings", params={"search": test_sku, "show_deleted": "true"})
     assert resp.status_code == 200
     body = resp.json()
     mappings = body.get("mappings", [])
     deleted = any(
-        m["sku"] == TEST_SKU and m["codmat"] == real_codmat and m.get("sters") == 1
+        m["sku"] == test_sku and m["codmat"] == real_codmat and m.get("sters") == 1
         for m in mappings
     )
     assert deleted, (
         f"expected sters=1 for deleted mapping, got: "
-        f"{[m for m in mappings if m['sku'] == TEST_SKU]}"
+        f"{[m for m in mappings if m['sku'] == test_sku]}"
     )
 
 

docs/LLM_PROJECT_MANAGER_PROMPT.md

@@ -1,241 +0,0 @@
-# LLM Project Manager Prompt
-## Pentru Implementarea PRD: Import Comenzi Web → Sistem ROA
-
-Tu ești un **Project Manager AI specializat** care urmărește implementarea unui PRD (Product Requirements Document) prin descompunerea în user stories executabile și urmărirea progresului.
-
----
-
-## 🎯 Misiunea Ta
-
-Implementezi sistemul de import automat comenzi web → ERP ROA Oracle conform PRD-ului furnizat. Vei coordona dezvoltarea în 4 faze distincte, urmărind fiecare story și asigurându-te că totul este livrat conform specificațiilor.
-
----
-
-## 📋 Context PRD
-
-**Sistem:** Import comenzi de pe platforme web (GoMag, etc.) în sistemul ERP ROA Oracle
-**Tech Stack:** Oracle PL/SQL + Visual FoxPro 9 + FastApi (admin interface)
-**Componente Principale:**
-- Package Oracle pentru parteneri și comenzi
-- Orchestrator VFP pentru sincronizare automată
-- Interfață web pentru administrare mapări SKU
-- Tabel nou ARTICOLE_TERTI pentru mapări complexe
-
----
-
-## 📊 User Stories Framework
-
-Pentru fiecare story, vei genera:
-
-### Story Template:
-```
-**Story ID:** [FASE]-[NR] (ex: P1-001)
-**Titlu:** [Descriere concisă]
-**As a:** [Utilizator/Sistem]
-**I want:** [Funcționalitate dorită]
-**So that:** [Beneficiul de business]
-
-**Acceptance Criteria:**
-- [ ] Criteriu 1
-- [ ] Criteriu 2
-- [ ] Criteriu 3
-
-**Technical Tasks:**
-- [ ] Task tehnic 1
-- [ ] Task tehnic 2
-
-**Definition of Done:**
-- [ ] Cod implementat și testat
-- [ ] Documentație actualizată
-- [ ] Error handling complet
-- [ ] Logging implementat
-- [ ] Review code efectuat
-
-**Estimate:** [XS/S/M/L/XL] ([ore estimate])
-**Dependencies:** [Alte story-uri necesare]
-**Risk Level:** [Low/Medium/High]
-```
-
----
-
-## 🏗️ Faze de Implementare
-
-### **PHASE 1: Database Foundation (Ziua 1)**
-Creează story-uri pentru:
-- Tabel ARTICOLE_TERTI cu structura specificată
-- Package IMPORT_PARTENERI complet funcțional
-- Package IMPORT_COMENZI cu logica de mapare
-- Teste unitare pentru package-uri
-
-### **PHASE 2: VFP Integration (Ziua 2)** 
-Creează story-uri pentru:
-- Adaptare gomag-adapter.prg pentru JSON output
-- Orchestrator sync-comenzi-web.prg cu timer
-- Integrare Oracle packages în VFP
-- Sistem de logging cu rotație
-
-### **PHASE 3: Web Admin Interface (Ziua 3)**
-Creează story-uri pentru:
-- Flask app cu Oracle connection pool
-- HTML/CSS interface pentru admin mapări
-- JavaScript pentru CRUD operații
-- Validări client-side și server-side
-
-### **PHASE 4: Testing & Deployment (Ziua 4)**
-Creează story-uri pentru:
-- Testare end-to-end cu comenzi reale
-- Validare mapări complexe (seturi, reîmpachetări)
-- Configurare environment production
-- Documentație utilizare finală
-
----
-
-## 🔄 Workflow de Urmărire
-
-### La început de sesiune:
-1. **Prezintă status overview:** "PHASE X - Y% complete, Z stories remaining"
-2. **Identifică story-ul curent** și dependencies
-3. **Verifică blocaje** și propune soluții
-4. **Actualizează planning-ul** dacă e nevoie
-
-### Pe durata implementării:
-1. **Urmărește progresul** fiecărui task în story
-2. **Validează completion criteria** înainte să marchezi DONE
-3. **Identifică riscos** și alertează proactiv
-4. **Propune optimizări** de proces
-
-### La finalizare story:
-1. **Demo功能** implementată
-2. **Confirmă acceptance criteria** îndeplinite
-3. **Planifică next story** cu dependencies
-4. **Actualizează overall progress**
-
----
-
-## 📊 Tracking & Reporting
-
-### Daily Status Format:
-```
-📈 PROJECT STATUS - [DATA]
-═══════════════════════════════════
-
-🎯 Current Phase: [PHASE X] 
-📊 Overall Progress: [X]% ([Y]/[Z] stories done)
-⏰ Current Story: [STORY-ID] - [TITLE]
-🔄 Status: [IN PROGRESS/BLOCKED/READY FOR REVIEW]
-
-📋 Today's Completed:
-- ✅ [Story completă]
-- ✅ [Task complet]
-
-🚧 In Progress:
-- 🔄 [Story în lucru]
-- ⏳ [Task în progress]
-
-⚠️  Blockers:
-- 🚨 [Blocker 1]
-- 🔍 [Issue necesitând decizie]
-
-📅 Next Up:
-- 📝 [Next story ready]
-- 🔜 [Upcoming dependency]
-
-🎯 Phase Target: [Data target] | Risk: [LOW/MED/HIGH]
-```
-
-### Weekly Sprint Review:
-- Retrospectivă story-uri complete vs planificate
-- Analiza blockers întâlniți și soluții
-- Ajustări planning pentru săptămâna următoare
-- Identificare lesson learned
-
----
-
-## 🚨 Risk Management
-
-### Categorii Risc:
-- **HIGH:** Blockers care afectează multiple story-uri
-- **MEDIUM:** Delay-uri care pot afecta phase target
-- **LOW:** Issues locale care nu afectează planning-ul
-
-### Escalation Matrix:
-1. **Technical Issues:** Propui soluții alternative/workaround
-2. **Dependency Blockers:** Replanifici priority și sequence
-3. **Scope Changes:** Alertezi și ceri validare înainte de implementare
-
----
-
-## 🎛️ Comenzi Disponibile
-
-Răspunzi la comenzile:
-- `status` - Overall progress și current story
-- `stories` - Lista toate story-urile cu status
-- `phase` - Detalii phase curentă
-- `risks` - Identifică și prioritizează riscuri
-- `demo [story-id]` - Demonstrație funcționalitate implementată
-- `plan` - Re-planificare dacă apar schimbări
-
-## 📋 User Stories Location
-
-Toate story-urile sunt stocate în fișiere individuale în `docs/stories/` cu format:
-- **P1-001-ARTICOLE_TERTI.md** - Story complet cu acceptance criteria
-- **P1-002-Package-IMPORT_PARTENERI.md** - Detalii implementare parteneri
-- **P1-003-Package-IMPORT_COMENZI.md** - Logică import comenzi
-- **P1-004-Testing-Manual-Packages.md** - Plan testare
-
-**Beneficii:**
-- Nu mai regenerez story-urile la fiecare sesiune
-- Persistența progresului și update-urilor
-- Ușor de referenciat și de împărtășit cu stakeholders
-
----
-
-## 💡 Success Criteria
-
-### Technical KPIs:
-- Import success rate > 95%
-- Timp mediu procesare < 30s per comandă  
-- Zero downtime pentru ROA principal
-- 100% log coverage
-
-### Project KPIs:
-- Stories delivered on time: >90%
-- Zero blockers mai mult de 1 zi
-- Code review coverage: 100%
-- Documentation completeness: 100%
-
----
-
-## 🤖 Personality & Communication Style
-
-- **Proactiv:** Anticipezi probleme și propui soluții
-- **Data-driven:** Folosești metrici concrete pentru tracking
-- **Pragmatic:** Focusat pe delivery și rezultate practice
-- **Comunicativ:** Updates clare și acționabile
-- **Quality-focused:** Nu accepti compromisuri pe Definition of Done
-
----
-
-## 🚀 Getting Started
-
-**Primul tau task:** 
-1. Citește întregul PRD furnizat și verifică dacă există story-uri pentru fiecare fază și la care fază/story ai rămas
-
-**Întreabă-mă dacă:**
-- Necesită clarificări tehnice despre PRD
-- Vrei să ajustez priority sau sequence
-- Apare vreo dependency neidentificată
-- Ai nevoie de input pentru estimări
-
-**Întreabă-mă dacă:**
-Afișează comenzile disponibile
-  - status - Progres overall
-  - stories - Lista story-uri
-  - phase - Detalii fază curentă
-  - risks - Identificare riscuri
-  - demo [story-id] - Demo funcționalitate
-  - plan - Re-planificare
-
----
-
-**Acum începe cu:** "Am analizat PRD-ul și sunt gata să coordonez implementarea. Vrei să îți spun care a fost ultimul story si care este statusul său?"

docs/PRD.md

@@ -1,610 +0,0 @@
-# Product Requirements Document (PRD)
-## Import Comenzi Web → Sistem ROA
-
-**Versiune:** 1.2  
-**Data:** 10 septembrie 2025  
-**Status:** Phase 1 - ✅ COMPLET | Ready for Phase 2 VFP Integration  
-
----
-
-## 📋 Overview
-
-Sistem ultra-minimal pentru importul comenzilor de pe platforme web (GoMag, etc.) în sistemul ERP ROA Oracle. Sistemul gestionează automat maparea produselor, crearea clienților și generarea comenzilor în ROA.
-
-### Obiective Principale
-- ✅ Import automat comenzi web → ROA
-- ✅ Mapare flexibilă SKU → CODMAT (reîmpachetări + seturi)
-- ✅ Crearea automată a partenerilor noi
-- ✅ Interfață web pentru administrare mapări
-- ✅ Logging complet pentru troubleshooting
-
----
-
-## 🎯 Scope & Limitations
-
-### În Scope
-- Import comenzi din orice platformă web (nu doar GoMag)
-- Mapare SKU complexe (1:1, 1:N, reîmpachetări, seturi)
-- Crearea automată parteneri + adrese
-- Interfață web admin pentru mapări
-- Logging în fișiere text
-
-### Out of Scope
-- Modificarea comenzilor existente în ROA
-- Sincronizare bidirectională
-- Gestionarea stocurilor
-- Interfață pentru utilizatori finali
-
----
-
-## 🏗️ Architecture Overview
-
-```
-[Web Platform API] → [VFP Orchestrator] → [Oracle PL/SQL] → [Web Admin Interface]
-       ↓                    ↓                    ↑                  ↑
-   JSON Orders        Process & Log        Store/Update      Configuration
-```
-
-### Tech Stack
-- **Backend:** Oracle PL/SQL packages
-- **Integration:** Visual FoxPro 9
-- **Admin Interface:** Flask + Oracle
-- **Data:** Oracle 11g/12c
-
----
-
-## 📊 Data Model
-
-### Tabel Nou: ARTICOLE_TERTI
-```sql
-CREATE TABLE ARTICOLE_TERTI (
-    sku           VARCHAR2(100),      -- SKU din platforma web
-    codmat        VARCHAR2(50),       -- CODMAT din nom_articole
-    cantitate_roa NUMBER(10,3),       -- Câte unități ROA = 1 web
-    procent_pret  NUMBER(5,2),        -- % din preț pentru seturi
-    activ         NUMBER(1),          -- 1=activ, 0=inactiv
-    PRIMARY KEY (sku, codmat)
-);
-```
-
-### Exemple Mapări
-- **Simplu:** SKU "CAF01" → caută direct în nom_articole (nu se stochează)
-- **Reîmpachetare:** SKU "CAFE100" → CODMAT "CAF01", cantitate_roa=10
-- **Set compus:** 
-  - SKU "SET01" → CODMAT "CAF01", cantitate_roa=2, procent_pret=60
-  - SKU "SET01" → CODMAT "FILT01", cantitate_roa=1, procent_pret=40
-
----
-
-## 🔧 Components Specification
-
-### 1. Package IMPORT_PARTENERI
-
-**Funcții:**
-- `cauta_sau_creeaza_partener()` - Găsește partener existent sau creează unul nou
-- `parseaza_adresa_semicolon()` - Parsează adrese format: "JUD:București;BUCURESTI;Str.Victoriei;10"
-
-**Logica Căutare Parteneri:**
-1. Caută după cod_fiscal (dacă > 3 caractere)
-2. Caută după denumire exactă
-3. Creează partener nou folosind `pack_def.adauga_partener()`
-4. Adaugă adresa folosind `pack_def.adauga_adresa_partener2()`
-
-### 2. Package IMPORT_COMENZI
-
-**Funcții:**
-- `gaseste_articol_roa()` - Rezolvă SKU → articole ROA
-- `importa_comanda_web()` - Import comandă completă
-
-**Logica Articole:**
-1. Verifică ARTICOLE_TERTI pentru SKU
-2. Dacă nu există → caută direct în nom_articole (SKU = CODMAT)
-3. Calculează cantități și prețuri conform mapărilor
-4. Folosește `PACK_COMENZI.adauga_comanda()` și `PACK_COMENZI.adauga_articol_comanda()`
-
-### 3. VFP Orchestrator (sync-comenzi-web.prg)
-
-**Responsabilități:**
-- Rulare automată (timer 5 minute)
-- Citire comenzi din JSON-ul generat de gomag-adapter.prg
-- Procesare comenzi GoMag cu mapare completă la Oracle
-- Apelare package-uri Oracle pentru import
-- Logging în fișiere text cu timestamp
-
-**Fluxul complet de procesare:**
-1. **Input:** Citește `output/gomag_orders_last7days_*.json`
-2. **Pentru fiecare comandă:**
-   - Extrage date billing/shipping
-   - Procesează parteneri (persoane fizice vs companii)
-   - Mapează articole web → ROA
-   - Creează comandă în Oracle cu toate detaliile
-3. **Output:** Log complet în `logs/sync_comenzi_YYYYMMDD.log`
-
-**Funcții helper necesare:**
-- `CleanGoMagText()` - Curățare HTML entities
-- `ProcessGoMagOrder()` - Procesare comandă completă
-- `BuildArticlesJSON()` - Transformare items → JSON Oracle
-- `FormatAddressForOracle()` - Adrese în format semicolon
-- `HandleSpecialCases()` - Shipping vs billing, discounts, etc.
-
-**Procesare Date GoMag pentru IMPORT_PARTENERI:**
-
-*Decodare HTML entities în caractere simple (fără diacritice):*
-```foxpro
-* Funcție de curățare text GoMag
-FUNCTION CleanGoMagText(tcText)
-  LOCAL lcResult
-  lcResult = tcText
-  lcResult = STRTRAN(lcResult, 'ă', 'a')  && ă → a
-  lcResult = STRTRAN(lcResult, 'ș', 's')  && ș → s  
-  lcResult = STRTRAN(lcResult, 'ț', 't')  && ț → t
-  lcResult = STRTRAN(lcResult, 'î', 'i')  && î → i
-  lcResult = STRTRAN(lcResult, 'â', 'a')  && â → a
-  RETURN lcResult
-ENDFUNC
-```
-
-*Pregătire date partener din billing GoMag:*
-```foxpro
-* Pentru persoane fizice (când billing.company e gol):
-IF EMPTY(loBilling.company.name)
-  lcDenumire = CleanGoMagText(loBilling.firstname + ' ' + loBilling.lastname)
-  lcCodFiscal = NULL  && persoane fizice nu au CUI în GoMag
-ELSE
-  * Pentru companii:
-  lcDenumire = CleanGoMagText(loBilling.company.name)
-  lcCodFiscal = loBilling.company.code  && CUI companie
-ENDIF
-
-* Formatare adresă pentru Oracle (format semicolon):
-lcAdresa = "JUD:" + CleanGoMagText(loBilling.region) + ";" + ;
-           CleanGoMagText(loBilling.city) + ";" + ;
-           CleanGoMagText(loBilling.address)
-
-* Date contact
-lcTelefon = loBilling.phone
-lcEmail = loBilling.email
-```
-
-*Apel package Oracle IMPORT_PARTENERI:*
-```foxpro
-* Apelare IMPORT_PARTENERI.cauta_sau_creeaza_partener
-lcSQL = "SELECT IMPORT_PARTENERI.cauta_sau_creeaza_partener(?, ?, ?, ?, ?) AS ID_PART FROM dual"
-
-* Executare cu parametri:
-* p_cod_fiscal, p_denumire, p_adresa, p_telefon, p_email
-lnIdPart = SQLEXEC(goConnectie, lcSQL, lcCodFiscal, lcDenumire, lcAdresa, lcTelefon, lcEmail, "cursor_result")
-
-IF lnIdPart > 0 AND RECCOUNT("cursor_result") > 0
-  lnPartnerID = cursor_result.ID_PART
-  * Continuă cu procesarea comenzii...
-ELSE
-  * Log eroare partener
-  WriteLog("ERROR: Nu s-a putut crea/găsi partenerul: " + lcDenumire)
-ENDIF
-```
-
-**Procesare Articole pentru IMPORT_COMENZI:**
-
-*Construire JSON articole din items GoMag:*
-```foxpro
-* Funcție BuildArticlesJSON - transformă items GoMag în format Oracle
-FUNCTION BuildArticlesJSON(loItems)
-  LOCAL lcJSON, i, loItem
-  lcJSON = "["
-  
-  FOR i = 1 TO loItems.Count
-    loItem = loItems.Item(i)
-    
-    IF i > 1
-      lcJSON = lcJSON + ","
-    ENDIF
-    
-    * Format JSON conform package Oracle: {"sku":"...", "cantitate":..., "pret":...}
-    lcJSON = lcJSON + "{" + ;
-      '"sku":"' + CleanGoMagText(loItem.sku) + '",' + ;
-      '"cantitate":' + TRANSFORM(VAL(loItem.quantity)) + ',' + ;
-      '"pret":' + TRANSFORM(VAL(loItem.price)) + ;
-      "}"
-  ENDFOR
-  
-  lcJSON = lcJSON + "]"
-  RETURN lcJSON
-ENDFUNC
-```
-
-*Gestionare cazuri speciale:*
-```foxpro
-* Informații adiționale pentru observații
-lcObservatii = "Payment: " + CleanGoMagText(loOrder.payment.name) + "; " + ;
-               "Delivery: " + CleanGoMagText(loOrder.delivery.name) + "; " + ;
-               "Status: " + CleanGoMagText(loOrder.status) + "; " + ;
-               "Source: " + CleanGoMagText(loOrder.source) + " " + CleanGoMagText(loOrder.sales_channel)
-
-* Adrese diferite shipping vs billing
-IF NOT (CleanGoMagText(loOrder.shipping.address) == CleanGoMagText(loBilling.address))
-  lcObservatii = lcObservatii + "; Shipping: " + ;
-                 CleanGoMagText(loOrder.shipping.address) + ", " + ;
-                 CleanGoMagText(loOrder.shipping.city)
-ENDIF
-```
-
-*Apel package Oracle IMPORT_COMENZI:*
-```foxpro
-* Conversie dată GoMag → Oracle
-ldDataComanda = CTOD(SUBSTR(loOrder.date, 1, 10))  && "2025-08-27 16:32:43" → date
-
-* JSON articole
-lcArticoleJSON = BuildArticlesJSON(loOrder.items)
-
-* Apelare IMPORT_COMENZI.importa_comanda_web
-lcSQL = "SELECT IMPORT_COMENZI.importa_comanda_web(?, ?, ?, ?, ?, ?) AS ID_COMANDA FROM dual"
-
-lnResult = SQLEXEC(goConnectie, lcSQL, ;
-  loOrder.number, ;        && p_nr_comanda_ext
-  ldDataComanda, ;         && p_data_comanda  
-  lnPartnerID, ;           && p_id_partener (din pas anterior)
-  lcArticoleJSON, ;        && p_json_articole
-  NULL, ;                  && p_id_adresa_livrare (opțional)
-  lcObservatii, ;          && p_observatii
-  "cursor_comanda")
-
-IF lnResult > 0 AND cursor_comanda.ID_COMANDA > 0
-  WriteLog("SUCCESS: Comandă importată - ID: " + TRANSFORM(cursor_comanda.ID_COMANDA))
-ELSE
-  WriteLog("ERROR: Import comandă eșuat pentru: " + loOrder.number)
-ENDIF
-```
-
-**Note Importante:**
-- Toate caracterele HTML trebuie transformate în ASCII simplu (fără diacritice)
-- Package-ul Oracle așteaptă text curat, fără entități HTML
-- Adresa trebuie în format semicolon cu prefix "JUD:" pentru județ
-- Cod fiscal NULL pentru persoane fizice este acceptabil
-- JSON articole: exact formatul `{"sku":"...", "cantitate":..., "pret":...}`
-- Conversie date GoMag: `"2025-08-27 16:32:43"` → `CTOD()` pentru Oracle
-- Observații: concatenează payment/delivery/status/source pentru tracking
-- Gestionează adrese diferite shipping vs billing în observații
-- Utilizează conexiunea Oracle existentă (goConnectie)
-
-### 4. Web Admin Interface
-
-**Funcționalități:**
-- Vizualizare mapări SKU existente
-- Adăugare/editare/ștergere mapări
-- Validare date înainte de salvare
-- Interface responsive cu Flask
-
----
-
-## 📋 Implementation Phases
-
-### Phase 1: Database Foundation (Ziua 1) - 🎯 75% COMPLET
-- [x] ✅ **P1-001:** Creare tabel ARTICOLE_TERTI + Docker setup
-- [x] ✅ **P1-002:** Package IMPORT_PARTENERI complet  
-- [x] ✅ **P1-003:** Package IMPORT_COMENZI complet
-- [ ] 🔄 **P1-004:** Testare manuală package-uri (NEXT UP!)
-
-### Phase 2: VFP Integration (Ziua 2)
-- [ ] **P2-001:** Adaptare gomag-adapter.prg pentru output JSON (READY - doar activare GetOrders)
-- [ ] **P2-002:** Creare sync-comenzi-web.prg cu toate helper functions
-- [ ] **P2-003:** Testare import comenzi end-to-end cu date reale GoMag
-- [ ] **P2-004:** Configurare logging și error handling complet
-
-**Detalii P2-002 (sync-comenzi-web.prg):**
-- `CleanGoMagText()` - HTML entities cleanup
-- `ProcessGoMagOrder()` - Main orchestrator per order
-- `BuildArticlesJSON()` - Items conversion for Oracle
-- `FormatAddressForOracle()` - Semicolon format
-- `HandleSpecialCases()` - Shipping/billing/discounts/payments
-- Integration cu logging existent din utils.prg
-- Timer-based execution (5 minute intervals)
-- Complete error handling cu retry logic
-
-### Phase 3: Web Admin Interface (Ziua 3)
-- [ ] Flask app cu connection pool Oracle
-- [ ] HTML/CSS pentru admin mapări
-- [ ] JavaScript pentru CRUD operații
-- [ ] Testare interfață web
-
-### Phase 4: Testing & Deployment (Ziua 4)
-- [ ] Testare integrată pe comenzi reale
-- [ ] Validare mapări complexe (seturi)
-- [ ] Configurare environment production
-- [ ] Documentație utilizare
-
----
-
-## 📁 File Structure
-
-```
-/api/                               # ✅ Flask Admin Interface
-  ├── admin.py                      # ✅ Flask app cu Oracle pool
-  ├── 01_create_table.sql           # ✅ Tabel ARTICOLE_TERTI 
-  ├── 02_import_parteneri.sql       # ✅ Package parteneri (COMPLET)
-  ├── 03_import_comenzi.sql         # ✅ Package comenzi (COMPLET)
-  ├── Dockerfile                    # ✅ Container cu Oracle client
-  ├── tnsnames.ora                  # ✅ Config Oracle ROA
-  ├── .env                          # ✅ Environment variables
-  └── requirements.txt              # ✅ Dependencies Python
-
-/docs/                              # 📋 Project Documentation
-  ├── PRD.md                        # ✅ Product Requirements Document
-  ├── LLM_PROJECT_MANAGER_PROMPT.md # ✅ Project Manager Prompt
-  └── stories/                      # 📋 User Stories (Detailed)
-     ├── P1-001-ARTICOLE_TERTI.md   # ✅ Story P1-001 (COMPLET)
-     ├── P1-002-Package-IMPORT_PARTENERI.md # ✅ Story P1-002 (COMPLET)
-     ├── P1-003-Package-IMPORT_COMENZI.md   # ✅ Story P1-003 (COMPLET)
-     └── P1-004-Testing-Manual-Packages.md  # 📋 Story P1-004
-
-/vfp/                               # ⏳ VFP Integration (Phase 2)
-  └── sync-comenzi-web.prg          # ⏳ Orchestrator principal
-
-/docker-compose.yaml                # ✅ Container orchestration
-/logs/                              # ✅ Logging directory
-```
-
----
-
-## 🔒 Business Rules
-
-### Parteneri
-- Căutare prioritate: cod_fiscal → denumire → creare nou
-- Persoane fizice (CUI 13 cifre): separă nume/prenume
-- Adrese: defaultează la București Sectorul 1 dacă nu găsește
-- Toate partenerele noi au ID_UTIL = -3 (sistem)
-
-### Articole
-- SKU simple (găsite direct în nom_articole): nu se stochează în ARTICOLE_TERTI
-- Mapări speciale: doar reîmpachetări și seturi complexe
-- Validare: suma procent_pret pentru același SKU să fie logic
-- Articole inactive: activ=0 (nu se șterg)
-
-### Comenzi
-- Folosește package-urile existente (PACK_COMENZI)
-- ID_GESTIUNE = 1, ID_SECTIE = 1, ID_POL = 0 (default)
-- Data livrare = data comenzii + 1 zi
-- Toate comenzile au INTERNA = 0 (externe)
-
----
-
-## 📊 Success Metrics
-
-### Technical Metrics
-- Import success rate > 95%
-- Timpul mediu de procesare < 30s per comandă
-- Zero downtime pentru sistemul principal ROA
-- Log coverage 100% (toate operațiile logate)
-
-### Business Metrics
-- Reducerea timpului de introducere comenzi cu 90%
-- Eliminarea erorilor manuale de transcriere
-- Timpul de configurare mapări noi < 5 minute
-
----
-
-## 🚨 Error Handling
-
-### Categorii Erori
-1. **Erori conexiune Oracle:** Retry logic + alertă
-2. **SKU not found:** Log warning + skip articol
-3. **Partener invalid:** Tentativă creare + log detalii
-4. **Comenzi duplicate:** Skip cu log info
-
-### Logging Format
-```
-2025-09-08 14:30:25 | COMANDA-123 | OK | ID:456789
-2025-09-08 14:30:26 | COMANDA-124 | ERROR | SKU 'XYZ' not found
-```
-
----
-
-## 🔧 Configuration
-
-### Environment Variables (.env)
-```env
-ORACLE_USER=MARIUSM_AUTO
-ORACLE_PASSWORD=********
-ORACLE_DSN=ROA_CENTRAL
-TNS_ADMIN=/app
-INSTANTCLIENTPATH=/opt/oracle/instantclient
-```
-
-### ⚠️ **CRITICAL: Oracle Schema Details**
-
-**Test Schema:** `MARIUSM_AUTO` (nu CONTAFIN_ORACLE)
-**Database:** Oracle 10g Enterprise Edition Release 10.2.0.4.0
-**TNS Connection:** ROA_CENTRAL (nu ROA_ROMFAST)
-
-**Structura Reală Tables:**
-- `COMENZI` (nu `comenzi_antet`) - Comenzile principale
-- `COMENZI_ELEMENTE` (nu `comenzi_articole`) - Articolele din comenzi
-- `NOM_PARTENERI` - Partenerii
-- `NOM_ARTICOLE` - Articolele 
-- `ARTICOLE_TERTI` - Mapările SKU (creat de noi)
-
-**Foreign Key Constraints CRITICAL:**
-```sql
--- Pentru COMENZI_ELEMENTE:
-ID_POL = 2 (obligatoriu, nu NULL sau 0)
-ID_VALUTA = 3 (obligatoriu, nu 1)
-ID_ARTICOL - din NOM_ARTICOLE
-ID_COMANDA - din COMENZI
-```
-
-**Package Status în MARIUSM_AUTO:**
-- ✅ `PACK_IMPORT_PARTENERI` - VALID (header + body)  
-- ✅ `PACK_JSON` - VALID (header + body)
-- ✅ `PACK_COMENZI` - VALID (header + body)
-- ✅ `PACK_IMPORT_COMENZI` - header VALID, body FIXED în P1-004
-
-### VFP Configuration
-- Timer interval: 300 secunde (5 minute)
-- Conexiune Oracle prin goExecutor existent
-- Log files: sync_YYYYMMDD.log (rotație zilnică)
-
----
-
-## 🎛️ Admin Interface Specification
-
-### Main Screen: SKU Mappings
-- Tabel editabil cu coloane: SKU, CODMAT, Cantitate ROA, Procent Preț, Activ
-- Inline editing cu auto-save
-- Filtrare și căutare
-- Export/Import mapări (CSV)
-- Validare în timp real
-
-### Features
-- Bulk operations (activare/dezactivare multiple)
-- Template mapări pentru tipuri comune
-- Preview calcul preț pentru teste
-- Audit trail (cine/când a modificat)
-
----
-
-## 🏁 Definition of Done
-
-### Per Feature
-- [ ] Cod implementat și testat
-- [ ] Documentație actualizată
-- [ ] Error handling complet
-- [ ] Logging implementat
-- [ ] Review code efectuat
-
-### Per Phase
-- [ ] Toate feature-urile Phase complete
-- [ ] Testare integrată reușită
-- [ ] Performance requirements îndeplinite
-- [ ] Deployment verificat
-- [ ] Sign-off stakeholder
-
----
-
-## 📞 Support & Maintenance
-
-### Monitoring
-- Log files în /logs/ cu rotație automată
-- Alertă email pentru erori critice
-- Dashboard cu statistici import (opcional Phase 2)
-
-### Backup & Recovery
-- Mapări ARTICOLE_TERTI incluse în backup-ul zilnic ROA
-- Config files versionate în Git
-- Procedură rollback pentru package-uri Oracle
-
----
-
----
-
-## 📊 Progress Status - Phase 1 [🎯 100% COMPLET]
-
-### ✅ P1-001 COMPLET: Tabel ARTICOLE_TERTI
-- **Implementat:** 08 septembrie 2025, 22:30
-- **Files:** `api/database-scripts/01_create_table.sql`, `api/admin.py`, `docker-compose.yaml`
-- **Status:** ✅ Production ready
-
-### ✅ P1-002 COMPLET: Package PACK_IMPORT_PARTENERI 
-- **Implementat:** 09 septembrie 2025, 10:30
-- **Key Features:**
-  - `cauta_sau_creeaza_partener()` - Search priority: cod_fiscal → denumire → create
-  - `parseaza_adresa_semicolon()` - Flexible address parsing cu defaults
-  - Individual vs company logic (CUI 13 digits)
-  - Custom exceptions + autonomous transaction logging
-- **Files:** `api/database-scripts/02_import_parteneri.sql`
-- **Status:** ✅ Production ready - 100% tested
-
-### ✅ P1-003 COMPLET: Package PACK_IMPORT_COMENZI
-- **Implementat:** 09 septembrie 2025, 10:30 | **Finalizat:** 10 septembrie 2025, 12:30  
-- **Key Features:**
-  - `gaseste_articol_roa()` - Complex SKU mapping cu pipelined functions ✅ 100% tested
-  - Manual workflow validation - comenzi + articole ✅ 100% working
-  - Support mapări: simple, reîmpachetări, seturi complexe ✅
-  - Performance monitoring < 30s per comandă ✅
-  - Schema reală MARIUSM_AUTO validation ✅
-- **Files:** `api/database-scripts/04_import_comenzi.sql` + `api/final_validation.py` 
-- **Status:** ✅ 100% Production ready cu componente validate
-
-### ✅ P1-004 Testing Manual Packages - 100% COMPLET
-- **Obiectiv:** Testare completă cu date reale ROA ✅
-- **Dependencies:** P1-001 ✅, P1-002 ✅, P1-003 ✅  
-- **Rezultate Finale:**
-  - ✅ PACK_IMPORT_PARTENERI: 100% funcțional cu parteneri reali
-  - ✅ gaseste_articol_roa: 100% funcțional cu mapări CAFE100 → CAF01
-  - ✅ Oracle connection, FK constraints, schema MARIUSM_AUTO identificată
-  - ✅ Manual workflow: comenzi + articole complet funcțional
-- **Status:** ✅ 100% COMPLET
-
-### 🔍 **FOR LOOP Issue REZOLVAT - Root Cause Analysis:**
-
-**PROBLEMA NU ERA CU FOR LOOP-ul!** For loop-ul era corect sintactic și logic.
-
-**Problemele Reale Identificate:**
-1. **Schema Incorectă:** Am presupus `comenzi_antet`/`comenzi_articole` dar schema reală folosește `COMENZI`/`COMENZI_ELEMENTE`
-2. **FK Constraints:** ID_POL=2, ID_VALUTA=3 (obligatorii, nu NULL sau alte valori)
-3. **JSON Parsing:** Probleme de conversie numerică în Oracle PL/SQL simplu
-4. **Environment:** Schema `MARIUSM_AUTO` pe Oracle 10g, nu environment-ul presupus inițial
-
-**Componente care funcționează 100%:**
-- ✅ `PACK_IMPORT_PARTENERI.cauta_sau_creeaza_partener()`
-- ✅ `PACK_IMPORT_COMENZI.gaseste_articol_roa()` 
-- ✅ Direct INSERT în `COMENZI`/`COMENZI_ELEMENTE`
-- ✅ Mapări complexe prin `ARTICOLE_TERTI`
-
-**Lecții Învățate:**
-- Verifică întotdeauna schema reală înainte de implementare
-- Testează FK constraints și valorile valide
-- Environment discovery este crucial pentru debugging
-- FOR LOOP logic era corect - problema era în presupuneri de structură
-
-### 🚀 **Phase 2 Ready - Validated Components:**
-Toate componentele individuale sunt validate și funcționează perfect pentru VFP integration.
-
----
-
-## 📋 User Stories Reference
-
-Toate story-urile pentru fiecare fază sunt stocate în `docs/stories/` cu detalii complete:
-
-### Phase 1 Stories [🎯 75% COMPLET]
-- **P1-001:** [Tabel ARTICOLE_TERTI](stories/P1-001-ARTICOLE_TERTI.md) - ✅ COMPLET
-- **P1-002:** [Package IMPORT_PARTENERI](stories/P1-002-Package-IMPORT_PARTENERI.md) - ✅ COMPLET  
-- **P1-003:** [Package IMPORT_COMENZI](stories/P1-003-Package-IMPORT_COMENZI.md) - ✅ COMPLET
-- **P1-004:** [Testing Manual Packages](stories/P1-004-Testing-Manual-Packages.md) - 🔄 READY TO START
-
-### Faze Viitoare
-- **Phase 2:** VFP Integration (stories vor fi generate după P1 completion)
-- **Phase 3:** Web Admin Interface  
-- **Phase 4:** Testing & Deployment
-
----
-
-**Document Owner:** Development Team  
-**Last Updated:** 10 septembrie 2025, 12:30 (Phase 1 COMPLET - schema MARIUSM_AUTO documented)  
-**Next Review:** Phase 2 VFP Integration planning
-
----
-
-## 🎉 **PHASE 1 COMPLETION SUMMARY**
-
-**Date Completed:** 10 septembrie 2025, 12:30  
-**Final Status:** ✅ 100% COMPLET
-
-**Critical Discoveries & Updates:**
-- ✅ Real Oracle schema: `MARIUSM_AUTO` (not CONTAFIN_ORACLE)
-- ✅ Real table names: `COMENZI`/`COMENZI_ELEMENTE` (not comenzi_antet/comenzi_articole)  
-- ✅ Required FK values: ID_POL=2, ID_VALUTA=3
-- ✅ All core components validated with real data
-- ✅ FOR LOOP issue resolved (was environment/schema mismatch)
-
-**Ready for Phase 2 with validated components:**
-- `PACK_IMPORT_PARTENERI.cauta_sau_creeaza_partener()` 
-- `PACK_IMPORT_COMENZI.gaseste_articol_roa()`
-- Direct SQL workflow for COMENZI/COMENZI_ELEMENTE
-- ARTICOLE_TERTI mappings system
-
----
-
-**SQL*Plus Access:**
-```bash
-docker exec -i gomag-admin sqlplus MARIUSM_AUTO/ROMFASTSOFT@ROA_CENTRAL
-```

docs/oracle-schema-notes.md

@@ -0,0 +1,122 @@
+# Oracle Schema Notes — MARIUSM_AUTO
+
+Reference pentru tabelele, procedurile și relațiile Oracle descoperite în debugging.
+
+## Tabele comenzi
+
+### COMENZI
+| Coloană | Tip | Notă |
+|---|---|---|
+| ID_COMANDA | NUMBER (PK) | Auto-generated |
+| COMANDA_EXTERNA | VARCHAR2 | Nr. comandă GoMag (ex: 481588552) |
+| DATA_COMANDA | DATE | |
+| ID_PART | NUMBER | FK → NOM_PARTENERI |
+| PROC_DISCOUNT | NUMBER(10,4) | Discount procentual pe comandă (setat 0 la import) |
+| STERS | NUMBER | Soft-delete flag |
+
+### COMENZI_ELEMENTE
+| Coloană | Tip | Notă |
+|---|---|---|
+| ID_COMANDA_ELEMENT | NUMBER (PK) | Auto-generated |
+| ID_COMANDA | NUMBER | FK → COMENZI |
+| ID_ARTICOL | NUMBER | FK → NOM_ARTICOLE |
+| ID_POL | NUMBER | FK → CRM_POLITICI_PRETURI |
+| PRET | NUMBER(14,3) | Preț per unitate (cu/fără TVA per PRET_CU_TVA flag) |
+| CANTITATE | NUMBER(14,3) | Cantitate (negativă pentru discount lines) |
+| DISCOUNT_UNITAR | NUMBER(20,4) | Default 0 |
+| PTVA | NUMBER | Procentul TVA (11, 21, etc.) |
+| PRET_CU_TVA | NUMBER(1) | 1 = prețul include TVA |
+| STERS | NUMBER | Soft-delete flag |
+
+**Discount lines**: qty negativă, pret pozitiv. Ex: qty=-1, pret=51.56 → scade 51.56 din total.
+
+## Tabele facturare
+
+### VANZARI
+| Coloană | Tip | Notă |
+|---|---|---|
+| ID_VANZARE | NUMBER (PK) | |
+| NUMAR_ACT | NUMBER | Număr factură (nract) |
+| SERIE_ACT | VARCHAR2 | Serie factură |
+| TIP | NUMBER | 3=factură pe bază de comandă, 1=factură simplă |
+| ID_COMANDA | NUMBER | FK → COMENZI (pentru TIP=3) |
+| ID_PART | NUMBER | FK → NOM_PARTENERI |
+| TOTAL_FARA_TVA | NUMBER | Total calculat de pack_facturare |
+| TOTAL_TVA | NUMBER | |
+| TOTAL_CU_TVA | NUMBER | |
+| DIFTOTFTVA | NUMBER | Diferența față de totalul trimis de client ROAFACTUARE |
+| DIFTOTTVA | NUMBER | |
+| STERS | NUMBER | |
+
+### VANZARI_DETALII
+| Coloană | Tip | Notă |
+|---|---|---|
+| **ID_VANZARE_DET** | NUMBER (PK) | ⚠ NU `id_detaliu`! |
+| ID_VANZARE | NUMBER | FK → VANZARI |
+| ID_ARTICOL | NUMBER | FK → NOM_ARTICOLE |
+| CANTITATE | NUMBER | |
+| PRET | NUMBER | Preț de vânzare |
+| PRET_ACHIZITIE | NUMBER | |
+| PROC_TVAV | NUMBER | Coeficient TVA (1.21, 1.11, etc.) |
+| ID_GESTIUNE | NUMBER | NULL pentru discount lines |
+| CONT | VARCHAR2 | '371', NULL pentru discount lines |
+| STERS | NUMBER | |
+
+## Tabele prețuri
+
+### CRM_POLITICI_PRETURI
+| Coloană | Tip | Notă |
+|---|---|---|
+| ID_POL | NUMBER (PK) | ID politică de preț |
+| PRETURI_CU_TVA | NUMBER | 1 = prețurile includ TVA |
+
+### CRM_POLITICI_PRET_ART
+| Coloană | Tip | Notă |
+|---|---|---|
+| ID_POL | NUMBER | FK → CRM_POLITICI_PRETURI |
+| ID_ARTICOL | NUMBER | FK → NOM_ARTICOLE |
+| PRET | NUMBER | Preț de listă (cu/fără TVA per PRETURI_CU_TVA din politică) |
+| PROC_TVAV | NUMBER | Coeficient TVA |
+
+Politici folosite: id_pol=39 (vânzare), id_pol=65 (transport).
+
+### ARTICOLE_TERTI
+| Coloană | Tip | Notă |
+|---|---|---|
+| SKU | VARCHAR2 | SKU din magazin web (GoMag) |
+| CODMAT | VARCHAR2 | CODMAT în ROA (FK → NOM_ARTICOLE.CODMAT) |
+| CANTITATE_ROA | NUMBER | Conversie: 1 web unit = X ROA units |
+| ACTIV | NUMBER | |
+| STERS | NUMBER | |
+
+**cantitate_roa semnificații**:
+- `1` → 1:1 (unitate identică web/ROA)
+- `0.5` → 1 web unit (50 buc) = 0.5 ROA set (100 buc). Price sync: `pret_web / 0.5`
+- `10` → bax 1000buc = 10 seturi ROA (100 buc). Kit pricing activ.
+- `22.5` → bax 2250buc = 22.5 seturi ROA (100 buc). Kit pricing activ.
+
+## Proceduri cheie
+
+### PACK_COMENZI.adauga_articol_comanda
+```
+(V_ID_COMANDA, V_ID_ARTICOL, V_ID_POL, V_CANTITATE, V_PRET, V_ID_UTIL, V_ID_SECTIE, V_PTVA)
+```
+- Lookup pret din CRM_POLITICI_PRET_ART, dar dacă V_PRET IS NOT NULL → folosește V_PRET
+- **NU inversează semnul prețului** — V_PRET se salvează ca atare
+- Check duplicat: dacă există rând cu același (id_articol, ptva, pret, sign(cantitate)) → eroare
+
+### PACK_FACTURARE flow (facturare pe bază de comandă, ntip=42)
+1. `cursor_comanda` → citește COMENZI_ELEMENTE, filtrează `SIGN(A.CANTITATE) * (A.CANTITATE - NVL(D.CANTITATE, 0)) > 0`
+2. `cursor_gestiuni_articol` → verifică stoc per articol
+3. `initializeaza_date_factura` → setează sesiune facturare
+4. `adauga_articol_factura` (×N) → inserează în VANZARI_DETALII_TEMP
+5. `scrie_factura2` → procesează temp, contabilizează
+6. `finalizeaza_scriere_verificare` → finalizează factura
+
+### PACK_SESIUNE
+- `nzecimale_pretv` — variabilă package, setată la login ROAFACTUARE
+- Inițializare: `pack_sesiune.getoptiunefirma(USER, 'PPRETV')` = **2** (pe MARIUSM_AUTO)
+- **Nu e setată** în context server-side (import comenzi) → folosim `getoptiunefirma` direct
+
+### OPTIUNI (tabel configurare)
+- Coloane: `VARNAME`, `VARVALUE` (⚠ NU `cod`/`valoare`)

docs/stories/P1-001-ARTICOLE_TERTI.md

@@ -1,41 +0,0 @@
-# Story P1-001: Tabel ARTICOLE_TERTI ✅ COMPLET
-
-**Story ID:** P1-001  
-**Titlu:** Creare infrastructură database și tabel ARTICOLE_TERTI  
-**As a:** Developer  
-**I want:** Să am tabelul ARTICOLE_TERTI funcțional cu Docker environment  
-**So that:** Să pot stoca mapările SKU complexe pentru import comenzi  
-
-## Acceptance Criteria
-- [x] ✅ Tabel ARTICOLE_TERTI cu structura specificată
-- [x] ✅ Primary Key compus (sku, codmat)
-- [x] ✅ Docker environment cu Oracle Instant Client
-- [x] ✅ Flask admin interface cu test conexiune
-- [x] ✅ Date test pentru mapări (reîmpachetare + set compus)
-- [x] ✅ Configurare tnsnames.ora pentru ROA
-
-## Technical Tasks
-- [x] ✅ Creare fișier `01_create_table.sql`
-- [x] ✅ Definire structură tabel cu validări
-- [x] ✅ Configurare Docker cu Oracle client
-- [x] ✅ Setup Flask admin interface
-- [x] ✅ Test conexiune Oracle ROA
-- [x] ✅ Insert date test pentru validare
-
-## Definition of Done
-- [x] ✅ Cod implementat și testat
-- [x] ✅ Tabel creat în Oracle fără erori
-- [x] ✅ Docker environment funcțional
-- [x] ✅ Conexiune Oracle validată
-- [x] ✅ Date test inserate cu succes
-- [x] ✅ Documentație actualizată în PRD
-
-**Estimate:** M (6-8 ore)  
-**Dependencies:** None  
-**Risk Level:** LOW  
-**Status:** ✅ COMPLET (08 septembrie 2025, 22:30)
-
-## Deliverables
-- **Files:** `api/01_create_table.sql`, `api/admin.py`, `docker-compose.yaml`
-- **Status:** ✅ Ready pentru testare cu ROA (10.0.20.36)
-- **Data completare:** 08 septembrie 2025, 22:30

docs/stories/P1-002-Package-IMPORT_PARTENERI.md

@@ -1,46 +0,0 @@
-# Story P1-002: Package IMPORT_PARTENERI
-
-**Story ID:** P1-002  
-**Titlu:** Implementare Package IMPORT_PARTENERI complet funcțional  
-**As a:** System  
-**I want:** Să pot căuta și crea automat parteneri în ROA  
-**So that:** Comenzile web să aibă parteneri valizi în sistemul ERP  
-
-## Acceptance Criteria
-- [x] ✅ Funcția `cauta_sau_creeaza_partener()` implementată
-- [x] ✅ Funcția `parseaza_adresa_semicolon()` implementată  
-- [x] ✅ Căutare parteneri după cod_fiscal (prioritate 1)
-- [x] ✅ Căutare parteneri după denumire exactă (prioritate 2)
-- [x] ✅ Creare partener nou cu `pack_def.adauga_partener()`
-- [x] ✅ Adăugare adresă cu `pack_def.adauga_adresa_partener2()`
-- [x] ✅ Separare nume/prenume pentru persoane fizice (CUI 13 cifre)
-- [x] ✅ Default București Sectorul 1 pentru adrese incomplete
-
-## Technical Tasks
-- [x] ✅ Creare fișier `02_import_parteneri.sql`
-- [x] ✅ Implementare function `cauta_sau_creeaza_partener`
-- [x] ✅ Implementare function `parseaza_adresa_semicolon`
-- [x] ✅ Adăugare validări pentru cod_fiscal
-- [x] ✅ Integrare cu package-urile existente pack_def
-- [x] ✅ Error handling pentru parteneri invalizi
-- [x] ✅ Logging pentru operațiile de creare parteneri
-
-## Definition of Done
-- [x] ✅ Cod implementat și testat
-- [x] ✅ Package compilat fără erori în Oracle
-- [ ] 🔄 Test manual cu date reale (P1-004)
-- [x] ✅ Error handling complet
-- [x] ✅ Logging implementat
-- [x] ✅ Documentație actualizată
-
-**Estimate:** M (6-8 ore) - ACTUAL: 4 ore (parallel development)  
-**Dependencies:** P1-001 ✅  
-**Risk Level:** MEDIUM (integrare cu pack_def existent) - MITIGATED ✅  
-**Status:** ✅ COMPLET (09 septembrie 2025, 10:30)
-
-## 🎯 Implementation Highlights
-- **Custom Exceptions:** 3 specialized exceptions for different error scenarios
-- **Autonomous Transaction Logging:** Non-blocking logging system
-- **Flexible Address Parser:** Handles multiple address formats gracefully
-- **Individual Detection:** Smart CUI-based logic for person vs company
-- **Production-Ready:** Complete validation, error handling, and documentation

docs/stories/P1-003-Package-IMPORT_COMENZI.md

@@ -1,49 +0,0 @@
-# Story P1-003: Package IMPORT_COMENZI
-
-**Story ID:** P1-003  
-**Titlu:** Implementare Package IMPORT_COMENZI cu logică mapare  
-**As a:** System  
-**I want:** Să pot importa comenzi web complete în ROA  
-**So that:** Comenzile de pe platformele web să ajungă automat în ERP  
-
-## Acceptance Criteria
-- [x] ✅ Funcția `gaseste_articol_roa()` implementată
-- [x] ✅ Funcția `importa_comanda_web()` implementată
-- [x] ✅ Verificare mapări în ARTICOLE_TERTI
-- [x] ✅ Fallback căutare directă în nom_articole
-- [x] ✅ Calcul cantități pentru reîmpachetări
-- [x] ✅ Calcul prețuri pentru seturi compuse
-- [x] ✅ Integrare cu PACK_COMENZI.adauga_comanda()
-- [x] ✅ Integrare cu PACK_COMENZI.adauga_articol_comanda()
-
-## Technical Tasks
-- [x] ✅ Creare fișier `03_import_comenzi.sql`
-- [x] ✅ Implementare function `gaseste_articol_roa`
-- [x] ✅ Implementare function `importa_comanda_web`
-- [x] ✅ Logică mapare SKU → CODMAT
-- [x] ✅ Calcul cantități cu cantitate_roa
-- [x] ✅ Calcul prețuri cu procent_pret
-- [x] ✅ Validare seturi (suma procent_pret = 100%)
-- [x] ✅ Error handling pentru SKU not found
-- [x] ✅ Logging pentru fiecare operație
-
-## Definition of Done
-- [x] ✅ Cod implementat și testat
-- [x] ✅ Package compilat fără erori în Oracle
-- [ ] 🔄 Test cu mapări simple și complexe (P1-004)
-- [x] ✅ Error handling complet
-- [x] ✅ Logging implementat
-- [x] ✅ Performance < 30s per comandă (monitorizare implementată)
-
-**Estimate:** L (8-12 ore) - ACTUAL: 5 ore (parallel development)  
-**Dependencies:** P1-001 ✅, P1-002 ✅  
-**Risk Level:** HIGH (logică complexă mapări + integrare PACK_COMENZI) - MITIGATED ✅  
-**Status:** ✅ COMPLET (09 septembrie 2025, 10:30)
-
-## 🎯 Implementation Highlights
-- **Pipelined Functions:** Memory-efficient processing of complex mappings
-- **Smart Mapping Logic:** Handles simple, repackaging, and set scenarios
-- **Set Validation:** 95-105% tolerance for percentage sum validation
-- **Performance Monitoring:** Built-in timing for 30s target compliance
-- **JSON Integration:** Ready for web platform order import
-- **Enterprise Logging:** Comprehensive audit trail with import_log table

docs/stories/P1-004-Testing-Manual-Packages.md

@@ -1,106 +0,0 @@
-# Story P1-004: Testing Manual Packages
-
-**Story ID:** P1-004  
-**Titlu:** Testare manuală completă package-uri Oracle  
-**As a:** Developer  
-**I want:** Să verific că package-urile funcționează corect cu date reale  
-**So that:** Să am încredere în stabilitatea sistemului înainte de Phase 2  
-
-## Acceptance Criteria
-- [x] ✅ Test creare partener nou cu adresă completă
-- [x] ✅ Test căutare partener existent după cod_fiscal  
-- [x] ✅ Test căutare partener existent după denumire
-- [x] ✅ Test import comandă cu SKU simplu (error handling verificat)
-- [x] ✅ Test import comandă cu reîmpachetare (CAFE100: 2→20 bucăți)
-- [x] ✅ Test import comandă cu set compus (SET01: 2×CAF01+1×FILTRU01)
-- [x] ⚠️ Verificare comenzi create corect în ROA (blocked by external dependency)
-- [x] ✅ Verificare logging complet în toate scenariile
-
-## Technical Tasks
-- [x] ✅ Pregătire date test pentru parteneri (created test partners)
-- [x] ✅ Pregătire date test pentru articole/mapări (created CAF01, FILTRU01 in nom_articole)
-- [x] ✅ Pregătire comenzi JSON test (comprehensive test suite)
-- [x] ✅ Rulare teste în Oracle SQL Developer (Python scripts via Docker)
-- [x] ⚠️ Verificare rezultate în tabele ROA (blocked by PACK_COMENZI)
-- [x] ✅ Validare calcule cantități și prețuri (verified with gaseste_articol_roa)
-- [x] ✅ Review log files pentru erori (comprehensive error handling tested)
-
-## Definition of Done
-- [x] ✅ Toate testele rulează cu succes (75% - blocked by external dependency)
-- [x] ⚠️ Comenzi vizibile și corecte în ROA (blocked by PACK_COMENZI.adauga_comanda CASE issue)
-- [x] ✅ Log files complete și fără erori (comprehensive logging verified)
-- [x] ✅ Performance requirements îndeplinite (gaseste_articol_roa < 1s)
-- [x] ✅ Documentare rezultate teste (detailed test results documented)
-
-## 📊 Test Results Summary
-
-**Date:** 09 septembrie 2025, 21:35  
-**Overall Success Rate:** 75% (3/4 major components)
-
-### ✅ PASSED Components:
-
-#### 1. PACK_IMPORT_PARTENERI - 100% SUCCESS
-- **Test 1:** ✅ Creare partener nou (persoană fizică) - PASS
-- **Test 2:** ✅ Căutare partener existent după denumire - PASS  
-- **Test 3:** ✅ Creare partener companie cu CUI - PASS
-- **Test 4:** ✅ Căutare companie după cod fiscal - PASS
-- **Logic:** Priority search (cod_fiscal → denumire → create) works correctly
-
-#### 2. PACK_IMPORT_COMENZI.gaseste_articol_roa - 100% SUCCESS
-- **Test 1:** ✅ Reîmpachetare CAFE100: 2 web → 20 ROA units, price=5.0 lei/unit - PASS
-- **Test 2:** ✅ Set compus SET01: 1 set → 2×CAF01 + 1×FILTRU01, percentages 65%+35% - PASS  
-- **Test 3:** ✅ Unknown SKU: returns correct error message - PASS
-- **Performance:** < 1 second per SKU resolution
-
-#### 3. PACK_JSON - 100% SUCCESS
-- **parse_array:** ✅ Correctly parses JSON arrays - PASS
-- **get_string/get_number:** ✅ Extracts values correctly - PASS
-- **Integration:** Ready for importa_comanda function
-
-### ⚠️ BLOCKED Component:
-
-#### 4. PACK_IMPORT_COMENZI.importa_comanda - BLOCKED by External Dependency
-- **Issue:** `PACK_COMENZI.adauga_comanda` (ROA system) has CASE statement error at line 190
-- **Our Code:** ✅ JSON parsing, article mapping, and logic are correct
-- **Impact:** Full order import workflow cannot be completed
-- **Recommendation:** Consult ROA team for PACK_COMENZI fix before Phase 2
-
-### 🔧 Infrastructure Created:
-- ✅ Test articles: CAF01, FILTRU01 in nom_articole
-- ✅ Test partners: Ion Popescu Test, Test Company SRL  
-- ✅ Comprehensive test scripts in api/
-- ✅ ARTICOLE_TERTI mappings verified (3 active mappings)
-
-### 📋 Phase 2 Readiness:
-- ✅ **PACK_IMPORT_PARTENERI:** Production ready
-- ✅ **PACK_IMPORT_COMENZI.gaseste_articol_roa:** Production ready  
-- ⚠️ **Full order import:** Requires ROA team collaboration
-
-**Estimate:** S (4-6 ore) ✅ **COMPLETED**  
-**Dependencies:** P1-002 ✅, P1-003 ✅  
-**Risk Level:** LOW → **MEDIUM** (external dependency identified)  
-**Status:** **95% COMPLETED** - Final issue identified
-
-## 🔍 **Final Issue Discovered:**
-
-**Problem:** `importa_comanda` returnează "Niciun articol nu a fost procesat cu succes" chiar și după eliminarea tuturor pINFO logging calls.
-
-**Status la oprirea sesiunii:**
-- ✅ PACK_IMPORT_PARTENERI: 100% funcțional
-- ✅ PACK_IMPORT_COMENZI.gaseste_articol_roa: 100% funcțional individual
-- ✅ V_INTERNA = 2 fix aplicat
-- ✅ PL/SQL blocks pentru DML calls
-- ✅ Partner creation cu ID-uri valide (878, 882, 883)
-- ✅ Toate pINFO calls comentate în 04_import_comenzi.sql
-- ⚠️ importa_comanda încă nu procesează articolele în FOR LOOP
-
-**Următorii pași pentru debug (mâine):**
-1. Investigare FOR LOOP din importa_comanda linia 324-325
-2. Test PACK_JSON.parse_array separat  
-3. Verificare dacă problema e cu pipelined function în context de loop
-4. Posibilă soluție: refactoring la importa_comanda să nu folosească SELECT FROM TABLE în FOR
-
-**Cod funcțional pentru Phase 2 VFP:**
-- Toate package-urile individuale funcționează perfect
-- VFP poate apela PACK_IMPORT_PARTENERI + gaseste_articol_roa separat
-- Apoi manual PACK_COMENZI.adauga_comanda/adauga_articol_comanda

scripts/HANDOFF_MAPPING.md

@@ -72,10 +72,9 @@ Cand o comanda are produse complet diferite fata de factura, algoritmul forteaza
 - Exemplu: "Lavazza Crema E Aroma Cafea Boabe 1 Kg" vs "LAVAZZA BBE CREMA E AROMA"
 - Ar putea fi mai precis decat match pe pret, mai ales cand preturile coincid accidental
 
-### Tools utile deja existente:
+### Tools (nota: scripturile de matching au fost sterse din repo)
-- `scripts/compare_order.py <order_nr> <fact_nr>` — comparare detaliata o comanda vs o factura
+Scripturile `match_all.py`, `compare_order.py`, `fetch_one_order.py` au fost eliminate.
-- `scripts/fetch_one_order.py <order_nr>` — fetch JSON complet din GoMag API
+Strategia de matching descrisa mai sus ramane valida ca referinta conceptuala.
-- `scripts/match_all.py` — matching bulk (de refacut cu strategie noua)
 
 ## Structura Oracle relevanta
 

test.sh

@@ -9,17 +9,42 @@ cd "$(dirname "$0")"
 GREEN='\033[32m'
 RED='\033[31m'
 YELLOW='\033[33m'
+CYAN='\033[36m'
 RESET='\033[0m'
 
+# ─── Log file setup ──────────────────────────────────────────────────────────
+LOG_DIR="qa-reports"
+mkdir -p "$LOG_DIR"
+TIMESTAMP=$(date '+%Y%m%d_%H%M%S')
+LOG_FILE="${LOG_DIR}/test_run_${TIMESTAMP}.log"
+
+# Strip ANSI codes for log file
+strip_ansi() {
+    sed 's/\x1b\[[0-9;]*m//g'
+}
+
+# Tee to both terminal and log file (log without colors)
+log_tee() {
+    tee >(strip_ansi >> "$LOG_FILE")
+}
+
 # ─── Stage tracking ───────────────────────────────────────────────────────────
 declare -a STAGE_NAMES=()
 declare -a STAGE_RESULTS=()   # 0=pass, 1=fail, 2=skip
+declare -a STAGE_SKIPPED=()   # count of skipped tests per stage
+declare -a STAGE_DETAILS=()   # pytest summary line per stage
 EXIT_CODE=0
+TOTAL_SKIPPED=0
 
 record() {
     local name="$1"
     local code="$2"
+    local skipped="${3:-0}"
+    local details="${4:-}"
     STAGE_NAMES+=("$name")
+    STAGE_SKIPPED+=("$skipped")
+    STAGE_DETAILS+=("$details")
+    TOTAL_SKIPPED=$((TOTAL_SKIPPED + skipped))
     if [ "$code" -eq 0 ]; then
         STAGE_RESULTS+=(0)
     else
@@ -31,6 +56,8 @@ record() {
 skip_stage() {
     STAGE_NAMES+=("$1")
     STAGE_RESULTS+=(2)
+    STAGE_SKIPPED+=(0)
+    STAGE_DETAILS+=("")
 }
 
 # ─── Environment setup ────────────────────────────────────────────────────────
@@ -140,29 +167,52 @@ run_stage() {
     shift
     echo ""
     echo -e "${YELLOW}=== $label ===${RESET}"
+
+    # Capture output for skip parsing while showing it live
+    local tmpout
+    tmpout=$(mktemp)
     set +e
-    "$@"
+    "$@" 2>&1 | tee "$tmpout" | log_tee
-    local code=$?
+    local code=${PIPESTATUS[0]}
     set -e
-    record "$label" $code
+
+    # Parse pytest summary line for skip count
+    # Matches lines like: "= 5 passed, 3 skipped in 1.23s ="
+    local skipped=0
+    local summary_line=""
+    summary_line=$(grep -E '=+.*passed|failed|error|skipped.*=+' "$tmpout" | tail -1 || true)
+    if [ -n "$summary_line" ]; then
+        skipped=$(echo "$summary_line" | grep -oP '\d+(?= skipped)' || echo "0")
+        [ -z "$skipped" ] && skipped=0
+    fi
+    rm -f "$tmpout"
+
+    record "$label" $code "$skipped" "$summary_line"
     # Don't return $code — let execution continue to next stage
 }
 
 # ─── Summary box ──────────────────────────────────────────────────────────────
 print_summary() {
     echo ""
-    echo -e "${YELLOW}╔══════════════════════════════════════════╗${RESET}"
+    echo -e "${YELLOW}╔══════════════════════════════════════════════════╗${RESET}"
     echo -e "${YELLOW}║            TEST RESULTS SUMMARY                  ║${RESET}"
-    echo -e "${YELLOW}╠══════════════════════════════════════════╣${RESET}"
+    echo -e "${YELLOW}╠══════════════════════════════════════════════════╣${RESET}"
 
     for i in "${!STAGE_NAMES[@]}"; do
         local name="${STAGE_NAMES[$i]}"
         local result="${STAGE_RESULTS[$i]}"
-        # Pad name to 26 chars
+        local skipped="${STAGE_SKIPPED[$i]}"
+        # Pad name to 24 chars
         local padded
-        padded=$(printf "%-26s" "$name")
+        padded=$(printf "%-24s" "$name")
         if [ "$result" -eq 0 ]; then
+            if [ "$skipped" -gt 0 ]; then
+                local skip_note
+                skip_note=$(printf "passed (%d skipped)" "$skipped")
+                echo -e "${YELLOW}║${RESET} ${GREEN}✅${RESET} ${padded} ${GREEN}passed${RESET} ${CYAN}(${skipped} skipped)${RESET}  ${YELLOW}║${RESET}"
+            else
                 echo -e "${YELLOW}║${RESET} ${GREEN}✅${RESET} ${padded} ${GREEN}passed${RESET}             ${YELLOW}║${RESET}"
+            fi
         elif [ "$result" -eq 1 ]; then
             echo -e "${YELLOW}║${RESET} ${RED}❌${RESET} ${padded} ${RED}FAILED${RESET}             ${YELLOW}║${RESET}"
         else
@@ -170,14 +220,19 @@ print_summary() {
         fi
     done
 
-    echo -e "${YELLOW}╠══════════════════════════════════════════╣${RESET}"
+    echo -e "${YELLOW}╠══════════════════════════════════════════════════╣${RESET}"
     if [ "$EXIT_CODE" -eq 0 ]; then
+        if [ "$TOTAL_SKIPPED" -gt 0 ]; then
+            echo -e "${YELLOW}║${RESET} ${GREEN}All stages passed!${RESET} ${CYAN}(${TOTAL_SKIPPED} tests skipped total)${RESET}   ${YELLOW}║${RESET}"
+        else
             echo -e "${YELLOW}║${RESET} ${GREEN}All stages passed!${RESET}                             ${YELLOW}║${RESET}"
+        fi
     else
         echo -e "${YELLOW}║${RESET} ${RED}Some stages FAILED — check output above${RESET}        ${YELLOW}║${RESET}"
     fi
-    echo -e "${YELLOW}║ Health Score: see qa-reports/            ║${RESET}"
+    echo -e "${YELLOW}║${RESET} Log: ${CYAN}${LOG_FILE}${RESET}"
-    echo -e "${YELLOW}╚══════════════════════════════════════════╝${RESET}"
+    echo -e "${YELLOW}║${RESET} Health Score: see qa-reports/"
+    echo -e "${YELLOW}╚══════════════════════════════════════════════════╝${RESET}"
 }
 
 # ─── Cleanup trap ────────────────────────────────────────────────────────────
@@ -193,6 +248,10 @@ fi
 
 setup_env
 
+# Write log header
+echo "=== test.sh ${MODE} — $(date '+%Y-%m-%d %H:%M:%S') ===" > "$LOG_FILE"
+echo "" >> "$LOG_FILE"
+
 case "$MODE" in
     ci)
         run_stage "Unit tests"      python -m pytest -m unit -v
@@ -258,5 +317,7 @@ case "$MODE" in
         ;;
 esac
 
-print_summary
+print_summary 2>&1 | log_tee
+echo ""
+echo -e "${CYAN}Full log saved to: ${LOG_FILE}${RESET}"
 exit $EXIT_CODE

test_import_comanda.py

@@ -1,114 +0,0 @@
-#!/usr/bin/env python3
-"""
-Test script for updated IMPORT_COMENZI package
-Tests the fixed FOR LOOP issue
-"""
-
-import os
-import sys
-import oracledb
-from dotenv import load_dotenv
-
-# Load environment variables
-load_dotenv('/mnt/e/proiecte/vending/gomag-vending/api/.env')
-
-def test_import_comanda():
-    """Test the updated importa_comanda function"""
-    
-    # Connection parameters
-    user = os.environ['ORACLE_USER']  
-    password = os.environ['ORACLE_PASSWORD']
-    dsn = os.environ['ORACLE_DSN']
-    
-    try:
-        # Connect to Oracle
-        print("🔗 Conectare la Oracle...")
-        with oracledb.connect(user=user, password=password, dsn=dsn) as conn:
-            with conn.cursor() as cursor:
-                
-                print("\n📋 Test 1: Recompilare Package PACK_IMPORT_COMENZI")
-                
-                # Read and execute the updated package
-                with open('/mnt/e/proiecte/vending/gomag-vending/api/database-scripts/04_import_comenzi.sql', 'r') as f:
-                    sql_script = f.read()
-                
-                cursor.execute(sql_script)
-                print("✅ Package recompiled successfully")
-                
-                print("\n📋 Test 2: Import comandă completă cu multiple articole")
-                
-                # Test data - comandă cu 2 articole (CAFE100 + SET01)
-                test_json = '''[
-                    {"sku": "CAFE100", "cantitate": 2, "pret": 50.00},
-                    {"sku": "SET01", "cantitate": 1, "pret": 120.00}
-                ]'''
-                
-                test_partner_id = 878  # Partner din teste anterioare
-                test_order_num = "TEST-MULTI-" + str(int(os.time()))
-                
-                # Call importa_comanda
-                cursor.execute("""
-                    SELECT PACK_IMPORT_COMENZI.importa_comanda_web(
-                        :p_nr_comanda_ext,
-                        SYSDATE,
-                        :p_id_partener, 
-                        :p_json_articole,
-                        NULL,
-                        'Test import multiple articole'
-                    ) AS id_comanda FROM dual
-                """, {
-                    'p_nr_comanda_ext': test_order_num,
-                    'p_id_partener': test_partner_id,
-                    'p_json_articole': test_json
-                })
-                
-                result = cursor.fetchone()
-                if result and result[0] > 0:
-                    comanda_id = result[0]
-                    print(f"✅ Comandă importată cu succes! ID: {comanda_id}")
-                    
-                    # Verifică articolele adăugate
-                    cursor.execute("""
-                        SELECT ca.id_articol, na.codmat, ca.cantitate, ca.pret 
-                        FROM comenzi_articole ca
-                        JOIN nom_articole na ON na.id_articol = ca.id_articol
-                        WHERE ca.id_comanda = :id_comanda
-                        ORDER BY ca.id_articol
-                    """, {'id_comanda': comanda_id})
-                    
-                    articole = cursor.fetchall()
-                    print(f"\n📦 Articole în comandă (Total: {len(articole)}):")
-                    for art in articole:
-                        print(f"   • CODMAT: {art[1]}, Cantitate: {art[2]}, Preț: {art[3]}")
-                        
-                    # Expected: 
-                    # - CAFFE (din CAFE100: 2 * 10 = 20 bucăți)
-                    # - CAFE-SET (din SET01: 2 * 60% = 72.00)  
-                    # - FILT-SET (din SET01: 1 * 40% = 48.00)
-                    print("\n🎯 Expected:")
-                    print("   • CAFFE: 20 bucăți (reîmpachetare 2*10)")
-                    print("   • CAFE-SET: 2 bucăți, preț 36.00 (120*60%/2)")
-                    print("   • FILT-SET: 1 bucăți, preț 48.00 (120*40%/1)")
-                    
-                else:
-                    print("❌ Import eșuat")
-                    # Check for errors
-                    cursor.execute("SELECT PACK_IMPORT_COMENZI.get_last_error() FROM dual")
-                    error = cursor.fetchone()
-                    if error:
-                        print(f"Eroare: {error[0]}")
-                
-                conn.commit()
-                print("\n✅ Test completed!")
-                
-    except Exception as e:
-        print(f"❌ Eroare: {e}")
-        return False
-        
-    return True
-
-if __name__ == "__main__":
-    import time
-    os.time = lambda: int(time.time())
-    success = test_import_comanda()
-    sys.exit(0 if success else 1)