pi-crew 0.9.51 → 0.9.52

package/src/schema/team-tool-schema.ts

@@ -1,415 +0,0 @@
-import { Type } from "@sinclair/typebox";
-
-const SkillOverride = Type.Unsafe({
-	description:
-		"Skill name(s) to add to role/default skills, an array of skill names, or false to disable all injected skills for this run.",
-	anyOf: [
-		{ type: "string", maxLength: 2048 },
-		{
-			type: "array",
-			maxItems: 32,
-			items: { type: "string", maxLength: 80 },
-		},
-		{ type: "boolean" },
-	],
-});
-
-const FreeformConfig = Type.Unsafe({
-	description: "Resource config for management actions.",
-	type: "object",
-	additionalProperties: true,
-});
-
-export const TeamToolParams = Type.Object({
-	action: Type.Optional(
-		Type.Union(
-			[
-				Type.Literal("run"),
-				Type.Literal("parallel"),
-				Type.Literal("plan"),
-				Type.Literal("status"),
-				Type.Literal("wait"),
-				Type.Literal("list"),
-				Type.Literal("get"),
-				Type.Literal("cancel"),
-				Type.Literal("retry"),
-				Type.Literal("resume"),
-				Type.Literal("respond"),
-				Type.Literal("create"),
-				Type.Literal("update"),
-				Type.Literal("delete"),
-				Type.Literal("doctor"),
-				Type.Literal("cleanup"),
-				Type.Literal("events"),
-				Type.Literal("artifacts"),
-				Type.Literal("worktrees"),
-				Type.Literal("forget"),
-				Type.Literal("summary"),
-				Type.Literal("prune"),
-				Type.Literal("export"),
-				Type.Literal("import"),
-				Type.Literal("imports"),
-				Type.Literal("help"),
-				Type.Literal("validate"),
-				Type.Literal("config"),
-				Type.Literal("init"),
-				Type.Literal("recommend"),
-				Type.Literal("autonomy"),
-				Type.Literal("api"),
-				Type.Literal("settings"),
-				Type.Literal("steer"),
-				Type.Literal("invalidate"),
-				Type.Literal("health"),
-				Type.Literal("graph"),
-				Type.Literal("onboard"),
-				Type.Literal("explain"),
-				Type.Literal("cache"),
-				Type.Literal("checkpoint"),
-				Type.Literal("search"),
-				Type.Literal("orchestrate"),
-				Type.Literal("schedule"),
-				Type.Literal("scheduled"),
-				Type.Literal("anchor"),
-				Type.Literal("auto-summarize"),
-				Type.Literal("auto_boomerang"),
-				Type.Literal("goal"),
-				Type.Literal("workflow-create"),
-				Type.Literal("workflow-get"),
-				Type.Literal("workflow-list"),
-				Type.Literal("workflow-save"),
-				Type.Literal("workflow-delete"),
-			],
-			{ description: "Team action. Defaults to 'list' when omitted." },
-		),
-	),
-	resource: Type.Optional(
-		Type.Union([Type.Literal("agent"), Type.Literal("team"), Type.Literal("workflow")], {
-			description: "Resource kind for get/create/update/delete/list. Defaults to all for list.",
-		}),
-	),
-	team: Type.Optional(
-		Type.String({
-			description: "Team name, e.g. default or implementation.",
-		}),
-	),
-	workflow: Type.Optional(Type.String({ description: "Workflow name, e.g. default or review." })),
-	role: Type.Optional(
-		Type.String({
-			description: "Role name to run directly within a team.",
-		}),
-	),
-	agent: Type.Optional(Type.String({ description: "Agent name to inspect or run directly." })),
-	goal: Type.Optional(Type.String({ description: "High-level objective for a team run." })),
-	chain: Type.Optional(
-		Type.String({
-			description:
-				'Chain expression: "step1 -> step2 -> step3". Runs each step as a sequential team run, passing handoff context forward via the goal text. Supports inline goals ("...") and @team references. e.g. chain=\'"Research X" -> "Analyze" -> "Write report"\'.',
-		}),
-	),
-	task: Type.Optional(
-		Type.String({
-			description: "Concrete task text for direct role/agent execution.",
-		}),
-	),
-	singleAgent: Type.Optional(
-		Type.Boolean({
-			description:
-				"When true (with action=plan), compose a single-agent sequential prompt for the workflow instead of a multi-agent plan. Cliff-resilient mode.",
-		}),
-	),
-	runId: Type.Optional(
-		Type.String({
-			description: "Run ID for status, cancel, or resume.",
-			pattern: "^[A-Za-z0-9_-]+$",
-		}),
-	),
-	taskId: Type.Optional(Type.String({ description: "Task ID for respond action." })),
-	message: Type.Optional(Type.String({ description: "Message for respond action." })),
-	async: Type.Optional(
-		Type.Boolean({
-			description: "Run in background when execution support is enabled.",
-		}),
-	),
-	details: Type.Optional(
-		Type.Boolean({
-			default: true,
-			description:
-				"(status) Output detail level. true (default) = full status (task graph, agents, effectiveness, events). false = compact summary (status, goal, task counts, and only failed/attention task errors) for quick checks.",
-		}),
-	),
-	workspaceMode: Type.Optional(
-		Type.Union([Type.Literal("single"), Type.Literal("worktree")], {
-			description: "Workspace isolation mode. Worktree mode is planned after MVP.",
-		}),
-	),
-	context: Type.Optional(
-		Type.Union([Type.Literal("fresh"), Type.Literal("fork")], {
-			description: "Child context mode for workers.",
-		}),
-	),
-	cwd: Type.Optional(Type.String({ description: "Working directory override." })),
-	model: Type.Optional(Type.String({ description: "Model override for direct runs." })),
-	skill: Type.Optional(SkillOverride),
-	scope: Type.Optional(
-		Type.Union([Type.Literal("user"), Type.Literal("project"), Type.Literal("both")], {
-			description: "Resource scope for discovery or management.",
-		}),
-	),
-	config: Type.Optional(FreeformConfig),
-	dryRun: Type.Optional(
-		Type.Boolean({
-			description: "Preview a management mutation without writing files.",
-		}),
-	),
-	confirm: Type.Optional(
-		Type.Boolean({
-			description: "Required for destructive management actions.",
-		}),
-	),
-	force: Type.Optional(
-		Type.Boolean({
-			description: "Override reference checks for destructive management actions.",
-		}),
-	),
-	keep: Type.Optional(
-		Type.Integer({
-			minimum: 0,
-			description: "Number of finished runs to keep for prune.",
-		}),
-	),
-	updateReferences: Type.Optional(
-		Type.Boolean({
-			description: "When renaming agents or workflows, update team references in the same project/user scope.",
-		}),
-	),
-	replyTo: Type.Optional(
-		Type.String({
-			description: "ID of the original mailbox message this is a reply to.",
-		}),
-	),
-	replyFrom: Type.Optional(Type.String({ description: "Task ID sending the reply." })),
-	replyDeadline: Type.Optional(Type.Integer({ description: "Ms epoch deadline for a reply." })),
-	planPath: Type.Optional(
-		Type.String({
-			description: "Path to a markdown plan document for orchestration.",
-		}),
-	),
-	subAction: Type.Optional(Type.String({ description: "Sub-action for schedule management (remove, disable, enable, update)." })),
-	jobId: Type.Optional(Type.String({ description: "Job ID for schedule management actions." })),
-	cron: Type.Optional(
-		Type.String({
-			description: "Cron expression for recurring scheduled runs (e.g., '0 9 * * MON').",
-		}),
-	),
-	interval: Type.Optional(
-		Type.Number({
-			description: "Interval in milliseconds between recurring scheduled runs.",
-		}),
-	),
-	once: Type.Optional(
-		Type.Union([Type.String(), Type.Number()], {
-			description: "ISO timestamp or epoch ms for a one-time scheduled run.",
-		}),
-	),
-	excludeContextBash: Type.Optional(
-		Type.Boolean({
-			description: "Mark certain bash commands as excludeFromContext to reduce context tokens (default: false).",
-		}),
-	),
-	// Budget tracking options
-	budgetTotal: Type.Optional(
-		Type.Number({
-			description:
-				"Total token budget for the run. When set, enables budget tracking with default 80% warning and 95% abort thresholds. Minimum 1000 — this is a MISCONFIGURATION GUARD (catches typos / silent-abort configs like budgetTotal:1, which would abort on turn 1), NOT a usefulness guarantee; a productive multi-turn goal needs far more than 1000 tokens.",
-			minimum: 1000,
-		}),
-	),
-	budgetUnlimited: Type.Optional(
-		Type.Boolean({
-			description:
-				"When true, skip budget enforcement entirely (explicit opt-out). Goal-start validation requires budgetTotal>=1000 OR budgetUnlimited:true; audit-logged when set. The validation itself is enforced in a later integration task.",
-		}),
-	),
-	budgetWarning: Type.Optional(
-		Type.Number({
-			description:
-				"Budget warning threshold as a fraction (0-1). Default: 0.8 (80%). Emits warning event when this threshold is crossed.",
-			minimum: 0,
-			maximum: 1,
-		}),
-	),
-	budgetAbort: Type.Optional(
-		Type.Number({
-			description:
-				"Budget abort threshold as a fraction (0-1). Default: 0.95 (95%). Aborts further execution when this threshold is crossed.",
-			minimum: 0,
-			maximum: 1,
-		}),
-	),
-	runKind: Type.Optional(
-		Type.Union([Type.Literal("team-run"), Type.Literal("goal-loop"), Type.Literal("dynamic-workflow")], {
-			description:
-				'Background dispatch discriminator. Default "team-run" runs the normal executeTeamRun workflow; "goal-loop" (P0/P1) and "dynamic-workflow" (P2/P3) dispatch to their respective background runners. Absent = "team-run" for backward compatibility.',
-		}),
-	),
-	tokenBudget: Type.Optional(
-		Type.Number({
-			description:
-				"Per-workflow token budget for dynamic-workflow runs. When set, ctx.agent() auto-rejects with ok:false once exhausted. Accumulated from each agent run's reported usage. Overrides workflow.maxTokenBudget.",
-			minimum: 0,
-		}),
-	),
-	args: Type.Optional(
-		// round-14 P1-5: typed workflow arguments. Type.Any() generates an empty {} schema
-		// (matches any JSON value) which is strict-provider friendly — no array type union.
-		// Description lives in the JSDoc / TeamToolParamsValue below to avoid the
-		// "description-only schema" strict-provider check.
-		Type.Any(),
-	),
-	analysis: Type.Optional(
-		Type.String({
-			maxLength: 100_000,
-			description:
-				"Inline analysis/context notes from the calling session. Persisted to artifacts/{runId}/shared/analysis.md (audit trail) and auto-injected into any workflow step declaring reads: analysis.md (e.g. builtin 'plan-execute'). Mutually exclusive with analysisPath. Ignored by goal-wrapped runs and chain dispatch in v1.",
-		}),
-	),
-	analysisPath: Type.Optional(
-		Type.String({
-			description:
-				"Path to an existing markdown analysis file (resolved within cwd). Copied into shared/analysis.md. Mutually exclusive with analysis.",
-		}),
-	),
-	focus: Type.Optional(
-		Type.String({
-			description:
-				"Sub-focus for the doctor action. 'zombies' runs a READ-ONLY scan for orphaned pi-crew sub-agent processes (identified by PI_CREW_KIND=subagent); it never kills and never matches the user's interactive main session.",
-		}),
-	),
-});
-
-export interface TeamToolParamsValue {
-	action?:
-		| "run"
-		| "parallel"
-		| "plan"
-		| "status"
-		| "wait"
-		| "list"
-		| "get"
-		| "cancel"
-		| "retry"
-		| "resume"
-		| "respond"
-		| "create"
-		| "update"
-		| "delete"
-		| "doctor"
-		| "cleanup"
-		| "events"
-		| "artifacts"
-		| "worktrees"
-		| "forget"
-		| "summary"
-		| "prune"
-		| "export"
-		| "import"
-		| "imports"
-		| "help"
-		| "validate"
-		| "config"
-		| "init"
-		| "recommend"
-		| "autonomy"
-		| "api"
-		| "settings"
-		| "steer"
-		| "invalidate"
-		| "health"
-		| "graph"
-		| "onboard"
-		| "explain"
-		| "cache"
-		| "checkpoint"
-		| "search"
-		| "orchestrate"
-		| "schedule"
-		| "scheduled"
-		| "goal"
-		| "workflow-create"
-		| "workflow-get"
-		| "workflow-list"
-		| "workflow-save"
-		| "workflow-delete";
-	resource?: "agent" | "team" | "workflow";
-	team?: string;
-	workflow?: string;
-	role?: string;
-	agent?: string;
-	goal?: string;
-	/** Chain expression: "step1 -> step2 -> step3". Runs each step as a sequential
-	 *  team run with handoff context passed forward via the goal. */
-	chain?: string;
-	task?: string;
-	singleAgent?: boolean;
-	runId?: string;
-	taskId?: string;
-	message?: string;
-	async?: boolean;
-	/** (status) Output detail level. false = compact summary. Default: true (full). */
-	details?: boolean;
-	workspaceMode?: "single" | "worktree";
-	context?: "fresh" | "fork";
-	cwd?: string;
-	model?: string;
-	skill?: string | string[] | boolean;
-	scope?: "user" | "project" | "both";
-	config?: Record<string, unknown>;
-	/** Sub-focus for the `doctor` action. `"zombies"` runs a READ-ONLY scan for
-	 *  orphaned pi-crew sub-agent processes (identified by PI_CREW_KIND=subagent);
-	 *  it never kills and never matches the user's interactive main session. */
-	focus?: string;
-	dryRun?: boolean;
-	confirm?: boolean;
-	force?: boolean;
-	keep?: number;
-	updateReferences?: boolean;
-	/** ID of the original mailbox message this is a reply to. */
-	replyTo?: string;
-	/** Task ID sending the reply. */
-	replyFrom?: string;
-	/** Ms epoch deadline for a reply. */
-	replyDeadline?: number;
-	/** Path to a markdown plan document for orchestration. */
-	planPath?: string;
-	cron?: string;
-	interval?: number;
-	once?: string | number;
-
-	// schedule sub-actions (removal/toggle/update of an existing job)
-	subAction?: string;
-	jobId?: string;
-	/** Mark certain bash commands as excludeFromContext to reduce context tokens (default: false). */
-	excludeContextBash?: boolean;
-	/** Total token budget for the run. When set, enables budget tracking (minimum 1000). */
-	budgetTotal?: number;
-	/** When true, skip budget enforcement entirely (explicit opt-out). */
-	budgetUnlimited?: boolean;
-	/** Budget warning threshold as a fraction (0-1). Default: 0.8. */
-	budgetWarning?: number;
-	/** Budget abort threshold as a fraction (0-1). Default: 0.95. */
-	budgetAbort?: number;
-	/** Background dispatch discriminator. Default "team-run". "goal-loop"/"dynamic-workflow" dispatch to their runners (P0/P2). */
-	runKind?: "team-run" | "goal-loop" | "dynamic-workflow";
-	/** Per-workflow token budget for dynamic-workflow runs (round-14 P1-2). */
-	tokenBudget?: number;
-	/** Typed workflow arguments for .dwf.ts scripts, accessible via ctx.args<T>() (round-14 P1-5). */
-	args?: unknown;
-	/** Inline analysis/context notes from the calling session. Persisted to
-	 *  artifacts/{runId}/shared/analysis.md (audit trail) and auto-injected into
-	 *  any workflow step declaring reads: analysis.md (e.g. builtin 'plan-execute').
-	 *  Mutually exclusive with `analysisPath`. */
-	analysis?: string;
-	/** Path to an existing analysis file resolved within `cwd`. Copied into
-	 *  shared/analysis.md. Mutually exclusive with `analysis`. */
-	analysisPath?: string;
-}

package/src/schema/validation-types.ts

@@ -1,159 +0,0 @@
-/**
- * Validation severity levels for pi-crew config.
- *
- * - Required fields invalid → always ERROR
- * - Optional fields invalid → ERROR (strict) / WARNING (lenient)
- * - Missing recommended optional fields → INFO
- * - Values outside recommended ranges → WARNING
- */
-
-import { Value } from "@sinclair/typebox/value";
-import { PiTeamsConfigSchema } from "./config-schema.ts";
-
-// ---------------------------------------------------------------------------
-// Types
-// ---------------------------------------------------------------------------
-
-export type ValidationSeverity = "ERROR" | "WARNING" | "INFO";
-
-export type ValidationMode = "strict" | "lenient";
-
-export interface ValidationFinding {
-	severity: ValidationSeverity;
-	message: string;
-	field?: string;
-	suggestion?: string;
-}
-
-export interface ValidationOutcome {
-	findings: ValidationFinding[];
-	mode: ValidationMode;
-	hasErrors: boolean;
-	hasWarnings: boolean;
-}
-
-// ---------------------------------------------------------------------------
-// Constants
-// ---------------------------------------------------------------------------
-
-const DEFAULT_MODE: ValidationMode = "strict";
-
-/** Recommended range constraints (not enforced by schema, but advisory). */
-const RECOMMENDED_RANGES: Record<string, { max: number; label: string }> = {
-	"limits.maxConcurrentWorkers": {
-		max: 8,
-		label: "maxConcurrentWorkers > 8 may degrade performance",
-	},
-	"limits.maxTaskDepth": {
-		max: 4,
-		label: "maxTaskDepth > 4 may cause excessive nesting",
-	},
-};
-
-/** Recommended optional top-level keys that should be explicitly set. */
-const RECOMMENDED_OPTIONAL_KEYS: readonly string[] = ["limits", "runtime"];
-
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-
-function isObject(value: unknown): value is Record<string, unknown> {
-	return value !== null && typeof value === "object" && !Array.isArray(value);
-}
-
-function getNestedValue(obj: Record<string, unknown>, dottedPath: string): unknown {
-	const parts = dottedPath.split(".");
-	let current: unknown = obj;
-	for (const part of parts) {
-		if (!isObject(current)) return undefined;
-		current = current[part];
-	}
-	return current;
-}
-
-// ---------------------------------------------------------------------------
-// Implementation
-// ---------------------------------------------------------------------------
-
-/**
- * Validate a raw config value with severity-tagged findings.
- *
- * @param raw   The raw config value to validate.
- * @param mode  Validation mode — 'strict' (default) or 'lenient'.
- * @returns     A `ValidationOutcome` with findings grouped by severity.
- */
-export function validateWithSeverity(raw: unknown, mode: ValidationMode = DEFAULT_MODE): ValidationOutcome {
-	const findings: ValidationFinding[] = [];
-
-	if (!isObject(raw)) {
-		findings.push({
-			severity: "ERROR",
-			message: "config must be an object",
-			field: "config",
-		});
-		return { findings, mode, hasErrors: true, hasWarnings: false };
-	}
-
-	// --- Schema-based validation ---
-	const isValid = Value.Check(PiTeamsConfigSchema, raw);
-	if (!isValid) {
-		for (const error of Value.Errors(PiTeamsConfigSchema, raw)) {
-			const errRecord = error as unknown as Record<string, unknown>;
-			const rawPath =
-				typeof errRecord.path === "string"
-					? errRecord.path
-					: typeof errRecord.instancePath === "string"
-						? errRecord.instancePath
-						: "config";
-
-			const message = typeof errRecord.message === "string" ? errRecord.message : "invalid value";
-
-			const field = rawPath.replace(/^\//, "").replace(/\//g, ".") || undefined;
-
-			// Additional properties / unknown keys are less severe in lenient mode.
-			// @sinclair/typebox uses keyword=undefined with message="Unexpected property" for these.
-			const isUnknownProperty =
-				errRecord.keyword === "additionalProperties" ||
-				message === "Unexpected property" ||
-				message.startsWith("Unexpected property");
-			if (isUnknownProperty) {
-				findings.push({
-					severity: mode === "lenient" ? "WARNING" : "ERROR",
-					message: `unknown property${field ? ` '${field}'` : ""}: ${message}`,
-					field,
-				});
-			} else {
-				findings.push({ severity: "ERROR", message, field });
-			}
-		}
-	}
-
-	// --- Missing recommended optional keys → INFO ---
-	for (const key of RECOMMENDED_OPTIONAL_KEYS) {
-		if (!(key in raw)) {
-			findings.push({
-				severity: "INFO",
-				message: `recommended config section '${key}' is not set`,
-				field: key,
-			});
-		}
-	}
-
-	// --- Values outside recommended ranges → WARNING ---
-	for (const [dottedPath, constraint] of Object.entries(RECOMMENDED_RANGES)) {
-		const value = getNestedValue(raw, dottedPath);
-		if (typeof value === "number" && value > constraint.max) {
-			findings.push({
-				severity: "WARNING",
-				message: constraint.label,
-				field: dottedPath,
-				suggestion: `consider using a value ≤ ${constraint.max}`,
-			});
-		}
-	}
-
-	const hasErrors = findings.some((f) => f.severity === "ERROR");
-	const hasWarnings = findings.some((f) => f.severity === "WARNING");
-
-	return { findings, mode, hasErrors, hasWarnings };
-}