@jiggai/kitchen 0.3.1 → 0.3.3

package/src/app/api/recipes/custom-team/route.ts

@@ -1,193 +0,0 @@
-import fs from "node:fs/promises";
-import path from "node:path";
-import { NextResponse } from "next/server";
-import { getWorkspaceRecipesDir } from "@/lib/paths";
-
-function isValidId(id: string) {
-  // Keep consistent with common recipe/team id expectations.
-  return /^[a-z0-9][a-z0-9_-]{1,62}$/.test(id);
-}
-
-function yamlEscape(s: string) {
-  // Minimal YAML escaping for simple strings.
-  const trimmed = s.replace(/\r\n/g, "\n").trimEnd();
-  if (!trimmed) return "\"\"";
-  // If it's a safe bare string, keep it readable.
-  if (/^[A-Za-z0-9 _.,:;()\[\]{}\-+/@]+$/.test(trimmed) && !/[:#\n]/.test(trimmed)) {
-    return trimmed;
-  }
-  return JSON.stringify(trimmed);
-}
-
-export async function POST(req: Request) {
-  const body = (await req.json()) as {
-    dryRun?: boolean;
-    recipeId?: string;
-    teamId?: string;
-    name?: string;
-    description?: string;
-    roles?: Array<{ roleId?: string; displayName?: string }>;
-  };
-
-  const dryRun = !!body.dryRun;
-  const recipeId = String(body.recipeId ?? body.teamId ?? "").trim();
-  const teamId = String(body.teamId ?? recipeId).trim();
-  const name = typeof body.name === "string" ? body.name.trim() : "";
-  const description = typeof body.description === "string" ? body.description.trim() : "";
-  const roles = Array.isArray(body.roles) ? body.roles : [];
-
-  if (!recipeId) return NextResponse.json({ ok: false, error: "recipeId is required" }, { status: 400 });
-  if (!teamId) return NextResponse.json({ ok: false, error: "teamId is required" }, { status: 400 });
-  if (!isValidId(recipeId)) {
-    return NextResponse.json(
-      { ok: false, error: "Invalid recipeId. Use lowercase letters/numbers with - or _ (2-63 chars)." },
-      { status: 400 },
-    );
-  }
-  if (!isValidId(teamId)) {
-    return NextResponse.json(
-      { ok: false, error: "Invalid teamId. Use lowercase letters/numbers with - or _ (2-63 chars)." },
-      { status: 400 },
-    );
-  }
-  if (!teamId.endsWith("-team")) {
-    return NextResponse.json(
-      { ok: false, error: "teamId must end with -team" },
-      { status: 400 },
-    );
-  }
-
-  if (roles.length < 1) {
-    return NextResponse.json({ ok: false, error: "Select at least one role/agent" }, { status: 400 });
-  }
-
-  const normalizedRoles = roles
-    .map((r) => ({
-      roleId: String(r.roleId ?? "").trim(),
-      displayName: typeof r.displayName === "string" ? r.displayName.trim() : "",
-    }))
-    .filter((r) => r.roleId);
-
-  if (normalizedRoles.length < 1) {
-    return NextResponse.json({ ok: false, error: "Each selected agent must have a roleId" }, { status: 400 });
-  }
-
-  const roleSet = new Set<string>();
-  for (const r of normalizedRoles) {
-    if (!isValidId(r.roleId)) {
-      return NextResponse.json(
-        { ok: false, error: `Invalid roleId: ${r.roleId}. Use lowercase letters/numbers with - or _.` },
-        { status: 400 },
-      );
-    }
-    if (roleSet.has(r.roleId)) {
-      return NextResponse.json({ ok: false, error: `Duplicate roleId: ${r.roleId}` }, { status: 400 });
-    }
-    roleSet.add(r.roleId);
-  }
-
-  const dir = await getWorkspaceRecipesDir();
-  const filePath = path.join(dir, `${recipeId}.md`);
-
-  try {
-    await fs.access(filePath);
-    return NextResponse.json({ ok: false, error: `Recipe already exists: ${recipeId}` }, { status: 409 });
-  } catch {
-    // ok
-  }
-
-  const lines: string[] = [];
-  lines.push("---");
-  lines.push(`id: ${recipeId}`);
-  lines.push(`name: ${yamlEscape(name || recipeId)}`);
-  lines.push(`version: 0.1.0`);
-  if (description) lines.push(`description: ${yamlEscape(description)}`);
-  lines.push("kind: team");
-  lines.push("requiredSkills: []");
-  lines.push("team:");
-  lines.push(`  teamId: ${teamId}`);
-  lines.push("agents:");
-
-  for (const r of normalizedRoles) {
-    lines.push(`  - role: ${r.roleId}`);
-    if (r.displayName) lines.push(`    name: ${yamlEscape(r.displayName)}`);
-    lines.push("    tools:");
-    lines.push("      profile: coding");
-    lines.push("      allow:");
-    lines.push("        - group:fs");
-    lines.push("        - group:web");
-    lines.push("        - group:runtime");
-    lines.push("      deny:");
-    lines.push("        - exec");
-  }
-
-  lines.push("templates:");
-
-  for (const r of normalizedRoles) {
-    const roleId = r.roleId;
-
-    lines.push(`  ${roleId}.soul: |`);
-    lines.push(`    # SOUL.md`);
-    lines.push(`    `);
-    lines.push(`    Role: ${roleId}`);
-    lines.push(`    `);
-
-    lines.push(`  ${roleId}.agents: |`);
-    lines.push(`    # AGENTS.md`);
-    lines.push(`    `);
-    lines.push(`    You are the ${roleId} role in team ${teamId}.`);
-    lines.push(`    `);
-
-    lines.push(`  ${roleId}.tools: |`);
-    lines.push(`    # TOOLS.md`);
-    lines.push(`    `);
-    lines.push(`    - (empty)`);
-    lines.push(`    `);
-
-    lines.push(`  ${roleId}.status: |`);
-    lines.push(`    # STATUS.md`);
-    lines.push(`    `);
-    lines.push(`    - (empty)`);
-    lines.push(`    `);
-
-    lines.push(`  ${roleId}.notes: |`);
-    lines.push(`    # NOTES.md`);
-    lines.push(`    `);
-    lines.push(`    - (empty)`);
-    lines.push("");
-  }
-
-  lines.push("files:");
-  lines.push("  - path: SOUL.md");
-  lines.push("    template: soul");
-  lines.push("    mode: createOnly");
-  lines.push("  - path: AGENTS.md");
-  lines.push("    template: agents");
-  lines.push("    mode: createOnly");
-  lines.push("  - path: TOOLS.md");
-  lines.push("    template: tools");
-  lines.push("    mode: createOnly");
-  lines.push("  - path: STATUS.md");
-  lines.push("    template: status");
-  lines.push("    mode: createOnly");
-  lines.push("  - path: NOTES.md");
-  lines.push("    template: notes");
-  lines.push("    mode: createOnly");
-
-  lines.push("---");
-  lines.push("");
-  lines.push("# Custom team recipe");
-  lines.push("");
-  lines.push("Generated by ClawKitchen Custom Team Builder.");
-  lines.push("");
-
-  const md = lines.join("\n");
-
-  if (dryRun) {
-    return NextResponse.json({ ok: true, dryRun: true, recipeId, teamId, filePath, md });
-  }
-
-  await fs.writeFile(filePath, md, "utf8");
-
-  return NextResponse.json({ ok: true, recipeId, teamId, filePath });
-}

package/src/app/api/recipes/delete/helpers.ts

@@ -1,65 +0,0 @@
-import fs from "node:fs/promises";
-import path from "node:path";
-import { runOpenClaw } from "@/lib/openclaw";
-
-export async function getAttachedTeams(workspaceRoot: string, recipeId: string): Promise<string[]> {
-  const attachedTeams: string[] = [];
-  const teamsRoot = path.resolve(workspaceRoot, "..");
-
-  try {
-    const entries = await fs.readdir(teamsRoot, { withFileTypes: true });
-    const workspaceDirs = entries.filter((e) => e.isDirectory() && e.name.startsWith("workspace-"));
-
-    for (const dirent of workspaceDirs) {
-      const metaPath = path.join(teamsRoot, dirent.name, "team.json");
-      try {
-        const raw = await fs.readFile(metaPath, "utf8");
-        const meta = JSON.parse(raw) as { recipeId?: unknown; teamId?: unknown };
-        if (String(meta.recipeId ?? "").trim() === recipeId) {
-          attachedTeams.push(
-            String(meta.teamId ?? dirent.name.replace(/^workspace-/, "")).trim() || dirent.name
-          );
-        }
-      } catch {
-        // ignore
-      }
-    }
-  } catch {
-    // ignore
-  }
-
-  return attachedTeams;
-}
-
-export async function getAttachedAgents(
-  workspaceRoot: string,
-  recipeId: string
-): Promise<{ attachedAgents: string[]; hasSameIdAgent: boolean }> {
-  const attachedAgents: string[] = [];
-  let hasSameIdAgent = false;
-
-  const agentsRes = await runOpenClaw(["agents", "list", "--json"]);
-  if (!agentsRes.ok) return { attachedAgents, hasSameIdAgent };
-
-  try {
-    const agents = JSON.parse(agentsRes.stdout) as Array<{ id?: unknown }>;
-    hasSameIdAgent = agents.some((a) => String(a.id ?? "").trim() === recipeId);
-
-    for (const a of agents) {
-      const agentId = String(a.id ?? "").trim();
-      if (!agentId) continue;
-      const metaPath = path.join(workspaceRoot, "agents", agentId, "agent.json");
-      try {
-        const raw = await fs.readFile(metaPath, "utf8");
-        const meta = JSON.parse(raw) as { recipeId?: unknown };
-        if (String(meta.recipeId ?? "").trim() === recipeId) attachedAgents.push(agentId);
-      } catch {
-        // ignore
-      }
-    }
-  } catch {
-    // ignore
-  }
-
-  return { attachedAgents, hasSameIdAgent };
-}

package/src/app/api/recipes/delete/route.ts

@@ -1,73 +0,0 @@
-import { NextResponse } from "next/server";
-import path from "node:path";
-import { runOpenClaw } from "@/lib/openclaw";
-import { findRecipeById, resolveRecipePath } from "@/lib/recipes";
-import fs from "node:fs/promises";
-import { getAttachedTeams, getAttachedAgents } from "./helpers";
-
-export async function POST(req: Request) {
-  const body = (await req.json()) as { id?: string };
-  const id = String(body.id ?? "").trim();
-  if (!id) return NextResponse.json({ ok: false, error: "id is required" }, { status: 400 });
-
-  const item = await findRecipeById(id);
-  if (!item) return NextResponse.json({ ok: false, error: `Recipe not found: ${id}` }, { status: 404 });
-  if (item.source === "builtin") {
-    return NextResponse.json({ ok: false, error: `Recipe ${id} is builtin and cannot be deleted` }, { status: 403 });
-  }
-
-  const workspaceRoot = (await runOpenClaw(["config", "get", "agents.defaults.workspace"]))?.stdout?.trim();
-  if (!workspaceRoot) {
-    return NextResponse.json({ ok: false, error: "agents.defaults.workspace not set" }, { status: 500 });
-  }
-  const allowedDir = path.resolve(workspaceRoot, "recipes") + path.sep;
-
-  const filePath = await resolveRecipePath(item);
-  const resolved = path.resolve(filePath);
-  if (!resolved.startsWith(allowedDir)) {
-    return NextResponse.json({ ok: false, error: `Refusing to delete non-workspace recipe path: ${resolved}` }, { status: 403 });
-  }
-
-  const kind = (item.kind ?? "team") as "team" | "agent";
-
-  if (kind === "team") {
-    const attachedTeams = await getAttachedTeams(workspaceRoot, id);
-    if (attachedTeams.length) {
-      return NextResponse.json(
-        {
-          ok: false,
-          error: `Team ${id} is in use by installed team(s): ${attachedTeams.join(", ")}. Remove the team(s) first, then delete the recipe. If no team is shown, you still have a .openclaw/workspace-${id} folder. Please remove the folder to delete this recipe.`,
-          details: { attachedTeams },
-        },
-        { status: 409 }
-      );
-    }
-  }
-
-  if (kind === "agent") {
-    const { attachedAgents, hasSameIdAgent } = await getAttachedAgents(workspaceRoot, id);
-    if (hasSameIdAgent) {
-      return NextResponse.json(
-        {
-          ok: false,
-          error: `Agent recipe ${id} cannot be deleted because an active agent exists with the same id: ${id}. Delete the agent first, then delete the recipe.`,
-          details: { agentId: id },
-        },
-        { status: 409 }
-      );
-    }
-    if (attachedAgents.length) {
-      return NextResponse.json(
-        {
-          ok: false,
-          error: `Agent recipe ${id} is in use by active agent(s): ${attachedAgents.join(", ")}. Delete the agent(s) first, then delete the recipe.`,
-          details: { attachedAgents },
-        },
-        { status: 409 }
-      );
-    }
-  }
-
-  await fs.rm(resolved, { force: true });
-  return NextResponse.json({ ok: true, deleted: resolved });
-}

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

@@ -1,21 +0,0 @@
-import { NextResponse } from "next/server";
-import { runOpenClaw } from "@/lib/openclaw";
-
-export async function GET() {
-  const { stdout, stderr } = await runOpenClaw(["recipes", "list"]);
-  if (stderr.trim()) {
-    // non-fatal warnings go to stderr sometimes; still try to parse stdout.
-  }
-
-  let data: unknown;
-  try {
-    data = JSON.parse(stdout);
-  } catch {
-    return NextResponse.json({ error: "Failed to parse openclaw recipes list output", stderr, stdout }, { status: 500 });
-  }
-
-  // NOTE: We intentionally return the full list here. openclaw can return both builtin + workspace
-  // entries for the same (kind,id). The UI can decide which to prefer depending on context.
-  const list = Array.isArray(data) ? data : [];
-  return NextResponse.json({ recipes: list, stderr });
-}

package/src/app/api/recipes/team-agents/__tests__/helpers.test.ts

@@ -1,156 +0,0 @@
-import { describe, expect, it, vi, beforeEach } from "vitest";
-import {
-  parseRecipeFrontmatter,
-  buildNextMarkdown,
-  handleRemove,
-  handleAdd,
-  handleAddLike,
-} from "../helpers";
-
-vi.mock("@/lib/openclaw", () => ({
-  runOpenClaw: vi.fn(),
-}));
-
-import { runOpenClaw } from "@/lib/openclaw";
-
-describe("recipes team-agents helpers", () => {
-  describe("parseRecipeFrontmatter", () => {
-    it("parses valid YAML with agents and templates", () => {
-      const yaml = `
-kind: team
-agents:
-  - role: lead
-    name: Lead
-templates:
-  lead.prompt: "# Prompt"
-`;
-      const { fm, agents, templates } = parseRecipeFrontmatter(yaml);
-      expect(fm.kind).toBe("team");
-      expect(agents).toHaveLength(1);
-      expect(agents[0]).toEqual({ role: "lead", name: "Lead" });
-      expect(templates).toEqual({ "lead.prompt": "# Prompt" });
-    });
-
-    it("handles empty YAML", () => {
-      const { fm, agents, templates } = parseRecipeFrontmatter("");
-      expect(fm).toEqual({});
-      expect(agents).toEqual([]);
-      expect(templates).toEqual({});
-    });
-
-    it("handles non-array agents", () => {
-      const { agents } = parseRecipeFrontmatter("agents: not-an-array");
-      expect(agents).toEqual([]);
-    });
-
-    it("handles non-object templates", () => {
-      const { templates } = parseRecipeFrontmatter("templates: []");
-      expect(templates).toEqual({});
-    });
-  });
-
-  describe("buildNextMarkdown", () => {
-    it("builds markdown with updated agents and templates", () => {
-      const fm = { kind: "team" };
-      const nextAgents = [{ role: "lead", name: "Lead" }];
-      const nextTemplates = { "lead.prompt": "content" };
-      const rest = "# Body";
-      const result = buildNextMarkdown(fm, nextAgents, nextTemplates, rest);
-      expect(result).toContain("---");
-      expect(result).toContain("kind: team");
-      expect(result).toContain("role: lead");
-      expect(result).toContain("# Body");
-    });
-
-    it("omits templates when empty", () => {
-      const fm = { kind: "team" };
-      const nextAgents: Array<Record<string, unknown>> = [];
-      const nextTemplates: Record<string, unknown> = {};
-      const result = buildNextMarkdown(fm, nextAgents, nextTemplates, "body");
-      expect(result).not.toContain("templates:");
-    });
-  });
-
-  describe("handleRemove", () => {
-    it("removes agent and role-prefixed templates", () => {
-      const agents = [
-        { role: "lead", name: "Lead" },
-        { role: "qa", name: "QA" },
-      ];
-      const templates = {
-        "lead.prompt": "x",
-        "qa.prompt": "y",
-        "other.stuff": "z",
-      };
-      const result = handleRemove(agents, templates, "lead");
-      expect(result.nextAgents).toHaveLength(1);
-      expect(result.nextAgents[0].role).toBe("qa");
-      expect(result.nextTemplates).toEqual({ "qa.prompt": "y", "other.stuff": "z" });
-      expect(result.addedRole).toBeNull();
-    });
-
-    it("returns addedRole null", () => {
-      const result = handleRemove([{ role: "x" }], {}, "x");
-      expect(result.addedRole).toBeNull();
-    });
-  });
-
-  describe("handleAdd", () => {
-    it("adds new agent when role not present", () => {
-      const agents: Array<Record<string, unknown>> = [];
-      const templates = {};
-      const result = handleAdd(agents, templates, "qa", "QA Lead");
-      expect(result.nextAgents).toHaveLength(1);
-      expect(result.nextAgents[0]).toEqual({ role: "qa", name: "QA Lead" });
-      expect(result.addedRole).toBe("qa");
-    });
-
-    it("updates existing agent when role present", () => {
-      const agents = [{ role: "qa", name: "Old" }];
-      const templates = {};
-      const result = handleAdd(agents, templates, "qa", "New Name");
-      expect(result.nextAgents).toHaveLength(1);
-      expect(result.nextAgents[0]).toEqual({ role: "qa", name: "New Name" });
-      expect(result.addedRole).toBe("qa");
-    });
-
-    it("copies templates without mutating", () => {
-      const agents: Array<Record<string, unknown>> = [];
-      const templates = { x: "y" };
-      const result = handleAdd(agents, templates, "qa", "");
-      expect(result.nextTemplates).toEqual({ x: "y" });
-    });
-  });
-
-  describe("handleAddLike", () => {
-    beforeEach(() => {
-      vi.mocked(runOpenClaw).mockResolvedValue({
-        ok: true,
-        stdout: JSON.stringify([{ id: "team1-lead" }]),
-        stderr: "",
-        exitCode: 0,
-      });
-    });
-
-    it("returns 400 when baseRole not found", async () => {
-      const agents: Array<Record<string, unknown>> = [];
-      const result = await handleAddLike(agents, {}, "lead", "New", "team1");
-      expect(result).toBeInstanceOf(Response);
-      const res = result as Response;
-      expect(res.status).toBe(400);
-      const json = await res.json();
-      expect(json.error).toContain("baseRole not found");
-    });
-
-    it("adds clone with suffixed role when base exists", async () => {
-      const agents = [{ role: "lead", name: "Lead" }];
-      const templates = { "lead.prompt": "content" };
-      const result = await handleAddLike(agents, templates, "lead", "Lead 2", "team1");
-      expect(result).not.toBeInstanceOf(Response);
-      const op = result as { nextAgents: Array<Record<string, unknown>>; nextTemplates: Record<string, unknown>; addedRole: string | null };
-      expect(op.nextAgents).toHaveLength(2);
-      expect(op.nextAgents[1].role).toMatch(/^lead(-\d+)?$/);
-      expect(op.addedRole).toBeTruthy();
-    });
-  });
-});

package/src/app/api/recipes/team-agents/helpers.ts

@@ -1,151 +0,0 @@
-import YAML from "yaml";
-import { NextResponse } from "next/server";
-import { runOpenClaw } from "@/lib/openclaw";
-import { splitRecipeFrontmatter, normalizeRole } from "@/lib/recipe-team-agents";
-
-export { splitRecipeFrontmatter as splitFrontmatter, normalizeRole };
-
-export function parseRecipeFrontmatter(yamlText: string) {
-  const fm = (YAML.parse(yamlText) ?? {}) as Record<string, unknown>;
-  const agentsRaw = fm.agents;
-  const agents: Array<Record<string, unknown>> = Array.isArray(agentsRaw)
-    ? (agentsRaw as Array<Record<string, unknown>>)
-    : [];
-  const templatesRaw = fm.templates;
-  const templates: Record<string, unknown> =
-    templatesRaw && typeof templatesRaw === "object" && !Array.isArray(templatesRaw)
-      ? (templatesRaw as Record<string, unknown>)
-      : {};
-  return { fm, agents, templates };
-}
-
-export function buildNextMarkdown(
-  fm: Record<string, unknown>,
-  nextAgents: Array<Record<string, unknown>>,
-  nextTemplates: Record<string, unknown>,
-  rest: string
-) {
-  const nextFm = {
-    ...fm,
-    agents: nextAgents,
-    ...(Object.keys(nextTemplates).length ? { templates: nextTemplates } : {}),
-  };
-  const nextYaml = YAML.stringify(nextFm).trimEnd();
-  return `---\n${nextYaml}\n---\n${rest}`;
-}
-
-export type OpResult = {
-  nextAgents: Array<Record<string, unknown>>;
-  nextTemplates: Record<string, unknown>;
-  addedRole: string | null;
-};
-
-export function handleRemove(
-  agents: Array<Record<string, unknown>>,
-  templates: Record<string, unknown>,
-  role: string
-): OpResult {
-  const nextAgents = agents.filter((a) => String(a.role ?? "") !== role);
-  const nextTemplates = { ...templates };
-  for (const k of Object.keys(nextTemplates)) {
-    if (k.startsWith(`${role}.`)) delete nextTemplates[k];
-  }
-  return { nextAgents, nextTemplates, addedRole: null };
-}
-
-export function handleAdd(
-  agents: Array<Record<string, unknown>>,
-  templates: Record<string, unknown>,
-  role: string,
-  name: string
-): OpResult {
-  const next = {
-    ...agents.find((a) => String(a.role ?? "") === role),
-    role,
-    ...(name ? { name } : {}),
-  };
-  const nextAgents = agents.slice();
-  const idx = nextAgents.findIndex((a) => String(a.role ?? "") === role);
-  if (idx === -1) nextAgents.push(next);
-  else nextAgents[idx] = next;
-  return { nextAgents, nextTemplates: { ...templates }, addedRole: role };
-}
-
-function maxSuffixFromUsedRoles(usedRoles: Set<string>, baseRole: string): number {
-  const escaped = baseRole.replace(/[-/\\^$*+?.()|[\]{}]/g, "\\$&");
-  let n = 1;
-  for (const r of usedRoles) {
-    if (r === baseRole) n = Math.max(n, 1);
-    const m = r.match(new RegExp(`^${escaped}-([0-9]+)$`));
-    if (m) {
-      const k = Number(m[1]);
-      if (Number.isFinite(k)) n = Math.max(n, k);
-    }
-  }
-  return n;
-}
-
-async function fetchExistingAgentIds(teamId: string): Promise<Set<string>> {
-  if (!teamId) return new Set<string>();
-  try {
-    const res = await runOpenClaw(["agents", "list", "--json"]);
-    if (!res.ok) return new Set<string>();
-    const items = JSON.parse(res.stdout) as Array<{ id?: unknown }>;
-    return new Set(items.map((a) => String(a.id ?? "").trim()).filter(Boolean));
-  } catch {
-    return new Set<string>();
-  }
-}
-
-function pickNextRole(
-  baseRole: string,
-  usedRoles: Set<string>,
-  existingAgentIds: Set<string>,
-  teamId: string,
-  n: number
-): string {
-  const isTaken = (role: string) =>
-    usedRoles.has(role) || (teamId ? existingAgentIds.has(`${teamId}-${role}`) : false);
-  if (!isTaken(baseRole)) return baseRole;
-  let i = Math.max(2, n + 1);
-  while (isTaken(`${baseRole}-${i}`)) i++;
-  return `${baseRole}-${i}`;
-}
-
-export async function handleAddLike(
-  agents: Array<Record<string, unknown>>,
-  templates: Record<string, unknown>,
-  baseRole: string,
-  name: string,
-  teamId: string
-): Promise<OpResult | NextResponse> {
-  const base = agents.find((a) => String(a.role ?? "") === baseRole);
-  if (!base) {
-    return NextResponse.json({ ok: false, error: `baseRole not found in recipe: ${baseRole}` }, { status: 400 });
-  }
-
-  const nextAgents = agents.slice();
-  const nextTemplates = { ...templates };
-  const usedRoles = new Set(nextAgents.map((a) => String(a.role ?? "").trim()).filter(Boolean));
-  const n = maxSuffixFromUsedRoles(usedRoles, baseRole);
-  const existingAgentIds = await fetchExistingAgentIds(teamId);
-  const nextRole = pickNextRole(baseRole, usedRoles, existingAgentIds, teamId, n);
-
-  const baseName = typeof (base as { name?: unknown }).name === "string" ? String((base as { name?: unknown }).name) : "";
-  const autoSuffix = nextRole === baseRole ? "" : String(nextRole.slice(baseRole.length + 1));
-  const suffixPart = autoSuffix ? ` ${autoSuffix}` : "";
-  const nextName = name || (baseName ? baseName + suffixPart : "");
-
-  const clone = { ...base, role: nextRole, ...(nextName ? { name: nextName } : {}) };
-  nextAgents.push(clone);
-
-  for (const [k, v] of Object.entries(templates)) {
-    if (!k.startsWith(`${baseRole}.`)) continue;
-    const suffix = k.slice(baseRole.length);
-    const nextKey = `${nextRole}${suffix}`;
-    if (nextTemplates[nextKey] === undefined) nextTemplates[nextKey] = v;
-  }
-
-  const addedRole = String((nextAgents[nextAgents.length - 1] as { role?: unknown } | undefined)?.role ?? "").trim() || null;
-  return { nextAgents, nextTemplates, addedRole };
-}