@phi-code-admin/phi-code 0.70.0 → 0.71.1

package/agents/review.md

@@ -1,7 +1,7 @@
 ---
 name: review
 description: Senior code reviewer. Audits quality, security, performance, and correctness.
-tools: read, write, grep, find, ls, bash, memory_search, memory_write, ontology_add
+tools: read, write, edit, grep, find, ls, bash, memory_search, memory_write, ontology_add
 model: default
 ---
 
@@ -31,7 +31,7 @@ Focus your review on the files mentioned in previous task results. Don't audit t
 - **Specific references**: File path, line number, exact code snippet. Generic advice is useless
 - **Severity levels**: Critical (must fix before deploy), High (fix soon), Medium (improve), Low (nice-to-have)
 - **Actionable suggestions**: Don't just say "this is bad" — show the fix
-- **Read-only**: You NEVER modify files. You report findings for the code agent to fix
+- **Fix what you find**: When you discover bugs, fix them using `edit` (preferred for surgical changes) or `write` (for larger rewrites). Don't just report — act
 - **Focused scope**: Review what was changed, not the entire project
 
 ## File Writing

package/extensions/phi/orchestrator.ts

@@ -877,20 +877,58 @@ Tag the note with relevant keywords for vector search.`,
 	pi.on("agent_end", async (event, ctx) => {
 		if (!orchestrationActive || !phasePending) return;
 
-		// Capture last assistant message for context passing
+		// 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 lastAssistant = messages.filter(m => m.role === 'assistant').pop();
-		let lastOutput = '';
-		if (lastAssistant?.content) {
-			const textParts = Array.isArray(lastAssistant.content) 
-				? lastAssistant.content.filter((c: any) => c.type === 'text').map((c: any) => c.text)
-				: [String(lastAssistant.content)];
-			lastOutput = textParts.join('\n').slice(0, 3000);
+		const filesWritten: string[] = [];
+		const filesEdited: string[] = [];
+		const errorsHit: string[] = [];
+		const testResults: string[] = [];
+		let toolCallCount = 0;
+
+		for (const msg of messages) {
+			if (msg.role === 'tool' || msg.role === 'function') {
+				toolCallCount++;
+				const content = Array.isArray(msg.content)
+					? msg.content.map((c: any) => c.text || '').join('')
+					: String(msg.content || '');
+				const name = (msg as any).name || '';
+				// 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));
+				}
+			}
 		}
 
-		// 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,6 +1,6 @@
 {
 	"name": "@phi-code-admin/phi-code",
-	"version": "0.70.0",
+	"version": "0.71.1",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"piConfig": {