@jiggai/kitchen 0.3.1 → 0.3.3

package/src/app/api/__tests__/teams-workflows-route.test.ts

@@ -1,110 +0,0 @@
-import { describe, expect, it, vi, beforeEach } from "vitest";
-import { GET, POST, DELETE } from "../teams/workflows/route";
-
-vi.mock("@/lib/workflows/storage", () => ({
-  listWorkflows: vi.fn(),
-  readWorkflow: vi.fn(),
-  writeWorkflow: vi.fn(),
-  deleteWorkflow: vi.fn(),
-}));
-
-import { listWorkflows, readWorkflow, writeWorkflow, deleteWorkflow } from "@/lib/workflows/storage";
-
-describe("api teams workflows route", () => {
-  beforeEach(() => {
-    vi.mocked(listWorkflows).mockReset();
-    vi.mocked(readWorkflow).mockReset();
-    vi.mocked(writeWorkflow).mockReset();
-    vi.mocked(deleteWorkflow).mockReset();
-  });
-
-  it("GET returns 400 when teamId missing", async () => {
-    const res = await GET(new Request("https://test"));
-    expect(res.status).toBe(400);
-    const json = await res.json();
-    expect(json.error).toBe("teamId is required");
-  });
-
-  it("GET returns 200 with list when no id", async () => {
-    vi.mocked(listWorkflows).mockResolvedValue({ ok: true, dir: "/tmp", files: ["a.workflow.json"] });
-    const res = await GET(new Request("https://test?teamId=team1"));
-    expect(res.status).toBe(200);
-    const json = await res.json();
-    expect(json.ok).toBe(true);
-    expect(json.files).toEqual(["a.workflow.json"]);
-    expect(listWorkflows).toHaveBeenCalledWith("team1");
-  });
-
-  it("GET returns 200 with workflow when id provided", async () => {
-    const wf = { id: "wf1", name: "Test", nodes: [], edges: [] };
-    vi.mocked(readWorkflow).mockResolvedValue({ ok: true, path: "/home/test/wf1.workflow.json", workflow: wf });
-    const res = await GET(new Request("https://test?teamId=team1&id=wf1"));
-    expect(res.status).toBe(200);
-    const json = await res.json();
-    expect(json.ok).toBe(true);
-    expect(json.workflow).toEqual(wf);
-    expect(readWorkflow).toHaveBeenCalledWith("team1", "wf1");
-  });
-
-  it("POST returns 400 when teamId missing", async () => {
-    const res = await POST(
-      new Request("https://test", {
-        method: "POST",
-        body: JSON.stringify({ workflow: { id: "wf1", name: "Test", nodes: [], edges: [] } }),
-      })
-    );
-    expect(res.status).toBe(400);
-    const json = await res.json();
-    expect(json.error).toBe("teamId is required");
-  });
-
-  it("POST returns 400 when workflow invalid", async () => {
-    const res = await POST(
-      new Request("https://test", {
-        method: "POST",
-        body: JSON.stringify({ teamId: "team1", workflow: {} }),
-      })
-    );
-    expect(res.status).toBe(400);
-    const json = await res.json();
-    expect(json.error).toMatch(/workflow.*required/i);
-  });
-
-  it("POST returns 200 when workflow valid", async () => {
-    const wf = { id: "wf1", name: "Test", nodes: [], edges: [] };
-    vi.mocked(writeWorkflow).mockResolvedValue({ ok: true, path: "/home/test/wf1.workflow.json" });
-    const res = await POST(
-      new Request("https://test", {
-        method: "POST",
-        body: JSON.stringify({ teamId: "team1", workflow: wf }),
-      })
-    );
-    expect(res.status).toBe(200);
-    const json = await res.json();
-    expect(json.ok).toBe(true);
-    expect(writeWorkflow).toHaveBeenCalledWith("team1", expect.objectContaining({ id: "wf1", name: "Test" }));
-  });
-
-  it("DELETE returns 400 when teamId missing", async () => {
-    const res = await DELETE(new Request("https://test"));
-    expect(res.status).toBe(400);
-    const json = await res.json();
-    expect(json.error).toBe("teamId is required");
-  });
-
-  it("DELETE returns 400 when id missing", async () => {
-    const res = await DELETE(new Request("https://test?teamId=team1"));
-    expect(res.status).toBe(400);
-    const json = await res.json();
-    expect(json.error).toBe("id is required");
-  });
-
-  it("DELETE returns 200 when valid", async () => {
-    vi.mocked(deleteWorkflow).mockResolvedValue({ ok: true, path: "/home/test/wf1.workflow.json", existed: true });
-    const res = await DELETE(new Request("https://test?teamId=team1&id=wf1"));
-    expect(res.status).toBe(200);
-    const json = await res.json();
-    expect(json.ok).toBe(true);
-    expect(deleteWorkflow).toHaveBeenCalledWith("team1", "wf1");
-  });
-});

package/src/app/api/__tests__/tickets-move-route.test.ts

@@ -1,60 +0,0 @@
-import { describe, expect, it, vi, beforeEach } from "vitest";
-import { POST } from "../tickets/move/route";
-
-vi.mock("@/lib/openclaw", () => ({ runOpenClaw: vi.fn() }));
-
-import { runOpenClaw } from "@/lib/openclaw";
-
-describe("api tickets move route", () => {
-  beforeEach(() => {
-    vi.mocked(runOpenClaw).mockReset();
-  });
-
-  it("returns 400 when ticket missing", async () => {
-    const res = await POST(new Request("https://test", { method: "POST", body: JSON.stringify({}) }));
-    expect(res.status).toBe(400);
-    const json = await res.json();
-    expect(json.error).toBe("Missing ticket");
-  });
-
-  it("returns 400 when destination invalid", async () => {
-    const res = await POST(
-      new Request("https://test", {
-        method: "POST",
-        body: JSON.stringify({ ticket: "T-1", to: "invalid" }),
-      })
-    );
-    expect(res.status).toBe(400);
-    const json = await res.json();
-    expect(json.error).toBe("Invalid destination stage");
-  });
-
-  it("returns 200 when move succeeds", async () => {
-    vi.mocked(runOpenClaw).mockResolvedValue({ ok: true, exitCode: 0, stdout: "", stderr: "" });
-    const res = await POST(
-      new Request("https://test", {
-        method: "POST",
-        body: JSON.stringify({ ticket: "T-1", to: "in-progress" }),
-      })
-    );
-    expect(res.status).toBe(200);
-    const json = await res.json();
-    expect(json.ok).toBe(true);
-    expect(runOpenClaw).toHaveBeenCalledWith(
-      expect.arrayContaining(["recipes", "move-ticket", "--ticket", "T-1", "--to", "in-progress"])
-    );
-  });
-
-  it("returns 500 when exec fails", async () => {
-    vi.mocked(runOpenClaw).mockRejectedValue(new Error("Command failed"));
-    const res = await POST(
-      new Request("https://test", {
-        method: "POST",
-        body: JSON.stringify({ ticket: "T-1", to: "done" }),
-      })
-    );
-    expect(res.status).toBe(500);
-    const json = await res.json();
-    expect(json.error).toBe("Command failed");
-  });
-});

package/src/app/api/agents/[id]/route.ts

@@ -1,83 +0,0 @@
-import fs from "node:fs/promises";
-import path from "node:path";
-import { NextResponse } from "next/server";
-
-import { runOpenClaw } from "@/lib/openclaw";
-import { parseTeamRoleWorkspace } from "@/lib/agent-workspace";
-
-async function resolveAgentWorkspace(agentId: string): Promise<string | null> {
-  try {
-    const { stdout } = await runOpenClaw(["agents", "list", "--json"]);
-    const list = JSON.parse(stdout) as Array<{ id: string; workspace?: string }>;
-    const agent = list.find((a) => a.id === agentId);
-    return agent?.workspace ? String(agent.workspace) : null;
-  } catch {
-    return null;
-  }
-}
-
-async function moveTeamRoleToTrashOrRemove(info: {
-  teamDir: string;
-  role: string;
-  roleDir: string;
-}): Promise<void> {
-  const trashDir = path.join(info.teamDir, ".trash", "roles");
-  const ts = new Date().toISOString().replace(/[:.]/g, "-");
-  const dst = path.join(trashDir, `${info.role}-${ts}`);
-  try {
-    await fs.mkdir(trashDir, { recursive: true });
-    await fs.rename(info.roleDir, dst);
-  } catch {
-    try {
-      await fs.rm(info.roleDir, { recursive: true, force: true });
-    } catch {
-      // ignore
-    }
-  }
-}
-
-export async function DELETE(_req: Request, { params }: { params: Promise<{ id: string }> }) {
-  try {
-    const { id } = await params;
-    const agentId = String(id ?? "").trim();
-    if (!agentId) return NextResponse.json({ ok: false, error: "agent id is required" }, { status: 400 });
-
-    const workspace = await resolveAgentWorkspace(agentId);
-
-    const result = await runOpenClaw(["agents", "delete", agentId, "--force", "--json"]);
-    if (!result.ok) {
-      return NextResponse.json(
-        {
-          ok: false,
-          error: `Failed to delete agent: ${agentId}`,
-          stderr: result.stderr || undefined,
-          stdout: result.stdout || undefined,
-        },
-        { status: 500 },
-      );
-    }
-
-    let parsed: unknown = null;
-    try {
-      parsed = JSON.parse(result.stdout);
-    } catch {
-      parsed = null;
-    }
-
-    if (workspace) {
-      const info = parseTeamRoleWorkspace(workspace);
-      if (info.kind === "teamRole") await moveTeamRoleToTrashOrRemove(info);
-    }
-
-    return NextResponse.json({ ok: true, result: parsed ?? result.stdout });
-  } catch (err: unknown) {
-    return NextResponse.json(
-      {
-        ok: false,
-        error: "Failed to delete agent",
-        message: err instanceof Error ? err.message : String(err),
-      },
-      { status: 500 },
-    );
-  }
-}

package/src/app/api/agents/add/route.ts

@@ -1,114 +0,0 @@
-import fs from "node:fs/promises";
-import os from "node:os";
-import path from "node:path";
-
-import { NextResponse } from "next/server";
-
-import { getKitchenApi } from "@/lib/kitchen-api";
-import { teamDirFromBaseWorkspace } from "@/lib/paths";
-
-function normalizeAgentId(id: string) {
-  const s = id.trim();
-  if (!s) throw new Error("agent id is required");
-  if (!/^[a-z0-9][a-z0-9-]{0,62}$/i.test(s)) {
-    throw new Error("agent id must match /^[a-z0-9][a-z0-9-]{0,62}$/i");
-  }
-  return s;
-}
-
-function buildAgentEntry(
-  body: { newAgentId?: string; agentId?: string; name?: string; emoji?: string; theme?: string; avatar?: string; model?: string },
-  newAgentId: string,
-  newWorkspace: string
-): Record<string, unknown> {
-  const identity: Record<string, unknown> = {};
-  if (typeof body.name === "string" && body.name.trim()) identity.name = body.name.trim();
-  if (typeof body.theme === "string" && body.theme.trim()) identity.theme = body.theme.trim();
-  if (typeof body.emoji === "string" && body.emoji.trim()) identity.emoji = body.emoji.trim();
-  if (typeof body.avatar === "string" && body.avatar.trim()) identity.avatar = body.avatar.trim();
-
-  return {
-    id: newAgentId,
-    workspace: newWorkspace,
-    ...(body.model ? { model: body.model } : {}),
-    identity,
-  };
-}
-
-export async function POST(req: Request) {
-  try {
-    const body = (await req.json()) as {
-      newAgentId?: string;
-      agentId?: string;
-      name?: string;
-      emoji?: string;
-      theme?: string;
-      avatar?: string;
-      model?: string;
-      overwrite?: boolean;
-    };
-
-    const newAgentId = normalizeAgentId(String(body.newAgentId ?? body.agentId ?? ""));
-    const overwrite = Boolean(body.overwrite);
-
-    const homeDir = os.homedir();
-    if (!homeDir) {
-      return NextResponse.json({ ok: false, error: "HOME is not set" }, { status: 500 });
-    }
-
-    const configPath = path.join(homeDir, ".openclaw", "openclaw.json");
-    const raw = await fs.readFile(configPath, "utf8");
-    const cfg = JSON.parse(raw) as {
-      agents?: { defaults?: { workspace?: string }; list?: Array<Record<string, unknown>> };
-    };
-
-    const baseWorkspace = String(cfg?.agents?.defaults?.workspace ?? "").trim();
-    if (!baseWorkspace) {
-      return NextResponse.json({ ok: false, error: "agents.defaults.workspace not set" }, { status: 500 });
-    }
-
-    const newWorkspace = teamDirFromBaseWorkspace(baseWorkspace, newAgentId);
-    const agentsList: Array<Record<string, unknown>> = Array.isArray(cfg?.agents?.list) ? cfg.agents.list : [];
-    const exists = agentsList.some((a) => String(a?.id ?? "").toLowerCase() === newAgentId.toLowerCase());
-    if (exists && !overwrite) {
-      return NextResponse.json({ ok: false, error: `Agent already exists: ${newAgentId}` }, { status: 409 });
-    }
-
-    const nextEntry = buildAgentEntry(body, newAgentId, newWorkspace);
-    const nextList = exists
-      ? agentsList.map((a) => (String(a?.id ?? "").toLowerCase() === newAgentId.toLowerCase() ? nextEntry : a))
-      : [...agentsList, nextEntry];
-
-  // Ensure workspace directory exists and seed IDENTITY.md for clarity.
-  await fs.mkdir(newWorkspace, { recursive: true });
-  const identityMd = `# IDENTITY.md\n\n- **Name:** ${String(body.name ?? "").trim() || newAgentId}\n- **Creature:**\n- **Vibe:**\n- **Emoji:** ${String(body.emoji ?? "").trim()}\n- **Avatar:** ${String(body.avatar ?? "").trim()}\n`;
-  await fs.writeFile(path.join(newWorkspace, "IDENTITY.md"), identityMd, "utf8");
-
-  // Persist to ~/.openclaw/openclaw.json
-  const nextCfg = {
-    ...cfg,
-    agents: {
-      ...(cfg.agents ?? {}),
-      list: nextList,
-    },
-  };
-
-  // Write atomically.
-  const tmpPath = `${configPath}.tmp`;
-  const bakPath = `${configPath}.bak.${new Date().toISOString().replace(/[:.]/g, "-")}`;
-  await fs.writeFile(tmpPath, JSON.stringify(nextCfg, null, 2) + "\n", "utf8");
-  await fs.copyFile(configPath, bakPath).catch(() => {});
-  await fs.rename(tmpPath, configPath);
-
-  // Restart gateway so the new agent is live.
-  const api = getKitchenApi();
-  await api.runtime.system.runCommandWithTimeout(["openclaw", "gateway", "restart"], { timeoutMs: 120000 });
-
-  return NextResponse.json({ ok: true, agentId: newAgentId, workspace: newWorkspace, restarted: true });
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    // Prefer 400 for validation/input errors; otherwise 500.
-    const status = /required|match \//i.test(msg) ? 400 : 500;
-    return NextResponse.json({ ok: false, error: msg }, { status });
-  }
-}

package/src/app/api/agents/file/route.ts

@@ -1,45 +0,0 @@
-import fs from "node:fs/promises";
-import path from "node:path";
-import { NextResponse } from "next/server";
-import { errorMessage } from "@/lib/errors";
-import { assertSafeRelativeFileName } from "@/lib/paths";
-import { getAgentContextFromBody, getAgentContextFromQuery } from "@/lib/api-route-helpers";
-
-export async function GET(req: Request) {
-  const ctx = await getAgentContextFromQuery(req);
-  if (ctx instanceof NextResponse) return ctx;
-  const { agentId, ws } = ctx;
-
-  const { searchParams } = new URL(req.url);
-  const name = String(searchParams.get("name") ?? "").trim();
-  if (!name) return NextResponse.json({ ok: false, error: "name is required" }, { status: 400 });
-
-  const safe = assertSafeRelativeFileName(name);
-  const filePath = path.join(ws, safe);
-
-  try {
-    const content = await fs.readFile(filePath, "utf8");
-    return NextResponse.json({ ok: true, agentId, workspace: ws, name: safe, filePath, content });
-  } catch (e: unknown) {
-    return NextResponse.json({ ok: false, error: errorMessage(e) }, { status: 404 });
-  }
-}
-
-export async function PUT(req: Request) {
-  const body = (await req.json()) as { agentId?: string; name?: string; content?: string };
-  const name = String(body.name ?? "").trim();
-  const content = typeof body.content === "string" ? body.content : null;
-  if (!name) return NextResponse.json({ ok: false, error: "name is required" }, { status: 400 });
-  if (content === null) return NextResponse.json({ ok: false, error: "content is required" }, { status: 400 });
-
-  const ctx = await getAgentContextFromBody(body);
-  if (ctx instanceof NextResponse) return ctx;
-  const { agentId, ws } = ctx;
-
-  const safe = assertSafeRelativeFileName(name);
-  const filePath = path.join(ws, safe);
-
-  await fs.mkdir(path.dirname(filePath), { recursive: true });
-  await fs.writeFile(filePath, content, "utf8");
-  return NextResponse.json({ ok: true, agentId, workspace: ws, name: safe, filePath });
-}

package/src/app/api/agents/files/route.ts

@@ -1,23 +0,0 @@
-import { NextResponse } from "next/server";
-import { getAgentContextFromQuery, listWorkspaceFiles } from "@/lib/api-route-helpers";
-
-export async function GET(req: Request) {
-  const ctx = await getAgentContextFromQuery(req);
-  if (ctx instanceof NextResponse) return ctx;
-  const { agentId, ws } = ctx;
-
-  const candidates = [
-    { name: "SOUL.md", required: true, rationale: "Agent persona/instructions" },
-    { name: "AGENTS.md", required: true, rationale: "Agent operating rules" },
-    { name: "TOOLS.md", required: true, rationale: "Agent local notes" },
-    { name: "STATUS.md", required: false, rationale: "Optional current status snapshot" },
-    { name: "NOTES.md", required: false, rationale: "Optional scratchpad" },
-    { name: "IDENTITY.md", required: false, rationale: "Optional identity (name/emoji/avatar)" },
-    { name: "USER.md", required: false, rationale: "Optional user profile" },
-    { name: "HEARTBEAT.md", required: false, rationale: "Optional periodic checklist" },
-    { name: "MEMORY.md", required: false, rationale: "Optional curated memory" },
-  ];
-
-  const files = await listWorkspaceFiles(ws, candidates);
-  return NextResponse.json({ ok: true, agentId, workspace: ws, files });
-}

package/src/app/api/agents/identity/route.ts

@@ -1,41 +0,0 @@
-import { NextResponse } from "next/server";
-
-import { runOpenClaw } from "@/lib/openclaw";
-
-type Body = {
-  agentId: string;
-  name?: string;
-  emoji?: string;
-  theme?: string;
-  avatar?: string;
-};
-
-export async function POST(req: Request) {
-  try {
-    const body = (await req.json()) as Body;
-    const agentId = body.agentId?.trim();
-
-    if (!agentId) {
-      return NextResponse.json({ error: "agentId is required" }, { status: 400 });
-    }
-
-    const args = ["agents", "set-identity", agentId];
-
-    if (body.name?.trim()) args.push("--name", body.name.trim());
-    if (body.emoji?.trim()) args.push("--emoji", body.emoji.trim());
-    if (body.theme?.trim()) args.push("--theme", body.theme.trim());
-    if (body.avatar?.trim()) args.push("--avatar", body.avatar.trim());
-
-    const { stdout, stderr } = await runOpenClaw(args);
-
-    return NextResponse.json({ ok: true, stdout, stderr });
-  } catch (err) {
-    return NextResponse.json(
-      {
-        error: "Failed to set agent identity",
-        message: err instanceof Error ? err.message : String(err),
-      },
-      { status: 500 },
-    );
-  }
-}

package/src/app/api/agents/route.ts

@@ -1,22 +0,0 @@
-import { NextResponse } from "next/server";
-import { type AgentListItem } from "@/lib/agents";
-import { errorMessage } from "@/lib/errors";
-import { runOpenClaw } from "@/lib/openclaw";
-
-export async function GET() {
-  try {
-    const { stdout, stderr } = await runOpenClaw(["agents", "list", "--json"]);
-
-    const agents = JSON.parse(stdout) as AgentListItem[];
-
-    return NextResponse.json({ agents, stderr: stderr || undefined });
-  } catch (err) {
-    return NextResponse.json(
-      {
-        error: "Failed to list agents",
-        message: errorMessage(err),
-      },
-      { status: 500 },
-    );
-  }
-}

package/src/app/api/agents/skills/install/route.ts

@@ -1,34 +0,0 @@
-import { NextResponse } from "next/server";
-import { runOpenClaw } from "@/lib/openclaw";
-import { parseTeamRoleWorkspace } from "@/lib/agent-workspace";
-import { installSkillErrorResponse } from "@/lib/api-route-helpers";
-
-export async function POST(req: Request) {
-  const body = (await req.json()) as { agentId?: string; skill?: string };
-  const agentId = String(body.agentId ?? "").trim();
-  const skill = String(body.skill ?? "").trim();
-
-  if (!agentId) return NextResponse.json({ ok: false, error: "agentId is required" }, { status: 400 });
-  if (!skill) return NextResponse.json({ ok: false, error: "skill is required" }, { status: 400 });
-
-  // For role-based team agents, install at team scope to avoid creating a separate
-  // workspace-<agentId> directory that diverges from roles/<role>.
-  let args = ["recipes", "install-skill", skill, "--agent-id", agentId, "--yes"];
-  try {
-    const { stdout } = await runOpenClaw(["agents", "list", "--json"]);
-    const list = JSON.parse(stdout) as Array<{ id: string; workspace?: string }>;
-    const agent = list.find((a) => a.id === agentId);
-    const ws = agent?.workspace ? String(agent.workspace) : "";
-    const info = parseTeamRoleWorkspace(ws);
-    if (info.kind === "teamRole") {
-      args = ["recipes", "install-skill", skill, "--team-id", info.teamId, "--yes"];
-    }
-  } catch {
-    // If anything goes wrong, fall back to agent scope.
-  }
-
-  const res = await runOpenClaw(args);
-  if (!res.ok) return installSkillErrorResponse(args, res, { scopeArgs: args });
-
-  return NextResponse.json({ ok: true, agentId, skill, scopeArgs: args, stdout: res.stdout, stderr: res.stderr });
-}

package/src/app/api/agents/skills/route.ts

@@ -1,39 +0,0 @@
-import fs from "node:fs/promises";
-import path from "node:path";
-import { NextResponse } from "next/server";
-import { parseTeamRoleWorkspace } from "@/lib/agent-workspace";
-import { getAgentContextFromQuery } from "@/lib/api-route-helpers";
-import { errorMessage } from "@/lib/errors";
-
-export async function GET(req: Request) {
-  const ctx = await getAgentContextFromQuery(req);
-  if (ctx instanceof NextResponse) return ctx;
-  const { agentId, ws } = ctx;
-
-  const info = parseTeamRoleWorkspace(ws);
-  const dirs: string[] = [path.join(ws, "skills")];
-  if (info.kind === "teamRole") {
-    dirs.push(path.join(info.teamDir, "skills"));
-  }
-
-  const skills = new Set<string>();
-  const notes: string[] = [];
-
-  for (const skillsDir of dirs) {
-    try {
-      const entries = await fs.readdir(skillsDir, { withFileTypes: true });
-      for (const e of entries) if (e.isDirectory()) skills.add(e.name);
-    } catch (e: unknown) {
-      notes.push(`${skillsDir}: ${errorMessage(e)}`);
-    }
-  }
-
-  return NextResponse.json({
-    ok: true,
-    agentId,
-    workspace: ws,
-    skillsDirs: dirs,
-    skills: Array.from(skills).sort((a, b) => a.localeCompare(b)),
-    note: notes.length ? notes.join("; ") : undefined,
-  });
-}

package/src/app/api/agents/update/route.ts

@@ -1,52 +0,0 @@
-import { NextResponse } from "next/server";
-import { gatewayConfigGet, gatewayConfigPatch } from "@/lib/gateway";
-
-function normalizeAgentId(id: string) {
-  const s = id.trim();
-  if (!s) throw new Error("agentId is required");
-  return s;
-}
-
-export async function POST(req: Request) {
-  const body = (await req.json()) as {
-    agentId?: string;
-    patch?: {
-      workspace?: string;
-      model?: string;
-      identity?: { name?: string; theme?: string; emoji?: string; avatar?: string };
-    };
-  };
-
-  const agentId = normalizeAgentId(String(body.agentId ?? ""));
-  const patch = body.patch ?? {};
-
-  const { raw } = await gatewayConfigGet();
-  const cfg = JSON.parse(raw) as { agents?: { list?: Array<Record<string, unknown>> } };
-
-  const list = Array.isArray(cfg.agents?.list) ? (cfg.agents?.list as Array<Record<string, unknown>>) : [];
-  const idx = list.findIndex((a) => String(a.id ?? "").toLowerCase() === agentId.toLowerCase());
-  if (idx === -1) return NextResponse.json({ ok: false, error: `Agent not found in config: ${agentId}` }, { status: 404 });
-
-  const current = list[idx] ?? {};
-  const currentIdentity = (current.identity ?? {}) as Record<string, unknown>;
-
-  const next: Record<string, unknown> = {
-    ...current,
-    ...(typeof patch.workspace === "string" && patch.workspace.trim() ? { workspace: patch.workspace.trim() } : {}),
-    ...(typeof patch.model === "string" && patch.model.trim() ? { model: patch.model.trim() } : {}),
-    identity: {
-      ...currentIdentity,
-      ...(typeof patch.identity?.name === "string" ? { name: patch.identity.name } : {}),
-      ...(typeof patch.identity?.theme === "string" ? { theme: patch.identity.theme } : {}),
-      ...(typeof patch.identity?.emoji === "string" ? { emoji: patch.identity.emoji } : {}),
-      ...(typeof patch.identity?.avatar === "string" ? { avatar: patch.identity.avatar } : {}),
-    },
-  };
-
-  const nextList = list.slice();
-  nextList[idx] = next;
-
-  await gatewayConfigPatch({ agents: { list: nextList } }, `ClawKitchen: update agent ${agentId}`);
-
-  return NextResponse.json({ ok: true, agentId });
-}