fix: ensure /app/data ownership on startup to prevent SQLITE_CANTOPEN (#42)

The entrypoint only ran chown when PUID/PGID differed from defaults,
but /app/data is a mounted volume with unpredictable ownership.
Now chown /app/data runs unconditionally on every startup.

Also added mkdir -p for the data directory and a clearer error message
when the database cannot be opened.

Author: johnpc

entrypoint.sh

@@ -7,7 +7,10 @@ PGID=${PGID:-1000}
 if [ "$(id -u node)" != "$PUID" ] || [ "$(id -g node)" != "$PGID" ]; then
   groupmod -o -g "$PGID" node
   usermod -o -u "$PUID" node
-  chown -R node:node /app /home/node
+  chown -R node:node /home/node
 fi
 
+# Always fix ownership on /app/data — it's a mounted volume with unpredictable ownership
+chown -R node:node /app/data
+
 exec gosu node "$@"

src/database.ts

@@ -1,4 +1,6 @@
 import Database from 'better-sqlite3';
+import * as fs from 'fs';
+import * as path from 'path';
 
 export interface Run {
   id: string;
@@ -42,7 +44,23 @@ export class SubsyncarrPlusDatabase {
   private db: Database.Database;
 
   constructor(dbPath: string) {
-    this.db = new Database(dbPath);
+    const dir = path.dirname(dbPath);
+    try {
+      fs.mkdirSync(dir, { recursive: true });
+    } catch {
+      // ignore — will fail on db open if truly inaccessible
+    }
+    try {
+      this.db = new Database(dbPath);
+    } catch (err: unknown) {
+      if (err instanceof Error && 'code' in err && err.code === 'SQLITE_CANTOPEN') {
+        throw new Error(
+          `Cannot open database at ${dbPath} — ensure the data directory exists and is writable. ` +
+            `If using Docker, check that your volume mount target exists and has correct permissions (PUID/PGID).`,
+        );
+      }
+      throw err;
+    }
     this.initSchema();
   }