@pi-harness/pi-harness 0.1.14 → 0.1.15

package/packages/core/src/plugins/cli-notifier.ts

@@ -1,194 +0,0 @@
-import { execFile } from "node:child_process";
-import { promisify } from "node:util";
-import type { Context } from "@deepseek-ai/cordis";
-import z from "@deepseek-ai/schemastery";
-import { Type } from "@earendil-works/pi-ai";
-import { defineTool, type AgentToolResult } from "@earendil-works/pi-coding-agent";
-
-const execFileAsync = promisify(execFile);
-const maxMessageLength = 2048;
-const maxTitleLength = 256;
-const maxReasonLength = 1024;
-const maxNotifications = 20;
-const defaultTimeoutMs = 10_000;
-const maxCommandOutputBytes = 256 * 1024;
-const windowsNotificationScript =
-  "$message = [System.Security.SecurityElement]::Escape([string]$env:PI_HARNESS_NOTIFICATION_MESSAGE); $title = [System.Security.SecurityElement]::Escape([string]$env:PI_HARNESS_NOTIFICATION_TITLE); [Windows.UI.Notifications.ToastNotificationManager, Windows.UI.Notifications, ContentType = WindowsRuntime] > $null; $xml = New-Object Windows.Data.Xml.Dom.XmlDocument; $xml.LoadXml(\"<toast><visual><binding template='ToastText02'><text id='1'>$message</text><text id='2'>$title</text></binding></visual></toast>\"); [Windows.UI.Notifications.ToastNotificationManager]::CreateToastNotifier('Pi Harness').Show([Windows.UI.Notifications.ToastNotification]::new($xml))";
-type Notification = { time: string; title: string; message: string; delivered: boolean; reason?: string };
-type NotificationResult = { title: string; message: string; delivered: boolean; platform: NodeJS.Platform; reason?: string };
-
-export interface CliNotifierPluginConfig {
-  enabled?: boolean;
-  title?: string;
-  timeoutMs?: number;
-}
-
-export const Config: z<CliNotifierPluginConfig> = z.object({
-  enabled: z.boolean().default(true),
-  title: z.string().max(maxTitleLength).default("Pi Harness"),
-  timeoutMs: z.number().min(100).max(60_000).step(1).default(defaultTimeoutMs),
-});
-
-function appleScriptString(value: string): string {
-  return `"${value
-    .replaceAll("\\", "\\\\")
-    .replaceAll('"', '\\"')
-    .replace(/[\r\n\u2028\u2029]/gu, " ")}"`;
-}
-
-function boundedEventMessage(value: string): string {
-  return value.length <= maxMessageLength ? value : `${value.slice(0, maxMessageLength - 1)}…`;
-}
-
-function throwIfAborted(signal: AbortSignal | undefined): void {
-  if (!signal?.aborted) return;
-  throw signal.reason instanceof Error ? signal.reason : new Error("Desktop notification was cancelled", { cause: signal.reason });
-}
-
-function normalizeText(value: unknown, label: "message" | "title", maximum: number): string {
-  if (typeof value !== "string") throw new Error(`Notification ${label} must be a string`);
-  if (value.length === 0 || value.length > maximum || value.trim().length === 0 || value.includes("\0"))
-    throw new Error(`Notification ${label} must be non-blank text between 1 and ${maximum} characters without null bytes`);
-  return value;
-}
-
-function normalizeTimeout(value: unknown): number {
-  if (typeof value !== "number" || !Number.isFinite(value)) return defaultTimeoutMs;
-  return Math.max(100, Math.min(60_000, Math.trunc(value)));
-}
-
-function boundedReason(value: unknown): string {
-  const reason = value instanceof Error ? value.message : String(value);
-  return reason.length <= maxReasonLength ? reason : `${reason.slice(0, maxReasonLength - 1)}…`;
-}
-
-async function deliver(title: string, message: string, timeoutMs: number, signal: AbortSignal): Promise<NotificationResult> {
-  const platform = process.platform;
-  const options = { timeout: timeoutMs, maxBuffer: maxCommandOutputBytes, signal };
-  if (platform === "darwin") {
-    await execFileAsync("osascript", ["-e", `display notification ${appleScriptString(message)} with title ${appleScriptString(title)}`], options);
-    return { title, message, delivered: true, platform };
-  }
-  if (platform === "linux") {
-    await execFileAsync("notify-send", ["--", title, message], options);
-    return { title, message, delivered: true, platform };
-  }
-  if (platform === "win32") {
-    await execFileAsync("powershell.exe", ["-NoProfile", "-NonInteractive", "-Command", windowsNotificationScript], {
-      ...options,
-      env: { ...process.env, PI_HARNESS_NOTIFICATION_MESSAGE: message, PI_HARNESS_NOTIFICATION_TITLE: title },
-    });
-    return { title, message, delivered: true, platform };
-  }
-  return { title, message, delivered: false, platform, reason: "unsupported-platform" };
-}
-
-export default {
-  name: "pi-cli-notifier",
-  inject: ["piPluginUi", "piTools"],
-  Config,
-  apply(context: Context, config: CliNotifierPluginConfig = {}) {
-    if (config.enabled !== undefined && typeof config.enabled !== "boolean") throw new Error("CLI Notifier enabled must be a boolean");
-    const enabled = config.enabled !== false;
-    const configuredTitle = config.title === undefined || config.title.trim() === "" ? "Pi Harness" : config.title.trim();
-    const defaultTitle = normalizeText(configuredTitle, "title", maxTitleLength);
-    const timeoutMs = normalizeTimeout(config.timeoutMs);
-    const lifecycle = new AbortController();
-    let deliveryQueue: Promise<void> = Promise.resolve();
-    const notifications: Notification[] = [];
-    const record = (result: NotificationResult): NotificationResult => {
-      notifications.unshift({ time: new Date().toISOString(), ...result });
-      notifications.splice(maxNotifications);
-      return result;
-    };
-    const notify = async (messageValue: unknown, titleValue: unknown = defaultTitle, signal = lifecycle.signal): Promise<NotificationResult> => {
-      const message = normalizeText(messageValue, "message", maxMessageLength);
-      const title = normalizeText(titleValue, "title", maxTitleLength);
-      throwIfAborted(signal);
-      if (!enabled) {
-        return record({ title, message, delivered: false, platform: process.platform, reason: "disabled" });
-      }
-      const operation = deliveryQueue.then(async () => {
-        throwIfAborted(signal);
-        try {
-          return record(await deliver(title, message, timeoutMs, signal));
-        } catch (error) {
-          throwIfAborted(signal);
-          const timedOut = typeof error === "object" && error !== null && "killed" in error && (error as { killed?: unknown }).killed === true;
-          const reason = timedOut ? `Notification command timed out after ${timeoutMs} ms` : boundedReason(error);
-          return record({ title, message, delivered: false, platform: process.platform, reason });
-        }
-      });
-      deliveryQueue = operation.then(
-        () => undefined,
-        () => undefined,
-      );
-      return operation;
-    };
-    const unregisterTool = context.piTools.register(
-      defineTool({
-        name: "cli_notify",
-        label: "CLI notify",
-        description: "Send a local desktop notification without invoking a shell.",
-        promptSnippet: "send a desktop notification when this task finishes",
-        parameters: Type.Object(
-          {
-            message: Type.String({ description: "Notification body", minLength: 1, maxLength: maxMessageLength }),
-            title: Type.Optional(Type.String({ description: "Notification title", minLength: 1, maxLength: maxTitleLength })),
-          },
-          { additionalProperties: false },
-        ),
-        executionMode: "sequential",
-        async execute(_toolCallId, params, signal): Promise<AgentToolResult<NotificationResult>> {
-          const record = params !== null && typeof params === "object" && !Array.isArray(params) ? (params as Record<string, unknown>) : {};
-          const actionSignal = signal === undefined ? lifecycle.signal : AbortSignal.any([signal, lifecycle.signal]);
-          const result = await notify(record.message, record.title ?? defaultTitle, actionSignal);
-          return {
-            content: [
-              { type: "text", text: result.delivered ? "Desktop notification sent." : `Desktop notification not sent: ${result.reason ?? "unknown reason"}` },
-            ],
-            details: structuredClone(result),
-          };
-        },
-      }),
-    );
-    const unsubscribeSession = context.on("pi/session-event", (event) => {
-      const current = event as {
-        type?: string;
-        messages?: readonly { role?: string; stopReason?: string; errorMessage?: string }[];
-        errorMessage?: string;
-        willRetry?: boolean;
-      };
-      if (current.type === "agent_end" && current.willRetry !== true) {
-        const last = current.messages?.at(-1);
-        if (last?.role === "assistant") {
-          const message =
-            last.stopReason === "error"
-              ? `Agent failed: ${last.errorMessage ?? "unknown error"}`
-              : last.stopReason === "aborted"
-                ? "Agent turn aborted."
-                : last.stopReason === "length"
-                  ? "Agent turn stopped at the model output limit."
-                  : "Agent turn completed.";
-          void notify(boundedEventMessage(message), defaultTitle, lifecycle.signal).catch(() => undefined);
-        }
-      } else if (current.type === "compaction_end" && current.errorMessage) {
-        void notify(boundedEventMessage(`Context compaction failed: ${current.errorMessage}`), defaultTitle, lifecycle.signal).catch(() => undefined);
-      }
-    });
-    const disposePanel = context.piPluginUi.register({
-      id: "cli-notifier-panel",
-      pluginId: "@pi-harness/core/plugins/cli-notifier",
-      title: "CLI Notifier",
-      description: "长任务完成后发送本机桌面通知。",
-      icon: "♢",
-      read: () => ({ enabled, platform: process.platform, timeoutMs, notifications: structuredClone(notifications) }),
-    });
-    context.effect(() => () => {
-      lifecycle.abort(new Error("CLI Notifier plugin disposed"));
-      unregisterTool();
-      unsubscribeSession();
-      disposePanel();
-    });
-  },
-};

package/packages/core/src/plugins/code2skill.ts

@@ -1,237 +0,0 @@
-import { lstat, mkdir, mkdtemp, rename, rm, writeFile } from "node:fs/promises";
-import { dirname, isAbsolute, join, sep } from "node:path";
-import type { Context } from "@deepseek-ai/cordis";
-import { Type } from "@earendil-works/pi-ai";
-import { defineTool, type AgentToolResult } from "@earendil-works/pi-coding-agent";
-import { readBoundedFile } from "../bounded-file.js";
-import { prepareWorkspaceFile, resolveExistingWorkspacePath } from "../workspace-path.js";
-import { EmptyConfig } from "../config.js";
-
-const maxFiles = 32;
-const maxNameLength = 128;
-const maxDescriptionLength = 1024;
-const maxPathLength = 4096;
-const maxFileBytes = 256 * 1024;
-const maxTotalBytes = 2 * 1024 * 1024;
-const parameterNames = new Set(["name", "description", "files"]);
-
-type SkillFile = { path: string; bytes: number };
-type SkillReport = { slug: string; directory: string; files: SkillFile[]; bytes: number };
-
-function slugify(name: string): string {
-  const slug = name
-    .trim()
-    .toLowerCase()
-    .replace(/[^a-z0-9]+/gu, "-")
-    .replace(/^-+|-+$/gu, "")
-    .slice(0, 64);
-  if (slug.length === 0) throw new Error("Skill name must contain at least one ASCII letter or number");
-  return slug;
-}
-
-function throwIfAborted(signal: AbortSignal): void {
-  if (!signal.aborted) return;
-  throw signal.reason instanceof Error ? signal.reason : new Error("Skill creation was cancelled", { cause: signal.reason });
-}
-
-function hasControlCharacters(value: string): boolean {
-  for (const character of value) {
-    const codePoint = character.codePointAt(0)!;
-    if (codePoint <= 31 || codePoint === 127) return true;
-  }
-  return false;
-}
-
-function parseParams(value: unknown): { name: string; description: string; files: string[] } {
-  if (value === null || typeof value !== "object" || Array.isArray(value)) throw new Error("Skill parameters must be an object");
-  let descriptors: PropertyDescriptorMap;
-  try {
-    const prototype = Object.getPrototypeOf(value) as unknown;
-    if (prototype !== Object.prototype && prototype !== null) throw new Error("Skill parameters must be a plain object");
-    descriptors = Object.getOwnPropertyDescriptors(value);
-  } catch (error) {
-    if (error instanceof Error && error.message === "Skill parameters must be a plain object") throw error;
-    throw new Error("Skill parameters must be an accessible plain object", { cause: error });
-  }
-  if (Reflect.ownKeys(descriptors).some((key) => typeof key !== "string" || !parameterNames.has(key)))
-    throw new Error("Skill parameters contain an unknown property");
-  if (Object.values(descriptors).some((descriptor) => !("value" in descriptor))) throw new Error("Skill parameters must use data properties");
-  const name: unknown = descriptors.name?.value;
-  const description: unknown = descriptors.description?.value;
-  const rawFiles: unknown = descriptors.files?.value;
-  if (typeof name !== "string" || name.trim().length === 0 || name.length > maxNameLength || name.includes("\0"))
-    throw new Error(`Skill name must be non-blank text between 1 and ${maxNameLength} characters`);
-  if (typeof description !== "string" || description.trim().length === 0 || description.length > maxDescriptionLength || description.includes("\0"))
-    throw new Error(`Skill description must be non-blank text between 1 and ${maxDescriptionLength} characters`);
-  if (!Array.isArray(rawFiles)) throw new Error(`Skill files must contain between 1 and ${maxFiles} source paths`);
-  let fileDescriptors: Record<PropertyKey, PropertyDescriptor>;
-  try {
-    if (Object.getPrototypeOf(rawFiles) !== Array.prototype) throw new Error("Skill files must be a plain array");
-    fileDescriptors = Object.getOwnPropertyDescriptors(rawFiles) as unknown as Record<PropertyKey, PropertyDescriptor>;
-  } catch (error) {
-    if (error instanceof Error && error.message === "Skill files must be a plain array") throw error;
-    throw new Error("Skill files must be an accessible plain array", { cause: error });
-  }
-  const length: unknown = fileDescriptors.length?.value;
-  if (typeof length !== "number" || !Number.isSafeInteger(length) || length === 0 || length > maxFiles)
-    throw new Error(`Skill files must contain between 1 and ${maxFiles} source paths`);
-  if (Reflect.ownKeys(fileDescriptors).some((key) => typeof key !== "string" || (key !== "length" && !/^(0|[1-9]\d*)$/u.test(key))))
-    throw new Error("Skill files contain an unknown property");
-  const files: string[] = [];
-  for (let index = 0; index < length; index += 1) {
-    const descriptor = fileDescriptors[String(index)];
-    if (descriptor === undefined || !("value" in descriptor)) throw new Error("Skill files must be a dense array of data properties");
-    const file: unknown = descriptor.value;
-    if (typeof file !== "string" || file.trim().length === 0 || file.length > maxPathLength || hasControlCharacters(file))
-      throw new Error(`Each skill file must be a non-empty relative path of at most ${maxPathLength} characters`);
-    files.push(file);
-  }
-  return { name, description, files };
-}
-
-function markdownLabel(value: string): string {
-  return value.replaceAll("\\", "\\\\").replaceAll("[", "\\[").replaceAll("]", "\\]");
-}
-
-function markdownTarget(value: string): string {
-  return value
-    .split("/")
-    .map((component) => encodeURIComponent(component).replace(/[!'()*]/gu, (character) => `%${character.codePointAt(0)!.toString(16).toUpperCase()}`))
-    .join("/");
-}
-
-function isMissing(error: unknown): boolean {
-  return typeof error === "object" && error !== null && "code" in error && (error as { code?: unknown }).code === "ENOENT";
-}
-
-async function assertSkillTargetAvailable(directory: string, slug: string, boundaryError: string): Promise<void> {
-  try {
-    const metadata = await lstat(directory);
-    if (metadata.isSymbolicLink()) throw new Error(boundaryError);
-    throw new Error(`Skill pack already exists: ${slug}`);
-  } catch (error) {
-    if (!isMissing(error)) throw error;
-  }
-}
-
-async function resolveWorkspaceFile(cwd: string, input: string): Promise<{ absolute: string; display: string }> {
-  if (typeof input !== "string" || input.trim() === "" || isAbsolute(input)) throw new Error("Skill files must be non-empty relative paths");
-  const resolved = await resolveExistingWorkspacePath(cwd, input, `Skill file must stay inside the workspace: ${input}`);
-  if (resolved.relativePath === ".") throw new Error(`Skill file must stay inside the workspace: ${input}`);
-  return { absolute: resolved.target, display: resolved.relativePath.split(sep).join("/") };
-}
-
-async function createSkill(context: Context, params: { name: string; description: string; files: string[] }, signal: AbortSignal): Promise<SkillReport> {
-  throwIfAborted(signal);
-  const cwd = context.piHarnessLaunch.cwd;
-  const displayName = params.name.trim().replace(/\s+/gu, " ");
-  const slug = slugify(displayName);
-  const description = params.description.trim();
-  const sources = [];
-  for (const file of params.files) {
-    throwIfAborted(signal);
-    sources.push(await resolveWorkspaceFile(cwd, file));
-  }
-  const unique = new Set(sources.map((source) => source.display));
-  if (unique.size !== sources.length) throw new Error("Skill files must be unique");
-  let bytes = 0;
-  const loaded: Array<SkillFile & { content: Buffer }> = [];
-  for (const source of sources) {
-    throwIfAborted(signal);
-    const content = await readBoundedFile(source.absolute, maxFileBytes, `Skill file ${source.display}`);
-    bytes += content.byteLength;
-    if (bytes > maxTotalBytes) throw new Error(`Skill sources exceed ${maxTotalBytes} bytes`);
-    loaded.push({ path: source.display, bytes: content.byteLength, content });
-  }
-  const boundaryError = "Skill output path must stay inside the workspace and must not overwrite an existing skill";
-  const boundary = await prepareWorkspaceFile(cwd, join(".pi", "skills", ".code2skill-boundary"), boundaryError);
-  const skillsDirectory = dirname(boundary.target);
-  const finalDirectory = join(skillsDirectory, slug);
-  await assertSkillTargetAvailable(finalDirectory, slug, boundaryError);
-  const files = loaded.map((file) => ({ path: file.path, bytes: file.bytes }));
-  const references = files.map((file) => `- [${markdownLabel(file.path)}](references/${markdownTarget(file.path)})`).join("\n");
-  const frontmatterDescription = description.replace(/\s+/gu, " ");
-  const markdown = `---\nname: ${slug}\ndescription: ${JSON.stringify(frontmatterDescription)}\n---\n\n# ${displayName}\n\n${description}\n\n## Reference files\n\n${references}\n`;
-  const stagingDirectory = await mkdtemp(join(skillsDirectory, ".code2skill-"));
-  try {
-    for (const [index, file] of loaded.entries()) {
-      throwIfAborted(signal);
-      const target = join(stagingDirectory, "references", ...file.path.split("/"));
-      await mkdir(dirname(target), { recursive: true });
-      await writeFile(target, loaded[index]!.content, { flag: "wx", mode: 0o600 });
-    }
-    throwIfAborted(signal);
-    await writeFile(join(stagingDirectory, "SKILL.md"), markdown, { encoding: "utf8", flag: "wx", mode: 0o600 });
-    throwIfAborted(signal);
-    await assertSkillTargetAvailable(finalDirectory, slug, boundaryError);
-    await rename(stagingDirectory, finalDirectory);
-  } catch (error) {
-    await rm(stagingDirectory, { recursive: true, force: true });
-    throw error;
-  }
-  return { slug, directory: join(".pi", "skills", slug), files, bytes };
-}
-
-export default {
-  name: "pi-code2skill",
-  inject: ["piHarnessLaunch", "piPluginUi", "piTools"],
-  Config: EmptyConfig,
-  apply(context: Context) {
-    const lifecycle = new AbortController();
-    let creationQueue: Promise<void> = Promise.resolve();
-    let generated = 0;
-    let latest: SkillReport | undefined;
-    const unregisterTool = context.piTools.register(
-      defineTool({
-        name: "skill_pack_create",
-        label: "Create skill pack",
-        description: "Package selected workspace source files into a local .pi/skills skill with a manifest and references.",
-        promptSnippet: "turn selected project files into a reusable skill pack",
-        parameters: Type.Object(
-          {
-            name: Type.String({ minLength: 1, maxLength: maxNameLength }),
-            description: Type.String({ minLength: 1, maxLength: maxDescriptionLength }),
-            files: Type.Array(Type.String({ minLength: 1, maxLength: maxPathLength, pattern: "^[^\\u0000-\\u001F\\u007F]+$" }), {
-              minItems: 1,
-              maxItems: maxFiles,
-            }),
-          },
-          { additionalProperties: false },
-        ),
-        executionMode: "sequential",
-        async execute(_toolCallId, params, signal): Promise<AgentToolResult<SkillReport>> {
-          const validated = parseParams(params);
-          const actionSignal = signal === undefined ? lifecycle.signal : AbortSignal.any([signal, lifecycle.signal]);
-          throwIfAborted(actionSignal);
-          const operation = creationQueue.then(async () => {
-            const report = await createSkill(context, validated, actionSignal);
-            latest = report;
-            generated += 1;
-            return {
-              content: [{ type: "text" as const, text: `Skill pack created: ${report.slug} (${report.files.length} file(s)).` }],
-              details: structuredClone(report),
-            };
-          });
-          creationQueue = operation.then(
-            () => undefined,
-            () => undefined,
-          );
-          return operation;
-        },
-      }),
-    );
-    const disposePanel = context.piPluginUi.register({
-      id: "code2skill-panel",
-      pluginId: "@pi-harness/core/plugins/code2skill",
-      title: "Code2Skill",
-      description: "把工作区代码打包为可复用的 Pi Skill，保留来源文件并生成 SKILL.md。",
-      icon: "✦",
-      read: () => ({ generated, latest: latest === undefined ? null : structuredClone(latest) }),
-    });
-    context.effect(() => () => {
-      lifecycle.abort(new Error("Code2Skill plugin disposed"));
-      unregisterTool();
-      disposePanel();
-    });
-  },
-};

package/packages/core/src/plugins/colleague-skill.ts

@@ -1,147 +0,0 @@
-import { randomUUID } from "node:crypto";
-import type { Context } from "@deepseek-ai/cordis";
-import { Type } from "@earendil-works/pi-ai";
-import { defineTool, type AgentToolResult } from "@earendil-works/pi-coding-agent";
-import { EmptyConfig } from "../config.js";
-
-const customType = "pi-harness/colleague-handoff";
-const maxRoleLength = 128;
-const maxObjectiveLength = 4_000;
-const maxContextLength = 4_000;
-const maxListTextLength = 1_000;
-const maxFileLength = 4_096;
-const maxListItems = 20;
-
-export interface ColleagueHandoffInput {
-  toRole: string;
-  objective: string;
-  context?: string;
-  constraints?: string[];
-  files?: string[];
-  acceptance?: string[];
-}
-
-export interface ColleagueHandoff {
-  id: string;
-  toRole: string;
-  objective: string;
-  context: string;
-  constraints: string[];
-  files: string[];
-  acceptance: string[];
-  createdAt: string;
-}
-
-function boundedText(value: unknown, field: string, maximum: number, required = false): string {
-  if (value === undefined && !required) return "";
-  if (typeof value !== "string") throw new Error(`${field} must be a string`);
-  const text = value.trim();
-  if (required && text.length === 0) throw new Error(`${field} is required`);
-  if (text.length > maximum) throw new Error(`${field} must be ${maximum} characters or fewer`);
-  if (text.includes("\0")) throw new Error(`${field} must not contain null bytes`);
-  return text;
-}
-
-function boundedList(values: unknown, field: string, itemMaximum: number): string[] {
-  if (values === undefined) return [];
-  if (!Array.isArray(values)) throw new Error(`${field} must be an array`);
-  if (values.length > maxListItems) throw new Error(`${field} must contain ${maxListItems} items or fewer`);
-  return values.map((value) => boundedText(value, `${field} item`, itemMaximum, true));
-}
-
-export function createColleagueHandoff(input: unknown, id: string, createdAt: string): ColleagueHandoff {
-  if (input === null || typeof input !== "object" || Array.isArray(input)) throw new Error("Handoff input must be an object");
-  const value = input as Record<string, unknown>;
-  const toRole = boundedText(value.toRole, "toRole", maxRoleLength, true);
-  const objective = boundedText(value.objective, "objective", maxObjectiveLength, true);
-  const validatedId = boundedText(id, "handoff id", 128, true);
-  const validatedCreatedAt = boundedText(createdAt, "createdAt", 64, true);
-  if (!Number.isFinite(Date.parse(validatedCreatedAt))) throw new Error("createdAt must be a valid timestamp");
-  return {
-    id: validatedId,
-    toRole,
-    objective,
-    context: boundedText(value.context, "context", maxContextLength),
-    constraints: boundedList(value.constraints, "constraints", maxListTextLength),
-    files: boundedList(value.files, "files", maxFileLength),
-    acceptance: boundedList(value.acceptance, "acceptance", maxListTextLength),
-    createdAt: validatedCreatedAt,
-  };
-}
-
-function readLatest(context: Context): ColleagueHandoff | undefined {
-  const entries = context.piSession.manager.getEntries();
-  for (let index = entries.length - 1; index >= 0; index -= 1) {
-    const entry = entries[index];
-    if (entry?.type !== "custom" || entry.customType !== customType || entry.data === null || typeof entry.data !== "object") continue;
-    const value = entry.data as Record<string, unknown>;
-    if (typeof value.id !== "string" || typeof value.createdAt !== "string") continue;
-    try {
-      return createColleagueHandoff(value, value.id, value.createdAt);
-    } catch {
-      continue;
-    }
-  }
-  return undefined;
-}
-
-function throwIfAborted(signal: AbortSignal): void {
-  if (!signal.aborted) return;
-  throw signal.reason instanceof Error ? signal.reason : new Error("Colleague handoff was cancelled", { cause: signal.reason });
-}
-
-export default {
-  name: "pi-colleague-skill",
-  inject: ["piSession", "piPluginUi", "piTools"],
-  Config: EmptyConfig,
-  apply(context: Context) {
-    const lifecycle = new AbortController();
-    let latest = readLatest(context);
-    const unregister = context.piTools.register(
-      defineTool({
-        name: "colleague_handoff",
-        label: "Colleague handoff",
-        description: "Create a durable, structured handoff packet for another role without starting a hidden agent or sending external messages.",
-        promptSnippet: "prepare a structured handoff for a colleague role",
-        parameters: Type.Object(
-          {
-            toRole: Type.String({ description: "Receiving role, for example reviewer or frontend", minLength: 1, maxLength: maxRoleLength }),
-            objective: Type.String({ description: "The concrete outcome the colleague should deliver", minLength: 1, maxLength: maxObjectiveLength }),
-            context: Type.Optional(Type.String({ maxLength: maxContextLength })),
-            constraints: Type.Optional(Type.Array(Type.String({ minLength: 1, maxLength: maxListTextLength }), { maxItems: maxListItems })),
-            files: Type.Optional(Type.Array(Type.String({ minLength: 1, maxLength: maxFileLength }), { maxItems: maxListItems })),
-            acceptance: Type.Optional(Type.Array(Type.String({ minLength: 1, maxLength: maxListTextLength }), { maxItems: maxListItems })),
-          },
-          { additionalProperties: false },
-        ),
-        executionMode: "sequential",
-        execute(_toolCallId, params, signal): Promise<AgentToolResult<ColleagueHandoff>> {
-          return Promise.resolve().then(() => {
-            const actionSignal = signal === undefined ? lifecycle.signal : AbortSignal.any([signal, lifecycle.signal]);
-            throwIfAborted(actionSignal);
-            const handoff = createColleagueHandoff(params, `handoff-${Date.now()}-${randomUUID().slice(0, 8)}`, new Date().toISOString());
-            context.piSession.manager.appendCustomEntry(customType, structuredClone(handoff));
-            latest = handoff;
-            return {
-              content: [{ type: "text" as const, text: `Handoff ${handoff.id} prepared for ${handoff.toRole}.` }],
-              details: structuredClone(handoff),
-            };
-          });
-        },
-      }),
-    );
-    const disposePanel = context.piPluginUi.register({
-      id: "colleague-skill-panel",
-      pluginId: "@pi-harness/core/plugins/colleague-skill",
-      title: "Colleague Skill",
-      description: "把任务、上下文、约束和验收条件整理成可追踪的角色交接包。",
-      icon: "⇄",
-      read: () => ({ latest: latest === undefined ? null : structuredClone(latest) }),
-    });
-    context.effect(() => () => {
-      lifecycle.abort(new Error("Colleague Skill plugin disposed"));
-      unregister();
-      disposePanel();
-    });
-  },
-};