mixdog 0.7.8 → 0.7.12

package/src/shared/open-url.mjs

@@ -0,0 +1,37 @@
+import { spawn } from 'child_process';
+
+/**
+ * Open a URL in the user's default browser. Best-effort and non-blocking —
+ * the caller always prints the URL too, so a failure here is never fatal.
+ *
+ * Platform dispatch (deterministic, not heuristic):
+ *  - Windows: `rundll32 url.dll,FileProtocolHandler <url>` passes the URL as a
+ *    single argv token, so query-string `&` separators are NOT re-parsed by a
+ *    shell. The old `start "<url>"` form is broken on Windows: cmd's `start`
+ *    treats the first quoted string as the WINDOW TITLE, so the URL is dropped
+ *    and nothing opens.
+ *  - macOS: `open <url>`.
+ *  - Linux/other: `xdg-open <url>`.
+ */
+export function openInBrowser(url) {
+  const u = String(url);
+  let cmd;
+  let args;
+  if (process.platform === 'win32') {
+    cmd = 'rundll32';
+    args = ['url.dll,FileProtocolHandler', u];
+  } else if (process.platform === 'darwin') {
+    cmd = 'open';
+    args = [u];
+  } else {
+    cmd = 'xdg-open';
+    args = [u];
+  }
+  try {
+    const child = spawn(cmd, args, { stdio: 'ignore', detached: true, windowsHide: true });
+    child.on('error', () => { /* opener missing — user uses the printed URL */ });
+    child.unref();
+  } catch {
+    /* spawn threw synchronously — user opens the printed URL manually */
+  }
+}

package/src/shared/seed.mjs

@@ -3,7 +3,7 @@ import { dirname, join } from 'path';
 import { fileURLToPath } from 'url';
 import { DEFAULT_PRESETS, DEFAULT_MAINTENANCE } from '../agent/orchestrator/config.mjs';
 import { writeFileAtomicSync, withFileLockSync } from './atomic-file.mjs';
-import { backupUserData, markUserDataInitialized, shouldSeedMissingUserData } from './user-data-guard.mjs';
+import { backupUserData, hasUserDataInitMarker, markUserDataInitialized, shouldSeedMissingUserData } from './user-data-guard.mjs';
 import { disableClaudeBuiltinsOnFirstInstall } from './disable-claude-builtins.mjs';
 
 const DEFAULTS_DIR = join(dirname(fileURLToPath(import.meta.url)), '..', '..', 'defaults');
@@ -49,12 +49,27 @@ const SEEDS = {
         };
         return JSON.stringify(composed, null, 2) + '\n';
     },
+    // Role→preset mapping consumed by Smart Bridge (loadResolvedRoles); without
+    // it on disk bridge roles fall back to the default preset. Baseline Lead
+    // workflow description ships alongside. Seeded HERE (not in setup-server)
+    // so this is the single first-install SSOT and the init marker is set once,
+    // after the whole default set lands.
+    'user-workflow.json': () => readFileSync(join(DEFAULTS_DIR, 'user-workflow.json'), 'utf8'),
+    'user-workflow.md': () => readFileSync(join(DEFAULTS_DIR, 'user-workflow.md'), 'utf8'),
 };
 
 export function ensureDataSeeds(dataDir) {
     if (!dataDir) return { created: [], skipped: [] };
     const created = [];
     const skipped = [];
+    // Capture fresh-install state ONCE, before the loop. The per-file
+    // markUserDataInitialized() below sets the marker as soon as the first seed
+    // lands; if we re-consulted the guard per file, every SUBSEQUENT first-time
+    // seed in this same pass would be refused (treated as a post-init deletion),
+    // which is exactly how user-workflow.json could end up permanently missing.
+    // On a fresh dir we seed the whole default set; once initialized, the guard
+    // governs (never recreate a file the user deleted on purpose).
+    const freshInstall = !hasUserDataInitMarker(dataDir);
     for (const [rel, bodyFn] of Object.entries(SEEDS)) {
         const full = join(dataDir, rel);
         if (existsSync(full)) {
@@ -62,7 +77,7 @@ export function ensureDataSeeds(dataDir) {
             skipped.push(rel);
             continue;
         }
-        if (!shouldSeedMissingUserData(dataDir, rel)) {
+        if (!freshInstall && !shouldSeedMissingUserData(dataDir, rel)) {
             skipped.push(rel);
             continue;
         }
@@ -112,7 +127,9 @@ export function ensureDataSeeds(dataDir) {
         }
     }
     if (created.length > 0) {
-        process.stderr.write(`[seed] created ${created.length} file(s): ${created.join(', ')}\n`);
+        if (process.env.MIXDOG_SETUP_QUIET !== '1') {
+            process.stderr.write(`[seed] created ${created.length} file(s): ${created.join(', ')}\n`);
+        }
         try { backupUserData(dataDir, 'post-seed'); } catch {}
     }
     return { created, skipped };

package/src/shared/user-data-guard.mjs

@@ -153,7 +153,9 @@ export function backupUserData(dataDir, reason = 'snapshot') {
   if (copied.length > 0) {
     markUserDataInitialized(dataDir);
     pruneBackups();
-    process.stderr.write(`[user-data-backup] ${reason}: copied ${copied.length} file(s) to ${backupDir}\n`);
+    if (process.env.MIXDOG_SETUP_QUIET !== '1') {
+      process.stderr.write(`[user-data-backup] ${reason}: copied ${copied.length} file(s) to ${backupDir}\n`);
+    }
   }
   return { dir: copied.length > 0 ? backupDir : null, copied };
 }

package/scripts/test-config-rmw-restore.mjs

@@ -1,122 +0,0 @@
-/**
- * Repro: malformed mixdog-config.json + writeSection('search', …) must not
- * wipe channels/memory/agent; restores from backup or throws.
- */
-import assert from 'node:assert/strict';
-import {
-  mkdtempSync,
-  mkdirSync,
-  writeFileSync,
-  readFileSync,
-  rmSync,
-} from 'fs';
-import { tmpdir } from 'os';
-import { join, dirname } from 'path';
-import { fileURLToPath } from 'url';
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-
-async function loadConfigModule(dataDir, backupRoot) {
-  process.env.CLAUDE_PLUGIN_DATA = dataDir;
-  process.env.MIXDOG_USER_DATA_BACKUP_ROOT = backupRoot;
-  process.env.MIXDOG_SKIP_USER_DATA_BACKUP = '1';
-  const url = new URL(`../src/shared/config.mjs?run=${Date.now()}`, import.meta.url).href;
-  return import(url);
-}
-
-function writeConfig(dataDir, obj) {
-  writeFileSync(
-    join(dataDir, 'mixdog-config.json'),
-    JSON.stringify(obj, null, 2) + '\n',
-    'utf8',
-  );
-}
-
-async function main() {
-  const dataDir = mkdtempSync(join(tmpdir(), 'mixdog-config-rmw-'));
-  const backupRoot = mkdtempSync(join(tmpdir(), 'mixdog-config-backup-'));
-
-  const prior = {
-    channels: { guild: '111' },
-    memory: { enabled: true },
-    agent: { presets: { default: { model: 'x' } } },
-  };
-  writeConfig(dataDir, prior);
-
-  process.env.CLAUDE_PLUGIN_DATA = dataDir;
-  process.env.MIXDOG_USER_DATA_BACKUP_ROOT = backupRoot;
-  const guardUrl = new URL(`../src/shared/user-data-guard.mjs?t=${Date.now()}`, import.meta.url).href;
-  const { backupUserData, markUserDataInitialized } = await import(guardUrl);
-  const snap = backupUserData(dataDir, 'test-fixture');
-  assert.ok(snap.dir, 'backup fixture should copy mixdog-config.json');
-
-  writeFileSync(join(dataDir, 'mixdog-config.json'), '{ not valid json\n', 'utf8');
-
-  const { writeSection } = await loadConfigModule(dataDir, backupRoot);
-  writeSection('search', { provider: 'brave' });
-
-  const onDisk = JSON.parse(readFileSync(join(dataDir, 'mixdog-config.json'), 'utf8'));
-  assert.deepEqual(onDisk.channels, prior.channels);
-  assert.deepEqual(onDisk.memory, prior.memory);
-  assert.deepEqual(onDisk.agent, prior.agent);
-  assert.deepEqual(onDisk.search, { provider: 'brave' });
-
-  const freshDir = mkdtempSync(join(tmpdir(), 'mixdog-config-fresh-'));
-  const { writeSection: writeFresh } = await loadConfigModule(freshDir, backupRoot);
-  writeFresh('search', { only: true });
-  const freshDisk = JSON.parse(readFileSync(join(freshDir, 'mixdog-config.json'), 'utf8'));
-  assert.deepEqual(freshDisk, { search: { only: true } });
-
-  const noBackupDir = mkdtempSync(join(tmpdir(), 'mixdog-config-noback-'));
-  markUserDataInitialized(noBackupDir);
-  writeFileSync(join(noBackupDir, 'mixdog-config.json'), '[]', 'utf8');
-  const { writeSection: writeNoBackup } = await loadConfigModule(
-    noBackupDir,
-    mkdtempSync(join(tmpdir(), 'empty-backup-')),
-  );
-  let threw = false;
-  try {
-    writeNoBackup('search', { x: 1 });
-  } catch (err) {
-    threw = true;
-    assert.match(String(err.message), /refusing section write/);
-  }
-  assert.equal(threw, true, 'malformed config with init marker and no backup must throw');
-
-  const pickRoot = mkdtempSync(join(tmpdir(), 'mixdog-config-pick-'));
-  const fullCfg = {
-    channels: { guild: '222' },
-    memory: { enabled: false },
-    agent: { presets: {} },
-  };
-  const oldDir = join(pickRoot, '2026-06-03T19-00-00-000Z-old-full');
-  const newDir = join(pickRoot, '2026-06-03T21-00-00-000Z-new-degenerate');
-  mkdirSync(oldDir, { recursive: true });
-  mkdirSync(newDir, { recursive: true });
-  writeFileSync(join(oldDir, 'mixdog-config.json'), JSON.stringify(fullCfg) + '\n', 'utf8');
-  writeFileSync(
-    join(newDir, 'mixdog-config.json'),
-    JSON.stringify({ search: { provider: 'tavily' } }) + '\n',
-    'utf8',
-  );
-  process.env.MIXDOG_USER_DATA_BACKUP_ROOT = pickRoot;
-  const pickUrl = new URL(`../src/shared/user-data-guard.mjs?pick=${Date.now()}`, import.meta.url).href;
-  const { loadLatestMixdogConfigFromBackup } = await import(pickUrl);
-  const picked = loadLatestMixdogConfigFromBackup(null);
-  assert.deepEqual(picked?.channels, fullCfg.channels);
-  assert.deepEqual(picked?.agent, fullCfg.agent);
-  assert.equal(picked?.search, undefined, 'must not restore newest search-only snapshot');
-  rmSync(pickRoot, { recursive: true, force: true });
-
-  rmSync(dataDir, { recursive: true, force: true });
-  rmSync(backupRoot, { recursive: true, force: true });
-  rmSync(freshDir, { recursive: true, force: true });
-  rmSync(noBackupDir, { recursive: true, force: true });
-
-  console.log('test-config-rmw-restore: ok');
-}
-
-main().catch((err) => {
-  console.error(err);
-  process.exit(1);
-});