fix(storage): split_part statt storage.foldername in Policies

Die Storage-API droppt/erstellt storage.foldername() bei ihren Boot-
Migrations neu. Policies, die davon abhingen, blockierten den Drop
('cannot drop function foldername') und schickten die Storage-API in
eine Crash-Loop. split_part(name,'/',1) liefert dieselbe erste
Pfad-Komponente (studio_id) ohne diese Kopplung.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

latest.json

@@ -1,11 +1,11 @@
 {
-  "version": "0.8.0",
-  "notes": "Rapport 0.8.0",
-  "pub_date": "2026-05-23T17:12:56Z",
+  "version": "0.8.2",
+  "notes": "Rapport 0.8.2",
+  "pub_date": "2026-05-23T17:54:59Z",
   "platforms": {
     "darwin-aarch64": {
-      "signature": "dW50cnVzdGVkIGNvbW1lbnQ6IHNpZ25hdHVyZSBmcm9tIHRhdXJpIHNlY3JldCBrZXkKUlVSUjV0czVoWDR4NGs2aHVOMzc4NUhtN092d1ZLa2Jobk1CY2prUkt6WlVFclRLQzF4Y3pDZk13djFiWkVUTm1QZU1LSTE4MmJoQjhtZ0ZBT2tQalBUaDdCMnRIM2ZzNmdnPQp0cnVzdGVkIGNvbW1lbnQ6IHRpbWVzdGFtcDoxNzc5NTU2Mzc2CWZpbGU6UkFQUE9SVCBQUkUtUkVMRUFTRS5hcHAudGFyLmd6CkNFaEtaQTF5SUJFMG84MVBNeEdaRXBleEtRZ1oyeUNjaU9ydWtzUkIxU21BU1RRY0I2QjREcEhEZlBEQitCRUQrNFowbFJ3V01aYjFhL2dwRGZDVkNBPT0K",
-      "url": "https://git.kgva.ch/karim/RAPPORT/releases/download/0.8.0/RAPPORT%20PRE-RELEASE.app.tar.gz"
+      "signature": "dW50cnVzdGVkIGNvbW1lbnQ6IHNpZ25hdHVyZSBmcm9tIHRhdXJpIHNlY3JldCBrZXkKUlVSUjV0czVoWDR4NHFSRmtHVUhpSkFlMDFwVS9XRGZjOGh1MnoybDNVcWh4ZkpIcGFQTFhVTHg5TVJ1SFA5L2kxbVQ5NUYvYmRsdlVmTHExcWZ4NThqeUZvQmk4WCtXTkEwPQp0cnVzdGVkIGNvbW1lbnQ6IHRpbWVzdGFtcDoxNzc5NTU4ODk5CWZpbGU6UkFQUE9SVCBQUkUtUkVMRUFTRS5hcHAudGFyLmd6Cjdvc2VDeW0rbEQvbnJDeDJzbTRIS1VQT1l0VU5pRmNNT1g0Wkc4NXB3UENGVjFPUFdRM0gveE1jcENKZjFMUTRCQ2tMMHNTUDBORkNiYVpOODZZdUJRPT0K",
+      "url": "https://git.kgva.ch/karim/RAPPORT/releases/download/0.8.2/RAPPORT%20PRE-RELEASE.app.tar.gz"
     }
   }
 }

package.json

@@ -1,7 +1,7 @@
 {
   "name": "rapport",
   "private": true,
-  "version": "0.8.0",
+  "version": "0.8.2",
   "type": "module",
   "scripts": {
     "dev": "vite",

src-tauri/Cargo.lock

@@ -2880,7 +2880,7 @@ dependencies = [
 
 [[package]]
 name = "rapport"
-version = "0.7.0"
+version = "0.8.1"
 dependencies = [
  "log",
  "serde",

src-tauri/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "rapport"
-version = "0.8.0"
+version = "0.8.2"
 description = "Rapport — Studio-Management für Architekturbüros"
 authors = ["Karim Gabriele Varano <karim@gabrielevarano.ch>"]
 license = "AGPL-3.0-or-later"

src-tauri/tauri.conf.json

@@ -1,7 +1,7 @@
 {
   "$schema": "../node_modules/@tauri-apps/cli/config.schema.json",
   "productName": "RAPPORT PRE-RELEASE",
-  "version": "0.8.0",
+  "version": "0.8.2",
   "identifier": "com.karimgabrielevarano.rapport",
   "build": {
     "frontendDist": "../dist",

src/App.jsx

@@ -297,8 +297,8 @@ export default function App() {
   const [modal, setModal] = useState(null);
   const [printContent, setPrintContent] = useState(null);
   const [darkMode, setDarkMode] = useState(() => localStorage.getItem("rapport_dark") === "1");
-  const [showChangelog, setShowChangelog] = useState(() => localStorage.getItem("rapport_changelog_seen") !== "0.8");
-  const [changelogVersion, setChangelogVersion] = useState("0.8");
+  const [showChangelog, setShowChangelog] = useState(() => localStorage.getItem("rapport_changelog_seen") !== "0.8.2");
+  const [changelogVersion, setChangelogVersion] = useState("0.8.2");
   const [showAbout, setShowAbout] = useState(false);
   const [navOpen, setNavOpen] = useState(false);
   const [expandedNav, setExpandedNav] = useState(new Set(["buchhaltung"]));
@@ -456,19 +456,21 @@ export default function App() {
   // Erst-Screen einer frischen Installation: «Lokal oder Cloud?».
   // Sichtbar solange der User die Wahl noch nicht getroffen hat UND es keine
   // lokalen Daten gibt. Sobald er gewählt hat, übernimmt der jeweilige Wizard.
+  // UpdateNotifier wird in allen Pre-Login-Screens mitgerendert, damit ein
+  // hängender Setup-Wizard sich via Auto-Update selbst befreien kann.
   const hasChosenBackend = localStorage.getItem("rapport_backend_chosen") === "1";
   if (!hasChosenBackend && isNewInstall && !data.settings.setupCompleted && !currentUser) {
-    return <BackendChoice />;
+    return <><BackendChoice /><UpdateNotifier /></>;
   }
 
   // Setup- und Migrations-Screens sind LocalStorage-Spezifika. Im Cloud-Modus
   // erfolgt Erst-Einrichtung über den Init-Dialog im Login.
   if (!isCloudBackend && isNewInstall && !data.settings.setupCompleted) {
-    return <Setup onComplete={handleSetupComplete} />;
+    return <><Setup onComplete={handleSetupComplete} /><UpdateNotifier /></>;
   }
 
   if (!isCloudBackend && !localStorage.getItem("rapport_v0_5_migrated")) {
-    return <MigrationScreen data={data} onComplete={handleSetupComplete} />;
+    return <><MigrationScreen data={data} onComplete={handleSetupComplete} /><UpdateNotifier /></>;
   }
 
   // Passwort-Reset hat höchste Priorität — User kommt von Mail-Link
@@ -493,12 +495,21 @@ export default function App() {
 
   // Cloud + Instanz ist leer (0 Studios) → mehrseitiger Setup-Wizard.
   // Cloud + Studios vorhanden → klassischer Login.
+  // UpdateNotifier wird hier auch gerendert, damit der Auto-Update-Check auch
+  // ohne Login läuft (sonst kommt man bei einem fehlerhaften Setup-Screen nie
+  // an ein neueres Build, das den Bug fixt).
   if (!currentUser) {
     if (isCloudBackend && cloudStudios !== null && cloudStudios.length === 0) {
       const cloudUrl = localStorage.getItem("rapport_cloud_url") || "";
-      return <CloudSetup cloudInit={cloudInit} cloudUrl={cloudUrl} />;
+      return <>
+        <CloudSetup cloudInit={cloudInit} cloudUrl={cloudUrl} />
+        <UpdateNotifier />
+      </>;
     }
-    return <Login verifyLogin={verifyLogin} settings={data.settings} version="0.8" />;
+    return <>
+      <Login verifyLogin={verifyLogin} settings={data.settings} version="0.8.2" />
+      <UpdateNotifier />
+    </>;
   }
 
   if (printContent) {
@@ -771,8 +782,8 @@ export default function App() {
           <div style={{ display: "flex", justifyContent: "space-between", alignItems: "center" }}>
             <button onClick={() => setShowAbout(true)} style={{ background: "none", border: "none", padding: 0, color: "#555", fontSize: 10, letterSpacing: "0.08em", cursor: "pointer", fontFamily: "inherit", textAlign: "left" }}
               onMouseEnter={e => e.currentTarget.style.color = "#aaa"} onMouseLeave={e => e.currentTarget.style.color = "#555"}>ÜBER RAPPORT</button>
-            <button onClick={() => { setChangelogVersion("0.8"); setShowChangelog(true); }} style={{ background: "none", border: "none", padding: 0, color: "#aaa", fontSize: 10, letterSpacing: "0.08em", cursor: "pointer", fontFamily: "inherit" }}
-              onMouseEnter={e => e.currentTarget.style.color = "#f0ede8"} onMouseLeave={e => e.currentTarget.style.color = "#aaa"}>0.8</button>
+            <button onClick={() => { setChangelogVersion("0.8.2"); setShowChangelog(true); }} style={{ background: "none", border: "none", padding: 0, color: "#aaa", fontSize: 10, letterSpacing: "0.08em", cursor: "pointer", fontFamily: "inherit" }}
+              onMouseEnter={e => e.currentTarget.style.color = "#f0ede8"} onMouseLeave={e => e.currentTarget.style.color = "#aaa"}>0.8.2</button>
           </div>
         </div>}
 
@@ -837,6 +848,18 @@ export default function App() {
 
       {showChangelog && (() => {
         const CHANGELOGS = {
+          "0.8.2": {
+            items: [
+              ["Selbstheilung für hängende 0.8.0-Installationen", "Wer von 0.7 auf 0.8 geupdated hat und in den Cloud-Setup-Wizard geschoben wurde, kommt mit 0.8.2 automatisch zurück in seinen Lokal-Modus. Der Auto-Recovery-Code erkennt: Cloud-Modus gesetzt + lokale Daten vorhanden + keine Cloud-Anmeldung → Cloud-Konfiguration wird zurückgenommen, alle Daten bleiben erhalten."],
+              ["Auto-Update auch ohne Login", "Bisher prüfte Rapport erst nach dem Login auf Updates — wer in einem fehlerhaften Setup-Bildschirm hing, kam nicht an den Bugfix. Jetzt läuft der Update-Check auch im «Lokal oder Cloud»-Wizard, im Login-Screen und im Cloud-Setup."],
+              ["Tauri ohne fest eingebaute Server-IP", "Die Desktop-App enthält keine vorkonfigurierte Cloud-Adresse mehr. Wer Cloud nutzen will, gibt die Server-Adresse beim Login aktiv ein — kein automatisches Vorausfüllen mit irrelevanten IPs."],
+            ],
+          },
+          "0.8.1": {
+            items: [
+              ["Update-Fix", "Behebt einen Fehler beim Upgrade von 0.7 auf 0.8: Lokal-Installationen wurden ungewollt in den Cloud-Modus geschoben und der Cloud-Setup-Wizard angezeigt, obwohl bereits lokale Daten vorhanden waren. Die App prüft jetzt vor einem automatischen Modus-Wechsel, ob lokale Daten existieren — und in Tauri-Installationen wird der Modus nie implizit gesetzt."],
+            ],
+          },
           "0.8": {
             items: [
               ["Cloud-Variante", "Rapport kann jetzt nicht nur lokal, sondern auch auf einem eigenen Server (Supabase) laufen. Beim ersten Öffnen wählt man «Lokal» oder «Cloud»; beide Modi haben dieselben Funktionen, Cloud zusätzlich Multi-User und Live-Sync zwischen Geräten."],
@@ -914,7 +937,7 @@ export default function App() {
           },
         };
         const versions = Object.keys(CHANGELOGS);
-        const current = CHANGELOGS[changelogVersion] || CHANGELOGS["0.8"];
+        const current = CHANGELOGS[changelogVersion] || CHANGELOGS["0.8.2"];
         return (
           <div style={{ position: "fixed", inset: 0, background: "rgba(0,0,0,0.55)", zIndex: 200, display: "flex", alignItems: "center", justifyContent: "center", padding: 24 }}>
             <div style={{ background: "#fff", borderRadius: 10, width: "100%", maxWidth: 480, boxShadow: "0 8px 40px rgba(0,0,0,0.18)", overflow: "hidden" }}>
@@ -943,7 +966,7 @@ export default function App() {
                 ))}
               </div>
               <div style={{ padding: "12px 32px 24px" }}>
-                <button className="btn btn-primary" style={{ width: "100%", fontSize: 13 }} onClick={() => { setShowChangelog(false); localStorage.setItem("rapport_changelog_seen", "0.8"); }}>
+                <button className="btn btn-primary" style={{ width: "100%", fontSize: 13 }} onClick={() => { setShowChangelog(false); localStorage.setItem("rapport_changelog_seen", "0.8.2"); }}>
                   Schliessen
                 </button>
               </div>
@@ -958,7 +981,7 @@ export default function App() {
             <div style={{ background: "#1a1a18", padding: "28px 32px 24px" }}>
               <div style={{ fontSize: 10, letterSpacing: "0.18em", color: "#b07848", marginBottom: 8, fontWeight: 600 }}>ÜBER RAPPORT</div>
               <div style={{ fontFamily: "'Playfair Display', serif", fontSize: 28, color: "#f0ede8", fontWeight: 400, lineHeight: 1.1 }}>Rapport</div>
-              <div style={{ fontSize: 11, color: "#888", marginTop: 6, letterSpacing: "0.04em" }}>Alpha 0.8 · Studio-Management für Architekturbüros</div>
+              <div style={{ fontSize: 11, color: "#888", marginTop: 6, letterSpacing: "0.04em" }}>Alpha 0.8.2 · Studio-Management für Architekturbüros</div>
             </div>
             <div style={{ padding: "20px 32px 8px" }}>
               <div style={{ fontSize: 11, fontWeight: 600, color: "#888", letterSpacing: "0.1em", marginBottom: 12 }}>LIZENZ</div>

src/storage/adapter.js

@@ -46,15 +46,37 @@ export class LocalStorageAdapter {
 }
 
 function createAdapter() {
-  // Build-time-Default für Web-Deploy: wenn eine Production-Build VITE_SUPABASE_URL
-  // gesetzt hat (z.B. via .env.production) und der User noch nichts gewählt hat,
-  // wird Cloud automatisch der Modus. Damit landet ein Browser, der app.rapport.kgva.ch
-  // öffnet, direkt im Cloud-Login (bzw. Init-Dialog wenn Server leer).
-  // Dev (`npm run dev`) ist ausgenommen — dort sieht der User weiterhin den
-  // BackendChoice-Screen, weil .env.local oft auf localhost zeigt.
-  if (typeof localStorage !== "undefined" && import.meta.env.PROD) {
-    const envUrl = import.meta.env.VITE_SUPABASE_URL;
-    if (envUrl && !localStorage.getItem("rapport_backend_chosen")) {
+  const isTauri = typeof window !== "undefined" && !!window.__TAURI_INTERNALS__;
+  // Build-time-URL nur für Web-Deploy gültig. Tauri-Builds ignorieren den
+  // eingebrannten Wert — Desktop-User geben die Server-URL aktiv ein.
+  // Der Anon-Key bleibt aus dem Build, weil er pro Cloud-Instanz konstant ist
+  // (kein User-Geheimnis, sondern Public-Konfig).
+  const envUrl = isTauri ? null : import.meta.env.VITE_SUPABASE_URL;
+  const envKey = import.meta.env.VITE_SUPABASE_ANON_KEY;
+
+  if (typeof localStorage !== "undefined") {
+    // ── Auto-Recovery für 0.8.0-Upgrade-Bug ──────────────────────────────
+    // 0.8.0 hat Cloud-Modus ungewollt gesetzt bei Lokal-Usern (siehe 0.8.1
+    // CHANGELOG). 0.8.1 verhindert das nur prospektiv, behebt aber den
+    // bestehenden Cloud-State nicht. Hier räumen wir nachträglich auf:
+    //   Cloud-Modus gesetzt + lokale Daten vorhanden + keine Cloud-Session
+    //   → Cloud-State löschen, User landet im BackendChoice oder Lokal-Modus.
+    const RECOVERY_MARKER = "rapport_080_recovery";
+    const backend = localStorage.getItem("rapport_backend");
+    const hasLocalData = !!localStorage.getItem(STORAGE_KEY);
+    const hasCloudSession = !!localStorage.getItem("rapport_supabase_session");
+    if (backend === "cloud" && hasLocalData && !hasCloudSession && !localStorage.getItem(RECOVERY_MARKER)) {
+      console.warn("Auto-Recovery: Cloud-Modus gesetzt, aber Lokal-Daten vorhanden und kein Cloud-Login — räume Cloud-State auf.");
+      localStorage.removeItem("rapport_backend");
+      localStorage.removeItem("rapport_backend_chosen");
+      localStorage.removeItem("rapport_cloud_url");
+      localStorage.setItem(RECOVERY_MARKER, "1");
+    }
+
+    // ── Auto-Cloud-Default für Web-Deploy ────────────────────────────────
+    // Nur wenn keine lokalen Daten existieren und der Browser auf einer
+    // konfigurierten Web-Instanz landet (envUrl aus build).
+    if (import.meta.env.PROD && envUrl && !localStorage.getItem("rapport_backend_chosen") && !hasLocalData) {
       localStorage.setItem("rapport_backend_chosen", "1");
       localStorage.setItem("rapport_backend", "cloud");
       if (!localStorage.getItem("rapport_cloud_url")) {
@@ -66,14 +88,15 @@ function createAdapter() {
   const backend = (typeof localStorage !== "undefined"
     && localStorage.getItem("rapport_backend")) || "local";
   if (backend === "cloud") {
-    const url = import.meta.env.VITE_SUPABASE_URL;
-    const key = import.meta.env.VITE_SUPABASE_ANON_KEY;
-    if (!url || !key) {
-      console.warn("rapport_backend=cloud, aber VITE_SUPABASE_URL/ANON_KEY fehlen — Fallback auf LocalStorage.");
+    // URL kommt bevorzugt aus localStorage (vom User eingegeben). Tauri hat
+    // gar keine env-URL; Web-Build hat sie ggf. als Fallback.
+    const url = (typeof localStorage !== "undefined" && localStorage.getItem("rapport_cloud_url")) || envUrl;
+    if (!url || !envKey) {
+      console.warn("rapport_backend=cloud, aber URL oder ANON_KEY fehlen — Fallback auf LocalStorage.");
       return new LocalStorageAdapter();
     }
     console.info("Storage-Adapter: SupabaseAdapter aktiv (URL:", url + ")");
-    return new SupabaseAdapter(url, key);
+    return new SupabaseAdapter(url, envKey);
   }
   return new LocalStorageAdapter();
 }

src/views/BackendChoice.jsx

@@ -7,7 +7,10 @@ import React from "react";
 //   Lokal → bestehender Setup.jsx
 //   Cloud → Login mit Init-Modus oder Login-Modus (je nach Studio-Vorhandensein)
 
-const envCloudUrl = import.meta.env.VITE_SUPABASE_URL || "";
+// Tauri-User geben die Server-URL immer aktiv ein. Build-time-URL ist nur
+// für Web-Deploy gedacht (z.B. app.rapport.kgva.ch → 127.0.0.1:54321 lokal).
+const isTauri = typeof window !== "undefined" && !!window.__TAURI_INTERNALS__;
+const envCloudUrl = isTauri ? "" : (import.meta.env.VITE_SUPABASE_URL || "");
 
 export default function BackendChoice() {
   const pick = (backend, cloudUrl = "") => {

supabase/migrations/0002_storage.sql

@@ -11,44 +11,55 @@
 -- Buckets sind PRIVATE — Zugriff nur über signierte URLs (zeitlich begrenzt).
 -- ============================================================================
 
-insert into storage.buckets (id, name, public)
+-- Hinweis: KEINE `public`-Spalte angeben. Beim Postgres-Init existiert sie in
+-- storage.buckets noch nicht (die fügt die Storage-API erst beim Boot per
+-- eigener Migration hinzu). Default ist `false` → Buckets sind privat, wie
+-- gewünscht. Würden wir `public` referenzieren, bräche der Init hier ab und
+-- ALLE folgenden Migrations (inkl. ensure_profile in 0005) liefen nicht mehr.
+insert into storage.buckets (id, name)
 values
-  ('receipts', 'receipts', false),
-  ('logos',    'logos',    false)
+  ('receipts', 'receipts'),
+  ('logos',    'logos')
 on conflict (id) do nothing;
 
 -- ────────────────────────────────────────────────────────────────────────────
 -- RLS-Policies auf storage.objects
 -- ────────────────────────────────────────────────────────────────────────────
 -- Prinzip: erste Pfad-Komponente ist studio_id; Zugriff nur wenn Member.
--- `(storage.foldername(name))[1]` gibt die erste Pfad-Komponente zurück.
+-- `split_part(name, '/', 1)` gibt die erste Pfad-Komponente zurück.
+--
+-- Bewusst NICHT storage.foldername() benutzen: die Storage-API droppt/erstellt
+-- diese Funktion bei ihren eigenen Boot-Migrations neu. Eine Policy-Abhängigkeit
+-- darauf würde diesen Drop blockieren ("cannot drop function foldername") und
+-- die Storage-API in eine Crash-Loop schicken. split_part ist ein eingebautes
+-- Postgres-Builtin ohne diese Kopplung.
 
 create policy "rapport_storage_read"
   on storage.objects for select
   using (
     bucket_id in ('receipts','logos')
-    and is_studio_member( (storage.foldername(name))[1]::uuid )
+    and is_studio_member( split_part(name, '/', 1)::uuid )
   );
 
 create policy "rapport_storage_insert"
   on storage.objects for insert
   with check (
     bucket_id in ('receipts','logos')
-    and is_studio_member( (storage.foldername(name))[1]::uuid )
+    and is_studio_member( split_part(name, '/', 1)::uuid )
   );
 
 create policy "rapport_storage_update"
   on storage.objects for update
   using (
     bucket_id in ('receipts','logos')
-    and is_studio_member( (storage.foldername(name))[1]::uuid )
+    and is_studio_member( split_part(name, '/', 1)::uuid )
   );
 
 create policy "rapport_storage_delete"
   on storage.objects for delete
   using (
     bucket_id in ('receipts','logos')
-    and is_studio_member( (storage.foldername(name))[1]::uuid )
+    and is_studio_member( split_part(name, '/', 1)::uuid )
   );
 
 -- ────────────────────────────────────────────────────────────────────────────