@alibaba-group/open-code-review 1.3.12 → 1.3.14

package/README.ja-JP.md

@@ -311,6 +311,7 @@ ocr review \
 | `--timeout` | — | `10` | 同時実行タスクのタイムアウト（分） |
 | `--audience` | — | `human` | `human`（進捗を表示）または`agent`（サマリーのみ） |
 | `--background` | `-b` | — | レビューのための任意の要件/ビジネスコンテキスト。`--commit`使用時に未指定の場合、コミットメッセージから自動取得 |
+| `--model` | — | — | このレビューでLLMモデルを選択または上書き |
 | `--rule` | — | — | カスタムJSONレビュールールへのパス |
 | `--max-tools` | — | 組み込み値 | ファイルごとのツール呼び出しラウンドの上限。テンプレートのデフォルトより大きい場合のみ有効 |
 | `--max-git-procs` | — | 組み込み値 | gitサブプロセスの最大同時実行数 |
@@ -337,6 +338,10 @@ ocr review --from main --to my-feature --concurrency 4
 # 特定のコミットを詳細なJSON出力でレビュー
 ocr review --commit abc123 --format json --audience agent
 
+# このレビューでモデルを選択またはオーバーライド
+ocr review --model claude-opus-4-6
+ocr review --commit abc123 --model claude-sonnet-4-6
+
 # 要件コンテキストを提供してより的確なレビューを実施
 ocr review --background "ログインAPIにレート制限を追加"
 
@@ -449,8 +454,9 @@ OCRは4層の優先度チェーンを使ってレビュールールを解決し
 | `providers.<name>.url` | string | プロバイダーのベースURLオーバーライド |
 | `providers.<name>.protocol` | string | `anthropic` \| `openai` |
 | `providers.<name>.model` | string | プロバイダーのモデル名 |
+| `providers.<name>.models` | array | 対話的選択に使う任意のプロバイダーモデル一覧 |
 | `providers.<name>.auth_header` | string | `x-api-key` \| `authorization` |
-| `custom_providers.<name>.*` | — | `providers.<name>.*`と同じフィールド |
+| `custom_providers.<name>.*` | — | 任意の`models`を含む`providers.<name>.*`と同じフィールド |
 | `llm.url` | string | `https://api.openai.com/v1/chat/completions` |
 | `llm.auth_token` | string | `sk-xxxxxxx` |
 | `llm.auth_header` | string | Anthropicのみ：`x-api-key` \| `authorization` |

package/README.ko-KR.md

@@ -311,6 +311,7 @@ ocr review \
 | `--timeout` | - | `10` | 동시 task timeout(분) |
 | `--audience` | - | `human` | `human`(progress 표시) 또는 `agent`(summary only) |
 | `--background` | `-b` | - | 리뷰를 위한 선택적 요구사항/비즈니스 컨텍스트. `--commit` 사용 시 미지정이면 commit message에서 자동 추출 |
+| `--model` | - | - | 이번 리뷰에서 LLM model 선택 또는 override |
 | `--rule` | - | - | custom JSON review rules 경로 |
 | `--max-tools` | - | built-in | 파일별 최대 tool call round. template default보다 클 때만 적용 |
 | `--max-git-procs` | - | built-in | 최대 동시 git subprocess 수 |
@@ -337,6 +338,10 @@ ocr review --from main --to my-feature --concurrency 4
 # 특정 commit을 verbose JSON output으로 리뷰
 ocr review --commit abc123 --format json --audience agent
 
+# 이번 리뷰에서 model 선택 또는 override
+ocr review --model claude-opus-4-6
+ocr review --commit abc123 --model claude-sonnet-4-6
+
 # 요구사항 컨텍스트를 제공하여 더 정확한 리뷰 수행
 ocr review --background "로그인 API에 rate limiting 추가"
 
@@ -407,8 +412,9 @@ Config file: `~/.opencodereview/config.json`
 | `providers.<name>.url` | string | Provider base URL override |
 | `providers.<name>.protocol` | string | `anthropic` \| `openai` |
 | `providers.<name>.model` | string | Provider의 model 이름 |
+| `providers.<name>.models` | array | 대화형 선택에 사용할 optional provider model 목록 |
 | `providers.<name>.auth_header` | string | `x-api-key` \| `authorization` |
-| `custom_providers.<name>.*` | — | `providers.<name>.*`과 동일한 필드 |
+| `custom_providers.<name>.*` | — | optional `models`를 포함한 `providers.<name>.*`과 동일한 필드 |
 | `llm.url` | string | `https://api.openai.com/v1/chat/completions` |
 | `llm.auth_token` | string | `sk-xxxxxxx` |
 | `llm.auth_header` | string | Anthropic only: `x-api-key` \| `authorization` |

package/README.md

@@ -313,6 +313,7 @@ See the [`examples/`](./examples/) directory for integration examples:
 | `--timeout` | — | `10` | Concurrent task timeout in minutes |
 | `--audience` | — | `human` | `human` (show progress) or `agent` (summary only) |
 | `--background` | `-b` | — | Optional requirement/business context for the review; auto-filled from commit message when using `--commit` |
+| `--model` | — | — | Select or override the LLM model for this review |
 | `--rule` | — | — | Path to custom JSON review rules |
 | `--max-tools` | — | built-in | Max tool call rounds per file; only takes effect when greater than template default |
 | `--max-git-procs` | — | built-in | Max concurrent git subprocesses |
@@ -339,6 +340,10 @@ ocr review --from main --to my-feature --concurrency 4
 # Review a specific commit with verbose JSON output
 ocr review --commit abc123 --format json --audience agent
 
+# Select or override model for this review
+ocr review --model claude-opus-4-6
+ocr review --commit abc123 --model claude-sonnet-4-6
+
 # Provide requirement context for more targeted review
 ocr review --background "Adding rate limiting to the login API"
 
@@ -451,8 +456,9 @@ Config file: `~/.opencodereview/config.json`
 | `providers.<name>.url` | string | Provider base URL override |
 | `providers.<name>.protocol` | string | `anthropic` \| `openai` |
 | `providers.<name>.model` | string | Model name for the provider |
+| `providers.<name>.models` | array | Optional provider model list for interactive selection |
 | `providers.<name>.auth_header` | string | `x-api-key` \| `authorization` |
-| `custom_providers.<name>.*` | — | Same fields as `providers.<name>.*` |
+| `custom_providers.<name>.*` | — | Same fields as `providers.<name>.*`, including optional `models` |
 | `llm.url` | string | `https://api.openai.com/v1/chat/completions` |
 | `llm.auth_token` | string | `sk-xxxxxxx` |
 | `llm.auth_header` | string | Anthropic only: `x-api-key` \| `authorization` |

package/README.ru-RU.md

@@ -113,18 +113,22 @@ sudo cp dist/opencodereview /usr/local/bin/ocr
 
 **Перед запуском ревью необходимо настроить LLM.**
 
+**Вариант A: интерактивная настройка (рекомендуется)**
+
+```bash
+ocr config provider          # Выбрать встроенного провайдера или добавить пользовательский
+ocr config model             # Выбрать модель для активного провайдера
+```
+
+![Provider setup](imgs/providers.jpg)
+
+**Вариант B: ручная настройка**
+
 ```bash
-# Вариант A: настройка через CLI
 ocr config set llm.url https://api.anthropic.com/v1/messages
 ocr config set llm.auth_token your-api-key-here
 ocr config set llm.model claude-opus-4-6
 ocr config set llm.use_anthropic true
-
-# Вариант B: переменные окружения (наивысший приоритет)
-export OCR_LLM_URL=https://api.anthropic.com/v1/messages
-export OCR_LLM_TOKEN=your-api-key-here
-export OCR_LLM_MODEL=claude-opus-4-6
-export OCR_USE_ANTHROPIC=true
 ```
 
 Конфигурация хранится в `~/.opencodereview/config.json`.
@@ -133,12 +137,19 @@ export OCR_USE_ANTHROPIC=true
 
 ```bash
 ocr config set llm.auth_header x-api-key
-# или
-export OCR_LLM_AUTH_HEADER=x-api-key
 ```
 
 Поддерживаемые значения: `x-api-key`, `authorization` (алиас: `bearer`). Прочие значения отклоняются с ошибкой.
 
+**Вариант C: переменные окружения (наивысший приоритет)**
+
+```bash
+export OCR_LLM_URL=https://api.anthropic.com/v1/messages
+export OCR_LLM_TOKEN=your-api-key-here
+export OCR_LLM_MODEL=claude-opus-4-6
+export OCR_USE_ANTHROPIC=true
+```
+
 Инструмент также совместим с переменными окружения Claude Code (`ANTHROPIC_BASE_URL`, `ANTHROPIC_AUTH_TOKEN`, `ANTHROPIC_MODEL`) и разбирает `~/.zshrc` / `~/.bashrc` в поисках соответствующих export'ов.
 
 > **Примечание для пользователей CC-Switch**: если вы используете [CC-Switch](https://github.com/farion1231/cc-switch) с включённым [routing service](https://www.ccswitch.io/en/docs?section=proxy&item=service), можно указать в `llm.url` адрес прокси CC-Switch без дополнительной настройки:
@@ -280,8 +291,11 @@ ocr review \
 |---------|-------|----------|
 | `ocr review` | `ocr r` | Запустить код-ревью |
 | `ocr rules check <file>` | — | Показать, какое правило ревью применяется к пути файла |
+| `ocr config provider` | — | Интерактивная настройка провайдера (встроенный, пользовательский или ручной) |
+| `ocr config model` | — | Интерактивный выбор модели для активного провайдера |
 | `ocr config set <key> <value>` | — | Установить значения конфигурации |
 | `ocr llm test` | — | Проверить подключение к LLM |
+| `ocr llm providers` | — | Показать список встроенных LLM-провайдеров |
 | `ocr viewer` | `ocr v` | Запустить WebUI-просмотрщик сессий на `localhost:5483` |
 | `ocr version` | — | Показать информацию о версии |
 
@@ -299,6 +313,7 @@ ocr review \
 | `--timeout` | — | `10` | Таймаут конкурентной задачи в минутах |
 | `--audience` | — | `human` | `human` (показывать прогресс) или `agent` (только сводка) |
 | `--background` | `-b` | — | Необязательный контекст требований/бизнес-логики для ревью; при `--commit` автоматически заполняется из сообщения коммита |
+| `--model` | — | — | Выбрать или переопределить LLM-модель для этого ревью |
 | `--rule` | — | — | Путь к пользовательским JSON-правилам ревью |
 | `--max-tools` | — | встроенное | Максимум раундов вызова инструментов на файл; действует, только если больше значения шаблона по умолчанию |
 | `--max-git-procs` | — | встроенное | Максимум одновременных git-подпроцессов |
@@ -307,6 +322,11 @@ ocr review \
 ## Примеры
 
 ```bash
+# Интерактивная настройка провайдера и модели
+ocr config provider
+ocr config model
+ocr llm providers
+
 # Показать, какие файлы попадут в ревью (без вызовов LLM)
 ocr review --preview
 ocr review -c abc123 -p
@@ -320,6 +340,10 @@ ocr review --from main --to my-feature --concurrency 4
 # Ревью конкретного коммита с подробным JSON-выводом
 ocr review --commit abc123 --format json --audience agent
 
+# Выбрать или переопределить модель для этого ревью
+ocr review --model claude-opus-4-6
+ocr review --commit abc123 --model claude-sonnet-4-6
+
 # Передать контекст требований для более прицельного ревью
 ocr review --background "Добавляем rate limiting в API логина"
 
@@ -427,6 +451,14 @@ OCR разрешает правила ревью по цепочке приор
 
 | Ключ | Тип | Пример |
 |------|-----|--------|
+| `provider` | string | `anthropic` \| `openai` \| `dashscope` \| `deepseek` \| `z-ai` |
+| `providers.<name>.api_key` | string | API-ключ провайдера |
+| `providers.<name>.url` | string | Переопределение base URL провайдера |
+| `providers.<name>.protocol` | string | `anthropic` \| `openai` |
+| `providers.<name>.model` | string | Имя модели провайдера |
+| `providers.<name>.models` | array | Необязательный список моделей для интерактивного выбора |
+| `providers.<name>.auth_header` | string | `x-api-key` \| `authorization` |
+| `custom_providers.<name>.*` | — | Те же поля, что и `providers.<name>.*`, включая необязательное `models` |
 | `llm.url` | string | `https://api.openai.com/v1/chat/completions` |
 | `llm.auth_token` | string | `sk-xxxxxxx` |
 | `llm.auth_header` | string | Только для Anthropic: `x-api-key` \| `authorization` |

package/README.zh-CN.md

@@ -311,6 +311,7 @@ ocr review \
 | `--timeout` | — | `10` | 并发任务超时时间（分钟） |
 | `--audience` | — | `human` | `human`（显示进度）或 `agent`（仅输出摘要） |
 | `--background` | `-b` | — | 可选的需求/业务背景信息；使用 `--commit` 时如未指定则自动从 commit message 中提取 |
+| `--model` | — | — | 为本次审查选择或覆盖 LLM 模型 |
 | `--rule` | — | — | 自定义 JSON 审查规则路径 |
 | `--max-tools` | — | 内置默认 | 每个文件的最大工具调用轮次；仅在大于模板默认值时生效 |
 | `--max-git-procs` | — | 内置默认 | 最大并发 git 子进程数 |
@@ -337,6 +338,10 @@ ocr review --from main --to my-feature --concurrency 4
 # 审查特定提交并以 JSON 格式输出详细信息
 ocr review --commit abc123 --format json --audience agent
 
+# 为本次审查选择或覆盖模型
+ocr review --model claude-opus-4-6
+ocr review --commit abc123 --model claude-sonnet-4-6
+
 # 提供需求背景以获得更有针对性的审查
 ocr review --background "为登录 API 添加限流"
 
@@ -439,8 +444,9 @@ OCR 通过四层优先级链解析评审规则。每层采用首次匹配原则
 | `providers.<name>.url` | string | 供应商 Base URL 覆盖 |
 | `providers.<name>.protocol` | string | `anthropic` \| `openai` |
 | `providers.<name>.model` | string | 供应商模型名称 |
+| `providers.<name>.models` | array | 用于交互式选择的可选供应商模型列表 |
 | `providers.<name>.auth_header` | string | `x-api-key` \| `authorization` |
-| `custom_providers.<name>.*` | — | 与 `providers.<name>.*` 相同的字段 |
+| `custom_providers.<name>.*` | — | 与 `providers.<name>.*` 相同的字段，包括可选的 `models` |
 | `llm.url` | string | `https://api.openai.com/v1/chat/completions` |
 | `llm.auth_token` | string | `sk-xxxxxxx` |
 | `llm.auth_header` | string | 仅 Anthropic：`x-api-key` \| `authorization` |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alibaba-group/open-code-review",
-  "version": "1.3.12",
+  "version": "1.3.14",
   "description": "OpenCodeReview CLI — AI-powered code review tool",
   "bin": {
     "ocr": "bin/ocr.js"

package/scripts/install.js

@@ -189,37 +189,46 @@ async function main() {
   }
 
   if (config.checksumPattern) {
+    const checksumUrl = buildUrl(config.checksumPattern, vars);
+    info("Verifying checksum...");
+    let shaContent;
     try {
-      const checksumUrl = buildUrl(config.checksumPattern, vars);
-      info("Verifying checksum...");
-      const shaContent = await downloadText(checksumUrl);
-      const actualSha = await computeChecksum(binaryDest);
-
-      let verified = false;
-      for (const line of shaContent.split("\n")) {
-        const trimmed = line.trim();
-        if (trimmed.includes(`-${os}-${arch}`)) {
-          const expectedSha = trimmed.split(/\s+/)[0].toLowerCase();
-          if (expectedSha) {
-            if (actualSha !== expectedSha) {
-              throw new Error(
-                `Checksum mismatch! Expected: ${expectedSha}, Got: ${actualSha}`
-              );
-            }
-            info("Checksum verified.");
-            verified = true;
-            break;
+      shaContent = await downloadText(checksumUrl);
+    } catch (e) {
+      try { fs.unlinkSync(binaryDest); } catch (_) {}
+      throw new Error(`Failed to download checksum from ${checksumUrl}: ${e.message}`);
+    }
+    let actualSha;
+    try {
+      actualSha = await computeChecksum(binaryDest);
+    } catch (e) {
+      try { fs.unlinkSync(binaryDest); } catch (_) {}
+      throw new Error(`Failed to compute checksum for ${binaryDest}: ${e.message}`);
+    }
+
+    let verified = false;
+    for (const line of shaContent.split("\n")) {
+      const trimmed = line.trim();
+      if (trimmed.includes(`-${os}-${arch}`)) {
+        const expectedSha = trimmed.split(/\s+/)[0].toLowerCase();
+        if (expectedSha) {
+          if (actualSha !== expectedSha) {
+            try { fs.unlinkSync(binaryDest); } catch (_) {}
+            throw new Error(
+              `Checksum mismatch! Expected: ${expectedSha}, Got: ${actualSha}`
+            );
           }
+          info("Checksum verified.");
+          verified = true;
+          break;
         }
       }
-      if (!verified) {
-        warn("No matching checksum entry found; skipping verification.");
-      }
-    } catch (e) {
-      if (e.message.includes("mismatch")) {
-        throw e;
-      }
-      warn(`Could not verify checksum: ${e.message}`);
+    }
+    if (!verified) {
+      try { fs.unlinkSync(binaryDest); } catch (_) {}
+      throw new Error(
+        `No matching checksum entry for ${os}-${arch} in ${checksumUrl}`
+      );
     }
   }
 

package/scripts/update.js

@@ -4,7 +4,6 @@
 const fs = require("fs");
 const path = require("path");
 const os = require("os");
-const http = require("http");
 const https = require("https");
 const { spawnSync } = require("child_process");
 
@@ -85,14 +84,14 @@ function fetchLatestVersion(pkg) {
   if (!pkgName) return Promise.resolve(null);
   const encodedName = pkgName.replace(/\//g, "%2F");
   const url = `${registry.replace(/\/$/, "")}/${encodedName}/latest`;
-  const client = url.startsWith("https") ? https : http;
+  if (!url.startsWith("https://")) return Promise.resolve(null);
 
   return new Promise((resolve) => {
     const options = {
       headers: { "User-Agent": "ocr-updater", Accept: "application/json" },
       timeout: 15000,
     };
-    const req = client
+    const req = https
       .get(url, options, (res) => {
         if (res.statusCode !== 200) {
           res.resume();
@@ -173,26 +172,38 @@ async function main() {
     }
 
     if (config.checksumPattern) {
+      const checksumUrl = buildUrl(config.checksumPattern, vars);
+      let shaContent;
       try {
-        const checksumUrl = buildUrl(config.checksumPattern, vars);
-        const shaContent = await downloadText(checksumUrl);
-        const actualSha = await computeChecksum(tempPath);
-
-        let verified = false;
-        for (const line of shaContent.split("\n")) {
-          const trimmed = line.trim();
-          if (trimmed.includes(`-${platform}-${arch}`)) {
-            const expectedSha = trimmed.split(/\s+/)[0].toLowerCase();
-            if (expectedSha && actualSha !== expectedSha) {
-              fs.unlinkSync(tempPath);
-              return;
-            }
-            verified = true;
-            break;
+        shaContent = await downloadText(checksumUrl);
+      } catch (_) {
+        fs.unlinkSync(tempPath);
+        return;
+      }
+      let actualSha;
+      try {
+        actualSha = await computeChecksum(tempPath);
+      } catch (_) {
+        fs.unlinkSync(tempPath);
+        return;
+      }
+
+      let verified = false;
+      for (const line of shaContent.split("\n")) {
+        const trimmed = line.trim();
+        if (trimmed.includes(`-${platform}-${arch}`)) {
+          const expectedSha = trimmed.split(/\s+/)[0].toLowerCase();
+          if (expectedSha && actualSha !== expectedSha) {
+            fs.unlinkSync(tempPath);
+            return;
           }
+          verified = true;
+          break;
         }
-      } catch (_) {
-        // checksum fetch failed, continue with the download
+      }
+      if (!verified) {
+        fs.unlinkSync(tempPath);
+        return;
       }
     }