groove-dev 0.27.40 → 0.27.42

package/node_modules/@groove-dev/gui/dist/index.html

@@ -6,7 +6,7 @@
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <link rel="icon" type="image/png" href="/favicon.png" />
     <title>Groove GUI</title>
-    <script type="module" crossorigin src="/assets/index-zzVaD3-G.js"></script>
+    <script type="module" crossorigin src="/assets/index-C1C2biHU.js"></script>
     <link rel="modulepreload" crossorigin href="/assets/vendor-C0HXlhrU.js">
     <link rel="modulepreload" crossorigin href="/assets/reactflow-BQPfi37R.js">
     <link rel="modulepreload" crossorigin href="/assets/codemirror-BBL3i_JW.js">

package/node_modules/@groove-dev/gui/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/gui",
-  "version": "0.27.40",
+  "version": "0.27.42",
   "description": "GROOVE GUI — visual agent control plane",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",

package/node_modules/@groove-dev/gui/src/components/ui/toast.jsx

@@ -36,7 +36,7 @@ const DURATIONS = {
 function ToastItem({ toast }) {
   const removeToast = useGrooveStore((s) => s.removeToast);
   const Icon = ICONS[toast.type] || Info;
-  const duration = DURATIONS[toast.type];
+  const duration = toast.persistent ? 0 : (toast.duration ?? DURATIONS[toast.type]);
 
   useEffect(() => {
     if (!duration) return;

package/node_modules/@groove-dev/gui/src/stores/groove.js

@@ -387,6 +387,7 @@ export const useGrooveStore = create((set, get) => ({
             'Project ready to preview',
             msg.url,
             { label: 'View Site', url: msg.url },
+            { persistent: true },
           );
           break;
 
@@ -660,11 +661,13 @@ export const useGrooveStore = create((set, get) => ({
 
   async deleteTeam(id) {
     const team = get().teams.find((t) => t.id === id);
-    if (team?.isDefault) { get().addToast('warning', 'Cannot delete the default team'); return; }
     try {
       await api.delete(`/teams/${encodeURIComponent(id)}`);
-      // WS team:deleted handler removes from array and switches activeTeamId
-      get().addToast('info', `Team "${team?.name}" deleted`);
+      // WS team:deleted handler removes from array and switches activeTeamId.
+      // Deleting the default team regenerates a fresh one server-side; the
+      // team:created event arrives separately so the list stays populated.
+      const wiped = team?.isDefault ? 'wiped' : 'deleted';
+      get().addToast('info', `Team "${team?.name}" ${wiped}`);
     } catch (err) {
       get().addToast('error', 'Failed to delete team', err.message);
     }
@@ -758,9 +761,11 @@ export const useGrooveStore = create((set, get) => ({
 
   // ── Toasts ────────────────────────────────────────────────
 
-  addToast(type, message, detail, action) {
+  addToast(type, message, detail, action, options = {}) {
     const id = ++toastCounter;
-    set((s) => ({ toasts: [...s.toasts, { id, type, message, detail, action }] }));
+    const persistent = !!options.persistent;
+    const duration = options.duration;
+    set((s) => ({ toasts: [...s.toasts, { id, type, message, detail, action, persistent, duration }] }));
   },
   removeToast(id) {
     set((s) => ({ toasts: s.toasts.filter((t) => t.id !== id) }));

package/node_modules/@groove-dev/gui/src/views/agents.jsx

@@ -239,14 +239,10 @@ export function TeamTabBar() {
               <ContextMenuItem onSelect={() => cloneTeam(team.id)}>
                 <Copy size={12} /> Clone
               </ContextMenuItem>
-              {!team.isDefault && (
-                <>
-                  <ContextMenuSeparator />
-                  <ContextMenuItem danger onSelect={() => deleteTeam(team.id)}>
-                    <Trash2 size={12} /> Delete
-                  </ContextMenuItem>
-                </>
-              )}
+              <ContextMenuSeparator />
+              <ContextMenuItem danger onSelect={() => deleteTeam(team.id)}>
+                <Trash2 size={12} /> {team.isDefault ? 'Wipe' : 'Delete'}
+              </ContextMenuItem>
             </ContextMenuContent>
           </ContextMenu>
         );

package/node_modules/@groove-dev/gui/src/views/settings.jsx

@@ -1080,6 +1080,19 @@ export default function SettingsView() {
                   </select>
                 </ConfigCard>
 
+                <ConfigCard icon={Cpu} label="Default Model" description="Model used for new agents. Auto routes by role.">
+                  <select
+                    value={config.defaultModel || ''}
+                    onChange={(e) => updateConfig('defaultModel', e.target.value || null)}
+                    className="w-full h-8 px-2.5 text-xs bg-surface-0 border border-border-subtle rounded-md text-text-0 font-mono focus:outline-none focus:ring-1 focus:ring-accent cursor-pointer"
+                  >
+                    <option value="">Auto (route by role)</option>
+                    {(providers.find((p) => p.id === (config.defaultProvider || 'claude-code'))?.models || []).map((m) => (
+                      <option key={m.id} value={m.id}>{m.name}</option>
+                    ))}
+                  </select>
+                </ConfigCard>
+
                 <ConfigCard icon={FolderOpen} label="Working Directory" description="Default root directory for new agents.">
                   <div className="flex items-center gap-1.5">
                     <code className="flex-1 h-8 px-2 flex items-center bg-surface-0 border border-border-subtle rounded-md text-2xs font-mono text-text-2 truncate min-w-0">

package/node_modules/@groove-dev/gui/vite.config.js

@@ -4,9 +4,6 @@ import tailwindcss from '@tailwindcss/vite';
 
 export default defineConfig({
   plugins: [tailwindcss(), react()],
-  define: {
-    __GROOVE_EDITION__: JSON.stringify(process.env.GROOVE_EDITION || 'community'),
-  },
   server: {
     port: 3142,
     proxy: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "groove-dev",
-  "version": "0.27.40",
+  "version": "0.27.42",
   "description": "Open-source agent orchestration layer — the AI company OS. Local model agent engine (GGUF/Ollama/llama-server), HuggingFace model browser, MCP integrations (Slack, Gmail, Stripe, 15+), agent scheduling (cron), business roles (CMO, CFO, EA). GUI dashboard, multi-agent coordination, zero cold-start, infinite sessions. Works with Claude Code, Codex, Gemini CLI, Ollama, any local model.",
   "license": "FSL-1.1-Apache-2.0",
   "author": "Groove Dev <hello@groovedev.ai> (https://groovedev.ai)",
@@ -53,10 +53,9 @@
     "dev:daemon": "npm run dev -w packages/daemon",
     "dev:gui": "npm run dev -w packages/gui",
     "build": "npm run build -w packages/gui",
-    "build:pro": "GROOVE_EDITION=pro npm run build -w packages/gui",
     "start:desktop": "npm run start -w packages/desktop",
     "build:desktop": "npm run build -w packages/desktop",
-    "dist:desktop": "npm run build:pro && npm run dist -w packages/desktop",
+    "dist:desktop": "npm run build && npm run dist -w packages/desktop",
     "test": "node --test packages/daemon/test/*.test.js",
     "prepublishOnly": "npm run build"
   },

package/packages/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/cli",
-  "version": "0.27.40",
+  "version": "0.27.42",
   "description": "GROOVE CLI — manage AI coding agents from your terminal",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",

package/packages/daemon/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/daemon",
-  "version": "0.27.40",
+  "version": "0.27.42",
   "description": "GROOVE daemon — agent orchestration engine",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",

package/packages/daemon/src/adaptive.js

@@ -3,6 +3,11 @@
 
 import { readFileSync, writeFileSync, existsSync } from 'fs';
 import { resolve } from 'path';
+import { minimatch } from 'minimatch';
+
+// Treat these scope entries as "unrestricted" — agent can touch any file
+// under its workingDir without counting as a scope violation.
+const UNRESTRICTED_SCOPE_PATTERNS = new Set(['**', '**/*', '*', '']);
 
 const DEFAULT_THRESHOLD = 0.75;
 const NUDGE_UP = 0.02;   // Good session → allow more context
@@ -186,14 +191,28 @@ export class AdaptiveThresholds {
           signals.toolFailures++;
         }
 
-        // Scope violations: writes outside declared scope
+        // Scope violations: writes outside declared scope. Use real glob matching
+        // (the naive substring check flagged every write when scope was `["**"]`
+        // because `file.includes("**")` is always false — which tanked the
+        // quality score and triggered false-positive rotations). An unrestricted
+        // scope (`**`, `**/*`, empty pattern) skips the check entirely.
         if (agentScope && agentScope.length > 0 && entry.input) {
           if (entry.tool === 'Write' || entry.tool === 'Edit') {
             const file = entry.input;
-            const inScope = agentScope.some((pattern) =>
-              file.includes(pattern.replace('/**', '').replace('**/', ''))
-            );
-            if (!inScope) signals.scopeViolations++;
+            const unrestricted = agentScope.some((p) => UNRESTRICTED_SCOPE_PATTERNS.has(String(p).trim()));
+            if (!unrestricted) {
+              const inScope = agentScope.some((pattern) => {
+                try {
+                  if (minimatch(file, pattern, { matchBase: true, dot: true })) return true;
+                  // Also try matching the basename and any path suffix, since
+                  // scope patterns are relative to the agent's workingDir and
+                  // the recorded input may be absolute.
+                  const idx = file.indexOf('/' + pattern.replace(/\/?\*\*\/?/g, '').replace(/^\//, ''));
+                  return idx >= 0;
+                } catch { return true; } // if the pattern is malformed, don't penalize
+              });
+              if (!inScope) signals.scopeViolations++;
+            }
           }
         }
       }

package/packages/daemon/src/api.js

@@ -15,13 +15,15 @@ import { ROLE_INTEGRATIONS } from './process.js';
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const pkgVersion = JSON.parse(readFileSync(new URL('../package.json', import.meta.url), 'utf8')).version;
-const isPro = process.env.GROOVE_EDITION === 'pro';
 
 let _daemon = null;
 
+// Single source of truth for Pro features: the signed-in user's subscription
+// status, populated by the daemon polling the backend with the stored JWT.
+// There is no build-time "Pro edition" flag — one binary, account-gated.
 function proOnly(req, res, next) {
   const sub = _daemon?.subscriptionCache || {};
-  if (isPro || sub.active) return next();
+  if (sub.active) return next();
   return res.status(403).json({
     error: 'Pro subscription required',
     edition: 'community',
@@ -129,6 +131,10 @@ export function createApi(app, daemon) {
         const team = daemon.teams.get(config.teamId);
         if (team?.workingDir) config.workingDir = team.workingDir;
       }
+      // Inherit configured default model if the request didn't pick one
+      if (!config.model && daemon.config?.defaultModel) {
+        config.model = daemon.config.defaultModel;
+      }
       const agent = await daemon.processes.spawn(config);
       daemon.audit.log('agent.spawn', { id: agent.id, role: agent.role, provider: agent.provider });
       res.status(201).json(agent);
@@ -295,7 +301,7 @@ export function createApi(app, daemon) {
     // verify the path matches the scope or belongs to no one.
     if (agent.scope && agent.scope.length > 0 && targets.length > 0) {
       for (const target of targets) {
-        const conflict = daemon.locks.check(agentId, target);
+        const conflict = daemon.locks.check(agentId, target, agent.workingDir);
         if (conflict.conflict) {
           daemon.audit.log('knock.denied', { agentId, toolName, target, owner: conflict.owner, pattern: conflict.pattern });
           daemon.broadcast({ type: 'knock:denied', agentId, agentName: agent.name, toolName, target, owner: conflict.owner, reason: 'scope_conflict' });
@@ -711,7 +717,7 @@ export function createApi(app, daemon) {
   app.get('/api/edition', (req, res) => {
     const sub = daemon.subscriptionCache || {};
     res.json({
-      edition: (isPro || sub.active) ? 'pro' : 'community',
+      edition: sub.active ? 'pro' : 'community',
       plan: sub.plan || 'community',
       subscriptionActive: sub.active || false,
       features: sub.features || [],
@@ -734,7 +740,7 @@ export function createApi(app, daemon) {
       host: daemon.host,
       port: daemon.port,
       projectDir: daemon.projectDir,
-      edition: (isPro || sub.active) ? 'pro' : 'community',
+      edition: sub.active ? 'pro' : 'community',
     });
   });
 
@@ -2650,11 +2656,15 @@ Keep responses concise. Help them think, don't lecture them about the system the
     if (!found) {
       return res.status(404).json({ error: 'No recommended team found. Run a planner first.' });
     }
+    const planPath = found.path;
+    const planContents = readFileSync(planPath, 'utf8');
     try {
-      const raw = JSON.parse(readFileSync(found.path, 'utf8'));
+      const raw = JSON.parse(planContents);
 
-      // Delete immediately after reading to prevent duplicate launches from poll races
-      try { unlinkSync(found.path); } catch { /* already gone */ }
+      // Delete immediately after reading to prevent duplicate launches from poll races.
+      // If every spawn below fails, we'll restore the plan from planContents so the
+      // user can retry without re-prompting the planner.
+      try { unlinkSync(planPath); } catch { /* already gone */ }
 
       // Support both old format (bare array) and new format ({ projectDir, agents, preview })
       let agentConfigs;
@@ -2834,6 +2844,14 @@ Keep responses concise. Help them think, don't lecture them about the system the
         daemon.preview.stashPlan(defaultTeamId, previewBlock, projectWorkingDir);
       }
 
+      // Restore the plan if nothing actually spawned or was reused — deleting
+      // it on a total failure leaves the team with no recovery path. A failed
+      // spawn (scope collision, provider unavailable, etc.) should be retryable
+      // once the user fixes the condition.
+      if (spawned.length === 0 && reused.length === 0 && failed.length > 0) {
+        try { writeFileSync(planPath, planContents); } catch { /* best-effort */ }
+      }
+
       daemon.audit.log('team.launch', {
         phase1: spawned.length, reused: reused.length, phase2Pending: phase2.length, failed: failed.length,
         agents: [...spawned, ...reused].map((a) => a.role), projectDir: projectDir || null, preview: !!previewBlock,

package/packages/daemon/src/lockmanager.js

@@ -18,7 +18,7 @@ const DEFAULT_OPERATION_TTL_MS = 10 * 60 * 1000; // 10 minutes
 export class LockManager {
   constructor(grooveDir) {
     this.path = resolve(grooveDir, 'locks.json');
-    this.locks = new Map(); // agentId -> glob patterns[]
+    this.locks = new Map(); // agentId -> { patterns, workingDir }
     this._compiledPatterns = new Map(); // agentId -> RegExp[]
     this.operations = new Map(); // agentId -> { name, resources, acquiredAt, expiresAt }
     this.load();
@@ -28,9 +28,11 @@ export class LockManager {
     if (existsSync(this.path)) {
       try {
         const data = JSON.parse(readFileSync(this.path, 'utf8'));
-        for (const [id, patterns] of Object.entries(data)) {
-          this.locks.set(id, patterns);
-          this._compilePatterns(id, patterns);
+        for (const [id, val] of Object.entries(data)) {
+          // Backward compat: old format stored just patterns array
+          const entry = Array.isArray(val) ? { patterns: val, workingDir: null } : val;
+          this.locks.set(id, entry);
+          this._compilePatterns(id, entry.patterns);
         }
       } catch {
         // Start fresh
@@ -51,8 +53,8 @@ export class LockManager {
     this._compiledPatterns.set(agentId, compiled);
   }
 
-  register(agentId, patterns) {
-    this.locks.set(agentId, patterns);
+  register(agentId, patterns, workingDir = null) {
+    this.locks.set(agentId, { patterns, workingDir: workingDir || null });
     this._compilePatterns(agentId, patterns);
     this.save();
   }
@@ -64,9 +66,13 @@ export class LockManager {
     this.save();
   }
 
-  check(agentId, filePath) {
+  // Scopes are per-team — only conflict with owners in the same workingDir.
+  // Pass workingDir=null to skip the filter (legacy behavior).
+  check(agentId, filePath, workingDir = null) {
     for (const [ownerId, compiled] of this._compiledPatterns) {
       if (ownerId === agentId) continue;
+      const ownerEntry = this.locks.get(ownerId);
+      if (workingDir && ownerEntry?.workingDir && ownerEntry.workingDir !== workingDir) continue;
       for (const { pattern, re } of compiled) {
         if (re && re.test(filePath)) {
           return { conflict: true, owner: ownerId, pattern };
@@ -111,11 +117,13 @@ export class LockManager {
   /**
    * Find any currently-locked agent whose scope overlaps with candidateScope.
    * Returns { overlap: true, owner, ... } for the first conflict, else {overlap:false}.
+   * Pass workingDir to limit the search to the same team folder (scopes are per-team).
    */
-  findOverlappingOwner(candidateScope) {
-    for (const [ownerId, patterns] of this.locks) {
-      const res = LockManager.scopesOverlap(candidateScope, patterns);
-      if (res.overlap) return { overlap: true, owner: ownerId, ownerScope: patterns, ...res };
+  findOverlappingOwner(candidateScope, workingDir = null) {
+    for (const [ownerId, entry] of this.locks) {
+      if (workingDir && entry.workingDir && entry.workingDir !== workingDir) continue;
+      const res = LockManager.scopesOverlap(candidateScope, entry.patterns);
+      if (res.overlap) return { overlap: true, owner: ownerId, ownerScope: entry.patterns, ...res };
     }
     return { overlap: false };
   }
@@ -140,7 +148,9 @@ export class LockManager {
   }
 
   getAll() {
-    return Object.fromEntries(this.locks);
+    const obj = {};
+    for (const [id, entry] of this.locks) obj[id] = entry.patterns;
+    return obj;
   }
 
   // --- Operation locks (coordination protocol) ---

package/packages/daemon/src/preview.js

@@ -76,28 +76,46 @@ export class PreviewService {
    * preview upfront at launch time and hand it back when the team completes.
    */
   async launch(teamId, workingDir, preview) {
+    this.daemon.audit?.log('preview.attempt', { teamId, workingDir, preview });
+
     if (!preview || !preview.kind || preview.kind === 'none' || preview.kind === 'cli') {
-      return { launched: false, reason: preview?.kind || 'no_preview' };
+      const result = { launched: false, reason: preview?.kind || 'no_preview' };
+      this.daemon.audit?.log('preview.skipped', { teamId, reason: result.reason });
+      return result;
     }
 
-    // Kill any existing preview for this team
     await this.kill(teamId);
 
-    const baseDir = preview.cwd
-      ? resolve(workingDir || this.daemon.projectDir, preview.cwd)
-      : resolve(workingDir || this.daemon.projectDir);
+    // Resolve cwd with a sensible fallback. The planner sometimes names the
+    // cwd after projectDir which is applied by api/launch → the actual project
+    // root. If that specific subdir doesn't exist, try workingDir itself.
+    const root = resolve(workingDir || this.daemon.projectDir);
+    const candidates = [];
+    if (preview.cwd) candidates.push(resolve(root, preview.cwd));
+    candidates.push(root);
+    const baseDir = candidates.find((p) => existsSync(p));
 
-    if (!existsSync(baseDir)) {
-      return { launched: false, reason: `cwd_missing: ${baseDir}` };
+    if (!baseDir) {
+      const result = { launched: false, reason: `cwd_missing: tried ${candidates.join(' and ')}` };
+      this.daemon.audit?.log('preview.failed', { teamId, reason: result.reason });
+      return result;
     }
 
+    let result;
     if (preview.kind === 'static-html') {
-      return this._launchStatic(teamId, baseDir, preview);
+      result = await this._launchStatic(teamId, baseDir, preview);
+    } else if (preview.kind === 'dev-server') {
+      result = await this._launchDevServer(teamId, baseDir, preview);
+    } else {
+      result = { launched: false, reason: `unknown_kind: ${preview.kind}` };
     }
-    if (preview.kind === 'dev-server') {
-      return this._launchDevServer(teamId, baseDir, preview);
+
+    if (result.launched) {
+      this.daemon.audit?.log('preview.launched', { teamId, url: result.url, kind: result.kind, baseDir });
+    } else {
+      this.daemon.audit?.log('preview.failed', { teamId, reason: result.reason, baseDir });
     }
-    return { launched: false, reason: `unknown_kind: ${preview.kind}` };
+    return result;
   }
 
   _launchStatic(teamId, baseDir, preview) {
@@ -238,6 +256,7 @@ export class PreviewService {
       if (entry.server) entry.server.close();
       if (entry.proc) entry.proc.kill('SIGTERM');
     } catch { /* best-effort */ }
+    this.daemon.audit?.log('preview.stopped', { teamId });
     this.daemon.broadcast({ type: 'preview:stopped', teamId });
     return true;
   }

package/packages/daemon/src/process.js

@@ -377,18 +377,21 @@ export class ProcessManager {
       }
     }
 
-    // Scope collision check: refuse to spawn if another running agent already
-    // claims overlapping files. Oversight roles (planner, QC, security) and
-    // the ambassador bypass since their job requires broad access.
+    // Scope collision check: refuse to spawn if another running agent in the
+    // SAME working directory already claims overlapping files. Scopes are
+    // relative patterns rooted at the agent's workingDir, so two agents in
+    // different workingDirs can't step on each other even if their patterns
+    // look identical. Oversight roles (planner, QC, security) and ambassadors
+    // bypass entirely since their job requires broad access.
     const SCOPE_BYPASS_ROLES = new Set(['planner', 'fullstack', 'qc', 'pm', 'supervisor', 'security', 'ambassador']);
     if (config.scope && config.scope.length > 0 && !SCOPE_BYPASS_ROLES.has(config.role) && !config.allowScopeOverlap) {
-      const conflict = locks.findOverlappingOwner(config.scope);
+      const conflict = locks.findOverlappingOwner(config.scope, config.workingDir);
       if (conflict.overlap) {
         const owner = registry.get(conflict.owner);
-        if (owner && owner.status === 'running') {
+        if (owner && owner.status === 'running' && owner.workingDir === config.workingDir) {
           const ownerScope = Array.isArray(conflict.ownerScope) ? conflict.ownerScope.join(', ') : '';
           throw new Error(
-            `Scope collision: ${config.role} scope [${config.scope.join(', ')}] overlaps with ${owner.name} (${owner.role}) which owns [${ownerScope}]. ` +
+            `Scope collision: ${config.role} scope [${config.scope.join(', ')}] overlaps with ${owner.name} (${owner.role}) which owns [${ownerScope}] in the same workspace. ` +
             `Two agents cannot edit the same files. Either narrow the scope or wait for ${owner.name} to finish.`
           );
         }
@@ -460,7 +463,7 @@ export class ProcessManager {
 
     // Register file locks for the agent's scope
     if (agent.scope && agent.scope.length > 0) {
-      locks.register(agent.id, agent.scope);
+      locks.register(agent.id, agent.scope, agent.workingDir);
     }
 
     // Register ambassador with federation system
@@ -1098,16 +1101,25 @@ For normal file edits within your scope, proceed without review.
    *   - The daemon has a preview plan stashed for this team (planner wrote one).
    *   - No pending phase 2 groups for this team (QC hasn't spawned yet).
    *   - Every non-planner team agent is in a terminal state.
-   *   - At least one non-planner agent completed successfully (something to preview).
-   * Clears the plan after launching so repeated completions don't re-fire.
+   *   - At least one non-planner agent completed successfully.
+   *
+   * The plan is intentionally NOT cleared on failure so the user can hit the
+   * "Launch Preview" button manually after fixing whatever blocked the auto
+   * attempt (wrong cwd, missing deps, etc). We guard against infinite re-fires
+   * with _previewAttempted per teamId.
    */
   _checkPreviewReady(teamId) {
     const preview = this.daemon.preview;
     if (!preview) return;
+    if (!this._previewAttempted) this._previewAttempted = new Set();
+    if (this._previewAttempted.has(teamId)) return;
+
     const plan = preview.getPlan(teamId);
-    if (!plan) return;
+    if (!plan) {
+      this.daemon.audit?.log('preview.skipped', { teamId, reason: 'no_plan_stashed' });
+      return;
+    }
 
-    // If a phase 2 group for this team is still pending, let it spawn first.
     const pendingPhase2 = this.daemon._pendingPhase2 || [];
     for (const group of pendingPhase2) {
       for (const id of group.waitFor) {
@@ -1123,7 +1135,7 @@ For normal file edits within your scope, proceed without review.
     const anyCompleted = teamAgents.some((a) => a.status === 'completed');
     if (!allDone || !anyCompleted) return;
 
-    preview.clearPlan(teamId);
+    this._previewAttempted.add(teamId);
     const workingDir = plan.workingDir;
     preview.launch(teamId, workingDir, plan.preview).then((result) => {
       if (!result.launched) {
@@ -1137,6 +1149,7 @@ For normal file edits within your scope, proceed without review.
       }
     }).catch((err) => {
       console.error(`[Groove] Preview launch error for team ${teamId}:`, err.message);
+      this.daemon.broadcast({ type: 'preview:failed', teamId, reason: err.message });
     });
   }
 
@@ -1510,7 +1523,7 @@ For normal file edits within your scope, proceed without review.
 
     // Re-register locks
     if (newAgent.scope && newAgent.scope.length > 0) {
-      locks.register(newAgent.id, newAgent.scope);
+      locks.register(newAgent.id, newAgent.scope, newAgent.workingDir);
     }
 
     // Spawn the resumed process

package/packages/daemon/src/teams.js

@@ -34,14 +34,30 @@ export class Teams {
   }
 
   _ensureDefault() {
-    const hasDefault = [...this.teams.values()].some((t) => t.isDefault);
-    if (!hasDefault) {
+    const defaultDir = resolve(this.daemon.projectDir, 'default');
+    const existing = [...this.teams.values()].find((t) => t.isDefault);
+
+    if (!existing) {
+      try { mkdirSync(defaultDir, { recursive: true }); } catch { /* may exist */ }
       const id = randomUUID().slice(0, 8);
-      const team = { id, name: 'Default', isDefault: true, createdAt: new Date().toISOString() };
-      // Default team uses the project directory (no subdirectory)
-      team.workingDir = this.daemon.projectDir;
+      const team = {
+        id,
+        name: 'Default',
+        isDefault: true,
+        workingDir: defaultDir,
+        createdAt: new Date().toISOString(),
+      };
       this.teams.set(id, team);
       this._save();
+      return;
+    }
+
+    // Migrate legacy default teams that pointed at the project root — give them
+    // their own folder so generated files don't pile up alongside source code.
+    if (!existing.workingDir || existing.workingDir === this.daemon.projectDir) {
+      try { mkdirSync(defaultDir, { recursive: true }); } catch { /* may exist */ }
+      existing.workingDir = defaultDir;
+      this._save();
     }
   }
 
@@ -125,12 +141,13 @@ export class Teams {
   }
 
   /**
-   * Delete a team — removes directory and all contents, moves agents to default.
+   * Delete a team — kills its agents, removes its directory, drops it from the
+   * registry. Deleting the default team regenerates a fresh empty one so users
+   * can wipe accumulated state and keep working without restarting the daemon.
    */
   delete(id) {
     const team = this.teams.get(id);
     if (!team) throw new Error('Team not found');
-    if (team.isDefault) throw new Error('Cannot delete the default team');
 
     // Kill any running agents in this team
     const agents = this.daemon.registry.getAll().filter((a) => a.teamId === id);
@@ -145,8 +162,13 @@ export class Teams {
       this.daemon.registry.remove(agent.id);
     }
 
-    // Remove the working directory
-    if (team.workingDir && !team.isDefault && existsSync(team.workingDir)) {
+    // Remove the team's working directory — refuse to nuke the project root
+    // (legacy default teams that were never migrated point there).
+    if (
+      team.workingDir &&
+      team.workingDir !== this.daemon.projectDir &&
+      existsSync(team.workingDir)
+    ) {
       try {
         rmSync(team.workingDir, { recursive: true, force: true });
       } catch (err) {
@@ -158,6 +180,13 @@ export class Teams {
     this._save();
     this.daemon.broadcast({ type: 'team:deleted', teamId: id });
 
+    // Always keep a default team available — regenerate one with a clean folder
+    if (team.isDefault) {
+      this._ensureDefault();
+      const fresh = this.getDefault();
+      if (fresh) this.daemon.broadcast({ type: 'team:created', team: fresh });
+    }
+
     // Clean up orphaned logs immediately — don't wait for the 24h GC cycle
     try { this.daemon._gc(); } catch { /* gc should never block deletion */ }
 

package/packages/daemon/src/tool-executor.js

@@ -158,7 +158,7 @@ export class ToolExecutor {
   _checkWriteScope(resolvedPath) {
     if (!this.daemon?.locks) return;
     const rel = relative(this.workingDir, resolvedPath);
-    const result = this.daemon.locks.check(this.agentId, rel);
+    const result = this.daemon.locks.check(this.agentId, rel, this.workingDir);
     if (result.conflict) {
       // Record conflict for supervisor + token savings
       if (this.daemon.supervisor) {