@linzumi/cli 1.0.87 → 1.0.89

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@linzumi/cli",
-  "version": "1.0.87",
+  "version": "1.0.89",
   "description": "Linzumi CLI \u2014 point a Codex agent at the real code on your laptop, with your team watching and steering from shared threads.",
   "type": "module",
   "bin": {

package/scripts/compute-node-reset-e2e.mjs

@@ -14,6 +14,16 @@
 //                  so there is nothing local whose loss could fork identity).
 //   P4 rename:     `linzumi node rename` renames the node and the list
 //                  read-back shows the new name (M1.8).
+//   P5 propagation: M1.8's leg of the plan's "Rename propagation test": with
+//                  a live viewerComputeNodesUpdated subscription open (the
+//                  same subscription class an open web client holds), a
+//                  CLI-issued rename is DELIVERED as a push frame carrying
+//                  the new name - no reload, no re-poll; the assertion
+//                  consumes only what the socket pushes. Subscribes with the
+//                  persisted ComputeNodeCliRenamePropagation document first
+//                  (the only shape an enforcing server accepts on the
+//                  __absinthe__ doc path), free-form fallback when the
+//                  server's manifest predates it (dev default posture).
 //
 // REQUIREMENTS: a server running M1.1 (join adoption) + M1.3 (the
 // executeComputeNodeCommand mutation) - D29 gates the CLI merge on M1.3
@@ -29,15 +39,35 @@
 // The script never touches the real ~/.linzumi: all CLI state is scoped into
 // a temp dir via LINZUMI_CONFIG_FILE, and the "config wipe" leg wipes THAT.
 import { spawn, spawnSync } from 'node:child_process';
-import { mkdtempSync, rmSync } from 'node:fs';
+import { mkdtempSync, readFileSync, rmSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { dirname, join } from 'node:path';
 import { fileURLToPath } from 'node:url';
+import WebSocket from 'ws';
 
 const here = dirname(fileURLToPath(import.meta.url));
 const pkgRoot = dirname(here);
 const cliBin = join(pkgRoot, 'bin', 'linzumi.js');
 
+// The subscription document for the P5 propagation leg - same single-source
+// JSON the CLI transport and the server manifest generator consume, so an
+// enforcing server (prod posture) accepts it by documentId.
+const renamePropagationDocument = JSON.parse(
+  readFileSync(
+    join(pkgRoot, 'src', 'computeNodePersistedDocuments.json'),
+    'utf8'
+  )
+).documents.find(
+  (document) => document.operationName === 'ComputeNodeCliRenamePropagation'
+);
+if (renamePropagationDocument === undefined) {
+  // Loud at module load: without this, a renamed/removed document in the JSON
+  // surfaces as an opaque TypeError deep inside subscribe().
+  throw new Error(
+    'ComputeNodeCliRenamePropagation not found in src/computeNodePersistedDocuments.json - the P5 leg needs it (test/computeNodeCli.test.ts pins it)'
+  );
+}
+
 const serverUrl = process.env.LINZUMI_E2E_URL ?? 'ws://127.0.0.1:4000';
 const token = process.env.LINZUMI_E2E_TOKEN;
 // Extra args appended to `linzumi connect` (space-separated), e.g.
@@ -77,6 +107,188 @@ function runCli(args, options = {}) {
   };
 }
 
+// Async variant for legs that must keep the event loop live while the CLI
+// runs (P5 holds an open websocket whose frames a spawnSync would starve).
+function runCliAsync(args, options = {}) {
+  return new Promise((resolve, reject) => {
+    const child = spawn('node', [cliBin, ...args], {
+      env: cliEnv(),
+      stdio: ['ignore', 'pipe', 'pipe'],
+    });
+    let stdout = '';
+    let stderr = '';
+    child.stdout.on('data', (chunk) => (stdout += chunk));
+    child.stderr.on('data', (chunk) => (stderr += chunk));
+    const timer = setTimeout(() => {
+      child.kill('SIGKILL');
+      reject(new Error(`CLI timed out: linzumi ${args.join(' ')}`));
+    }, options.timeoutMs ?? 60_000);
+    child.on('error', (error) => {
+      clearTimeout(timer);
+      reject(error);
+    });
+    child.on('close', (code) => {
+      clearTimeout(timer);
+      resolve({ code, stdout, stderr });
+    });
+  });
+}
+
+// A minimal Phoenix-v2 GraphQL subscription client, standing in for the open
+// web client of the plan's Rename propagation test: connect /socket, join
+// __absinthe__:control, push the subscription "doc" (documentId first;
+// free-form fallback when the server's persisted-document store predates it),
+// then surface every subscription:data frame. No polling anywhere.
+function openRenamePropagationSubscription() {
+  const socketUrl = new URL(
+    serverUrl.replace(/^http/, 'ws').replace(/\/$/, '') + '/socket/websocket'
+  );
+  socketUrl.searchParams.set('vsn', '2.0.0');
+  socketUrl.searchParams.set('token', token);
+
+  const ws = new WebSocket(socketUrl.toString());
+  const controlTopic = '__absinthe__:control';
+  const pending = new Map(); // ref -> {resolve, reject}
+  const dataListeners = new Set();
+  // Cancel hooks for in-flight waitForNodeFrame waits, so close() can clear
+  // their timers/listeners on error paths (never an orphaned 20s timeout).
+  const activeWaitCancels = new Set();
+  let nextRef = 0;
+
+  function push(event, payload) {
+    const ref = String((nextRef += 1));
+    // Phoenix v2 JSON frames: [join_ref, ref, topic, event, payload]; the
+    // control topic's join_ref is the join push's own ref ("1").
+    ws.send(JSON.stringify(['1', ref, controlTopic, event, payload]));
+    return new Promise((resolve, reject) => {
+      pending.set(ref, { resolve, reject });
+      setTimeout(() => {
+        if (pending.delete(ref)) {
+          reject(new Error(`no phx_reply for ${event} within 15s`));
+        }
+      }, 15_000);
+    });
+  }
+
+  ws.on('message', (raw) => {
+    let frame;
+    try {
+      frame = JSON.parse(String(raw));
+    } catch {
+      return;
+    }
+    const [, ref, topic, event, payload] = frame;
+    if (event === 'phx_reply' && pending.has(ref)) {
+      const waiter = pending.get(ref);
+      pending.delete(ref);
+      waiter.resolve(payload);
+      return;
+    }
+    if (event === 'subscription:data') {
+      for (const listener of dataListeners) {
+        listener(topic, payload);
+      }
+    }
+  });
+
+  const opened = new Promise((resolve, reject) => {
+    ws.on('open', resolve);
+    ws.on('error', reject);
+  });
+
+  return {
+    async subscribe() {
+      await opened;
+      const joinReply = await push('phx_join', {});
+      if (joinReply?.status !== 'ok') {
+        throw new Error(
+          `__absinthe__:control join failed: ${JSON.stringify(joinReply)}`
+        );
+      }
+
+      // Persisted documentId first (the only shape an enforcing server
+      // accepts); free-form fallback mirrors the CLI's HTTP transport.
+      let docReply = await push('doc', {
+        documentId: renamePropagationDocument.documentId,
+        operationName: renamePropagationDocument.operationName,
+      });
+      const replyCodes = (docReply?.response?.errors ?? [])
+        .map((error) => error?.extensions?.code)
+        .filter((code) => typeof code === 'string');
+      if (
+        docReply?.status !== 'ok' &&
+        replyCodes.includes('PERSISTED_DOCUMENT_NOT_FOUND')
+      ) {
+        docReply = await push('doc', { query: renamePropagationDocument.text });
+      }
+      if (
+        docReply?.status !== 'ok' ||
+        typeof docReply?.response?.subscriptionId !== 'string'
+      ) {
+        throw new Error(
+          `viewerComputeNodesUpdated subscribe failed: ${JSON.stringify(docReply)}`
+        );
+      }
+      return docReply.response.subscriptionId;
+    },
+    // Resolves with the first PUSHED viewerComputeNodesUpdated frame matching
+    // `predicate`; other frames on the topic (e.g. adoption events) are
+    // skipped, never treated as failures. The returned promise is
+    // pre-guarded against unhandled rejection (a P5 error path abandons it
+    // before awaiting) and close() cancels its timer + listener so a failed
+    // run never leaves a live timeout keeping the process alive.
+    waitForNodeFrame(subscriptionId, predicate, timeoutMs = 20_000) {
+      const framePromise = new Promise((resolve, reject) => {
+        const timer = setTimeout(() => {
+          cancel(
+            new Error(
+              `no matching viewerComputeNodesUpdated push within ${timeoutMs}ms`
+            )
+          );
+        }, timeoutMs);
+        const listener = (topic, payload) => {
+          if (
+            topic !== subscriptionId &&
+            payload?.subscriptionId !== subscriptionId
+          ) {
+            return;
+          }
+          const node = payload?.result?.data?.viewerComputeNodesUpdated;
+          if (node !== undefined && predicate(node)) {
+            clearTimeout(timer);
+            dataListeners.delete(listener);
+            activeWaitCancels.delete(cancel);
+            resolve(node);
+          }
+        };
+        const cancel = (error) => {
+          clearTimeout(timer);
+          dataListeners.delete(listener);
+          activeWaitCancels.delete(cancel);
+          reject(error);
+        };
+        activeWaitCancels.add(cancel);
+        dataListeners.add(listener);
+      });
+      // An error before the await abandons this promise; the guard keeps the
+      // eventual timeout/cancel rejection from becoming an unhandled
+      // rejection (Node exits non-zero on those before cleanup finishes).
+      framePromise.catch(() => {});
+      return framePromise;
+    },
+    close() {
+      for (const cancel of [...activeWaitCancels]) {
+        cancel(new Error('subscription closed'));
+      }
+      try {
+        ws.close();
+      } catch {
+        // Best-effort teardown only.
+      }
+    },
+  };
+}
+
 function nodeList() {
   const result = runCli([
     'node',
@@ -238,6 +450,53 @@ try {
     renamed?.name === newName,
     JSON.stringify(renamed)
   );
+
+  // P5: rename propagation (M1.8's contribution to the plan's Rename
+  // propagation test - program plan Definitions + M1 acceptance (b)). The
+  // subscription is opened BEFORE the rename runs and the assertion consumes
+  // only pushed frames: delivery without reload or re-poll.
+  const subscription = openRenamePropagationSubscription();
+  try {
+    const subscriptionId = await subscription.subscribe();
+    const propagatedName = `e2e propagated ${Date.now()}`;
+    const framePromise = subscription.waitForNodeFrame(
+      subscriptionId,
+      (node) => node.id === successor.id && node.name === propagatedName
+    );
+
+    const propagatedRename = await runCliAsync([
+      'node',
+      'rename',
+      propagatedName,
+      '--node',
+      successor.id,
+      '--linzumi-url',
+      serverUrl,
+      '--token',
+      token,
+    ]);
+    check(
+      'P5 rename exits 0',
+      propagatedRename.code === 0,
+      propagatedRename.stderr
+    );
+
+    const frame = await framePromise;
+    check(
+      'P5 subscription push delivered the renamed node (no re-poll)',
+      frame.id === successor.id && frame.name === propagatedName,
+      JSON.stringify(frame)
+    );
+    check(
+      'P5 pushed frame carries the successor generation',
+      frame.resetGeneration === successor.resetGeneration,
+      JSON.stringify(frame)
+    );
+  } catch (error) {
+    check('P5 rename propagation', false, error.message);
+  } finally {
+    subscription.close();
+  }
 } finally {
   rmSync(stateRoot, { recursive: true, force: true });
   rmSync(workDir, { recursive: true, force: true });