@sickr/replay 0.9.0-beta.3 → 0.9.0

package/dist/live.js

@@ -183,16 +183,7 @@ async function sessionLoop(creds, urlid, opts) {
             tailTimer = setInterval(() => pumpNewLines(ws, offsets, opts), 500);
         });
         ws.addEventListener('message', (ev) => {
-            // Decode payload — the `ws` npm package delivers text frames as a
-            // Buffer (not a string) on the browser-style addEventListener API.
-            // Without explicit decode we'd silently treat every message as '{}'.
-            let raw = '';
-            const d = ev.data;
-            if (typeof d === 'string')
-                raw = d;
-            else if (d && typeof d.toString === 'function') {
-                raw = d.toString('utf8');
-            }
+            const raw = decodeWsPayload(ev.data);
             if (opts.verbose)
                 process.stderr.write(`sickr: ws recv (${raw.length}b): ${raw.slice(0, 120)}\n`);
             let m;
@@ -268,6 +259,33 @@ function pumpNewLines(ws, offsets, opts) {
     }
     writeOffsets(offsets);
 }
+/** Decode a WebSocket `MessageEvent.data` payload into a UTF-8 string.
+ *  The browser WebSocket gives strings for text frames; the Node `ws`
+ *  package gives Buffers; some shims give ArrayBuffer/Uint8Array. Reading
+ *  any of those as `typeof === 'string'` silently produces empty JSON,
+ *  which dropped steer messages on the CLI side in 0.9.0-beta.2 (#bug6).
+ *  Always run inbound frames through this. Exported for test coverage. */
+export function decodeWsPayload(data) {
+    if (typeof data === 'string')
+        return data;
+    if (data == null)
+        return '';
+    // Buffer / Uint8Array / ArrayBuffer all expose toString — but ArrayBuffer's
+    // default toString returns "[object ArrayBuffer]", so coerce via Uint8Array.
+    if (data instanceof ArrayBuffer)
+        return Buffer.from(new Uint8Array(data)).toString('utf8');
+    if (ArrayBuffer.isView(data))
+        return Buffer.from(data.buffer, data.byteOffset, data.byteLength).toString('utf8');
+    if (typeof data.toString === 'function') {
+        try {
+            return data.toString('utf8');
+        }
+        catch {
+            return '';
+        }
+    }
+    return '';
+}
 function isAlive(pid) {
     try {
         process.kill(pid, 0);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sickr/replay",
-  "version": "0.9.0-beta.3",
+  "version": "0.9.0",
   "type": "module",
   "description": "npx @sickr/replay — local Claude Code audit + one-click share. The free wedge into SICKR.",
   "bin": { "replay": "dist/cli.js", "sickr": "dist/cli.js" },
@@ -9,7 +9,8 @@
   "scripts": {
     "build": "tsc",
     "test": "vitest run",
-    "dev": "tsc -w"
+    "dev": "tsc -w",
+    "prepublishOnly": "npm run build && npm test && node scripts/pre-publish-check.mjs"
   },
   "engines": { "node": ">=20" },
   "license": "UNLICENSED",