@tuongaz/seeflow 0.1.105 → 0.1.107

package/src/share.ts

@@ -81,13 +81,31 @@ import { type SseTap, createSseTap } from './share/sse-tap.ts';
 // re-validate join/leave with their strict required fields so malformed
 // join/leave frames are dropped rather than treated as sideband. Cursor,
 // viewport, and any other future `kind` no-op in v1 per the design doc.
-const PresenceBaseSchema = z.object({ kind: z.string() }).passthrough();
-const PresenceJoinSchema = z.object({
-  kind: z.literal('join'),
-  peerId: z.string(),
-  displayName: z.string(),
-});
-const PresenceLeaveSchema = z.object({ kind: z.literal('leave'), peerId: z.string() });
+// Presence payload shapes accepted on the wire. The cloud relay uses
+// `state: 'joined' | 'left' | 'active' | 'idle' | 'host-offline'`; older host
+// builds emitted `kind: 'join' | 'leave'`. We accept either to stay
+// forward/backward compatible.
+const PresenceBaseSchema = z
+  .object({
+    kind: z.string().optional(),
+    state: z.string().optional(),
+  })
+  .passthrough();
+const PresenceJoinSchema = z
+  .object({
+    peerId: z.string(),
+    displayName: z.string(),
+    kind: z.literal('join').optional(),
+    state: z.literal('joined').optional(),
+  })
+  .passthrough();
+const PresenceLeaveSchema = z
+  .object({
+    peerId: z.string(),
+    kind: z.literal('leave').optional(),
+    state: z.literal('left').optional(),
+  })
+  .passthrough();
 
 export interface PeerSummary {
   peerId: string;
@@ -800,15 +818,43 @@ export function createShareController(deps: ShareDeps): ShareController {
     });
   };
 
+  // The studio API receives a flat node-patch body (`name`, `description`,
+  // color tokens, `width` / `height`, …) which `mergeNodeUpdates` dispatches
+  // into `node.data.*`. The peer SPA applies the wire diff as a shallow merge
+  // against the node root, so a flat patch would land at the root and leave
+  // `data.*` stale — the renderer reads `data.name` and would silently keep
+  // the old label. Normalize the wire shape by reading the post-mutation node
+  // from the watcher's snapshot and forwarding the full merged `data` (plus
+  // `position` / `type` if they moved). Keeps the peer's shallow-merge
+  // semantics correct without changing the wire schema.
+  const NODE_ROOT_PATCH_KEYS = new Set(['position', 'type']);
+  const canonicalizePatchNode = (op: RpcOp): RpcOp => {
+    if (op.op !== 'patchNode') return op;
+    const watcher = deps.operationsDeps?.watcher;
+    const snap = watcher?.snapshot(op.flowId);
+    const node = snap?.flow?.nodes.find((n) => n.id === op.nodeId);
+    if (!node) return op;
+    const newPatch: Record<string, unknown> = {};
+    for (const key of Object.keys(op.patch)) {
+      if (NODE_ROOT_PATCH_KEYS.has(key)) {
+        newPatch[key] = (node as unknown as Record<string, unknown>)[key];
+      } else {
+        newPatch.data = node.data;
+      }
+    }
+    return { ...op, patch: newPatch };
+  };
+
   // Build + emit a `node-patched` envelope for an accepted op. Centralized so
   // peer-rpc and host-local edits assemble the wire payload identically (only
   // the `attributedTo` value differs). Returns the version assigned so callers
   // can echo it back into rpc-result if needed.
   const broadcastNodePatched = (
-    op: RpcOp,
+    rawOp: RpcOp,
     outcome: RpcDispatchOutcome,
     attributedTo: { peerId: string; displayName: string },
   ): number => {
+    const op = canonicalizePatchNode(rawOp);
     const nextVersion = (flowVersions.get(op.flowId) ?? 0) + 1;
     flowVersions.set(op.flowId, nextVersion);
     const diff = computeNodePatchedDiff(op, outcome);
@@ -986,6 +1032,50 @@ export function createShareController(deps: ShareDeps): ShareController {
     }
   };
 
+  // Emit a `flow-snapshot` envelope to a freshly-joined peer so its canvas can
+  // render the host's flow graph. Prefers the watcher's resolved snapshot
+  // (file refs inlined, style merged, schema-validated) so component / icon
+  // / image nodes render without the peer needing to walk the host's
+  // filesystem. Falls back to raw on-disk JSON for setups that boot without
+  // a watcher (CLI smoke tests, integration). Without this the peer is stuck
+  // on "CONNECTING…" forever (viewer's share-session.tsx blocks until
+  // `snapshot` is populated). Errors only warn — the peer can refresh /
+  // rotate to retry.
+  const emitFlowSnapshotForPeer = (peerConnId: string): void => {
+    const registry = deps.operationsDeps?.registry;
+    const watcher = deps.operationsDeps?.watcher;
+    if (!registry) return;
+    const entries = registry.list();
+    if (entries.length === 0) return;
+    const flows: Record<string, unknown> = {};
+    let activeFlowId: string | null = null;
+    for (const entry of entries) {
+      let flow: unknown = null;
+      const snap = watcher?.snapshot(entry.id) ?? watcher?.reparse(entry.id) ?? null;
+      if (snap?.valid && snap.flow) {
+        flow = snap.flow;
+      } else {
+        try {
+          flow = JSON.parse(readFileSync(join(entry.repoPath, entry.flowPath), 'utf8'));
+        } catch (err) {
+          console.warn(`[share] flow-snapshot read failed for ${entry.id}:`, err);
+          continue;
+        }
+      }
+      flows[entry.id] = flow;
+      if (activeFlowId === null) activeFlowId = entry.id;
+      if (entry.isDefault) activeFlowId = entry.id;
+    }
+    if (!activeFlowId) return;
+    try {
+      broadcast(
+        makeEnvelope('flow-snapshot', { flows, activeFlowId }, { to: peerConnId, from: 'host' }),
+      );
+    } catch (err) {
+      console.warn('[share] flow-snapshot broadcast failed:', err);
+    }
+  };
+
   const handleFrame = (env: Envelope) => {
     if (env.type === 'presence') {
       const base = PresenceBaseSchema.safeParse(env.payload);
@@ -993,7 +1083,15 @@ export function createShareController(deps: ShareDeps): ShareController {
         console.warn('[share] dropped presence frame: invalid payload');
         return;
       }
-      const kind = base.data.kind;
+      // Relay uses `state: 'joined'/'left'/...`; older host builds emitted
+      // `kind: 'join'/'leave'`. Map either onto our internal verbs.
+      const kind =
+        base.data.kind ??
+        (base.data.state === 'joined'
+          ? 'join'
+          : base.data.state === 'left'
+            ? 'leave'
+            : base.data.state);
       if (kind === 'join') {
         const parsed = PresenceJoinSchema.safeParse(env.payload);
         if (!parsed.success) {
@@ -1020,6 +1118,10 @@ export function createShareController(deps: ShareDeps): ShareController {
           ...current,
           peers: [...current.peers, { peerId, displayName, joinedAt: Date.now() }],
         });
+        // Prime the new peer with the host's flow graph first — without this
+        // the peer is stuck on "CONNECTING…" forever (viewer waits on
+        // snapshot before rendering the canvas).
+        emitFlowSnapshotForPeer(env.from);
         // Prime the new peer with a one-shot files-manifest snapshot so its
         // canvas can render placeholder sizing before any file-request fires.
         // Fire-and-forget; emit failures are warned, never propagated.

package/dist/web/assets/channel-CrVPaFj0.js

@@ -1 +0,0 @@
-import{U as a,C as n}from"./mermaid.core-BzxthQo1.js";const t=(r,o)=>a.lang.round(n.parse(r)[o]);export{t as c};

package/dist/web/assets/classDiagram-4FO5ZUOK-yNcOh78q.js

@@ -1 +0,0 @@
-import{s as a,a as s,c as e,C as t}from"./chunk-727SXJPM-kQNWdsxV.js";import{a as i}from"./mermaid.core-BzxthQo1.js";import"./index-COPB-kTN.js";import"./chunk-FMBD7UC4-CUar6TO-.js";import"./chunk-ND2GUHAM-CNZ3763c.js";import"./chunk-55IACEB6-5ygNV__l.js";import"./chunk-2J33WTMH-BPHmI1De.js";import"./purify.es-CLGrRn1w.js";import"./step-CWvwoXpJ.js";var b={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{b as diagram};

package/dist/web/assets/classDiagram-v2-Q7XG4LA2-yNcOh78q.js

@@ -1 +0,0 @@
-import{s as a,a as s,c as e,C as t}from"./chunk-727SXJPM-kQNWdsxV.js";import{a as i}from"./mermaid.core-BzxthQo1.js";import"./index-COPB-kTN.js";import"./chunk-FMBD7UC4-CUar6TO-.js";import"./chunk-ND2GUHAM-CNZ3763c.js";import"./chunk-55IACEB6-5ygNV__l.js";import"./chunk-2J33WTMH-BPHmI1De.js";import"./purify.es-CLGrRn1w.js";import"./step-CWvwoXpJ.js";var b={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{b as diagram};

package/dist/web/assets/stateDiagram-v2-BHNVJYJU-C30cJIVE.js

@@ -1 +0,0 @@
-import{b as r,a as e,s as a,S as s}from"./chunk-AQP2D5EJ-DzeeRd3w.js";import{a as i}from"./mermaid.core-BzxthQo1.js";import"./index-COPB-kTN.js";import"./chunk-55IACEB6-5ygNV__l.js";import"./chunk-2J33WTMH-BPHmI1De.js";import"./purify.es-CLGrRn1w.js";import"./step-CWvwoXpJ.js";var n={parser:a,get db(){return new s(2)},renderer:e,styles:r,init:i(t=>{t.state||(t.state={}),t.state.arrowMarkerAbsolute=t.arrowMarkerAbsolute},"init")};export{n as diagram};