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

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,16 +877,25 @@ 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
+		// 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.
 		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 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 lastOutput = bestOutput.slice(0, 4000);
 
 		// Inject previous phase output into next phase
 		if (lastOutput && phaseQueue.length > 0) {

package/package.json

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