@nielpattin/pi-simplify 0.2.1 → 0.2.3

package/CHANGELOG.md

@@ -0,0 +1,13 @@
+# Changelog
+
+## [0.2.1](https://github.com/MattDevy/pi-extensions/compare/pi-simplify-v0.2.0...pi-simplify-v0.2.1) (2026-05-14)
+
+### Bug Fixes
+
+- **deps:** Migrate to [@earendil-works](https://github.com/earendil-works) namespace ([7124e84](https://github.com/MattDevy/pi-extensions/commit/7124e846c29d92f605ceaed1c5581910a91d8d3d))
+
+## [0.2.0](https://github.com/MattDevy/pi-extensions/compare/pi-simplify-v0.1.0...pi-simplify-v0.2.0) (2026-04-10)
+
+### Features
+
+- Add pi-simplify code review extension ([#94](https://github.com/MattDevy/pi-extensions/issues/94)) ([aded3cd](https://github.com/MattDevy/pi-extensions/commit/aded3cd6ea928e93b50e787b4c6443bc2d3f8684))

package/package.json

@@ -1,6 +1,6 @@
 {
    "name": "@nielpattin/pi-simplify",
-   "version": "0.2.1",
+   "version": "0.2.3",
    "description": "A Pi extension that reviews recently changed code for clarity, consistency, and maintainability.",
    "keywords": [
       "ai",
@@ -31,9 +31,8 @@
    },
    "files": [
       "dist",
-      "src",
-      "!src/**/*.test.ts",
-      "README.md"
+      "README.md",
+      "CHANGELOG.md"
    ],
    "type": "module",
    "main": "./dist/index.js",
@@ -49,6 +48,10 @@
       "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": {
+      "@earendil-works/pi-ai": "^0.74.1",
+      "@earendil-works/pi-coding-agent": "^0.74.1",
+      "@earendil-works/pi-tui": "^0.74.1",
+      "@sinclair/typebox": "^0.34.49",
       "@types/node": "^25.7.0",
       "@typescript-eslint/eslint-plugin": "^8.0.0",
       "@typescript-eslint/parser": "^8.0.0",
@@ -67,7 +70,7 @@
    },
    "pi": {
       "extensions": [
-         "src/index.ts"
+         "./dist/index.js"
       ]
    }
 }

package/src/git-diff.ts

@@ -1,59 +0,0 @@
-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

@@ -1,9 +0,0 @@
-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

@@ -1,29 +0,0 @@
-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

@@ -1,42 +0,0 @@
-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

@@ -1,10 +0,0 @@
-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;
-}