agentgui 1.0.921 → 1.0.923

package/AGENTS.md

@@ -73,4 +73,16 @@ Only after the real generator is patched will agentgui sessions run autonomously
 
 **acptoapi (c:\dev\acptoapi) merged Claude Code history routes as of 2026-05-02; ccsniff package is no longer needed.**
 
-History functionality (`GET /v1/history/*` endpoints) is now built into acptoapi. Routes: `snapshot` (event/session/project/tool/error counts + byte/date range), `sessions` (list with title/project/cwd/counts), `sessions/:sid/events` (flattened events), `search` (BM25 with snippets), `reindex` (rebuild index), `stream` (SSE). Implementation: `lib/history/` (bm25.js for tokenize/buildIndex/search/snippet, watcher.js for JsonlWatcher + JsonlReplayer, index.js for HistoryStore singleton + flattenEvent). Reads `~/.claude/projects` by default; override with `CLAUDE_PROJECTS_DIR` env var. The ccsniff package itself is no longer required — acptoapi covers the functionality entirely.
+History functionality (`GET /v1/history/*` endpoints) is now built into acptoapi. Routes: `snapshot` (event/session/project/tool/error counts + byte/date range), `sessions` (list with title/project/cwd/counts), `sessions/:sid/events` (flattened events), `search` (BM25 with snippets), `reindex` (rebuild index), `stream` (SSE). Implementation: `lib/history/` (bm25.js for tokenize/buildIndex/search/snippet, watcher.js for JsonlWatcher + JsonlReplayer, index.js for HistoryStore singleton + flattenEvent). Reads `~/.claude/projects` by default; override with `CLAUDE_PROJECTS_DIR` env var. The ccsniff package itself is no longer needed — acptoapi covers the functionality entirely.
+
+## buildSystemPrompt System Prompt for claude-code
+
+**`lib/provider-config.js` buildSystemPrompt() must return '' for claude-code agent; returning "Model: X." breaks conversation resume.**
+
+The function previously returned "Model: X." when agentId was 'claude-code' and model was non-null. This caused `buildArgs` in lib/claude-runner-agents.js to pass `--append-system-prompt "Model: X."` to the claude CLI, which triggers "argument missing" error on conversation resume. Fix: return '' early when agentId is 'claude-code' or falsy. The model is already passed via `--model` flag; system prompt is only for non-claude-code agents.
+
+## WebSocket Sync Endpoint Testing
+
+**WebSocket `/sync` endpoint — message ordering requires registering handler BEFORE sending.**
+
+Server sends `sync_connected` with `clientId` on connect. Legacy handler (`lib/ws-legacy-handlers.js`) handles `ping→pong`, `subscribe→subscription_confirmed`, `get_subscriptions→subscriptions`, `unsubscribe`, `latency_report`. All responses use codec encode/decode (`lib/codec.js`). Pattern: queue outbound messages and use a waiters array + sequential promises to avoid race between send and handler registration. Test structure: `const queued = []; let waiting; ws.on('message', ...); queued.forEach(msg => ws.send(msg)); waiting.resolve(...)` ensures the handler is live before messages flow.

package/lib/provider-config.js

@@ -3,11 +3,10 @@ import path from 'path';
 import os from 'os';
 
 export function buildSystemPrompt(agentId, model, subAgent) {
+  if (!agentId || agentId === 'claude-code') return '';
   const parts = [];
-  if (agentId && agentId !== 'claude-code') {
-    const displayAgentId = agentId.split('-·-')[0];
-    parts.push(`Use ${displayAgentId} subagent for all tasks.`);
-  }
+  const displayAgentId = agentId.split('-·-')[0];
+  parts.push(`Use ${displayAgentId} subagent for all tasks.`);
   if (model) parts.push(`Model: ${model}.`);
   if (subAgent) parts.push(`Subagent: ${subAgent}.`);
   return parts.join(' ');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentgui",
-  "version": "1.0.921",
+  "version": "1.0.923",
   "description": "Multi-agent ACP client with real-time communication",
   "type": "module",
   "main": "electron/main.js",