Merge pull request #35 from maribakulj/claude/analyze-app-issues-MYBpt

fix: page IDs with trailing dots, Reader profile handling, Editor 404 UX

backend/app/api/v1/ingest.py

@@ -40,8 +40,13 @@ _ALLOWED_MIME_PREFIXES = ("image/",)
 
 
 def _sanitize_label(label: str) -> str:
-    """Nettoie un folio_label : garde uniquement alphanum, -, _, ."""
+    """Nettoie un folio_label : garde uniquement alphanum, -, _, .
+
+    Les points en début/fin sont supprimés pour éviter des IDs de page
+    se terminant par un point (problèmes de routing et de chemins fichier).
+    """
     clean = Path(label).name  # retire tout chemin
+    clean = clean.strip(".")  # supprimer les points en début/fin
     if not _SAFE_LABEL_RE.match(clean) or not clean:
         clean = re.sub(r"[^\w\-\.]", "_", clean) or "page"
     return clean

frontend/src/pages/Editor.tsx

@@ -67,7 +67,12 @@ export default function Editor() {
       const ext = m.extensions as { region_validations?: Record<string, string> } | undefined
       setRegionValidations(ext?.region_validations ?? {})
     } catch (e: unknown) {
-      setError((e as Error).message)
+      const msg = (e as Error).message ?? ''
+      if (msg.includes('404')) {
+        setError('Cette page n\'a pas encore ete analysee par l\'IA. Lancez le pipeline depuis Administration.')
+      } else {
+        setError(msg)
+      }
     } finally {
       setLoading(false)
     }

frontend/src/pages/Reader.tsx

@@ -34,12 +34,19 @@ export default function Reader() {
   const [error, setError] = useState<string | null>(null)
 
   useEffect(() => {
-    Promise.all([fetchPages(manuscriptId), fetchProfile(profileId)])
+    const loadPages = fetchPages(manuscriptId)
+    const loadProfile = profileId
+      ? fetchProfile(profileId).catch(() => null)
+      : Promise.resolve(null)
+
+    Promise.all([loadPages, loadProfile])
       .then(([pgs, prof]) => {
         const sorted = [...pgs].sort((a, b) => a.sequence - b.sequence)
         setPages(sorted)
-        setProfile(prof)
-        setVisibleLayers(new Set(prof.active_layers))
+        if (prof) {
+          setProfile(prof)
+          setVisibleLayers(new Set(prof.active_layers))
+        }
       })
       .catch((e: Error) => setError(e.message))
       .finally(() => setLoading(false))