Skip to content

优化自动获取 Cookie 登录链路并补齐安装版 QA 回归 - #724

Open
guanyuefu-hub wants to merge 6 commits into
algerkong:mainfrom
guanyuefu-hub:codex/ampl-immersive-theme-system
Open

优化自动获取 Cookie 登录链路并补齐安装版 QA 回归#724
guanyuefu-hub wants to merge 6 commits into
algerkong:mainfrom
guanyuefu-hub:codex/ampl-immersive-theme-system

Conversation

@guanyuefu-hub

@guanyuefu-hub guanyuefu-hub commented Jul 12, 2026

Copy link
Copy Markdown

Summary

  • Improve auto Cookie login window status flow and waiting feedback.
  • Add installed-release QA scripts for release smoke, auto Cookie waiting state, and real-login closure.
  • Align installed-release smoke scripts with the current UI/store contracts to avoid stale-selector timeouts and Windows temp cleanup false failures.
  • Add release smoke and delivery documentation.

Commits

  • 06c183c fix(login): optimize auto Cookie waiting state and login guidance
  • d0690de test(qa): add installed-release regression and real-login closure scripts
  • 81b7a98 docs(qa): add installed-release delivery notes and QA entry
  • 4c56914 merge(main): sync upstream main and resolve loginWindow conflict
  • f0b2ee2 fix(qa): fix installed-release smoke first-run preconditions
  • 79cfe9b fix(qa): align installed-release smoke with current UI

Conflict Resolution

  • Synced with origin/main.
  • Resolved conflict in src/main/modules/loginWindow.ts.
  • Preserved auto Cookie status machine, attempt isolation, and status events.
  • Kept upstream fix: login window title is resolved at window-open time from current i18n state.

Verified Locally

  • node --check for changed QA scripts
  • npx eslint for changed QA scripts
  • npm run typecheck
  • npm run build
  • Installed-release smoke suite with AMPL_EXE_PATH=dist/win-unpacked/AlgerMusicPlayer.exe: pass=true, 7/7 items ok=true
    • theme-preset
    • search-playback
    • download-postactions
    • local-music-maintenance
    • local-music-health-repair
    • account-journey
    • auto-cookie-window

Current PR State

  • mergeable = MERGEABLE
  • mergeStateStatus = UNSTABLE
  • GitHub currently reports no checks: no checks reported

@guanyuefu-hub
guanyuefu-hub requested a review from algerkong as a code owner July 12, 2026 01:28
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