@geminixiang/pi-simplify 0.0.7 → 0.0.9

package/.pi/skills/release/SKILL.md

@@ -11,15 +11,15 @@ Repo defaults:
 - remote: `origin`
 - repo: `geminixiang/pi-simplify`
 - npm package: `@geminixiang/pi-simplify`
-- use `package.json` version as the git tag and GitHub release title
+- use `package.json` version for npm/package files, and `v<version>` for the git tag and GitHub release title
 - versions with `-alpha.`, `-beta.`, or `-rc.` are prereleases by default
 
 ## Version rules
 
 Examples:
 
-- stable: `0.2.1`, `0.3.0`, `1.0.0`
-- prerelease: `0.2.0-beta.8`, `0.3.0-rc.1`, `1.0.0-alpha.2`
+- stable package version: `0.2.1`, `0.3.0`, `1.0.0` → release tag/title: `v0.2.1`, `v0.3.0`, `v1.0.0`
+- prerelease package version: `0.2.0-beta.8`, `0.3.0-rc.1`, `1.0.0-alpha.2` → release tag/title: `v0.2.0-beta.8`, `v0.3.0-rc.1`, `v1.0.0-alpha.2`
 
 Guidance:
 
@@ -100,17 +100,17 @@ Write concise notes focused on user-visible changes, usually with:
 - `### Docs and maintenance`
 - `### Verification`
 
-Write notes to `/tmp/pi-simplify-release-<version>.md`. Keep them consistent with the CHANGELOG entry from step 3.
+Write notes to `/tmp/pi-simplify-release-<version>.md`. Keep them consistent with the CHANGELOG entry from step 3. Use `v<previous-version>...v<version>` in compare links.
 
 ### 6. Create or update release
 
 Prerelease:
 
 ```bash
-gh release create <version> \
+gh release create v<version> \
   --repo geminixiang/pi-simplify \
   --target main \
-  --title <version> \
+  --title v<version> \
   --notes-file /tmp/pi-simplify-release-<version>.md \
   --prerelease
 ```
@@ -118,14 +118,14 @@ gh release create <version> \
 Stable:
 
 ```bash
-gh release create <version> \
+gh release create v<version> \
   --repo geminixiang/pi-simplify \
   --target main \
-  --title <version> \
+  --title v<version> \
   --notes-file /tmp/pi-simplify-release-<version>.md
 ```
 
-If it already exists, use `gh release edit <version> ...` and keep prerelease/stable intent consistent.
+If it already exists, use `gh release edit v<version> ...` and keep prerelease/stable intent consistent.
 
 ## Report back
 

package/CHANGELOG.md

@@ -6,6 +6,29 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ## [Unreleased]
 
+## [0.0.9] - 2026-05-27
+
+### Added
+
+- Support simplifying specific folders and files as snapshot reviews.
+- Add a preset selector for choosing uncommitted changes or folder snapshot mode.
+
+### Fixed
+
+- Respect configured selection keybindings in simplify TUI selectors.
+- Keep release tags and GitHub release titles v-prefixed.
+
+## [0.0.8] - 2026-05-25
+
+### Added
+
+- Support simplifying specific folders and files as snapshot reviews.
+- Add a preset selector for choosing uncommitted changes or folder snapshot mode.
+
+### Fixed
+
+- Keep release tags and GitHub release titles v-prefixed.
+
 ## [0.0.7] - 2026-05-25
 
 ### Changed

package/README.md

@@ -30,12 +30,22 @@ pi install npm:@geminixiang/pi-simplify
 /simplify
 ```
 
-Analyzes all git changes and presents cleanup candidates:
+Opens a preset selector. Choose uncommitted changes or a folder snapshot.
+
+Uncommitted mode analyzes all git changes and presents cleanup candidates:
 
 - **Safe** (green) - auto-selected, will be deleted
 - **Confirm** (yellow) - delete after user confirms
 - **Review** (orange) - user should review first
 
+### Simplify Folders
+
+```
+/simplify folder src docs
+```
+
+Analyzes the specified folders/files as a snapshot (not a diff), even when there are no local git changes.
+
 ### With Focus
 
 ```

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geminixiang/pi-simplify",
-  "version": "0.0.7",
+  "version": "0.0.9",
   "description": "Simplify: Review changed code for reuse, quality, and efficiency",
   "keywords": [
     "pi-package"

package/prompt.ts

@@ -50,8 +50,8 @@ Before flagging, **search the codebase** (utility directories, shared modules, f
 
 ## How to Analyze
 
-1. Run \`git diff\` to see what changed.
-2. For each change, classify as **Essential** / **Residual** / **Legacy** (don't flag legacy code unless the diff touches it).
+1. Unless the user explicitly asks for a folder/snapshot review, run \`git diff\` to see what changed. For folder/snapshot review, read the files directly under the requested paths and ignore files outside that scope.
+2. For each change or in-scope snapshot finding, classify as **Essential** / **Residual** / **Legacy** (don't flag legacy code unless the diff touches it; in folder/snapshot mode, flag only in-scope code that is clearly removable or simplifiable).
 3. For each finding:
    - Identify exact file and line(s)
    - Assign **category** (reuse / quality / efficiency) and **risk**

package/selector.ts

@@ -1,11 +1,10 @@
-import { getSelectListTheme, type ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
 import {
-  matchesKey,
-  Key,
-  SelectList,
-  wrapTextWithAnsi,
-  type SelectItem,
-} from "@earendil-works/pi-tui";
+  getSelectListTheme,
+  keyHint,
+  rawKeyHint,
+  type ExtensionCommandContext,
+} from "@earendil-works/pi-coding-agent";
+import { matchesKey, SelectList, wrapTextWithAnsi, type SelectItem } from "@earendil-works/pi-tui";
 
 import type { Action, SimplifyResult } from "./types.js";
 
@@ -64,7 +63,7 @@ export async function showCandidateSelector(
       Math.min(selectedIndex - Math.floor(maxVisible / 2), selectableItems.length - maxVisible),
     );
 
-  const result = await ctx.ui.custom<SimplifyResult[]>((tui, theme, _keybindings, done) => {
+  const result = await ctx.ui.custom<SimplifyResult[]>((tui, theme, keybindings, done) => {
     const updateListLabel = (index: number) => {
       const item = listItems[index];
       const { candidate } = selectableItems[index];
@@ -82,6 +81,34 @@ export async function showCandidateSelector(
     };
     selectList.onCancel = () => done([]);
 
+    const submitSelection = () => {
+      done(Array.from(selectedSet).map((i) => selectableItems[i].candidate));
+    };
+
+    const cancelSelection = () => {
+      done([]);
+    };
+
+    const toggleCurrent = () => {
+      if (selectedSet.has(cursorPos)) {
+        selectedSet.delete(cursorPos);
+      } else {
+        selectedSet.add(cursorPos);
+      }
+      updateListLabel(cursorPos);
+      tui.requestRender();
+    };
+
+    const toggleAll = () => {
+      if (selectedSet.size === selectableItems.length) {
+        selectedSet.clear();
+      } else {
+        for (let i = 0; i < selectableItems.length; i++) selectedSet.add(i);
+      }
+      updateAllListLabels();
+      tui.requestRender();
+    };
+
     return {
       render(width: number) {
         selectList.setSelectedIndex(cursorPos);
@@ -119,14 +146,23 @@ export async function showCandidateSelector(
           );
           lines.push(theme.fg("muted", `${detailPrefix}Kind: ${current.category}`));
           lines.push(theme.fg("muted", `${detailPrefix}Fix: ${ACTION_LABEL[current.action]}`));
-          lines.push(theme.fg("borderMuted", `${detailIndent}└─ space selects this fix`));
+          lines.push(
+            theme.fg("borderMuted", `${detailIndent}└─ ${rawKeyHint("space", "selects this fix")}`),
+          );
         }
 
         lines.push("");
         lines.push(
           theme.fg(
             "muted",
-            `Selected: ${selectedSet.size} • ↑↓ navigate • space toggle • a select all • enter confirm • esc cancel`,
+            [
+              `Selected: ${selectedSet.size}`,
+              rawKeyHint("↑↓", "navigate"),
+              rawKeyHint("space", "toggle"),
+              rawKeyHint("a", "select all"),
+              keyHint("tui.select.confirm", "confirm"),
+              keyHint("tui.select.cancel", "cancel"),
+            ].join(" • "),
           ),
         );
         return lines;
@@ -135,35 +171,12 @@ export async function showCandidateSelector(
         selectList.invalidate();
       },
       handleInput(data: string) {
-        if (matchesKey(data, Key.enter)) {
-          const results = Array.from(selectedSet).map((i) => selectableItems[i].candidate);
-          done(results);
-          return;
-        }
-        if (matchesKey(data, Key.escape)) {
-          done([]);
-          return;
-        }
-        if (matchesKey(data, Key.space)) {
-          if (selectedSet.has(cursorPos)) {
-            selectedSet.delete(cursorPos);
-          } else {
-            selectedSet.add(cursorPos);
-          }
-          updateListLabel(cursorPos);
-          tui.requestRender();
-          return;
-        }
-        if (matchesKey(data, "a")) {
-          if (selectedSet.size === selectableItems.length) {
-            selectedSet.clear();
-          } else {
-            for (let i = 0; i < selectableItems.length; i++) selectedSet.add(i);
-          }
-          updateAllListLabels();
-          tui.requestRender();
-          return;
-        }
+        if (keybindings.matches(data, "tui.select.confirm")) return submitSelection();
+        if (keybindings.matches(data, "tui.select.cancel")) return cancelSelection();
+
+        // Extra shortcuts: these selector-specific actions are not registered pi keybinding ids.
+        if (matchesKey(data, "space")) return toggleCurrent();
+        if (matchesKey(data, "a")) return toggleAll();
 
         const previousCursorPos = cursorPos;
         selectList.handleInput(data);

package/workflow.ts

@@ -1,8 +1,11 @@
 import {
   defineTool,
+  DynamicBorder,
+  keyHint,
   type ExtensionAPI,
   type ExtensionCommandContext,
 } from "@earendil-works/pi-coding-agent";
+import { Container, SelectList, Text, type SelectItem } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
 
 import { SIMPLIFY_PROMPT } from "./prompt.js";
@@ -26,6 +29,58 @@ const actionSchema = Type.Union([
   Type.Literal("parallelize"),
 ]);
 
+type SimplifyTarget = { type: "uncommitted" } | { type: "folder"; paths: string[] };
+type ParsedArgs = {
+  target: SimplifyTarget | null;
+  additionalFocus?: string;
+  error?: string;
+};
+
+const SIMPLIFY_PRESETS = [
+  { value: "uncommitted", label: "Simplify uncommitted changes", description: "" },
+  { value: "folder", label: "Simplify a folder (or more)", description: "(snapshot, not diff)" },
+] as const;
+
+type SimplifyPresetValue = (typeof SIMPLIFY_PRESETS)[number]["value"];
+
+function parseSimplifyPaths(value: string): string[] {
+  return value
+    .split(/\s+/)
+    .map((item) => item.trim())
+    .filter((item) => item.length > 0);
+}
+
+function parseArgs(args: string): ParsedArgs {
+  const trimmed = args.trim();
+  if (!trimmed) return { target: null };
+
+  const [subcommand, ...rest] = trimmed.split(/\s+/);
+  if (subcommand?.toLowerCase() !== "folder") {
+    return { target: { type: "uncommitted" }, additionalFocus: trimmed };
+  }
+
+  const paths = parseSimplifyPaths(rest.join(" "));
+  if (paths.length === 0)
+    return { target: null, error: "Usage: /simplify folder <path> [path...]" };
+  return { target: { type: "folder", paths } };
+}
+
+async function hasUncommittedChanges(pi: ExtensionAPI): Promise<boolean> {
+  const { stdout, code } = await pi.exec("git", ["status", "--porcelain"]);
+  return code === 0 && stdout.trim().length > 0;
+}
+
+function buildSimplifyPrompt(target: SimplifyTarget, additionalFocus?: string): string {
+  let fullPrompt = SIMPLIFY_PROMPT;
+
+  if (target.type === "folder") {
+    fullPrompt += `\n\n## Scope Override: Folder Snapshot\n\nReview only the code in these repository-relative paths: ${target.paths.join(", ")}\n\nThis is a snapshot review, not a git diff review. Read files directly under these paths instead of relying on \`git diff\`. Only return candidates whose \`file\` is inside one of these paths.`;
+  }
+
+  if (additionalFocus?.trim()) fullPrompt += `\n\n## Additional Focus\n\n${additionalFocus.trim()}`;
+  return fullPrompt;
+}
+
 function registerSimplifyCandidatesTool(
   pi: ExtensionAPI,
   state: {
@@ -107,6 +162,79 @@ function registerSimplifyCandidatesTool(
   );
 }
 
+async function showFolderInput(ctx: ExtensionCommandContext): Promise<SimplifyTarget | null> {
+  const result = await ctx.ui.editor(
+    "Enter folders/files to simplify (space-separated or one per line):",
+    ".",
+  );
+
+  if (!result?.trim()) return null;
+  const paths = parseSimplifyPaths(result);
+  if (paths.length === 0) return null;
+
+  return { type: "folder", paths };
+}
+
+async function showSimplifyTargetSelector(
+  ctx: ExtensionCommandContext,
+  pi: ExtensionAPI,
+): Promise<SimplifyTarget | null> {
+  const presetItems: SelectItem[] = SIMPLIFY_PRESETS.map((preset) => ({
+    value: preset.value,
+    label: preset.label,
+    description: preset.description,
+  }));
+  const smartDefault = (await hasUncommittedChanges(pi)) ? "uncommitted" : "folder";
+  const smartDefaultIndex = presetItems.findIndex((item) => item.value === smartDefault);
+
+  while (true) {
+    const result = await ctx.ui.custom<SimplifyPresetValue | null>((tui, theme, _kb, done) => {
+      const container = new Container();
+      container.addChild(new DynamicBorder((str) => theme.fg("accent", str)));
+      container.addChild(new Text(theme.fg("accent", theme.bold("Select a simplify preset"))));
+
+      const selectList = new SelectList(presetItems, Math.min(presetItems.length, 10), {
+        selectedPrefix: (text) => theme.fg("accent", text),
+        selectedText: (text) => theme.fg("accent", text),
+        description: (text) => theme.fg("muted", text),
+        scrollInfo: (text) => theme.fg("dim", text),
+        noMatch: (text) => theme.fg("warning", text),
+      });
+
+      if (smartDefaultIndex >= 0) selectList.setSelectedIndex(smartDefaultIndex);
+      selectList.onSelect = (item) => done(item.value as SimplifyPresetValue);
+      selectList.onCancel = () => done(null);
+
+      container.addChild(selectList);
+      container.addChild(
+        new Text(
+          `${keyHint("tui.select.confirm", "confirm")}  ${keyHint("tui.select.cancel", "cancel")}`,
+        ),
+      );
+      container.addChild(new DynamicBorder((str) => theme.fg("accent", str)));
+
+      return {
+        render(width: number) {
+          return container.render(width);
+        },
+        invalidate() {
+          container.invalidate();
+        },
+        handleInput(data: string) {
+          selectList.handleInput(data);
+          tui.requestRender();
+        },
+      };
+    });
+
+    if (!result) return null;
+    if (result === "uncommitted") return { type: "uncommitted" };
+
+    const target = await showFolderInput(ctx);
+    if (target) return target;
+  }
+}
+
 async function applyFindings(
   ctx: ExtensionCommandContext,
   selected: SimplifyResult[],
@@ -167,7 +295,8 @@ export function registerSimplifyWorkflow(pi: ExtensionAPI) {
   });
 
   pi.registerCommand("simplify", {
-    description: "Review changed code for reuse, quality, and efficiency, then apply fixes",
+    description:
+      "Review changed code or folders for reuse, quality, and efficiency, then apply fixes",
     handler: async (args: string, ctx: ExtensionCommandContext) => {
       if (!ctx.hasUI) {
         ctx.ui.notify("Simplify requires interactive mode", "error");
@@ -188,16 +317,29 @@ export function registerSimplifyWorkflow(pi: ExtensionAPI) {
         return;
       }
 
-      const { stdout: gitStatus } = await pi.exec("git", ["status", "--porcelain"]);
-      if (!gitStatus.trim()) {
-        ctx.ui.notify("No git changes found. Simplify reviews uncommitted changes.", "info");
+      const parsed = parseArgs(args);
+      if (parsed.error) {
+        ctx.ui.notify(parsed.error, "error");
+        return;
+      }
+
+      let target = parsed.target;
+      if (!target) target = await showSimplifyTargetSelector(ctx, pi);
+      if (!target) {
+        ctx.ui.notify("Simplify cancelled", "info");
+        return;
+      }
+
+      if (target.type === "uncommitted" && !(await hasUncommittedChanges(pi))) {
+        ctx.ui.notify("No git changes found. Choose folder mode to simplify a snapshot.", "info");
         return;
       }
 
-      ctx.ui.notify("Analyzing code for review findings...", "info");
+      const targetHint =
+        target.type === "folder" ? `folders: ${target.paths.join(", ")}` : "uncommitted changes";
+      ctx.ui.notify(`Analyzing ${targetHint} for review findings...`, "info");
 
-      let fullPrompt = SIMPLIFY_PROMPT;
-      if (args.trim()) fullPrompt += `\n\n## Additional Focus\n\n${args.trim()}`;
+      const fullPrompt = buildSimplifyPrompt(target, parsed.additionalFocus);
 
       latestToolCandidates = null;
       const turnStarted = new Promise<void>((r) => {