groove-dev 0.27.37 → 0.27.39

package/packages/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-Df4O6yJI.js"></script>
+    <script type="module" crossorigin src="/assets/index-BRZ_leqO.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/packages/gui/package.json

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

package/packages/gui/src/components/onboarding/setup-wizard.jsx

@@ -19,7 +19,7 @@ const PROVIDERS = [
     id: 'claude-code',
     name: 'Claude Code',
     subtitle: 'by Anthropic',
-    models: ['Opus 4.6', 'Sonnet 4.6', 'Haiku 4.5'],
+    models: ['Opus 4.7', 'Opus 4.6', 'Sonnet 4.6', 'Haiku 4.5'],
     authType: 'Subscription or API key',
     authModes: ['subscription', 'apikey'],
     recommended: true,

package/packages/gui/src/components/ui/toast.jsx

@@ -64,6 +64,18 @@ function ToastItem({ toast }) {
           <p className="text-xs text-text-3 font-sans mt-0.5">{toast.detail}</p>
         )}
       </div>
+      {toast.action?.url && (
+        <button
+          onClick={(e) => {
+            e.stopPropagation();
+            try { window.open(toast.action.url, '_blank', 'noopener'); } catch {}
+            removeToast(toast.id);
+          }}
+          className="text-xs font-medium text-accent hover:text-accent-hover bg-surface-5 hover:bg-surface-6 px-3 py-1.5 rounded transition-colors cursor-pointer flex-shrink-0 whitespace-nowrap"
+        >
+          {toast.action.label || 'Open'}
+        </button>
+      )}
       <button
         onClick={(e) => { e.stopPropagation(); removeToast(toast.id); }}
         className="p-1.5 text-text-4 hover:text-text-1 hover:bg-surface-5 rounded transition-colors cursor-pointer flex-shrink-0 z-10"

package/packages/gui/src/stores/groove.js

@@ -381,6 +381,39 @@ export const useGrooveStore = create((set, get) => ({
           get().addToast('info', `QC agent ${msg.name} auto-spawned`, 'Auditing phase 1 work');
           break;
 
+        case 'preview:ready':
+          get().addToast(
+            'success',
+            'Project ready to preview',
+            msg.url,
+            { label: 'View Site', url: msg.url },
+          );
+          break;
+
+        case 'preview:failed':
+          get().addToast(
+            'warning',
+            'Preview could not launch',
+            msg.reason ? String(msg.reason).slice(0, 200) : 'Unknown error',
+          );
+          break;
+
+        case 'preview:stopped':
+          break;
+
+        case 'agent:stalled': {
+          const name = msg.agentName || msg.agentId;
+          const secs = Math.round((msg.silentMs || 0) / 1000);
+          get().addToast('warning', `${name} may be stalled`, `No output for ${secs}s — API stream may be hung`);
+          break;
+        }
+
+        case 'knock:denied': {
+          const name = msg.agentName || msg.agentId;
+          get().addToast('warning', `${name} blocked`, `${msg.toolName} on ${msg.target} — ${msg.reason || 'scope conflict'}`);
+          break;
+        }
+
         case 'phase2:failed':
           get().addToast('error', `QC agent failed to spawn`, msg.error || 'Unknown error');
           break;
@@ -725,9 +758,9 @@ export const useGrooveStore = create((set, get) => ({
 
   // ── Toasts ────────────────────────────────────────────────
 
-  addToast(type, message, detail) {
+  addToast(type, message, detail, action) {
     const id = ++toastCounter;
-    set((s) => ({ toasts: [...s.toasts, { id, type, message, detail }] }));
+    set((s) => ({ toasts: [...s.toasts, { id, type, message, detail, action }] }));
   },
   removeToast(id) {
     set((s) => ({ toasts: s.toasts.filter((t) => t.id !== id) }));

package/plans/chat-persistence-refactor.md

@@ -1,154 +0,0 @@
-# Chat & Activity Persistence Refactor
-
-## Context
-
-On 2026-04-14, node positions were failing to persist across team switches. Six
-attempted fixes in `packages/gui/src/views/agents.jsx` all failed because they
-targeted the wrong layer. On-screen debug logging (since removed) revealed the
-actual cause:
-
-```
-SAVE.err: Failed to execute 'setItem' on 'Storage': Setting the value of
-'groove:nodePositions:...' exceeded the quota.
-```
-
-`localStorage` was full. `chatHistory` (100 msgs/agent) and `activityLog`
-(200 events/agent) were persisted on every agent stdout event and never pruned
-when agents were killed. Over time they consumed the browser's ~5–10 MB quota,
-so every new `setItem` silently failed — including node positions, panel
-widths, and anything else.
-
-## What's already shipped
-
-Two stopgap fixes are in `main` so the app works today:
-
-- `packages/gui/src/stores/groove.js` — on every WebSocket `state` broadcast,
-  prune `tokenTimeline`, `chatHistory`, and `activityLog` entries whose agent
-  IDs are no longer in the live registry. Keeps localStorage bounded by live
-  agents instead of all-agents-ever.
-- `packages/gui/src/views/agents.jsx` `savePositions()` — on quota error,
-  drop `groove:chatHistory` + `groove:activityLog` and retry.
-
-These are sufficient for normal usage (5–20 concurrent agents with moderate
-output). They do not fix the underlying architectural problem.
-
-## Remaining failure modes
-
-1. **Single chatty agent hits quota alone.** Messages are count-capped (100)
-   but not size-capped. One agent dumping 50 KB tool outputs × 100 = 5 MB,
-   quota gone before any prune runs.
-2. **Main-thread stalls.** Every stdout event does a synchronous
-   `JSON.stringify` of the full `chatHistory` object + `setItem`. Scales O(n
-   agents × m messages). Worse with more agents.
-3. **Browser cache clears wipe history.** `localStorage` is per-origin,
-   per-browser. Clearing the Electron webview or switching devices loses all
-   conversation state.
-4. **Multi-client divergence.** If/when we support web GUI + Electron hitting
-   the same daemon, each has its own localStorage — they'll show different
-   chat histories for the same agent.
-5. **No querying.** Want "last message across all agents" or "search for
-   'error' in activity"? Not possible without deserializing everything.
-
-## Proposed architecture
-
-Move chat, activity, and timeline to daemon-side disk persistence. Keep
-`localStorage` for tiny UI state only (node positions, viewport, panel widths,
-`activeTeamId`, `expandedNodes`).
-
-### Daemon changes (`packages/daemon/src/`)
-
-New module `chatlog.js`:
-- Append-only JSONL files at `.groove/chat/<agentId>.jsonl` and
-  `.groove/activity/<agentId>.jsonl`.
-- Each line: `{ t, from, text, type?, subtype? }` for chat;
-  `{ t, text, type, subtype }` for activity.
-- Write on every message — fs.appendFile with a 0o600 perm. No in-memory
-  buffer needed; kernel page cache handles batching.
-- Add rotation: if a file exceeds ~5 MB, rename to `.1.jsonl` and start fresh.
-  Keep at most N=3 rotations. (Protects against a runaway agent eating disk.)
-- On agent kill/completion: leave files alone. GC cleans them later (existing
-  `_gc()` in `index.js` already handles `GROOVE_AGENT_LOGS/` — extend to
-  `.groove/chat/` and `.groove/activity/`.)
-
-New endpoints in `api.js`:
-- `GET /api/agents/:id/chat?cursor=<lineNumber>&limit=100` — returns
-  `{ lines: [...], nextCursor, hasMore }`. Paginate from newest backwards
-  (like tail -f) so the UI loads the last 100 first.
-- `GET /api/agents/:id/activity?cursor=...&limit=...` — same shape.
-- `DELETE /api/agents/:id/chat` — for future "clear history" UI.
-
-Integrate into `process.js` stdout handler:
-- When we currently emit `agent:stdout` / `agent:activity` over WebSocket,
-  also call `chatlog.appendChat(agentId, entry)` or
-  `chatlog.appendActivity(agentId, entry)`. WebSocket push stays for live
-  streaming; JSONL is the durable tail.
-
-### GUI changes (`packages/gui/src/`)
-
-In `stores/groove.js`:
-- Remove `chatHistory` and `activityLog` from the persisted keys. Kill the
-  `persistJSON('groove:chatHistory', ...)` and `persistJSON('groove:activityLog', ...)`
-  calls in the stdout handler.
-- Keep them as in-memory state only, scoped to agents the user has opened in
-  this session. When user opens an agent panel, hydrate from
-  `GET /api/agents/:id/chat` + `GET /api/agents/:id/activity`.
-- When user closes the panel or switches away, drop from memory after a grace
-  period (e.g., 5 min LRU). WebSocket push still updates the in-memory copy
-  for the active agent.
-- Optional: cap total in-memory messages (e.g., 1000 across all agents) with
-  LRU eviction by agent.
-
-In `components/agents/agent-chat.jsx` (or wherever chat is rendered):
-- On mount: dispatch `fetchAgentHistory(agentId)` if not already loaded.
-- Show a "Load more" button or scroll-to-top infinite scroll for pagination.
-
-### What stays in localStorage
-
-- `groove:nodePositions:<teamId>` — tiny, per-team, bounded by agent count.
-- `groove:teamViewports`, `groove:teamOrder`, `groove:activeTeamId`.
-- `groove:detailWidth`, `groove:terminalVisible`, `groove:terminalHeight`,
-  `groove:editorSidebarWidth`, `groove:expandedNodes`.
-- `groove:onboardingComplete`, `groove:storeVersion`.
-
-None of these individually exceed a few KB. Total <100 KB in the worst case.
-
-## Implementation order
-
-1. Add `chatlog.js` + wire JSONL writes in `process.js`. Tests: spawn an
-   agent, verify `.groove/chat/<id>.jsonl` grows; kill it, restart daemon,
-   verify file still there.
-2. Add GET endpoints in `api.js` + GUI fetch helpers. Test: hit endpoint with
-   curl, verify cursor pagination.
-3. Add in-memory + lazy-load path in `stores/groove.js`. Keep localStorage
-   writes in parallel behind a feature flag (`groove:chatPersist=daemon`) so
-   we can A/B before flipping default.
-4. Flip default, remove localStorage writes for chatHistory/activityLog,
-   remove the prune-stale-agents stopgap in the `state` handler.
-5. Add GC for `.groove/chat/*.jsonl` and `.groove/activity/*.jsonl` in
-   `index.js` `_gc()` (dead agents after N days).
-
-## Estimated effort
-
-2–3 hours focused work. Bulk is the GUI refactor — the daemon side is a thin
-append-only writer + two endpoints.
-
-## Files touched (expected)
-
-- `packages/daemon/src/chatlog.js` (new)
-- `packages/daemon/src/process.js` (add appendChat/appendActivity calls)
-- `packages/daemon/src/api.js` (two new GET endpoints)
-- `packages/daemon/src/index.js` (import chatlog; extend `_gc()`)
-- `packages/gui/src/stores/groove.js` (remove persistJSON for chat/activity;
-  add fetchAgentHistory action; drop prune-stale-agents stopgap from `state`
-  handler)
-- `packages/gui/src/components/agents/agent-chat.jsx` (hydrate on mount)
-- `packages/gui/src/components/agents/agent-feed.jsx` (same for activity)
-
-## When to do this
-
-Not urgent. The current prune+retry stopgap handles normal usage. Priority
-bumps if:
-- A user reports chat/activity loss that prune-on-broadcast didn't catch.
-- We add multi-client access (web GUI alongside Electron).
-- We add chat search or cross-agent analytics.
-- An agent with huge tool outputs single-handedly hits quota in production.