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

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` | — | Показать информацию о версии |
 
@@ -307,6 +321,11 @@ ocr review \
 ## Примеры
 
 ```bash
+# Интерактивная настройка провайдера и модели
+ocr config provider
+ocr config model
+ocr llm providers
+
 # Показать, какие файлы попадут в ревью (без вызовов LLM)
 ocr review --preview
 ocr review -c abc123 -p

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alibaba-group/open-code-review",
-  "version": "1.3.12",
+  "version": "1.3.13",
   "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;
       }
     }