pi-shit 0.1.0

package/README.md

@@ -0,0 +1,44 @@
+# pi-shit
+
+Combined Pi package for personal extensions + skills.
+
+## Structure
+
+- `extensions/` → Pi extensions (for example `deep-review`, `plan-mode`)
+- `skills/` → Pi skills (including `pr-context-packer`)
+- `themes/` → Pi themes (`rose-pine`, `rose-pine-dawn`)
+
+## Install in Pi
+
+```bash
+pi install npm:pi-shit
+```
+
+Or from git/local:
+
+```bash
+pi install git:github.com/ferologics/pi-shit
+pi install /path/to/pi-shit
+```
+
+## Sync
+
+This repo is assembled from:
+
+- `skills/` ← `pi-skills` (git subtree)
+- `extensions/` ← `pi-extensions` (git subtree)
+- `themes/` ← `zenobi-us/pi-rose-pine` (via `just update-themes`)
+
+Update all sources with:
+
+```bash
+just update
+```
+
+Or update individually:
+
+```bash
+just update-skills
+just update-extensions
+just update-themes
+```

package/extensions/README.md

@@ -0,0 +1,23 @@
+# pi-extensions
+
+Custom extensions for [pi-coding-agent](https://github.com/badlogic/pi-mono).
+
+## Extensions
+
+| Extension                     | Description                                                                   |
+| ----------------------------- | ----------------------------------------------------------------------------- |
+| [`deep-review`](deep-review/) | Run context-pack + direct OpenAI Responses deep review with live streaming UI |
+| [`pi-notify`](pi-notify/)     | Desktop notifications when the agent finishes                                 |
+| [`plan-mode`](plan-mode/)     | Read-only plan mode with progress tracking and questionnaire support          |
+
+Each extension folder contains full usage details and examples.
+
+## Setup
+
+Symlink extensions to `~/.pi/agent/extensions/`:
+
+```bash
+ln -s ~/dev/pi-extensions/deep-review ~/.pi/agent/extensions/
+ln -s ~/dev/pi-extensions/pi-notify ~/.pi/agent/extensions/
+ln -s ~/dev/pi-extensions/plan-mode ~/.pi/agent/extensions/
+```

package/extensions/deep-review/README.md

@@ -0,0 +1,56 @@
+# Deep Review Extension
+
+Run a 2-phase deep PR review flow inside your current Pi session:
+
+1. Context packing via nested `pi -p` + `/skill:pr-context-packer`
+2. Direct OpenAI Responses API streaming (thinking + final answer)
+
+## Commands
+
+- `/deep-review <query> [options]`
+- `/deep-review-stop`
+
+## Defaults
+
+- Model: `gpt-5.2`
+- Reasoning effort: `xhigh`
+- Summary: `auto` (shows readable reasoning summary deltas)
+- Verbosity: `medium`
+- Base ref: auto-detected by `pr-context-packer`
+
+## Options
+
+- `--query <text>` (alternative to positional query; cannot be combined with positional query text)
+- `--project <path>`
+- `--base <ref>`
+- `--model <id>`
+- `--effort minimal|low|medium|high|xhigh`
+- `--verbosity low|medium|high`
+- `--summary auto|detailed|null`
+- `--no-summary` (shortcut for `--summary null`)
+- `--org <id>`
+- `--project-id <id>`
+- `--debug`
+- `--help`
+
+## Example
+
+```text
+/deep-review "find bugs and regressions"
+```
+
+## Notes
+
+- The context-pack subprocess is launched with explicit skill scope:
+  - `--no-skills --skill <bundled skills/pr-context-packer/SKILL.md>`
+- `deep-review` fails fast if that bundled skill file is missing.
+- Intended package layout is `pi-shit` (`extensions/` + `skills/` in one package).
+- Scribe expansion stays enabled in context-packer (no disable flag in this extension).
+- Use `--no-summary` if you want parity mode without readable reasoning summary text.
+- The command updates Pi UI live with a compact status widget (phase + stream progress).
+- Streamed thinking/answer text is not previewed live in the widget; full markdown answer is posted at completion.
+- Context-pack stage output is posted into the current session.
+- Final response includes duration, token usage, and estimated cost.
+- Final answer/thinking/report are written to a temp output folder and linked in the result.
+- The extension attempts to copy the final answer to clipboard automatically.
+- For auth, `OPENAI_API_KEY` is recommended.

package/extensions/deep-review/index.test.ts

@@ -0,0 +1,97 @@
+import { beforeAll, describe, expect, it, vi } from "vitest";
+import * as os from "node:os";
+import * as path from "node:path";
+import { mkdtemp, writeFile } from "node:fs/promises";
+
+vi.mock("@mariozechner/pi-ai", () => ({
+    calculateCost: vi.fn(),
+    getModel: vi.fn(),
+}));
+
+vi.mock("@mariozechner/pi-coding-agent", () => ({
+    getMarkdownTheme: vi.fn(() => ({})),
+}));
+
+vi.mock("@mariozechner/pi-tui", () => ({
+    Markdown: class {},
+}));
+
+let splitArgs: (input: string, platform?: NodeJS.Platform) => string[];
+let parseOptions: (rawArgs: string, cwd: string) => { ok: boolean; message?: string };
+let extractContextPackPath: (output: string) => Promise<string | undefined>;
+let normalizeSectionLikeBoldMarkdown: (markdown: string) => string;
+let parseSseStream: (
+    body: ReadableStream<Uint8Array>,
+) => AsyncGenerator<{ type?: string; [key: string]: unknown }, void, void>;
+
+beforeAll(async () => {
+    const mod = await import("./index.js");
+    splitArgs = mod.splitArgs;
+    parseOptions = mod.parseOptions;
+    extractContextPackPath = mod.extractContextPackPath;
+    normalizeSectionLikeBoldMarkdown = mod.normalizeSectionLikeBoldMarkdown;
+    parseSseStream = mod.parseSseStream;
+});
+
+describe("splitArgs", () => {
+    it("keeps Windows backslashes on win32", () => {
+        const tokens = splitArgs('--project C:\\repo --query "review"', "win32");
+        expect(tokens).toEqual(["--project", "C:\\repo", "--query", "review"]);
+    });
+
+    it("supports escaped spaces on posix", () => {
+        const tokens = splitArgs("--project /tmp/my\\ repo --query test", "darwin");
+        expect(tokens).toEqual(["--project", "/tmp/my repo", "--query", "test"]);
+    });
+});
+
+describe("parseOptions", () => {
+    it("rejects positional + --query together", () => {
+        const parsed = parseOptions('positional --query "flag"', "/tmp");
+        expect(parsed.ok).toBe(false);
+        expect(parsed.message).toContain("both positionally and via --query");
+    });
+});
+
+describe("extractContextPackPath", () => {
+    it("returns an existing candidate path", async () => {
+        const dir = await mkdtemp(path.join(os.tmpdir(), "deep-review-test-"));
+        const realPath = path.join(dir, "pr-context.txt");
+        await writeFile(realPath, "test", "utf8");
+
+        const output = `Output: /nope/pr-context.txt\nOutput: ${realPath}`;
+        const extracted = await extractContextPackPath(output);
+
+        expect(extracted).toBe(realPath);
+    });
+});
+
+describe("normalizeSectionLikeBoldMarkdown", () => {
+    it("converts standalone bold lines to headings", () => {
+        const input = "**Overview**\nBody text";
+        const output = normalizeSectionLikeBoldMarkdown(input);
+        expect(output).toContain("### Overview");
+        expect(output).toContain("Body text");
+    });
+});
+
+describe("parseSseStream", () => {
+    it("parses JSON SSE events and ignores DONE", async () => {
+        const encoder = new TextEncoder();
+        const stream = new ReadableStream<Uint8Array>({
+            start(controller) {
+                controller.enqueue(encoder.encode('data: {"type":"response.created"}\n\n'));
+                controller.enqueue(encoder.encode("data: [DONE]\n\n"));
+                controller.close();
+            },
+        });
+
+        const events: Array<{ type?: string }> = [];
+        for await (const event of parseSseStream(stream)) {
+            events.push(event);
+        }
+
+        expect(events).toHaveLength(1);
+        expect(events[0].type).toBe("response.created");
+    });
+});