@cybermem/dashboard 0.13.16 → 0.13.17

package/app/api/backup/route.ts

@@ -1,12 +1,13 @@
-import { execSync } from 'child_process'
-import { createReadStream, rmSync, statSync } from 'fs'
-import { NextResponse } from 'next/server'
-import { tmpdir } from 'os'
-import { join } from 'path'
+import { resolveDataDir } from "@/lib/resolve-data-dir";
+import { execSync } from "child_process";
+import { createReadStream, rmSync, statSync } from "fs";
+import { NextResponse } from "next/server";
+import { tmpdir } from "os";
+import { join } from "path";
 
-export const dynamic = 'force-dynamic'
+export const dynamic = "force-dynamic";
 
-const DATA_DIR = process.env.DATA_DIR || '/data'
+const DATA_DIR = resolveDataDir();
 
 /**
  * GET /api/backup
@@ -14,52 +15,55 @@ const DATA_DIR = process.env.DATA_DIR || '/data'
  */
 export async function GET() {
   try {
-    const timestamp = new Date().toISOString().replace(/[:.]/g, '-').slice(0, 19)
-    const backupName = `cybermem-backup-${timestamp}.tar.gz`
-    const tmpPath = join(tmpdir(), backupName)
+    const timestamp = new Date()
+      .toISOString()
+      .replace(/[:.]/g, "-")
+      .slice(0, 19);
+    const backupName = `cybermem-backup-${timestamp}.tar.gz`;
+    const tmpPath = join(tmpdir(), backupName);
 
     try {
       // Create backup via tar command (available in Node alpine image)
-      execSync(`tar -czf "${tmpPath}" -C "${DATA_DIR}" .`, { stdio: 'pipe' })
+      execSync(`tar -czf "${tmpPath}" -C "${DATA_DIR}" .`, { stdio: "pipe" });
 
       // Get file stats
-      const stats = statSync(tmpPath)
+      const stats = statSync(tmpPath);
 
       // Stream the file
-      const stream = createReadStream(tmpPath)
+      const stream = createReadStream(tmpPath);
 
       // Convert Node stream to Web ReadableStream
       const webStream = new ReadableStream({
         start(controller) {
-          stream.on('data', (chunk) => controller.enqueue(chunk))
-          stream.on('end', () => {
-            controller.close()
+          stream.on("data", (chunk) => controller.enqueue(chunk));
+          stream.on("end", () => {
+            controller.close();
             // Clean up temp file after streaming
-            try { rmSync(tmpPath) } catch {}
-          })
-          stream.on('error', (err) => controller.error(err))
-        }
-      })
+            try {
+              rmSync(tmpPath);
+            } catch {}
+          });
+          stream.on("error", (err) => controller.error(err));
+        },
+      });
 
       return new NextResponse(webStream, {
         headers: {
-          'Content-Type': 'application/gzip',
-          'Content-Disposition': `attachment; filename="${backupName}"`,
-          'Content-Length': String(stats.size)
-        }
-      })
-
+          "Content-Type": "application/gzip",
+          "Content-Disposition": `attachment; filename="${backupName}"`,
+          "Content-Length": String(stats.size),
+        },
+      });
     } catch (tarError: any) {
       return NextResponse.json(
         { error: `Backup failed: ${tarError.message}` },
-        { status: 500 }
-      )
+        { status: 500 },
+      );
     }
-
   } catch (error: any) {
     return NextResponse.json(
-      { error: error.message || 'Unknown error' },
-      { status: 500 }
-    )
+      { error: error.message || "Unknown error" },
+      { status: 500 },
+    );
   }
 }

package/app/api/reset/route.ts

@@ -1,11 +1,11 @@
+import { resolveDataDir } from "@/lib/resolve-data-dir";
 import { readdirSync, statSync, unlinkSync } from "fs";
 import { NextRequest, NextResponse } from "next/server";
-import { homedir } from "os";
-import { join, resolve } from "path";
+import { join } from "path";
 
 export const dynamic = "force-dynamic";
 
-const DATA_DIR = process.env.DATA_DIR || resolve(homedir(), ".cybermem/data");
+const DATA_DIR = resolveDataDir();
 
 /**
  * POST /api/reset

package/app/api/restore/route.ts

@@ -1,12 +1,13 @@
-import { execSync } from 'child_process'
-import { readdirSync, rmSync, unlinkSync, writeFileSync } from 'fs'
-import { NextRequest, NextResponse } from 'next/server'
-import { tmpdir } from 'os'
-import { join } from 'path'
+import { resolveDataDir } from "@/lib/resolve-data-dir";
+import { execSync } from "child_process";
+import { readdirSync, rmSync, unlinkSync, writeFileSync } from "fs";
+import { NextRequest, NextResponse } from "next/server";
+import { tmpdir } from "os";
+import { join } from "path";
 
-export const dynamic = 'force-dynamic'
+export const dynamic = "force-dynamic";
 
-const DATA_DIR = process.env.DATA_DIR || '/data'
+const DATA_DIR = resolveDataDir();
 
 /**
  * POST /api/restore
@@ -16,66 +17,69 @@ const DATA_DIR = process.env.DATA_DIR || '/data'
  */
 export async function POST(request: NextRequest) {
   try {
-    const formData = await request.formData()
-    const file = formData.get('backup') as File | null
+    const formData = await request.formData();
+    const file = formData.get("backup") as File | null;
 
     if (!file) {
       return NextResponse.json(
         { error: 'No backup file provided. Upload as "backup" field.' },
-        { status: 400 }
-      )
+        { status: 400 },
+      );
     }
 
     // Validate file type
-    if (!file.name.endsWith('.tar.gz') && !file.name.endsWith('.tgz')) {
+    if (!file.name.endsWith(".tar.gz") && !file.name.endsWith(".tgz")) {
       return NextResponse.json(
-        { error: 'Invalid file type. Must be .tar.gz or .tgz' },
-        { status: 400 }
-      )
+        { error: "Invalid file type. Must be .tar.gz or .tgz" },
+        { status: 400 },
+      );
     }
 
-    const tmpPath = join(tmpdir(), `restore-${Date.now()}.tar.gz`)
+    const tmpPath = join(tmpdir(), `restore-${Date.now()}.tar.gz`);
 
     try {
       // Write uploaded file to temp location
-      const buffer = Buffer.from(await file.arrayBuffer())
-      writeFileSync(tmpPath, buffer)
+      const buffer = Buffer.from(await file.arrayBuffer());
+      writeFileSync(tmpPath, buffer);
 
       // Remove existing database files
-      const existingFiles = readdirSync(DATA_DIR)
+      const existingFiles = readdirSync(DATA_DIR);
       for (const f of existingFiles) {
-        if (f.startsWith('openmemory.sqlite')) {
-          try { rmSync(join(DATA_DIR, f)) } catch {}
+        if (f.startsWith("openmemory.sqlite")) {
+          try {
+            rmSync(join(DATA_DIR, f));
+          } catch {}
         }
       }
 
       // Extract backup to data directory
-      execSync(`tar -xzf "${tmpPath}" -C "${DATA_DIR}"`, { stdio: 'pipe' })
+      execSync(`tar -xzf "${tmpPath}" -C "${DATA_DIR}"`, { stdio: "pipe" });
 
       // Clean up temp file
-      unlinkSync(tmpPath)
+      unlinkSync(tmpPath);
 
       return NextResponse.json({
         success: true,
-        message: 'Database restored successfully. Restart openmemory container to apply.',
+        message:
+          "Database restored successfully. Restart openmemory container to apply.",
         restartRequired: true,
-        restartCommand: 'docker restart cybermem-mcp'
-      })
-
+        restartCommand: "docker restart cybermem-mcp",
+      });
     } catch (restoreError: any) {
       // Clean up temp file on error
-      try { unlinkSync(tmpPath) } catch {}
+      try {
+        unlinkSync(tmpPath);
+      } catch {}
 
       return NextResponse.json(
         { error: `Restore failed: ${restoreError.message}` },
-        { status: 500 }
-      )
+        { status: 500 },
+      );
     }
-
   } catch (error: any) {
     return NextResponse.json(
-      { error: error.message || 'Unknown error' },
-      { status: 500 }
-    )
+      { error: error.message || "Unknown error" },
+      { status: 500 },
+    );
   }
 }

package/components/dashboard/settings-modal.tsx

@@ -191,7 +191,11 @@ export default function SettingsModal({ onClose }: { onClose: () => void }) {
         description: data.message || "Database wiped successfully.",
       });
     } catch (err: any) {
+      setShowResetConfirm(false);
       setOperationStatus({ type: "error", message: err.message });
+      toast.error("Reset Failed", {
+        description: err.message,
+      });
     } finally {
       setIsResetting(false);
     }

package/e2e/reset-db.spec.ts

@@ -276,4 +276,73 @@ test.describe("Reset DB - Settings Modal", () => {
 
     console.log("✅ Cancel closes confirmation, Settings modal persists");
   });
+
+  test("6. Reset API never returns ENOENT (DATA_DIR resolved)", async ({
+    request,
+  }) => {
+    // Call the reset API directly and verify we never get ENOENT
+    const res = await request.post("/api/reset", {
+      data: { confirm: "RESET" },
+      headers: { "Content-Type": "application/json" },
+    });
+
+    const body = await res.json();
+
+    // Must NOT contain ENOENT — this was the RPi bug
+    if (body.error) {
+      expect(body.error).not.toContain("ENOENT");
+      expect(body.error).not.toContain("no such file or directory");
+    }
+
+    // Should succeed (200) — file deletion works
+    expect(res.status()).toBe(200);
+    expect(body.success).toBe(true);
+    console.log(
+      `✅ Reset API succeeded without ENOENT (deleted ${body.deletedCount} files)`,
+    );
+  });
+
+  test("7. On API error, confirmation modal closes and error toast shown", async ({
+    page,
+  }) => {
+    await openSettings(page);
+
+    const resetBtn = page.getByRole("button", { name: /Reset DB/i });
+    await resetBtn.click();
+
+    await expect(
+      page.getByRole("heading", { name: "Reset Database" }),
+    ).toBeVisible();
+
+    // Intercept the reset API to simulate a server error
+    await page.route("**/api/reset", async (route) => {
+      await route.fulfill({
+        status: 500,
+        contentType: "application/json",
+        body: JSON.stringify({ error: "Simulated server error for E2E test" }),
+      });
+    });
+
+    const confirmBtn = page.getByRole("button", { name: /Reset Database/i });
+    const input = page.getByPlaceholder(/Type "RESET" to confirm/i);
+
+    await input.click();
+    await input.fill("RESET");
+    await expect(confirmBtn).toBeEnabled();
+    await confirmBtn.click();
+
+    // Confirmation modal MUST close even on error
+    await expect(page.getByPlaceholder(/Type "RESET" to confirm/i)).toBeHidden({
+      timeout: 10000,
+    });
+
+    // Error toast must appear
+    await expect(
+      page.locator('[data-sonner-toaster] [data-type="error"]'),
+    ).toBeVisible({ timeout: 5000 });
+
+    console.log(
+      "✅ On API error: modal closed, error toast shown (not hidden behind modal)",
+    );
+  });
 });

package/lib/resolve-data-dir.ts

@@ -0,0 +1,17 @@
+import { existsSync } from "fs";
+import { homedir } from "os";
+import { resolve } from "path";
+
+/**
+ * Resolves the CyberMem data directory depending on the runtime environment:
+ *
+ * 1. Explicit `DATA_DIR` env var (highest priority)
+ * 2. Docker volume mount at `/data` (container runtime)
+ * 3. `~/.cybermem/data` (local dev / Next.js standalone)
+ */
+export function resolveDataDir(): string {
+  const envDataDir = process.env.DATA_DIR?.trim();
+  if (envDataDir) return envDataDir;
+  if (existsSync("/data")) return "/data";
+  return resolve(homedir(), ".cybermem/data");
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cybermem/dashboard",
-  "version": "0.13.16",
+  "version": "0.13.17",
   "description": "CyberMem Monitoring Dashboard",
   "homepage": "https://cybermem.dev",
   "repository": {