@kelceyp/caw-server 1.0.273 → 1.0.274

package/dist/embedding-worker.js

@@ -8,9 +8,17 @@
  * IPC protocol (JSON-lines over stdin/stdout):
  *   Startup:  sends  { ready: true }  after pipeline is loaded
  *             sends  { error: "..." } if startup fails
- *   Request:  parent → worker: { texts: ["...", "..."] }\n
- *   Response: worker → parent: { vectors: [[...], ...] }\n
- *                           or { error: "..." }\n
+ *   Request:  parent → worker: { id, texts: ["...", "..."], maxLength? }\n
+ *   Response: worker → parent: { id, vectors: [[...], ...] }\n
+ *                           or { id, error: "..." }\n
+ *
+ *   Every request line carries a numeric `id` allocated by the parent; every
+ *   response echoes the same `id` so the parent can pair responses by id (not
+ *   by order of arrival). Requests that are unparseable JSON, missing a
+ *   numeric id, or fail the texts-array validation are dropped rather than
+ *   answered — without a correlatable id the parent has no way to pair such
+ *   a response. In-package this path is unreachable (the provider always
+ *   sends well-formed lines); the drop is defence-in-depth for version skew.
  *
  * Vectors are raw (un-normalised) Float32Arrays serialised as plain number arrays.
  * Normalisation is performed by EmbeddingProvider after receiving the response.
@@ -81,13 +89,18 @@ const run = async () => {
         try {
             req = JSON.parse(line);
         } catch {
-            sendLine({ error: `Invalid JSON request: ${line}` });
+            // Cannot echo an id we could not parse — drop silently.
             return;
         }
 
-        const { texts, maxLength } = req;
+        const { id, texts, maxLength } = req;
+        if (typeof id !== 'number') {
+            // No correlatable id — drop silently (see IPC protocol note).
+            return;
+        }
         if (!Array.isArray(texts)) {
-            sendLine({ error: 'Request must have a texts array' });
+            // Malformed request with a valid id — drop silently rather than
+            // answer with an id-less error.
             return;
         }
 
@@ -120,9 +133,9 @@ const run = async () => {
             const embedRss = Math.round(process.memoryUsage().rss / 1024 / 1024);
             const shapeStr = maxLength !== undefined ? `[${texts.length}×${maxLength}]` : `[${texts.length}×var]`;
             logStderr('debug', `Embed #${embedCallCount}: shape=${shapeStr}, RSS: ${embedRss}MB`);
-            sendLine({ vectors });
+            sendLine({ id, vectors });
         } catch (err) {
-            sendLine({ error: `Embedding failed: ${err.message}` });
+            sendLine({ id, error: `Embedding failed: ${err.message}` });
         }
     });