fix(dicts): fix youdaotrans and caiyun translation engines

- youdaotrans: use free demo API, no API key required
- caiyun: require user API token (website redesigned)
- youdao: limit to short text (max: 5 words)
- reorder: youdaotrans before caiyun
- pdf.js: fix Windows compatibility (use curl)
- README: add Node.js 17+ build instructions

Author: bylsxy

.gitignore

@@ -97,3 +97,4 @@ test/**/response
 
 
 **/.DS_Store
+scripts/pdfjs.zip  

README-zh.md

@@ -45,6 +45,21 @@ yarn build
 
 在 `build/` 目录下可查看针对各个浏览器打包好的扩展包。
 
+### Node.js 17+ 用户注意
+
+如果使用 Node.js 17 或更高版本，构建时可能遇到 OpenSSL 错误。请在构建前设置环境变量：
+
+```bash
+# Linux/macOS
+export NODE_OPTIONS=--openssl-legacy-provider
+
+# Windows (cmd)
+set NODE_OPTIONS=--openssl-legacy-provider
+
+# Windows (PowerShell)
+$env:NODE_OPTIONS="--openssl-legacy-provider"
+```
+
 ## 开发
 
 见[项目贡献指南](./CONTRIBUTING-zh.md)。

README.md

@@ -49,6 +49,21 @@ yarn build
 
 Artifacts can be found in `build/`.
 
+### Note for Node.js 17+
+
+If you are using Node.js 17 or later, you may encounter an OpenSSL error. Set this environment variable before building:
+
+```bash
+# Linux/macOS
+export NODE_OPTIONS=--openssl-legacy-provider
+
+# Windows (cmd)
+set NODE_OPTIONS=--openssl-legacy-provider
+
+# Windows (PowerShell)
+$env:NODE_OPTIONS="--openssl-legacy-provider"
+```
+
 ## Development
 
 See the [contributing guide](./CONTRIBUTING.md).

scripts/pdf.js

@@ -12,7 +12,7 @@ if (!shell.which('git')) {
 }
 
 const cacheDir = 'pdf'
-const repoRoot = 'pdf'
+let repoRoot = 'pdf'
 const publicPDFRoot = path.join(__dirname, '../assets/pdf')
 const pdfFiles = [
   'build/pdf.js',
@@ -27,22 +27,38 @@ const files = [...pdfFiles, ...pdfDirs]
 
 shell.cd(path.resolve(__dirname))
 
-shell.rm('-rf', cacheDir)
+const distRoot = getPdfDistRoot()
+const useLocalDist =
+  !!distRoot && fs.existsSync(path.join(distRoot, 'web/viewer.js'))
 
-exec(
-  `wget https://github.com/mozilla/pdf.js/releases/download/v2.16.105/pdfjs-2.16.105-dist.zip -O pdfjs.tar.gz &&
-  mkdir -p ${cacheDir} &&
-  tar -xzvf pdfjs.tar.gz -C ${cacheDir}`,
-  'Error: download failed'
-)
+if (useLocalDist) {
+  shell.echo('Using pdfjs-dist from node_modules.')
+  repoRoot = distRoot
+} else {
+  shell.rm('-rf', cacheDir)
+
+  shell.mkdir('-p', cacheDir)
+
+  const zipName = 'pdfjs.zip'
+  const zipPath = path.join(__dirname, zipName)
+
+  const downloadCmd =
+    `curl -L --retry 3 --retry-delay 2 --max-time 300 -k https://github.com/mozilla/pdf.js/releases/download/v2.16.105/pdfjs-2.16.105-dist.zip -o "${zipPath}"` +
+    ` || curl -L --retry 3 --retry-delay 2 --max-time 300 -k https://ghproxy.com/https://github.com/mozilla/pdf.js/releases/download/v2.16.105/pdfjs-2.16.105-dist.zip -o "${zipPath}"` +
+    ` || curl -L --retry 3 --retry-delay 2 --max-time 300 -k https://download.fastgit.org/mozilla/pdf.js/releases/download/v2.16.105/pdfjs-2.16.105-dist.zip -o "${zipPath}"`
 
-shell.cd('./' + cacheDir)
+  exec(downloadCmd, 'Error: download failed')
+
+  exec(`tar -xf "${zipPath}" -C "${cacheDir}"`, 'Error: download failed')
+
+  repoRoot = path.join(__dirname, cacheDir)
+}
 
 startUpgrade()
 
 async function startUpgrade() {
   shell.echo('\nChecking files.')
-  await Promise.all(files.map(p => exists(path.join(__dirname, repoRoot, p))))
+  await Promise.all(files.map(p => exists(resolveRepoPath(p))))
 
   shell.echo('\nModifying files.')
   await Promise.all([modifyViewrJS(), modifyViewerHTML()])
@@ -62,7 +78,7 @@ async function startUpgrade() {
 }
 
 async function modifyViewrJS() {
-  const viewerPath = path.join(__dirname, repoRoot, 'web/viewer.js')
+  const viewerPath = resolveRepoPath('web/viewer.js')
   let file = await fs.readFile(viewerPath, 'utf8')
 
   file = '/* saladict */ window.__SALADICT_PDF_PAGE__ = true;\n' + file
@@ -97,7 +113,7 @@ async function modifyViewrJS() {
 }
 
 async function modifyViewerHTML() {
-  const viewerPath = path.join(__dirname, repoRoot, 'web/viewer.html')
+  const viewerPath = resolveRepoPath('web/viewer.html')
   let file = await fs.readFile(viewerPath, 'utf8')
 
   if (!file.includes(`</body>`)) {
@@ -126,6 +142,21 @@ function cleanInit() {
   })
 }
 
+function getPdfDistRoot() {
+  try {
+    return path.dirname(require.resolve('pdfjs-dist/package.json'))
+  } catch (e) {
+    return null
+  }
+}
+
+function resolveRepoPath(subPath) {
+  if (path.isAbsolute(repoRoot)) {
+    return path.join(repoRoot, subPath)
+  }
+  return path.join(__dirname, repoRoot, subPath)
+}
+
 async function exists(path) {
   try {
     await fs.access(path)
@@ -152,26 +183,26 @@ async function cloneFiles() {
   for (const pdfFile of pdfFiles) {
     const targetPath = path.join(publicPDFRoot, pdfFile)
     await fs.ensureFile(targetPath)
-    await fs.copy(path.join(__dirname, repoRoot, pdfFile), targetPath)
+    await fs.copy(resolveRepoPath(pdfFile), targetPath)
   }
 
   const restPdfDirs = pdfDirs.filter(name => name !== 'web/locale')
 
   for (const pdfDir of restPdfDirs) {
     const targetPath = path.join(publicPDFRoot, pdfDir)
     await fs.ensureDir(targetPath)
-    await fs.copy(path.join(__dirname, repoRoot, pdfDir), targetPath)
+    await fs.copy(resolveRepoPath(pdfDir), targetPath)
   }
 
   // copy locale.properties
   await fs.ensureDir(path.join(publicPDFRoot, 'web/locale'))
   await fs.copy(
-    path.join(__dirname, repoRoot, 'web/locale/locale.properties'),
+    resolveRepoPath('web/locale/locale.properties'),
     path.join(publicPDFRoot, 'web/locale/locale.properties')
   )
 
   const locales = (
-    await fs.readdir(path.join(__dirname, repoRoot, 'web/locale'))
+    await fs.readdir(resolveRepoPath('web/locale'))
   ).filter(
     name =>
       name.startsWith('en') ||
@@ -182,9 +213,6 @@ async function cloneFiles() {
   for (const locale of locales) {
     const targetPath = path.join(publicPDFRoot, 'web/locale', locale)
     await fs.ensureDir(targetPath)
-    await fs.copy(
-      path.join(__dirname, repoRoot, 'web/locale', locale),
-      targetPath
-    )
+    await fs.copy(resolveRepoPath(path.join('web/locale', locale)), targetPath)
   }
 }

src/app-config/profiles.ts

@@ -42,8 +42,8 @@ export function _getDefaultProfile(id?: string) {
         'youdao',
         'urban',
         'vocabulary',
-        'caiyun',
         'youdaotrans',
+        'caiyun',
         'zdic',
         'guoyu',
         'liangan',
@@ -203,8 +203,8 @@ export function translation(): ProfileStorage {
     'google',
     'tencent',
     'baidu',
-    'caiyun',
     'youdaotrans',
+    'caiyun',
     'zdic',
     'guoyu',
     'liangan'

src/components/MachineTrans/engine.ts

@@ -1,6 +1,6 @@
 import { DictID, AppConfig } from '@/app-config'
 import { Language } from '@opentranslate/languages'
-import { Translator } from '@opentranslate/translator'
+import { Translator, TranslateResult } from '@opentranslate/translator'
 import { DictItem, SelectOptions } from '@/app-config/dicts'
 import { isContainJapanese, isContainKorean } from '@/_helpers/lang-check'
 import { DictSearchResult } from '../dictionaries/helpers'
@@ -257,3 +257,91 @@ export function machineResult<ID extends DictID>(
     catalog
   }
 }
+
+export const MACHINE_TRANSLATE_CHAR_LIMIT = 4500
+
+function chunkText(text: string, limit = MACHINE_TRANSLATE_CHAR_LIMIT) {
+  const chunks: string[] = []
+  let current = ''
+
+  const pushCurrent = () => {
+    if (current) {
+      chunks.push(current)
+      current = ''
+    }
+  }
+
+  for (const raw of text.split(/\n+/)) {
+    const part = raw.trim()
+    if (!part) {
+      continue
+    }
+
+    if (part.length > limit) {
+      pushCurrent()
+      for (let i = 0; i < part.length; i += limit) {
+        chunks.push(part.slice(i, i + limit))
+      }
+      continue
+    }
+
+    const candidate = current ? `${current}\n${part}` : part
+    if (candidate.length > limit) {
+      pushCurrent()
+      current = part
+    } else {
+      current = candidate
+    }
+  }
+
+  pushCurrent()
+
+  return chunks.length ? chunks : ['']
+}
+
+export async function translateInChunks(
+  translator: Translator,
+  text: string,
+  sl: Language,
+  tl: Language,
+  config: Record<string, unknown> | undefined,
+  limit = MACHINE_TRANSLATE_CHAR_LIMIT
+): Promise<TranslateResult> {
+  const segments = chunkText(text, limit)
+
+  const aggregated: TranslateResult = {
+    from: sl,
+    to: tl,
+    origin: { paragraphs: [] },
+    trans: { paragraphs: [] }
+  }
+
+  let currentSl = sl
+  let currentTl = tl
+
+  for (const segment of segments) {
+    const res = await translator.translate(
+      segment,
+      currentSl,
+      currentTl,
+      config
+    )
+
+    aggregated.from = res.from
+    aggregated.to = res.to
+    aggregated.origin.paragraphs.push(...res.origin.paragraphs)
+    aggregated.trans.paragraphs.push(...res.trans.paragraphs)
+
+    if (res.origin.tts && !aggregated.origin.tts) {
+      aggregated.origin.tts = res.origin.tts
+    }
+    if (res.trans.tts && !aggregated.trans.tts) {
+      aggregated.trans.tts = res.trans.tts
+    }
+
+    currentSl = res.from as Language
+    currentTl = res.to as Language
+  }
+
+  return aggregated
+}

src/components/dictionaries/baidu/engine.ts

@@ -5,7 +5,8 @@ import {
   MachineTranslateResult,
   MachineTranslatePayload,
   getMTArgs,
-  machineResult
+  machineResult,
+  translateInChunks
 } from '@/components/MachineTrans/engine'
 import { BaiduLanguage } from './config'
 
@@ -56,8 +57,31 @@ export const search: SearchFunction<
   const key = config.dictAuth.baidu.key
   const translatorConfig = appid && key ? { appid, key } : undefined
 
+  if (!translatorConfig) {
+    return machineResult(
+      {
+        result: {
+          id: 'baidu',
+          slInitial: 'hide',
+          sl,
+          tl,
+          searchText: { paragraphs: [''] },
+          trans: { paragraphs: [''] },
+          requireCredential: true
+        }
+      },
+      translator.getSupportLanguages()
+    )
+  }
+
   try {
-    const result = await translator.translate(text, sl, tl, translatorConfig)
+    const result = await translateInChunks(
+      translator,
+      text,
+      sl,
+      tl,
+      translatorConfig
+    )
     return machineResult(
       {
         result: {