@jpkme/diff-review 0.1.10 → 0.1.12

package/web/build/server/chunks/{remote-1g4ayag-B67QMR_2.js → git-review-CpO_G8z8.js}

@@ -1,9 +1,10 @@
-import { e as error, g as get_request_store, b as stringify_remote_arg, M as MUTATIVE_METHODS, w as with_request_store, c as create_remote_key, n as noop, H as HttpError, S as SvelteKitError, h as handle_error_and_jsonify, _ as __exportAll, i as init_remote_functions } from './index.js-CL95e0Gl.js';
-import { join, basename, resolve } from 'node:path';
-import { existsSync, readFileSync, statSync, readdirSync, mkdtempSync, rmSync, writeFileSync, mkdirSync } from 'node:fs';
+import { e as error, g as get_request_store, b as stringify_remote_arg, M as MUTATIVE_METHODS, w as with_request_store, c as create_remote_key, n as noop, H as HttpError, S as SvelteKitError, h as handle_error_and_jsonify } from './index.js-yLYmrAWy.js';
+import { m as makeOutputFilename, a as assertGitRepo, b as branchTips, f as fingerprint, e as ensureReviewDir, l as listBranches, g as getRemoteUrl, t as tryResolveGitRoot, i as isOutputFilename, d as decodePayload, c as locateReviewFile, r as reviewDir, h as companionPaths, s as showFileLines, j as loadBranchDiff, k as git, F as FINDING_CLASSES, n as FINDING_SEVERITIES, o as FINDING_KINDS, G as GROUP_KINDS, W as WHY_SOURCES, B as BLOCK_OPS, S as SKIP_REASONS } from './validate-BsTKyhsq.js';
+import { existsSync, readFileSync, statSync, readdirSync, writeFileSync, mkdtempSync, rmSync, mkdirSync } from 'node:fs';
 import { execFileSync, spawnSync } from 'node:child_process';
-import { createHash } from 'node:crypto';
 import { tmpdir, platform } from 'node:os';
+import { join, resolve, basename } from 'node:path';
+import { createHash } from 'node:crypto';
 import process$1 from 'node:process';
 
 //#region node_modules/@sveltejs/kit/src/runtime/app/server/remote/shared.js
@@ -957,502 +958,6 @@ Object.defineProperty(query, "live", {
 	enumerable: true
 });
 
-//#region ../lib/fingerprint.mjs
-/**
-* Tips cortos de SHA. Sin APIs de Node: la UI lo importa para armar el nombre del JSON.
-*/
-function shortSha(sha, n = 10) {
-	const hex = String(sha || "").toLowerCase().replace(/[^a-f0-9]/g, "");
-	return hex.length >= 7 ? hex.slice(0, n) : "";
-}
-function fingerprint(branchSha, baseSha) {
-	const a = shortSha(branchSha);
-	const b = shortSha(baseSha);
-	if (!a || !b) return null;
-	return `${a}-${b}`;
-}
-//#endregion
-//#region ../lib/output-name.mjs
-/**
-* Nombre del JSON que escribe el agente.
-* Preferido: tips cortos branch/base. Legacy (con stamp o sin tips) sigue válido al importar.
-*/
-/** Prefijo del formato actual. */
-var OUTPUT_PREFIX = "diff-rev";
-/** Nombre legacy fijo. */
-var OUTPUT_FILENAME_LEGACY = `diff-review-output.json`;
-/**
-* Válidos:
-* - diff-rev_{branchTip}_{baseTip}.json
-* - diff-report_{branchTip}-{baseTip}_YYYYMMDD-HHmmss.json (legacy)
-* - diff-review-output.json / diff-review-output-YYYYMMDD-HHmmss.json (legacy)
-*/
-var OUTPUT_FILENAME_RE = /^(?:diff-rev_[0-9a-f]{7,12}_[0-9a-f]{7,12}|diff-report_[0-9a-f]{7,12}-[0-9a-f]{7,12}_\d{8}-\d{6}|diff-review-output(?:-\d{8}-\d{6})?)\.json$/i;
-/**
-* @param {{ branchSha?: string, baseSha?: string }} [input]
-* @returns {string}
-*   con tips: diff-rev_aaaaaaaaaa_bbbbbbbbbb.json
-*   sin tips: diff-review-output.json
-*/
-function makeOutputFilename(input = {}) {
-	const opts = input instanceof Date ? {} : input || {};
-	const fp = fingerprint(opts.branchSha, opts.baseSha);
-	if (fp) {
-		const [branchTip, baseTip] = fp.split("-");
-		if (branchTip && baseTip) return `${OUTPUT_PREFIX}_${branchTip}_${baseTip}.json`;
-	}
-	return OUTPUT_FILENAME_LEGACY;
-}
-function isOutputFilename(name) {
-	return OUTPUT_FILENAME_RE.test(String(name || "").trim());
-}
-//#endregion
-//#region ../lib/json-payload.mjs
-function tryParseJson(s) {
-	try {
-		return JSON.parse(s);
-	} catch {
-		return null;
-	}
-}
-/** Extrae un objeto JSON del texto del modelo (fences, prosa alrededor, envelope). */
-function decodePayload(text) {
-	const raw = String(text || "").trim();
-	if (!raw) throw new Error("Respuesta vacía.");
-	const fence = raw.match(/```(?:json)?\s*([\s\S]*?)```/i);
-	if (fence) {
-		const parsed = tryParseJson(fence[1].trim());
-		if (parsed && typeof parsed === "object") return parsed;
-	}
-	const direct = tryParseJson(raw);
-	if (direct && typeof direct === "object") return direct;
-	const start = raw.indexOf("{");
-	const end = raw.lastIndexOf("}");
-	if (start >= 0 && end > start) {
-		const parsed = tryParseJson(raw.slice(start, end + 1));
-		if (parsed && typeof parsed === "object") return parsed;
-	}
-	throw new Error("No se pudo interpretar la respuesta como JSON.");
-}
-//#endregion
-//#region ../lib/config.mjs
-var DEFAULT_IGNORE = [
-	"**/package-lock.json",
-	"**/pnpm-lock.yaml",
-	"**/yarn.lock",
-	"**/bun.lock",
-	"**/bun.lockb",
-	"**/*.g.cs",
-	"**/*Snapshot.cs",
-	"**/*.Designer.cs",
-	"**/node_modules/**",
-	"**/.svelte-kit/**",
-	"**/dist/**",
-	"**/build/**",
-	"**/*.min.js",
-	"**/*.min.css",
-	"**/CODE_REVIEW_CORRECTNESS_FINDINGS.md"
-];
-var STORE_DIFF_MAX = 4e5;
-var GUIDELINE_FILES = [
-	"REVIEW.md",
-	"CLAUDE.md",
-	"AGENTS.md"
-];
-function unquote(s) {
-	const t = String(s || "").trim();
-	if (t.startsWith("\"") && t.endsWith("\"") || t.startsWith("'") && t.endsWith("'")) return t.slice(1, -1);
-	return t;
-}
-/** Minimal YAML: `key: value` and `key:\n  - item`. */
-function parseSimpleYaml(text) {
-	const result = {};
-	let key = null;
-	for (const raw of String(text || "").split(/\r?\n/)) {
-		if (!raw.trim() || raw.trim().startsWith("#")) continue;
-		const list = raw.match(/^\s+-\s+(.*)$/);
-		if (list && key) {
-			if (!Array.isArray(result[key])) result[key] = [];
-			result[key].push(unquote(list[1]));
-			continue;
-		}
-		const kv = raw.match(/^([A-Za-z][\w]*)\s*:\s*(.*)$/);
-		if (!kv) continue;
-		key = kv[1];
-		const v = kv[2].trim();
-		result[key] = v === "" ? [] : unquote(v);
-	}
-	return result;
-}
-function globToRegExp(glob) {
-	let g = String(glob || "").replace(/\\/g, "/");
-	if (!g.includes("/")) g = `**/${g}`;
-	let s = "";
-	for (let i = 0; i < g.length; i++) {
-		const c = g[i];
-		if (c === "*" && g[i + 1] === "*") if (g[i + 2] === "/") {
-			s += "(?:.*/)?";
-			i += 2;
-		} else {
-			s += ".*";
-			i += 1;
-		}
-		else if (c === "*") s += "[^/]*";
-		else if (c === "?") s += "[^/]";
-		else if ("+()[]{}|.^$".includes(c)) s += `\\${c}`;
-		else s += c;
-	}
-	return new RegExp(`^${s}$`, "i");
-}
-function globMatch(filePath, pattern) {
-	const norm = String(filePath || "").replace(/\\/g, "/");
-	return globToRegExp(pattern).test(norm);
-}
-function isIgnored(filePath, patterns) {
-	return (patterns || []).some((p) => globMatch(filePath, p));
-}
-function isTestPath(filePath) {
-	const p = String(filePath || "").replace(/\\/g, "/");
-	return /(^|\/)(__tests__|tests?|spec)s?(\/|$)/i.test(p) || /\.(test|spec)\.[^.]+$/i.test(p);
-}
-function readIfExists(path) {
-	try {
-		return existsSync(path) ? readFileSync(path, "utf8") : null;
-	} catch {
-		return null;
-	}
-}
-function loadReviewConfig(repo) {
-	const jsonText = readIfExists(join(repo, ".diff-review.json"));
-	const yamlText = readIfExists(join(repo, ".diff-review.yml")) || readIfExists(join(repo, ".diff-review.yaml"));
-	let extra = {};
-	if (jsonText) try {
-		extra = JSON.parse(jsonText);
-	} catch {
-		extra = {};
-	}
-	else if (yamlText) extra = parseSimpleYaml(yamlText);
-	const extraIgnore = Array.isArray(extra.ignore) ? extra.ignore.map(String) : [];
-	const rulesFile = extra.rulesFile ? String(extra.rulesFile) : "REVIEW.md";
-	const rules = readIfExists(join(repo, rulesFile)) || "";
-	const model = extra.model && extra.model !== "default" ? String(extra.model) : null;
-	return {
-		ignore: [...DEFAULT_IGNORE, ...extraIgnore],
-		model,
-		rulesFile,
-		rules: rules.trim()
-	};
-}
-/** CLAUDE.md / REVIEW.md / AGENTS.md from repo root and immediate parent dirs of touched files. */
-function collectGuidelines(repo, filePaths, { rulesFile = "REVIEW.md", maxChars = 8e3, maxTotalChars = 12e3, maxDepth = 1, skipNames = [] } = {}) {
-	const skip = new Set(skipNames);
-	const names = [.../* @__PURE__ */ new Set([rulesFile, ...GUIDELINE_FILES])].filter((name) => !skip.has(name));
-	const seen = /* @__PURE__ */ new Set();
-	const out = [];
-	const addRel = (rel) => {
-		const norm = String(rel || "").replace(/\\/g, "/").replace(/^\/+/, "");
-		if (!norm || seen.has(norm)) return;
-		const text = readIfExists(join(repo, ...norm.split("/")));
-		if (!text || !text.trim()) return;
-		seen.add(norm);
-		out.push({
-			path: norm,
-			text: text.trim()
-		});
-	};
-	for (const name of names) addRel(name);
-	const depth = Math.max(1, Number(maxDepth) || 1);
-	for (const filePath of filePaths || []) {
-		const parts = String(filePath || "").replace(/\\/g, "/").split("/").filter(Boolean);
-		const minLen = Math.max(1, parts.length - depth);
-		for (let len = parts.length - 1; len >= minLen; len--) {
-			const dir = parts.slice(0, len).join("/");
-			for (const name of names) addRel(`${dir}/${name}`);
-		}
-	}
-	let total = 0;
-	const capped = [];
-	for (const item of out) {
-		if (total >= maxTotalChars) break;
-		const room = maxTotalChars - total;
-		const slice = item.text.slice(0, Math.min(maxChars, room));
-		if (!slice) continue;
-		capped.push({
-			path: item.path,
-			text: slice
-		});
-		total += slice.length;
-	}
-	return capped;
-}
-function capDiff(diff, max = STORE_DIFF_MAX) {
-	const text = String(diff || "");
-	if (text.length <= max) return {
-		diff: text,
-		diffChars: text.length,
-		truncated: false
-	};
-	return {
-		diff: `${text.slice(0, max)}\n\n… [diff recortado: se guardan ${max} de ${text.length} caracteres]`,
-		diffChars: text.length,
-		truncated: true
-	};
-}
-//#endregion
-//#region ../lib/git.mjs
-function assertGitRepo(repo) {
-	const raw = String(repo || "").trim();
-	if (!raw) throw new Error("Indicá la ruta absoluta de un repositorio git.");
-	if (!/[\\/]/.test(raw) && !/^[A-Za-z]:/.test(raw)) throw new Error(`"${raw}" no es una ruta. Pegá la ruta absoluta del repo (ej. C:\\tienda-argenta\\tienda-argenta-2).`);
-	const resolved = resolve(raw);
-	if (!resolved || resolved === ".") throw new Error("Indicá la ruta absoluta de un repositorio git.");
-	if (!existsSync(resolved)) throw new Error(`No existe la carpeta: ${resolved}`);
-	if (!existsSync(join(resolved, ".git"))) throw new Error(`No es un repositorio git (falta .git): ${resolved}`);
-	return resolved;
-}
-/**
-* Si `cwd` está dentro de un repo git, devuelve la raíz absoluta; si no, null.
-* Acepta subcarpetas (usa `git rev-parse --show-toplevel`).
-*/
-function tryResolveGitRoot(cwd) {
-	const raw = String(cwd || "").trim();
-	if (!raw) return null;
-	const dir = resolve(raw);
-	if (!existsSync(dir)) return null;
-	try {
-		const root = execFileSync("git", ["rev-parse", "--show-toplevel"], {
-			cwd: dir,
-			encoding: "utf8",
-			windowsHide: true,
-			stdio: [
-				"ignore",
-				"pipe",
-				"ignore"
-			]
-		}).trim();
-		return root ? resolve(root) : null;
-	} catch {
-		return null;
-	}
-}
-function git(repo, args, { maxBuffer = 64 * 1024 * 1024 } = {}) {
-	return execFileSync("git", args, {
-		cwd: repo,
-		encoding: "utf8",
-		maxBuffer,
-		windowsHide: true
-	});
-}
-function resolveRev(repo, ref) {
-	return git(repo, ["rev-parse", ref]).trim();
-}
-function branchTips(repo, branch, base) {
-	return {
-		branchSha: resolveRev(repo, branch),
-		baseSha: resolveRev(repo, base)
-	};
-}
-function commitLog(repo, mergeBase, branch, limit = 25) {
-	try {
-		return git(repo, [
-			"log",
-			"--oneline",
-			"--no-merges",
-			"-n",
-			String(limit),
-			`${mergeBase}..${branch}`
-		]).trim().split("\n").filter(Boolean);
-	} catch {
-		return [];
-	}
-}
-var FILE_LINE_CACHE = /* @__PURE__ */ new Map();
-var FILE_LINE_CACHE_MAX = 48;
-function rememberFileLines(key, lines) {
-	if (FILE_LINE_CACHE.has(key)) FILE_LINE_CACHE.delete(key);
-	FILE_LINE_CACHE.set(key, lines);
-	while (FILE_LINE_CACHE.size > FILE_LINE_CACHE_MAX) {
-		const oldest = FILE_LINE_CACHE.keys().next().value;
-		FILE_LINE_CACHE.delete(oldest);
-	}
-}
-function showFileLines(repo, rev, path) {
-	const resolved = assertGitRepo(repo);
-	const safeRev = String(rev || "").trim();
-	const safePath = String(path || "").replace(/\\/g, "/").trim();
-	if (!safeRev || !/^[A-Za-z0-9._/\-]+$/.test(safeRev)) throw new Error("Rev inválido.");
-	if (!safePath || safePath.startsWith("/") || safePath.includes("..") || safePath.includes("\0")) throw new Error("Ruta inválida.");
-	const key = `${resolved}\0${safeRev}\0${safePath}`;
-	if (FILE_LINE_CACHE.has(key)) return FILE_LINE_CACHE.get(key);
-	let text;
-	try {
-		text = git(resolved, ["show", `${safeRev}:${safePath}`]);
-	} catch {
-		rememberFileLines(key, null);
-		return null;
-	}
-	if (text.includes("\0")) {
-		rememberFileLines(key, null);
-		return null;
-	}
-	const raw = text.replace(/\r\n/g, "\n");
-	const lines = raw.endsWith("\n") ? raw.slice(0, -1).split("\n") : raw.split("\n");
-	rememberFileLines(key, lines);
-	return lines;
-}
-function listBranches(repo) {
-	const names = git(repo, [
-		"for-each-ref",
-		"--sort=-committerdate",
-		"--format=%(refname:short)",
-		"refs/heads",
-		"refs/remotes"
-	]).split("\n").map((s) => s.trim()).filter(Boolean).map((name) => name.replace(/^origin\//, "")).filter((name) => name !== "HEAD" && !name.endsWith("/HEAD"));
-	const unique = [...new Set(names)];
-	const current = git(repo, [
-		"rev-parse",
-		"--abbrev-ref",
-		"HEAD"
-	]).trim();
-	const bases = unique.filter((n) => n === "develop" || n === "main" || n === "master");
-	return {
-		branches: unique,
-		current,
-		defaultBase: bases.includes("develop") ? "develop" : bases.includes("main") ? "main" : bases[0] || "develop"
-	};
-}
-/** URL del remote (origin, o el primero disponible). Vacío si no hay. */
-function getRemoteUrl(repo, preferred = "origin") {
-	try {
-		const url = git(repo, [
-			"remote",
-			"get-url",
-			preferred
-		]).trim();
-		if (url) return url;
-	} catch {}
-	try {
-		const remotes = git(repo, ["remote"]).split("\n").map((s) => s.trim()).filter(Boolean);
-		if (!remotes.length) return "";
-		return git(repo, [
-			"remote",
-			"get-url",
-			remotes[0]
-		]).trim();
-	} catch {
-		return "";
-	}
-}
-//#endregion
-//#region ../lib/extract.mjs
-function extractBranchDiff({ repo, branch, base = "develop" }) {
-	if (!branch) throw new Error("Falta el branch a revisar.");
-	const config = loadReviewConfig(repo);
-	const mergeBase = git(repo, [
-		"merge-base",
-		base,
-		branch
-	]).trim();
-	const branchSha = git(repo, ["rev-parse", branch]).trim();
-	const baseSha = git(repo, ["rev-parse", base]).trim();
-	const nameStatus = git(repo, [
-		"diff",
-		mergeBase,
-		branch,
-		"--name-status"
-	]).trim().split("\n").filter(Boolean).map((line) => {
-		const [status, ...rest] = line.split("	");
-		const path = rest[rest.length - 1];
-		const oldPath = rest.length > 1 ? rest[0] : null;
-		let changeType = "modified";
-		if (status.startsWith("A")) changeType = "added";
-		else if (status.startsWith("D")) changeType = "deleted";
-		else if (status.startsWith("R")) changeType = "renamed";
-		return {
-			path,
-			oldPath,
-			changeType
-		};
-	});
-	const ignored = [];
-	const included = [];
-	for (const f of nameStatus) if (isIgnored(f.path, config.ignore)) ignored.push(f.path);
-	else included.push(f);
-	const files = included.map((f) => {
-		let raw = "";
-		try {
-			raw = git(repo, [
-				"diff",
-				mergeBase,
-				branch,
-				"--",
-				f.path
-			]);
-			const idx = raw.indexOf("\n@@");
-			if (idx !== -1) raw = raw.slice(idx + 1);
-		} catch {
-			raw = "";
-		}
-		const capped = capDiff(raw);
-		return {
-			path: f.path,
-			oldPath: f.oldPath || void 0,
-			changeType: f.changeType,
-			diff: capped.diff,
-			diffChars: capped.diffChars,
-			truncated: capped.truncated,
-			modelChars: capped.diff.length,
-			group: ""
-		};
-	});
-	const shortstat = git(repo, [
-		"diff",
-		mergeBase,
-		branch,
-		"--shortstat"
-	]).trim();
-	const insMatch = shortstat.match(/(\d+) insertion/);
-	const delMatch = shortstat.match(/(\d+) deletion/);
-	return {
-		version: 2,
-		repo,
-		branch,
-		baseBranch: base,
-		mergeBase,
-		branchSha,
-		baseSha,
-		generatedAt: (/* @__PURE__ */ new Date()).toISOString(),
-		intent: "",
-		stats: {
-			filesChanged: files.length,
-			filesIgnored: ignored.length,
-			filesTruncated: files.filter((f) => f.truncated).length,
-			insertions: insMatch ? Number(insMatch[1]) : 0,
-			deletions: delMatch ? Number(delMatch[1]) : 0
-		},
-		context: {
-			commits: commitLog(repo, mergeBase, branch),
-			rules: config.rules,
-			rulesFile: config.rulesFile,
-			ignored: ignored.slice(0, 40),
-			testsTouched: files.filter((f) => isTestPath(f.path)).map((f) => f.path),
-			testsMissing: files.filter((f) => !isTestPath(f.path)).length,
-			guidelines: collectGuidelines(repo, files.map((f) => f.path), {
-				rulesFile: config.rulesFile,
-				skipNames: config.rules.trim() ? [config.rulesFile] : [],
-				maxDepth: 1,
-				maxChars: 8e3,
-				maxTotalChars: 12e3
-			})
-		},
-		files,
-		groups: [],
-		blocks: [],
-		findings: [],
-		skipped: []
-	};
-}
-//#endregion
 //#region ../lib/remote-repo.mjs
 var CACHE_ROOT = join(tmpdir(), "diff-review-clones");
 var MAX_DEEPEN_ROUNDS = 8;
@@ -1610,59 +1115,6 @@ function ensureRemoteWorktree(url, branch, base = "develop") {
 	return dir;
 }
 //#endregion
-//#region ../lib/validate.mjs
-var GROUP_KINDS = [
-	"feat",
-	"fix",
-	"refactor",
-	"perf",
-	"test",
-	"chore",
-	"docs",
-	"infra"
-];
-var BLOCK_OPS = [
-	"add",
-	"mod",
-	"del",
-	"rename"
-];
-var WHY_SOURCES = [
-	"code",
-	"commit",
-	"pr",
-	"issue",
-	"inferred"
-];
-var FINDING_CLASSES = ["risk", "quality"];
-var FINDING_SEVERITIES = [
-	"high",
-	"med",
-	"low",
-	"nit"
-];
-var FINDING_KINDS = [
-	"bug",
-	"race",
-	"auth",
-	"security",
-	"api-break",
-	"missing-test",
-	"perf",
-	"maintainability",
-	"docs",
-	"other"
-];
-var SKIP_REASONS = [
-	"generated",
-	"lockfile",
-	"format",
-	"vendored",
-	"binary",
-	"trivial",
-	"ignored"
-];
-//#endregion
 //#region ../lib/review.mjs
 var MAX = {
 	intent: 700,
@@ -1672,7 +1124,7 @@ var MAX = {
 	why: 240,
 	finding: 360
 };
-function text$1(value, max) {
+function text(value, max) {
 	const raw = String(value ?? "").replace(/\s+/g, " ").trim();
 	return raw.length > max ? `${raw.slice(0, max - 1).trimEnd()}…` : raw;
 }
@@ -1716,8 +1168,8 @@ function normalizeGroups(raw) {
 		groups.push({
 			id,
 			kind: oneOf(item.kind, GROUP_KINDS, "chore"),
-			title: text$1(item.title, MAX.groupTitle) || `${oneOf(item.kind, GROUP_KINDS, "chore")}: cambios`,
-			intent: text$1(item.intent, MAX.groupIntent)
+			title: text(item.title, MAX.groupTitle) || `${oneOf(item.kind, GROUP_KINDS, "chore")}: cambios`,
+			intent: text(item.intent, MAX.groupIntent)
 		});
 	}
 	return groups;
@@ -1738,13 +1190,13 @@ function normalizeBlocks(raw, { groupIds, knownPaths }) {
 			id,
 			group,
 			file,
-			lines: range ? formatLineRange(range) : text$1(item.lines, 24),
+			lines: range ? formatLineRange(range) : text(item.lines, 24),
 			side: range?.side || "new",
 			start: range?.start ?? 0,
 			end: range?.end ?? 0,
 			op: oneOf(item.op, BLOCK_OPS, "mod"),
-			what: text$1(item.what, MAX.what),
-			why: text$1(item.why, MAX.why),
+			what: text(item.what, MAX.what),
+			why: text(item.why, MAX.why),
 			source: oneOf(item.source, WHY_SOURCES, "inferred")
 		});
 	}
@@ -1764,7 +1216,7 @@ function normalizeFindings(raw, { blockIds, knownPaths }) {
 	const findings = [];
 	for (const item of raw || []) {
 		const file = String(item?.file || item?.path || "").trim();
-		const what = text$1(item?.what || item?.title, MAX.finding);
+		const what = text(item?.what || item?.title, MAX.finding);
 		if (!file || !what) continue;
 		if (knownPaths.size && !knownPaths.has(file)) continue;
 		const cls = oneOf(item.class, FINDING_CLASSES, "risk");
@@ -1781,7 +1233,7 @@ function normalizeFindings(raw, { blockIds, knownPaths }) {
 			line,
 			block: blockIds.has(block) ? block : "",
 			what,
-			fix: text$1(item.fix || item.detail, MAX.finding)
+			fix: text(item.fix || item.detail, MAX.finding)
 		};
 		const key = findingKey(finding);
 		if (seen.has(key)) continue;
@@ -1858,7 +1310,7 @@ function buildReview({ skeleton, payload, notes = [] }) {
 	return {
 		...skeleton,
 		version: 2,
-		intent: text$1(payload?.intent, MAX.intent),
+		intent: text(payload?.intent, MAX.intent),
 		groups: liveGroups,
 		blocks,
 		findings,
@@ -2494,7 +1946,7 @@ function takePendingImport() {
 	}
 }
 /**
-* Busca un JSON de review en la raíz del repo local.
+* Busca un JSON de review en `.diff-review/` o (formato viejo) en la raíz del repo local.
 * Prefiere `preferredName` si existe; si no, el más reciente que matchee el patrón.
 */
 function detectReviewOutput(repo, preferredName) {
@@ -2508,8 +1960,8 @@ function detectReviewOutput(repo, preferredName) {
 	}
 	const preferred = String(preferredName || "").trim();
 	if (preferred && isOutputFilename(preferred)) {
-		const preferredPath = join(root, preferred);
-		if (existsSync(preferredPath)) try {
+		const preferredPath = locateReviewFile(root, preferred);
+		if (preferredPath) try {
 			return {
 				filename: preferred,
 				path: preferredPath,
@@ -2518,10 +1970,16 @@ function detectReviewOutput(repo, preferredName) {
 		} catch {}
 	}
 	let best = null;
-	try {
-		for (const name of readdirSync(root)) {
+	for (const dir of [reviewDir(root), root]) {
+		let names;
+		try {
+			names = readdirSync(dir);
+		} catch {
+			continue;
+		}
+		for (const name of names) {
 			if (!isOutputFilename(name)) continue;
-			const full = join(root, name);
+			const full = join(dir, name);
 			try {
 				const st = statSync(full);
 				if (!st.isFile()) continue;
@@ -2532,12 +1990,10 @@ function detectReviewOutput(repo, preferredName) {
 				};
 			} catch {}
 		}
-	} catch {
-		return null;
 	}
 	return best;
 }
-/** Lee y parsea un JSON de review en la raíz del repo (solo nombres válidos). */
+/** Lee y parsea un JSON de review de `.diff-review/` o la raíz del repo (solo nombres válidos). */
 function readReviewOutput(repo, filename) {
 	const raw = String(repo || "").trim();
 	const name = basename(String(filename || "").trim());
@@ -2548,8 +2004,8 @@ function readReviewOutput(repo, filename) {
 	} catch {
 		return null;
 	}
-	const filePath = join(root, name);
-	if (!existsSync(filePath)) return null;
+	const filePath = locateReviewFile(root, name);
+	if (!filePath) return null;
 	try {
 		return {
 			filename: name,
@@ -2559,7 +2015,21 @@ function readReviewOutput(repo, filename) {
 		return null;
 	}
 }
-async function pickLocalFolder$1() {
+/** Escribe (o pisa) el JSON de review en `.diff-review/` para poder abrirlo luego solo con el puntero. */
+function writeReviewOutput(repo, filename, payload) {
+	const raw = String(repo || "").trim();
+	const name = basename(String(filename || "").trim());
+	if (!raw || !name || !isOutputFilename(name) || looksLikeGitUrl(raw)) throw new Error("Repo local y nombre de salida válidos requeridos para guardar el JSON.");
+	const root = assertGitRepo(raw);
+	ensureReviewDir(root);
+	const path = companionPaths(root, name).json;
+	writeFileSync(path, `${JSON.stringify(payload, null, 2)}\n`, "utf8");
+	return {
+		filename: name,
+		payload
+	};
+}
+async function pickLocalFolder() {
 	return pickFolder();
 }
 /** Nombre de salida con tips cortos del branch y la base (repo local). */
@@ -2572,8 +2042,12 @@ function resolveOutputName(repo, branch, base) {
 		fingerprint: null
 	};
 	try {
-		const tips = branchTips(assertGitRepo(raw), b, baseRef);
+		const root = assertGitRepo(raw);
+		const tips = branchTips(root, b, baseRef);
 		const fp = fingerprint(tips.branchSha, tips.baseSha);
+		try {
+			ensureReviewDir(root);
+		} catch {}
 		return {
 			filename: makeOutputFilename({
 				branchSha: tips.branchSha,
@@ -2590,12 +2064,13 @@ function resolveOutputName(repo, branch, base) {
 		};
 	}
 }
-/** Diálogo nativo de JSON; abre en la raíz del repo si es local. */
+/** Diálogo nativo de JSON; abre en `.diff-review/` (o la raíz) si el repo es local. */
 async function pickReviewJsonFile(defaultDir) {
 	const dir = String(defaultDir || "").trim();
 	let start = dir;
 	if (dir && !looksLikeGitUrl(dir)) try {
-		start = assertGitRepo(dir);
+		const root = assertGitRepo(dir);
+		start = existsSync(reviewDir(root)) ? reviewDir(root) : root;
 	} catch {
 		start = dir;
 	}
@@ -2631,10 +2106,23 @@ function loadFileLines(input) {
 	}
 }
 function hydrateReviewPayload(input) {
-	const skeleton = extractBranchDiff({
-		repo: resolveWorktree(resolveSource(input.source, input.repo), input.repo, input.branch, input.base || "develop"),
+	const repo = resolveWorktree(resolveSource(input.source, input.repo), input.repo, input.branch, input.base || "develop");
+	const base = input.base || "develop";
+	let filename = String(input.outputFilename || "").trim();
+	if (!filename || !isOutputFilename(filename)) try {
+		const tips = branchTips(repo, input.branch, base);
+		filename = makeOutputFilename({
+			branchSha: tips.branchSha,
+			baseSha: tips.baseSha
+		});
+	} catch {
+		filename = "";
+	}
+	const skeleton = loadBranchDiff({
+		repo,
 		branch: input.branch,
-		base: input.base || "develop"
+		base,
+		...filename ? { filename } : {}
 	});
 	const extraNotes = Array.isArray(input.payload?.notes) ? input.payload.notes.filter((n) => typeof n === "string") : [];
 	const built = buildReview({
@@ -2653,128 +2141,6 @@ function hydrateReviewPayload(input) {
 	};
 	return JSON.parse(JSON.stringify(doc));
 }
-//#endregion
-//#region src/lib/report/git.remote.ts
-var git_remote_exports = /* @__PURE__ */ __exportAll({
-	detectOutput: () => detectOutput,
-	fileLines: () => fileLines,
-	hydrateFromGit: () => hydrateFromGit,
-	launchContext: () => launchContext,
-	loadDetectedOutput: () => loadDetectedOutput,
-	loadRepo: () => loadRepo,
-	outputName: () => outputName,
-	pendingImport: () => pendingImport,
-	pickLocalFolder: () => pickLocalFolder,
-	pickReviewFile: () => pickReviewFile
-});
-function fail(err) {
-	const message = describeGitError(err) || "No se pudo hablar con git.";
-	console.error("[diff-review git]", message, err);
-	error(400, message);
-}
-var loadRepo = /* @__PURE__ */ query("unchecked", async (input) => {
-	try {
-		return loadRepoInfo(input?.source, String(input?.repo || "").trim());
-	} catch (err) {
-		fail(err);
-	}
-});
-/** Repo del CWD de lanzamiento, o null si no hay git. */
-var launchContext = /* @__PURE__ */ query(async () => {
-	try {
-		return loadLaunchRepoInfo();
-	} catch (err) {
-		console.error("[diff-review git] launch context", describeGitError(err), err);
-		return null;
-	}
-});
-/** JSON pendiente del CLI (DIFF_REVIEW_IMPORT_FILE); se consume al leer. */
-var pendingImport = /* @__PURE__ */ query(async () => {
-	try {
-		return takePendingImport();
-	} catch (err) {
-		console.error("[diff-review git] pending import", describeGitError(err), err);
-		return null;
-	}
-});
-/** ¿Hay un JSON de review en la raíz del repo local? */
-var detectOutput = /* @__PURE__ */ query("unchecked", async (input) => {
-	try {
-		return detectReviewOutput(String(input?.repo || "").trim(), input?.preferredName);
-	} catch (err) {
-		console.error("[diff-review git] detect output", describeGitError(err), err);
-		return null;
-	}
-});
-/** Nombre sugerido con tips cortos (branch/base). */
-var outputName = /* @__PURE__ */ query("unchecked", async (input) => {
-	try {
-		return resolveOutputName(String(input?.repo || "").trim(), String(input?.branch || "").trim(), String(input?.base || "develop").trim());
-	} catch (err) {
-		console.error("[diff-review git] output name", describeGitError(err), err);
-		return {
-			filename: makeOutputFilename({}),
-			fingerprint: null
-		};
-	}
-});
-/** Lee un JSON detectado en el repo (sin diálogo). */
-var loadDetectedOutput = /* @__PURE__ */ command("unchecked", async (input) => {
-	try {
-		return readReviewOutput(String(input?.repo || "").trim(), String(input?.filename || "").trim());
-	} catch (err) {
-		fail(err);
-	}
-});
-var pickLocalFolder = /* @__PURE__ */ command("unchecked", async (_input) => {
-	try {
-		return await pickLocalFolder$1();
-	} catch (err) {
-		fail(err);
-	}
-});
-/** Diálogo nativo de JSON; `directory` = raíz del repo local. */
-var pickReviewFile = /* @__PURE__ */ command("unchecked", async (input) => {
-	try {
-		const picked = await pickReviewJsonFile(input?.directory);
-		if (!picked) return null;
-		return {
-			filename: basename(picked.path),
-			payload: decodePayload(picked.text)
-		};
-	} catch (err) {
-		fail(err);
-	}
-});
-/** Líneas del archivo en el branch revisado, para expandir el contexto oculto de un diff. */
-var fileLines = /* @__PURE__ */ query("unchecked", async (input) => loadFileLines({
-	repo: String(input?.repo || "").trim(),
-	branch: String(input?.branch || "").trim(),
-	base: String(input?.base || "develop").trim(),
-	source: input?.source,
-	path: String(input?.path || "").trim()
-}));
-var hydrateFromGit = /* @__PURE__ */ command("unchecked", async (input) => {
-	try {
-		return hydrateReviewPayload({
-			repo: String(input?.repo || "").trim(),
-			branch: String(input?.branch || "").trim(),
-			base: String(input?.base || "develop").trim(),
-			source: input?.source,
-			payload: input?.payload
-		});
-	} catch (err) {
-		fail(err);
-	}
-});
-init_remote_functions(git_remote_exports, "src/lib/report/git.remote.ts", "1g4ayag");
-for (const [name, fn] of Object.entries(git_remote_exports)) {
-	fn.__.id = "1g4ayag/" + name;
-	fn.__.name = name;
-}
-
-//#region \0sveltekit-remote:1g4ayag
-var _sveltekit_remote_1g4ayag_default = git_remote_exports;
 
-export { _sveltekit_remote_1g4ayag_default as default };
-//# sourceMappingURL=remote-1g4ayag-B67QMR_2.js.map
+export { loadLaunchRepoInfo as a, detectReviewOutput as b, command as c, describeGitError as d, readReviewOutput as e, pickReviewJsonFile as f, loadFileLines as g, hydrateReviewPayload as h, loadRepoInfo as l, pickLocalFolder as p, query as q, resolveOutputName as r, takePendingImport as t, writeReviewOutput as w };
+//# sourceMappingURL=git-review-CpO_G8z8.js.map