@tuongaz/seeflow 0.1.83 → 0.1.85

package/src/proxy.ts

@@ -16,7 +16,7 @@ import { realpathSync } from 'node:fs';
 import { join, resolve, sep } from 'node:path';
 import type { EventBus } from './events.ts';
 import { type ProcessSpawner, type SpawnHandle, defaultProcessSpawner } from './process-spawner.ts';
-import type { PlayAction, ResetAction } from './schema.ts';
+import type { PlayAction } from './schema.ts';
 import { shortId } from './short-id.ts';
 
 export interface PlayResult {
@@ -73,31 +73,6 @@ function resolveScript(cwd: string, nodeId: string, scriptPath: string): Resolve
   return { ok: true, absPath: realTarget };
 }
 
-// Legacy anchor for resetAction (kept until resetAction gets its own design
-// round). Same realpath escape check as resolveScript, but rooted at the
-// project root rather than a per-node folder.
-function resolveResetScript(cwd: string, scriptPath: string): Resolved {
-  const projectRoot = cwd;
-  let realRoot: string;
-  try {
-    realRoot = realpathSync(projectRoot);
-  } catch {
-    return { ok: false };
-  }
-  const target = resolve(projectRoot, scriptPath);
-  let realTarget: string;
-  try {
-    realTarget = realpathSync(target);
-  } catch {
-    return { ok: false };
-  }
-  const rootWithSep = realRoot.endsWith(sep) ? realRoot : realRoot + sep;
-  if (realTarget !== realRoot && !realTarget.startsWith(rootWithSep)) {
-    return { ok: false };
-  }
-  return { ok: true, absPath: realTarget };
-}
-
 // Copy `process.env` into a string-only record, then layer the per-run extras.
 // Bun.spawn's env contract is `Record<string, string>` so the undefineds that
 // `process.env` advertises in its type must be filtered out first.
@@ -130,54 +105,6 @@ async function writeStdinPayload(handle: SpawnHandle, input: unknown): Promise<v
   }
 }
 
-// Live play-script handles indexed by flowId. Populated by runPlay() on spawn;
-// entries are removed when each handle's `exited` promise resolves (success
-// AND error paths). `stopAllPlays(flowId)` consults this map to terminate
-// every in-flight play for a demo on /reset.
-const livePlayHandles = new Map<string, Set<SpawnHandle>>();
-
-function registerLiveHandle(flowId: string, handle: SpawnHandle): void {
-  let set = livePlayHandles.get(flowId);
-  if (!set) {
-    set = new Set();
-    livePlayHandles.set(flowId, set);
-  }
-  set.add(handle);
-  handle.exited.finally(() => {
-    const current = livePlayHandles.get(flowId);
-    if (!current) return;
-    current.delete(handle);
-    if (current.size === 0) livePlayHandles.delete(flowId);
-  });
-}
-
-async function killWithGrace(handle: SpawnHandle): Promise<void> {
-  handle.kill('SIGTERM');
-  let graceTimer: ReturnType<typeof setTimeout> | undefined;
-  const gracePromise = new Promise<'grace'>((res) => {
-    graceTimer = setTimeout(() => res('grace'), SIGKILL_GRACE_MS);
-  });
-  const winner = await Promise.race([handle.exited.then(() => 'exited' as const), gracePromise]);
-  if (graceTimer) clearTimeout(graceTimer);
-  if (winner === 'grace') {
-    handle.kill('SIGKILL');
-    await handle.exited;
-  }
-}
-
-// Kill every live play-script for `flowId` (SIGTERM → 2s grace → SIGKILL in
-// parallel) and wait for each to exit. Idempotent on an unknown flowId. The
-// map is keyed by flowId so a stop on demo A never touches demo B.
-export async function stopAllPlays(flowId: string): Promise<void> {
-  const set = livePlayHandles.get(flowId);
-  if (!set || set.size === 0) return;
-  const handles = [...set];
-  // Clear eagerly so a parallel runPlay can't double-count an entry we're
-  // about to await on. The exited.finally() will no-op the second delete.
-  livePlayHandles.delete(flowId);
-  await Promise.all(handles.map((h) => killWithGrace(h)));
-}
-
 export async function runPlay(options: RunPlayOptions): Promise<PlayResult> {
   const { events, flowId, nodeId, cwd, action } = options;
   const spawner = options.spawner ?? defaultProcessSpawner;
@@ -229,8 +156,6 @@ export async function runPlay(options: RunPlayOptions): Promise<PlayResult> {
     return { runId, error: message };
   }
 
-  registerLiveHandle(flowId, handle);
-
   // Drain stdout AND stderr CONCURRENTLY with the process running so OS pipe
   // buffers (~64 KB) don't fill up and deadlock the child.
   const stdoutPromise = new Response(handle.stdout).text();
@@ -303,120 +228,3 @@ export async function runPlay(options: RunPlayOptions): Promise<PlayResult> {
   });
   return { runId, error: message };
 }
-
-export interface RunResetOptions {
-  events: EventBus;
-  flowId: string;
-  /** Project root (`<repoPath>`). Script resolves under `<cwd>/`. */
-  cwd: string;
-  action: ResetAction;
-  /** Injectable for tests; defaults to `defaultProcessSpawner`. */
-  spawner?: ProcessSpawner;
-}
-
-export interface ResetResult {
-  ok: boolean;
-  body?: unknown;
-  error?: string;
-}
-
-// Run the demo's one-shot `resetAction` script. Same spawn discipline as
-// runPlay (realpath-guarded scriptPath, concurrent stdout/stderr drain,
-// optional stdin payload, SIGTERM→2s→SIGKILL escalation on timeout) but the
-// lifecycle SSE event is the single `demo:reset` broadcast that mirrors the
-// returned shape. Callers (the /reset endpoint) decide what HTTP status to
-// surface; this returns `{ ok }` plus body/error so the endpoint can map.
-export async function runReset(options: RunResetOptions): Promise<ResetResult> {
-  const { events, flowId, cwd, action } = options;
-  const spawner = options.spawner ?? defaultProcessSpawner;
-
-  // resetAction is anchored at the project root — design defers per-node
-  // resetAction to a later round (decision #7).
-  const resolved = resolveResetScript(cwd, action.scriptPath);
-  if (!resolved.ok) {
-    events.broadcast({
-      type: 'demo:reset',
-      flowId,
-      payload: { ok: false, error: SCRIPT_PATH_ESCAPE },
-    });
-    return { ok: false, error: SCRIPT_PATH_ESCAPE };
-  }
-
-  const wantsStdin = action.input !== undefined;
-  const env = buildChildEnv({ SEEFLOW_DEMO_ID: flowId });
-
-  let handle: SpawnHandle;
-  try {
-    handle = spawner.spawn({
-      cmd: [action.interpreter, ...(action.args ?? []), resolved.absPath],
-      cwd,
-      env,
-      stdin: wantsStdin ? 'pipe' : 'ignore',
-    });
-  } catch (err) {
-    const message = err instanceof Error ? err.message : String(err);
-    events.broadcast({
-      type: 'demo:reset',
-      flowId,
-      payload: { ok: false, error: message },
-    });
-    return { ok: false, error: message };
-  }
-
-  const stdoutPromise = new Response(handle.stdout).text();
-  const stderrPromise = new Response(handle.stderr).text();
-
-  if (wantsStdin) {
-    await writeStdinPayload(handle, action.input);
-  }
-
-  const timeoutMs = action.timeoutMs ?? DEFAULT_TIMEOUT_MS;
-  let timer: ReturnType<typeof setTimeout> | undefined;
-  const timeoutPromise = new Promise<'timeout'>((res) => {
-    timer = setTimeout(() => res('timeout'), timeoutMs);
-  });
-  const exitPromise = handle.exited.then((code) => ({ code }) as const);
-
-  const race = await Promise.race([exitPromise, timeoutPromise]);
-  if (timer) clearTimeout(timer);
-
-  if (race === 'timeout') {
-    await killWithGrace(handle);
-    await Promise.allSettled([stdoutPromise, stderrPromise]);
-    const message = `reset script timed out after ${timeoutMs}ms`;
-    events.broadcast({
-      type: 'demo:reset',
-      flowId,
-      payload: { ok: false, error: message },
-    });
-    return { ok: false, error: message };
-  }
-
-  const code = race.code;
-  const [stdout, stderr] = await Promise.all([stdoutPromise, stderrPromise]);
-
-  if (code === 0) {
-    let body: unknown;
-    try {
-      body = JSON.parse(stdout);
-    } catch {
-      body = stdout;
-    }
-    events.broadcast({
-      type: 'demo:reset',
-      flowId,
-      payload: { ok: true, body },
-    });
-    return { ok: true, body };
-  }
-
-  const lastLine = lastNonEmptyLine(stderr);
-  const truncated = lastLine.slice(0, STDERR_TRUNCATE);
-  const message = truncated.length > 0 ? truncated : `reset script exited with code ${code}`;
-  events.broadcast({
-    type: 'demo:reset',
-    flowId,
-    payload: { ok: false, error: message },
-  });
-  return { ok: false, error: message };
-}

package/src/schema-catalog.ts

@@ -27,7 +27,6 @@ import {
   FlowTextNodeSchema,
   FlowUserNodeSchema,
   PlayActionSchema,
-  ResetActionSchema,
   StatusActionSchema,
   StatusReportSchema,
   StyleSchema,
@@ -63,7 +62,7 @@ const CATEGORIES: SchemaCategory[] = [
   {
     name: 'action',
     description:
-      'playAction, statusAction, resetAction, statusReport, plus componentAction (the set | script discriminated union dispatched on component-node action handles).',
+      'playAction, statusAction, statusReport, plus componentAction (the set | script discriminated union dispatched on component-node action handles).',
   },
   {
     name: 'componentSpec',
@@ -112,7 +111,6 @@ const PAYLOADS: Record<string, SchemaPayload> = {
     schemas: {
       playAction: toJsonSchema(PlayActionSchema),
       statusAction: toJsonSchema(StatusActionSchema),
-      resetAction: toJsonSchema(ResetActionSchema),
       statusReport: toJsonSchema(StatusReportSchema),
       componentAction: toJsonSchema(ComponentActionSchema),
     },

package/src/schema.ts

@@ -115,9 +115,6 @@ export const ScriptActionSchema = z.object({
 
 export const PlayActionSchema = ScriptActionSchema;
 
-// resetAction is a one-shot script with the same shape as a play script.
-export const ResetActionSchema = ScriptActionSchema;
-
 // Long-running status script. Same spawn shape as ScriptAction (interpreter +
 // args + scriptPath) but no stdin payload and a much longer max lifetime since
 // these processes tick continuously and stream StatusReports to stdout.
@@ -395,10 +392,6 @@ export const ResolvedFlowSchema = z
     description: z.string().optional(),
     nodes: z.array(NodeSchema),
     connectors: z.array(ConnectorSchema),
-    // Optional one-shot script the studio runs when the user clicks Restart.
-    // The studio kills every live play + status script for the flow BEFORE
-    // invoking this script, so the script sees no stragglers.
-    resetAction: ResetActionSchema.optional(),
   })
   .superRefine((resolved, ctx) => {
     const nodeIds = new Set(resolved.nodes.map((n) => n.id));
@@ -476,7 +469,6 @@ export type EdgePinSide = z.infer<typeof EdgePinSideSchema>;
 export type PlayAction = z.infer<typeof PlayActionSchema>;
 export type StatusAction = z.infer<typeof StatusActionSchema>;
 export type StatusReport = z.infer<typeof StatusReportSchema>;
-export type ResetAction = z.infer<typeof ResetActionSchema>;
 export type StateSource = z.infer<typeof StateSourceSchema>;
 
 // =============================================================================
@@ -648,7 +640,6 @@ export const FlowSchema = z
     version: z.literal(2),
     name: z.string().min(1),
     description: z.string().optional(),
-    resetAction: ResetActionSchema.optional(),
     nodes: z.array(FlowNodeSchema),
     connectors: z.array(FlowConnectorSchema),
   })
@@ -687,7 +678,6 @@ export const FlowEnvelopeSchema = z
     version: z.literal(2),
     name: z.string().min(1),
     description: z.string().optional(),
-    resetAction: ResetActionSchema.optional(),
     nodes: z.array(z.unknown().describe('See `seeflow schema node`')),
     connectors: z.array(z.unknown().describe('See `seeflow schema connector`')),
   })

package/src/server.ts

@@ -52,8 +52,7 @@ export interface CreateAppOptions {
    *  proxy.ts pick `defaultProcessSpawner`. Tests use this to drive runPlay
    *  with an in-memory fake spawner. */
   processSpawner?: ProcessSpawner;
-  /** Inject a ProxyFacade — tests use this to short-circuit runPlay /
-   *  runReset / stopAllPlays and assert call order. */
+  /** Inject a ProxyFacade — tests use this to short-circuit runPlay. */
   proxy?: ProxyFacade;
   /** Per-process token gating `Origin: null` requests (sandboxed MCP App
    *  iframe). Generated at studio boot; delivered to the iframe via

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

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

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

@@ -1 +0,0 @@
-import{s as a,a as s,c as e,C as t}from"./chunk-727SXJPM-DTQLQ_nx.js";import{a as i}from"./mermaid.core-Cf2CUPpR.js";import"./index-CaE1zYAX.js";import"./chunk-FMBD7UC4-Cqo28fuY.js";import"./chunk-ND2GUHAM-DcquCeO4.js";import"./chunk-55IACEB6-D_PUAOtb.js";import"./chunk-2J33WTMH-Cr7utMp9.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-DGPggFxp.js

@@ -1 +0,0 @@
-import{s as a,a as s,c as e,C as t}from"./chunk-727SXJPM-DTQLQ_nx.js";import{a as i}from"./mermaid.core-Cf2CUPpR.js";import"./index-CaE1zYAX.js";import"./chunk-FMBD7UC4-Cqo28fuY.js";import"./chunk-ND2GUHAM-DcquCeO4.js";import"./chunk-55IACEB6-D_PUAOtb.js";import"./chunk-2J33WTMH-Cr7utMp9.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};