From cf8cfb740cb3cf30e1077959b669b6ff0726da84 Mon Sep 17 00:00:00 2001
From: Stephen Shaw <sshaw@decriptor.com>
Date: Wed, 29 Apr 2026 20:38:10 -0600
Subject: [PATCH] configurator: colocate Synology backup default under
 /volume1/docker

Match the QNAP convention of keeping roon and roon-backups together
under the container app-data root. Previously backup defaulted to
/volume1/roon-backups while roon lived at /volume1/docker/roon.
---
 configurator/e2e/configurator.spec.ts       | 2 +-
 configurator/public/platforms/synology.json | 2 +-
 configurator/src/urlState.test.ts           | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/configurator/e2e/configurator.spec.ts b/configurator/e2e/configurator.spec.ts
index e1510d0..7078172 100644
--- a/configurator/e2e/configurator.spec.ts
+++ b/configurator/e2e/configurator.spec.ts
@@ -45,7 +45,7 @@ test.describe('Platform switching', () => {
     await page.locator('#platform-select').selectOption('synology');
     await expect(page.locator('#vol-roon')).toHaveValue('/volume1/docker/roon');
     await expect(page.locator('#vol-music')).toHaveValue('/volume1/music');
-    await expect(page.locator('#vol-backup')).toHaveValue('/volume1/roon-backups');
+    await expect(page.locator('#vol-backup')).toHaveValue('/volume1/docker/roon-backups');
     expect(await getOutput(page)).toContain('- /volume1/music:/Music');
   });
 });
diff --git a/configurator/public/platforms/synology.json b/configurator/public/platforms/synology.json
index f66cbf3..66fe714 100644
--- a/configurator/public/platforms/synology.json
+++ b/configurator/public/platforms/synology.json
@@ -3,7 +3,7 @@
   "label": "Synology",
   "roon": "/volume1/docker/roon",
   "music": "/volume1/music",
-  "backup": "/volume1/roon-backups",
+  "backup": "/volume1/docker/roon-backups",
   "prefix": "/volume1/",
   "hint": "Paths set for Synology DSM. Adjust volume1 if needed.",
   "rootPattern": "^/volume\\d+/"
diff --git a/configurator/src/urlState.test.ts b/configurator/src/urlState.test.ts
index 160318b..bc1b0fa 100644
--- a/configurator/src/urlState.test.ts
+++ b/configurator/src/urlState.test.ts
@@ -16,7 +16,7 @@ const platforms: PlatformMap = {
     id: 'synology', label: 'Synology',
     roon: '/volume1/docker/roon',
     music: '/volume1/music',
-    backup: '/volume1/roon-backups',
+    backup: '/volume1/docker/roon-backups',
     prefix: '/volume1/',
     hint: '',
     rootPattern: '^/volume\\d+/',
@@ -66,7 +66,7 @@ describe('encode', () => {
     const cfg = baseConfig({
       volRoon: '/volume1/docker/roon',
       volMusic: '/volume1/music',
-      volBackup: '/volume1/roon-backups',
+      volBackup: '/volume1/docker/roon-backups',
     });
     const params = encode(cfg, 'synology', platforms, 'qnap');
     expect(params.get('p')).toBe('synology');