feat(task-20): Timmy responds to Workshop input bar with AI
## Task Task #20: Timmy responds to Workshop input bar — make the "Say something to Timmy…" input bar actually trigger an AI response shown in Timmy's speech bubble. ## What was built ### Server (artifacts/api-server/src/lib/agent.ts) - Added `chatReply(userText)` method to AgentService - Uses claude-haiku (cheaper eval model) with a wizard persona system prompt - 150-token limit so replies fit in the speech bubble - Stub mode: returns one of 4 wizard-themed canned replies after 400ms delay - Real mode: calls Anthropic with wizard persona, truncates to 250 chars ### Server (artifacts/api-server/src/routes/events.ts) - Imported agentService - Added per-visitor rate limit system: 3 replies/minute per visitorId (in-memory Map) - Added broadcastToAll() helper for broadcasting to all WS clients - Updated visitor_message handler: 1. Broadcasts visitor message to all watchers as before 2. Checks rate limit — if exceeded, sends polite "I need a moment…" reply 3. Fire-and-forget async AI call: - Broadcasts agent_state: gamma=working (crystal ball pulses) - Calls agentService.chatReply() - Broadcasts agent_state: gamma=idle - Broadcasts chat: agentId=timmy, text=reply to ALL clients - Logs world event "visitor:reply" ### Frontend (the-matrix/js/websocket.js) - Updated case 'chat' handler to differentiate message sources: - agentId === 'timmy': speech bubble + event log entry "Timmy: <text>" - agentId === 'visitor': event log only (don't hijack speech bubble) - everything else (delta/alpha/beta payment notifications): speech bubble ## What was already working (no change needed) - Enter key on input bar (ui.js already had keydown listener) - Input clearing after send (already in ui.js) - Speech bubble rendering (setSpeechBubble already existed in agents.js) - WebSocket sendVisitorMessage already exported from websocket.js ## Tests - 27/27 testkit PASS (no regressions) - TypeScript: 0 errors - Vite build: clean (the-matrix rebuilt)
This commit is contained in:
@@ -31,12 +31,43 @@ import type { Server } from "http";
|
||||
import { eventBus, type BusEvent } from "../lib/event-bus.js";
|
||||
import { makeLogger } from "../lib/logger.js";
|
||||
import { getWorldState, setAgentStateInWorld } from "../lib/world-state.js";
|
||||
import { agentService } from "../lib/agent.js";
|
||||
import { db, worldEvents } from "@workspace/db";
|
||||
|
||||
const logger = makeLogger("ws-events");
|
||||
|
||||
const PING_INTERVAL_MS = 30_000;
|
||||
|
||||
// ── Per-visitor rate limit (3 replies/minute) ─────────────────────────────────
|
||||
const CHAT_RATE_LIMIT = 3;
|
||||
const CHAT_RATE_WINDOW_MS = 60_000;
|
||||
|
||||
interface RateLimitEntry {
|
||||
count: number;
|
||||
resetAt: number;
|
||||
}
|
||||
|
||||
const visitorRateLimits = new Map<string, RateLimitEntry>();
|
||||
|
||||
function checkChatRateLimit(visitorId: string): boolean {
|
||||
const now = Date.now();
|
||||
const entry = visitorRateLimits.get(visitorId);
|
||||
if (!entry || now > entry.resetAt) {
|
||||
visitorRateLimits.set(visitorId, { count: 1, resetAt: now + CHAT_RATE_WINDOW_MS });
|
||||
return true;
|
||||
}
|
||||
if (entry.count >= CHAT_RATE_LIMIT) return false;
|
||||
entry.count++;
|
||||
return true;
|
||||
}
|
||||
|
||||
function broadcastToAll(wss: WebSocketServer, payload: object): void {
|
||||
const str = JSON.stringify(payload);
|
||||
wss.clients.forEach((c) => {
|
||||
if (c.readyState === 1) c.send(str);
|
||||
});
|
||||
}
|
||||
|
||||
function updateAgentWorld(agentId: string, state: string): void {
|
||||
try {
|
||||
setAgentStateInWorld(agentId, state);
|
||||
@@ -262,11 +293,41 @@ export function attachWebSocketServer(server: Server): void {
|
||||
}
|
||||
if (msg.type === "visitor_message" && msg.text) {
|
||||
const text = String(msg.text).slice(0, 500);
|
||||
wss.clients.forEach(c => {
|
||||
if (c.readyState === 1) {
|
||||
c.send(JSON.stringify({ type: "chat", agentId: "visitor", text }));
|
||||
|
||||
// Broadcast visitor message to all watchers
|
||||
broadcastToAll(wss, { type: "chat", agentId: "visitor", text });
|
||||
|
||||
// Rate-limit Timmy's AI replies per visitor
|
||||
const visId = String(msg.visitorId ?? ip);
|
||||
if (!checkChatRateLimit(visId)) {
|
||||
send(socket, {
|
||||
type: "chat",
|
||||
agentId: "timmy",
|
||||
text: "I need a moment to gather my thoughts… try again shortly.",
|
||||
});
|
||||
return;
|
||||
}
|
||||
|
||||
// Fire-and-forget AI reply
|
||||
void (async () => {
|
||||
try {
|
||||
// Signal that Timmy is thinking
|
||||
broadcastToAll(wss, { type: "agent_state", agentId: "gamma", state: "working" });
|
||||
updateAgentWorld("gamma", "working");
|
||||
|
||||
const reply = await agentService.chatReply(text);
|
||||
|
||||
broadcastToAll(wss, { type: "agent_state", agentId: "gamma", state: "idle" });
|
||||
updateAgentWorld("gamma", "idle");
|
||||
broadcastToAll(wss, { type: "chat", agentId: "timmy", text: reply });
|
||||
|
||||
void logWorldEvent("visitor:reply", reply.slice(0, 100), "timmy");
|
||||
} catch (err) {
|
||||
broadcastToAll(wss, { type: "agent_state", agentId: "gamma", state: "idle" });
|
||||
updateAgentWorld("gamma", "idle");
|
||||
logger.warn("chatReply failed", { err: String(err) });
|
||||
}
|
||||
});
|
||||
})();
|
||||
}
|
||||
} catch {
|
||||
/* ignore malformed messages */
|
||||
|
||||
Reference in New Issue
Block a user