kodevu 0.1.30 → 0.1.32

package/README.md

@@ -1,63 +1,55 @@
 # Kodevu
 
-A Node.js tool that polls new SVN revisions or Git commits, fetches each change diff directly from the repository, sends the diff to a supported reviewer CLI, and writes review results to report files.
+A Node.js tool that fetches Git commits or SVN revisions, sends the diff to a supported AI reviewer CLI, and writes review results to report files.
 
 ## Workflow
 
 1. Detect the repository type automatically (Git or SVN).
-2. Read the latest change from `target`.
-3. Find changes that have not been reviewed yet.
-4. For each change:
-   - load metadata and changed paths from SVN or Git
-   - generate a unified diff for that single revision or commit
-   - send the diff and change metadata to the configured reviewer CLI
-   - allow the reviewer to inspect related local repository files in read-only mode when a local workspace is available
-   - write the result to `~/.kodevu/` (Markdown by default; optional JSON via config)
-5. Update `~/.kodevu/state.json` so the same change is not reviewed twice.
+2. Fetch the specified revision(s) as requested by the user:
+   - A single specific revision/commit via `--rev`.
+   - The latest $N$ revisions/commits via `--last` (default: 1).
+3. For each change:
+   - Load metadata and changed paths from SVN or Git.
+   - Generate a unified diff for that single revision or commit.
+   - Send the diff and change metadata to the configured reviewer CLI.
+   - Allow the reviewer to inspect related local repository files in read-only mode when a local workspace is available.
+   - Write the result to `~/.kodevu/` (Markdown by default; optional JSON via config).
+
+**Note**: Kodevu is stateless. It does not track which changes have been reviewed previously.
 
 ## Quick start
 
+Review the latest commit in your repository:
+
 ```bash
 npx kodevu /path/to/your/repo
 ```
 
-No config file is required for the default flow.
-Review reports are written to `~/.kodevu/` as Markdown (`.md`) by default, and review state is stored at `~/.kodevu/state.json`.
-
-If you want a config file, run `npx kodevu init` to create `./config.json` in the current directory.
-
-## Setup
+Review the latest 3 commits:
 
 ```bash
-npx kodevu init
+npx kodevu /path/to/your/repo --last 3
 ```
 
-This creates `config.json` in the current directory from built-in defaults.
-You only need this when you want to override defaults such as `reviewer` or output paths.
-
-If you want a different path:
+Review a specific commit:
 
 ```bash
-npx kodevu init --config ./config.current.json
+npx kodevu /path/to/your/repo --rev abc1234
 ```
 
-Then edit `config.json` if you want custom settings.
+Review reports are written to `~/.kodevu/` as Markdown (`.md`) by default.
 
-If you do not pass `--config`, Kodevu will try to load `./config.json` from the current directory only when that file exists. Otherwise it runs with built-in defaults.
-
-## Run
+## Setup
 
-Run:
+If you want to customize settings beyond the defaults:
 
 ```bash
-npx kodevu /path/to/your/repo
+npx kodevu init
 ```
 
-Run with debug logs:
+This creates `config.json` in the current directory. You only need this when you want to override defaults such as `reviewer` or output paths.
 
-```bash
-npx kodevu /path/to/your/repo --debug
-```
+## Run
 
 Specify the output language (e.g., Chinese):
 
@@ -65,60 +57,43 @@ Specify the output language (e.g., Chinese):
 npx kodevu /path/to/your/repo --lang zh
 ```
 
-Use a custom config path only when needed:
+Run with debug logs:
 
 ```bash
-npx kodevu --config ./config.current.json
+npx kodevu /path/to/your/repo --debug
 ```
 
-Or combine a config file with a positional target override:
+Use a custom config path:
 
 ```bash
-npx kodevu /path/to/your/repo --config ./config.current.json
+npx kodevu --config ./my-config.json
 ```
 
-`--debug` / `-d` is a CLI-only switch. It is not read from `config.json`.
-
-## Config
-
-- `target`: required repository target; can be provided by config or as the CLI positional argument
-- `reviewer`: `codex`, `gemini`, `copilot`, or `auto`; default `auto`
-- `prompt`: additional instructions for the reviewer; usually empty by default as the core instructions are built-in
-- `lang`: output language for the review (e.g., `zh`, `en`, `auto`); default `auto`
-- `outputDir`: report output directory; default `~/.kodevu`
-- `outputFormats`: report formats to generate; supports `markdown` and `json`; default `["markdown"]`
-- `stateFilePath`: review state file path; default `~/.kodevu/state.json`
-- `commandTimeoutMs`: timeout for a single review command execution in milliseconds
-- `maxRevisionsPerRun`: cap the number of pending changes per polling cycle
-
-Internal defaults:
+## Config / CLI Options
 
-- by default, review reports and state are stored under `~/.kodevu`; first run starts from the current latest change instead of replaying full history
-- if `./config.json` is absent, Kodevu still runs with built-in defaults as long as you pass a positional `target`
-- Kodevu invokes `git`, `svn`, and the configured reviewer CLI from `PATH`; when `reviewer` is `auto`, it randomly selects one from the installed reviewer CLIs it can find in `PATH`; debug logging is enabled only by passing `--debug` or `-d`
+- `target`: Repository target path or SVN URL. (CLI positional argument overrides config)
+- `reviewer`: `codex`, `gemini`, `copilot`, or `auto` (default: `auto`).
+- `rev`: A specific revision or commit hash to review.
+- `last`: Number of latest revisions to review (default: 1 if `rev` is not set).
+- `lang`: Output language for the review (e.g., `zh`, `en`, `auto`).
+- `prompt`: Additional instructions for the reviewer.
+- `outputDir`: Report output directory (default: `~/.kodevu`).
+- `outputFormats`: Report formats to generate (supports `markdown` and `json`; default: `["markdown"]`).
+- `commandTimeoutMs`: Timeout for a single review command execution in milliseconds.
+- `maxRevisionsPerRun`: Cap the number of revisions handled in one run.
 
 ## Target Rules
 
 - For SVN, `target` can be a working copy path or repository URL.
 - For Git, `target` must be a local repository path or a subdirectory inside a local repository.
-- The tool tries Git first for existing local paths, then falls back to SVN.
+- The tool tries Git first, then falls back to SVN.
 
 ## Notes
 
-- `reviewer: "codex"` uses `codex exec` with the diff embedded in the prompt.
-- `reviewer: "gemini"` uses `gemini -p` in non-interactive mode.
-- `reviewer: "copilot"` uses `copilot -p` in non-interactive mode.
-- `reviewer: "auto"` probes `codex`, `gemini`, and `copilot` in `PATH`, then randomly chooses one of the available CLIs for this run.
-- Large diffs are truncated before being sent to the reviewer or written into the report once they exceed the configured line or character limits.
-- For Git targets and local SVN working copies, the reviewer command runs from the repository workspace so it can inspect related files beyond the diff when needed.
-- For remote SVN URLs without a local working copy, the review still relies on the diff and change metadata only.
-- By default, SVN reports are written as `<YYYYMMDD-HHmmss>-svn-r<revision>.md`.
-- By default, Git reports are written as `<YYYYMMDD-HHmmss>-git-<short-commit-hash>.md`.
-- If `outputFormats` includes `json`, matching `.json` files are generated alongside Markdown reports.
-- `~/.kodevu/state.json` stores per-project checkpoints keyed by repository identity; only the v2 multi-project structure is supported.
-- If the reviewer command exits non-zero or times out, the report is still written, but the state is not advanced so the change can be retried later.
-- Review instructions are built into the tool in English to ensure consistent logic across reviewers. The `lang` setting (CLI `--lang` or config `lang`) determines the language AI uses for the resulting review. When set to `auto`, Kodevu detects the language from the system environment (`LANG`, `LC_ALL`, or system locale).
-- Each report includes a `Token Usage` section recording token consumption for the review task. When the reviewer CLI outputs token statistics (via stderr), those are used directly (`source: "reviewer"`). Otherwise tokens are estimated at ~4 characters per token (`source: "estimate"`). The JSON report contains a `tokenUsage` object with `inputTokens`, `outputTokens`, `totalTokens`, and `source`.
+- `reviewer: "auto"` probes `codex`, `gemini`, and `copilot` in `PATH`, then selects an available one.
+- Large diffs are truncated based on internal limits to fit AI context windows.
+- For Git targets and local SVN working copies, the reviewer command runs from the repository workspace so it can inspect related files beyond the diff.
+- If the reviewer command exits non-zero or times out, the report is still written containing the error details.
 
 ## License
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.30",
+  "version": "0.1.32",
   "license": "MIT",
   "type": "module",
   "description": "Poll SVN revisions or Git commits, send each change diff to a reviewer CLI, and write configurable review reports.",

package/src/config.js

@@ -12,12 +12,13 @@ const defaultConfig = {
   target: "",
   lang: "auto",
   outputDir: defaultStorageDir,
-  stateFilePath: path.join(defaultStorageDir, "state.json"),
   logsDir: path.join(defaultStorageDir, "logs"),
   commandTimeoutMs: 600000,
   prompt: "",
   maxRevisionsPerRun: 5,
-  outputFormats: ["markdown"]
+  outputFormats: ["markdown"],
+  rev: "",
+  last: 0
 };
 
 const configTemplate = {
@@ -26,11 +27,12 @@ const configTemplate = {
   lang: defaultConfig.lang,
   prompt: defaultConfig.prompt,
   outputDir: "~/.kodevu",
-  stateFilePath: "~/.kodevu/state.json",
   logsDir: "~/.kodevu/logs",
   commandTimeoutMs: defaultConfig.commandTimeoutMs,
   maxRevisionsPerRun: defaultConfig.maxRevisionsPerRun,
-  outputFormats: defaultConfig.outputFormats
+  outputFormats: defaultConfig.outputFormats,
+  rev: "",
+  last: 1
 };
 
 function resolveConfigPath(baseDir, value) {
@@ -145,6 +147,8 @@ export function parseCliArgs(argv) {
     reviewer: "",
     lang: "",
     prompt: "",
+    rev: "",
+    last: "",
     commandExplicitlySet: false
   };
 
@@ -208,6 +212,26 @@ export function parseCliArgs(argv) {
       continue;
     }
 
+    if (value === "--rev" || value === "-v") {
+      const rev = argv[index + 1];
+      if (!rev || rev.startsWith("-")) {
+        throw new Error(`Missing value for ${value}`);
+      }
+      args.rev = rev;
+      index += 1;
+      continue;
+    }
+
+    if (value === "--last" || value === "-n") {
+      const last = argv[index + 1];
+      if (!last || last.startsWith("-")) {
+        throw new Error(`Missing value for ${value}`);
+      }
+      args.last = last;
+      index += 1;
+      continue;
+    }
+
     if (!value.startsWith("-") && args.command === "run" && !args.target) {
       args.target = value;
       continue;
@@ -258,6 +282,14 @@ export async function loadConfig(configPath, cliArgs = {}) {
     config.lang = cliArgs.lang;
   }
 
+  if (cliArgs.rev) {
+    config.rev = cliArgs.rev;
+  }
+
+  if (cliArgs.last) {
+    config.last = cliArgs.last;
+  }
+
   if (!config.target) {
     throw new Error('Missing target. Pass `npx kodevu <repo-path>` or set "target" in config.json.');
   }
@@ -284,12 +316,16 @@ export async function loadConfig(configPath, cliArgs = {}) {
   config.configPath = loadedConfigPath;
   config.baseDir = baseDir;
   config.outputDir = resolveConfigPath(config.baseDir, config.outputDir);
-  config.stateFilePath = resolveConfigPath(config.baseDir, config.stateFilePath);
   config.logsDir = resolveConfigPath(config.baseDir, config.logsDir);
   config.maxRevisionsPerRun = Number(config.maxRevisionsPerRun);
   config.commandTimeoutMs = Number(config.commandTimeoutMs);
+  config.last = Number(config.last);
   config.outputFormats = normalizeOutputFormats(config.outputFormats, loadedConfigPath);
 
+  if (!config.rev && (isNaN(config.last) || config.last <= 0)) {
+    config.last = 1;
+  }
+
   if (!Number.isInteger(config.maxRevisionsPerRun) || config.maxRevisionsPerRun <= 0) {
     throw new Error(`"maxRevisionsPerRun" must be a positive integer${loadedConfigPath ? ` in ${loadedConfigPath}` : ""}`);
   }
@@ -317,6 +353,8 @@ Options:
   --reviewer, -r Override reviewer (codex | gemini | copilot | auto)
   --prompt, -p   Override prompt
   --lang, -l     Override output language (e.g. zh, en, auto)
+  --rev, -v      Review a specific revision or commit hash
+  --last, -n     Review the latest N revisions (default: 1)
   --debug, -d    Print extra debug information to the console
   --help, -h     Show help
 

package/src/git-client.js

@@ -124,6 +124,16 @@ export async function getPendingCommits(config, targetInfo, startExclusive, endI
   return splitLines(result.stdout).slice(0, limit);
 }
 
+export async function getLatestCommitIds(config, targetInfo, limit) {
+  const result = await runGit(
+    config,
+    ["rev-list", "-n", String(limit), "HEAD", ...buildPathArgs(targetInfo)],
+    { cwd: targetInfo.repoRootPath, trim: true }
+  );
+  // Reverse to get chronological order (oldest to newest among the latest n)
+  return splitLines(result.stdout).reverse();
+}
+
 export async function getCommitDiff(config, targetInfo, commitHash) {
   const result = await runGit(
     config,

package/src/report-generator.js

@@ -1,5 +1,13 @@
-export const CORE_REVIEW_INSTRUCTION =
-  "Please strictly review the current changes, prioritizing bugs, regression risks, compatibility issues, security concerns, boundary condition flaws, and missing tests. Please use Markdown for your response. If no clear flaws are found, write \"No clear flaws found\" and supplement with residual risks.";
+export function getCoreReviewInstruction(lang) {
+  const lowLang = (lang || "").toLowerCase();
+  if (lowLang.startsWith("zh")) {
+    if (lowLang === "zh-tw" || lowLang === "zh-hk") {
+      return "請嚴格審查目前的更改，優先考慮錯誤、回歸風險、相容性問題、安全問題、邊界條件缺陷和缺失的測試。請使用 Markdown 進行回覆。如果未發現明顯缺陷，請寫「未發現明顯缺陷」並補充風險。";
+    }
+    return "请严格审查当前的更改，优先处理 Bug、回归风险、兼容性问题、安全问题、边界条件缺陷和缺失的测试。请使用 Markdown 进行回复。如果未发现明显缺陷，请写“未发现明显缺陷”并补充残留风险。";
+  }
+  return "Please strictly review the current changes, prioritizing bugs, regression risks, compatibility issues, security concerns, boundary condition flaws, and missing tests. Please use Markdown for your response. If no clear flaws are found, write \"No clear flaws found\" and supplement with residual risks.";
+}
 
 export function getReviewWorkspaceRoot(config, backend, targetInfo) {
   if (backend.kind === "git" && targetInfo.repoRootPath) {
@@ -30,13 +38,77 @@ function getLanguageDisplayName(lang) {
   return lang;
 }
 
+const LOCALIZED_PHRASES = {
+  en: {
+    workspaceRoot: "You are running inside a read-only workspace rooted at:",
+    noWorkspace: "No local repository workspace is available for this review run.",
+    besidesDiff: "Besides the diff below, you may read other related files in the workspace when needed to understand call sites, shared utilities, configuration, tests, or data flow. Do not modify files or rely on shell commands.",
+    reviewFromDiff: "Review primarily from the diff below. Do not assume access to other local files, shell commands, or repository history.",
+    fileRefs: "Use plain text file references like path/to/file.js:123. Do not emit clickable workspace links.",
+    repoType: "Repository Type",
+    changeId: "Change ID",
+    author: "Author",
+    date: "Date",
+    changedFiles: "Changed files",
+    commitMessage: "Commit message",
+    diffNoteTruncated: "Diff delivery note: the diff was truncated before being sent to the reviewer to stay within configured size limits. Original diff size was {originalLineCount} lines / {originalCharCount} chars, and the included excerpt is {outputLineCount} lines / {outputCharCount} chars. Use the changed file list and inspect related workspace files when needed.",
+    diffNoteFull: "Diff delivery note: the full diff is included. Size is {originalLineCount} lines / {originalCharCount} chars.",
+    langRule: "--- IMPORTANT LANGUAGE RULE ---\nYou MUST respond strictly in {langName}. No other language should be used for the explanation and comments."
+  },
+  zh: {
+    workspaceRoot: "你正运行在一个只读工作区内，根目录为：",
+    noWorkspace: "此审查运行没有可用的本地仓库工作区。",
+    besidesDiff: "除了下面的 Diff，你可以在需要时阅读工作区中的其他相关文件，以了解调用点、共享工具、配置、测试或数据流。请勿修改文件或依赖 Shell 命令。",
+    reviewFromDiff: "主要根据下面的 Diff 进行审查。不要假设可以访问其他本地文件、Shell 命令或仓库历史。",
+    fileRefs: "使用纯文本文件引用，如 path/to/file.js:123。不要生成可点击的工作区链接。",
+    repoType: "仓库类型",
+    changeId: "变更 ID",
+    author: "作者",
+    date: "日期",
+    changedFiles: "已变更文件",
+    commitMessage: "提交信息",
+    diffNoteTruncated: "Diff 交付说明：Diff 在发送给审查者之前已被截斷，以保持在配置的大小限制内。原始 Diff 大小为 {originalLineCount} 行 / {originalCharCount} 个字符，包含的摘录为 {outputLineCount} 行 / {outputCharCount} 个字符。在需要时使用已更正文件列表并检查相关工作区文件。",
+    diffNoteFull: "Diff 交付说明：包含完整的 Diff。大小为 {originalLineCount} 行 / {originalCharCount} 个字符。",
+    langRule: "--- 重要语言规则 ---\n你必须严格使用 {langName} 进行回复。解释、评论和总结均不得使用其他语言。"
+  },
+  "zh-tw": {
+    workspaceRoot: "你正運行在一個唯讀工作區內，根目錄為：",
+    noWorkspace: "此審查運行沒有可用的本地倉庫工作區。",
+    besidesDiff: "除了下面的 Diff，你可以在需要時閱讀工作區中的其他相關文件，以了解調用點、共享工具、配置、測試或資料流。請勿修改文件或依賴 Shell 命令。",
+    reviewFromDiff: "主要根據下面的 Diff 進行審查。不要假設可以訪問其他本地文件、Shell 命令或倉庫歷史。",
+    fileRefs: "使用純文本文件引用，如 path/to/file.js:123。不要生成可點擊的工作區連結。",
+    repoType: "倉庫類型",
+    changeId: "變更 ID",
+    author: "作者",
+    date: "日期",
+    changedFiles: "已變更文件",
+    commitMessage: "提交信息",
+    diffNoteTruncated: "Diff 交付說明：Diff 在傳送給審查者之前已被截斷，以保持在配置的大小限制內。原始 Diff 大小為 {originalLineCount} 行 / {originalCharCount} 個字符，包含的摘錄為 {outputLineCount} 行 / {outputCharCount} 個字符。在需要時使用已更正文件列表並檢查相關工作區文件。",
+    diffNoteFull: "Diff 交付說明：包含完整的 Diff。大小為 {originalLineCount} 行 / {originalCharCount} 個字符。",
+    langRule: "--- 重要語言規則 ---\n你必須嚴格使用 {langName} 進行回覆。解釋、評論和總結均不得使用其他語言。"
+  }
+};
+
+function getPhrase(key, lang, placeholders = {}) {
+  const lowLang = (lang || "en").toLowerCase();
+  const langKey = lowLang.startsWith("zh") ? (lowLang === "zh-tw" || lowLang === "zh-hk" ? "zh-tw" : "zh") : "en";
+  let phrase = LOCALIZED_PHRASES[langKey][key] || LOCALIZED_PHRASES.en[key];
+
+  for (const [k, v] of Object.entries(placeholders)) {
+    phrase = phrase.replace(`{${k}}`, v);
+  }
+  return phrase;
+}
+
 export function buildPrompt(config, backend, targetInfo, details, reviewDiffPayload) {
   const fileList = details.changedPaths.map((item) => `${item.action} ${item.relativePath}`).join("\n");
   const workspaceRoot = getReviewWorkspaceRoot(config, backend, targetInfo);
   const canReadRelatedFiles = backend.kind === "git" || Boolean(targetInfo.workingCopyPath);
 
-  const langName = getLanguageDisplayName(config.resolvedLang);
-  const lowLang = (config.resolvedLang || "").toLowerCase();
+  const lang = config.resolvedLang || "en";
+  const langName = getLanguageDisplayName(lang);
+  const lowLang = lang.toLowerCase();
+
   let langInstruction = `IMPORTANT: Your entire response MUST be in ${langName}. All explanations, comments, and structure should strictly follow the ${langName} language rules.`;
 
   if (lowLang.startsWith("zh")) {
@@ -47,39 +119,41 @@ export function buildPrompt(config, backend, targetInfo, details, reviewDiffPayl
     }
   }
 
-
-
   return [
-    CORE_REVIEW_INSTRUCTION,
     langInstruction,
+    getCoreReviewInstruction(lang),
     config.prompt,
     canReadRelatedFiles
-      ? `You are running inside a read-only workspace rooted at: ${workspaceRoot}`
-      : "No local repository workspace is available for this review run.",
+      ? `${getPhrase("workspaceRoot", lang)} ${workspaceRoot}`
+      : getPhrase("noWorkspace", lang),
     canReadRelatedFiles
-      ? "Besides the diff below, you may read other related files in the workspace when needed to understand call sites, shared utilities, configuration, tests, or data flow. Do not modify files or rely on shell commands."
-      : "Review primarily from the diff below. Do not assume access to other local files, shell commands, or repository history.",
-    "Use plain text file references like path/to/file.js:123. Do not emit clickable workspace links.",
-    `Repository Type: ${backend.displayName}`,
-    `Change ID: ${details.displayId}`,
-    `Author: ${details.author}`,
-    `Date: ${details.date || "unknown"}`,
-    `Changed files:\n${fileList || "(none)"}`,
-    `Commit message:\n${details.message || "(empty)"}`,
+      ? getPhrase("besidesDiff", lang)
+      : getPhrase("reviewFromDiff", lang),
+    getPhrase("fileRefs", lang),
+    `${getPhrase("repoType", lang)}: ${backend.displayName}`,
+    `${getPhrase("changeId", lang)}: ${details.displayId}`,
+    `${getPhrase("author", lang)}: ${details.author}`,
+    `${getPhrase("date", lang)}: ${details.date || "unknown"}`,
+    `${getPhrase("changedFiles", lang)}:\n${fileList || "(none)"}`,
+    `${getPhrase("commitMessage", lang)}:\n${details.message || "(empty)"}`,
     reviewDiffPayload.wasTruncated
-      ? `Diff delivery note: the diff was truncated before being sent to the reviewer to stay within configured size limits. Original diff size was ${reviewDiffPayload.originalLineCount} lines / ${reviewDiffPayload.originalCharCount} chars, and the included excerpt is ${reviewDiffPayload.outputLineCount} lines / ${reviewDiffPayload.outputCharCount} chars. Use the changed file list and inspect related workspace files when needed.`
-      : `Diff delivery note: the full diff is included. Size is ${reviewDiffPayload.originalLineCount} lines / ${reviewDiffPayload.originalCharCount} chars.`,
-    `--- IMPORTANT LANGUAGE RULE ---\nYou MUST respond strictly in ${langName}. No other language should be used for the explanation and comments.${
-      lowLang.startsWith("zh")
+      ? getPhrase("diffNoteTruncated", lang, {
+          originalLineCount: reviewDiffPayload.originalLineCount,
+          originalCharCount: reviewDiffPayload.originalCharCount,
+          outputLineCount: reviewDiffPayload.outputLineCount,
+          outputCharCount: reviewDiffPayload.outputCharCount
+        })
+      : getPhrase("diffNoteFull", lang, {
+          originalLineCount: reviewDiffPayload.originalLineCount,
+          originalCharCount: reviewDiffPayload.originalCharCount
+        }),
+    getPhrase("langRule", lang, { langName }) +
+      (lowLang.startsWith("zh")
         ? lowLang === "zh-tw" || lowLang === "zh-hk"
           ? "\n請務必完全使用繁體中文進行回覆，所有的審查分析、注釋和總結都必須使用繁體中文。"
           : "\n请务必完全使用简体中文进行回复，所有的审查分析、注释和总结都必须使用简体中文。"
-        : ""
-    }`
-
+        : "")
   ].filter(Boolean).join("\n\n");
-
-
 }
 
 export function formatTokenUsage(tokenUsage) {

package/src/review-runner.js

@@ -7,14 +7,6 @@ import {
   writeTextFile,
   writeJsonFile
 } from "./utils.js";
-import {
-  loadState,
-  saveState,
-  getProjectState,
-  updateProjectState,
-  readLastReviewedId,
-  buildStateSnapshot
-} from "./state-manager.js";
 import {
   shouldWriteFormat,
   buildReport,
@@ -158,45 +150,19 @@ export async function runReviewCycle(config) {
 
   const { backend, targetInfo } = await resolveRepositoryContext(config);
   logger.debug(
-    `Resolved repository context: backend=${backend.kind} target=${targetInfo.targetDisplay || config.target} stateKey=${targetInfo.stateKey}`
-  );
-  const latestChangeId = await backend.getLatestChangeId(config, targetInfo);
-  const stateFile = await loadState(config.stateFilePath);
-  const projectState = getProjectState(stateFile, targetInfo);
-  let lastReviewedId = readLastReviewedId(projectState, backend, targetInfo);
-  logger.debug(
-    `Checkpoint status: latest=${backend.formatChangeId(latestChangeId)} lastReviewed=${lastReviewedId ? backend.formatChangeId(lastReviewedId) : "(none)"}`
+    `Resolved repository context: backend=${backend.kind} target=${targetInfo.targetDisplay || config.target}`
   );
 
-  if (lastReviewedId) {
-    const checkpointIsValid = await backend.isValidCheckpoint(config, targetInfo, lastReviewedId, latestChangeId);
-
-    if (!checkpointIsValid) {
-      logger.info("Saved review state no longer matches repository history. Resetting checkpoint.");
-      lastReviewedId = null;
-    }
-  }
-
   let changeIdsToReview = [];
 
-  if (!lastReviewedId) {
-    changeIdsToReview = [latestChangeId];
-    logger.info(`Initialized state to review the latest ${backend.changeName} ${backend.formatChangeId(latestChangeId)} first.`);
+  if (config.rev) {
+    changeIdsToReview = [config.rev];
   } else {
-    changeIdsToReview = await backend.getPendingChangeIds(
-      config,
-      targetInfo,
-      lastReviewedId,
-      latestChangeId,
-      config.maxRevisionsPerRun
-    );
+    changeIdsToReview = await backend.getLatestChangeIds(config, targetInfo, config.last || 1);
   }
 
-  logger.debug(`Planned ${changeIdsToReview.length} ${backend.changeName}(s) for this cycle.`);
-
   if (changeIdsToReview.length === 0) {
-    const lastKnownId = lastReviewedId ? backend.formatChangeId(lastReviewedId) : "(none)";
-    logger.info(`No new ${backend.changeName}s. Last reviewed: ${lastKnownId}`);
+    logger.info("No changes found to review.");
     return;
   }
 
@@ -215,20 +181,12 @@ export async function runReviewCycle(config) {
       updateOverallProgress(progress, index, changeIdsToReview.length, fraction, `${displayId} | ${stage}`);
     };
 
-    let result;
-
     try {
-      result = await reviewChange(config, backend, targetInfo, changeId, { update: syncOverallProgress, log: (message) => progress.log(message) });
-      syncOverallProgress(0.94, "saving checkpoint");
+      await reviewChange(config, backend, targetInfo, changeId, { update: syncOverallProgress, log: (message) => progress.log(message) });
+      updateOverallProgress(progress, index + 1, changeIdsToReview.length, 0, `finished ${displayId}`);
     } catch (error) {
       progress.fail(`failed at ${displayId} (${index}/${changeIdsToReview.length} completed)`);
       throw error;
     }
-
-    const nextProjectSnapshot = buildStateSnapshot(backend, targetInfo, changeId);
-    await saveState(config.stateFilePath, updateProjectState(stateFile, targetInfo, nextProjectSnapshot));
-    stateFile.projects[targetInfo.stateKey] = nextProjectSnapshot;
-    logger.debug(`Saved checkpoint for ${backend.formatChangeId(changeId)} to ${config.stateFilePath}.`);
-    updateOverallProgress(progress, index + 1, changeIdsToReview.length, 0, `finished ${displayId}`);
   }
 }

package/src/svn-client.js

@@ -112,6 +112,20 @@ export async function getPendingRevisions(config, targetInfo, startExclusive, en
     .slice(0, limit);
 }
 
+export async function getLatestRevisionIds(config, targetInfo, limit) {
+  const result = await runCommand(
+    SVN_COMMAND,
+    ["log", "--xml", "--quiet", "-l", String(limit), "-r", "HEAD:1", getRemoteTarget(targetInfo, config)],
+    { encoding: COMMAND_ENCODING, trim: true, debug: config.debug }
+  );
+  const parsed = xmlParser.parse(result.stdout);
+
+  return asArray(parsed?.log?.logentry)
+    .map((entry) => Number(entry?.revision))
+    .filter((revision) => Number.isInteger(revision))
+    .sort((left, right) => left - right);
+}
+
 export async function getRevisionDiff(config, revision) {
   const result = await runCommand(
     SVN_COMMAND,

package/src/vcs-client.js

@@ -42,8 +42,8 @@ function createSvnBackend() {
     async getLatestChangeId(config, targetInfo) {
       return await svnClient.getLatestRevision(config, targetInfo);
     },
-    async getPendingChangeIds(config, targetInfo, startExclusive, endInclusive, limit) {
-      return await svnClient.getPendingRevisions(config, targetInfo, startExclusive, endInclusive, limit);
+    async getLatestChangeIds(config, targetInfo, limit) {
+      return await svnClient.getLatestRevisionIds(config, targetInfo, limit);
     },
     async getChangeDiff(config, targetInfo, revision) {
       return await svnClient.getRevisionDiff(config, revision);
@@ -59,16 +59,6 @@ function createSvnBackend() {
         message: details.message,
         changedPaths: details.changedPaths
       };
-    },
-    async isValidCheckpoint() {
-      return true;
-    },
-    toStateValue(revision) {
-      return Number(revision);
-    },
-    fromStateValue(state) {
-      const id = state.lastReviewedId;
-      return Number.isInteger(id) ? id : null;
     }
   };
 }
@@ -99,8 +89,8 @@ function createGitBackend() {
     async getLatestChangeId(config, targetInfo) {
       return await gitClient.getLatestCommit(config, targetInfo);
     },
-    async getPendingChangeIds(config, targetInfo, startExclusive, endInclusive, limit) {
-      return await gitClient.getPendingCommits(config, targetInfo, startExclusive, endInclusive, limit);
+    async getLatestChangeIds(config, targetInfo, limit) {
+      return await gitClient.getLatestCommitIds(config, targetInfo, limit);
     },
     async getChangeDiff(config, targetInfo, commitHash) {
       return await gitClient.getCommitDiff(config, targetInfo, commitHash);
@@ -116,16 +106,6 @@ function createGitBackend() {
         message: details.message,
         changedPaths: details.changedPaths
       };
-    },
-    async isValidCheckpoint(config, targetInfo, checkpointCommit, latestCommit) {
-      return await gitClient.isValidCheckpoint(config, targetInfo, checkpointCommit, latestCommit);
-    },
-    toStateValue(commitHash) {
-      return String(commitHash);
-    },
-    fromStateValue(state) {
-      const id = state.lastReviewedId;
-      return typeof id === "string" && id ? id : null;
     }
   };
 }

package/src/state-manager.js

@@ -1,61 +0,0 @@
-import fs from "node:fs/promises";
-import path from "node:path";
-import { pathExists, ensureDir } from "./utils.js";
-
-function normalizeStateFile(state) {
-  if (!state || typeof state !== "object" || Array.isArray(state)) {
-    throw new Error('State file must be a JSON object with shape {"version":2,"projects":{...}}.');
-  }
-
-  if (state.version !== 2) {
-    throw new Error('State file version must be 2.');
-  }
-
-  if (!state.projects || typeof state.projects !== "object" || Array.isArray(state.projects)) {
-    throw new Error('State file must contain a "projects" object.');
-  }
-
-  return {
-    version: 2,
-    projects: state.projects
-  };
-}
-
-export async function loadState(stateFile) {
-  if (!(await pathExists(stateFile))) {
-    return { version: 2, projects: {} };
-  }
-
-  const raw = await fs.readFile(stateFile, "utf8");
-  return normalizeStateFile(JSON.parse(raw));
-}
-
-export async function saveState(stateFile, state) {
-  await ensureDir(path.dirname(stateFile));
-  await fs.writeFile(stateFile, `${JSON.stringify(state, null, 2)}\n`, "utf8");
-}
-
-export function getProjectState(stateFile, targetInfo) {
-  return stateFile.projects?.[targetInfo.stateKey] ?? {};
-}
-
-export function updateProjectState(stateFile, targetInfo, projectState) {
-  return {
-    version: 2,
-    projects: {
-      ...(stateFile.projects || {}),
-      [targetInfo.stateKey]: projectState
-    }
-  };
-}
-
-export function readLastReviewedId(state, backend, targetInfo) {
-  return backend.fromStateValue(state);
-}
-
-export function buildStateSnapshot(backend, targetInfo, changeId) {
-  return {
-    lastReviewedId: backend.toStateValue(changeId),
-    updatedAt: new Date().toISOString()
-  };
-}