From e76b05499323cb9cda23810254b7087b914a3088 Mon Sep 17 00:00:00 2001
From: Terranom674 <thomas_dannenberg@web.de>
Date: Mon, 17 Aug 2026 20:07:35 +0200
Subject: [PATCH] Handle legacy source views while building manifests

---
 runtime/lib/build_manifest.py | 50 +++++++++++++++++++++++++++++------
 1 file changed, 42 insertions(+), 8 deletions(-)

diff --git a/runtime/lib/build_manifest.py b/runtime/lib/build_manifest.py
index 98584f7..08c20bf 100644
--- a/runtime/lib/build_manifest.py
+++ b/runtime/lib/build_manifest.py
@@ -28,18 +28,43 @@ def read_config(path: Path) -> dict[str, tuple[str, Path]]:
     return result
 
 
-def query_rows(args: argparse.Namespace) -> list[list[str]]:
-    password = args.password_file.read_text(encoding="utf-8").strip()
-    env = os.environ.copy()
-    env["PGPASSWORD"] = password
-    sql = f"SELECT share_id, item_type, display_name, storage_id, source_path FROM {args.view} ORDER BY share_id"
+def run_query(args: argparse.Namespace, env: dict[str, str], sql: str) -> subprocess.CompletedProcess[str]:
     command = [
         "psql", "-X", "-A", "-F", "\t", "-t",
         "-h", args.host, "-p", str(args.port), "-U", args.user, "-d", args.database,
         "-c", sql,
     ]
-    completed = subprocess.run(command, env=env, check=True, text=True, capture_output=True)
-    return list(csv.reader(completed.stdout.splitlines(), delimiter="\t"))
+    return subprocess.run(command, env=env, check=False, text=True, capture_output=True)
+
+
+def query_rows(args: argparse.Namespace) -> list[list[str]]:
+    password = args.password_file.read_text(encoding="utf-8").strip()
+    env = os.environ.copy()
+    env["PGPASSWORD"] = password
+
+    modern_sql = f"SELECT share_id, item_type, display_name, storage_id, source_path FROM {args.view} ORDER BY share_id"
+    completed = run_query(args, env, modern_sql)
+    if completed.returncode == 0:
+        return list(csv.reader(completed.stdout.splitlines(), delimiter="\t"))
+
+    # Existing installations may still expose the original four-column view.
+    # Keep them usable and derive folder/file from the resolved source path.
+    if "item_type" not in completed.stderr or "does not exist" not in completed.stderr:
+        raise RuntimeError(completed.stderr.strip() or "Nextcloud source view query failed")
+
+    legacy_sql = f"SELECT share_id, display_name, storage_id, source_path FROM {args.view} ORDER BY share_id"
+    completed = run_query(args, env, legacy_sql)
+    if completed.returncode != 0:
+        raise RuntimeError(completed.stderr.strip() or "legacy Nextcloud source view query failed")
+
+    rows: list[list[str]] = []
+    for row in csv.reader(completed.stdout.splitlines(), delimiter="\t"):
+        if len(row) == 4:
+            share_id, display_name, storage_id, source_path = row
+            rows.append([share_id, "", display_name, storage_id, source_path])
+        else:
+            rows.append(row)
+    return rows
 
 
 def contained_join(root: Path, relative: str) -> Path:
@@ -67,7 +92,7 @@ def build(args: argparse.Namespace) -> dict[str, object]:
 
         share_id, item_type, display_name, storage_id, source_path = row
         item_type = item_type.strip().lower()
-        if item_type not in {"folder", "file"}:
+        if item_type and item_type not in {"folder", "file"}:
             errors.append(f"share {share_id}: unsupported item_type {item_type!r}")
             continue
 
@@ -90,6 +115,15 @@ def build(args: argparse.Namespace) -> dict[str, object]:
             errors.append(f"share {share_id}: storage mount unavailable: {mount}")
             continue
 
+        if not item_type:
+            if source.is_dir():
+                item_type = "folder"
+            elif source.is_file():
+                item_type = "file"
+            else:
+                errors.append(f"share {share_id}: source unavailable: {source}")
+                continue
+
         if item_type == "folder":
             if not source.is_dir():
                 errors.append(f"share {share_id}: source directory unavailable: {source}")