fix sakura api base handling

hgmzhn · hgmzhn · commit e60d46adcd62 · 2026-06-28T19:51:27.000+08:00
diff --git a/doc/CHANGELOG_v3.0.0.md b/doc/CHANGELOG_v3.0.0.md
@@ -0,0 +1,24 @@
+# v3.0.0 更新日志
+
+发布日期：2026-06-28
+
+## 🐛 修复与改进
+
+### AI 渲染器文本替换修复
+- 修复了在使用 AI 渲染器（OpenAI / Gemini）时，`text_replacements.yaml` 中配置的文本替换规则（如通用/横排/竖排符号或文本正则替换）不生效的问题。现在 AI 渲染模块在组装翻译 Prompt 前会正确加载并应用该规则。
+
+### AI 渲染器前置画框与坐标去除（极简参考原文渲染 + 彻底擦除原文防重叠）
+- 彻底去除了 AI 渲染器在发送的前置图像上绘制红色编号框与数字的逻辑，直接发送干净的去字/原图背景。
+- 移除了冗长且大模型不易精确理解的绝对像素坐标，改为极简提示词，让模型在渲染时根据“原文（original text）”参考定位气泡。
+- **强制擦除原文防重叠**：优化了默认提示词，明确要求大模型在渲染翻译前，必须**完全擦除原图上对应的日语文本**，防止翻译后的文字与原日语文字在气泡中重叠或并存。同时，也会自动识别并干净擦除所有残留的日语注音假名（Furigana/Rubi 字符）以及漏检文本，并将其转换为翻译渲染，保证整页漫画画面百分之百干净。
+
+### 修复异常文本误判导致 API 密钥被误拉黑的问题
+- 修复了当 AI 渲染失败抛出通用 `RuntimeError` 异常时，其文本描述中包含 "HTTP 404" 的排查细节信息会被轮询系统（`api_key_rotation.py`）的正则表达式抓取，误判定为发生了真实的 HTTP 404 永久网络错误，从而导致该 API Key 被错误的拉黑屏蔽。现在系统在提取状态码时会自动过滤并跳过 `RuntimeError`、`ValueError` 等内置异常类型的正则匹配，避免发生误拉黑。
+
+### 修复渲染报错被错误归类为“模型不支持多模态”的问题
+- 修复了当 AI 渲染发生其他错误（例如 API 404 响应细节描述）而抛出异常时，由于异常消息中带有 `"vision"` 或 `"image_url"` 关键字，导致客户端 UI 界面（`app_logic.py`）将其误判并显示为“模型不支持多模态输入”引导信息的 Bug。现已增加豁免判定，排除由渲染器（Renderer）抛出的此类异常误匹配。
+
+### 修复 Sakura API 地址更新与失败处理
+- 修复 Sakura 翻译器只读取启动时 `SAKURA_API_BASE` 的问题。现在 UI、Web 用户配置或预设中修改 Sakura 地址后，会在翻译任务解析配置时重新应用，避免地址改为 8081 后实际仍连接 8080。
+- Sakura 翻译开始时会在日志中输出当前连接地址，连接重试与最终失败日志也会带上地址，便于定位实际请求的端口。
+- 修复 Sakura 连接失败达到重试上限后返回原文、导致流程继续渲染的问题。现在会抛出异常并终止任务，避免把原文误当翻译结果。
diff --git a/manga_translator/server/routes/config.py b/manga_translator/server/routes/config.py
@@ -84,6 +84,8 @@
     "GEMINI_API_KEY",
     "GEMINI_API_BASE",
     "GEMINI_MODEL",
+    "SAKURA_API_BASE",
+    "SAKURA_DICT_PATH",
     "OCR_OPENAI_API_KEY",
     "OCR_OPENAI_API_BASE",
     "OCR_OPENAI_MODEL",
diff --git a/manga_translator/server/runtime_api.py b/manga_translator/server/runtime_api.py
@@ -1,6 +1,11 @@
 from typing import Optional
 
 RUNTIME_API_ENV_PRIORITY = {
+    "translator": {
+        "sakura": {
+            "api_base": ["SAKURA_API_BASE"],
+        },
+    },
     "ocr": {
         "openai": {
             "api_key": ["OCR_OPENAI_API_KEY"],
diff --git a/manga_translator/translators/sakura.py b/manga_translator/translators/sakura.py
@@ -9,9 +9,13 @@
 import logging
 from typing import Callable, List, Tuple
 
+from ..runtime_api_resolver import get_runtime_api_override
 from .common import CommonTranslator, validate_openai_response
 from .keys import SAKURA_API_BASE, SAKURA_DICT_PATH
 
+DEFAULT_SAKURA_API_BASE = 'http://127.0.0.1:8080/v1'
+DEFAULT_SAKURA_DICT_PATH = './dict/sakura_dict.txt'
+
 
 class SakuraDict():
     def __init__(self, path: str, logger: logging.Logger) -> None:
@@ -209,12 +213,10 @@ class SakuraTranslator(CommonTranslator):
 
     def __init__(self):
         super().__init__()
-        self.client = openai.AsyncOpenAI(api_key = openai.api_key or 'empty')
-        if "/v1" not in SAKURA_API_BASE:
-            self.client.base_url = SAKURA_API_BASE + "/v1"
-        else:
-            self.client.base_url = SAKURA_API_BASE
-        self.client.api_key = "sk-114514"
+        self.api_base = self._normalize_api_base(os.getenv('SAKURA_API_BASE') or SAKURA_API_BASE)
+        self.dict_path = os.getenv('SAKURA_DICT_PATH') or SAKURA_DICT_PATH or DEFAULT_SAKURA_DICT_PATH
+        self.client = None
+        self._setup_client()
         self.temperature = 0.3
         self.top_p = 0.3
         self.frequency_penalty = 0.1
@@ -223,8 +225,56 @@ def __init__(self):
         self._heart_pattern = re.compile(r'❤')
         self.sakura_dict = SakuraDict(self.get_dict_path(), self.logger)
 
+    @staticmethod
+    def _normalize_api_base(api_base: str) -> str:
+        normalized = (
+            str(api_base or SAKURA_API_BASE or DEFAULT_SAKURA_API_BASE).strip()
+            or DEFAULT_SAKURA_API_BASE
+        )
+        normalized = normalized.rstrip("/")
+        if "/v1" not in normalized:
+            normalized = f"{normalized}/v1"
+        return normalized
+
+    def _setup_client(self):
+        self.client = openai.AsyncOpenAI(
+            api_key="sk-114514",
+            base_url=self.api_base,
+        )
+
+    def parse_args(self, config):
+        super().parse_args(config)
+        translator_args = self._resolve_translator_config(config)
+        user_env_vars = getattr(config, "_user_env_vars", None) or {}
+        runtime_override = get_runtime_api_override(config, "translator", "sakura")
+
+        api_base = (
+            self._get_config_value(translator_args, "user_api_base", None)
+            or runtime_override.get("api_base")
+            or user_env_vars.get("SAKURA_API_BASE")
+            or os.getenv("SAKURA_API_BASE")
+            or SAKURA_API_BASE
+            or DEFAULT_SAKURA_API_BASE
+        )
+        api_base = self._normalize_api_base(api_base)
+        if api_base != self.api_base:
+            self.api_base = api_base
+            self._setup_client()
+            self.logger.info(f"Sakura API base updated: {self.api_base}")
+
+        dict_path = (
+            user_env_vars.get("SAKURA_DICT_PATH")
+            or os.getenv("SAKURA_DICT_PATH")
+            or SAKURA_DICT_PATH
+            or DEFAULT_SAKURA_DICT_PATH
+        )
+        dict_path = str(dict_path or DEFAULT_SAKURA_DICT_PATH).strip() or DEFAULT_SAKURA_DICT_PATH
+        if dict_path != self.dict_path:
+            self.dict_path = dict_path
+            self.sakura_dict = SakuraDict(self.get_dict_path(), self.logger)
+
     def get_dict_path(self):
-        return SAKURA_DICT_PATH
+        return getattr(self, 'dict_path', None) or SAKURA_DICT_PATH or DEFAULT_SAKURA_DICT_PATH
 
     def detect_and_caculate_repeats(self, s: str, threshold: int = _REPEAT_DETECT_THRESHOLD, remove_all=True) -> Tuple[bool, str, int, str]:
         """
@@ -457,6 +507,7 @@ def _delete_quotation_mark(self, texts: List[str]) -> List[str]:
 
     async def _translate(self, from_lang: str, to_lang: str, queries: List[str], ctx=None) -> List[str]:
         self.logger.debug(f'Temperature: {self.temperature}, TopP: {self.top_p}')
+        self.logger.info(f'Sakura当前连接地址: {self.api_base}')
         self.logger.debug(f'原文： {queries}')
         text_prompt = '\n'.join(queries)
         self.logger.debug('-- Sakura Prompt --\n' + self._format_prompt_log(text_prompt) + '\n\n')
@@ -500,11 +551,9 @@ async def _handle_translation_request(self, prompt) -> str:
             except (openai.APIError, openai.APIConnectionError) as e:
                 server_error_attempt += 1
                 if server_error_attempt >= self._RETRY_ATTEMPTS:
-                    self.logger.error(f'Sakura API请求失败。错误信息： {e}')
-                    if isinstance(prompt, list):
-                        return "\n".join(prompt)
-                    return str(prompt)
-                self.logger.warning(f'Sakura因服务器错误而进行重试。尝试次数： {server_error_attempt}，错误信息： {e}')
+                    self.logger.error(f'Sakura API请求失败。地址：{self.api_base}，错误信息： {e}')
+                    raise Exception(f'Sakura API请求失败（地址：{self.api_base}）：{e}') from e
+                self.logger.warning(f'Sakura因服务器错误而进行重试。地址：{self.api_base}，尝试次数： {server_error_attempt}，错误信息： {e}')
 
         return response
 
