@xrkseek/harness 0.6.8 → 0.6.10

package/node_modules/@xrkseek/server-face/dist/agent-roster-tools.js

@@ -1,19 +1,24 @@
 /**
- * Model-facing team_list / team_save / team_import / team_ready.
- * Pack import materializes under agent-rosters/packs/ (child-only skills).
+ * Model-facing team_list / save / import / ready / export / remove /
+ * team_self (child own surfaces). Packs under agent-rosters/packs/.
  */
+import { writeFileSync } from "node:fs";
 import path from "node:path";
 import { readSessionEvents } from "@xrkseek/core-session";
 import { isAttachmentError } from "@xrkseek/attachment";
 import { canvasWorkspaceIdForSession } from "./canvas-tools.js";
 import { isAgentTeamSpawnRole } from "./agent-team-roles.js";
-import { GLOBAL_ROSTER_ID, memberIdProblem, catalogMemberWriteProblem, parseMemberToolPolicy, parseRosterScope, rosterPublicFields, } from "./agent-roster-store.js";
-import { parseAgentMemberCard } from "./agent-member-card.js";
-import { loadAgentMemberPack, loadAgentMemberPackBytes, materializeAgentMemberPack, notReadyPackHints, } from "./agent-member-pack.js";
-import { captureSessionPlaybook } from "./capture-member-playbook.js";
+import { GLOBAL_ROSTER_ID, memberIdProblem, catalogMemberWriteProblem, parseMemberInject, parseMemberToolPolicy, parseRosterScope, rosterPublicFields, } from "./agent-roster-store.js";
+import { parseAgentMemberCard, } from "./agent-member-card.js";
+import { AGENT_MEMBER_PACK_LAYOUT, MEMBER_PACK_BRIEFCASE, buildAgentMemberPackZip, collectPackSkillFiles, collectPackWorkspaceFiles, listMemberPackInventory, loadAgentMemberPack, loadAgentMemberPackBytes, materializeAgentMemberPack, mergeIntoMemberPack, notReadyPackHints, syncMemberPackPlaybook, } from "./agent-member-pack.js";
+import { withAdvancedPackPathGuidance } from "./capture-member-playbook.js";
 import { resolveSessionCwd } from "./session-cwd.js";
 import { referencedAttachmentIds } from "./session-attachment.js";
-import { liveRosterCatalog, packShareFlowHint, rosterScopeVisibility, teamListDescription, } from "./session-tool-copy.js";
+import { dispatchFaceMethod } from "./dispatch.js";
+import { liveRosterCatalog, rosterScopeVisibility, teamListDescription, } from "./session-tool-copy.js";
+function teamPresent(title, kind, args = {}) {
+    return { card: "generic", title, kind, rawInput: args };
+}
 function missingMemberHint(runtime, canvasId, id, tool) {
     const home = runtime.agentRoster.findHome(id);
     if (!home) {
@@ -86,35 +91,204 @@ function enrichListedMember(runtime, canvasId, row) {
         ...rosterPublicFields(row),
         visibility: rosterScopeVisibility(row.scope === "global" ? "global" : "workspace", rosterWs),
     };
-    if (row.kind !== "advanced" || row.ready)
+    if (row.kind !== "advanced")
         return pub;
-    return {
+    const packOpts = {
+        rosterWorkspaceId: rosterWs,
+        memberId: row.id,
+        productHome: runtime.agentRoster.home,
+    };
+    const inventory = listMemberPackInventory(packOpts);
+    const base = {
         ...pub,
-        ...notReadyPackHints({
-            rosterWorkspaceId: rosterWs,
-            memberId: row.id,
-            productHome: runtime.agentRoster.home,
+        ...(inventory
+            ? {
+                packHome: inventory.packHome,
+                playbookPath: inventory.playbookPath,
+                ...(inventory.envPrepPath
+                    ? { envPrepPath: inventory.envPrepPath }
+                    : {}),
+                ...(inventory.materialsPath
+                    ? {
+                        materialsPath: inventory.materialsPath,
+                        materialsTop: inventory.materialsTop,
+                    }
+                    : {}),
+                skills: inventory.skills,
+                briefcase: MEMBER_PACK_BRIEFCASE,
+            }
+            : {}),
+    };
+    if (row.ready)
+        return base;
+    return {
+        ...base,
+        ...notReadyPackHints(packOpts),
+    };
+}
+function resolveCwdPaths(runtime, sessionId, rows) {
+    const cwd = resolveSessionCwd(runtime, sessionId);
+    if (!Array.isArray(rows))
+        return [];
+    return rows
+        .filter((s) => typeof s === "string" && s.trim().length > 0)
+        .map((s) => path.isAbsolute(s.trim()) ? s.trim() : path.resolve(cwd, s.trim()));
+}
+/**
+ * Mint an advanced pack member (playbook + optional skills/files/env).
+ * Shared by team_save — team_import only installs card=/pack=.
+ */
+function mintAdvancedMember(input) {
+    const { runtime, sessionId, canvasId, scope, name, playbook, } = input;
+    if (!name.trim() || !playbook.trim()) {
+        return {
+            ok: false,
+            content: "team_save: advanced needs name and playbook (from the real scenario)",
+        };
+    }
+    const toolsPolicy = parseMemberToolPolicy(input.tools);
+    if (input.tools !== undefined && input.tools !== null && !toolsPolicy) {
+        return {
+            ok: false,
+            content: 'team_save: tools needs { mode: "allow"|"deny", names: [...] }',
+        };
+    }
+    const card = {
+        name: name.trim(),
+        playbook: withAdvancedPackPathGuidance(playbook),
+        role: isAgentTeamSpawnRole(input.roleRaw) ? input.roleRaw : "worker",
+        ...(input.brief?.trim() ? { brief: input.brief.trim() } : {}),
+        ...(input.inject === "subagent" || input.inject === "minimal"
+            ? { inject: input.inject }
+            : {}),
+        ...(toolsPolicy ? { tools: toolsPolicy } : {}),
+    };
+    const skillPaths = resolveCwdPaths(runtime, sessionId, input.skills);
+    let skillsFiles;
+    if (skillPaths.length > 0) {
+        const bundled = collectPackSkillFiles(skillPaths);
+        if (!bundled.ok) {
+            return { ok: false, content: `team_save: ${bundled.error}` };
+        }
+        skillsFiles = bundled.files;
+    }
+    const filePaths = resolveCwdPaths(runtime, sessionId, input.files);
+    let workspaceFiles;
+    if (filePaths.length > 0) {
+        const bundled = collectPackWorkspaceFiles(filePaths);
+        if (!bundled.ok) {
+            return { ok: false, content: `team_save: ${bundled.error}` };
+        }
+        workspaceFiles = bundled.files;
+    }
+    const envPrep = input.envPrep?.trim() || undefined;
+    const zip = buildAgentMemberPackZip(card, {
+        ...(skillsFiles && skillsFiles.size > 0 ? { skillsFiles } : {}),
+        ...(workspaceFiles && workspaceFiles.size > 0 ? { workspaceFiles } : {}),
+        ...(envPrep ? { envPrep } : {}),
+    });
+    const loaded = loadAgentMemberPackBytes(zip);
+    if (!loaded.ok) {
+        return { ok: false, content: `team_save: ${loaded.error}` };
+    }
+    const rosterWs = scope === "global" ? GLOBAL_ROSTER_ID : canvasId;
+    const member = runtime.agentRoster.upsertAtScope(canvasId, scope, {
+        ...loaded.pack.card,
+        kind: "advanced",
+        ready: false,
+        ...(input.id?.trim() ? { id: input.id.trim() } : {}),
+    });
+    if (!member) {
+        return { ok: false, content: "team_save: name and playbook required" };
+    }
+    const placed = materializeAgentMemberPack(loaded.pack, {
+        rosterWorkspaceId: rosterWs,
+        memberId: member.id,
+        productHome: runtime.agentRoster.home,
+    });
+    return {
+        ok: true,
+        content: JSON.stringify({
+            workspaceId: rosterWs,
+            scope,
+            visibility: rosterScopeVisibility(scope, rosterWs),
+            member,
+            source: "team_save",
+            kind: "advanced",
+            packDir: placed.packDir,
+            skillNames: placed.skillNames,
+            ready: false,
+            ...(placed.envPrep
+                ? {
+                    envPrep: placed.envPrep,
+                    envHint: "Review envPrep here (or hand zip to a friend). Then team_ready. Do not auto-run scripts.",
+                }
+                : {
+                    envHint: "No env/SETUP.md — team_ready when configured; friend gets playbook (± skills).",
+                }),
+            next: "team_ready → subagent member_id",
+            layout: AGENT_MEMBER_PACK_LAYOUT,
+            pathHint: `Spawn keeps parent workspace cwd. ${MEMBER_PACK_BRIEFCASE} = pack_home (injected) — child may read/edit/copy into workspace. Share zip with relative paths under the pack tree.`,
+            shareHint: "Friend: team_export zip / UI import, or they team_import pack=… → follow envPrep with main session → team_ready. " +
+                "scope=global keeps it on every canvas team_list.",
         }),
     };
 }
-/** Register team_list / team_save / team_import / team_ready (or list-only). */
+function patchExistingMember(runtime, canvasId, existing, patch) {
+    const name = (patch.name ?? existing.name).trim();
+    let playbook = (patch.playbook ?? existing.playbook).trim();
+    if (existing.kind === "advanced" && patch.playbook !== undefined) {
+        playbook = withAdvancedPackPathGuidance(playbook);
+    }
+    if (!name || !playbook)
+        return undefined;
+    const scope = existing.scope === "global" ? "global" : "workspace";
+    const inject = patch.inject ?? existing.inject;
+    const tools = patch.tools === undefined
+        ? existing.tools
+        : patch.tools === null
+            ? undefined
+            : patch.tools;
+    const member = runtime.agentRoster.upsertAtScope(canvasId, scope, {
+        id: existing.id,
+        name,
+        playbook,
+        role: isAgentTeamSpawnRole(existing.role) ? existing.role : "worker",
+        kind: existing.kind,
+        ready: existing.ready,
+        brief: patch.brief !== undefined ? patch.brief : existing.brief,
+        inject,
+        ...(tools ? { tools } : patch.tools === null ? { tools: null } : {}),
+        appearance: existing.appearance,
+    });
+    if (!member)
+        return undefined;
+    if (existing.kind === "advanced") {
+        const rosterWs = scope === "global" ? GLOBAL_ROSTER_ID : canvasId;
+        syncMemberPackPlaybook({
+            rosterWorkspaceId: rosterWs,
+            memberId: member.id,
+            playbook: member.playbook,
+            productHome: runtime.agentRoster.home,
+        });
+    }
+    return runtime.agentRoster.get(canvasId, member.id);
+}
+/** Register team_list / team_save / team_import / team_ready / team_remove (or list + team_self). */
 export function bindAgentRosterTools(tools, options) {
     const { runtime, sessionId, listOnly = false } = options;
+    const selfMemberId = options.selfMemberId?.trim() || undefined;
     const ws = () => canvasWorkspaceIdForSession(runtime, sessionId);
     registerTool(tools, {
         name: "team_list",
         description: "List Agent Team members (干员) for this workspace + global.",
         parameters: { type: "object", properties: {}, additionalProperties: false },
         isConcurrencySafe: () => true,
-        presentCall: () => ({
-            card: "generic",
-            title: "team_list",
-            kind: "search",
-            rawInput: {},
-        }),
+        presentCall: () => teamPresent("team_list", "search"),
         dynamicSchema: () => ({
             description: teamListDescription({
                 listOnly,
+                ...(selfMemberId ? { selfRevise: true } : {}),
                 catalog: liveRosterCatalog(runtime, sessionId),
             }),
         }),
@@ -126,33 +300,139 @@ export function bindAgentRosterTools(tools, options) {
             return { content: JSON.stringify({ workspaceId, members }) };
         },
     });
-    if (listOnly)
+    if (listOnly) {
+        if (selfMemberId) {
+            registerTool(tools, {
+                name: "team_self",
+                description: "Revise YOUR 干员 surfaces (this member_id): playbook, inject, tools, brief. " +
+                    "Child tool/inject surface ≠ parent. Durable for next spawn. " +
+                    "Cannot mint, import, export, or edit other members.",
+                parameters: {
+                    type: "object",
+                    properties: {
+                        playbook: {
+                            type: "string",
+                            description: "Replacement playbook (max 20000). Omit to keep.",
+                        },
+                        brief: {
+                            type: "string",
+                            description: "Replacement duty line. Omit to keep.",
+                        },
+                        name: {
+                            type: "string",
+                            description: "Optional display-name tweak. Omit to keep.",
+                        },
+                        inject: {
+                            type: "string",
+                            description: "minimal (thin context) | subagent (keep project docs). Omit to keep.",
+                        },
+                        tools: {
+                            type: "object",
+                            properties: {
+                                mode: { type: "string" },
+                                names: { type: "array", items: { type: "string" } },
+                            },
+                            additionalProperties: false,
+                            description: "allow|deny list on top of role. Pass null to clear.",
+                        },
+                    },
+                    additionalProperties: false,
+                },
+                presentCall: (args) => teamPresent("team_self", "execute", args),
+                execute: async (args) => {
+                    const a = readArgs(args);
+                    const canvasId = ws();
+                    const existing = runtime.agentRoster.get(canvasId, selfMemberId);
+                    if (!existing || existing.catalog) {
+                        return {
+                            content: `team_self: member "${selfMemberId}" not editable here`,
+                            isError: true,
+                        };
+                    }
+                    const hasPlaybook = typeof a.playbook === "string";
+                    const hasBrief = typeof a.brief === "string";
+                    const hasName = typeof a.name === "string" && a.name.trim().length > 0;
+                    const hasInject = a.inject === "minimal" || a.inject === "subagent";
+                    const hasTools = a.tools !== undefined;
+                    if (!hasPlaybook && !hasBrief && !hasName && !hasInject && !hasTools) {
+                        return {
+                            content: "team_self: pass playbook, brief, inject, and/or tools (optional name)",
+                            isError: true,
+                        };
+                    }
+                    const toolsPatch = a.tools === null
+                        ? null
+                        : hasTools
+                            ? parseMemberToolPolicy(a.tools)
+                            : undefined;
+                    if (hasTools && a.tools !== null && !toolsPatch) {
+                        return {
+                            content: 'team_self: tools needs { mode: "allow"|"deny", names: [...] }',
+                            isError: true,
+                        };
+                    }
+                    const patched = patchExistingMember(runtime, canvasId, existing, {
+                        ...(hasName ? { name: String(a.name).trim() } : {}),
+                        ...(hasPlaybook ? { playbook: String(a.playbook) } : {}),
+                        ...(hasBrief ? { brief: String(a.brief) } : {}),
+                        ...(hasInject
+                            ? { inject: parseMemberInject(a.inject, existing.inject) }
+                            : {}),
+                        ...(hasTools ? { tools: toolsPatch ?? null } : {}),
+                    });
+                    if (!patched) {
+                        return { content: "team_self: name and playbook required", isError: true };
+                    }
+                    return {
+                        content: JSON.stringify({
+                            member: {
+                                ...enrichListedMember(runtime, canvasId, patched),
+                                playbook: patched.playbook,
+                            },
+                            next: "next subagent member_id uses these surfaces; this turn already started",
+                        }),
+                    };
+                },
+            });
+        }
         return;
+    }
     registerTool(tools, {
         name: "team_save",
-        description: "Mint/update a thin 干员 (always ready). Not propose_skill. Catalog mem_seed_* are read-only — omit id. " +
-            "from_session=true fills name/playbook from this chat. scope global|workspace. " +
-            "Advanced pack members: use team_import pack= (not team_save). " +
-            packShareFlowHint(),
+        description: "Mint or revise a 干员 (template → skill xrk-delegate). " +
+            "Thin: name+playbook (+inject/tools). " +
+            `Advanced ${MEMBER_PACK_BRIEFCASE}: kind=advanced + skills/files/env_prep (playbook.md · skills/ · workspace/ · env/SETUP.md) → team_ready → team_export. ` +
+            "Revise any custom member with id= (omit unchanged fields); advanced may also merge skills/files/env_prep into 公文包. " +
+            "Install friend zip/json → team_import (not this).",
         parameters: {
             type: "object",
             properties: {
                 id: {
                     type: "string",
-                    description: "mem_ then [A-Za-z0-9][A-Za-z0-9._-]*, max 64 chars. Omit to mint.",
+                    description: "Existing mem_… to revise (thin or advanced). Omit to mint. Pattern: mem_[A-Za-z0-9][A-Za-z0-9._-]*",
+                },
+                name: {
+                    type: "string",
+                    description: "Short label (e.g. PPT排版员). On revise, omit to keep.",
                 },
-                name: { type: "string" },
                 playbook: {
                     type: "string",
-                    description: "Standalone child instructions (max 20000 chars).",
+                    description: "Durable how-to. Prefer: Role; Steps; Constraints; Acceptance; for advanced also list what is in 公文包 (briefcase). " +
+                        "On revise, omit to keep. Max 20000. Template: skill xrk-delegate.",
                 },
                 role: { type: "string" },
-                from_session: { type: "boolean" },
-                brief: { type: "string" },
-                scope: { type: "string", description: "global | workspace (default workspace)" },
+                kind: {
+                    type: "string",
+                    description: "thin (default) | advanced. Also advanced when skills/files/env_prep set.",
+                },
+                brief: { type: "string", description: "One-line duty for team_list." },
+                scope: {
+                    type: "string",
+                    description: "global | workspace. Thin default workspace; advanced default global.",
+                },
                 inject: {
                     type: "string",
-                    description: "minimal (default) | subagent",
+                    description: "minimal (default) | subagent (keep project docs)",
                 },
                 tools: {
                     type: "object",
@@ -161,16 +441,26 @@ export function bindAgentRosterTools(tools, options) {
                         names: { type: "array", items: { type: "string" } },
                     },
                     additionalProperties: false,
+                    description: "Child tool allow|deny on top of role.",
+                },
+                env_prep: {
+                    type: "string",
+                    description: `Advanced ${MEMBER_PACK_BRIEFCASE}: env/SETUP.md (links, installs). Never auto-run. Merge on id= revise.`,
+                },
+                skills: {
+                    type: "array",
+                    items: { type: "string" },
+                    description: `Advanced: skill dirs → ${MEMBER_PACK_BRIEFCASE} skills/<id>/ (full tree incl. references/). Merge on id=.`,
+                },
+                files: {
+                    type: "array",
+                    items: { type: "string" },
+                    description: `Advanced: paths → ${MEMBER_PACK_BRIEFCASE} workspace/ (materials; not session cwd). Merge on id=.`,
                 },
             },
             additionalProperties: false,
         },
-        presentCall: (args) => ({
-            card: "generic",
-            title: "team_save",
-            kind: "execute",
-            rawInput: args,
-        }),
+        presentCall: (args) => teamPresent("team_save", "execute", args),
         execute: async (args) => {
             const a = readArgs(args);
             const idProblem = memberIdProblem(a.id);
@@ -179,14 +469,15 @@ export function bindAgentRosterTools(tools, options) {
             const catalogProblem = catalogMemberWriteProblem(a.id);
             if (catalogProblem)
                 return { content: `team_save: ${catalogProblem}`, isError: true };
-            const captured = a.from_session === true
-                ? captureSessionPlaybook(runtime, sessionId)
-                : undefined;
-            const name = String(a.name ?? "").trim() || captured?.name || "";
-            const playbook = String(a.playbook ?? "").trim() || captured?.playbook || "";
+            const name = String(a.name ?? "").trim();
+            const playbook = String(a.playbook ?? "").trim();
             const roleRaw = typeof a.role === "string" ? a.role : "worker";
             const canvasId = ws();
-            const scope = parseRosterScope(a.scope);
+            const wantsAdvanced = a.kind === "advanced" ||
+                (typeof a.env_prep === "string" && a.env_prep.trim().length > 0) ||
+                (Array.isArray(a.skills) && a.skills.length > 0) ||
+                (Array.isArray(a.files) && a.files.length > 0);
+            const scope = parseRosterScope(a.scope ?? (wantsAdvanced ? "global" : "workspace"));
             if (scope === undefined) {
                 return {
                     content: 'team_save: scope must be "workspace" or "global"',
@@ -195,13 +486,169 @@ export function bindAgentRosterTools(tools, options) {
             }
             if (typeof a.id === "string" && a.id.trim()) {
                 const existing = runtime.agentRoster.get(canvasId, a.id.trim());
-                if (existing?.kind === "advanced") {
+                if (!existing) {
+                    return {
+                        content: missingMemberHint(runtime, canvasId, a.id.trim(), "team_save"),
+                        isError: true,
+                    };
+                }
+                const toolsPatch = a.tools === null
+                    ? null
+                    : a.tools !== undefined
+                        ? parseMemberToolPolicy(a.tools)
+                        : undefined;
+                if (a.tools !== undefined && a.tools !== null && !toolsPatch) {
                     return {
-                        content: "team_save: refuse — this id is an advanced pack member. " +
-                            "Prep with team_list envPrep, then team_ready (or import a new pack). Do not downgrade via team_save.",
+                        content: 'team_save: tools needs { mode: "allow"|"deny", names: [...] }',
                         isError: true,
                     };
                 }
+                const hasName = name.length > 0;
+                const hasPlaybook = playbook.length > 0;
+                const hasSurface = hasName ||
+                    hasPlaybook ||
+                    typeof a.brief === "string" ||
+                    a.inject === "minimal" ||
+                    a.inject === "subagent" ||
+                    a.tools !== undefined;
+                if (existing.kind === "advanced") {
+                    const rosterWs = existing.scope === "global" ? GLOBAL_ROSTER_ID : canvasId;
+                    const packOpts = {
+                        rosterWorkspaceId: rosterWs,
+                        memberId: existing.id,
+                        productHome: runtime.agentRoster.home,
+                    };
+                    let inventory = listMemberPackInventory(packOpts);
+                    const skillPaths = resolveCwdPaths(runtime, sessionId, a.skills);
+                    const filePaths = resolveCwdPaths(runtime, sessionId, a.files);
+                    const envPrep = typeof a.env_prep === "string" && a.env_prep.trim()
+                        ? a.env_prep.trim()
+                        : undefined;
+                    if (skillPaths.length > 0 || filePaths.length > 0 || envPrep) {
+                        let skillsFiles;
+                        if (skillPaths.length > 0) {
+                            const bundled = collectPackSkillFiles(skillPaths);
+                            if (!bundled.ok) {
+                                return { content: `team_save: ${bundled.error}`, isError: true };
+                            }
+                            skillsFiles = bundled.files;
+                        }
+                        let workspaceFiles;
+                        if (filePaths.length > 0) {
+                            const bundled = collectPackWorkspaceFiles(filePaths);
+                            if (!bundled.ok) {
+                                return { content: `team_save: ${bundled.error}`, isError: true };
+                            }
+                            workspaceFiles = bundled.files;
+                        }
+                        const merged = mergeIntoMemberPack({
+                            ...packOpts,
+                            ...(skillsFiles ? { skillsFiles } : {}),
+                            ...(workspaceFiles ? { workspaceFiles } : {}),
+                            ...(envPrep ? { envPrep } : {}),
+                        });
+                        if (!merged.ok) {
+                            return { content: `team_save: ${merged.error}`, isError: true };
+                        }
+                        inventory = merged.inventory;
+                    }
+                    let patched = existing;
+                    if (hasSurface) {
+                        const next = patchExistingMember(runtime, canvasId, existing, {
+                            ...(hasName ? { name } : {}),
+                            ...(hasPlaybook ? { playbook } : {}),
+                            ...(typeof a.brief === "string" ? { brief: a.brief } : {}),
+                            ...(a.inject === "minimal" || a.inject === "subagent"
+                                ? { inject: a.inject }
+                                : {}),
+                            ...(a.tools !== undefined ? { tools: toolsPatch ?? null } : {}),
+                        });
+                        if (!next) {
+                            return {
+                                content: "team_save: name and playbook required",
+                                isError: true,
+                            };
+                        }
+                        patched = next;
+                    }
+                    else if (skillPaths.length === 0 &&
+                        filePaths.length === 0 &&
+                        !envPrep) {
+                        return {
+                            content: "team_save: pass playbook/brief/inject/tools and/or skills/files/env_prep to revise",
+                            isError: true,
+                        };
+                    }
+                    return {
+                        content: JSON.stringify({
+                            workspaceId: patched.scope === "global" ? "global" : canvasId,
+                            member: {
+                                ...enrichListedMember(runtime, canvasId, patched),
+                                playbook: patched.playbook,
+                            },
+                            kind: "advanced",
+                            briefcase: MEMBER_PACK_BRIEFCASE,
+                            ...(inventory ? { inventory } : {}),
+                            next: patched.ready
+                                ? "subagent member_id · team_export to share"
+                                : "follow envPrep → team_ready → subagent member_id",
+                        }),
+                    };
+                }
+                // Thin revise (or upgrade path falls through to mint when wantsAdvanced).
+                if (!wantsAdvanced) {
+                    if (!hasSurface) {
+                        return {
+                            content: "team_save: pass playbook/brief/inject/tools to revise thin",
+                            isError: true,
+                        };
+                    }
+                    const patched = patchExistingMember(runtime, canvasId, existing, {
+                        ...(hasName ? { name } : {}),
+                        ...(hasPlaybook ? { playbook } : {}),
+                        ...(typeof a.brief === "string" ? { brief: a.brief } : {}),
+                        ...(a.inject === "minimal" || a.inject === "subagent"
+                            ? { inject: a.inject }
+                            : {}),
+                        ...(a.tools !== undefined ? { tools: toolsPatch ?? null } : {}),
+                    });
+                    if (!patched) {
+                        return {
+                            content: "team_save: name and playbook required",
+                            isError: true,
+                        };
+                    }
+                    return {
+                        content: JSON.stringify({
+                            workspaceId: patched.scope === "global" ? "global" : canvasId,
+                            member: enrichListedMember(runtime, canvasId, patched),
+                            kind: "thin",
+                            next: "subagent member_id",
+                        }),
+                    };
+                }
+            }
+            if (wantsAdvanced) {
+                const minted = mintAdvancedMember({
+                    runtime,
+                    sessionId,
+                    canvasId,
+                    scope,
+                    name,
+                    playbook,
+                    roleRaw,
+                    ...(typeof a.brief === "string" ? { brief: a.brief } : {}),
+                    inject: a.inject,
+                    tools: a.tools,
+                    ...(typeof a.env_prep === "string" ? { envPrep: a.env_prep } : {}),
+                    skills: a.skills,
+                    files: a.files,
+                    ...(typeof a.id === "string" ? { id: a.id } : {}),
+                });
+                return {
+                    content: minted.content,
+                    ...(minted.ok ? {} : { isError: true }),
+                };
             }
             const toolsPolicy = parseMemberToolPolicy(a.tools);
             const member = runtime.agentRoster.upsertAtScope(canvasId, scope, {
@@ -232,45 +679,48 @@ export function bindAgentRosterTools(tools, options) {
     });
     registerTool(tools, {
         name: "team_import",
-        description: packShareFlowHint(),
+        description: "Install a friend-shared 干员: card= (thin JSON) or pack= (advanced zip/dir/attachment → 公文包/briefcase). " +
+            "Then follow envPrep with this main session → team_ready. Illegal pack: inspect/fix → re-import. " +
+            "After import you may team_save id= to edit playbook or merge skills/files. Mint new → team_save (not this).",
         parameters: {
             type: "object",
             properties: {
                 card: {
                     type: "string",
-                    description: "Thin JSON (kind xrk-agent-member). Omit when pack is set.",
+                    description: "Thin JSON (kind xrk-agent-member). Omit when pack.",
                 },
                 pack: {
                     type: "string",
                     description: "Marked pack zip/dir path (cwd-relative OK) or upload id " +
-                        "(attachment:sha256:… / sha256:… from the chat file handle). Omit when card is set.",
+                        "(attachment:sha256:… / sha256:…). Omit when card.",
                 },
                 scope: {
                     type: "string",
-                    description: "global | workspace. global = every workspace; workspace = this canvas only. " +
-                        "Defaults: card→workspace, pack→global (pass workspace for project-only packs).",
+                    description: "global | workspace. Defaults: card→workspace; pack→global (shareable).",
                 },
             },
             additionalProperties: false,
         },
-        presentCall: (args) => ({
-            card: "generic",
-            title: "team_import",
-            kind: "execute",
-            rawInput: args,
-        }),
+        presentCall: (args) => teamPresent("team_import", "execute", args),
         execute: async (args) => {
             const a = readArgs(args);
+            if (a.from_session === true) {
+                return {
+                    content: "team_import: minting moved to team_save (kind=advanced + name/playbook/skills/files). " +
+                        "team_import is only card= or pack= install.",
+                    isError: true,
+                };
+            }
             const hasCard = typeof a.card === "string" && a.card.trim().length > 0;
             const hasPack = typeof a.pack === "string" && a.pack.trim().length > 0;
             if (hasCard === hasPack) {
                 return {
-                    content: "team_import: set exactly one of card (JSON) or pack (path|attachment id)",
+                    content: "team_import: set exactly one of card (JSON) or pack (path|attachment). " +
+                        "Mint new → team_save.",
                     isError: true,
                 };
             }
-            // Pack defaults global (shareable); card defaults workspace. Either may override.
-            const scope = parseRosterScope(a.scope ?? (hasPack ? "global" : "workspace"));
+            const scope = parseRosterScope(a.scope ?? (hasCard ? "workspace" : "global"));
             if (scope === undefined) {
                 return { content: 'team_import: scope must be "workspace" or "global"', isError: true };
             }
@@ -299,8 +749,8 @@ export function bindAgentRosterTools(tools, options) {
                     }),
                 };
             }
-            const packRaw = String(a.pack);
-            const loaded = await loadPackFromArg(runtime, sessionId, packRaw);
+            const packLabel = String(a.pack);
+            const loaded = await loadPackFromArg(runtime, sessionId, packLabel);
             if (!loaded.ok)
                 return { content: `team_import: ${loaded.error}`, isError: true };
             const member = runtime.agentRoster.upsertAtScope(canvasId, scope, {
@@ -322,31 +772,37 @@ export function bindAgentRosterTools(tools, options) {
                     visibility,
                     member,
                     source: "pack",
-                    pack: packRaw,
+                    pack: packLabel,
                     packDir: placed.packDir,
                     skillNames: placed.skillNames,
                     ready: false,
                     ...(placed.envPrep
                         ? {
                             envPrep: placed.envPrep,
-                            envHint: "Review envPrep in this parent session, then call team_ready. Do not auto-run untrusted scripts.",
+                            envHint: "Review envPrep here (or hand zip to a friend). Then team_ready. Do not auto-run scripts.",
                         }
                         : {
-                            envHint: "No env/SETUP.md — call team_ready when the member is configured.",
+                            envHint: "No env/SETUP.md — team_ready when configured; friend gets playbook (± skills).",
                         }),
-                    next: "team_ready → subagent member_id (pack skills inject into the child catalog only)",
+                    next: "team_ready → subagent member_id",
+                    layout: AGENT_MEMBER_PACK_LAYOUT,
+                    pathHint: `Spawn keeps parent workspace cwd. ${MEMBER_PACK_BRIEFCASE} = pack_home (injected) — child may read/edit/copy into workspace. Share zip with relative paths under the pack tree.`,
+                    shareHint: "Friend: team_export zip / UI import, or they team_import pack=… → follow envPrep with main session → team_ready. " +
+                        "scope=global keeps it on every canvas team_list.",
                 }),
             };
         },
     });
     registerTool(tools, {
         name: "team_ready",
-        description: "Mark an advanced pack 干员 ready after env prep / playbook review. " +
-            "Required before subagent member_id when team_list shows not-ready. Thin members stay ready.",
+        description: "Mark an advanced 干员 ready after env prep. " +
+            "Triggers: team_list shows not-ready; UI zip import; user says 帮我就绪 / mark ready / prep done. " +
+            "Flow: team_list → read envPrep → help install if needed → team_ready id=. " +
+            "Thin members stay ready. Required before subagent member_id on advanced.",
         parameters: {
             type: "object",
             properties: {
-                id: { type: "string", description: "mem_… member id" },
+                id: { type: "string", description: "mem_… from team_list (not-ready advanced)" },
                 ready: {
                     type: "boolean",
                     description: "Default true. Pass false to mark not-ready again.",
@@ -355,12 +811,7 @@ export function bindAgentRosterTools(tools, options) {
             required: ["id"],
             additionalProperties: false,
         },
-        presentCall: (args) => ({
-            card: "generic",
-            title: "team_ready",
-            kind: "execute",
-            rawInput: args,
-        }),
+        presentCall: (args) => teamPresent("team_ready", "execute", args),
         execute: async (args) => {
             const a = readArgs(args);
             const id = String(a.id ?? "").trim();
@@ -380,5 +831,133 @@ export function bindAgentRosterTools(tools, options) {
             };
         },
     });
+    registerTool(tools, {
+        name: "team_export",
+        description: "Export a custom 干员 for the user: thin→JSON, advanced→zip (公文包/briefcase: playbook+skills+references+workspace+env). " +
+            "Writes to dir= (default session cwd). Catalog mem_seed_* cannot export — mint a copy first. " +
+            "User said 导出/分享这个干员 / export member. UI Export is equivalent.",
+        parameters: {
+            type: "object",
+            properties: {
+                id: { type: "string", description: "mem_… from team_list (not catalog)" },
+                dir: {
+                    type: "string",
+                    description: "Output directory (cwd-relative or absolute). Default: session cwd.",
+                },
+            },
+            required: ["id"],
+            additionalProperties: false,
+        },
+        presentCall: (args) => teamPresent("team_export", "execute", args),
+        execute: async (args) => {
+            const a = readArgs(args);
+            const id = String(a.id ?? "").trim();
+            if (!id)
+                return { content: "team_export: id required", isError: true };
+            const exported = await dispatchFaceMethod(runtime, "team.export", `te-${Date.now()}`, {
+                sessionId,
+                id,
+            });
+            if (!exported.result.ok) {
+                const msg = typeof exported.result.error === "object" &&
+                    exported.result.error &&
+                    "message" in exported.result.error
+                    ? String(exported.result.error.message)
+                    : "export failed";
+                return { content: `team_export: ${msg}`, isError: true };
+            }
+            const value = exported.result.value;
+            const cwd = resolveSessionCwd(runtime, sessionId);
+            const dirRaw = typeof a.dir === "string" ? a.dir.trim() : "";
+            const outDir = dirRaw
+                ? path.isAbsolute(dirRaw)
+                    ? dirRaw
+                    : path.resolve(cwd, dirRaw)
+                : cwd;
+            const outPath = path.join(outDir, value.fileName);
+            try {
+                if (value.format === "json" && typeof value.text === "string") {
+                    writeFileSync(outPath, value.text, "utf8");
+                }
+                else if (value.format === "zip" && typeof value.base64 === "string") {
+                    writeFileSync(outPath, Buffer.from(value.base64, "base64"));
+                }
+                else {
+                    return { content: "team_export: empty payload", isError: true };
+                }
+            }
+            catch (err) {
+                return {
+                    content: `team_export: write failed — ${err instanceof Error ? err.message : String(err)}`,
+                    isError: true,
+                };
+            }
+            return {
+                content: JSON.stringify({
+                    format: value.format,
+                    path: outPath,
+                    fileName: value.fileName,
+                    ...(value.packIncomplete
+                        ? { packIncomplete: true, hint: value.hint }
+                        : {}),
+                    next: value.format === "zip"
+                        ? "Hand zip → team_import pack= → follow envPrep → team_ready"
+                        : "Hand JSON → team_import card= or UI import",
+                }),
+            };
+        },
+    });
+    registerTool(tools, {
+        name: "team_remove",
+        description: "Uninstall a custom 干员 (not catalog mem_seed_*). Like deleting a skill. " +
+            "Pass id from team_list; scope global|workspace must match where it lives.",
+        parameters: {
+            type: "object",
+            properties: {
+                id: { type: "string", description: "mem_… to delete" },
+                scope: {
+                    type: "string",
+                    description: "global | workspace — must match team_list scope for that id",
+                },
+            },
+            required: ["id", "scope"],
+            additionalProperties: false,
+        },
+        presentCall: (args) => teamPresent("team_remove", "execute", args),
+        execute: async (args) => {
+            const a = readArgs(args);
+            const id = String(a.id ?? "").trim();
+            if (!id)
+                return { content: "team_remove: id required", isError: true };
+            const catalogProblem = catalogMemberWriteProblem(id);
+            if (catalogProblem) {
+                return { content: `team_remove: ${catalogProblem}`, isError: true };
+            }
+            const scope = parseRosterScope(a.scope);
+            if (scope === undefined) {
+                return {
+                    content: 'team_remove: scope must be "workspace" or "global"',
+                    isError: true,
+                };
+            }
+            const canvasId = ws();
+            const rosterWs = scope === "global" ? GLOBAL_ROSTER_ID : canvasId;
+            const ok = runtime.agentRoster.remove(rosterWs, id);
+            if (!ok) {
+                return {
+                    content: missingMemberHint(runtime, canvasId, id, "team_remove"),
+                    isError: true,
+                };
+            }
+            return {
+                content: JSON.stringify({
+                    workspaceId: rosterWs,
+                    deleted: true,
+                    id,
+                    next: "gone from team_list; zip/JSON copies elsewhere are untouched",
+                }),
+            };
+        },
+    });
 }
 //# sourceMappingURL=agent-roster-tools.js.map