groove-dev 0.27.156 → 0.27.159

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

@@ -4,7 +4,7 @@ import { useGrooveStore } from '../stores/groove';
 import { Button } from '../components/ui/button';
 import { ScrollArea } from '../components/ui/scroll-area';
 import { Dialog, DialogContent } from '../components/ui/dialog';
-import { BookOpen, Plus, Search, Trash2, Pencil, ChevronRight, Hash, FolderOpen, Clock, Save, Link2, FileText, Sparkles, HelpCircle, GripVertical } from 'lucide-react';
+import { BookOpen, Plus, Search, Trash2, Pencil, ChevronRight, Hash, FolderOpen, Clock, Save, Link2, FileText, Sparkles, HelpCircle, GripVertical, CornerLeftUp } from 'lucide-react';
 
 const COMMANDS = [
   { cmd: 'save',     args: '#tag',                desc: 'Save the message and send it to the agent' },
@@ -114,7 +114,7 @@ function MemoryCard({ item, onEdit, onDelete }) {
   );
 }
 
-function EditorModal({ open, onOpenChange, editing, onSave }) {
+function EditorModal({ open, onOpenChange, editing, onSave, onRename }) {
   const [tag, setTag] = useState('');
   const [content, setContent] = useState('');
   const textareaRef = useRef(null);
@@ -132,9 +132,15 @@ function EditorModal({ open, onOpenChange, editing, onSave }) {
     }
   }, [open]);
 
-  const handleSave = () => {
+  const handleSave = async () => {
     if (!tag.trim() || editing?.readOnly) return;
-    onSave(tag.trim(), content);
+    const originalTag = editing?.tag || '';
+    const newTag = tag.trim();
+    if (!editing?.isNew && newTag !== originalTag) {
+      await onRename(originalTag, newTag, content);
+    } else {
+      onSave(newTag, content);
+    }
     onOpenChange(false);
   };
 
@@ -148,27 +154,30 @@ function EditorModal({ open, onOpenChange, editing, onSave }) {
   const isNew = editing?.isNew;
   const readOnly = editing?.readOnly;
   const title = readOnly ? `#${editing?.tag || ''}` : isNew ? 'New Memory' : `Edit #${editing?.tag || ''}`;
+  const tagChanged = !isNew && tag.trim() !== (editing?.tag || '');
 
   return (
     <Dialog open={open} onOpenChange={onOpenChange}>
       <DialogContent title={title} description="Memory content" className="max-w-2xl">
         <div className="p-5 space-y-4" onKeyDown={handleKeyDown}>
-          {isNew && (
-            <div>
-              <label className="block text-xs font-medium text-text-2 mb-1.5">Tag</label>
-              <div className="flex items-center gap-1">
-                <span className="text-sm text-text-3">#</span>
-                <input
-                  type="text"
-                  value={tag}
-                  onChange={(e) => setTag(e.target.value.replace(/[^a-zA-Z0-9/_-]/g, '').toLowerCase())}
-                  placeholder="project/feature-name"
-                  className="flex-1 px-2 py-1.5 text-sm font-mono rounded-md bg-surface-0 border border-border text-text-0 placeholder:text-text-4 focus:outline-none focus:ring-1 focus:ring-accent"
-                />
-              </div>
-              <p className="text-2xs text-text-4 mt-1">Use / for hierarchy: groove/memory-system</p>
+          <div>
+            <label className="block text-xs font-medium text-text-2 mb-1.5">
+              {isNew ? 'Tag' : 'Title'}
+              {tagChanged && <span className="ml-2 text-2xs text-warning">(will rename)</span>}
+            </label>
+            <div className="flex items-center gap-1">
+              <span className="text-sm text-text-3">#</span>
+              <input
+                type="text"
+                value={tag}
+                onChange={(e) => !readOnly && setTag(e.target.value.replace(/[^a-zA-Z0-9/_-]/g, '').toLowerCase())}
+                readOnly={readOnly}
+                placeholder="project/feature-name"
+                className="flex-1 px-2 py-1.5 text-sm font-mono rounded-md bg-surface-0 border border-border text-text-0 placeholder:text-text-4 focus:outline-none focus:ring-1 focus:ring-accent"
+              />
             </div>
-          )}
+            {isNew && <p className="text-2xs text-text-4 mt-1">Use / for hierarchy: groove/memory-system</p>}
+          </div>
           <div>
             {!readOnly && <label className="block text-xs font-medium text-text-2 mb-1.5">Content</label>}
             <textarea
@@ -196,7 +205,7 @@ function EditorModal({ open, onOpenChange, editing, onSave }) {
               {!readOnly && (
                 <Button variant="primary" size="sm" onClick={handleSave} disabled={!tag.trim()}>
                   <Save size={14} />
-                  Save
+                  {tagChanged ? 'Rename & Save' : 'Save'}
                 </Button>
               )}
             </div>
@@ -249,7 +258,7 @@ function InstructModal({ open, onOpenChange }) {
   );
 }
 
-function TreeItem({ tag, label, isDoc, indent, isDragOver, onSelect, onDragStart, onDragOver, onDragLeave, onDrop }) {
+function TreeItem({ tag, label, isDoc, indent, isDragOver, onSelect, onEdit, onDelete, onDragStart, onDragOver, onDragLeave, onDrop }) {
   return (
     <div
       draggable
@@ -257,19 +266,34 @@ function TreeItem({ tag, label, isDoc, indent, isDragOver, onSelect, onDragStart
       onDragOver={(e) => { e.preventDefault(); e.dataTransfer.dropEffect = 'move'; onDragOver?.(tag); }}
       onDragLeave={() => onDragLeave?.()}
       onDrop={(e) => { e.preventDefault(); onDrop?.(e.dataTransfer.getData('text/plain'), tag); }}
-      onClick={() => onSelect({ tag })}
       className={`flex items-center gap-1.5 w-full px-2 py-1.5 rounded-md text-xs transition-colors cursor-pointer group ${isDragOver ? 'bg-accent/15 border border-accent/30 border-dashed' : 'hover:bg-surface-2'}`}
       style={indent ? { paddingLeft: `${8 + indent * 16}px` } : undefined}
     >
       <GripVertical size={10} className="text-text-4 opacity-0 group-hover:opacity-50 flex-shrink-0 cursor-grab" />
       <Hash size={11} className="text-text-4 flex-shrink-0" />
-      <span className="font-medium text-text-2 truncate">{label}</span>
+      <span className="font-medium text-text-2 truncate flex-1" onClick={() => onSelect({ tag })}>{label}</span>
       {isDoc && <Sparkles size={9} className="text-purple flex-shrink-0" />}
+      <div className="flex items-center gap-0.5 opacity-0 group-hover:opacity-100 transition-opacity flex-shrink-0">
+        <button
+          onClick={(e) => { e.stopPropagation(); onEdit?.({ tag }); }}
+          className="p-1 rounded text-text-4 hover:text-accent hover:bg-accent/10 transition-colors cursor-pointer"
+          title="Edit"
+        >
+          <Pencil size={11} />
+        </button>
+        <button
+          onClick={(e) => { e.stopPropagation(); onDelete?.(tag); }}
+          className="p-1 rounded text-text-4 hover:text-danger hover:bg-danger/10 transition-colors cursor-pointer"
+          title="Delete"
+        >
+          <Trash2 size={11} />
+        </button>
+      </div>
     </div>
   );
 }
 
-function TreeGroup({ node, onSelect, dragOverTag, onDragStart, onDragOver, onDragLeave, onDrop }) {
+function TreeGroup({ node, onSelect, onEdit, onDelete, dragOverTag, onDragStart, onDragOver, onDragLeave, onDrop }) {
   const [expanded, setExpanded] = useState(true);
   const hasChildren = node.children && node.children.length > 0;
 
@@ -278,7 +302,8 @@ function TreeGroup({ node, onSelect, dragOverTag, onDragStart, onDragOver, onDra
       <TreeItem
         tag={node.tag} label={node.tag} isDoc={node.type === 'doc'}
         isDragOver={dragOverTag === node.tag}
-        onSelect={onSelect} onDragStart={onDragStart} onDragOver={onDragOver} onDragLeave={onDragLeave} onDrop={onDrop}
+        onSelect={onSelect} onEdit={onEdit} onDelete={onDelete}
+        onDragStart={onDragStart} onDragOver={onDragOver} onDragLeave={onDragLeave} onDrop={onDrop}
       />
     );
   }
@@ -305,14 +330,16 @@ function TreeGroup({ node, onSelect, dragOverTag, onDragStart, onDragOver, onDra
             <TreeItem
               tag={node.tag} label={node.tag} isDoc={node.type === 'doc'} indent={1}
               isDragOver={false}
-              onSelect={onSelect} onDragStart={onDragStart} onDragOver={onDragOver} onDragLeave={onDragLeave} onDrop={onDrop}
+              onSelect={onSelect} onEdit={onEdit} onDelete={onDelete}
+              onDragStart={onDragStart} onDragOver={onDragOver} onDragLeave={onDragLeave} onDrop={onDrop}
             />
           )}
           {node.children.map((child) => (
             <TreeItem
               key={child.tag} tag={child.tag} label={child.tag.split('/').pop()} isDoc={child.type === 'doc'} indent={1}
               isDragOver={dragOverTag === child.tag}
-              onSelect={onSelect} onDragStart={onDragStart} onDragOver={onDragOver} onDragLeave={onDragLeave} onDrop={onDrop}
+              onSelect={onSelect} onEdit={onEdit} onDelete={onDelete}
+              onDragStart={onDragStart} onDragOver={onDragOver} onDragLeave={onDragLeave} onDrop={onDrop}
             />
           ))}
         </div>
@@ -335,7 +362,7 @@ export default function MemoryView() {
   const setKeeperEditing = useGrooveStore((s) => s.setKeeperEditing);
 
   const [search, setSearch] = useState('');
-  const [viewMode, setViewMode] = useState('list');
+  const [viewMode, setViewMode] = useState('tree');
   const [editorOpen, setEditorOpen] = useState(false);
   const [dragOverTag, setDragOverTag] = useState(null);
   const [draggingTag, setDraggingTag] = useState(null);
@@ -346,10 +373,23 @@ export default function MemoryView() {
     if (keeperEditing) setEditorOpen(true);
   }, [keeperEditing]);
 
-  const filtered = search
-    ? keeperItems.filter((item) => item.tag.includes(search.toLowerCase()))
+  const searchLower = search.toLowerCase();
+  const filtered = searchLower
+    ? keeperItems.filter((item) => item.tag.includes(searchLower))
     : keeperItems;
 
+  const filteredTree = searchLower
+    ? keeperTree
+        .map((node) => {
+          const tagMatch = node.tag.includes(searchLower);
+          const matchingChildren = (node.children || []).filter((c) => c.tag.includes(searchLower));
+          if (tagMatch) return node;
+          if (matchingChildren.length > 0) return { ...node, children: matchingChildren };
+          return null;
+        })
+        .filter(Boolean)
+    : keeperTree;
+
   const handleNew = () => {
     setKeeperEditing({ tag: '', content: '', isNew: true });
     setEditorOpen(true);
@@ -370,6 +410,12 @@ export default function MemoryView() {
     setKeeperEditing(null);
   };
 
+  const handleRename = async (oldTag, newTag, content) => {
+    await moveKeeperItem(oldTag, newTag);
+    await updateKeeperItem(newTag, content);
+    setKeeperEditing(null);
+  };
+
   const handleEditorClose = (open) => {
     setEditorOpen(open);
     if (!open) setKeeperEditing(null);
@@ -384,10 +430,9 @@ export default function MemoryView() {
     setDragOverTag(null);
     setDraggingTag(null);
     if (!sourceTag || !targetTag || sourceTag === targetTag) return;
-    // Don't drop onto self or own children
     if (targetTag.startsWith(sourceTag + '/')) return;
     const sourceName = sourceTag.split('/').pop();
-    const newTag = targetTag + '/' + sourceName;
+    const newTag = targetTag === '__root__' ? sourceName : targetTag + '/' + sourceName;
     if (sourceTag === newTag) return;
     try {
       await moveKeeperItem(sourceTag, newTag);
@@ -464,24 +509,50 @@ export default function MemoryView() {
             </div>
           </div>
         ) : viewMode === 'tree' ? (
-          <div className="p-3 space-y-0.5" onDragOver={(e) => e.preventDefault()}>
-            {keeperTree.map((node) => (
-              <TreeGroup
-                key={node.tag} node={node} onSelect={handleTreeSelect}
-                dragOverTag={dragOverTag}
-                onDragStart={(tag) => setDraggingTag(tag)}
-                onDragOver={(tag) => { if (tag !== draggingTag) setDragOverTag(tag); }}
-                onDragLeave={() => setDragOverTag(null)}
-                onDrop={handleDrop}
-              />
-            ))}
-          </div>
+          filteredTree.length === 0 ? (
+            <div className="flex flex-col items-center justify-center h-32 gap-2">
+              <Search size={18} className="text-text-4" />
+              <p className="text-xs text-text-3">No memories matching &ldquo;{search}&rdquo;</p>
+            </div>
+          ) : (
+            <div className="p-3 space-y-0.5" onDragOver={(e) => e.preventDefault()}>
+              {filteredTree.map((node) => (
+                <TreeGroup
+                  key={node.tag} node={node} onSelect={handleTreeSelect}
+                  onEdit={handleEdit} onDelete={(tag) => deleteKeeperItem(tag)}
+                  dragOverTag={dragOverTag}
+                  onDragStart={(tag) => setDraggingTag(tag)}
+                  onDragOver={(tag) => { if (tag !== draggingTag) setDragOverTag(tag); }}
+                  onDragLeave={() => setDragOverTag(null)}
+                  onDrop={handleDrop}
+                />
+              ))}
+              {draggingTag && draggingTag.includes('/') && (
+                <div
+                  onDragOver={(e) => { e.preventDefault(); e.dataTransfer.dropEffect = 'move'; setDragOverTag('__root__'); }}
+                  onDragLeave={() => setDragOverTag(null)}
+                  onDrop={(e) => { e.preventDefault(); handleDrop(e.dataTransfer.getData('text/plain'), '__root__'); }}
+                  className={`flex items-center gap-2 px-3 py-2 mt-2 rounded-md border border-dashed text-xs transition-colors ${dragOverTag === '__root__' ? 'border-accent/50 bg-accent/10 text-accent' : 'border-border text-text-4'}`}
+                >
+                  <CornerLeftUp size={12} />
+                  <span>Drop here to move to root</span>
+                </div>
+              )}
+            </div>
+          )
         ) : (
-          <div className="p-3 space-y-2">
-            {filtered.map((item) => (
-              <MemoryCard key={item.tag} item={item} onEdit={handleEdit} onDelete={(tag) => deleteKeeperItem(tag)} />
-            ))}
-          </div>
+          filtered.length === 0 ? (
+            <div className="flex flex-col items-center justify-center h-32 gap-2">
+              <Search size={18} className="text-text-4" />
+              <p className="text-xs text-text-3">No memories matching &ldquo;{search}&rdquo;</p>
+            </div>
+          ) : (
+            <div className="p-3 space-y-2">
+              {filtered.map((item) => (
+                <MemoryCard key={item.tag} item={item} onEdit={handleEdit} onDelete={(tag) => deleteKeeperItem(tag)} />
+              ))}
+            </div>
+          )
         )}
       </ScrollArea>
 
@@ -491,6 +562,7 @@ export default function MemoryView() {
         onOpenChange={handleEditorClose}
         editing={keeperEditing}
         onSave={handleSave}
+        onRename={handleRename}
       />
 
       {/* Instruct Modal (command reference) */}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "groove-dev",
-  "version": "0.27.156",
+  "version": "0.27.159",
   "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)",

package/packages/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/cli",
-  "version": "0.27.156",
+  "version": "0.27.159",
   "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.156",
+  "version": "0.27.159",
   "description": "GROOVE daemon — agent orchestration engine",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",

package/packages/daemon/src/journalist.js

@@ -483,6 +483,9 @@ export class Journalist {
       'Be specific. Name files, functions, and line numbers. Do not summarize vaguely.',
       'Preserve the investigation narrative — the next agent needs to understand the',
       'journey, not just the destination.',
+      'IMPORTANT: Focus on what the user ASKED the agent to do (the original task).',
+      'Do NOT include observations about unrelated code issues, potential improvements,',
+      'or things the agent noticed but did not act on. The next agent must stay on task.',
       'Keep your response under 4000 characters.',
       '',
       '---',
@@ -1024,7 +1027,13 @@ export class Journalist {
       originalTask ? `## Original Task\n\n${originalTask}\n` : '',
       ``,
       agent.role === 'planner' ? 'CRITICAL: You are a PLANNING ONLY agent. Do NOT implement code. Route all work to your team via .groove/recommended-team.json.\n' : '',
-      `Continue seamlessly — finish the work and deliver the output.`,
+      `## Instructions`,
+      ``,
+      `Continue and finish the in-progress task — deliver the output. Stay focused on that specific task only.`,
+      `- Do NOT explore the codebase looking for other things to fix or improve`,
+      `- Do NOT start new work outside the original task scope`,
+      `- Do NOT act on TODO comments, code quality issues, or improvements you notice in passing`,
+      `- If the task is already complete, report what was accomplished and STOP — await new instructions from the user`,
     ].filter(Boolean).join('\n');
 
     // Hard cap: 16000 chars — investigation context needs room to preserve the full narrative
@@ -1142,34 +1151,60 @@ export class Journalist {
    * Build a full context-resume prompt that preserves the conversation
    * thread so a fresh agent picks up where the previous session left off.
    */
-  buildConversationResumePrompt(agent, userMessage) {
+  buildConversationResumePrompt(agent, userMessage, { isRotation = false, reason } = {}) {
     const thread = this.extractConversationThread(agent);
     if (!thread) return null;
 
     const constraints = this.daemon.memory?.getConstraintsMarkdown(2000) || '';
     const discoveries = this.daemon.memory?.getDiscoveriesMarkdown(agent.role, 5, 1000, agent.scope, agent.teamId) || '';
 
+    // Extract the user's original task from the conversation — the first substantial
+    // user message is almost always the task assignment. This anchors the new agent.
+    const originalTask = this._extractOriginalTask(thread);
+
+    // Rotation and idle-resume need very different framing. During rotation the agent
+    // has no new user message — it must continue the exact in-progress task without
+    // drifting. During idle-resume the user explicitly sent a message to continue.
+    const isIdleResume = !isRotation && userMessage && userMessage.trim().length > 0;
+
+    const taskFocusBlock = isRotation ? [
+      `## CRITICAL: Task Focus`,
+      ``,
+      `You were auto-rotated (reason: ${reason || 'context_management'}) — this is a routine context refresh, NOT a new assignment.`,
+      originalTask ? `Your task: ${originalTask}` : '',
+      ``,
+      `Rules:`,
+      `- Continue ONLY the task described in the conversation below`,
+      `- Do NOT explore the codebase looking for other things to fix or improve`,
+      `- Do NOT start new work that was not part of the original task`,
+      `- Do NOT act on TODO comments, code quality issues, or improvements you notice in passing`,
+      `- If the task is complete, report what was done and STOP — await new instructions from the user`,
+      `- If the task is in progress, pick up exactly where the previous session left off`,
+      ``,
+    ] : [];
+
     let prompt = [
       `# Session Context Resume`,
       ``,
-      `You are continuing a session that went idle. Below is the full conversation`,
-      `from your previous session — your actual exchanges with the user. Pick up`,
-      `exactly where you left off. The user's new message follows at the end.`,
+      isRotation
+        ? `You are continuing a session after an automatic context rotation. Your context was refreshed but your task has NOT changed. The conversation below is your previous session — continue the same work.`
+        : `You are continuing a session that went idle. Below is the full conversation from your previous session — your actual exchanges with the user. Pick up exactly where you left off. The user's new message follows at the end.`,
       ``,
       `Role: ${agent.role} | Provider: ${agent.provider} | Scope: ${agent.scope?.join(', ') || 'unrestricted'}`,
       agent.workingDir ? `Working directory: ${agent.workingDir}` : '',
       ``,
+      ...taskFocusBlock,
       constraints ? `## Project Constraints\n\n${constraints}\n` : '',
       discoveries ? `## Known Issues & Fixes\n\n${discoveries}\n` : '',
       `## Previous Conversation\n\n${thread}`,
       ``,
       `---`,
       ``,
-      `## New Message From User`,
-      ``,
-      userMessage,
+      isIdleResume ? `## New Message From User\n\n${userMessage}` : '',
       ``,
-      `Continue seamlessly from the conversation above. You have the full context of what was discussed, what was tried, what worked and what didn't. Do not ask the user to repeat anything.`,
+      isRotation
+        ? `Continue the in-progress task from the conversation above. Stay focused on that task only. Do not ask the user to repeat anything. If the task was already completed, state that and wait for new instructions.`
+        : `Continue seamlessly from the conversation above. You have the full context of what was discussed, what was tried, what worked and what didn't. Do not ask the user to repeat anything.`,
     ].filter(Boolean).join('\n');
 
     // Hard cap at 80K chars (~20K tokens) to leave plenty of room in context window
@@ -1180,23 +1215,24 @@ export class Journalist {
         prompt = [
           `# Session Context Resume`,
           ``,
-          `You are continuing a session that went idle. Below is the conversation`,
-          `from your previous session (older turns summarized to fit). Pick up`,
-          `exactly where you left off.`,
+          isRotation
+            ? `You are continuing after an automatic context rotation. Your task has NOT changed.`
+            : `You are continuing a session that went idle (older turns summarized to fit). Pick up exactly where you left off.`,
           ``,
           `Role: ${agent.role} | Scope: ${agent.scope?.join(', ') || 'unrestricted'}`,
           agent.workingDir ? `Working directory: ${agent.workingDir}` : '',
           ``,
+          ...taskFocusBlock,
           constraints ? `## Project Constraints\n\n${constraints}\n` : '',
           `## Previous Conversation\n\n${smallerThread}`,
           ``,
           `---`,
           ``,
-          `## New Message From User`,
-          ``,
-          userMessage,
+          isIdleResume ? `## New Message From User\n\n${userMessage}` : '',
           ``,
-          `Continue seamlessly. Do not ask the user to repeat anything.`,
+          isRotation
+            ? `Continue the in-progress task only. Do not explore or start new work. If done, state that and wait.`
+            : `Continue seamlessly. Do not ask the user to repeat anything.`,
         ].filter(Boolean).join('\n');
       }
     }
@@ -1204,6 +1240,15 @@ export class Journalist {
     return prompt;
   }
 
+  _extractOriginalTask(thread) {
+    // Find the first substantial USER message in the thread — that's the task.
+    const match = thread.match(/\[USER\]:\n([\s\S]*?)(?=\n\n---|\n\[CLAUDE\]:|$)/);
+    if (!match) return '';
+    const firstMsg = match[1].trim();
+    if (firstMsg.length < 10) return '';
+    return firstMsg.length > 500 ? firstMsg.slice(0, 500) + '...' : firstMsg;
+  }
+
   // --- Workspace Grouping ---
 
   /**

package/packages/daemon/src/process.js

@@ -507,6 +507,7 @@ export class ProcessManager {
 
     if (finalStatus === 'completed' && agent.role === 'planner') {
       this._extractRecommendedTeam(agent, logPath);
+      this._consumeRecommendedTeamAutonomous(agent);
       this._handleReviewComplete(agent);
     }
 
@@ -534,7 +535,7 @@ export class ProcessManager {
 
     this._checkPhase2(agent.id);
 
-    if (finalStatus === 'completed' && agent.role === 'fullstack' && agent.teamId) {
+    if (finalStatus === 'completed' && agent.role === 'fullstack' && agent.teamId && agent.metadata?.isQcPhase2) {
       this._triggerReview(agent);
     }
 
@@ -1755,6 +1756,114 @@ For normal file edits within your scope, proceed without review.
     } catch { /* best effort */ }
   }
 
+  /**
+   * Daemon-autonomous consumption of recommended-team.json.
+   * If the file exists and the GUI hasn't already consumed it (no _pendingPhase2
+   * for this team), broadcast a notification so the GUI picks it up on next tick.
+   * This closes the race where GUI polling stops before the file is written.
+   */
+  _consumeRecommendedTeamAutonomous(agent) {
+    try {
+      const workDir = agent.workingDir || this.daemon.projectDir;
+      const targetPath = resolve(workDir, '.groove', 'recommended-team.json');
+      if (!existsSync(targetPath)) return;
+
+      const teamId = agent.teamId || null;
+
+      // If phase 2 is already pending for this team, GUI already consumed it
+      const pending = this.daemon._pendingPhase2 || [];
+      if (teamId && pending.some(g => g.agents.some(a => a.teamId === teamId))) return;
+
+      // Broadcast so the GUI knows to fetch — even if its polling interval was cleared
+      this.daemon.broadcast({
+        type: 'recommended-team:ready',
+        teamId,
+        agentId: agent.id,
+        agentName: agent.name,
+      });
+
+      // Delayed self-consumption: if the GUI doesn't consume within 5s, daemon does it directly
+      setTimeout(() => {
+        if (!existsSync(targetPath)) return; // GUI consumed it
+        try {
+          const raw = JSON.parse(readFileSync(targetPath, 'utf8'));
+          delete raw._meta;
+          const agentConfigs = Array.isArray(raw) ? raw : (raw.agents || []);
+          if (agentConfigs.length === 0) return;
+
+          const phase1 = agentConfigs.filter(a => !a.phase || a.phase === 1);
+          const phase2 = agentConfigs.filter(a => a.phase === 2);
+          if (phase1.length === 0) return;
+
+          // Check again — GUI may have consumed during the timeout
+          const currentPending = this.daemon._pendingPhase2 || [];
+          if (teamId && currentPending.some(g => g.agents.some(a => a.teamId === teamId))) return;
+
+          const baseDir = agent.workingDir || this.daemon.projectDir;
+          const projectDir = raw.projectDir || null;
+          let projectWorkingDir = baseDir;
+          if (projectDir) {
+            const safeName = String(projectDir).replace(/[^a-zA-Z0-9_-]/g, '-').slice(0, 64);
+            projectWorkingDir = resolve(baseDir, safeName);
+            mkdirSync(projectWorkingDir, { recursive: true });
+          }
+
+          const defaultTeamId = teamId || this.daemon.teams.getDefault()?.id || null;
+          const phase1Ids = [];
+
+          // Spawn phase 1 agents
+          const spawnPromises = phase1.map(async (config) => {
+            try {
+              const validated = validateAgentConfig({
+                role: config.role,
+                scope: config.scope || [],
+                prompt: config.prompt || '',
+                provider: config.provider || agent.provider || this.daemon.config?.defaultProvider,
+                model: config.model || agent.model || this.daemon.config?.defaultModel || 'auto',
+                permission: config.permission || 'auto',
+                workingDir: config.workingDir || projectWorkingDir,
+                name: config.name || undefined,
+              });
+              validated.teamId = defaultTeamId;
+              const spawned = await this.spawn(validated);
+              phase1Ids.push(spawned.id);
+              return spawned;
+            } catch (err) {
+              console.error(`[Groove] Autonomous team launch: failed to spawn ${config.role}: ${err.message}`);
+              return null;
+            }
+          });
+
+          Promise.all(spawnPromises).then(() => {
+            // Set up phase 2 pending
+            if (phase2.length > 0 && phase1Ids.length > 0) {
+              this.daemon._pendingPhase2 = this.daemon._pendingPhase2 || [];
+              this.daemon._pendingPhase2.push({
+                waitFor: phase1Ids,
+                agents: phase2.map(c => ({
+                  role: c.role, scope: c.scope || [], prompt: c.prompt || '',
+                  provider: c.provider || agent.provider || this.daemon.config?.defaultProvider,
+                  model: c.model || agent.model || this.daemon.config?.defaultModel || 'auto',
+                  permission: c.permission || 'auto',
+                  workingDir: c.workingDir || projectWorkingDir,
+                  name: c.name || undefined,
+                  teamId: defaultTeamId,
+                })),
+              });
+            }
+
+            // Clean up the file
+            try { unlinkSync(targetPath); } catch { /* */ }
+            this.daemon.audit?.log('team.autonomousLaunch', { teamId: defaultTeamId, phase1: phase1Ids.length, phase2: phase2.length });
+            console.log(`[Groove] Autonomous team launch: ${phase1Ids.length} phase 1 agents spawned for team ${defaultTeamId}`);
+          });
+        } catch (err) {
+          console.error(`[Groove] Autonomous team consumption failed: ${err.message}`);
+        }
+      }, 5000);
+    } catch { /* best effort */ }
+  }
+
   /**
    * Check if a completed/crashed agent was the last phase 1 agent in a team.
    * If so, auto-spawn the phase 2 (QC/finisher) agents.
@@ -1834,13 +1943,26 @@ For normal file edits within your scope, proceed without review.
           try {
             const validated = validateAgentConfig(config);
             if (!validated.teamId) validated.teamId = this.daemon.teams.getDefault()?.id || null;
+            validated.metadata = { ...(validated.metadata || {}), isQcPhase2: true };
+            const existingId = existing?.id || null;
             const p = this.spawn(validated).then((agent) => {
+              registry.update(agent.id, { metadata: { ...(agent.metadata || {}), isQcPhase2: true } });
               this.daemon.broadcast({
                 type: 'phase2:spawned',
                 agentId: agent.id,
+                oldAgentId: existingId,
                 name: agent.name,
                 role: agent.role,
               });
+              if (existingId) {
+                this.daemon.broadcast({
+                  type: 'rotation:complete',
+                  agentId: agent.id,
+                  oldAgentId: existingId,
+                  agentName: agent.name,
+                  reason: 'phase2_respawn',
+                });
+              }
               this.daemon.audit.log('phase2.autoSpawn', { id: agent.id, name: agent.name, role: agent.role });
             }).catch((err) => {
               console.error(`[Groove] Phase 2 spawn failed for ${config.role}: ${err.message}`);
@@ -1884,6 +2006,7 @@ For normal file edits within your scope, proceed without review.
     const teamId = agent.teamId;
     if (!teamId) return;
     if (this._reviewTriggered.has(teamId)) return;
+    this._reviewTriggered.add(teamId);
 
     const registry = this.daemon.registry;
     const teamAgents = registry.getAll().filter(a => a.teamId === teamId);
@@ -1898,7 +2021,6 @@ For normal file edits within your scope, proceed without review.
       a.role !== 'planner' && (a.status === 'running' || a.status === 'starting'));
     if (hasRunning) return;
 
-    this._reviewTriggered.add(teamId);
     this._reviewPending.add(teamId);
 
     const journalist = this.daemon.journalist;
@@ -1912,6 +2034,12 @@ For normal file edits within your scope, proceed without review.
       for (const f of (journalist?.getAgentFiles(a) || [])) allFiles.add(f);
     }
 
+    if (!originalSpec.trim() && !plannerResult.trim() && allFiles.size === 0) {
+      console.log(`[Groove] Review skipped for team ${teamId}: no spec, plan, or files to review`);
+      this._reviewPending.delete(teamId);
+      return;
+    }
+
     const reviewPrompt = `You are reviewing a completed team build against the original specification.
 
 ## Original Task