pulls.review 0.1.0 → 0.2.1

package/dist/index.mjs

@@ -1,126 +1,67 @@
-#!/usr/bin/env node
-import { readFileSync } from "node:fs";
+import { t as devframe_default } from "./devframe-CX5butdC.mjs";
+import { readFile } from "node:fs/promises";
 import process from "node:process";
+import { fileURLToPath } from "node:url";
 import { parseArgs } from "node:util";
-import { githubTokenFromEnv, llmSettingsFromEnv } from "@pulls.review/core/env";
-import { runLlmAnalysis } from "@pulls.review/core/llm";
-import { Diagnostic, formatDiagnostic } from "nostics";
-import { DEFAULT_LOCALE, isLocale } from "@pulls.review/core/locales";
-import { resolveModel } from "@pulls.review/core/analyze";
-import { diagnostics } from "@pulls.review/core/diagnostics";
-import { createGithubPullRequestSource } from "@pulls.review/core/github";
-import { staticCredentials } from "@pulls.review/core/types";
-//#region src/config.ts
-/** The first value that is set and not blank: an Action passes unset inputs as `''`. */
-function first(...values) {
-	return values.find((value) => value);
-}
-function resolveGithubToken(env) {
-	const token = githubTokenFromEnv(env);
-	if (!token) throw new Error("A GitHub token is required: set GITHUB_TOKEN (or PULLS_REVIEW_GITHUB_TOKEN).");
-	return token;
-}
-function resolveLocale(env, flags = {}) {
-	const requested = first(flags.locale, env.PULLS_REVIEW_LOCALE);
-	if (requested === void 0) return DEFAULT_LOCALE;
-	if (!isLocale(requested)) throw new Error(`Unsupported locale "${requested}".`);
-	return requested;
-}
-/**
-* `owner/repo#123` or a github.com PR URL; in a workflow run, the PR the event is about.
-*/
-function resolveTarget(arg, env, event) {
-	if (arg) {
-		const match = arg.match(/^(?:https:\/\/github\.com\/)?([^/\s]+)\/([^/#\s]+)(?:#|\/pull\/)(\d+)\/?$/);
-		if (!match) throw new Error(`Cannot read a pull request from "${arg}": expected owner/repo#123 or a github.com pull request URL.`);
-		return {
-			owner: match[1],
-			repo: match[2],
-			number: match[3]
-		};
-	}
-	const [owner, repo] = env.GITHUB_REPOSITORY?.split("/") ?? [];
-	const number = event?.pull_request?.number;
-	if (!owner || !repo || number === void 0) throw new Error("No pull request given. Pass owner/repo#123, or run on a pull_request event in GitHub Actions.");
-	return {
-		owner,
-		repo,
-		number: String(number)
-	};
-}
-//#endregion
-//#region src/run.ts
-/** The author GitHub shows for a workflow's `GITHUB_TOKEN`, which cannot look itself up at `/user`. */
-const ACTIONS_BOT_LOGIN = "github-actions[bot]";
-function describeProgress(progress) {
-	switch (progress.kind) {
-		case "thinking": return `[${progress.step}] thinking`;
-		case "reading": return `[${progress.step}] reading ${progress.paths.length} file${progress.paths.length === 1 ? "" : "s"}`;
-		case "organizing": return `[${progress.step}] organizing groups`;
+import { createDevServer } from "devframe/adapters/dev";
+import { H3 } from "h3";
+import { isLocalBranch, parseTarget, readCurrentBranch, resolveDefaultBranch } from "@pulls.review/core/local";
+//#region src/page.ts
+/** The page an argument opens (see the app's local routes). */
+async function pageFor(cwd, arg, worktree) {
+	if (worktree) return "/worktree";
+	if (!arg) {
+		const [current, defaultBranch] = await Promise.all([readCurrentBranch(cwd), resolveDefaultBranch(cwd)]);
+		return current && defaultBranch && current !== defaultBranch.name ? `/branch/${current}` : "/";
 	}
-}
-/**
-* Fetches the PR, analyzes it and upserts the shared-analysis comment, exactly like the
-* site's "share as comment" - one comment per author, found by its marker. A comment
-* already carrying the PR's current head sha is left alone, so re-runs on an unchanged
-* PR cost no model call.
-*/
-async function run({ pr, githubToken, llm, locale, log }, analyze) {
-	const resolved = resolveModel(llm);
-	if (!resolved) throw diagnostics.llmNotConfigured();
-	const source = createGithubPullRequestSource(pr, staticCredentials(githubToken));
-	const [diff, login, comments] = await Promise.all([
-		source.fetch(),
-		source.viewer().then((viewer) => viewer?.login ?? ACTIONS_BOT_LOGIN, () => ACTIONS_BOT_LOGIN),
-		source.sharing.list()
-	]);
-	const headSha = diff.head?.sha ?? "";
-	const existing = comments.find((comment) => comment.login === login);
-	if (existing && existing.analysis.headSha === headSha) return {
-		status: "up-to-date",
-		url: existing.url
-	};
-	log(`Analyzing ${pr.owner}/${pr.repo}#${pr.number} (${diff.files.length} files) with ${resolved.model.provider}/${resolved.model.id}`);
-	const { result } = await analyze(diff, resolved, locale, { onProgress: (progress) => log(describeProgress(progress)) });
-	return {
-		status: "posted",
-		url: (await source.sharing.upsert(login, {
-			headSha,
-			result
-		}, existing)).url
-	};
+	if (parseTarget(arg).kind === "range") return `/compare/${arg}`;
+	return await isLocalBranch(cwd, arg) ? `/branch/${arg}` : `/commit/${arg}`;
 }
 //#endregion
 //#region src/index.ts
-const HELP = `Usage: pulls.review [owner/repo#123 | https://github.com/owner/repo/pull/123] [options]
+const HELP = `Usage: pulls.review [target] [options]
+
+Reviews local git changes in the browser, grouped and summarized. Run it inside
+a git repository. The target only picks the page that opens; any other can be
+picked from the browser.
 
-Analyzes the pull request with an LLM and posts (or updates) the grouped summary as
-a PR comment that pulls.review picks up. Without a target, reads the pull_request
-event of the running GitHub Actions job.
+Target (git revision syntax):
+  (none)     the current branch against the default branch, or the ref picker
+             when the default branch itself is checked out
+  <branch>   what a branch adds since it forked from the default branch
+  <rev>      one commit against its parent
+  A...B      what B adds since it forked from A
+  A..B       the tree diff between two revisions
 
 Options:
-  --provider <name>  gateway | anthropic | openai-compatible  (PULLS_REVIEW_PROVIDER)
-  --model <id>       model id for the provider                 (PULLS_REVIEW_MODEL)
-  --locale <tag>     language of the summaries, default en     (PULLS_REVIEW_LOCALE)
+  --worktree     open the uncommitted changes against HEAD
+  --port <port>  port to listen on
+  --no-open      do not open the browser
   -h, --help
 
-Environment:
-  GITHUB_TOKEN           token with pull-requests: write       (PULLS_REVIEW_GITHUB_TOKEN)
-  AI_GATEWAY_API_KEY     Vercel AI Gateway key                  (PULLS_REVIEW_API_KEY)
-  ANTHROPIC_API_KEY      Anthropic key                          (PULLS_REVIEW_API_KEY)
-  OPENAI_API_KEY         OpenAI-compatible key                  (PULLS_REVIEW_API_KEY)
-  OPENAI_BASE_URL        OpenAI-compatible endpoint             (PULLS_REVIEW_BASE_URL)
+To review a GitHub pull request in CI, use @pulls.review/actions.
 `;
-function readEvent(path) {
-	return path ? JSON.parse(readFileSync(path, "utf-8")) : void 0;
+/**
+* devframe's SPA fallback skips any path that looks like a file, and refs do: `main...feat`
+* ends in `.feat`, `v1.2` in `.2`. Pages whose path carries a ref get the SPA's
+* `index.html` here, before devframe's static handler sees them.
+*/
+function createAppWithRefRoutes() {
+	const app = new H3();
+	const index = fileURLToPath(new URL("./client/index.html", import.meta.url));
+	app.use(async (event, next) => {
+		if (!/^\/(?:compare|branch|commit)\//.test(event.url.pathname) || !event.req.headers.get("accept")?.includes("text/html")) return next();
+		return new Response(await readFile(index, "utf8"), { headers: { "Content-Type": "text/html; charset=utf-8" } });
+	});
+	return app;
 }
 async function main() {
 	const { values: flags, positionals } = parseArgs({
 		options: {
-			provider: { type: "string" },
-			model: { type: "string" },
-			locale: { type: "string" },
-			help: {
+			"worktree": { type: "boolean" },
+			"port": { type: "string" },
+			"no-open": { type: "boolean" },
+			"help": {
 				type: "boolean",
 				short: "h"
 			}
@@ -128,21 +69,21 @@ async function main() {
 		allowPositionals: true
 	});
 	if (flags.help) {
-		console.log(HELP);
+		process.stdout.write(HELP);
 		return;
 	}
-	const env = process.env;
-	const outcome = await run({
-		pr: resolveTarget(positionals[0], env, readEvent(env.GITHUB_EVENT_PATH)),
-		githubToken: resolveGithubToken(env),
-		llm: llmSettingsFromEnv(env, flags),
-		locale: resolveLocale(env, flags),
-		log: (line) => console.log(line)
-	}, runLlmAnalysis);
-	console.log(outcome.status === "up-to-date" ? `Comment already covers the current head: ${outcome.url}` : `Posted ${outcome.url}`);
+	const page = encodeURI(await pageFor(process.cwd(), positionals[0], !!flags.worktree));
+	await createDevServer(devframe_default, {
+		app: createAppWithRefRoutes(),
+		port: flags.port ? Number(flags.port) : void 0,
+		openBrowser: flags["no-open"] ? false : page,
+		onReady: ({ origin }) => {
+			process.stdout.write(`pulls.review is serving ${origin}${page}\n`);
+		}
+	});
 }
 main().catch((error) => {
-	console.error(error instanceof Diagnostic ? formatDiagnostic(error) : error instanceof Error ? error.message : String(error));
+	console.error(error instanceof Error ? error.message : error);
 	process.exitCode = 1;
 });
 //#endregion

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "pulls.review",
   "type": "module",
-  "version": "0.1.0",
-  "description": "Analyze a GitHub pull request with pulls.review and post the grouped summary as a PR comment.",
+  "version": "0.2.1",
+  "description": "Review local git changes - the working tree, a commit or any range - grouped and summarized in the browser.",
   "author": "Anthony Fu <hi@antfu.me>",
   "license": "MIT",
   "funding": "https://github.com/sponsors/antfu",
@@ -16,30 +16,38 @@
   "keywords": [
     "pull-request",
     "code-review",
-    "github-actions",
-    "llm"
+    "git",
+    "devframe"
   ],
+  "exports": {
+    "./devframe": "./dist/devframe.mjs"
+  },
   "bin": {
-    "pulls.review": "./dist/index.mjs",
-    "pulls-review": "./dist/index.mjs"
+    "pulls.review": "./bin.mjs",
+    "pulls-review": "./bin.mjs"
   },
   "files": [
+    "bin.mjs",
     "dist"
   ],
   "engines": {
     "node": ">=20"
   },
   "dependencies": {
-    "@pulls.review/core": "^0.1.0",
-    "nostics": "^1.3.0"
+    "@pulls.review/core": "^0.2.1",
+    "devframe": "^1.2.0",
+    "h3": "^2.0.1-rc.32",
+    "unstorage": "^1.17.5"
   },
   "devDependencies": {
     "@types/node": "^26.6.4",
+    "tsdown": "^0.23.0",
     "typescript": "^6.0.3",
     "vitest": "^5.0.3"
   },
   "scripts": {
-    "build": "tsdown",
+    "build": "tsdown && pnpm -F @pulls.review/app run build:local",
+    "dev": "node bin.mjs",
     "typecheck": "tsc --noEmit",
     "test": "vitest run"
   }