@phi-code-admin/phi-code 0.69.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, 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,9 +31,13 @@ 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
+- Use the `write` tool to create review reports and documentation
+- NEVER use `bash` with `cat > file` as a workaround — always use the `write` tool directly
+
 ## Output Format
 
 ### 🔴 Critical / High

package/extensions/phi/orchestrator.ts

@@ -707,6 +707,12 @@ After implementation, use \`memory_write\` to save a summary of what was built,
 - NEVER run a server with \`&\` without cleanup. Always use: \`timeout 15 bash -c 'node src/index.js & PID=$!; sleep 2; curl ...; kill $PID'\`
 - ALWAYS kill background processes after testing
 - If a test hangs, use \`timeout\` to prevent deadlock
+- 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\`
+- 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.`,
 			},
@@ -871,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.69.0",
+	"version": "0.71.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"piConfig": {
@@ -61,7 +61,7 @@
 		"phi-code-tui": "^0.56.3",
 		"proper-lockfile": "^4.1.2",
 		"sigma-agents": "^0.1.6",
-		"sigma-memory": "0.2.0",
+		"sigma-memory": "0.2.1",
 		"sigma-skills": "^0.1.2",
 		"strip-ansi": "^7.1.0",
 		"undici": "^7.19.1",