@nielpattin/pi-simplify 0.2.1

package/README.md

@@ -0,0 +1,50 @@
+# pi-simplify
+
+A [Pi](https://github.com/nicholasgasior/pi-coding-agent) extension that reviews recently changed code for clarity, consistency, and maintainability improvements.
+
+## Installation
+
+```bash
+pi install npm:pi-simplify
+```
+
+## Usage
+
+### Review all uncommitted changes
+
+```
+/simplify
+```
+
+### Review only staged changes
+
+```
+/simplify --staged
+```
+
+### Review specific files
+
+```
+/simplify src/foo.ts src/bar.ts
+```
+
+### Diff against a specific branch
+
+```
+/simplify --ref=main
+```
+
+## What it does
+
+When invoked, `/simplify` detects changed files (via `git diff`) and instructs the agent to review them with these principles:
+
+- **Preserve functionality**: never change what the code does
+- **Apply project standards**: follow conventions from CLAUDE.md / AGENTS.md
+- **Enhance clarity**: reduce complexity, eliminate redundancy, improve naming
+- **Maintain balance**: avoid over-simplification
+
+The agent reads each file, applies improvements one at a time, runs tests to verify nothing breaks, and summarizes the changes.
+
+## License
+
+MIT

package/dist/git-diff.d.ts

@@ -0,0 +1,3 @@
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import type { ChangedFile, SimplifyOptions } from "./types.js";
+export declare function getChangedFiles(pi: ExtensionAPI, cwd: string, options: SimplifyOptions): Promise<ChangedFile[]>;

package/dist/git-diff.js

@@ -0,0 +1,50 @@
+const STATUS_MAP = {
+    M: "modified",
+    A: "added",
+    R: "renamed",
+    C: "copied",
+};
+function parseDiffOutput(stdout) {
+    const files = [];
+    for (const line of stdout.split("\n")) {
+        if (!line.trim())
+            continue;
+        const parts = line.split("\t");
+        const statusCode = parts[0]?.[0];
+        if (!statusCode)
+            continue;
+        const status = STATUS_MAP[statusCode];
+        if (!status)
+            continue;
+        // Renamed (R100\told\tnew) and copied (C100\told\tnew) have two paths; use the new one.
+        const path = status === "renamed" || status === "copied" ? parts[2] : parts[1];
+        if (path) {
+            files.push({ path, status });
+        }
+    }
+    return files;
+}
+export async function getChangedFiles(pi, cwd, options) {
+    if (options.files.length > 0) {
+        return options.files.map((path) => ({ path, status: "modified" }));
+    }
+    const args = ["diff", "--name-status"];
+    if (options.staged) {
+        args.push("--cached");
+    }
+    else {
+        args.push(options.ref);
+    }
+    const result = await pi.exec("git", args, { cwd });
+    if (result.code === 0) {
+        const files = parseDiffOutput(result.stdout);
+        if (files.length > 0)
+            return files;
+    }
+    // Fallback: diff against previous commit
+    const fallback = await pi.exec("git", ["diff", "--name-status", "HEAD~1"], { cwd });
+    if (fallback.code === 0) {
+        return parseDiffOutput(fallback.stdout);
+    }
+    return [];
+}

package/dist/index.d.ts

@@ -0,0 +1,2 @@
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+export default function (pi: ExtensionAPI): void;

package/dist/index.js

@@ -0,0 +1,7 @@
+import { handleSimplifyCommand, COMMAND_NAME } from "./simplify-command.js";
+export default function (pi) {
+    pi.registerCommand(COMMAND_NAME, {
+        description: "Review recently changed files for clarity, consistency, and maintainability improvements",
+        handler: (args, ctx) => handleSimplifyCommand(args, ctx, pi),
+    });
+}

package/dist/prompt-builder.d.ts

@@ -0,0 +1,2 @@
+import type { ChangedFile } from "./types.js";
+export declare function buildSimplifyPrompt(files: readonly ChangedFile[]): string;

package/dist/prompt-builder.js

@@ -0,0 +1,26 @@
+export function buildSimplifyPrompt(files) {
+    const fileList = files.map((f) => `- ${f.path} (${f.status})`).join("\n");
+    return `Review the following recently changed files and apply simplification improvements.
+
+## Principles
+
+- **Preserve functionality**: Never change what the code does. All existing tests must continue to pass.
+- **Apply project standards**: Follow any conventions from CLAUDE.md or AGENTS.md in this project.
+- **Enhance clarity**: Reduce unnecessary complexity and nesting, eliminate redundant code and abstractions, improve variable and function names, consolidate related logic, remove unnecessary comments that describe obvious code. Avoid nested ternary operators: prefer switch statements or if/else chains for multiple conditions.
+- **Maintain balance**: Do not over-simplify. Avoid overly clever solutions that are hard to understand. Do not combine too many concerns into single functions. Do not remove helpful abstractions. Prioritize readability over fewer lines.
+
+## Scope
+
+Only review and modify these files:
+${fileList}
+
+## Process
+
+1. Read each file listed above
+2. Identify concrete improvements (dead code, unclear names, redundant logic, inconsistent patterns)
+3. Apply changes one file at a time
+4. After all changes, run existing tests to verify nothing is broken
+5. Summarize what you changed and why
+
+Do NOT add new features, change public APIs, or refactor code outside the listed files.`;
+}

package/dist/simplify-command.d.ts

@@ -0,0 +1,5 @@
+import type { ExtensionAPI, ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
+import type { SimplifyOptions } from "./types.js";
+export declare const COMMAND_NAME = "simplify";
+export declare function parseArgs(args: string): SimplifyOptions;
+export declare function handleSimplifyCommand(args: string, ctx: ExtensionCommandContext, pi: ExtensionAPI): Promise<void>;

package/dist/simplify-command.js

@@ -0,0 +1,31 @@
+import { getChangedFiles } from "./git-diff.js";
+import { buildSimplifyPrompt } from "./prompt-builder.js";
+export const COMMAND_NAME = "simplify";
+export function parseArgs(args) {
+    const tokens = args.trim().split(/\s+/).filter(Boolean);
+    const files = [];
+    let ref = "HEAD";
+    let staged = false;
+    for (const token of tokens) {
+        if (token === "--staged") {
+            staged = true;
+        }
+        else if (token.startsWith("--ref=")) {
+            ref = token.slice("--ref=".length);
+        }
+        else {
+            files.push(token);
+        }
+    }
+    return { files, ref, staged };
+}
+export async function handleSimplifyCommand(args, ctx, pi) {
+    const options = parseArgs(args);
+    const files = await getChangedFiles(pi, ctx.cwd, options);
+    if (files.length === 0) {
+        ctx.ui.notify("No changed files found. Specify file paths or make some changes first.", "info");
+        return;
+    }
+    const prompt = buildSimplifyPrompt(files);
+    pi.sendUserMessage(prompt, { deliverAs: "followUp" });
+}

package/dist/types.d.ts

@@ -0,0 +1,9 @@
+export interface ChangedFile {
+    readonly path: string;
+    readonly status: "modified" | "added" | "renamed" | "copied";
+}
+export interface SimplifyOptions {
+    readonly files: readonly string[];
+    readonly ref: string;
+    readonly staged: boolean;
+}

package/package.json

@@ -0,0 +1,73 @@
+{
+   "name": "@nielpattin/pi-simplify",
+   "version": "0.2.1",
+   "description": "A Pi extension that reviews recently changed code for clarity, consistency, and maintainability.",
+   "keywords": [
+      "ai",
+      "ai-agent",
+      "code-review",
+      "coding-assistant",
+      "developer-tools",
+      "llm",
+      "pi-coding-agent",
+      "pi-extension",
+      "pi-package",
+      "refactor",
+      "simplify"
+   ],
+   "homepage": "https://github.com/nielpattin/pi-packages/tree/main/pi-simplify#readme",
+   "bugs": {
+      "url": "https://github.com/nielpattin/pi-packages/issues"
+   },
+   "license": "MIT",
+   "author": "Matt Devy",
+   "repository": {
+      "type": "git",
+      "url": "git+https://github.com/nielpattin/pi-packages.git",
+      "directory": "pi-simplify"
+   },
+   "publishConfig": {
+      "access": "public"
+   },
+   "files": [
+      "dist",
+      "src",
+      "!src/**/*.test.ts",
+      "README.md"
+   ],
+   "type": "module",
+   "main": "./dist/index.js",
+   "types": "./dist/index.d.ts",
+   "scripts": {
+      "clean": "node -e \"const fs=require('node:fs');fs.rmSync('dist',{recursive:true,force:true});fs.rmSync('tsconfig.build.tsbuildinfo',{force:true});\"",
+      "build": "npm run clean && tsc -p tsconfig.build.json",
+      "typecheck": "tsc --noEmit",
+      "test": "vitest run",
+      "lint": "eslint src/",
+      "check": "vitest run && eslint src/ && tsc --noEmit",
+      "prepublishOnly": "npm run build && npm run check",
+      "prepack": "node -e \"const fs=require('node:fs');if(!fs.existsSync('dist')){console.error('Error: dist/ missing. Run npm run build first.');process.exit(1);}\""
+   },
+   "devDependencies": {
+      "@types/node": "^25.7.0",
+      "@typescript-eslint/eslint-plugin": "^8.0.0",
+      "@typescript-eslint/parser": "^8.0.0",
+      "eslint": "^10.3.0",
+      "typescript": "^6.0.3",
+      "vitest": "^4.1.6"
+   },
+   "peerDependencies": {
+      "@earendil-works/pi-ai": "^0.74.0",
+      "@earendil-works/pi-coding-agent": "^0.74.0",
+      "@earendil-works/pi-tui": "^0.74.0",
+      "@sinclair/typebox": "^0.34.0"
+   },
+   "engines": {
+      "node": ">=18"
+   },
+   "pi": {
+      "extensions": [
+         "src/index.ts"
+      ]
+   }
+}

package/src/git-diff.ts

@@ -0,0 +1,59 @@
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import type { ChangedFile, SimplifyOptions } from "./types.js";
+
+const STATUS_MAP: Record<string, ChangedFile["status"]> = {
+   M: "modified",
+   A: "added",
+   R: "renamed",
+   C: "copied",
+};
+
+function parseDiffOutput(stdout: string): ChangedFile[] {
+   const files: ChangedFile[] = [];
+
+   for (const line of stdout.split("\n")) {
+      if (!line.trim()) continue;
+
+      const parts = line.split("\t");
+      const statusCode = parts[0]?.[0];
+      if (!statusCode) continue;
+
+      const status = STATUS_MAP[statusCode];
+      if (!status) continue;
+
+      // Renamed (R100\told\tnew) and copied (C100\told\tnew) have two paths; use the new one.
+      const path = status === "renamed" || status === "copied" ? parts[2] : parts[1];
+      if (path) {
+         files.push({ path, status });
+      }
+   }
+
+   return files;
+}
+
+export async function getChangedFiles(pi: ExtensionAPI, cwd: string, options: SimplifyOptions): Promise<ChangedFile[]> {
+   if (options.files.length > 0) {
+      return options.files.map((path) => ({ path, status: "modified" as const }));
+   }
+
+   const args = ["diff", "--name-status"];
+   if (options.staged) {
+      args.push("--cached");
+   } else {
+      args.push(options.ref);
+   }
+
+   const result = await pi.exec("git", args, { cwd });
+   if (result.code === 0) {
+      const files = parseDiffOutput(result.stdout);
+      if (files.length > 0) return files;
+   }
+
+   // Fallback: diff against previous commit
+   const fallback = await pi.exec("git", ["diff", "--name-status", "HEAD~1"], { cwd });
+   if (fallback.code === 0) {
+      return parseDiffOutput(fallback.stdout);
+   }
+
+   return [];
+}

package/src/index.ts

@@ -0,0 +1,9 @@
+import type { ExtensionAPI, ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
+import { handleSimplifyCommand, COMMAND_NAME } from "./simplify-command.js";
+
+export default function (pi: ExtensionAPI): void {
+   pi.registerCommand(COMMAND_NAME, {
+      description: "Review recently changed files for clarity, consistency, and maintainability improvements",
+      handler: (args: string, ctx: ExtensionCommandContext) => handleSimplifyCommand(args, ctx, pi),
+   });
+}

package/src/prompt-builder.ts

@@ -0,0 +1,29 @@
+import type { ChangedFile } from "./types.js";
+
+export function buildSimplifyPrompt(files: readonly ChangedFile[]): string {
+   const fileList = files.map((f) => `- ${f.path} (${f.status})`).join("\n");
+
+   return `Review the following recently changed files and apply simplification improvements.
+
+## Principles
+
+- **Preserve functionality**: Never change what the code does. All existing tests must continue to pass.
+- **Apply project standards**: Follow any conventions from CLAUDE.md or AGENTS.md in this project.
+- **Enhance clarity**: Reduce unnecessary complexity and nesting, eliminate redundant code and abstractions, improve variable and function names, consolidate related logic, remove unnecessary comments that describe obvious code. Avoid nested ternary operators: prefer switch statements or if/else chains for multiple conditions.
+- **Maintain balance**: Do not over-simplify. Avoid overly clever solutions that are hard to understand. Do not combine too many concerns into single functions. Do not remove helpful abstractions. Prioritize readability over fewer lines.
+
+## Scope
+
+Only review and modify these files:
+${fileList}
+
+## Process
+
+1. Read each file listed above
+2. Identify concrete improvements (dead code, unclear names, redundant logic, inconsistent patterns)
+3. Apply changes one file at a time
+4. After all changes, run existing tests to verify nothing is broken
+5. Summarize what you changed and why
+
+Do NOT add new features, change public APIs, or refactor code outside the listed files.`;
+}

package/src/simplify-command.ts

@@ -0,0 +1,42 @@
+import type { ExtensionAPI, ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
+import { getChangedFiles } from "./git-diff.js";
+import { buildSimplifyPrompt } from "./prompt-builder.js";
+import type { SimplifyOptions } from "./types.js";
+
+export const COMMAND_NAME = "simplify";
+
+export function parseArgs(args: string): SimplifyOptions {
+   const tokens = args.trim().split(/\s+/).filter(Boolean);
+   const files: string[] = [];
+   let ref = "HEAD";
+   let staged = false;
+
+   for (const token of tokens) {
+      if (token === "--staged") {
+         staged = true;
+      } else if (token.startsWith("--ref=")) {
+         ref = token.slice("--ref=".length);
+      } else {
+         files.push(token);
+      }
+   }
+
+   return { files, ref, staged };
+}
+
+export async function handleSimplifyCommand(
+   args: string,
+   ctx: ExtensionCommandContext,
+   pi: ExtensionAPI,
+): Promise<void> {
+   const options = parseArgs(args);
+   const files = await getChangedFiles(pi, ctx.cwd, options);
+
+   if (files.length === 0) {
+      ctx.ui.notify("No changed files found. Specify file paths or make some changes first.", "info");
+      return;
+   }
+
+   const prompt = buildSimplifyPrompt(files);
+   pi.sendUserMessage(prompt, { deliverAs: "followUp" });
+}

package/src/types.ts

@@ -0,0 +1,10 @@
+export interface ChangedFile {
+   readonly path: string;
+   readonly status: "modified" | "added" | "renamed" | "copied";
+}
+
+export interface SimplifyOptions {
+   readonly files: readonly string[];
+   readonly ref: string;
+   readonly staged: boolean;
+}