@evandu/super-pi 0.1.109 → 0.1.111

package/dist/server/store.d.ts

@@ -91,6 +91,31 @@ export declare class Store {
      */
     setDefaultModel(model?: ModelSelection): Promise<void>;
     snapshot(): AppState;
+    /**
+     * Whether a conversation already holds a message with this id.
+     *
+     * A LIVE read with no clone, because its one caller is the send path's duplicate check: a resend
+     * that arrives within milliseconds of the original must be answered without paying for a full
+     * copy of every conversation in the app. The answer is a boolean and the message is not handed
+     * out, so nothing can mutate the store through it.
+     */
+    hasMessage(conversationId: string, messageId: string): boolean;
+    /**
+     * One conversation and its workspace, as DETACHED copies, without cloning the whole state.
+     *
+     * The alternative - what the run path used to do - was two `snapshot()` calls, and `snapshot()`
+     * deep-clones every conversation, every message and every tool result in the app. That is the
+     * shape of the bug it fixes: reading one conversation's model cost O(entire state), and it sat on
+     * the path between a click on 发送 and the frame that tells the browser a turn has started.
+     *
+     * The two objects are still CLONES, because every caller of this is allowed to keep them after the
+     * store has moved on (a streamed turn holds its conversation for minutes), and handing out live
+     * references would turn the first such caller into a corruption bug that only shows up under load.
+     */
+    turnContext(conversationId: string): {
+        conversation: Conversation;
+        workspace: Workspace;
+    } | undefined;
     createWorkspace(name: string, path?: string): Promise<Workspace>;
     renameWorkspace(workspaceId: string, name: string): Promise<Workspace>;
     /**

package/dist/server/store.js

@@ -684,6 +684,38 @@ export class Store {
         }
     }
     snapshot() { return structuredClone(this.state); }
+    /**
+     * Whether a conversation already holds a message with this id.
+     *
+     * A LIVE read with no clone, because its one caller is the send path's duplicate check: a resend
+     * that arrives within milliseconds of the original must be answered without paying for a full
+     * copy of every conversation in the app. The answer is a boolean and the message is not handed
+     * out, so nothing can mutate the store through it.
+     */
+    hasMessage(conversationId, messageId) {
+        return Boolean(this.state.conversations.find((item) => item.id === conversationId)?.messages.some((message) => message.id === messageId));
+    }
+    /**
+     * One conversation and its workspace, as DETACHED copies, without cloning the whole state.
+     *
+     * The alternative - what the run path used to do - was two `snapshot()` calls, and `snapshot()`
+     * deep-clones every conversation, every message and every tool result in the app. That is the
+     * shape of the bug it fixes: reading one conversation's model cost O(entire state), and it sat on
+     * the path between a click on 发送 and the frame that tells the browser a turn has started.
+     *
+     * The two objects are still CLONES, because every caller of this is allowed to keep them after the
+     * store has moved on (a streamed turn holds its conversation for minutes), and handing out live
+     * references would turn the first such caller into a corruption bug that only shows up under load.
+     */
+    turnContext(conversationId) {
+        const conversation = this.state.conversations.find((item) => item.id === conversationId);
+        if (!conversation)
+            return undefined;
+        const workspace = this.state.workspaces.find((item) => item.id === conversation.workspaceId);
+        if (!workspace)
+            return undefined;
+        return { conversation: structuredClone(conversation), workspace: structuredClone(workspace) };
+    }
     async createWorkspace(name, path) {
         const cleanName = name.trim();
         if (!cleanName || cleanName.length > 80)

package/dist/server/ws-server.js

@@ -64,6 +64,16 @@ export class WsBroadcaster {
     }
     broadcast(event) {
         const streaming = event.type === "message.delta" || event.type === "message.thinking";
+        /**
+         * SERIALIZED ONCE, NOT ONCE PER SOCKET. A `state` frame carries the whole application state, so
+         * `JSON.stringify` on it is the most expensive thing this method does - and every open tab used
+         * to pay for its own copy of the same string. The bytes are identical (no socket-specific field),
+         * so the string is built once and handed to every socket.
+         *
+         * Stream deltas stay per-event: they are coalesced per socket, so there is no single payload to
+         * share, and their cost is one small string.
+         */
+        const payload = streaming ? undefined : JSON.stringify(event);
         for (const socket of this.sockets) {
             if (socket.readyState !== WebSocket.OPEN)
                 continue;
@@ -72,7 +82,7 @@ export class WsBroadcaster {
             else {
                 // A state/tool/lifecycle event is a transcript boundary. Flush its preceding text first.
                 this.flushStreaming(socket);
-                socket.send(JSON.stringify(event));
+                socket.send(payload ?? JSON.stringify(event));
             }
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@evandu/super-pi",
-  "version": "0.1.109",
+  "version": "0.1.111",
   "description": "Web control plane and companion for Pi",
   "license": "MIT",
   "type": "module",
@@ -95,6 +95,7 @@
     "scripts/tool-register-env.test.mjs",
     "scripts/bg-task-zombie.test.mjs",
     "scripts/scoped-sidebar-tabs.test.mjs",
+    "scripts/send-retry.test.mjs",
     "scripts/git-diff-view.test.mjs",
     "scripts/fork-server.test.mjs",
     "scripts/styles-no-dangling.test.mjs",
@@ -164,6 +165,19 @@
     "scripts/knowledge-base-browser.test.mjs"
   ],
   "dependencies": {
+    "@codemirror/commands": "^6.11.1",
+    "@codemirror/lang-css": "^6.3.1",
+    "@codemirror/lang-go": "^6.0.1",
+    "@codemirror/lang-html": "^6.4.12",
+    "@codemirror/lang-javascript": "^6.2.5",
+    "@codemirror/lang-json": "^6.0.2",
+    "@codemirror/lang-markdown": "^6.5.2",
+    "@codemirror/lang-python": "^6.2.1",
+    "@codemirror/lang-rust": "^6.0.2",
+    "@codemirror/language": "^6.13.1",
+    "@codemirror/search": "^6.7.2",
+    "@codemirror/state": "^6.7.6",
+    "@codemirror/view": "^6.43.15",
     "@earendil-works/pi-coding-agent": "^0.86.1",
     "@git-diff-view/react": "^0.1.8",
     "@types/diff": "^5.2.3",