@inceptionstack/pi-hard-no 1.0.2 → 1.1.0

package/architect.ts

@@ -44,11 +44,16 @@ You have tools available (read, bash, grep, find, ls) to explore the full codeba
 - Do public APIs have adequate comments/types?
 - Are new files/modules properly documented?
 
-## Response format
-If everything looks good at the big-picture level, say "LGTM — architecture looks solid."
-If there are issues, list them as bullet points with severity (high/medium/low).
-Focus on systemic issues that individual mini-reviews would miss.
-Do NOT repeat issues that were already found and fixed in mini-reviews.`;
+## Response format — terse, technical, low-bandwidth
+
+LGTM or bullets. Nothing else. Agent reads this, not a human — minimum words, maximum signal.
+
+- If everything is fine: one line — "LGTM — architecture looks solid."
+- Otherwise: bullet per issue, each: **<severity>:** <area/file> — <problem>; <fix>.
+- Max 3 lines per issue, 1 line preferred.
+- No preamble, no recap of what you checked, no praise.
+- Skip anything already fixed in mini-reviews.
+- Technical shorthand OK (race, coupling, leak, god-object, dead code).`;
 
 /**
  * Load architect review rules from .hardno/architect.md.

package/changes.ts

@@ -370,7 +370,16 @@ export function hasGitCommitCommand(toolCalls: TrackedToolCall[]): boolean {
   return toolCalls.some((tc) => {
     if (tc.name !== "bash") return false;
     const cmd = String(tc.input?.command ?? "");
-    return /\bgit(?:\s+-C\s+\S+)?\s+commit\b/.test(cmd);
+    // Direct: git commit
+    if (/\bgit(?:\s+-C\s+\S+)?\s+commit\b/.test(cmd)) return true;
+    // Subprocess wrapper: perl/python/node/ruby calling git commit
+    if (
+      /\b(?:python3?|node|perl|ruby)\b/.test(cmd) &&
+      /\bgit\b/.test(cmd) &&
+      /\bcommit\b/.test(cmd)
+    )
+      return true;
+    return false;
   });
 }
 

package/commands.ts

@@ -1,4 +1,4 @@
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 
 import { type AutoReviewSettings, configDirs } from "./settings";
 import { buildReviewPrompt } from "./prompt";

package/context.ts

@@ -6,7 +6,7 @@
  * Falls back gracefully when git is unavailable.
  */
 
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { truncateDiff } from "./helpers";
 import { filterIgnored } from "./ignore";
 import { log } from "./logger";
@@ -627,6 +627,12 @@ export async function getBestReviewContent(
 
   const allowLastCommitFallback = hasGitCommitCommand(agentToolCalls);
 
+  // Also allow last-commit fallback when the agent modified files via edit/write
+  // tools but the working tree is clean (implying an undetected commit occurred,
+  // e.g. via subprocess wrapper that evaded hasGitCommitCommand detection).
+  const agentModifiedFiles = agentToolCalls.some((tc) => tc.name === "write" || tc.name === "edit");
+  const shouldFallbackToLastCommit = allowLastCommitFallback || agentModifiedFiles;
+
   if (gitRoots && gitRoots.size > 0) {
     const result = await getContentFromGitRoots(
       pi,
@@ -635,7 +641,7 @@ export async function getBestReviewContent(
       summarySection,
       onStatus,
       lim,
-      allowLastCommitFallback,
+      shouldFallbackToLastCommit,
     );
     if (result) return result;
   }
@@ -643,7 +649,7 @@ export async function getBestReviewContent(
   const cwdResult = await getContentFromCwd(pi, ignorePatterns, summarySection, onStatus, lim);
   if (cwdResult) return cwdResult;
 
-  if (allowLastCommitFallback) {
+  if (shouldFallbackToLastCommit) {
     const lastCommitResult = await getContentFromLastCommit(
       pi,
       ignorePatterns,

package/git-roots.ts

@@ -6,7 +6,7 @@
 
 import { dirname, resolve, isAbsolute } from "node:path";
 import { homedir } from "node:os";
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 
 /**
  * Find the git repo root for a given directory.

package/index.ts

@@ -21,7 +21,7 @@
  *   or: cp index.ts ~/.pi/agent/extensions/pi-hard-no.ts
  */
 
-import { type ExtensionAPI, isToolCallEventType } from "@mariozechner/pi-coding-agent";
+import { type ExtensionAPI, isToolCallEventType } from "@earendil-works/pi-coding-agent";
 
 import {
   type AutoReviewSettings,

package/judge.ts

@@ -26,7 +26,7 @@ import {
   SessionManager,
   createAgentSession,
   type AgentSessionEvent,
-} from "@mariozechner/pi-coding-agent";
+} from "@earendil-works/pi-coding-agent";
 
 import { log } from "./logger";
 

package/message-sender.ts

@@ -1,4 +1,4 @@
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 
 import { log } from "./logger";
 import type { ReviewResult } from "./reviewer";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inceptionstack/pi-hard-no",
-  "version": "1.0.2",
+  "version": "1.1.0",
   "type": "module",
   "description": "Pi extension — automatic code review after every agent turn",
   "license": "MIT",
@@ -40,11 +40,11 @@
     "check": "npm run typecheck && npm run lint && npm run format:check && npm run test"
   },
   "peerDependencies": {
-    "@mariozechner/pi-coding-agent": "*"
+    "@earendil-works/pi-coding-agent": "*"
   },
   "devDependencies": {
     "@eslint/js": "^9.27.0",
-    "@mariozechner/pi-coding-agent": "^0.69.0",
+    "@earendil-works/pi-coding-agent": "^0.74.0",
     "@types/node": "^22.15.17",
     "eslint": "^9.27.0",
     "prettier": "^3.5.3",

package/prompt.ts

@@ -70,21 +70,33 @@ export const DEFAULT_AUTO_REVIEW_RULES = `## What to review (in priority order)
 
 // ── Part 3: Suffix (always included, not user-editable) ──
 
-export const PROMPT_SUFFIX = `## Response format
+export const PROMPT_SUFFIX = `## Output style — terse, technical, low-bandwidth
+
+Write as if on a slow link: minimum words, maximum signal. The agent reads this,
+not a human skimming prose.
+
+- Max 3 lines per issue. One line if location + problem + fix fits in one.
+- Each issue = location · problem · fix. Nothing else.
+- No preamble, no recap, no "I noticed", no "consider", no hedging.
+- Technical shorthand OK (e.g. "OBO", "race", "null deref", "unbounded loop").
+- Don't restate code — point to it by file:line.
+- Don't explain why the rule exists. Just the hit.
+
+## Response format
 
 Your response MUST follow this exact structure:
 
-1. (If issues found) List of bullet points, each: - **<Severity>:** <file/location> — <one-line explanation>
+1. (If issues found) List of bullet points, each: - **<Severity>:** <file/location> — <problem>; <fix>
    Severity is one of: High, Medium, Low.
 2. (If no issues) Write a single line: No issues found.
 3. On the final line of your response, output exactly ONE of these verdict tags:
    - <verdict>LGTM</verdict>  — if no real bugs were found
    - <verdict>ISSUES_FOUND</verdict>  — if you flagged any issue above
 
-## Example — issues found
+## Example — issues found (terse)
 
-    - **High:** test-bugs.ts:12 — Off-by-one error: i <= items.length should be i < items.length.
-    - **High:** test-bugs.ts:6 — Hardcoded API key sk-prod-... leaks a secret.
+    - **High:** test-bugs.ts:12 — OBO in \`for (i<=len)\`; use \`<\`.
+    - **High:** test-bugs.ts:6 — hardcoded key \`sk-prod-…\`; move to env.
 
     <verdict>ISSUES_FOUND</verdict>
 

package/reviewer.ts

@@ -17,7 +17,7 @@ import {
   AuthStorage,
   ModelRegistry,
   type AgentSessionEvent,
-} from "@mariozechner/pi-coding-agent";
+} from "@earendil-works/pi-coding-agent";
 
 import { log, logReview, safeStringify, type ReviewToolCall } from "./logger";
 

package/session-kind.ts

@@ -52,7 +52,7 @@
  * using distinct mock objects stay isolated without an explicit reset.
  */
 
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 
 import { log } from "./logger";