Add plugin OCR 翻译 v1.0.0#194
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a new OCR translation plugin for ZTools, featuring a Vue 3 frontend and a Python-based local OCR server using RapidOCR. The plugin supports various cloud providers and a local engine with pre-processing capabilities. Feedback focused on improving the robustness of the Python server's header parsing, preventing UI blocking by using asynchronous file operations in the Electron preload script, and enhancing type safety by enabling strict mode in the TypeScript configuration.
| return | ||
|
|
||
| try: | ||
| content_length = int(self.headers.get("Content-Length", "0")) |
There was a problem hiding this comment.
直接使用 int() 转换 Content-Length 可能会因为请求头缺失或格式非法而抛出 ValueError。建议增加校验或异常处理以提高服务器的健壮性。
| content_length = int(self.headers.get("Content-Length", "0")) | |
| cl_header = self.headers.get("Content-Length", "0") | |
| content_length = int(cl_header) if cl_header.isdigit() else 0 |
| readImageAsDataUrl(file) { | ||
| const ext = path.extname(file).slice(1).toLowerCase() | ||
| const mimeType = imageMimeTypes[ext] || 'application/octet-stream' | ||
| const base64 = fs.readFileSync(file).toString('base64') |
There was a problem hiding this comment.
在 Electron 环境中，fs.readFileSync 是同步阻塞操作。如果读取的图片文件较大，会直接导致渲染进程 UI 冻结，影响用户体验。建议改用异步的 fs.promises.readFile 并将 readImageAsDataUrl 设为 async 函数，同时更新调用方（如 index.vue 中的 loadImageFromFile）。
| "strict": false, | ||
| "noImplicitAny": false, |
There was a problem hiding this comment.
对于新创建的 TypeScript 项目，建议开启 strict 模式。当前的配置关闭了严格检查并允许隐式的 any 类型，这会降低类型系统的保护作用，增加维护成本。开启 strict: true 会自动包含 noImplicitAny 等多项安全检查。
| "strict": false, | |
| "noImplicitAny": false, | |
| "strict": true, |
|
|
感谢佬的贡献，经测试发现两个问题
|
|
好的感谢,我这就修复本地RapidOCR封装后没考虑mac环境,
|
- feat: OCR 翻译插件 v1.0.0 初始版本，支持多引擎图片文字识别 - fix: 修复 macOS onnxruntime 兼容性，移除 Python 一键安装，纯内置 RapidOCR 运行时，新增 Logo
- feat: OCR 翻译插件 v1.0.0 初始版本，支持多引擎图片文字识别 - fix: 修复 macOS onnxruntime 兼容性，移除 Python 一键安装，纯内置 RapidOCR 运行时，新增 Logo - fix: 修复 macOS/Linux 构建脚本，使用真实 OCR 服务代码
2 participants
插件信息
本次变更
截图 / 演示
自检清单
plugins/ocr-new/目录此 PR 由 ztools-plugin-cli 自动管理：每次
ztools publish在分支上追加一个 commit，PR 链接保持不变。