mixdog 0.7.18 → 0.8.1

package/scripts/bump.mjs

@@ -1,80 +0,0 @@
-#!/usr/bin/env node
-// bump.mjs — single source of truth for the mixdog release version.
-//
-// Bumps the THREE version surfaces in lockstep so the npm and marketplace
-// channels can never drift again:
-//   1. package.json                       version  (npm channel)
-//   2. .claude-plugin/plugin.json         version  (Claude Code plugin / marketplace update trigger)
-//   3. .claude-plugin/marketplace.json    plugin source `ref` -> `v<new>` (pins marketplace to the tagged release)
-//
-// Run by the gated release.yml pipeline, or locally:
-//   node scripts/bump.mjs <patch|minor|major|X.Y.Z>
-//
-// Invariant-based (no heuristic fallback): any inconsistent on-disk state
-// throws and aborts the release rather than guessing.
-
-import { readFileSync, writeFileSync } from 'node:fs';
-import { fileURLToPath } from 'node:url';
-import { dirname, join } from 'node:path';
-
-const ROOT = join(dirname(fileURLToPath(import.meta.url)), '..');
-const PKG = join(ROOT, 'package.json');
-const PLUGIN = join(ROOT, '.claude-plugin', 'plugin.json');
-const MARKET = join(ROOT, '.claude-plugin', 'marketplace.json');
-
-const SEMVER = /^(\d+)\.(\d+)\.(\d+)$/;
-
-function readJson(p) {
-  return JSON.parse(readFileSync(p, 'utf8'));
-}
-function writeJson(p, obj) {
-  writeFileSync(p, JSON.stringify(obj, null, 2) + '\n');
-}
-
-function nextVersion(current, arg) {
-  const m = SEMVER.exec(current);
-  if (!m) throw new Error(`package.json version is not X.Y.Z: "${current}"`);
-  const [major, minor, patch] = m.slice(1).map(Number);
-  if (arg === 'major') return `${major + 1}.0.0`;
-  if (arg === 'minor') return `${major}.${minor + 1}.0`;
-  if (arg === 'patch') return `${major}.${minor}.${patch + 1}`;
-  if (SEMVER.test(arg)) return arg;
-  throw new Error(`bump arg must be patch|minor|major|X.Y.Z, got: "${arg}"`);
-}
-
-const arg = process.argv[2];
-if (!arg) throw new Error('usage: node scripts/bump.mjs <patch|minor|major|X.Y.Z>');
-
-const pkg = readJson(PKG);
-const plugin = readJson(PLUGIN);
-const market = readJson(MARKET);
-
-if (typeof pkg.version !== 'string') throw new Error('package.json has no version string');
-if (typeof plugin.version !== 'string') throw new Error('plugin.json has no version string');
-
-const current = pkg.version;
-const next = nextVersion(current, arg);
-const tag = `v${next}`;
-
-const entry = (market.plugins || []).find((p) => p.name === plugin.name);
-if (!entry) throw new Error(`marketplace.json has no plugin entry named "${plugin.name}"`);
-if (!entry.source || typeof entry.source !== 'object') {
-  throw new Error('marketplace plugin entry is missing a source object');
-}
-
-// All three surfaces move together or not at all.
-pkg.version = next;
-plugin.version = next;
-entry.source.ref = tag;
-
-writeJson(PKG, pkg);
-writeJson(PLUGIN, plugin);
-writeJson(MARKET, market);
-
-console.log(`bump: ${current} -> ${next} (tag ${tag})`);
-console.log('  updated package.json, .claude-plugin/plugin.json, marketplace source ref');
-
-// Expose results to GitHub Actions when running in CI.
-if (process.env.GITHUB_OUTPUT) {
-  writeFileSync(process.env.GITHUB_OUTPUT, `version=${next}\ntag=${tag}\n`, { flag: 'a' });
-}

package/scripts/check-json.mjs

@@ -1,45 +0,0 @@
-#!/usr/bin/env node
-import { readFileSync, readdirSync } from 'fs';
-import { join, relative } from 'path';
-
-const ROOT = process.cwd();
-const SKIP_DIRS = new Set([
-  '.git',
-  '.mixdog',
-  'build',
-  'coverage',
-  'dist',
-  'node_modules',
-  'refs',
-  'target',
-]);
-
-function walk(dir, out = []) {
-  for (const entry of readdirSync(dir, { withFileTypes: true })) {
-    if (entry.isDirectory()) {
-      if (SKIP_DIRS.has(entry.name)) continue;
-      walk(join(dir, entry.name), out);
-      continue;
-    }
-    if (entry.isFile() && entry.name.endsWith('.json')) out.push(join(dir, entry.name));
-  }
-  return out;
-}
-
-const files = walk(ROOT).sort();
-let failed = 0;
-for (const file of files) {
-  try {
-    JSON.parse(readFileSync(file, 'utf8'));
-  } catch (e) {
-    failed++;
-    process.stderr.write(`FAIL ${relative(ROOT, file)}: ${e.message}\n`);
-  }
-}
-
-if (failed > 0) {
-  process.stderr.write(`JSON check failed: ${failed}/${files.length}\n`);
-  process.exit(1);
-}
-
-process.stdout.write(`JSON check passed: ${files.length} files\n`);

package/scripts/check-syntax-changed.mjs

@@ -1,102 +0,0 @@
-#!/usr/bin/env node
-import { spawnSync } from 'child_process';
-import { existsSync, statSync } from 'fs';
-import { join, relative, resolve } from 'path';
-import { fileURLToPath } from 'url';
-
-const ROOT = process.cwd();
-const EXTENSIONS = new Set(['.mjs', '.cjs']);
-
-function extname(file) {
-  const idx = file.lastIndexOf('.');
-  return idx >= 0 ? file.slice(idx) : '';
-}
-
-function runGit(args) {
-  const res = spawnSync('git', args, {
-    cwd: ROOT,
-    encoding: 'utf8',
-    stdio: ['ignore', 'pipe', 'pipe'],
-  });
-  return res;
-}
-
-function isGitRepo() {
-  const res = runGit(['rev-parse', '--is-inside-work-tree']);
-  return res.status === 0 && res.stdout.trim() === 'true';
-}
-
-function collectChanged() {
-  const out = new Set();
-  const diff = runGit(['diff', '--name-only', 'HEAD']);
-  if (diff.status === 0) {
-    for (const line of diff.stdout.split(/\r?\n/)) {
-      const p = line.trim();
-      if (p) out.add(p);
-    }
-  }
-  const untracked = runGit(['ls-files', '--others', '--exclude-standard']);
-  if (untracked.status === 0) {
-    for (const line of untracked.stdout.split(/\r?\n/)) {
-      const p = line.trim();
-      if (p) out.add(p);
-    }
-  }
-  return [...out];
-}
-
-function fallbackFull() {
-  const here = fileURLToPath(import.meta.url);
-  const full = resolve(here, '..', 'check-syntax.mjs');
-  const res = spawnSync(process.execPath, [full], {
-    cwd: ROOT,
-    stdio: 'inherit',
-  });
-  process.exit(res.status ?? 1);
-}
-
-if (!isGitRepo()) {
-  process.stdout.write('check:changed: not a git repo, running full check:syntax\n');
-  fallbackFull();
-}
-
-const candidates = collectChanged()
-  .filter((p) => EXTENSIONS.has(extname(p)))
-  .map((p) => join(ROOT, p))
-  .filter((abs) => {
-    if (!existsSync(abs)) return false;
-    try {
-      return statSync(abs).isFile();
-    } catch {
-      return false;
-    }
-  });
-
-const files = [...new Set(candidates)].sort();
-
-if (files.length === 0) {
-  process.stdout.write('check:changed: no changed .mjs/.cjs files, running full check:syntax\n');
-  fallbackFull();
-}
-
-let failed = 0;
-for (const file of files) {
-  const rel = relative(ROOT, file);
-  const res = spawnSync('node', ['--check', file], {
-    encoding: 'utf8',
-    stdio: ['ignore', 'pipe', 'pipe'],
-  });
-  if (res.status !== 0) {
-    failed++;
-    process.stderr.write(`FAIL ${rel}\n`);
-    if (res.stderr) process.stderr.write(res.stderr);
-    if (res.stdout) process.stderr.write(res.stdout);
-  }
-}
-
-if (failed > 0) {
-  process.stderr.write(`JavaScript syntax check (changed) failed: ${failed}/${files.length}\n`);
-  process.exit(1);
-}
-
-process.stdout.write(`JavaScript syntax check (changed) passed: ${files.length} files\n`);

package/scripts/check-syntax.mjs

@@ -1,58 +0,0 @@
-#!/usr/bin/env node
-import { readdirSync } from 'fs';
-import { join, relative } from 'path';
-import { spawnSync } from 'child_process';
-
-const ROOT = process.cwd();
-const SKIP_DIRS = new Set([
-  '.git',
-  '.mixdog',
-  'build',
-  'coverage',
-  'dist',
-  'node_modules',
-  'refs',
-  'target',
-]);
-const EXTENSIONS = new Set(['.js', '.cjs', '.mjs']);
-
-function extname(file) {
-  const idx = file.lastIndexOf('.');
-  return idx >= 0 ? file.slice(idx) : '';
-}
-
-function walk(dir, out = []) {
-  for (const entry of readdirSync(dir, { withFileTypes: true })) {
-    if (entry.isDirectory()) {
-      if (SKIP_DIRS.has(entry.name)) continue;
-      walk(join(dir, entry.name), out);
-      continue;
-    }
-    if (!entry.isFile()) continue;
-    if (EXTENSIONS.has(extname(entry.name))) out.push(join(dir, entry.name));
-  }
-  return out;
-}
-
-const files = walk(ROOT).sort();
-let failed = 0;
-for (const file of files) {
-  const rel = relative(ROOT, file);
-  const res = spawnSync('node', ['--check', file], {
-    encoding: 'utf8',
-    stdio: ['ignore', 'pipe', 'pipe'],
-  });
-  if (res.status !== 0) {
-    failed++;
-    process.stderr.write(`FAIL ${rel}\n`);
-    if (res.stderr) process.stderr.write(res.stderr);
-    if (res.stdout) process.stderr.write(res.stdout);
-  }
-}
-
-if (failed > 0) {
-  process.stderr.write(`JavaScript syntax check failed: ${failed}/${files.length}\n`);
-  process.exit(1);
-}
-
-process.stdout.write(`JavaScript syntax check passed: ${files.length} files\n`);

package/scripts/code-graph-batch.test.mjs

@@ -1,33 +0,0 @@
-// Regression: code_graph runs batchable symbol modes (find_symbol/callers/
-// callees/references) once per name when several are requested, so N lookups
-// cost ONE call. A single name (or single-element symbols[]) stays byte-identical
-// to the legacy single-symbol output (no per-symbol header).
-// Uses this repo's own exported symbols. Run: node scripts/code-graph-batch.test.mjs
-import { executeCodeGraphTool } from '../src/agent/orchestrator/tools/code-graph.mjs';
-
-const CWD = process.cwd();
-const call = (args) => executeCodeGraphTool('code_graph', args, CWD, null, {});
-let pass = 0, fail = 0;
-const ok = (label, cond, detail = '') => {
-  console.log(`${cond ? 'PASS' : 'FAIL'}  ${label}${detail ? '  -> ' + detail : ''}`);
-  cond ? pass++ : fail++;
-};
-
-// 1. Batch via symbols[] — one call, one section per symbol.
-const r1 = await call({ mode: 'find_symbol', symbols: ['peekShellJob', 'killShellJob'] });
-ok('batch find_symbol via symbols[]', /# find_symbol peekShellJob/.test(r1) && /# find_symbol killShellJob/.test(r1));
-
-// 2. Batch via space-separated symbol field (most portable across MCP schema caches).
-const r2 = await call({ mode: 'callers', symbol: 'peekShellJob killShellJob' });
-ok('batch callers via space-separated symbol', /# callers peekShellJob/.test(r2) && /# callers killShellJob/.test(r2));
-
-// 3. Single symbol — unchanged, NO batch header.
-const r3 = await call({ mode: 'find_symbol', symbol: 'peekShellJob' });
-ok('single find_symbol unchanged (no header)', !/# find_symbol /.test(r3) && r3.length > 0);
-
-// 4. Single-element symbols[] resolves (no "provide symbol" error, no header).
-const r4 = await call({ mode: 'find_symbol', symbols: ['peekShellJob'] });
-ok('single-element symbols[] resolves', !/# find_symbol /.test(r4) && r4.length > 0 && !/provide .symbol./.test(r4));
-
-console.log(`\n${fail === 0 ? 'ALL OK' : 'FAILURES'}: ${pass} passed, ${fail} failed`);
-process.exit(fail === 0 ? 0 : 1);

package/scripts/config-preserve-smoke.mjs

@@ -1,180 +0,0 @@
-import { mkdtempSync, rmSync, mkdirSync, writeFileSync, readFileSync } from 'fs';
-import { join } from 'path';
-import { tmpdir } from 'os';
-
-// This smoke exercises config-merge preservation, not the user-data backup
-// path, so skip the backup entirely — otherwise updateSection writes real
-// backups under ~/.claude/backups/mixdog-user-data on every run. The seed
-// guard also drops `.initialized-*.json` markers into the backup ROOT (not
-// gated by the skip flag), so redirect that root into the temp dir too.
-process.env.MIXDOG_SKIP_USER_DATA_BACKUP = '1';
-
-const tmp = mkdtempSync(join(tmpdir(), 'mixdog-config-preserve-'));
-process.env.CLAUDE_PLUGIN_DATA = tmp;
-process.env.MIXDOG_USER_DATA_BACKUP_ROOT = join(tmp, 'backups');
-mkdirSync(tmp, { recursive: true });
-
-const { updateSection, readSection } = await import('../src/shared/config.mjs');
-const {
-  mergeConfig,
-  mergeAgentConfig,
-  mergeMemoryConfig,
-  mergeSearchConfig,
-  mergeEndpointConfig,
-  mergeWebhookEndpointConfig,
-} = await import('../setup/config-merge.mjs');
-
-let failed = 0;
-function assert(label, cond) {
-  const pass = !!cond;
-  process.stdout.write(`${label}: ${pass ? 'PASS' : 'FAIL'}\n`);
-  if (!pass) failed++;
-}
-
-// (a) ngrokDomain survives partial webhook save
-updateSection('channels', () => ({
-  backend: 'discord',
-  webhook: { enabled: true, ngrokDomain: 'x.ngrok-free.dev' },
-}));
-updateSection('channels', (current) => mergeConfig(current, {
-  webhook: { enabled: true, respectQuiet: false },
-}));
-const chA = readSection('channels');
-assert('webhook.ngrokDomain preserved', chA?.webhook?.ngrokDomain === 'x.ngrok-free.dev');
-assert('webhook.respectQuiet updated', chA?.webhook?.respectQuiet === false);
-
-// (d) a plaintext authtoken in the on-disk webhook is NEVER re-persisted
-// (keychain-only invariant) while ngrokDomain still survives the merge.
-updateSection('channels', () => ({
-  backend: 'discord',
-  webhook: { enabled: true, ngrokDomain: 'y.ngrok-free.dev', authtoken: 'PLAINTEXT-MUST-DROP' },
-}));
-updateSection('channels', (current) => mergeConfig(current, {
-  webhook: { enabled: true, respectQuiet: true },
-}));
-const chD = readSection('channels');
-assert('webhook plaintext authtoken stripped', chD?.webhook?.authtoken === undefined);
-assert('webhook.ngrokDomain survives authtoken strip', chD?.webhook?.ngrokDomain === 'y.ngrok-free.dev');
-
-// (b) schedules.respectQuiet sidecar survives partial schedules save
-updateSection('channels', (current) => ({
-  ...current,
-  schedules: { respectQuiet: false, customSidecar: 'keep-me' },
-}));
-updateSection('channels', (current) => mergeConfig(current, {
-  schedules: { respectQuiet: true },
-}));
-const chB = readSection('channels');
-assert('schedules.customSidecar preserved', chB?.schedules?.customSidecar === 'keep-me');
-assert('schedules.respectQuiet updated', chB?.schedules?.respectQuiet === true);
-
-// (c) agent unmanaged top-level field survives partial bridge save
-updateSection('agent', () => ({
-  customSidecar: 'agent-keep',
-  bridge: { maxWorkers: 2 },
-}));
-updateSection('agent', (current) => mergeAgentConfig(current, { bridge: { timeoutMs: 90000 } }));
-const agent = readSection('agent');
-assert('agent.customSidecar preserved', agent?.customSidecar === 'agent-keep');
-assert('agent.bridge merged', agent?.bridge?.maxWorkers === 2 && agent?.bridge?.timeoutMs === 90000);
-
-// (e) schedule endpoint config.json — REAL disk round-trip: skill sidecars
-// (timezone/mode) survive a partial UI edit-save.
-{
-  const dir = join(tmp, 'schedules', 'sched1');
-  mkdirSync(dir, { recursive: true });
-  const cfgPath = join(dir, 'config.json');
-  writeFileSync(cfgPath, JSON.stringify({ time: '09:00', days: 'daily', timezone: 'America/Chicago', mode: 'cron' }, null, 2));
-  const incoming = { time: '10:00', days: 'weekday', model: 'fast', enabled: true };
-  const merged = mergeEndpointConfig(JSON.parse(readFileSync(cfgPath, 'utf8')), incoming);
-  writeFileSync(cfgPath, JSON.stringify(merged, null, 2));
-  const after = JSON.parse(readFileSync(cfgPath, 'utf8'));
-  assert('schedule.timezone preserved (disk)', after.timezone === 'America/Chicago');
-  assert('schedule.mode preserved (disk)', after.mode === 'cron');
-  assert('schedule.time updated (disk)', after.time === '10:00');
-}
-
-// (f) webhook endpoint config.json — REAL disk round-trip: omitted secret +
-// parser survive a partial edit-save, and role stays pinned to webhook-handler.
-{
-  const dir = join(tmp, 'webhooks', 'wh1');
-  mkdirSync(dir, { recursive: true });
-  const cfgPath = join(dir, 'config.json');
-  writeFileSync(cfgPath, JSON.stringify({ secret: 'hmac-key', parser: 'github', channel: 'main', role: 'webhook-handler' }, null, 2));
-  const incoming = { channel: 'alerts', model: 'default' };
-  const merged = mergeWebhookEndpointConfig(JSON.parse(readFileSync(cfgPath, 'utf8')), incoming);
-  writeFileSync(cfgPath, JSON.stringify(merged, null, 2));
-  const after = JSON.parse(readFileSync(cfgPath, 'utf8'));
-  assert('webhook.secret preserved (disk)', after.secret === 'hmac-key');
-  assert('webhook.parser preserved (disk)', after.parser === 'github');
-  assert('webhook.channel updated (disk)', after.channel === 'alerts');
-  assert('webhook.role pinned (disk)', after.role === 'webhook-handler');
-}
-
-// (g) mergeKept hardening: a stray array/string on EITHER side never spreads
-// into numeric keys.
-{
-  const m1 = mergeEndpointConfig({ keep: 1 }, ['x']);
-  assert('mergeKept ignores array incoming', m1.keep === 1 && m1['0'] === undefined);
-  const m2 = mergeEndpointConfig(['y'], { keep: 2 });
-  assert('mergeKept ignores array existing', m2.keep === 2 && m2['0'] === undefined);
-}
-
-// (h) memory — partial save preserves providers / user / cycle1 omitted keys
-// (POST /memory/config → updateSection + mergeMemoryConfig).
-updateSection('memory', () => ({
-  providers: { openai: { model: 'gpt-4o', temperature: 0.2 } },
-  user: { name: 'Alice', title: 'Engineer' },
-  cycle1: { interval: 60, timeout: 120000, batchSize: 8 },
-}));
-updateSection('memory', (current) => mergeMemoryConfig(current, {
-  cycle1: { interval: 90 },
-}));
-const mem = readSection('memory');
-assert('memory.providers preserved', mem?.providers?.openai?.model === 'gpt-4o' && mem?.providers?.openai?.temperature === 0.2);
-assert('memory.user preserved', mem?.user?.name === 'Alice' && mem?.user?.title === 'Engineer');
-assert('memory.cycle1 omitted keys preserved', mem?.cycle1?.timeout === 120000 && mem?.cycle1?.batchSize === 8);
-assert('memory.cycle1.interval updated', mem?.cycle1?.interval === 90);
-
-// (i) search — partial save preserves provider / models / crawl omitted keys
-// (POST /search/config → updateSection + mergeSearchConfig).
-updateSection('search', () => ({
-  provider: 'tavily',
-  models: { anthropic: 'claude-haiku', openai: 'gpt-4o-mini' },
-  crawl: { enabled: true, maxDepth: 2, respectRobots: true },
-  requestTimeoutMs: 30000,
-  rawSearch: { maxResults: 8, credentials: {} },
-}));
-updateSection('search', (current) => mergeSearchConfig(current, {
-  requestTimeoutMs: 45000,
-}));
-const sr = readSection('search');
-assert('search.provider preserved', sr?.provider === 'tavily');
-assert('search.models preserved', sr?.models?.anthropic === 'claude-haiku' && sr?.models?.openai === 'gpt-4o-mini');
-assert('search.crawl preserved', sr?.crawl?.enabled === true && sr?.crawl?.maxDepth === 2 && sr?.crawl?.respectRobots === true);
-assert('search.requestTimeoutMs updated', sr?.requestTimeoutMs === 45000);
-
-// (j) channels.channelsConfig — intentional FULL REPLACE: deleting a channel in
-// the UI must drop its disk row (no stale entry); a partial save that omits
-// channelsConfig entirely must preserve the existing map (length>0 guard).
-updateSection('channels', () => ({
-  backend: 'discord',
-  channelsConfig: {
-    'chan-a': { name: 'alpha', enabled: true },
-    'chan-b': { name: 'beta', enabled: true },
-  },
-}));
-updateSection('channels', (current) => mergeConfig(current, {
-  channelsConfig: { 'chan-a': { name: 'alpha', enabled: false } },
-}));
-const chJ1 = readSection('channels');
-assert('channelsConfig deleted channel removed', chJ1?.channelsConfig?.['chan-b'] === undefined);
-assert('channelsConfig kept channel updated', chJ1?.channelsConfig?.['chan-a']?.enabled === false);
-updateSection('channels', (current) => mergeConfig(current, {
-  webhook: { enabled: true },
-}));
-const chJ2 = readSection('channels');
-assert('channelsConfig survives unrelated partial save', chJ2?.channelsConfig?.['chan-a']?.name === 'alpha');
-
-try { rmSync(tmp, { recursive: true, force: true }); } catch {}
-process.exit(failed ? 1 : 0);