fix(sprint-f4-f5): tests faux-positifs, frontend bugs, Dockerfile unique

Sprint F4 — Correction des tests faux-positifs :
- IIIF double-slash : le replace("://","X") masquait le bug, corrigé
avec split("://",1) + vérification du path
- METS OBJID : ajout assert is not None avant comparaison
- Corrections archive : vérification du contenu JSON archivé (pas juste
le path), assertion version == 1 dans l'archive
- Ingest idempotence : vérification BDD (count pages == 2) après
réingestion, pas juste les compteurs de réponse

Sprint F5 — Frontend + infra :
- Editor imageUrl : remplacé `master ? '' : ''` par
`master?.image?.derivative_web ?? master?.image?.master ?? ''`
- SearchBar : connecté onSelectResult → onOpenPage → Editor dans
Home.tsx et App.tsx (navigation fonctionnelle)
- api.ts : warning console si VITE_API_URL absent en production
- Dockerfile : supprimé infra/Dockerfile (copie divergente),
Dockerfile racine est désormais la source unique

477 tests passants, 0 échecs.

https://claude.ai/code/session_015Lht7wNQRzhUaLw94dE9z9

Dockerfile

@@ -1,6 +1,6 @@
 # Scriptorium AI — image de production (multi-stage)
 # Ce fichier est utilisé par HuggingFace Spaces (SDK docker, détection automatique).
-# Il doit rester synchronisé avec infra/Dockerfile.
+# Source unique — le fichier infra/Dockerfile a été supprimé pour éviter la divergence.
 #
 # Build depuis la racine du dépôt :
 #   docker build -t scriptorium-ai .

backend/tests/test_api_corrections.py

@@ -238,13 +238,13 @@ async def test_corrections_archives_old_version(async_client, db_session, monkey
     ms = await _create_manuscript(db_session, corpus.id)
     page = await _create_page(db_session, ms.id)
 
-    written_paths: list[str] = []
+    written_data: dict[str, str] = {}
 
     monkeypatch.setattr(Path, "exists", lambda self: True)
     monkeypatch.setattr(Path, "read_text", lambda self, **kw: _make_master(page.id, version=1))
 
     def _capture_write(self: Path, content: str, **kw: object) -> None:
-        written_paths.append(str(self))
+        written_data[str(self)] = content
 
     monkeypatch.setattr(Path, "write_text", _capture_write)
 
@@ -254,10 +254,17 @@ async def test_corrections_archives_old_version(async_client, db_session, monkey
     )
 
     # Deux écritures attendues : master_v1.json (archive) + master.json (nouveau)
+    written_paths = list(written_data.keys())
     assert len(written_paths) >= 2
     assert any("master_v1.json" in p for p in written_paths)
     assert any("master.json" in p and "master_v" not in p for p in written_paths)
 
+    # Vérifier que l'archive contient bien la version originale (v1)
+    import json as _json
+    archive_path = next(p for p in written_paths if "master_v1.json" in p)
+    archive_data = _json.loads(written_data[archive_path])
+    assert archive_data["editorial"]["version"] == 1
+
 
 @pytest.mark.asyncio
 async def test_corrections_multiple_fields(async_client, db_session, monkeypatch):

backend/tests/test_api_ingest.py

@@ -457,6 +457,15 @@ async def test_reingest_manifest_skips_existing_pages(async_client, db_session,
     assert data2["pages_created"] == 0
     assert data2["pages_skipped"] == 2
 
+    # Vérifier que la BDD n'a bien que 2 pages (pas de doublons)
+    from sqlalchemy import select as sa_select
+    from app.models.corpus import PageModel
+    page_result = await db_session.execute(
+        sa_select(PageModel).where(PageModel.manuscript_id == data1["manuscript_id"])
+    )
+    pages_in_db = list(page_result.scalars().all())
+    assert len(pages_in_db) == 2
+
 
 @pytest.mark.asyncio
 async def test_reingest_images_skips_existing_pages(async_client, db_session):

backend/tests/test_export_iiif.py

@@ -480,7 +480,10 @@ def test_base_url_trailing_slash_stripped():
     """Un base_url avec slash final ne génère pas de double slash dans les IDs."""
     pages = [_make_page("ms-0001r", "0001r", 1)]
     manifest = generate_manifest(pages, _base_meta(), "https://example.com/")
-    assert "//" not in manifest["id"].replace("://", "X")
+    manifest_id = manifest["id"]
+    # Retirer le protocole puis vérifier qu'il n'y a pas de double slash
+    without_protocol = manifest_id.split("://", 1)[1]
+    assert "//" not in without_protocol
 
 
 # ---------------------------------------------------------------------------

backend/tests/test_export_mets.py

@@ -195,7 +195,9 @@ def test_generate_mets_namespace(beatus_pages, beatus_meta):
 
 def test_generate_mets_objid(beatus_pages, beatus_meta):
     root = _parse(generate_mets(beatus_pages, beatus_meta))
-    assert root.get("OBJID") == "BnF-Latin-8878"
+    objid = root.get("OBJID")
+    assert objid is not None, "OBJID attribute absent du root mets"
+    assert objid == "BnF-Latin-8878"
 
 
 def test_generate_mets_label(beatus_pages, beatus_meta):

frontend/src/App.tsx

@@ -42,6 +42,7 @@ export default function App() {
       onOpenManuscript={(manuscriptId, profileId) =>
         setView({ name: 'reader', manuscriptId, profileId })
       }
+      onOpenPage={(pageId) => setView({ name: 'editor', pageId })}
       onAdmin={() => setView({ name: 'admin' })}
     />
   )

frontend/src/lib/api.ts

@@ -1,5 +1,13 @@
 const BASE_URL: string = import.meta.env.VITE_API_URL ?? ''
 
+if (!BASE_URL && import.meta.env.PROD) {
+  console.warn(
+    '[Scriptorium] VITE_API_URL non défini en production. ' +
+    'Les appels API utiliseront des chemins relatifs, ce qui peut échouer ' +
+    'si le frontend n\'est pas servi par le même domaine que le backend.'
+  )
+}
+
 // ── Types ─────────────────────────────────────────────────────────────────────
 
 export interface ProviderInfo {

frontend/src/pages/Editor.tsx

@@ -119,7 +119,7 @@ export default function Editor({ pageId, onBack }: Props) {
     return <div className="p-8 text-red-600">Erreur : {error}</div>
   }
 
-  const imageUrl = master ? '' : '' // image path not directly stored on PageMaster
+  const imageUrl = master?.image?.derivative_web ?? master?.image?.master ?? ''
   const regions = master?.layout?.regions ?? []
 
   return (

frontend/src/pages/Home.tsx

@@ -10,10 +10,11 @@ import {
 
 interface Props {
   onOpenManuscript: (manuscriptId: string, profileId: string) => void
+  onOpenPage?: (pageId: string) => void
   onAdmin: () => void
 }
 
-export default function Home({ onOpenManuscript, onAdmin }: Props) {
+export default function Home({ onOpenManuscript, onOpenPage, onAdmin }: Props) {
   const [corpora, setCorpora] = useState<Corpus[]>([])
   const [loading, setLoading] = useState(true)
   const [error, setError] = useState<string | null>(null)
@@ -73,7 +74,7 @@ export default function Home({ onOpenManuscript, onAdmin }: Props) {
           </p>
         </div>
         <div className="flex items-center gap-4">
-          <SearchBar />
+          <SearchBar onSelectResult={onOpenPage ? (r) => onOpenPage(r.page_id) : undefined} />
           <AdminNav onClick={onAdmin} />
         </div>
       </header>

infra/Dockerfile

@@ -1,71 +0,0 @@
-# Scriptorium AI — image de production (multi-stage)
-# Ce fichier est la copie exacte de Dockerfile (racine).
-# Build depuis la racine du dépôt :
-#   docker build -f infra/Dockerfile -t scriptorium-ai .
-#
-# Structure attendue dans l'image :
-#   /app/backend/app/   ← source Python (importable via PYTHONPATH)
-#   /app/profiles/      ← profils JSON
-#   /app/prompts/       ← templates de prompts
-#   /app/static/        ← frontend React buildé
-#   /app/data/          ← créé vide ; à monter en volume pour les artefacts
-
-# ── Stage 1 : build du frontend React ────────────────────────────────────────
-FROM node:20-slim AS frontend-builder
-
-WORKDIR /frontend
-
-# Installer les dépendances (cache layer séparé)
-COPY frontend/package.json ./
-RUN npm install
-
-# Copier les sources et builder
-COPY frontend/ ./
-RUN npm run build
-
-# ── Stage 2 : image Python finale ────────────────────────────────────────────
-FROM python:3.11-slim
-
-WORKDIR /app
-
-# ── Dépendances Python ─────────────────────────────────────────────────────
-# On copie uniquement pyproject.toml pour exploiter le cache de layers Docker.
-# Un stub app/__init__.py satisfait setuptools (discover packages) sans avoir
-# besoin de copier tout le code source à ce stade.
-COPY backend/pyproject.toml /tmp/build/
-RUN mkdir -p /tmp/build/app \
-    && touch /tmp/build/app/__init__.py \
-    && pip install --no-cache-dir --upgrade /tmp/build/ \
-    && rm -rf /tmp/build
-
-# ── Layer dédié mistralai — invalide le cache HF si v0.x est présent ─────
-# Layer séparé de l'install principal pour forcer la mise à jour même si
-# HuggingFace réutilise le layer pyproject.toml depuis un build antérieur.
-RUN pip install --no-cache-dir 'mistralai>=1.0,<2.0'
-
-# ── Code source backend ────────────────────────────────────────────────────
-COPY backend/app ./backend/app
-COPY profiles/ ./profiles/
-COPY prompts/ ./prompts/
-
-# ── Frontend buildé ────────────────────────────────────────────────────────
-COPY --from=frontend-builder /frontend/dist ./static
-
-# ── Répertoire des artefacts (vide dans l'image ; monté en volume) ─────────
-RUN mkdir -p /app/data
-
-# ── Secrets Google AI : JAMAIS dans l'image (R06) ─────────────────────────
-# Passer au runtime via -e ou docker-compose environment :
-#   AI_PROVIDER, GOOGLE_AI_STUDIO_API_KEY, GOOGLE_AI_API_KEY,
-#   GOOGLE_VERTEX_PROJECT, GOOGLE_VERTEX_LOCATION
-
-# PYTHONPATH permet l'import `app.main:app` depuis /app/backend/app/
-ENV PYTHONPATH=/app/backend
-ENV PROFILES_DIR=/app/profiles
-ENV PROMPTS_DIR=/app/prompts
-ENV DATA_DIR=/app/data
-
-EXPOSE 7860
-
-# 1 worker au MVP — pas de Gunicorn, pas de multiprocessing
-CMD ["uvicorn", "app.main:app", "--host", "0.0.0.0", "--port", "7860", "--workers", "1"]