Skip to content

refactor(m11-04): decouper les ecrans de scan camera et code-barres (M-11-#184) - #198

Merged
donovan-dev-web merged 2 commits into
developfrom
feature/m11-04-decouper-ecrans-scan
Sep 8, 2026
Merged

donovan-dev-web merged 2 commits into
developfrom
feature/m11-04-decouper-ecrans-scan

Conversation

@donovan-dev-web

Copy link
Copy Markdown
Owner

Objectif

Refactor: split the two barcode/OCR scan screens so each working file is well under 300 lines (M-11-#184).

Fichiers modifies

  • src/app/scan/camera.tsx: 831 -> 128 lines
    • extract src/components/scan/use-ocr-analysis.ts (OCR polling hook, state machine, US-ID-08/09)
    • extract src/components/scan/ocr-analysis.ts (pure helpers + types: DetectedInfo, buildManualParams, confidenceLabel)
    • extract ocr-analyzing-overlay.tsx / ocr-confirm-overlay.tsx / ocr-result-overlay.tsx (view states)
    • extract camera-ocr-styles.ts (themes)
  • src/app/scan/barcode.tsx: 551 -> 94 lines
    • extract src/components/scan/use-barcode-scanning.ts (stabilizer, single + continuous flow)
    • extract barcode-overlay-controls.tsx / barcode-pending-sheets.tsx / barcode-styles.ts

Nouveaux tests

  • src/components/tests/ocr-analysis.test.ts: buildManualParams, confidenceLabel, hasAnyDetected

Verifications

  • 303 tests passent (dont 26 tests ecrans scan inchanges)
  • Lint: 0 erreurs / 0 warnings
  • Aucun changement de comportement: tous les testID et exports par defaut preserves

Closes #184

#184)

- barcode-pending-sheets : confirm/success/unknown + bandeau continu
- ocr-analyzing-overlay : champs détectés, guide, torche, validation
- ocr-confirm-overlay : édition des champs, filtrage du numéro, actions
@donovan-dev-web
donovan-dev-web merged commit 2072f82 into develop Sep 8, 2026
1 check passed
@donovan-dev-web
donovan-dev-web deleted the feature/m11-04-decouper-ecrans-scan branch September 8, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant