Skip to content

fix: cross-platform Windows/Linux support for browser auth and Chrome detection#4

Open
nikitabuida wants to merge 1 commit into
ForgetMeAI:mainfrom
nikitabuida:fix/win-cross-platform
Open

fix: cross-platform Windows/Linux support for browser auth and Chrome detection#4
nikitabuida wants to merge 1 commit into
ForgetMeAI:mainfrom
nikitabuida:fix/win-cross-platform

Conversation

@nikitabuida

Copy link
Copy Markdown

Исправления кросс-платформенной совместимости (Windows + Linux)

Проблемы

  1. defaultChromeExecutable() не содержал путей для Windows — браузер не находился
  2. import.meta.url сравнение в zai_browser_auth.js использовало сырую строку \ ile://\, что не работает на Windows (разный формат URL)
  3. readToken() падал с \Execution context was destroyed\ при навигации (Google OAuth редирект)
  4. cleanChromeProfileLocks() возвращал пути с обратной косой чертой на Windows
  5. import_zai_curl.js использовал жёстко заданный /tmp/\ путь

Изменения

Файл Изменение
\src/providers/zaiBrowser.js\ Добавлены пути Chrome/Edge для Windows; нормализация /\ в cleanChromeProfileLocks
\scripts/zai_browser_auth.js\ \
eadToken()\ обёрнут в try/catch; \import.meta.url\ через \pathToFileURL\
\scripts/import_zai_curl.js\ /tmp/\ заменён на \os.tmpdir()\

… detection

- Add Windows Chrome/Edge search paths to defaultChromeExecutable()
- Fix import.meta.url comparison using pathToFileURL for Windows compat
- Wrap readToken() in try/catch to handle navigation destroyed context
- Normalize path separators in cleanChromeProfileLocks()
- Replace hardcoded /tmp/ fallback with os.tmpdir() in import_zai_curl
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant