pi-crew 0.1.32 → 0.1.33

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-crew",
-  "version": "0.1.32",
+  "version": "0.1.33",
   "description": "Pi extension for coordinated AI teams, workflows, worktrees, and async task orchestration",
   "author": "baphuongna",
   "license": "MIT",

package/src/config/config.ts

@@ -210,6 +210,18 @@ function mergeConfig(base: PiTeamsConfig, override: PiTeamsConfig): PiTeamsConfi
 			},
 		};
 	}
+	if (base.tools || override.tools) {
+		merged.tools = {
+			...(base.tools ?? {}),
+			...withoutUndefined((override.tools ?? {}) as Record<string, unknown>),
+		};
+	}
+	if (base.telemetry || override.telemetry) {
+		merged.telemetry = {
+			...(base.telemetry ?? {}),
+			...withoutUndefined((override.telemetry ?? {}) as Record<string, unknown>),
+		};
+	}
 	if (merged.agents?.overrides && Object.keys(merged.agents.overrides).length === 0) delete merged.agents.overrides;
 	return merged;
 }
@@ -409,6 +421,26 @@ function parseAgentsConfig(value: unknown): CrewAgentsConfig | undefined {
 	return Object.values(agents).some((entry) => entry !== undefined) ? agents : undefined;
 }
 
+function parseToolsConfig(value: unknown): CrewToolsConfig | undefined {
+	const obj = asRecord(value);
+	if (!obj) return undefined;
+	const tools: CrewToolsConfig = {
+		enableClaudeStyleAliases: parseWithSchema(Type.Boolean(), obj.enableClaudeStyleAliases),
+		enableSteer: parseWithSchema(Type.Boolean(), obj.enableSteer),
+		terminateOnForeground: parseWithSchema(Type.Boolean(), obj.terminateOnForeground),
+	};
+	return Object.values(tools).some((entry) => entry !== undefined) ? tools : undefined;
+}
+
+function parseTelemetryConfig(value: unknown): CrewTelemetryConfig | undefined {
+	const obj = asRecord(value);
+	if (!obj) return undefined;
+	const telemetry: CrewTelemetryConfig = {
+		enabled: parseWithSchema(Type.Boolean(), obj.enabled),
+	};
+	return Object.values(telemetry).some((entry) => entry !== undefined) ? telemetry : undefined;
+}
+
 export function parseConfig(raw: unknown): PiTeamsConfig {
 	const obj = asRecord(raw);
 	if (!obj) return {};
@@ -423,6 +455,8 @@ export function parseConfig(raw: unknown): PiTeamsConfig {
 		control: parseControlConfig(obj.control),
 		worktree: parseWorktreeConfig(obj.worktree),
 		agents: parseAgentsConfig(obj.agents),
+		tools: parseToolsConfig(obj.tools),
+		telemetry: parseTelemetryConfig(obj.telemetry),
 		ui: parseUiConfig(obj.ui),
 	};
 }

package/src/extension/autonomous-policy.ts

@@ -5,22 +5,61 @@ import { allTeams, discoverTeams } from "../teams/discover-teams.ts";
 import { allWorkflows, discoverWorkflows } from "../workflows/discover-workflows.ts";
 
 const DEFAULT_MAGIC_KEYWORDS: Record<string, string[]> = {
-	implementation: ["autoteam", "team:", "implementation-team"],
+	implementation: ["autoteam", "team:", "implementation-team", "pi-crew", "dùng team", "use team"],
 	review: ["review-team", "security review", "code review"],
 	fastFix: ["fast-fix", "quick fix"],
 	research: ["research-team", "deep research"],
 };
 
+const BULLET_OR_NUMBERED_TASK_RE = /^\s*(?:[-*•]|\d+[.)])\s+\S+/;
+const ACTIONABLE_TASK_TERMS: readonly string[] = Array.from(new Set([
+	"implement",
+	"refactor",
+	"migrate",
+	"fix",
+	"add",
+	"update",
+	"test",
+	"review",
+	"research",
+	"analyze",
+	"document",
+	"docs",
+	"sửa",
+	"thêm",
+	"cập nhật",
+	"kiểm thử",
+	"nghiên cứu",
+	"phân tích",
+	"viết docs",
+]));
+
 function mergeMagicKeywords(configured: Record<string, string[]> | undefined): Record<string, string[]> {
 	return { ...DEFAULT_MAGIC_KEYWORDS, ...(configured ?? {}) };
 }
 
+function actionableLineCount(prompt: string): number {
+	return prompt
+		.split(/\r?\n/)
+		.map((line) => line.trim())
+		.filter((line) => BULLET_OR_NUMBERED_TASK_RE.test(line) && ACTIONABLE_TASK_TERMS.some((term) => line.toLowerCase().includes(term)))
+		.length;
+}
+
+function hasTaskListSignal(prompt: string): boolean {
+	const lower = prompt.toLowerCase();
+	const bulletCount = prompt.split(/\r?\n/).filter((line) => BULLET_OR_NUMBERED_TASK_RE.test(line)).length;
+	const explicitList = ["các task", "danh sách task", "todo", "tasks sau", "task list", "làm lần lượt"].some((term) => lower.includes(term));
+	return bulletCount >= 3 || actionableLineCount(prompt) >= 2 || (explicitList && bulletCount >= 2);
+}
+
 export function detectTeamIntent(prompt: string, config: PiTeamsAutonomousConfig = {}): string[] {
 	const lower = prompt.toLowerCase();
 	const matches: string[] = [];
 	for (const [intent, keywords] of Object.entries(mergeMagicKeywords(config.magicKeywords))) {
 		if (keywords.some((keyword) => lower.includes(keyword.toLowerCase()))) matches.push(intent);
 	}
+	if (hasTaskListSignal(prompt) && !matches.includes("taskList")) matches.push("taskList");
 	return matches;
 }
 
@@ -39,8 +78,15 @@ export function buildAutonomousPolicy(prompt: string, config: PiTeamsAutonomousC
 		`Autonomy profile: ${effective.profile}.`,
 		"You have access to the `team` tool for coordinated multi-agent work. Use it proactively when the task benefits from specialized roles, planning, review, verification, durable artifacts, async execution, or worktree isolation.",
 		"",
+		"Decision framework (not keyword-only):",
+		"- Treat a user-supplied task list with 2+ actionable bullets/numbered items as a delegation candidate even when no pi-crew keyword appears.",
+		"- Prefer `team` when tasks span multiple files/subsystems, require sequencing, combine implementation + tests/docs/review, or need independent exploration before edits.",
+		"- If unsure whether subtasks conflict, call `team` with action='recommend' first instead of manually splitting work.",
+		"- For assisted/aggressive autonomy and non-trivial multi-task work, prefer a team run or plan over direct single-agent execution.",
+		"",
 		"Use `team` automatically when:",
 		"- The task spans multiple files, subsystems, or unclear code areas.",
+		"- The prompt contains a non-trivial task list, roadmap, checklist, migration plan, or ordered implementation plan.",
 		"- The task requires planning before implementation.",
 		"- The task asks for implementation plus tests, review, verification, migration, architecture, security review, or debugging.",
 		"- The task would benefit from explorer/planner/executor/reviewer/verifier roles.",
@@ -48,6 +94,7 @@ export function buildAutonomousPolicy(prompt: string, config: PiTeamsAutonomousC
 		"Do not use `team` when:",
 		"- The user asks a simple factual question or tiny single-file edit.",
 		"- The user explicitly asks you to work directly without delegation.",
+		"- The tasks clearly modify the same small file region and can be completed safer by one agent without parallel fanout.",
 		"- The action is destructive (`delete`, `forget`, `prune`, forced cleanup) and the user has not explicitly confirmed it.",
 		"",
 		"Recommended mappings:",
@@ -60,9 +107,16 @@ export function buildAutonomousPolicy(prompt: string, config: PiTeamsAutonomousC
 		"- Before claiming delegated work is complete, inspect the run with action='status' or action='summary'.",
 		"- Unsure or risky work -> action='plan' first, then run the selected team.",
 		"",
+		"Conflict-safe task splitting:",
+		"- Do not parallelize subtasks that may edit the same file, same symbol, same migration path, package manifest, lockfile, or generated schema unless a planner explicitly sequences them.",
+		"- For potential overlap, use plan/recommend first, assign one owner per file/symbol, and require workers to report intended changed files before editing.",
+		"- Prefer workspaceMode: 'worktree' for parallel implementation in clean git repositories, but still avoid merging overlapping edits without review.",
+		"- If workers discover overlap, blockers, missing requirements, or need leader decisions, they must use mailbox/status artifacts to ask the leader/orchestrator and pause risky edits.",
+		"- The leader should resolve conflicts by sequencing, narrowing scope, or reassigning ownership before continuing.",
+		"",
 		asyncGuidance,
 		worktreeGuidance,
-		intents.length > 0 ? `Detected pi-crew routing keywords/intents in the user prompt: ${intents.join(", ")}. Consider the matching team workflow if appropriate.` : "No explicit pi-crew magic keyword was detected; decide based on task complexity and risk.",
+		intents.length > 0 ? `Detected pi-crew routing signals/intents in the user prompt: ${intents.join(", ")}. Consider the matching team workflow if appropriate.` : "No explicit pi-crew routing signal was detected; decide based on complexity, risk, task-list structure, and conflict potential.",
 	].join("\n");
 }
 

package/src/extension/team-recommendation.ts

@@ -26,8 +26,8 @@ const REVIEW_TERMS = ["review", "audit", "security", "vulnerability", "diff", "p
 const RESEARCH_TERMS = ["research", "investigate", "compare", "analyze", "document", "docs", "explain", "architecture", "đọc sâu", "source", "projects"];
 const PARALLEL_RESEARCH_RE = /(?:đọc sâu|deep read|deep research|source audit|multiple projects|các project|pi-\*|source\/|@source)/i;
 const FAST_FIX_TERMS = ["quick fix", "fast-fix", "small bug", "typo", "one-line", "minor", "lint"];
-const IMPLEMENTATION_TERMS = ["implement", "refactor", "migrate", "feature", "tests", "test", "integration", "upgrade", "build", "create", "add"];
-const RISKY_TERMS = ["migration", "refactor", "large", "multiple", "parallel", "concurrent", "risky", "critical"];
+const IMPLEMENTATION_TERMS = ["implement", "refactor", "migrate", "feature", "tests", "test", "integration", "upgrade", "build", "create", "add", "fix", "update", "sửa", "thêm", "cập nhật", "kiểm thử"];
+const RISKY_TERMS = ["migration", "refactor", "large", "multiple", "parallel", "concurrent", "risky", "critical", "nhiều file", "nhiều task"];
 const NUMBERED_LINE_RE = /^\s*\d+[.)]\s+(.+)$/;
 const BULLETED_LINE_RE = /^\s*[-*•]\s+(.+)$/;
 const CONJUNCTION_SPLIT_RE = /\s+(?:and|,\s*and|,)\s+/i;
@@ -67,12 +67,14 @@ export function decomposeGoal(goal: string): { strategy: DecompositionStrategy;
 		const subtask = makeSubtask(goal);
 		return { strategy: "atomic", subtasks: [subtask], fanout: 1 };
 	}
-	if (lines.length >= 2 && lines.every((line) => NUMBERED_LINE_RE.test(line))) {
-		const subtasks = lines.map((line) => makeSubtask(line.match(NUMBERED_LINE_RE)?.[1] ?? line));
+	const numberedLines = lines.map((line) => line.match(NUMBERED_LINE_RE)?.[1]).filter((line): line is string => line !== undefined);
+	if (numberedLines.length >= 2 && numberedLines.length >= lines.length - 1) {
+		const subtasks = numberedLines.map((line) => makeSubtask(line));
 		return { strategy: "numbered", subtasks, fanout: subtasks.length };
 	}
-	if (lines.length >= 2 && lines.every((line) => BULLETED_LINE_RE.test(line))) {
-		const subtasks = lines.map((line) => makeSubtask(line.match(BULLETED_LINE_RE)?.[1] ?? line));
+	const bulletedLines = lines.map((line) => line.match(BULLETED_LINE_RE)?.[1]).filter((line): line is string => line !== undefined);
+	if (bulletedLines.length >= 2 && bulletedLines.length >= lines.length - 1) {
+		const subtasks = bulletedLines.map((line) => makeSubtask(line));
 		return { strategy: "bulleted", subtasks, fanout: subtasks.length };
 	}
 	if (lines.length === 1) {
@@ -94,6 +96,7 @@ function metadataMatches(goal: string, values: string[] | undefined): string[] {
 export function recommendTeam(goal: string, config: PiTeamsAutonomousConfig = {}, resources?: { teams?: TeamConfig[]; agents?: AgentConfig[] }): TeamRecommendation {
 	const normalized = goal.toLowerCase();
 	const intents = detectTeamIntent(goal, config);
+	const decomposition = decomposeGoal(goal);
 	const reasons: string[] = [];
 	let team: TeamRecommendation["team"] = "default";
 	let workflow: TeamRecommendation["workflow"] = "default";
@@ -138,10 +141,15 @@ export function recommendTeam(goal: string, config: PiTeamsAutonomousConfig = {}
 		workflow = "fast-fix";
 		confidence = "high";
 		reasons.push(`Small fix terms detected: ${fastFixMatches.join(", ") || "fast-fix intent"}.`);
+	} else if (intents.includes("taskList")) {
+		team = "implementation";
+		workflow = "implementation";
+		confidence = "high";
+		reasons.push(`Actionable multi-item task list detected (${decomposition.fanout} bullet${decomposition.fanout === 1 ? "" : "s"}); use coordinated implementation planning.`);
 	} else if (intents.includes("implementation") || implementationMatches.length > 0) {
 		team = "implementation";
 		workflow = "implementation";
-		confidence = implementationMatches.length >= 2 || riskyMatches.length > 0 ? "high" : "medium";
+		confidence = implementationMatches.length >= 2 || riskyMatches.length > 0 || decomposition.fanout >= 2 ? "high" : "medium";
 		reasons.push(`Implementation terms detected: ${implementationMatches.join(", ") || "implementation intent"}.`);
 	} else {
 		action = "plan";
@@ -149,7 +157,7 @@ export function recommendTeam(goal: string, config: PiTeamsAutonomousConfig = {}
 		reasons.push("No strong team-specific intent detected; start with planning/default discovery.");
 	}
 
-	const decomposition = decomposeGoal(goal);
+
 	if (decomposition.strategy !== "atomic") reasons.push(`Goal decomposes into ${decomposition.subtasks.length} subtasks using ${decomposition.strategy} parsing.`);
 	const async = config.preferAsyncForLongTasks === true && (wordCount(goal) > 24 || riskyMatches.length > 0 || implementationMatches.length >= 2 || decomposition.fanout >= 3);
 	const workspaceMode = config.allowWorktreeSuggestion === false ? "single" : (riskyMatches.length > 0 && team === "implementation" ? "worktree" : "single");

package/src/runtime/task-packet.ts

@@ -43,12 +43,13 @@ export function buildTaskPacket(input: BuildTaskPacketInput): TaskPacket {
 		branchPolicy: input.manifest.workspaceMode === "worktree" ? "Use the assigned task worktree and avoid modifying the leader checkout." : "Use the current checkout; do not create branches unless explicitly requested.",
 		acceptanceTests: [],
 		commitPolicy: "Do not commit unless explicitly requested by the user or workflow.",
-		reportingContract: "Report changed files, verification evidence, blockers, and next recommended action.",
-		escalationPolicy: "Stop and report if scope is ambiguous, destructive action is needed, permissions are missing, or verification cannot be completed.",
+		reportingContract: "Report intended/changed files, verification evidence, blockers, conflict risks, and next recommended action.",
+		escalationPolicy: "Stop and report if scope is ambiguous, destructive action is needed, permissions are missing, verification cannot be completed, or edits may overlap with another worker/task.",
 		constraints: [
 			"Stay within the assigned task scope.",
 			"Do not claim completion without verification evidence.",
 			"Use mailbox/API state for coordination when available.",
+			"Do not make overlapping edits to the same file/symbol without explicit leader sequencing or ownership guidance.",
 		],
 		expectedArtifacts: ["prompt", "result", "verification"],
 		verification: defaultVerificationContract(input.step),
@@ -66,6 +67,14 @@ export function validateTaskPacket(packet: TaskPacket): TaskPacketValidationResu
 	if ((packet.scope === "module" || packet.scope === "single_file" || packet.scope === "custom") && !packet.scopePath?.trim()) {
 		errors.push(`scopePath is required for scope '${packet.scope}'`);
 	}
+	if (packet.constraints.length === 0) errors.push("constraints must contain at least one entry");
+	for (const [index, constraint] of packet.constraints.entries()) {
+		if (!constraint.trim()) errors.push(`constraints contains an empty value at index ${index}`);
+	}
+	if (packet.expectedArtifacts.length === 0) errors.push("expectedArtifacts must contain at least one entry");
+	for (const [index, artifact] of packet.expectedArtifacts.entries()) {
+		if (!artifact.trim()) errors.push(`expectedArtifacts contains an empty value at index ${index}`);
+	}
 	for (const [index, test] of packet.acceptanceTests.entries()) {
 		if (!test.trim()) errors.push(`acceptanceTests contains an empty value at index ${index}`);
 	}

package/src/runtime/task-runner/prompt-builder.ts

@@ -23,6 +23,9 @@ export function coordinationBridgeInstructions(task: TeamTaskState): string {
 		`Mailbox target for this task: ${task.id}`,
 		"Use the run mailbox contract for coordination with the leader/orchestrator:",
 		"- If blocked or uncertain, report the blocker in your final result and, when mailbox tools/API are available, send an inbox/outbox message addressed to the leader.",
+		"- Ask the leader before editing when scope is ambiguous, requirements conflict, destructive action is needed, or you discover likely overlap with another task.",
+		"- Before making non-trivial edits, state intended changed files in your notes/result; if another worker may touch the same file/symbol, pause and request sequencing/ownership guidance.",
+		"- Do not resolve cross-worker conflicts silently. Escalate via mailbox/result with: file/symbol, conflicting task if known, proposed owner, and safest next step.",
 		"- If nudged, answer with current status, blocker, or smallest next step.",
 		"- Treat inherited/dependency context as reference-only; do not continue the parent conversation directly.",
 		"- Completion handoff should include: DONE/FAILED, summary, changed/read files, verification evidence, and remaining risks.",