@phi-code-admin/phi-code 0.58.6 → 0.58.8

package/extensions/phi/orchestrator.ts

@@ -144,6 +144,12 @@ export default function orchestratorExtension(pi: ExtensionAPI) {
 		task: TaskDef,
 		agentDefs: Map<string, AgentDef>,
 		cwd: string,
+		sharedContext: {
+			projectTitle: string;
+			projectDescription: string;
+			specSummary: string;
+			completedTasks: Array<{ index: number; title: string; agent: string; output: string }>;
+		},
 		timeoutMs: number = 300000,
 	): Promise<TaskResult> {
 		return new Promise((resolve) => {
@@ -153,11 +159,42 @@ export default function orchestratorExtension(pi: ExtensionAPI) {
 			const phiBin = findPhiBinary();
 			const startTime = Date.now();
 
-			let taskPrompt = `Task: ${task.title}\n\n${task.description}`;
+			// Build prompt with shared context
+			let taskPrompt = "";
+
+			// Inject shared project context (lightweight, always included)
+			taskPrompt += `# Project Context\n\n`;
+			taskPrompt += `**Project:** ${sharedContext.projectTitle}\n`;
+			taskPrompt += `**Description:** ${sharedContext.projectDescription}\n\n`;
+
+			if (sharedContext.specSummary) {
+				taskPrompt += `## Specification Summary\n${sharedContext.specSummary}\n\n`;
+			}
+
+			// Inject results from dependency tasks (only the ones this task depends on)
+			const deps = task.dependencies || [];
+			if (deps.length > 0) {
+				const depResults = sharedContext.completedTasks.filter(ct => deps.includes(ct.index));
+				if (depResults.length > 0) {
+					taskPrompt += `## Previous Task Results (your dependencies)\n\n`;
+					for (const dep of depResults) {
+						const truncatedOutput = dep.output.length > 1500 ? dep.output.slice(0, 1500) + "\n...(truncated)" : dep.output;
+						taskPrompt += `### Task ${dep.index}: ${dep.title} [${dep.agent}]\n\`\`\`\n${truncatedOutput}\n\`\`\`\n\n`;
+					}
+				}
+			}
+
+			// The actual task
+			taskPrompt += `---\n\n# Your Task\n\n**${task.title}**\n\n${task.description}`;
 			if (task.subtasks && task.subtasks.length > 0) {
-				taskPrompt += "\n\nSub-tasks:\n" + task.subtasks.map((st, i) => `${i + 1}. ${st}`).join("\n");
+				taskPrompt += "\n\n## Sub-tasks\n" + task.subtasks.map((st, i) => `${i + 1}. ${st}`).join("\n");
 			}
-			taskPrompt += "\n\nComplete this task. Be thorough and precise. Report what you did.";
+			taskPrompt += `\n\n---\n\n## Instructions\n`;
+			taskPrompt += `- You are an isolated agent with your own context. Work independently.\n`;
+			taskPrompt += `- Use the project context and dependency results above to inform your work.\n`;
+			taskPrompt += `- Follow the output format defined in your system prompt.\n`;
+			taskPrompt += `- Be precise. Reference specific file paths and line numbers.\n`;
+			taskPrompt += `- Report exactly what you did, what worked, and what didn't.\n`;
 
 			const args: string[] = [];
 			if (phiBin === "npx") args.push("@phi-code-admin/phi-code");
@@ -199,17 +236,25 @@ export default function orchestratorExtension(pi: ExtensionAPI) {
 		tasks: TaskDef[],
 		todoFile: string,
 		notify: (msg: string, type: "info" | "error" | "warning") => void,
+		projectContext?: { title: string; description: string; specSummary: string },
 	): Promise<{ results: TaskResult[]; progressFile: string }> {
 		const agentDefs = loadAgentDefs();
 		const progressFile = todoFile.replace("todo-", "progress-");
 		const progressPath = join(plansDir, progressFile);
 		let progress = `# Progress: ${todoFile}\n\n`;
 		progress += `**Started:** ${new Date().toLocaleString()}\n`;
-		progress += `**Tasks:** ${tasks.length}\n**Mode:** parallel (dependency-aware)\n\n`;
+		progress += `**Tasks:** ${tasks.length}\n**Mode:** parallel (dependency-aware, shared context)\n\n`;
 		await writeFile(progressPath, progress, "utf-8");
 
+		// Shared context for sub-agents
+		const sharedContext = {
+			projectTitle: projectContext?.title || "Project",
+			projectDescription: projectContext?.description || "",
+			specSummary: projectContext?.specSummary || "",
+			completedTasks: [] as Array<{ index: number; title: string; agent: string; output: string }>,
+		};
+
 		// Build dependency graph
-		// Task indices are 1-based in the plan files
 		const completed = new Set<number>();
 		const failed = new Set<number>();
 		const results: TaskResult[] = [];
@@ -268,22 +313,29 @@ export default function orchestratorExtension(pi: ExtensionAPI) {
 				notify(`⏳ Task ${idx + 1}: **${t.title}** [${t.agent || "code"}]`, "info");
 			}
 
-			// Launch all ready tasks simultaneously
+			// Launch all ready tasks simultaneously (each gets shared context)
 			const promises = readyIndices.map(async (idx) => {
 				const task = tasks[idx];
-				const result = await executeTask(task, agentDefs, process.cwd());
+				const result = await executeTask(task, agentDefs, process.cwd(), sharedContext);
 				result.taskIndex = idx + 1;
 				return result;
 			});
 
 			const waveResults = await Promise.all(promises);
 
-			// Process results
+			// Process results and feed into shared context for next wave
 			for (const result of waveResults) {
 				results.push(result);
 
 				if (result.status === "success") {
 					completed.add(result.taskIndex);
+					// Add to shared context so dependent tasks can see this result
+					sharedContext.completedTasks.push({
+						index: result.taskIndex,
+						title: result.title,
+						agent: result.agent,
+						output: result.output,
+					});
 				} else {
 					failed.add(result.taskIndex);
 				}
@@ -402,7 +454,7 @@ export default function orchestratorExtension(pi: ExtensionAPI) {
 		promptSnippet: "Plan + execute projects in parallel waves. Each sub-agent gets isolated context + model. Use prompt-architect patterns for structured task descriptions.",
 		promptGuidelines: [
 			"When asked to plan or build a project: analyze the request thoroughly, then call the orchestrate tool. It plans AND executes automatically.",
-			"CRITICAL: Each task description must be SELF-CONTAINED. The sub-agent has NO shared history, NO access to the conversation, NO memory of previous tasks. Include ALL context it needs: file paths, expected behavior, relevant code patterns, and success criteria.",
+			"CRITICAL: Each task description must be SELF-CONTAINED. The sub-agent has NO access to this conversation. It receives: (1) project context (title, description, spec summary) automatically, (2) outputs from its dependency tasks automatically, (3) your task description. So include specific details: file paths, expected behavior, code patterns, success criteria. Don't repeat the project description — that's injected automatically.",
 			"Structure each task description using the prompt-architect pattern: [CONTEXT] what exists and why → [TASK] what to do specifically → [FORMAT] expected output → [CONSTRAINTS] rules and limitations.",
 			"Assign agent types strategically: 'explore' (read-only analysis, codebase understanding), 'plan' (architecture, design decisions), 'code' (implementation, file creation/modification), 'test' (write + run tests, validate behavior), 'review' (security audit, quality check, read-only).",
 			"Set dependencies to maximize parallelism: tasks without dependencies run simultaneously in the same wave. Only add dependencies when a task truly needs another task's output.",
@@ -459,7 +511,18 @@ export default function orchestratorExtension(pi: ExtensionAPI) {
 				notify(`📋 Plan created: **${p.title}** (${p.tasks.length} tasks)\nNow executing with sub-agents...`, "info");
 
 				// Auto-execute all tasks
-				const { results, progressFile } = await executePlan(p.tasks, todoFile, notify);
+				// Build spec summary for shared context
+				const specSummary = [
+					`Goals: ${p.goals.join("; ")}`,
+					`Requirements: ${p.requirements.join("; ")}`,
+					p.architecture?.length ? `Architecture: ${p.architecture.join("; ")}` : "",
+					p.constraints?.length ? `Constraints: ${p.constraints.join("; ")}` : "",
+				].filter(Boolean).join("\n");
+
+				const { results, progressFile } = await executePlan(
+					p.tasks, todoFile, notify,
+					{ title: p.title, description: p.description, specSummary },
+				);
 
 				const succeeded = results.filter(r => r.status === "success").length;
 				const failed = results.filter(r => r.status === "error").length;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@phi-code-admin/phi-code",
-	"version": "0.58.6",
+	"version": "0.58.8",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"piConfig": {