@jiggai/kitchen 0.1.8 → 0.1.10-canary.0

package/src/app/tickets/[ticket]/page.tsx

@@ -1,6 +1,9 @@
 import Link from "next/link";
 import { getTicketMarkdown } from "@/lib/tickets";
 
+// Ticket detail should always reflect current stage/file; do not cache.
+export const dynamic = "force-dynamic";
+
 export default async function TicketDetailPage({
   params,
 }: {

package/src/app/tickets/page.tsx

@@ -1,6 +1,9 @@
 import { listTickets } from "@/lib/tickets";
 import { TicketsBoardClient } from "@/app/tickets/TicketsBoardClient";
 
+// Tickets reflect live filesystem state; do not cache.
+export const dynamic = "force-dynamic";
+
 export default async function TicketsPage() {
   const tickets = await listTickets();
   return <TicketsBoardClient tickets={tickets} />;

package/src/components/AppShell.tsx

@@ -1,65 +1,279 @@
 "use client";
 
 import Link from "next/link";
-import { usePathname } from "next/navigation";
+import { usePathname, useRouter } from "next/navigation";
+import { useEffect, useMemo, useState } from "react";
 import { ThemeToggle } from "@/components/ThemeToggle";
 import { ToastProvider } from "@/components/ToastProvider";
 
-function NavLink({ href, label }: { href: string; label: string }) {
-  const pathname = usePathname();
-  const active = pathname === href || (href !== "/" && pathname.startsWith(href));
+function Icon({ children }: { children: React.ReactNode }) {
+  return (
+    <span
+      className="grid size-7 place-items-center text-[color:var(--ck-text-secondary)]"
+      aria-hidden
+    >
+      {children}
+    </span>
+  );
+}
 
+function SideNavLink({
+  href,
+  label,
+  icon,
+  active,
+  collapsed,
+}: {
+  href: string;
+  label: string;
+  icon: React.ReactNode;
+  active: boolean;
+  collapsed: boolean;
+}) {
   return (
     <Link
       href={href}
+      title={label}
       className={
-        "rounded-[var(--ck-radius-sm)] px-3 py-1.5 text-sm font-medium transition-colors " +
-        (active
-          ? "bg-[var(--ck-accent-red)] text-white"
-          : "text-[color:var(--ck-text-secondary)] hover:bg-[color:var(--ck-bg-glass)] hover:text-[color:var(--ck-text-primary)]")
+        (
+          active
+            ? "flex items-center rounded-[var(--ck-radius-sm)] bg-white/10 text-sm font-medium text-[color:var(--ck-text-primary)]"
+            : "flex items-center rounded-[var(--ck-radius-sm)] text-sm font-medium text-[color:var(--ck-text-secondary)] transition-colors hover:bg-white/5 hover:text-[color:var(--ck-text-primary)]"
+        ) +
+        (collapsed ? " justify-center px-0 py-3" : " gap-4 px-4 py-3")
       }
     >
-      {label}
+      {icon}
+      {collapsed ? null : <span>{label}</span>}
     </Link>
   );
 }
 
 export function AppShell({ children }: { children: React.ReactNode }) {
+  const pathname = usePathname() || "/";
+  const router = useRouter();
+
+  const currentTeamId = useMemo(() => {
+    const m = pathname.match(/^\/teams\/([^/]+)(?:\/|$)/);
+    return m ? decodeURIComponent(m[1]) : null;
+  }, [pathname]);
+
+  const [collapsed, setCollapsed] = useState(() => {
+    try {
+      const v = localStorage.getItem("ck-leftnav-collapsed");
+      return v === "1";
+    } catch {
+      return false;
+    }
+  });
+
+  function toggleCollapsed() {
+    setCollapsed((v) => {
+      const next = !v;
+      try {
+        localStorage.setItem("ck-leftnav-collapsed", next ? "1" : "0");
+      } catch {
+        // ignore
+      }
+      return next;
+    });
+  }
+
+  // Minimal team list (for switcher). We rely on existing /api/recipes.
+  const [teamIds, setTeamIds] = useState<string[]>([]);
+  useEffect(() => {
+    (async () => {
+      try {
+        const res = await fetch("/api/agents", { cache: "no-store" });
+        const json = await res.json();
+        const agents = Array.isArray(json.agents)
+          ? (json.agents as Array<{ workspace?: string }> )
+          : [];
+
+        const s = new Set<string>();
+        for (const a of agents) {
+          const ws = String(a.workspace ?? "");
+          const parts = ws.split("/").filter(Boolean);
+          const wsPart = parts.find((p) => p.startsWith("workspace-")) ?? "";
+          if (!wsPart) continue;
+          const teamId = wsPart.slice("workspace-".length);
+          if (teamId && teamId !== "workspace") s.add(teamId);
+        }
+
+        setTeamIds(Array.from(s).sort());
+      } catch {
+        setTeamIds([]);
+      }
+    })();
+  }, []);
+
+  const globalNav = [
+    {
+      href: `/`,
+      label: "Home",
+      icon: (
+        <Icon>
+          <svg viewBox="0 0 24 24" className="h-6 w-6" fill="none" stroke="currentColor" strokeWidth="2">
+            <path d="M3 11l9-8 9 8" />
+            <path d="M5 10v10h14V10" />
+          </svg>
+        </Icon>
+      ),
+    },
+    {
+      href: `/recipes`,
+      label: "Recipes",
+      icon: (
+        <Icon>
+          <svg viewBox="0 0 24 24" className="h-6 w-6" fill="none" stroke="currentColor" strokeWidth="2">
+            <path d="M6 4h12v16H6z" />
+            <path d="M9 8h6" />
+            <path d="M9 12h6" />
+          </svg>
+        </Icon>
+      ),
+    },
+    {
+      href: `/tickets`,
+      label: "Tickets",
+      icon: (
+        <Icon>
+          <svg viewBox="0 0 24 24" className="h-6 w-6" fill="none" stroke="currentColor" strokeWidth="2">
+            <path d="M4 7h16v4a2 2 0 0 1 0 4v4H4v-4a2 2 0 0 0 0-4z" />
+          </svg>
+        </Icon>
+      ),
+    },
+    {
+      href: `/goals`,
+      label: "Goals",
+      icon: (
+        <Icon>
+          <svg viewBox="0 0 24 24" className="h-6 w-6" fill="none" stroke="currentColor" strokeWidth="2">
+            <circle cx="12" cy="12" r="9" />
+            <path d="M12 7v5l3 3" />
+          </svg>
+        </Icon>
+      ),
+    },
+    {
+      href: `/cron-jobs`,
+      label: "Cron jobs",
+      icon: (
+        <Icon>
+          <svg viewBox="0 0 24 24" className="h-6 w-6" fill="none" stroke="currentColor" strokeWidth="2">
+            <circle cx="12" cy="12" r="9" />
+            <path d="M12 7v5" />
+          </svg>
+        </Icon>
+      ),
+    },
+    {
+      href: `/settings`,
+      label: "Settings",
+      icon: (
+        <Icon>
+          <svg viewBox="0 0 24 24" className="h-6 w-6" fill="none" stroke="currentColor" strokeWidth="2">
+            <path d="M12 15.5a3.5 3.5 0 1 0 0-7 3.5 3.5 0 0 0 0 7Z" />
+            <path d="M19.4 15a7.9 7.9 0 0 0 .1-1l2-1.5-2-3.5-2.4.5a7.8 7.8 0 0 0-1.7-1L13.5 3h-4L8.6 6.5a7.8 7.8 0 0 0-1.7 1L4.5 7l-2 3.5 2 1.5a7.9 7.9 0 0 0 .1 1l-2 1.5 2 3.5 2.4-.5a7.8 7.8 0 0 0 1.7 1L10.5 21h4l.9-3.5a7.8 7.8 0 0 0 1.7-1l2.4.5 2-3.5-2-1.5Z" />
+          </svg>
+        </Icon>
+      ),
+    },
+  ];
+
+  const sideWidth = collapsed ? "w-16" : "w-72";
+
   return (
     <ToastProvider>
-      <div className="min-h-screen">
-      <header className="sticky top-0 z-50 border-b border-[color:var(--ck-border-subtle)] bg-[color:var(--ck-bg-glass)] backdrop-blur-[var(--ck-glass-blur)]">
-        <div className="mx-auto flex max-w-6xl items-center justify-between gap-4 px-4 py-3">
-          <div className="flex items-center gap-3">
-            <Link href="/" className="text-sm font-semibold tracking-tight">
-              Claw Kitchen
+      <div className="flex h-dvh w-dvw overflow-hidden">
+        <aside className={`flex shrink-0 flex-col border-r border-[color:var(--ck-border-subtle)] bg-[color:var(--ck-bg-glass)] backdrop-blur-[var(--ck-glass-blur)] ${sideWidth}`}>
+          <div className="flex h-14 items-center justify-between gap-2 border-b border-[color:var(--ck-border-subtle)] px-3">
+            <Link href="/" className="text-sm font-semibold tracking-tight" title="Home">
+              {collapsed ? "CK" : "Claw Kitchen"}
             </Link>
-            <nav className="hidden items-center gap-1 sm:flex">
-              <NavLink href="/recipes" label="Recipes" />
-              <NavLink href="/tickets" label="Tickets" />
-              <NavLink href="/goals" label="Goals" />
-              {/* Channels hidden for release */}
-              <NavLink href="/cron-jobs" label="Cron jobs" />
-              <NavLink href="/settings" label="Settings" />
-            </nav>
+            <button
+              onClick={toggleCollapsed}
+              className="rounded-[var(--ck-radius-sm)] px-2 py-1 text-sm text-[color:var(--ck-text-secondary)] hover:bg-white/5 hover:text-[color:var(--ck-text-primary)]"
+              title={collapsed ? "Expand" : "Collapse"}
+            >
+              {collapsed ? "»" : "«"}
+            </button>
           </div>
 
-          <div className="flex items-center gap-2">
+          <div className="border-b border-[color:var(--ck-border-subtle)] p-2">
+            {collapsed ? (
+              <button
+                className="w-full rounded-[var(--ck-radius-sm)] bg-white/5 px-2 py-2 text-xs font-medium text-[color:var(--ck-text-secondary)] hover:bg-white/10"
+                title={currentTeamId ? `Team: ${currentTeamId}` : "Select team"}
+                onClick={() => {
+                  if (teamIds[0]) router.push(`/teams/${encodeURIComponent(teamIds[0])}`);
+                }}
+              >
+                👥
+              </button>
+            ) : (
+              <div className="flex flex-col gap-1">
+                <div className="px-2 pt-1 text-xs font-semibold uppercase tracking-wide text-[color:var(--ck-text-tertiary)]">
+                  Team
+                </div>
+                <select
+                  value={currentTeamId ?? ""}
+                  onChange={(e) => {
+                    const id = e.target.value;
+                    if (id) router.push(`/teams/${encodeURIComponent(id)}`);
+                  }}
+                  className="w-full rounded-[var(--ck-radius-sm)] border border-white/10 bg-black/25 px-2 py-2 text-sm text-[color:var(--ck-text-primary)]"
+                >
+                  <option value="">(select)</option>
+                  {teamIds.map((id) => (
+                    <option key={id} value={id}>
+                      {id}
+                    </option>
+                  ))}
+                </select>
+              </div>
+            )}
+          </div>
+
+          <nav className="min-h-0 flex-1 overflow-auto p-2">
+            <div className={collapsed ? "mt-2 px-2 pb-2 pt-2" : "mt-2 px-2 pb-2 pt-2"}>
+              {/* section label intentionally omitted */}
+            </div>
+            {globalNav.map((it) => (
+              <SideNavLink
+                key={it.href}
+                href={it.href}
+                label={it.label}
+                icon={<span aria-hidden>{it.icon}</span>}
+                collapsed={collapsed}
+                active={pathname === it.href || pathname.startsWith(it.href + "/")}
+              />
+            ))}
+          </nav>
+
+          <div className="flex items-center justify-between gap-2 border-t border-[color:var(--ck-border-subtle)] p-2">
             <a
               href="https://github.com/JIGGAI/ClawRecipes/tree/main/docs"
               target="_blank"
               rel="noreferrer"
-              className="rounded-[var(--ck-radius-sm)] px-3 py-1.5 text-sm font-medium text-[color:var(--ck-text-secondary)] transition-colors hover:bg-[color:var(--ck-bg-glass)] hover:text-[color:var(--ck-text-primary)]"
+              className={
+                collapsed
+                  ? "mx-auto rounded-[var(--ck-radius-sm)] px-2 py-2 text-sm text-[color:var(--ck-text-secondary)] hover:bg-white/5 hover:text-[color:var(--ck-text-primary)]"
+                  : "rounded-[var(--ck-radius-sm)] px-3 py-2 text-sm font-medium text-[color:var(--ck-text-secondary)] hover:bg-white/5 hover:text-[color:var(--ck-text-primary)]"
+              }
+              title="Docs"
             >
-              Docs
+              {collapsed ? "📖" : "Docs"}
             </a>
-            <ThemeToggle />
+            {collapsed ? null : <ThemeToggle />}
           </div>
-        </div>
-      </header>
+        </aside>
 
-      <main className="mx-auto max-w-6xl px-4 py-8">{children}</main>
-    </div>
+        <div className="min-w-0 flex-1">
+          <main className="h-full overflow-auto p-4 md:p-6">{children}</main>
+        </div>
+      </div>
     </ToastProvider>
   );
 }

package/src/lib/workflows/README.md

@@ -0,0 +1,11 @@
+This folder contains the (early) file-first workflow model for ClawKitchen's Visual Workflows MVP.
+
+Source of truth lives in each team workspace:
+
+- shared-context/workflows/<id>.workflow.json
+
+API (temporary):
+
+- GET /api/teams/workflows?teamId=... (list)
+- GET /api/teams/workflows?teamId=...&id=... (read)
+- POST /api/teams/workflows { teamId, workflow } (write)

package/src/lib/workflows/runs-storage.ts

@@ -0,0 +1,71 @@
+import fs from "node:fs/promises";
+import path from "node:path";
+import { getTeamWorkspaceDir } from "@/lib/paths";
+import { assertSafeWorkflowId } from "@/lib/workflows/storage";
+import type { WorkflowRunFileV1 } from "@/lib/workflows/runs-types";
+
+const RUNS_DIR = path.join("shared-context", "workflow-runs");
+
+export function assertSafeRunId(runId: string) {
+  const id = String(runId ?? "").trim();
+  if (!id) throw new Error("run id is required");
+  if (!/^[a-z0-9][a-z0-9-]{0,80}$/.test(id)) {
+    throw new Error("Invalid run id. Use lowercase letters, numbers, and dashes (max 81 chars).");
+  }
+  return id;
+}
+
+export async function getWorkflowRunsDir(teamId: string, workflowId: string) {
+  const wfId = assertSafeWorkflowId(workflowId);
+  const teamDir = await getTeamWorkspaceDir(teamId);
+  return path.join(teamDir, RUNS_DIR, wfId);
+}
+
+export function workflowRunFileName(runId: string) {
+  return `${runId}.run.json`;
+}
+
+export async function listWorkflowRuns(teamId: string, workflowId: string) {
+  const dir = await getWorkflowRunsDir(teamId, workflowId);
+  try {
+    const entries = await fs.readdir(dir, { withFileTypes: true });
+    const files = entries
+      .filter((e) => e.isFile() && e.name.endsWith(".run.json"))
+      .map((e) => e.name)
+      .sort()
+      .reverse();
+    return { ok: true as const, dir, files };
+  } catch (err: unknown) {
+    if (err && typeof err === "object" && (err as { code?: unknown }).code === "ENOENT") {
+      return { ok: true as const, dir, files: [] as string[] };
+    }
+    throw err;
+  }
+}
+
+export async function readWorkflowRun(teamId: string, workflowId: string, runId: string) {
+  const wfId = assertSafeWorkflowId(workflowId);
+  const rId = assertSafeRunId(runId);
+  const dir = await getWorkflowRunsDir(teamId, wfId);
+  const p = path.join(dir, workflowRunFileName(rId));
+  const raw = await fs.readFile(p, "utf8");
+  const parsed = JSON.parse(raw) as WorkflowRunFileV1;
+  return { ok: true as const, path: p, run: parsed };
+}
+
+export async function writeWorkflowRun(teamId: string, workflowId: string, run: WorkflowRunFileV1) {
+  const wfId = assertSafeWorkflowId(workflowId);
+  const rId = assertSafeRunId(run.id);
+  const dir = await getWorkflowRunsDir(teamId, wfId);
+  await fs.mkdir(dir, { recursive: true });
+  const p = path.join(dir, workflowRunFileName(rId));
+  const toWrite: WorkflowRunFileV1 = {
+    ...run,
+    schema: "clawkitchen.workflow-run.v1",
+    id: rId,
+    workflowId: wfId,
+    teamId,
+  };
+  await fs.writeFile(p, JSON.stringify(toWrite, null, 2) + "\n", "utf8");
+  return { ok: true as const, path: p };
+}

package/src/lib/workflows/runs-types.ts

@@ -0,0 +1,32 @@
+export type WorkflowRunNodeResultV1 = {
+  nodeId: string;
+  status: "pending" | "running" | "waiting" | "success" | "error" | "skipped";
+  startedAt?: string;
+  endedAt?: string;
+  output?: unknown;
+  error?: { message: string; stack?: string } | string;
+};
+
+export type WorkflowRunApprovalV1 = {
+  nodeId: string;
+  state: "pending" | "approved" | "changes_requested" | "canceled";
+  requestedAt?: string;
+  decidedAt?: string;
+  /** Freeform notes from the approver (optional) */
+  note?: string;
+};
+
+export type WorkflowRunFileV1 = {
+  schema: "clawkitchen.workflow-run.v1";
+  id: string;
+  workflowId: string;
+  teamId?: string;
+  startedAt: string;
+  endedAt?: string;
+  status: "running" | "waiting_for_approval" | "success" | "error" | "canceled";
+  /** Optional high-level summary */
+  summary?: string;
+  nodes?: WorkflowRunNodeResultV1[];
+  approval?: WorkflowRunApprovalV1;
+  meta?: Record<string, unknown>;
+};

package/src/lib/workflows/storage.ts

@@ -0,0 +1,82 @@
+import fs from "node:fs/promises";
+import path from "node:path";
+import { getTeamWorkspaceDir } from "@/lib/paths";
+import type { WorkflowFileV1 } from "@/lib/workflows/types";
+
+const WORKFLOWS_DIR = path.join("shared-context", "workflows");
+
+export function workflowFileName(workflowId: string) {
+  return `${workflowId}.workflow.json`;
+}
+
+export function assertSafeWorkflowId(workflowId: string) {
+  const id = String(workflowId ?? "").trim();
+  if (!id) throw new Error("workflow id is required");
+  if (!/^[a-z0-9][a-z0-9-]{0,62}$/.test(id)) {
+    throw new Error(
+      "Invalid workflow id. Use lowercase letters, numbers, and dashes (max 63 chars), e.g. marketing-cadence-v1"
+    );
+  }
+  return id;
+}
+
+export async function getTeamWorkflowsDir(teamId: string) {
+  const teamDir = await getTeamWorkspaceDir(teamId);
+  return path.join(teamDir, WORKFLOWS_DIR);
+}
+
+export async function listWorkflows(teamId: string) {
+  const dir = await getTeamWorkflowsDir(teamId);
+  try {
+    const entries = await fs.readdir(dir, { withFileTypes: true });
+    const files = entries
+      .filter((e) => e.isFile() && e.name.endsWith(".workflow.json"))
+      .map((e) => e.name)
+      .sort();
+
+    return { ok: true as const, dir, files };
+  } catch (err: unknown) {
+    if (err && typeof err === "object" && (err as { code?: unknown }).code === "ENOENT") {
+      return { ok: true as const, dir, files: [] as string[] };
+    }
+    throw err;
+  }
+}
+
+export async function readWorkflow(teamId: string, workflowId: string) {
+  const id = assertSafeWorkflowId(workflowId);
+  const dir = await getTeamWorkflowsDir(teamId);
+  const p = path.join(dir, workflowFileName(id));
+  const raw = await fs.readFile(p, "utf8");
+  const parsed = JSON.parse(raw) as WorkflowFileV1;
+  return { ok: true as const, path: p, workflow: parsed };
+}
+
+export async function writeWorkflow(teamId: string, workflow: WorkflowFileV1) {
+  const id = assertSafeWorkflowId(workflow.id);
+  const dir = await getTeamWorkflowsDir(teamId);
+  await fs.mkdir(dir, { recursive: true });
+  const p = path.join(dir, workflowFileName(id));
+  const toWrite: WorkflowFileV1 = {
+    ...workflow,
+    schema: "clawkitchen.workflow.v1",
+    id,
+  };
+  await fs.writeFile(p, JSON.stringify(toWrite, null, 2) + "\n", "utf8");
+  return { ok: true as const, path: p };
+}
+
+export async function deleteWorkflow(teamId: string, workflowId: string) {
+  const id = assertSafeWorkflowId(workflowId);
+  const dir = await getTeamWorkflowsDir(teamId);
+  const p = path.join(dir, workflowFileName(id));
+  try {
+    await fs.unlink(p);
+  } catch (err: unknown) {
+    if (err && typeof err === "object" && (err as { code?: unknown }).code === "ENOENT") {
+      return { ok: true as const, path: p, existed: false as const };
+    }
+    throw err;
+  }
+  return { ok: true as const, path: p, existed: true as const };
+}

package/src/lib/workflows/types.ts

@@ -0,0 +1,50 @@
+export type WorkflowTriggerCronV1 = {
+  kind: "cron";
+  id: string;
+  name?: string;
+  enabled?: boolean;
+  /** Standard 5-field cron expression */
+  expr: string;
+  /** IANA timezone, e.g. America/New_York */
+  tz?: string;
+};
+
+export type WorkflowTriggerV1 = WorkflowTriggerCronV1;
+
+export type WorkflowNodeV1 = {
+  id: string;
+  type:
+    | "start"
+    | "end"
+    | "llm"
+    | "tool"
+    | "condition"
+    | "delay"
+    | "human_approval";
+  name?: string;
+  /** UI layout hints (optional) */
+  x?: number;
+  y?: number;
+  /** Node-type-specific config */
+  config?: Record<string, unknown>;
+};
+
+export type WorkflowEdgeV1 = {
+  id: string;
+  from: string;
+  to: string;
+  /** Optional edge label/condition */
+  label?: string;
+};
+
+export type WorkflowFileV1 = {
+  schema: "clawkitchen.workflow.v1";
+  id: string;
+  name: string;
+  version?: number;
+  timezone?: string;
+  triggers?: WorkflowTriggerV1[];
+  nodes: WorkflowNodeV1[];
+  edges: WorkflowEdgeV1[];
+  meta?: Record<string, unknown>;
+};

package/src/lib/workflows/validate.ts

@@ -0,0 +1,70 @@
+import type { WorkflowEdgeV1, WorkflowFileV1, WorkflowNodeV1, WorkflowTriggerV1 } from "@/lib/workflows/types";
+
+export type WorkflowValidationResult = {
+  errors: string[];
+  warnings: string[];
+};
+
+function isFiveFieldCron(expr: string) {
+  const parts = String(expr || "")
+    .trim()
+    .split(/\s+/)
+    .filter(Boolean);
+  return parts.length === 5;
+}
+
+function uniqCount(values: string[]) {
+  const set = new Set<string>();
+  for (const v of values) set.add(v);
+  return set.size;
+}
+
+export function validateWorkflowFileV1(wf: WorkflowFileV1): WorkflowValidationResult {
+  const errors: string[] = [];
+  const warnings: string[] = [];
+
+  if (wf.schema !== "clawkitchen.workflow.v1") errors.push(`schema must be clawkitchen.workflow.v1 (got ${String(wf.schema)})`);
+  if (!String(wf.id || "").trim()) errors.push("id is required");
+  if (!String(wf.name || "").trim()) errors.push("name is required");
+
+  const nodes: WorkflowNodeV1[] = Array.isArray(wf.nodes) ? wf.nodes : [];
+  const edges: WorkflowEdgeV1[] = Array.isArray(wf.edges) ? wf.edges : [];
+  const triggers: WorkflowTriggerV1[] = Array.isArray(wf.triggers) ? wf.triggers : [];
+
+  const nodeIds = nodes.map((n) => String(n?.id ?? "").trim()).filter(Boolean);
+  if (nodeIds.length !== nodes.length) errors.push("all nodes must have a non-empty id");
+  if (uniqCount(nodeIds) !== nodeIds.length) errors.push("node ids must be unique");
+
+  const edgeIds = edges.map((e) => String(e?.id ?? "").trim()).filter(Boolean);
+  if (edgeIds.length !== edges.length) errors.push("all edges must have a non-empty id");
+  if (uniqCount(edgeIds) !== edgeIds.length) errors.push("edge ids must be unique");
+
+  const nodeIdSet = new Set(nodeIds);
+  for (const e of edges) {
+    const from = String(e?.from ?? "").trim();
+    const to = String(e?.to ?? "").trim();
+    if (!from || !to) {
+      errors.push(`edge ${String(e?.id ?? "(missing id)")} must have from/to`);
+      continue;
+    }
+    if (!nodeIdSet.has(from)) errors.push(`edge ${String(e.id)} references missing from node: ${from}`);
+    if (!nodeIdSet.has(to)) errors.push(`edge ${String(e.id)} references missing to node: ${to}`);
+  }
+
+  const starts = nodes.filter((n) => n.type === "start");
+  const ends = nodes.filter((n) => n.type === "end");
+  if (!starts.length) warnings.push("no start node found");
+  if (starts.length > 1) warnings.push("multiple start nodes found (MVP supports this, but execution semantics may be ambiguous)");
+  if (!ends.length) warnings.push("no end node found");
+
+  for (const t of triggers) {
+    if (t.kind === "cron") {
+      if (!String(t.id || "").trim()) errors.push("cron trigger missing id");
+      if (!String(t.expr || "").trim()) errors.push(`cron trigger ${String(t.id || "(missing id)")} missing expr`);
+      else if (!isFiveFieldCron(t.expr)) warnings.push(`cron trigger ${String(t.id)} expr is not 5-field: ${String(t.expr)}`);
+      if (t.tz && !String(t.tz).includes("/")) warnings.push(`cron trigger ${String(t.id)} tz doesn't look like an IANA timezone: ${String(t.tz)}`);
+    }
+  }
+
+  return { errors, warnings };
+}