fix(build): resolve an explicit relative backupFolder against root

The previous root fix only pre-resolved backupFolder's *default* value
against root; an explicitly-provided relative backupFolder (e.g.
plugin.ts's syncBackupsDir, which is always a defined string) stayed
relative. checkMeta()/saveBackup() then resolved it via a bare
path.resolve(this.backupFolder, ...), falling back to process.cwd().
Resolve it against root unconditionally in the constructor instead.

Author: sergak01

src/lib/dist.service.ts

@@ -132,7 +132,7 @@ export class DistService {
 
     const {
       root = process.cwd(),
-      backupFolder = path.resolve(root, 'backups'),
+      backupFolder = 'backups',
       distZipFolder = path.resolve(root, 'dist-zip'),
       distZipFilename = createDefaultZipFileName(this.pageName),
       backupNameTemplate = `{${TEMPLATE_PART_PAGE_NAME}}-{${TEMPLATE_PART_DATE}}.zip`,
@@ -143,7 +143,10 @@ export class DistService {
     } = syncOptions || {};
 
     this.root = root;
-    this.backupFolder = backupFolder;
+    // Resolve eagerly (not just at the default) — checkMeta()/saveBackup() use this value
+    // directly without prepending root, so an explicit relative backupFolder must be
+    // anchored here or it silently falls back to process.cwd() at those call sites.
+    this.backupFolder = path.resolve(root, backupFolder);
     this.backupNameTemplate = backupNameTemplate;
     this.dateFormat = dateFormat;
 

tests/unit/lib/dist.service.root.spec.ts

@@ -76,6 +76,22 @@ describe('DistService — root-scoped path resolution', () => {
     expect(readBack?.toString('utf-8')).toBe('{"a":1}');
   });
 
+  it('resolves an explicit relative backupFolder against the explicit root, not process.cwd()', async () => {
+    // eslint-disable-next-line no-new -- the constructor's fire-and-forget syncMeta() creates the folder
+    new DistService('test-app', {
+      root: projectRoot,
+      backupFolder: 'backups',
+    });
+
+    // Don't call checkMeta() ourselves: the constructor already kicked one off in the
+    // background (unawaited), and a second concurrent call races its mkdir. Poll instead.
+    await vi.waitFor(() => {
+      expect(fs.existsSync(path.join(projectRoot, 'backups'))).toBe(true);
+    });
+
+    expect(fs.existsSync(path.join(unrelatedCwd, 'backups'))).toBe(false);
+  });
+
   it('defaults root to process.cwd() when not provided', async () => {
     const service = new DistService('test-app', {});