@rootplatform/review-my-pr 1.0.0

package/README.md

@@ -0,0 +1,164 @@
+# review-my-pr
+
+CLI tool that generates a code review prompt from your git diff and copies it to the clipboard. Paste it into any LLM chat to get an instant code review.
+
+
+## Usage
+
+Run from any git repository on a feature branch:
+
+```bash
+# Auto-detects main/master as the base branch
+review-my-pr
+
+# Specify a base branch
+review-my-pr --base develop
+
+# Exclude additional file patterns
+review-my-pr --exclude "*.generated.ts" --exclude "docs/**"
+```
+
+### Options
+
+| Flag | Description |
+|---|---|
+| `-b, --base <branch>` | Base branch to diff against. Auto-detects `origin/main`, `origin/master`, `main`, or `master` if not specified. |
+| `-e, --exclude <pattern>` | Additional git pathspec exclusion pattern. Can be repeated for multiple patterns. |
+| `-V, --version` | Show version number |
+| `-h, --help` | Show help |
+
+### Example Output
+
+```
+Generating diff from origin/master...
+✅ Diff generated successfully
+✅ Found 2 changed file(s) (65 lines)
+
+✅ Review prompt copied to clipboard!
+Just paste it into your chat with Cmd+V
+```
+
+## Configuration
+
+Create a `.review-my-pr/` directory in your repository root to customize the review prompt. The directory contains two files:
+
+```
+.review-my-pr/
+  prompt.md       # The review prompt template
+  config.json     # Category definitions and settings
+```
+
+If this directory is not present, a sensible default prompt is used.
+
+### `prompt.md` - Prompt Template
+
+Use these placeholders in your template and they will be replaced with actual values:
+
+| Placeholder | Description |
+|---|---|
+| `{{DIFF}}` | The full git diff content |
+| `{{BASE_BRANCH}}` | The base branch name (e.g. `origin/master`) |
+| `{{CHANGED_FILES}}` | List of all changed files |
+| `{{SRC_FILES}}` | Source files only (excludes tests, mocks, fixtures) |
+| `{{TEST_FILES}}` | Test/spec files only |
+| `{{MIGRATION_FILES}}` | Database migration files |
+| `{{HTTP_FILES}}` | HTTP layer files |
+| `{{PR_TITLE}}` | PR title (requires `gh` CLI) |
+| `{{PR_BODY}}` | PR description body (requires `gh` CLI) |
+| `{{PR_METADATA}}` | Formatted PR title + description block |
+| `{{CATEGORY.NAME}}` | Custom category defined in `config.json` (see below) |
+
+If `{{DIFF}}` is not present in your template, the diff will be appended automatically at the end.
+
+### `config.json` - Custom File Categories
+
+Define custom file categories to group changed files by regex patterns. Reference them in `prompt.md` with `{{CATEGORY.NAME}}`.
+
+```json
+{
+  "categories": {
+    "SRC_FILES": {
+      "exclude": "\\.(test|spec)\\.(ts|tsx|js|jsx)$|__mocks__|fixtures"
+    },
+    "TEST_FILES": {
+      "pattern": "\\.(test|spec)\\.(ts|tsx|js|jsx)$"
+    },
+    "MIGRATION_FILES": {
+      "pattern": "migrations/\\d+_.*\\.(ts|js)$"
+    },
+    "SCHEMAS": {
+      "pattern": "\\.graphql$|\\.schema\\.ts$"
+    }
+  }
+}
+```
+
+| Property | Description |
+|---|---|
+| `pattern` | Regex to include matching files. If omitted, all files are included (before `exclude` is applied). |
+| `exclude` | Regex to exclude matching files. Applied after `pattern`. |
+
+Categories are independent -- a file can appear in multiple categories.
+
+### Example
+
+`.review-my-pr/config.json`:
+
+```json
+{
+  "categories": {
+    "SRC_FILES": { "exclude": "\\.(test|spec)\\.(ts|tsx|js|jsx)$|__mocks__|fixtures" },
+    "TEST_FILES": { "pattern": "\\.(test|spec)\\.(ts|tsx|js|jsx)$" }
+  }
+}
+```
+
+`.review-my-pr/prompt.md`:
+
+```markdown
+You are an expert code reviewer for our TypeScript backend.
+
+## Context
+- Stack: TypeScript, Node.js, PostgreSQL
+- Focus: bugs, security, performance, test coverage
+
+## Source Files
+{{CATEGORY.SRC_FILES}}
+
+## Test Files
+{{CATEGORY.TEST_FILES}}
+
+## Diff
+
+\`\`\`diff
+{{DIFF}}
+\`\`\`
+```
+
+The built-in placeholders (`{{SRC_FILES}}`, `{{TEST_FILES}}`, etc.) still work and use the default categorization patterns. Custom categories via `{{CATEGORY.*}}` give you full control over the regex patterns.
+
+## Default Exclusions
+
+The following file patterns are always excluded from the diff:
+
+- `**/package-lock.json`
+- `**/yarn.lock`
+- `**/pnpm-lock.yaml`
+- `**/test-results/**`
+- `**/playwright-report/**`
+
+Use `--exclude` to add more patterns on top of these.
+
+## PR Metadata
+
+If you have the [GitHub CLI](https://cli.github.com/) (`gh`) installed and your branch has an open PR, the tool will automatically include the PR title and description in the prompt.
+
+## Development
+
+```bash
+git clone <repo-url>
+cd review-my-pr
+npm install
+npm run build
+npm link   # makes `review-my-pr` available globally
+```

package/dist/index.d.ts

@@ -0,0 +1,2 @@
+#!/usr/bin/env node
+export {};

package/dist/index.js

@@ -0,0 +1,18 @@
+#!/usr/bin/env node
+import { Command } from "commander";
+import { runReview } from "./review.js";
+const program = new Command();
+program
+    .name("review-my-pr")
+    .description("Generate a code review prompt from your git diff and copy it to the clipboard")
+    .version("1.0.0")
+    .option("-b, --base <branch>", "base branch to diff against (default: auto-detect main/master)")
+    .option("-e, --exclude <pattern>", "additional git pathspec exclusion (repeatable)", (val, acc) => [...acc, val], [])
+    .action(async (options) => {
+    await runReview({
+        base: options.base,
+        exclude: options.exclude,
+    });
+});
+program.parse();
+//# sourceMappingURL=index.js.map

package/dist/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";AAEA,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AACpC,OAAO,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAExC,MAAM,OAAO,GAAG,IAAI,OAAO,EAAE,CAAC;AAE9B,OAAO;KACJ,IAAI,CAAC,cAAc,CAAC;KACpB,WAAW,CACV,+EAA+E,CAChF;KACA,OAAO,CAAC,OAAO,CAAC;KAChB,MAAM,CAAC,qBAAqB,EAAE,gEAAgE,CAAC;KAC/F,MAAM,CACL,yBAAyB,EACzB,gDAAgD,EAChD,CAAC,GAAW,EAAE,GAAa,EAAE,EAAE,CAAC,CAAC,GAAG,GAAG,EAAE,GAAG,CAAC,EAC7C,EAAc,CACf;KACA,MAAM,CAAC,KAAK,EAAE,OAA8C,EAAE,EAAE;IAC/D,MAAM,SAAS,CAAC;QACd,IAAI,EAAE,OAAO,CAAC,IAAI;QAClB,OAAO,EAAE,OAAO,CAAC,OAAO;KACzB,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEL,OAAO,CAAC,KAAK,EAAE,CAAC"}

package/dist/lib/clipboard.d.ts

@@ -0,0 +1,2 @@
+export declare function copyToClipboard(text: string): Promise<void>;
+export declare function getPasteHint(): string;

package/dist/lib/clipboard.js

@@ -0,0 +1,10 @@
+import clipboardy from "clipboardy";
+export async function copyToClipboard(text) {
+    await clipboardy.write(text);
+}
+export function getPasteHint() {
+    return process.platform === "darwin"
+        ? "Just paste it into your chat with Cmd+V"
+        : "Just paste it into your chat with Ctrl+V";
+}
+//# sourceMappingURL=clipboard.js.map

package/dist/lib/clipboard.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"clipboard.js","sourceRoot":"","sources":["../../src/lib/clipboard.ts"],"names":[],"mappings":"AAAA,OAAO,UAAU,MAAM,YAAY,CAAC;AAEpC,MAAM,CAAC,KAAK,UAAU,eAAe,CAAC,IAAY;IAChD,MAAM,UAAU,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;AAC/B,CAAC;AAED,MAAM,UAAU,YAAY;IAC1B,OAAO,OAAO,CAAC,QAAQ,KAAK,QAAQ;QAClC,CAAC,CAAC,yCAAyC;QAC3C,CAAC,CAAC,0CAA0C,CAAC;AACjD,CAAC"}

package/dist/lib/git.d.ts

@@ -0,0 +1,27 @@
+export declare function isGitRepo(): boolean;
+export declare function getRepoRoot(): string;
+export declare function detectBaseBranch(explicit?: string): string;
+export declare function ensureBranchUpToDate(baseBranch: string): void;
+export interface DiffResult {
+    content: string;
+    changedFiles: string[];
+    lineCount: number;
+}
+export declare function getDiff(baseBranch: string, extraExclusions?: string[]): DiffResult;
+export interface CategorizedFiles {
+    source: string[];
+    test: string[];
+    migration: string[];
+    http: string[];
+}
+export declare function categorizeFiles(files: string[]): CategorizedFiles;
+export interface CategoryDefinition {
+    pattern?: string;
+    exclude?: string;
+}
+export declare function categorizeFilesWithConfig(files: string[], categories: Record<string, CategoryDefinition>): Record<string, string[]>;
+export interface PrMetadata {
+    title: string;
+    body: string;
+}
+export declare function getPrMetadata(): PrMetadata | null;

package/dist/lib/git.js

@@ -0,0 +1,127 @@
+import { execFileSync, execSync } from "node:child_process";
+export function isGitRepo() {
+    try {
+        execSync("git rev-parse --is-inside-work-tree", { stdio: "pipe" });
+        return true;
+    }
+    catch {
+        return false;
+    }
+}
+export function getRepoRoot() {
+    return execSync("git rev-parse --show-toplevel", { stdio: "pipe" })
+        .toString()
+        .trim();
+}
+function branchExists(branch) {
+    try {
+        execSync(`git rev-parse --verify ${branch}`, { stdio: "pipe" });
+        return true;
+    }
+    catch {
+        return false;
+    }
+}
+export function detectBaseBranch(explicit) {
+    if (explicit) {
+        if (!branchExists(explicit)) {
+            throw new Error(`Branch '${explicit}' does not exist`);
+        }
+        return explicit;
+    }
+    const candidates = ["origin/main", "origin/master", "main", "master"];
+    for (const branch of candidates) {
+        if (branchExists(branch)) {
+            return branch;
+        }
+    }
+    throw new Error("Could not detect a base branch. No main/master branch found.\n" +
+        "Use --base <branch> to specify one explicitly.");
+}
+export function ensureBranchUpToDate(baseBranch) {
+    const behindCount = execFileSync("git", ["rev-list", "--count", `HEAD..${baseBranch}`], { stdio: "pipe" })
+        .toString()
+        .trim();
+    if (parseInt(behindCount, 10) > 0) {
+        throw new Error(`Your branch is behind ${baseBranch} by ${behindCount} commit(s). ` +
+            `Please update your branch and try again.`);
+    }
+}
+const DEFAULT_EXCLUSIONS = [
+    ":(exclude)**/package-lock.json",
+    ":(exclude)**/yarn.lock",
+    ":(exclude)**/pnpm-lock.yaml",
+    ":(exclude)**/test-results/**",
+    ":(exclude)**/playwright-report/**",
+];
+export function getDiff(baseBranch, extraExclusions = []) {
+    const exclusions = [
+        ...DEFAULT_EXCLUSIONS,
+        ...extraExclusions.map((e) => `:(exclude)${e}`),
+    ];
+    const pathspec = ["--", ".", ...exclusions];
+    const content = execFileSync("git", ["diff", baseBranch, ...pathspec], {
+        stdio: "pipe",
+        maxBuffer: 50 * 1024 * 1024,
+    })
+        .toString()
+        .trim();
+    const changedFiles = execFileSync("git", ["diff", "--name-only", baseBranch, ...pathspec], { stdio: "pipe", maxBuffer: 10 * 1024 * 1024 })
+        .toString()
+        .trim()
+        .split("\n")
+        .filter(Boolean);
+    const lineCount = content ? content.split("\n").length : 0;
+    return { content, changedFiles, lineCount };
+}
+export function categorizeFiles(files) {
+    const testPattern = /\.(test|spec)\.(ts|tsx|js|jsx)$/;
+    const mockPattern = /__mocks__|fixtures/;
+    const migrationPattern = /migrations\/\d+_.*\.(ts|js)$/;
+    const httpPattern = /\/http\//;
+    const test = files.filter((f) => testPattern.test(f));
+    const migration = files.filter((f) => migrationPattern.test(f));
+    const http = files.filter((f) => httpPattern.test(f));
+    const source = files.filter((f) => !testPattern.test(f) && !mockPattern.test(f));
+    return { source, test, migration, http };
+}
+export function categorizeFilesWithConfig(files, categories) {
+    const result = {};
+    for (const [name, def] of Object.entries(categories)) {
+        const includeRe = def.pattern ? new RegExp(def.pattern) : null;
+        const excludeRe = def.exclude ? new RegExp(def.exclude) : null;
+        result[name] = files.filter((f) => {
+            const included = includeRe ? includeRe.test(f) : true;
+            const excluded = excludeRe ? excludeRe.test(f) : false;
+            return included && !excluded;
+        });
+    }
+    return result;
+}
+export function getPrMetadata() {
+    try {
+        execSync("which gh", { stdio: "pipe" });
+    }
+    catch {
+        return null;
+    }
+    try {
+        const title = execSync("gh pr view --json title -q .title", {
+            stdio: "pipe",
+        })
+            .toString()
+            .trim();
+        const body = execSync("gh pr view --json body -q .body", {
+            stdio: "pipe",
+        })
+            .toString()
+            .trim();
+        if (!title)
+            return null;
+        return { title, body };
+    }
+    catch {
+        return null;
+    }
+}
+//# sourceMappingURL=git.js.map

package/dist/lib/git.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"git.js","sourceRoot":"","sources":["../../src/lib/git.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AAE5D,MAAM,UAAU,SAAS;IACvB,IAAI,CAAC;QACH,QAAQ,CAAC,qCAAqC,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAC;QACnE,OAAO,IAAI,CAAC;IACd,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,KAAK,CAAC;IACf,CAAC;AACH,CAAC;AAED,MAAM,UAAU,WAAW;IACzB,OAAO,QAAQ,CAAC,+BAA+B,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC;SAChE,QAAQ,EAAE;SACV,IAAI,EAAE,CAAC;AACZ,CAAC;AAED,SAAS,YAAY,CAAC,MAAc;IAClC,IAAI,CAAC;QACH,QAAQ,CAAC,0BAA0B,MAAM,EAAE,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAC;QAChE,OAAO,IAAI,CAAC;IACd,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,KAAK,CAAC;IACf,CAAC;AACH,CAAC;AAED,MAAM,UAAU,gBAAgB,CAAC,QAAiB;IAChD,IAAI,QAAQ,EAAE,CAAC;QACb,IAAI,CAAC,YAAY,CAAC,QAAQ,CAAC,EAAE,CAAC;YAC5B,MAAM,IAAI,KAAK,CAAC,WAAW,QAAQ,kBAAkB,CAAC,CAAC;QACzD,CAAC;QACD,OAAO,QAAQ,CAAC;IAClB,CAAC;IAED,MAAM,UAAU,GAAG,CAAC,aAAa,EAAE,eAAe,EAAE,MAAM,EAAE,QAAQ,CAAC,CAAC;IAEtE,KAAK,MAAM,MAAM,IAAI,UAAU,EAAE,CAAC;QAChC,IAAI,YAAY,CAAC,MAAM,CAAC,EAAE,CAAC;YACzB,OAAO,MAAM,CAAC;QAChB,CAAC;IACH,CAAC;IAED,MAAM,IAAI,KAAK,CACb,gEAAgE;QAC9D,gDAAgD,CACnD,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,oBAAoB,CAAC,UAAkB;IACrD,MAAM,WAAW,GAAG,YAAY,CAC9B,KAAK,EACL,CAAC,UAAU,EAAE,SAAS,EAAE,SAAS,UAAU,EAAE,CAAC,EAC9C,EAAE,KAAK,EAAE,MAAM,EAAE,CAClB;SACE,QAAQ,EAAE;SACV,IAAI,EAAE,CAAC;IAEV,IAAI,QAAQ,CAAC,WAAW,EAAE,EAAE,CAAC,GAAG,CAAC,EAAE,CAAC;QAClC,MAAM,IAAI,KAAK,CACb,yBAAyB,UAAU,OAAO,WAAW,cAAc;YACjE,0CAA0C,CAC7C,CAAC;IACJ,CAAC;AACH,CAAC;AAQD,MAAM,kBAAkB,GAAG;IACzB,gCAAgC;IAChC,wBAAwB;IACxB,6BAA6B;IAC7B,8BAA8B;IAC9B,mCAAmC;CACpC,CAAC;AAEF,MAAM,UAAU,OAAO,CACrB,UAAkB,EAClB,kBAA4B,EAAE;IAE9B,MAAM,UAAU,GAAG;QACjB,GAAG,kBAAkB;QACrB,GAAG,eAAe,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,aAAa,CAAC,EAAE,CAAC;KAChD,CAAC;IAEF,MAAM,QAAQ,GAAG,CAAC,IAAI,EAAE,GAAG,EAAE,GAAG,UAAU,CAAC,CAAC;IAE5C,MAAM,OAAO,GAAG,YAAY,CAAC,KAAK,EAAE,CAAC,MAAM,EAAE,UAAU,EAAE,GAAG,QAAQ,CAAC,EAAE;QACrE,KAAK,EAAE,MAAM;QACb,SAAS,EAAE,EAAE,GAAG,IAAI,GAAG,IAAI;KAC5B,CAAC;SACC,QAAQ,EAAE;SACV,IAAI,EAAE,CAAC;IAEV,MAAM,YAAY,GAAG,YAAY,CAC/B,KAAK,EACL,CAAC,MAAM,EAAE,aAAa,EAAE,UAAU,EAAE,GAAG,QAAQ,CAAC,EAChD,EAAE,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,EAAE,GAAG,IAAI,GAAG,IAAI,EAAE,CAC/C;SACE,QAAQ,EAAE;SACV,IAAI,EAAE;SACN,KAAK,CAAC,IAAI,CAAC;SACX,MAAM,CAAC,OAAO,CAAC,CAAC;IAEnB,MAAM,SAAS,GAAG,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC;IAE3D,OAAO,EAAE,OAAO,EAAE,YAAY,EAAE,SAAS,EAAE,CAAC;AAC9C,CAAC;AASD,MAAM,UAAU,eAAe,CAAC,KAAe;IAC7C,MAAM,WAAW,GAAG,iCAAiC,CAAC;IACtD,MAAM,WAAW,GAAG,oBAAoB,CAAC;IACzC,MAAM,gBAAgB,GAAG,8BAA8B,CAAC;IACxD,MAAM,WAAW,GAAG,UAAU,CAAC;IAE/B,MAAM,IAAI,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC;IACtD,MAAM,SAAS,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC;IAChE,MAAM,IAAI,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC;IACtD,MAAM,MAAM,GAAG,KAAK,CAAC,MAAM,CACzB,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC,CACpD,CAAC;IAEF,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC;AAC3C,CAAC;AAOD,MAAM,UAAU,yBAAyB,CACvC,KAAe,EACf,UAA8C;IAE9C,MAAM,MAAM,GAA6B,EAAE,CAAC;IAE5C,KAAK,MAAM,CAAC,IAAI,EAAE,GAAG,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,EAAE,CAAC;QACrD,MAAM,SAAS,GAAG,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;QAC/D,MAAM,SAAS,GAAG,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;QAE/D,MAAM,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE;YAChC,MAAM,QAAQ,GAAG,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;YACtD,MAAM,QAAQ,GAAG,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC;YACvD,OAAO,QAAQ,IAAI,CAAC,QAAQ,CAAC;QAC/B,CAAC,CAAC,CAAC;IACL,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC;AAOD,MAAM,UAAU,aAAa;IAC3B,IAAI,CAAC;QACH,QAAQ,CAAC,UAAU,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAC;IAC1C,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,CAAC;QACH,MAAM,KAAK,GAAG,QAAQ,CAAC,mCAAmC,EAAE;YAC1D,KAAK,EAAE,MAAM;SACd,CAAC;aACC,QAAQ,EAAE;aACV,IAAI,EAAE,CAAC;QAEV,MAAM,IAAI,GAAG,QAAQ,CAAC,iCAAiC,EAAE;YACvD,KAAK,EAAE,MAAM;SACd,CAAC;aACC,QAAQ,EAAE;aACV,IAAI,EAAE,CAAC;QAEV,IAAI,CAAC,KAAK;YAAE,OAAO,IAAI,CAAC;QACxB,OAAO,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC;IACzB,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC"}

package/dist/lib/prompt.d.ts

@@ -0,0 +1,16 @@
+import type { CategorizedFiles, CategoryDefinition, PrMetadata } from "./git.js";
+export interface ReviewConfig {
+    categories?: Record<string, CategoryDefinition>;
+}
+export interface PromptContext {
+    baseBranch: string;
+    diffContent: string;
+    changedFiles: string[];
+    categorizedFiles: CategorizedFiles;
+    customCategories?: Record<string, string[]>;
+    prMetadata: PrMetadata | null;
+}
+export declare function hasCustomPrompt(repoRoot: string): boolean;
+export declare function loadTemplate(repoRoot: string): string;
+export declare function loadConfig(repoRoot: string): ReviewConfig;
+export declare function buildPrompt(template: string, context: PromptContext): string;

package/dist/lib/prompt.js

@@ -0,0 +1,164 @@
+import { readFileSync, existsSync } from "node:fs";
+import { join } from "node:path";
+const CONFIG_DIR = ".review-my-pr";
+const PROMPT_FILE = "prompt.md";
+const CONFIG_FILE = "config.json";
+const DEFAULT_PROMPT = `You are an expert code reviewer. Below is a git diff comparing a feature branch to {{BASE_BRANCH}}.
+
+{{PR_METADATA}}
+## Your Task
+
+### 0. High-Level Summary
+In 2-3 sentences, describe:
+- **Product impact**: What does this change deliver for users or customers?
+- **Engineering approach**: Key patterns or architectural decisions.
+
+### 1. Critical Review Focus
+Prioritize your review on these areas (in order of importance):
+
+**🐛 Bugs & Logic Errors** (HIGHEST PRIORITY)
+- Off-by-one errors, null/undefined handling, race conditions
+- Incorrect conditional logic, missing edge cases
+- Type mismatches, incorrect API contract usage
+- Transaction handling issues (missing rollback, deadlocks)
+
+**💥 Breaking Changes** (CRITICAL)
+- API contract changes (request/response shape changes)
+- Database schema changes without proper migrations
+- Backward compatibility issues
+
+**🔒 Security Issues**
+- Authentication/authorization bypasses
+- Injection vulnerabilities (SQL, command, etc.)
+- Sensitive data exposure in logs or responses
+- Missing access control checks
+
+**⚡ Performance Regressions**
+- N+1 query patterns
+- Missing database indexes on new columns
+- Unbounded queries without pagination
+- Unnecessary computation in hot paths
+
+**🧪 Test Coverage Gaps**
+- New code paths without tests
+- Modified logic without updated tests
+- Missing error case coverage
+
+### 2. Review Guidelines
+- **Only report issues you're confident about (>80% certainty)**
+- **Don't flag stylistic preferences** if the code follows existing patterns in the codebase
+- **If you see an unfamiliar pattern**, assume it may be a project convention before flagging
+- **Focus on functional correctness** over cosmetic improvements
+- **Skip categories with no relevant changes** - don't force issues where none exist
+
+### 3. Issue Format
+For each real issue found:
+
+📍 \`path/to/file.ts:line-range\`
+**[🔴 Critical | 🟠 Major | 🟡 Minor | 🔵 Enhancement]** Brief title
+
+→ **Problem**: What's wrong and why it matters
+→ **Fix**: Specific suggestion or code snippet
+
+**Confidence**: 🔴 Certain / 🟡 Likely / 🟢 Suggestion
+
+### 4. Output Structure
+
+\`\`\`
+## Summary
+[2-3 sentence overview]
+
+## Prioritized Issues
+
+### 🔴 Critical
+[Issues that must be fixed before merge - bugs, security, breaking changes]
+
+### 🟠 Major
+[Issues that should be fixed - logic problems, missing error handling]
+
+### 🟡 Minor
+[Issues worth addressing - code clarity, minor optimizations]
+
+### 🔵 Enhancements
+[Nice-to-haves - suggestions for improvement]
+
+## ✅ Highlights
+[Positive findings, well-implemented patterns, good practices observed]
+\`\`\`
+
+## Changed Files
+{{CHANGED_FILES}}
+
+## Diff
+
+\`\`\`diff
+{{DIFF}}
+\`\`\`
+
+---
+
+**CRITICAL INSTRUCTION**: Your primary job is catching bugs and breaking changes. Be thorough but avoid false positives. If something looks unusual but might be intentional, note it as a question rather than an issue.`;
+export function hasCustomPrompt(repoRoot) {
+    return existsSync(join(repoRoot, CONFIG_DIR, PROMPT_FILE));
+}
+export function loadTemplate(repoRoot) {
+    const promptPath = join(repoRoot, CONFIG_DIR, PROMPT_FILE);
+    if (existsSync(promptPath)) {
+        return readFileSync(promptPath, "utf-8");
+    }
+    return DEFAULT_PROMPT;
+}
+export function loadConfig(repoRoot) {
+    const configPath = join(repoRoot, CONFIG_DIR, CONFIG_FILE);
+    if (!existsSync(configPath)) {
+        return {};
+    }
+    try {
+        return JSON.parse(readFileSync(configPath, "utf-8"));
+    }
+    catch {
+        return {};
+    }
+}
+function formatFileList(files) {
+    if (files.length === 0)
+        return "_None_";
+    return files.map((f) => `- ${f}`).join("\n");
+}
+function buildPrMetadataBlock(meta) {
+    if (!meta)
+        return "";
+    return `## PR Metadata
+**Title**: ${meta.title}
+**Description**: ${meta.body}
+
+`;
+}
+export function buildPrompt(template, context) {
+    const { baseBranch, diffContent, changedFiles, categorizedFiles, customCategories, prMetadata, } = context;
+    const diffBlock = diffContent;
+    const fencedDiff = "```diff\n" + diffContent + "\n```";
+    let prompt = template;
+    prompt = prompt.replaceAll("{{BASE_BRANCH}}", baseBranch);
+    prompt = prompt.replaceAll("{{CHANGED_FILES}}", formatFileList(changedFiles));
+    prompt = prompt.replaceAll("{{SRC_FILES}}", formatFileList(categorizedFiles.source));
+    prompt = prompt.replaceAll("{{TEST_FILES}}", formatFileList(categorizedFiles.test));
+    prompt = prompt.replaceAll("{{MIGRATION_FILES}}", formatFileList(categorizedFiles.migration));
+    prompt = prompt.replaceAll("{{HTTP_FILES}}", formatFileList(categorizedFiles.http));
+    prompt = prompt.replaceAll("{{PR_TITLE}}", prMetadata?.title ?? "");
+    prompt = prompt.replaceAll("{{PR_BODY}}", prMetadata?.body ?? "");
+    prompt = prompt.replaceAll("{{PR_METADATA}}", buildPrMetadataBlock(prMetadata));
+    if (customCategories) {
+        for (const [name, files] of Object.entries(customCategories)) {
+            prompt = prompt.replaceAll(`{{CATEGORY.${name}}}`, formatFileList(files));
+        }
+    }
+    if (prompt.includes("{{DIFF}}")) {
+        prompt = prompt.replaceAll("{{DIFF}}", diffBlock);
+    }
+    else {
+        prompt += "\n\n## Diff\n\n" + fencedDiff;
+    }
+    return prompt;
+}
+//# sourceMappingURL=prompt.js.map

package/dist/lib/prompt.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"prompt.js","sourceRoot":"","sources":["../../src/lib/prompt.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACnD,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AAOjC,MAAM,UAAU,GAAG,eAAe,CAAC;AACnC,MAAM,WAAW,GAAG,WAAW,CAAC;AAChC,MAAM,WAAW,GAAG,aAAa,CAAC;AAElC,MAAM,cAAc,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;0NA8FmM,CAAC;AAe3N,MAAM,UAAU,eAAe,CAAC,QAAgB;IAC9C,OAAO,UAAU,CAAC,IAAI,CAAC,QAAQ,EAAE,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC;AAC7D,CAAC;AAED,MAAM,UAAU,YAAY,CAAC,QAAgB;IAC3C,MAAM,UAAU,GAAG,IAAI,CAAC,QAAQ,EAAE,UAAU,EAAE,WAAW,CAAC,CAAC;IAE3D,IAAI,UAAU,CAAC,UAAU,CAAC,EAAE,CAAC;QAC3B,OAAO,YAAY,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;IAC3C,CAAC;IAED,OAAO,cAAc,CAAC;AACxB,CAAC;AAED,MAAM,UAAU,UAAU,CAAC,QAAgB;IACzC,MAAM,UAAU,GAAG,IAAI,CAAC,QAAQ,EAAE,UAAU,EAAE,WAAW,CAAC,CAAC;IAE3D,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC,EAAE,CAAC;QAC5B,OAAO,EAAE,CAAC;IACZ,CAAC;IAED,IAAI,CAAC;QACH,OAAO,IAAI,CAAC,KAAK,CAAC,YAAY,CAAC,UAAU,EAAE,OAAO,CAAC,CAAiB,CAAC;IACvE,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,CAAC;IACZ,CAAC;AACH,CAAC;AAED,SAAS,cAAc,CAAC,KAAe;IACrC,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,QAAQ,CAAC;IACxC,OAAO,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC/C,CAAC;AAED,SAAS,oBAAoB,CAAC,IAAuB;IACnD,IAAI,CAAC,IAAI;QAAE,OAAO,EAAE,CAAC;IAErB,OAAO;aACI,IAAI,CAAC,KAAK;mBACJ,IAAI,CAAC,IAAI;;CAE3B,CAAC;AACF,CAAC;AAED,MAAM,UAAU,WAAW,CAAC,QAAgB,EAAE,OAAsB;IAClE,MAAM,EACJ,UAAU,EACV,WAAW,EACX,YAAY,EACZ,gBAAgB,EAChB,gBAAgB,EAChB,UAAU,GACX,GAAG,OAAO,CAAC;IAEZ,MAAM,SAAS,GAAG,WAAW,CAAC;IAC9B,MAAM,UAAU,GAAG,WAAW,GAAG,WAAW,GAAG,OAAO,CAAC;IAEvD,IAAI,MAAM,GAAG,QAAQ,CAAC;IAEtB,MAAM,GAAG,MAAM,CAAC,UAAU,CAAC,iBAAiB,EAAE,UAAU,CAAC,CAAC;IAC1D,MAAM,GAAG,MAAM,CAAC,UAAU,CAAC,mBAAmB,EAAE,cAAc,CAAC,YAAY,CAAC,CAAC,CAAC;IAC9E,MAAM,GAAG,MAAM,CAAC,UAAU,CACxB,eAAe,EACf,cAAc,CAAC,gBAAgB,CAAC,MAAM,CAAC,CACxC,CAAC;IACF,MAAM,GAAG,MAAM,CAAC,UAAU,CACxB,gBAAgB,EAChB,cAAc,CAAC,gBAAgB,CAAC,IAAI,CAAC,CACtC,CAAC;IACF,MAAM,GAAG,MAAM,CAAC,UAAU,CACxB,qBAAqB,EACrB,cAAc,CAAC,gBAAgB,CAAC,SAAS,CAAC,CAC3C,CAAC;IACF,MAAM,GAAG,MAAM,CAAC,UAAU,CACxB,gBAAgB,EAChB,cAAc,CAAC,gBAAgB,CAAC,IAAI,CAAC,CACtC,CAAC;IACF,MAAM,GAAG,MAAM,CAAC,UAAU,CAAC,cAAc,EAAE,UAAU,EAAE,KAAK,IAAI,EAAE,CAAC,CAAC;IACpE,MAAM,GAAG,MAAM,CAAC,UAAU,CAAC,aAAa,EAAE,UAAU,EAAE,IAAI,IAAI,EAAE,CAAC,CAAC;IAClE,MAAM,GAAG,MAAM,CAAC,UAAU,CACxB,iBAAiB,EACjB,oBAAoB,CAAC,UAAU,CAAC,CACjC,CAAC;IAEF,IAAI,gBAAgB,EAAE,CAAC;QACrB,KAAK,MAAM,CAAC,IAAI,EAAE,KAAK,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,gBAAgB,CAAC,EAAE,CAAC;YAC7D,MAAM,GAAG,MAAM,CAAC,UAAU,CAAC,cAAc,IAAI,IAAI,EAAE,cAAc,CAAC,KAAK,CAAC,CAAC,CAAC;QAC5E,CAAC;IACH,CAAC;IAED,IAAI,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,EAAE,CAAC;QAChC,MAAM,GAAG,MAAM,CAAC,UAAU,CAAC,UAAU,EAAE,SAAS,CAAC,CAAC;IACpD,CAAC;SAAM,CAAC;QACN,MAAM,IAAI,iBAAiB,GAAG,UAAU,CAAC;IAC3C,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC"}

package/dist/review.d.ts

@@ -0,0 +1,5 @@
+export interface ReviewOptions {
+    base?: string;
+    exclude?: string[];
+}
+export declare function runReview(options: ReviewOptions): Promise<void>;

package/dist/review.js

@@ -0,0 +1,67 @@
+import { isGitRepo, getRepoRoot, detectBaseBranch, ensureBranchUpToDate, getDiff, categorizeFiles, categorizeFilesWithConfig, getPrMetadata, } from "./lib/git.js";
+import { loadTemplate, loadConfig, buildPrompt, hasCustomPrompt } from "./lib/prompt.js";
+import { copyToClipboard, getPasteHint } from "./lib/clipboard.js";
+export async function runReview(options) {
+    if (!isGitRepo()) {
+        console.error("❌ Not a git repository. Run this command from inside a git repo.");
+        process.exit(1);
+    }
+    const repoRoot = getRepoRoot();
+    let baseBranch;
+    try {
+        baseBranch = detectBaseBranch(options.base);
+    }
+    catch (err) {
+        console.error(`❌ ${err.message}`);
+        process.exit(1);
+    }
+    try {
+        ensureBranchUpToDate(baseBranch);
+    }
+    catch (err) {
+        console.error(`❌ ${err.message}`);
+        process.exit(1);
+    }
+    console.log(`Generating diff from ${baseBranch}...`);
+    const diff = getDiff(baseBranch, options.exclude);
+    if (!diff.content) {
+        console.error(`❌ No changes found compared to ${baseBranch}`);
+        process.exit(1);
+    }
+    console.log("✅ Diff generated successfully");
+    console.log(`✅ Found ${diff.changedFiles.length} changed file(s) (${diff.lineCount} lines)`);
+    console.log("");
+    if (diff.lineCount > 2000) {
+        console.log(`⚠️  Large diff (${diff.lineCount} lines). Consider reviewing in chunks for better results.`);
+        console.log("");
+    }
+    const categorized = categorizeFiles(diff.changedFiles);
+    const prMetadata = getPrMetadata();
+    const template = loadTemplate(repoRoot);
+    const config = loadConfig(repoRoot);
+    const isCustom = hasCustomPrompt(repoRoot);
+    const customCategories = config.categories
+        ? categorizeFilesWithConfig(diff.changedFiles, config.categories)
+        : undefined;
+    const prompt = buildPrompt(template, {
+        baseBranch,
+        diffContent: diff.content,
+        changedFiles: diff.changedFiles,
+        categorizedFiles: categorized,
+        customCategories,
+        prMetadata,
+    });
+    try {
+        await copyToClipboard(prompt);
+        if (isCustom) {
+            console.log("📋 Using custom prompt from .review-my-pr/prompt.md");
+        }
+        console.log("✅ Review prompt copied to clipboard!");
+        console.log(getPasteHint());
+    }
+    catch {
+        console.error("❌ Failed to copy to clipboard. Outputting prompt to stdout:\n");
+        console.log(prompt);
+    }
+}
+//# sourceMappingURL=review.js.map

package/dist/review.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review.js","sourceRoot":"","sources":["../src/review.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,SAAS,EACT,WAAW,EACX,gBAAgB,EAChB,oBAAoB,EACpB,OAAO,EACP,eAAe,EACf,yBAAyB,EACzB,aAAa,GACd,MAAM,cAAc,CAAC;AACtB,OAAO,EAAE,YAAY,EAAE,UAAU,EAAE,WAAW,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAC;AACzF,OAAO,EAAE,eAAe,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAOnE,MAAM,CAAC,KAAK,UAAU,SAAS,CAAC,OAAsB;IACpD,IAAI,CAAC,SAAS,EAAE,EAAE,CAAC;QACjB,OAAO,CAAC,KAAK,CAAC,kEAAkE,CAAC,CAAC;QAClF,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;IAED,MAAM,QAAQ,GAAG,WAAW,EAAE,CAAC;IAE/B,IAAI,UAAkB,CAAC;IACvB,IAAI,CAAC;QACH,UAAU,GAAG,gBAAgB,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;IAC9C,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,OAAO,CAAC,KAAK,CAAC,KAAM,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;QAC7C,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;IAED,IAAI,CAAC;QACH,oBAAoB,CAAC,UAAU,CAAC,CAAC;IACnC,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,OAAO,CAAC,KAAK,CAAC,KAAM,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;QAC7C,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;IAED,OAAO,CAAC,GAAG,CAAC,wBAAwB,UAAU,KAAK,CAAC,CAAC;IAErD,MAAM,IAAI,GAAG,OAAO,CAAC,UAAU,EAAE,OAAO,CAAC,OAAO,CAAC,CAAC;IAElD,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,CAAC;QAClB,OAAO,CAAC,KAAK,CAAC,kCAAkC,UAAU,EAAE,CAAC,CAAC;QAC9D,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;IAED,OAAO,CAAC,GAAG,CAAC,+BAA+B,CAAC,CAAC;IAC7C,OAAO,CAAC,GAAG,CACT,WAAW,IAAI,CAAC,YAAY,CAAC,MAAM,qBAAqB,IAAI,CAAC,SAAS,SAAS,CAChF,CAAC;IACF,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;IAEhB,IAAI,IAAI,CAAC,SAAS,GAAG,IAAI,EAAE,CAAC;QAC1B,OAAO,CAAC,GAAG,CACT,mBAAmB,IAAI,CAAC,SAAS,2DAA2D,CAC7F,CAAC;QACF,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;IAClB,CAAC;IAED,MAAM,WAAW,GAAG,eAAe,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IACvD,MAAM,UAAU,GAAG,aAAa,EAAE,CAAC;IAEnC,MAAM,QAAQ,GAAG,YAAY,CAAC,QAAQ,CAAC,CAAC;IACxC,MAAM,MAAM,GAAG,UAAU,CAAC,QAAQ,CAAC,CAAC;IACpC,MAAM,QAAQ,GAAG,eAAe,CAAC,QAAQ,CAAC,CAAC;IAE3C,MAAM,gBAAgB,GAAG,MAAM,CAAC,UAAU;QACxC,CAAC,CAAC,yBAAyB,CAAC,IAAI,CAAC,YAAY,EAAE,MAAM,CAAC,UAAU,CAAC;QACjE,CAAC,CAAC,SAAS,CAAC;IAEd,MAAM,MAAM,GAAG,WAAW,CAAC,QAAQ,EAAE;QACnC,UAAU;QACV,WAAW,EAAE,IAAI,CAAC,OAAO;QACzB,YAAY,EAAE,IAAI,CAAC,YAAY;QAC/B,gBAAgB,EAAE,WAAW;QAC7B,gBAAgB;QAChB,UAAU;KACX,CAAC,CAAC;IAEH,IAAI,CAAC;QACH,MAAM,eAAe,CAAC,MAAM,CAAC,CAAC;QAC9B,IAAI,QAAQ,EAAE,CAAC;YACb,OAAO,CAAC,GAAG,CAAC,qDAAqD,CAAC,CAAC;QACrE,CAAC;QACD,OAAO,CAAC,GAAG,CAAC,sCAAsC,CAAC,CAAC;QACpD,OAAO,CAAC,GAAG,CAAC,YAAY,EAAE,CAAC,CAAC;IAC9B,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,CAAC,KAAK,CAAC,+DAA+D,CAAC,CAAC;QAC/E,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;IACtB,CAAC;AACH,CAAC"}