@cueloop/daemon 0.1.0-alpha.111 → 0.1.0-alpha.112

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cueloop/daemon",
-  "version": "0.1.0-alpha.111",
+  "version": "0.1.0-alpha.112",
   "description": "The cueloop session daemon and its client: socket protocol, atomic store, resumable waits",
   "homepage": "https://github.com/mmurakaru/cueloop#readme",
   "bugs": {
@@ -44,8 +44,8 @@
     "access": "public"
   },
   "dependencies": {
-    "@cueloop/extension-api": "0.1.0-alpha.111",
-    "@cueloop/schema": "0.1.0-alpha.111",
+    "@cueloop/extension-api": "0.1.0-alpha.112",
+    "@cueloop/schema": "0.1.0-alpha.112",
     "@pierre/diffs": "^1.6.1",
     "effect": "^4.0.6",
     "smol-toml": "^1.9.1",

package/src/protocol.test.ts

@@ -0,0 +1,37 @@
+import { expect, test } from "bun:test";
+import { LineBuffer } from "./protocol";
+
+test("partial and coalesced frames keep order and leave the last frame pending", () => {
+  const buffer = new LineBuffer();
+  const lines: string[] = [];
+
+  for (const chunk of [' {"a":', '1}\n\n {"b":2}\n{"c":', "3}", "\n"])
+    buffer.push(chunk, (line) => lines.push(line));
+
+  expect(lines).toEqual(['{"a":1}', '{"b":2}', '{"c":3}']);
+});
+
+test("large fragmented frames preserve text and allow a following frame", () => {
+  const buffer = new LineBuffer();
+  const text = JSON.stringify({ text: "漢字 🌍 ".repeat(150_000) });
+  const lines: string[] = [];
+
+  for (let offset = 0; offset < text.length; offset += 4096)
+    buffer.push(text.slice(offset, offset + 4096), (line) => lines.push(line));
+
+  expect(lines).toEqual([]);
+  buffer.push('\n{"done":true}\n', (line) => lines.push(line));
+  expect(lines).toEqual([text, '{"done":true}']);
+});
+
+test("a callback can supply more frames without changing pending frame order", () => {
+  const buffer = new LineBuffer();
+  const lines: string[] = [];
+
+  buffer.push("first\nsecond\n", (line) => {
+    lines.push(line);
+    buffer.push("third\n", (next) => lines.push(next));
+  });
+
+  expect(lines).toEqual(["first", "second", "third"]);
+});

package/src/protocol.ts

@@ -53,16 +53,27 @@ export function parseInboundFrame(line: string): Frame {
 /** Incremental NDJSON splitter; tolerates partial writes. */
 export class LineBuffer {
   private buffered = "";
+  private fragments: string[] = [];
 
   push(chunk: string, onLine: (line: string) => void): void {
     this.buffered += chunk;
     for (;;) {
       const newlineIndex = this.buffered.indexOf("\n");
 
-      if (newlineIndex === -1) return;
+      if (newlineIndex === -1) {
+        if (this.buffered) this.fragments.push(this.buffered);
 
-      const line = this.buffered.slice(0, newlineIndex).trim();
+        this.buffered = "";
 
+        return;
+      }
+
+      const fragment = this.buffered.slice(0, newlineIndex);
+      const line = this.fragments.length
+        ? [...this.fragments, fragment].join("").trim()
+        : fragment.trim();
+
+      this.fragments = [];
       this.buffered = this.buffered.slice(newlineIndex + 1);
 
       if (line) onLine(line);