@phi-code-admin/phi-code 0.71.0 → 0.72.0

package/extensions/phi/orchestrator.ts

@@ -527,6 +527,8 @@ export default function orchestratorExtension(pi: ExtensionAPI) {
 		const review = getModel("review");
 
 		const ts = timestamp();
+		// Inject runtime info so agents can adapt to the host OS
+		const runtimeInfo = `\n\nRuntime: ${process.platform} (${process.arch})`;
 
 		return [
 			{
@@ -534,19 +536,23 @@ export default function orchestratorExtension(pi: ExtensionAPI) {
 				agent: loadAgentDef("explore"),
 				instruction: `You are the EXPLORE agent. Analyze the project requirements and existing codebase.
 
+**FIRST ACTION (MANDATORY):** Call \`memory_search\` with project-relevant keywords to check for prior context.
+
 **Project Request:** ${description}
 
 **Your tasks:**
-1. List all existing files and read key ones
-2. Identify tech stack, patterns, and constraints
-3. Create a STRUCTURED PROJECT BRIEF in \`.phi/plans/brief-${ts}.md\`:
+1. Call \`memory_search\` with project-relevant keywords (MANDATORY)
+2. List all existing files and read key ones
+3. Identify tech stack, patterns, and constraints
+4. Create a STRUCTURED PROJECT BRIEF in \`.phi/plans/brief-${ts}.md\`:
    - Context: what exists now
    - Objective: what needs to be built
    - Requirements: specific features needed
    - Tech decisions: frameworks, patterns to use
    - Constraints: what to NOT break
+5. Write your findings to \`.phi/plans/explore-${ts}.md\`
 
-**Step 4:** Write your findings to \`.phi/plans/explore-${ts}.md\`
+**LAST ACTION (MANDATORY):** Call \`memory_write\` to save your exploration findings for downstream agents.
 
 **Knowledge Graph:**
 After your analysis, use \`ontology_add\` to save key project entities AND their relations:
@@ -576,7 +582,7 @@ After your analysis, use \`ontology_add\` to save key project entities AND their
 - Every function fully implemented
 - Follow existing patterns if codebase exists
 - [Any other specific constraints]
-\`\`\``,
+\`\`\`` + runtimeInfo,
 			},
 			{
 				key: "plan", label: "📐 Phase 2 — PLAN", model: plan.preferred, fallback: plan.fallback,
@@ -616,7 +622,7 @@ After your analysis, use \`ontology_add\` to save key project entities AND their
 - Dependencies: Task 1
 \`\`\`
 
-Before finishing, use \`memory_write\` to save your plan summary with relevant tags for future reference.`,
+Before finishing, use \`memory_write\` to save your plan summary with relevant tags for future reference.` + runtimeInfo,
 			},
 			{
 				key: "code", label: "💻 Phase 3 — CODE", model: code.preferred, fallback: code.fallback,
@@ -663,7 +669,7 @@ After implementation, use \`memory_write\` to save a summary of what was built,
 **CRITICAL RULES:**
 - Write ONE file per tool call — NEVER combine multiple files in a single response
 - Keep each file under 500 lines. If longer, split into modules
-- After writing each file, verify it exists with \`ls\` before proceeding`,
+- After writing ALL files, verify they exist with a single \`find . -name '*.ts' -o -name '*.js' -o -name '*.html' | sort\`` + runtimeInfo,
 			},
 			{
 				key: "test", label: "🧪 Phase 4 — TEST", model: test.preferred, fallback: test.fallback,
@@ -710,20 +716,23 @@ After implementation, use \`memory_write\` to save a summary of what was built,
 - NEVER put tool calls inside thinking blocks. Always use the proper JSON tool call format
 - NEVER modify source code permanently for testing. Use environment variables: \`PORT=3001 node server.js\` instead of editing files
 - NEVER create .env files with fake credentials. Use inline env vars: \`API_KEY=test node server.js\`
-- For port conflicts on Windows, use: \`netstat -ano | findstr :PORT\` and \`taskkill /PID <pid> /F\`
-- For port conflicts on Linux/Mac, use: \`lsof -ti:PORT | xargs kill -9\`
+- For port conflicts, prefer: \`npx kill-port PORT\` (cross-platform)
+- On Windows fallback: \`netstat -ano | findstr :PORT\` and \`taskkill /PID <pid> /F\`
+- On Linux/Mac fallback: \`lsof -ti:PORT | xargs kill -9\`
 - Always clean up after tests: kill background processes, remove temp files
 
-After testing, use \`memory_write\` to save test results, bugs found, and lessons learned.`,
+After testing, use \`memory_write\` to save test results, bugs found, and lessons learned.` + runtimeInfo,
 			},
 			{
 				key: "review", label: "🔍 Phase 5 — REVIEW", model: review.preferred, fallback: review.fallback,
 				agent: loadAgentDef("review"),
 				instruction: `You are the REVIEW agent. Final quality review.
 
+**FIRST ACTIONS (MANDATORY — do these before anything else):**
+1. Call \`memory_search\` to find all notes from previous phases (explore, plan, code, test)
+2. Call \`ontology_query\` to understand the full project architecture and entity relationships
+
 **Context Retrieval:**
-1. Use \`memory_search\` to find all notes from previous phases (explore, plan, code, test)
-2. Use \`ontology_query\` to understand the full project architecture
 3. Review all \`.phi/plans/*.md\` files for complete context
 
 **Project Request:** ${description}
@@ -765,11 +774,13 @@ After testing, use \`memory_write\` to save test results, bugs found, and lesson
 ✅ Project ready for production / ❌ Issues need resolution
 \`\`\`
 
-After your review, use \`memory_write\` to save:
+After your review, use \`memory_write\` ONCE to save:
 - Key lessons learned about this project type
 - Patterns that worked well
 - Common mistakes to avoid in future projects
-Tag the note with relevant keywords for vector search.`,
+Tag the note with relevant keywords for vector search.
+
+**Important:** Write lessons-learned ONCE. Do not call memory_write twice with the same filename or duplicate content.` + runtimeInfo,
 			},
 		];
 	}
@@ -837,10 +848,16 @@ Tag the note with relevant keywords for vector search.`,
 
 	function sendNextPhase(ctx: any) {
 		if (phaseQueue.length === 0) {
+			// All phases done — clean up and notify
 			setOrchestrationActive(false);
 			phasePending = false;
 			deactivateAgent();
-			ctx.ui.notify(`\n✅ **All 5 phases complete!**`, "info");
+			try {
+				ctx.ui.notify(`\n✅ **All 5 phases complete!**`, "info");
+			} catch {
+				// Fallback: inject completion message into conversation stream
+				try { pi.sendUserMessage(`✅ All 5 phases complete! The project is ready for review.`); } catch { /* best effort */ }
+			}
 			return;
 		}
 
@@ -875,31 +892,69 @@ Tag the note with relevant keywords for vector search.`,
 	// That's why phases 2-5 never executed.
 
 	pi.on("agent_end", async (event, ctx) => {
-		if (!orchestrationActive || !phasePending) return;
+		if (!orchestrationActive) return;
+		// Allow completion even if phasePending was cleared by a duplicate event
+		if (!phasePending) {
+			// If queue is empty and orchestration is active, force completion
+			if (phaseQueue.length === 0) {
+				sendNextPhase(ctx);
+			}
+			return;
+		}
 
-		// Capture the most informative assistant message for context passing
-		// The last message is often trivial ("Good, file created."). Find the longest
-		// text-only assistant message instead — that's usually the real analysis/plan.
+		// Build a structured summary of what happened in this phase
+		// Instead of raw LLM text, extract concrete actions: files created/modified,
+		// errors encountered, test results. This gives the next phase actionable context.
 		const messages = event.messages || [];
-		const assistantMessages = messages.filter(m => m.role === 'assistant');
-		let bestOutput = '';
-		let bestLength = 0;
-		for (const msg of assistantMessages) {
-			if (!msg.content) continue;
-			const textParts = Array.isArray(msg.content)
-				? msg.content.filter((c: any) => c.type === 'text').map((c: any) => c.text)
-				: [String(msg.content)];
-			const combined = textParts.join('\n');
-			if (combined.length > bestLength) {
-				bestLength = combined.length;
-				bestOutput = combined;
+		const filesWritten: string[] = [];
+		const filesEdited: string[] = [];
+		const errorsHit: string[] = [];
+		const testResults: string[] = [];
+		let toolCallCount = 0;
+
+		for (const msg of messages) {
+			// Pi uses role: "toolResult" instead of "tool"
+			if (msg.role === 'tool' || msg.role === 'function' || msg.role === 'toolResult') {
+				toolCallCount++;
+				const content = Array.isArray(msg.content)
+					? msg.content.map((c: any) => c.text || '').join('')
+					: String(msg.content || '');
+				const name = (msg as any).name || (msg as any).toolName || '';
+				// Track writes
+				if (name === 'write' && content.includes('Successfully wrote')) {
+					const match = content.match(/wrote \d+ bytes to (.+)/);
+					if (match) filesWritten.push(match[1]);
+				}
+				// Track edits
+				if (name === 'edit' && !content.includes('ERR')) {
+					const match = content.match(/edited (.+)/) || content.match(/in (.+)/);
+					if (match) filesEdited.push(match[1]);
+				}
+				// Track errors
+				if (content.includes('ERR:') || content.includes('Error:') || content.includes('FAIL')) {
+					const preview = content.slice(0, 150).replace(/\n/g, ' ');
+					errorsHit.push(`${name}: ${preview}`);
+				}
+				// Track test results
+				if (content.includes('PASS') || content.includes('✅') || content.includes('✗') || content.includes('❌')) {
+					const lines = content.split('\n').filter(l => /PASS|FAIL|✅|❌|✗/.test(l));
+					testResults.push(...lines.slice(0, 10));
+				}
 			}
 		}
-		const lastOutput = bestOutput.slice(0, 4000);
 
-		// Inject previous phase output into next phase
-		if (lastOutput && phaseQueue.length > 0) {
-			phaseQueue[0].instruction += `\n\n**Previous phase output (summary):**\n${lastOutput}`;
+		// Build the summary
+		const summaryParts: string[] = [];
+		summaryParts.push(`Tool calls: ${toolCallCount}`);
+		if (filesWritten.length > 0) summaryParts.push(`Files created/written: ${filesWritten.join(', ')}`);
+		if (filesEdited.length > 0) summaryParts.push(`Files edited: ${filesEdited.join(', ')}`);
+		if (testResults.length > 0) summaryParts.push(`Test results:\n${testResults.join('\n')}`);
+		if (errorsHit.length > 0) summaryParts.push(`Errors encountered: ${errorsHit.length}\n${errorsHit.slice(0, 5).join('\n')}`);
+		const phaseSummary = summaryParts.join('\n');
+
+		// Inject structured summary into next phase
+		if (phaseSummary && phaseQueue.length > 0) {
+			phaseQueue[0].instruction += `\n\n**Previous phase summary:**\n${phaseSummary}`;
 		}
 
 		// Phase complete — chain to next

package/package.json

@@ -1,107 +1,107 @@
 {
-	"name": "@phi-code-admin/phi-code",
-	"version": "0.71.0",
-	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
-	"type": "module",
-	"piConfig": {
-		"name": "phi",
-		"configDir": ".phi"
-	},
-	"bin": {
-		"phi": "dist/cli.js"
-	},
-	"main": "./dist/index.js",
-	"types": "./dist/index.d.ts",
-	"exports": {
-		".": {
-			"types": "./dist/index.d.ts",
-			"import": "./dist/index.js"
-		},
-		"./hooks": {
-			"types": "./dist/core/hooks/index.d.ts",
-			"import": "./dist/core/hooks/index.js"
-		}
-	},
-	"files": [
-		"dist",
-		"skills",
-		"agents",
-		"extensions",
-		"README.md",
-		"LICENSE",
-		"CHANGELOG.md",
-		"scripts"
-	],
-	"scripts": {
-		"clean": "shx rm -rf dist",
-		"dev": "tsgo -p tsconfig.build.json --watch --preserveWatchOutput",
-		"build": "tsgo -p tsconfig.build.json && shx chmod +x dist/cli.js && npm run copy-assets",
-		"build:binary": "npm --prefix ../tui run build && npm --prefix ../ai run build && npm --prefix ../agent run build && npm run build && bun build --compile ./dist/cli.js --outfile dist/pi && npm run copy-binary-assets",
-		"copy-assets": "./scripts/copy-assets.sh",
-		"copy-binary-assets": "shx cp package.json dist/ && shx cp README.md dist/ && shx cp CHANGELOG.md dist/ && shx mkdir -p dist/theme && shx cp src/modes/interactive/theme/*.json dist/theme/ && shx mkdir -p dist/export-html/vendor && shx cp src/core/export-html/template.html dist/export-html/ && shx cp src/core/export-html/vendor/*.js dist/export-html/vendor/ && shx cp -r docs dist/ && shx cp -r examples dist/ && shx cp ../../node_modules/@silvia-odwyer/photon-node/photon_rs_bg.wasm dist/",
-		"test": "vitest --run",
-		"prepublishOnly": "npm run clean && npm run build",
-		"postinstall": "node scripts/postinstall.cjs"
-	},
-	"dependencies": {
-		"@mariozechner/jiti": "^2.6.2",
-		"@silvia-odwyer/photon-node": "^0.3.4",
-		"chalk": "^5.5.0",
-		"cli-highlight": "^2.1.11",
-		"diff": "^8.0.2",
-		"extract-zip": "^2.0.1",
-		"file-type": "^21.1.1",
-		"glob": "^13.0.1",
-		"hosted-git-info": "^9.0.2",
-		"ignore": "^7.0.5",
-		"marked": "^15.0.12",
-		"minimatch": "^10.2.3",
-		"phi-code-agent": "^0.56.3",
-		"phi-code-ai": "^0.56.3",
-		"phi-code-tui": "^0.56.3",
-		"proper-lockfile": "^4.1.2",
-		"sigma-agents": "^0.1.6",
-		"sigma-memory": "0.2.1",
-		"sigma-skills": "^0.1.2",
-		"strip-ansi": "^7.1.0",
-		"undici": "^7.19.1",
-		"yaml": "^2.8.2"
-	},
-	"overrides": {
-		"rimraf": "6.1.2",
-		"gaxios": {
-			"rimraf": "6.1.2"
-		}
-	},
-	"optionalDependencies": {
-		"@mariozechner/clipboard": "^0.3.2"
-	},
-	"devDependencies": {
-		"@types/diff": "^7.0.2",
-		"@types/hosted-git-info": "^3.0.5",
-		"@types/ms": "^2.1.0",
-		"@types/node": "^24.3.0",
-		"@types/proper-lockfile": "^4.1.4",
-		"shx": "^0.4.0",
-		"typescript": "^5.7.3",
-		"vitest": "^3.2.4"
-	},
-	"keywords": [
-		"ai",
-		"coding-agent",
-		"cli",
-		"typescript",
-		"sub-agents",
-		"memory"
-	],
-	"author": "Mario Zechner",
-	"license": "MIT",
-	"repository": {
-		"type": "git",
-		"url": "https://github.com/uglyswap/phi-code.git"
-	},
-	"homepage": "https://github.com/uglyswap/phi-code",
-	"engines": {
-		"node": ">=20.6.0"
-	}
+  "name": "@phi-code-admin/phi-code",
+  "version": "0.72.0",
+  "description": "Coding agent CLI with read, bash, edit, write tools and session management",
+  "type": "module",
+  "piConfig": {
+    "name": "phi",
+    "configDir": ".phi"
+  },
+  "bin": {
+    "phi": "dist/cli.js"
+  },
+  "main": "./dist/index.js",
+  "types": "./dist/index.d.ts",
+  "exports": {
+    ".": {
+      "types": "./dist/index.d.ts",
+      "import": "./dist/index.js"
+    },
+    "./hooks": {
+      "types": "./dist/core/hooks/index.d.ts",
+      "import": "./dist/core/hooks/index.js"
+    }
+  },
+  "files": [
+    "dist",
+    "skills",
+    "agents",
+    "extensions",
+    "README.md",
+    "LICENSE",
+    "CHANGELOG.md",
+    "scripts"
+  ],
+  "scripts": {
+    "clean": "shx rm -rf dist",
+    "dev": "tsgo -p tsconfig.build.json --watch --preserveWatchOutput",
+    "build": "tsgo -p tsconfig.build.json && shx chmod +x dist/cli.js && npm run copy-assets",
+    "build:binary": "npm --prefix ../tui run build && npm --prefix ../ai run build && npm --prefix ../agent run build && npm run build && bun build --compile ./dist/cli.js --outfile dist/pi && npm run copy-binary-assets",
+    "copy-assets": "./scripts/copy-assets.sh",
+    "copy-binary-assets": "shx cp package.json dist/ && shx cp README.md dist/ && shx cp CHANGELOG.md dist/ && shx mkdir -p dist/theme && shx cp src/modes/interactive/theme/*.json dist/theme/ && shx mkdir -p dist/export-html/vendor && shx cp src/core/export-html/template.html dist/export-html/ && shx cp src/core/export-html/vendor/*.js dist/export-html/vendor/ && shx cp -r docs dist/ && shx cp -r examples dist/ && shx cp ../../node_modules/@silvia-odwyer/photon-node/photon_rs_bg.wasm dist/",
+    "test": "vitest --run",
+    "prepublishOnly": "npm run clean && npm run build",
+    "postinstall": "node scripts/postinstall.cjs"
+  },
+  "dependencies": {
+    "@mariozechner/jiti": "^2.6.2",
+    "@silvia-odwyer/photon-node": "^0.3.4",
+    "chalk": "^5.5.0",
+    "cli-highlight": "^2.1.11",
+    "diff": "^8.0.2",
+    "extract-zip": "^2.0.1",
+    "file-type": "^21.1.1",
+    "glob": "^13.0.1",
+    "hosted-git-info": "^9.0.2",
+    "ignore": "^7.0.5",
+    "marked": "^15.0.12",
+    "minimatch": "^10.2.3",
+    "phi-code-agent": "^0.56.3",
+    "phi-code-ai": "^0.56.3",
+    "phi-code-tui": "^0.56.3",
+    "proper-lockfile": "^4.1.2",
+    "sigma-agents": "^0.1.6",
+    "sigma-memory": "0.2.1",
+    "sigma-skills": "^0.1.2",
+    "strip-ansi": "^7.1.0",
+    "undici": "^7.19.1",
+    "yaml": "^2.8.2"
+  },
+  "overrides": {
+    "rimraf": "6.1.2",
+    "gaxios": {
+      "rimraf": "6.1.2"
+    }
+  },
+  "optionalDependencies": {
+    "@mariozechner/clipboard": "^0.3.2"
+  },
+  "devDependencies": {
+    "@types/diff": "^7.0.2",
+    "@types/hosted-git-info": "^3.0.5",
+    "@types/ms": "^2.1.0",
+    "@types/node": "^24.3.0",
+    "@types/proper-lockfile": "^4.1.4",
+    "shx": "^0.4.0",
+    "typescript": "^5.7.3",
+    "vitest": "^3.2.4"
+  },
+  "keywords": [
+    "ai",
+    "coding-agent",
+    "cli",
+    "typescript",
+    "sub-agents",
+    "memory"
+  ],
+  "author": "Mario Zechner",
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/uglyswap/phi-code.git"
+  },
+  "homepage": "https://github.com/uglyswap/phi-code",
+  "engines": {
+    "node": ">=20.6.0"
+  }
 }