@j-o-r/hello-dave 0.0.6 → 0.0.8

package/docs/toolset.md

@@ -0,0 +1,164 @@
+# ToolSet (\`lib/ToolSet.js\`)
+
+## Overview
+**ToolSet** manages LLM function calls (tools): registration, execution, listing. Integrates with \`Prompt.js\` (via \`execute(prompt)\`) and \`AgentManager.js\` (pre-configured modes, generics). Supports JSON Schema params, async methods. Exports from \`lib/genericToolset.js\`: \`toolsPool\` (11 pre-built tools).
+
+**Uses named exports from \`lib/index.js\` - NO default export.**
+
+Key Features:
+- **Modes**: \`toolChoice: 'auto'|'none'|'required'\` (default \`'auto'\`).
+- **Validation**: Name regex \`/^\[#![a-z_0-9]{2,}$/\`.
+- **Events** (via \`Prompt\`): \`tool_request\`, \`tool_error\`, \`tool_response\`.
+- **Execution**: \`execute(prompt)\` processes \`function_request\`s from last message → calls → adds \`function_response\`s.
+
+## Constructor
+\`\`\`javascript
+import { ToolSet } from '@j-o-r/hello-dave';
+
+new ToolSet(choice = 'auto');  // 'auto'|'none'|'required'
+\`\`\`
+
+## Methods
+| Method | Args | Returns | Description |
+|--------|------|---------|-------------|
+| \`add(name, desc, params, method)\` | \`string\`, \`string\`, \`TSSchema\`, \`async (params) => *\` | \`void\` | Register tool. Overwrites if exists. |
+| \`get(name)\` | \`string\` | \`TSTool\` | Get tool details. |
+| \`delete(name)\` | \`string\` | \`void\` | Remove tool. |
+| \`has(name)\` | \`string\` | \`boolean\` | Exists? |
+| \`list()\` | - | \`TSToolListItem[]\` (sorted) | All tools (name/desc/params). |
+| \`call(name, params)\` | \`string\`, \`object\` | \`Promise<*\>\` | Execute tool. |
+| \`execute(prompt)\` | \`Prompt\` | \`Promise<void>\` | Process last message \`function_request\`s → execute → add \`function_response\`s to \`tool\` role. |
+| \`length\` | - | \`number\` | Tool count. |
+| \`toolChoice\` | - | \`string\` | Getter for mode. |
+
+**TSTool**: \`{description: string, parameters: TSSchema, method: async (params) => *}\`  
+**TSSchema**: OpenAI-style JSON Schema (\`{type: 'object', properties: {...}, required: [...]}\`).
+
+## Generic Tools (\`lib/genericToolset.js\` → \`toolsPool\`)
+| `syntax_check` | Validate syntax (JS/Py/Bash/JSON) via utils/syntax_check.sh. |\n|-----|-----|
+Pre-built \`ToolSet('auto')\` with **11 tools**. Copied via \`AgentManager.addGenericToolcall(name)\` or \`toolsetMode: 'auto'\`.
+
+| Tool Name                  | Description |
+|----------------------------|-------------|
+| \`javascript_interpreter\` | Execute ESM ES6 JavaScript on \`node\`. \`console.log\` captures output. cwd: \`~/devpri/js/hello-dave\`. |
+| \`get_user_env\`           | Get user environment (name, system, city/region/country/timezone, external IP). |
+| \`execute_bash_script\`    | Execute raw Bash script/command (no escaping; verbatim \`$\| < > & " ' \\\\` newlines \`$(())\` \`[[ ]]\`. Heredoc-safe). Ubuntu 25.10. |
+| \`send_email\`             | Send email via \`msmtp\` (to, subject, body). |
+| \`open_link\`              | Open URL/file with \`xdg-open\`. |
+| \`execute_remote_script\`  | Execute raw Bash on remote via SSH (\`ssh://user@host[:port]\`). |
+| \`history_search\`         | Search chat sessions: \`"(todo\\|task)"\` or \`"package.json"\`. Regex in \`.cache/[app]/[prompt]/sessions/*.ndjson\`. |
+| \`read_file\`              | Read raw file in CWD (relative path, no \`/\` \`..\` \`\\\\\`). |
+| \`write_file\`             | Write raw content to file in CWD (relative, as-is; no escaping). Returns bytes written. |
+| \`memory_recall\`          | Recall persistent memory entries from agent-specific storage (\`.cache/[agent]/memory.ndjson\`). |
+| \`memory_write\`           | Write/update persistent memory entries for long-term state (\`.cache/[agent]/memory.ndjson\`). |
+
+## Memory Tools (New)
+These tools provide persistent, agent-specific memory storage in NDJSON format (\`.cache/[agent-name]/memory.ndjson\`). Append-only; latest value per key. Ideal for token efficiency by offloading state/decisions from context.
+
+### \`memory_recall\`
+**Description**: Retrieve the latest values for specified keys from memory file. Returns JSON: \`{key: value, ...}\` or \`{}\` if missing.
+
+**Parameters**:
+\`\`\`json
+{
+  "type": "object",
+  "properties": {
+    "keys": {
+      "type": "array",
+      "items": { "type": "string" },
+      "description": "List of memory keys to recall, e.g., ['current_task', 'last_decision']"
+    }
+  },
+  "required": ["keys"]
+}
+\`\`\`
+
+**Example Usage in Agent Prompt**:
+\`\`\`
+Before proceeding, recall your current task and previous decision:
+- Call memory_recall with keys: ["current_task", "last_decision"]
+Then, use the recalled info to avoid repetition.
+\`\`\`
+
+**Token Efficiency**: Stores tasks/decisions externally. Recall only needed state (e.g., 100 tokens) vs. full history (10k+ tokens), preventing loops/re-reasoning.
+
+### \`memory_write\`
+**Description**: Append/update key-value pairs to memory (overwrites prior same-key entries). Returns confirmation with stored entries.
+
+**Parameters**:
+\`\`\`json
+{
+  "type": "object",
+  "properties": {
+    "entries": {
+      "type": "array",
+      "items": {
+        "type": "object",
+        "properties": {
+          "key": { "type": "string", "description": "Unique key (e.g., 'current_task')" },
+          "value": { "type": "string", "description": "JSON-stringifiable value" }
+        },
+        "required": ["key", "value"]
+      },
+      "description": "List of {key, value} pairs to store"
+    }
+  },
+  "required": ["entries"]
+}
+\`\`\`
+
+**Example Usage in Agent Prompt**:
+\`\`\`
+After analyzing, store the plan:
+- Call memory_write with entries: [{"key": "current_task", "value": "Implement login feature"}, {"key": "step", "value": "1/5"}]
+\`\`\`
+
+**Token Efficiency**: Persist intermediate results across calls/sessions. Avoids context explosion/loops by recalling compact state instead of regenerating.
+
+## Integration with AgentManager
+In \`setup()\`:
+\`\`\`javascript
+agent.setup({
+  toolsetMode: 'auto',  // Loads all 11 generic tools
+  // or 'required': Empty ToolSet (add manually)
+});
+agent.addGenericToolcall('memory_recall');  // Selective copy post-setup
+\`\`\`
+
+**Custom Tools**:
+\`\`\`javascript
+import { AgentManager } from '@j-o-r/hello-dave';
+
+agent.setup({ toolsetMode: 'required' });
+const ts = agent.getToolset();
+ts.add('math', 'Add numbers', {
+  type: 'object',
+  properties: {a: {type: 'number'}, b: {type: 'number'}},
+  required: ['a', 'b']
+}, async ({a, b}) => a + b);
+\`\`\`
+
+**Workflow** (in \`Prompt.call()\` / API adapters):
+1. LLM → \`function_request\`s (assistant message).
+2. \`ToolSet.execute(prompt)\` → parallel \`call()\` → \`function_response\`s (tool message).
+3. Recurse until text response.
+
+## Examples
+### Standalone ToolSet
+\`\`\`javascript
+import { ToolSet } from '@j-o-r/hello-dave';
+
+const ts = new ToolSet('required');
+ts.add('hello', 'Say hello', {type: 'object', properties: {name: {type: 'string'}}, required: ['name']},
+  async ({name}) => \`Hello, \${name}!\`);
+console.log(await ts.call('hello', {name: 'World'}));  // "Hello, World!"
+\`\`\`
+
+### With Prompt/AgentManager
+See [AgentManager](./agent-manager.md#custom-tools--generics), [Prompt](./prompt-class.md).
+
+**Updated:** March 26, 2026 (11 generics from \`lib/genericToolset.js\`).
+
+See also: [GenericToolset Source](./generic-toolset.md).
+\n## Syntax Validation\nSee [tools-syntax-validation.md](tools-syntax-validation.md) for `write_file` JS checks + `syntax_check` tool.
+\n## Path Resolution\nSee [path-resolution-best-practices.md](path-resolution-best-practices.md) for __dirname vs cwd() in tools (e.g., syntax_check.sh).\\n

package/docs/xai-responses.md

@@ -0,0 +1,111 @@
+# xAI Responses API Integration (`lib/API/x.ai/responses.js`)
+
+## Overview
+This module provides a seamless wrapper for the [xAI Responses API](https://docs.x.ai/docs/api-reference#create-new-response) (formerly Grok API), adapted for the hello-dave framework. It integrates directly with `Prompt.js` and `ToolSet.js`, enabling tool-using agents via `AgentManager`.
+
+**Uses named exports from `lib/index.js` - NO default export.**
+
+Key adaptations:
+- **Unified Interface**: Uses `Prompt` messages/history for input; parses `output` into assistant/tool responses.
+- **Recursive Tool Calls**: Automatically handles `function_call` → execute tools → `function_call_output` loops (up to `GLOBAL.max_recursive_requests`).
+- **Tool Support**: Converts `ToolSet` to xAI `function` tools; supports native xAI tools like `web_search`, `x_search`.
+- **Reasoning & Search**: Configurable `reasoning.effort` (`low`/`medium`/`high`), `search_parameters`.
+- **Streaming**: Not yet implemented (future).
+- **Auth**: Requires `XAIKEY` env var.
+
+Default model: `grok-4-fast-reasoning`. Supports: `grok-4-fast-reasoning`, `grok-4-fast-non-reasoning`, etc.
+
+## Usage in AgentManager
+```javascript
+import { AgentManager } from '@j-o-r/hello-dave';
+
+const agent = new AgentManager({ name: 'weatheragent', secret: 'optional-ws-secret' });
+agent.setup({
+  prompt: 'You are a precise weather expert. Use tools for real-time data.',
+  api: 'xai',
+  options: { model: 'grok-4-fast-reasoning', reasoning: { effort: 'high' } },
+  toolsetMode: 'auto',  // Adds generic tools like web_search
+  contextWindow: 128000
+});
+agent.addGenericToolcall('web_search');  // Optional: Add extras
+
+// CLI mode
+agent.start();  // Interactive chat
+
+// Or direct call
+const response = await agent.directCall('Weather in Amsterdam today?');
+console.log(response);  // Handles search/tools automatically
+```
+
+Under the hood:
+1. `Prompt.setAdaptor(API.text['xai'], toolset, options)` → uses `responses.request()`.
+2. Converts history to xAI `input[]` (system/user/assistant/tool).
+3. POST to `https://api.x.ai/v1/responses`.
+4. Parses `output` → adds text/reasoning/tools to `Prompt`.
+5. If tools needed, `ToolSet.execute()` → recurse until complete.
+
+## Options (XAIOptions)
+| Property | Type | Default | Description |
+|----------|------|---------|-------------|
+| `model` | string | `'grok-4-fast-non-reasoning'` | e.g., `'grok-4-fast-reasoning'` |
+| `input` | `XAIInput[]` | `[{role: 'system', content: 'Be precise'}]` | Messages/history |
+| `reasoning.effort` | `'low'\|'medium'\|'high'` | `'medium'` | Reasoning depth |
+| `reasoning.summary` | `'auto'\|'concise'\|'detailed'` | `'auto'` | Reasoning output style |
+| `temperature` | number | `1` (0-2) | Sampling |
+| `parallel_tool_calls` | boolean | `true` | Parallel functions |
+| `tool_choice` | `'auto'` | `'auto'` | Tool selection |
+| `tools` | `XAIFunctionTool[]` | `[]` | From `ToolSet` |
+| `store` | boolean | `false` | Persist response (still bills tokens) |
+| `max_output_tokens` | number | `4000` | Limit |
+| `search_parameters` | object | - | `{max_search_results: 4, mode: 'auto', return_citations: true}` |
+
+**Tool Types**:
+- `function`: From `ToolSet` (name, description, parameters JSONSchema).
+- Native: `web_search`, `x_search` (filters: dates, domains, etc.).
+
+## Example Request (Generated Internally)
+```json
+{
+  "model": "grok-4-fast-reasoning",
+  "input": [
+    {"role": "system", "content": "You are a precise weather expert."},
+    {"role": "user", "content": "Weather in Amsterdam today?"}
+  ],
+  "parallel_tool_calls": true,
+  "reasoning": {"effort": "high"},
+  "search_parameters": {"mode": "auto", "return_citations": true},
+  "tools": [{"type": "function", "name": "web_search", ...}]
+}
+```
+
+## Example Response (Parsed to Prompt)
+- **Text + Citations**: Added as `assistant` text + `log` annotations (URLs).
+- **Function Calls**: `assistant` with `function_request` → `ToolSet.execute()`.
+- **Reasoning**: Prepended as `reasoning` multimodal text.
+
+Sample parsed:
+```
+Today is [date]. Weather: Cloudy, 5°C... [citations]
+```
+Tokens: `input_tokens`, `output_tokens` (incl. `reasoning_tokens`) logged in `Prompt.records`.
+
+## Advanced: Custom Tools
+Extend `ToolSet` before `setup()`:
+```javascript
+import { ToolSet } from '@j-o-r/hello-dave';
+
+const ts = new ToolSet('required');
+ts.add('get_weather', 'Get weather for a location', {
+  type: 'object',
+  properties: { location: {type: 'string'} },
+  required: ['location']
+}, async (params) => `Weather: ${params.location} - 20°C`);  // Handler
+agent.setup({ ..., toolsetMode: ts });
+```
+
+## Errors & Limits
+- `Max recursive calls`: `GLOBAL.max_recursive_requests`.
+- Missing `XAIKEY`: Throws.
+- Invalid last message: Must end in `user`/`tool`.
+
+See `lib/API/x.ai/responses.js` source for full JSDoc/types. Cross-links: [Prompt](../prompt-class.md), [ToolSet](../toolset.md), [AgentManager](agent-manager.md).

package/docs/xai_collections.md

@@ -0,0 +1,106 @@
+[x.ai collections](https://docs.x.ai/docs/guides/using-collections/api)
+
+
+Creating a collection:
+
+```bash
+curl https://management-api.x.ai/v1/collections \
+  -X POST \
+  -H "Content-Type: application/json" \
+  -H "Authorization: Bearer $XAI_MANAGEMENT_API_KEY" \
+  -d '{"collection_name": "SEC Filings"}'
+```
+
+Listing collections:
+```bash
+curl https://management-api.x.ai/v1/collections \
+  -H "Authorization: Bearer $XAI_MANAGEMENT_API_KEY"
+```
+
+Viewing collection configuration:
+```bash
+'curl https://management-api.x.ai/v1/collections/collection_dbc087b1-6c99-493d-86c6-b401fee34a9d \
+  -H "Authorization: Bearer $XAI_MANAGEMENT_API_KEY"
+```
+
+Updating collection configuration
+```bash
+curl https://management-api.x.ai/v1/collections/collection_dbc087b1-6c99-493d-86c6-b401fee34a9d \
+  -X PUT \
+  -H "Content-Type: application/json" \
+  -H "Authorization: Bearer $XAI_MANAGEMENT_API_KEY" \
+  -d '{"collection_name": "SEC Filings (New)"}'
+```
+
+Uploading documents:
+
+```bash
+# Step 1: Upload file
+curl https://api.x.ai/v1/files \
+  -H "Authorization: Bearer $XAI_API_KEY" \
+  -F file=@tesla-20241231.html
+
+# Step 2: Add file to collection (use file_id from step 1)
+curl -X POST https://management-api.x.ai/v1/collections/$COLLECTION_ID/documents/$FILE_ID \
+  -H "Authorization: Bearer $XAI_MANAGEMENT_API_KEY"
+
+```
+
+Uploading with metadata fields
+
+```bash
+curl https://management-api.x.ai/v1/collections/collection_dbc087b1-6c99-493d-86c6-b401fee34a9d/documents \
+  -H "Authorization: Bearer $XAI_MANAGEMENT_API_KEY" \
+  -F "name=paper.pdf" \
+  -F "data=@paper.pdf" \
+  -F "content_type=application/pdf" \
+  -F 'fields={"author": "Sandra Kim", "year": "2024", "title": "Q3 Revenue Analysis"}'
+
+```
+
+Searching documents
+
+```bash
+curl https://api.x.ai/v1/documents/search \
+  -H "Content-Type: application/json" \
+  -H "Authorization: Bearer $XAI_API_KEY" \
+  -d '{
+    "query": "What were the key revenue drivers based on the SEC filings?",
+    "source": {
+      "collection_ids": ["collection_dbc087b1-6c99-493d-86c6-b401fee34a9d"]
+    }
+  }'
+```
+
+```bash
+curl https://api.x.ai/v1/documents/search \
+  -H "Content-Type: application/json" \
+  -H "Authorization: Bearer $XAI_API_KEY" \
+  -d '{
+      "query": "What were the key revenue drivers based on the SEC filings?",
+      "source": {
+          "collection_ids": [
+              "collection_dbc087b1-6c99-493d-86c6-b401fee34a9d"
+          ]
+      },
+      "retrieval_mode": {"type": "hybrid"}
+}'
+```
+
+
+Deleting a document:
+
+```bash
+curl https://management-api.x.ai/v1/collections/collection_dbc087b1-6c99-493d-86c6-b401fee34a9d/documents/file_55a709d4-8edc-4f83-84d9-9f04fe49f832 \
+  -X DELETE \
+  -H "Authorization: Bearer $XAI_MANAGEMENT_API_KEY"
+
+```
+
+Deleting a collection
+
+```bash
+curl https://management-api.x.ai/v1/collections/collection_dbc087b1-6c99-493d-86c6-b401fee34a9d \
+  -X DELETE \
+  -H "Authorization: Bearer $XAI_MANAGEMENT_API_KEY"
+```

package/lib/AgentClient.js

@@ -1,58 +1,82 @@
-/*
-Websocket client for AI sessions
-*/
-import { WebSocketClient } from "@j-o-r/apiserver";
+import { WebSocketClient } from '@j-o-r/apiserver';
+/**
+ * @module lib/AgentClient
+ * @exports default AgentClient
+ * Websocket client for AI agent sessions, wrapping a Prompt/ToolSet instance with robust queue-based message handling, auto-reconnect, and epoch-based resets.
+ */
 
 /**
-* @typedef {import('./API/openai.com/reponses/text.js').request} OARequest
-* @typedef {import('./API/x.ai/text.js').request} XRequest
-* @typedef {import('./API/anthropic.com/text.js').request} ANTHRequest
-* 
-* @typedef {import('./API/x.ai/text.js').XOptions} XOptions
-* @typedef {import('./API/openai.com/reponses/text.js').OAOptions} OAOptions
-* @typedef {import('./API/anthropic.com/text.js').ANTHOptions} ANTHOptions
-*
-* @typedef {import('./Prompt.js').default} Prompt
-* @typedef {import('./ToolSet.js').default} ToolSet
-*/
+ * @typedef {import('./API/openai.com/reponses/text.js').request} OARequest
+ * @typedef {import('./API/x.ai/text.js').request} XRequest
+ * @typedef {import('./API/anthropic.com/text.js').request} ANTHRequest
+ * 
+ * @typedef {import('./API/x.ai/text.js').XOptions} XOptions
+ * @typedef {import('./API/openai.com/reponses/text.js').OAOptions} OAOptions
+ * @typedef {import('./API/anthropic.com/text.js').ANTHOptions} ANTHOptions
+ *
+ * @typedef {import('./Prompt.js').default} Prompt
+ * @typedef {import('./ToolSet.js').default} ToolSet
+ */
 /**
-* @typedef {Object} WSOptions
-* @property {Prompt} prompt - The prompt session
-* @property {ToolSet} [toolset] - The toolset
-* @property {string} description - Custom introduction message.
-* @property {string} name - Logical name: e.g. search, code, os, support.
-* @property {string} secret - Websocket server access secret.
-* @property {string} [url='ws://127.0.0.1:8000/ws?params=1234']
-* @property {number} [intervalMs=250] - How often to pull one message from the queue.
-*/
+ * @typedef {Object} WSOptions
+ * @property {Prompt} prompt - The prompt session instance.
+ * @property {ToolSet} [toolset] - Optional toolset for the prompt.
+ * @property {string} description - Custom introduction message for the agent.
+ * @property {string} name - Logical name (e.g., 'search', 'code', 'os').
+ * @property {string} secret - Websocket server access secret (must match server).
+ * @property {string} [url='ws://127.0.0.1:8000/ws'] - WebSocket URL.
+ * @property {number} [intervalMs=2000] - Polling interval for queue processing (ms).
+ * @example { prompt, name: 'code-agent', secret: 'abc123', url: 'ws://localhost:8001/ws' }
+ */
+
 /**
-* Websocket Client
-* Wrapper combining a session and a websocket
-*/
+ * AgentClient: WebSocket client wrapper for AI agent sessions.
+ * Combines Prompt/ToolSet with WS communication for bidirectional queries, responses, tool calls, and resets.
+ * Features: Sequential message queue (one-at-a-time processing), auto-reconnect on close, epoch invalidation for resets.
+ * Logs events via console; emits via underlying Prompt (tool_request, tool_error, etc.).
+ * @emits tool_request - When tools are requested (via #prompt).
+ * @emits tool_error - When tools fail (via #prompt).
+ * @example
+ * const client = new AgentClient({
+ *   prompt: myPrompt,
+ *   name: 'code',
+ *   description: 'Code execution agent',
+ *   secret: 'mysecret',
+ *   intervalMs: 1000
+ * });
+ */
 class AgentClient {
-	/** @type {Prompt} */
+	/** @private @type {Prompt} */
 	#prompt;
+	/** @private @type {string} */
 	#description = '';
+	/** @private @type {string} */
 	#name;
+	/** @private @type {string} */
 	#secret = '';
-	/** @type {WebSocketClient} */
+	/** @private @type {import('@j-o-r/apiserver').WebSocketClient} */
 	#ws;
+	/** @private @type {string} */
 	#url = 'ws://127.0.0.1:8000/ws';
 
-	// Message queue and processing flag to ensure one-at-a-time handling
-	/** @type {Array<any>} */
+	/** @private @type {Array&lt;Object&gt;} Message queue for sequential processing. */
 	#queue = [];
+	/** @private @type {boolean} Flag to prevent reentrant processing. */
 	#processing = false;
-	// Bump this to invalidate in-flight work (hard reset)
+	/** @private @type {number} Epoch counter; increments on reset to invalidate in-flight work. */
 	#epoch = 0;
 
-	// Interval-based scheduler (no tight while-loop)
+	/** @private @type {NodeJS.Timeout|null} Interval ID for scheduler. */
 	#intervalId = null;
+	/** @private @type {number} Queue polling interval (ms). */
 	#intervalMs = 2000;
 
 	/**
-	 * Creates an instance of CLILoader.
-	 * @param {WSOptions} options - The options to configure the CLILoader.
+	 * Initializes the AgentClient with configuration options.
+	 * Registers Prompt events for logging and starts the WebSocket connection.
+	 * @param {WSOptions} options - Configuration object.
+	 * @throws {TypeError} If required options (e.g., prompt, secret) are invalid/missing.
+	 * @example See WSOptions.
 	 */
 	constructor(options) {
 		if (options.url) this.#url = options.url;
@@ -64,11 +88,13 @@ class AgentClient {
 		this.#registerPromptEvents();
 		this._start();
 	}
-  /**
-  * Informative.
-  * See what is happening
-  */
-	#registerPromptEvents () {
+
+	/**
+	 * @private
+	 * Registers Prompt events for logging tool requests/errors.
+	 * @returns {void}
+	 */
+	#registerPromptEvents() {
 		const events = Object.keys(this.#prompt.EVENTS);
 		events.forEach((evt) => {
 			this.#prompt.on(evt, (_msg) => {
@@ -82,72 +108,85 @@ class AgentClient {
 			});
 		});
 	}
+
+	/**
+	 * @private
+	 * Starts the WebSocket connection, sends introduction, sets up handlers.
+	 * Auto-retries on close after 5s.
+	 * @returns {void}
+	 */
 	_start() {
 		const url = this.#url + '?wssrc_id=' + this.#secret;
-		this.#ws = new WebSocketClient(url);  // Match server port/path
+		this.#ws = new WebSocketClient(url);
 		this.#ws.onopen = () => {
-			// Send my introduction to the server
 			this.#ws.send(JSON.stringify({
 				action: 'agent_introduction',
 				name: this.#name,
 				content: this.#description,
 				id: new Date().getTime()
 			}));
-			console.log('introduction_send:>>')
+			console.log('introduction_send:>>');
 			console.log(this.#name);
 			console.log(this.#description);
 		};
 		this.#ws.onmessage = (m) => {
-			// Enqueue messages; processing happens sequentially
 			this.incoming(m);
 		};
 		this.#ws.onclose = () => {
 			console.error('CLOSE: Lost Connection, retry in 5 seconds');
-			// Stop ticking while disconnected
 			this.#stopInterval();
 			setTimeout(() => {
 				this._start();
-			}, 5000)
+			}, 5000);
 		};
-		this.#ws.onerror = (e) => {
-
-		}
+		this.#ws.onerror = (e) => {};
 	}
+
 	/**
-	* Enqueue an incoming websocket message and trigger the processor.
-	*/
+	 * Enqueues an incoming WebSocket message and starts interval processing if needed.
+	 * Immediately handles 'reset' actions (clears queue, resets prompt, increments epoch).
+	 * @param {MessageEvent} e - Raw WebSocket message event.
+	 * @returns {void}
+	 * @throws {SyntaxError} If JSON parse fails.
+	 */
 	incoming(e) {
-		const message = JSON.parse(e.data);
-		// Handle resets immediately: do not enqueue, just clear the queue and reset the prompt
+		let message;
+		try {
+			message = JSON.parse(e.data);
+		} catch (err) {
+			console.error('Invalid JSON in incoming message');
+			return;
+		}
 		if (message?.action === "reset") {
 			console.log("<RESET:incoming>");
-			// Invalidate in-flight work
 			this.#epoch++;
-			// Empty the queue immediately
 			this.#queue = [];
-			// Stop the scheduler; any in-flight handler will finish, but no further items will run
 			this.#stopInterval();
-			// Reset the prompt/session state
 			this.#prompt.reset();
-			return; // Do not enqueue this message
+			return;
 		}
 		this.#queue.push(message);
-		// Start interval-based processing (no while-loop)
 		this.#startInterval();
 	}
 
 	/**
-	 * Start the interval that pulls one message at each tick.
+	 * @private
+	 * Starts the interval ticker to process one queued message per tick.
+	 * Idempotent; skips if already running.
+	 * @returns {void}
 	 */
 	#startInterval() {
 		if (this.#intervalId) return;
 		this.#intervalId = setInterval(() => {
-			// We intentionally do not await here; #processOne guards reentrancy
 			void this.#processOne();
 		}, this.#intervalMs);
 	}
 
-	/** Stop the interval if it is running. */
+	/**
+	 * @private
+	 * Stops the processing interval.
+	 * @returns {void}
+	 */
 	#stopInterval() {
 		if (this.#intervalId) {
 			clearInterval(this.#intervalId);
@@ -156,13 +195,15 @@ class AgentClient {
 	}
 
 	/**
-	 * Internal: process at most ONE message. Called on an interval tick.
+	 * @private
+	 * Processes exactly one queued message (guards reentrancy).
+	 * Stops interval if queue empty post-process.
+	 * @returns {Promise&lt;void&gt;}
 	 */
 	async #processOne() {
 		if (this.#processing) return;
 		const message = this.#queue.shift();
 		if (!message) {
-			// Nothing to do; stop ticking until a new message arrives
 			this.#stopInterval();
 			return;
 		}
@@ -172,13 +213,17 @@ class AgentClient {
 			await this.#handleMessage(message, epochAtStart);
 		} finally {
 			this.#processing = false;
-			// If the queue drained, we can stop; else the next tick will pick the next one
 			if (this.#queue.length === 0) this.#stopInterval();
 		}
 	}
 
 	/**
-	 * Internal: handle a single message
+	 * @private
+	 * Handles a single enqueued message (query/response/reset).
+	 * Sends responses/errors back via WS.
+	 * @param {Object} message - Parsed message.
+	 * @param {number} epochAtStart - Epoch at start; aborts if changed.
+	 * @returns {Promise&lt;void&gt;}
 	 */
 	async #handleMessage(message, epochAtStart) {
 		if (message.action === 'agent_query') {
@@ -193,7 +238,6 @@ class AgentClient {
 			}
 		} else if (message.action === 'reset') {
 			console.log('<RESET>');
-			// Empty the que;
 			this.#queue = [];
 			this.#epoch++;
 			this.#prompt.reset();