Skip to content

Dev cy 02 #901

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
apoint123 merged 4 commits into from
Feb 18, 2026
Merged

Dev cy 02 #901

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
ecb3286
chore: 将 @electron/node-gyp 依赖从 git 协议更改为 https tarball
kazukokawagawa Feb 18, 2026
b74a89f
refactor(store): 移除重复的状态属性并添加进度属性
kazukokawagawa Feb 18, 2026
9f475d5
fix(音频播放器): 修复加载时暂停状态不一致的问题
kazukokawagawa Feb 18, 2026
ca20cfc
refactor(stores): 调整状态存储中字段顺序
kazukokawagawa Feb 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions pnpm-lock.yaml
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

11 changes: 11 additions & 0 deletions src/core/audio-player/ffmpeg-engine/FFmpegAudioPlayer.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -157,6 +157,9 @@ export class FFmpegAudioPlayer extends BaseAudioPlayer {
this.dispatch("loadstart");

this.init();
if (this.audioCtx && this.audioCtx.state === "running") {
await this.audioCtx.suspend().catch(() => undefined);
}

try {
if (this.worker) {
Expand Down Expand Up @@ -186,6 +189,10 @@ export class FFmpegAudioPlayer extends BaseAudioPlayer {
file: file,
chunkSize: 4096 * 8,
});
this.isWorkerPaused = true;
await this.requestWorker({ type: "PAUSE" }).catch(() => {
this.isWorkerPaused = false;
});
} else {
await this.loadSrc(url as string);
}
Expand Down Expand Up @@ -233,6 +240,10 @@ export class FFmpegAudioPlayer extends BaseAudioPlayer {

this.runFetchLoop(url, 0, this.fileSize);
await initWorkerPromise;
this.isWorkerPaused = true;
await this.requestWorker({ type: "PAUSE" }).catch(() => {
this.isWorkerPaused = false;
});
} catch (e) {
const err = toError(e);
console.error("[Player] LoadSrc error:", err);
Expand Down
3 changes: 1 addition & 2 deletions src/stores/status.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -179,10 +179,9 @@ export const useStatusStore = defineStore("status", {
playListShow: false,
showFullPlayer: false,
playerMetaShow: true,
automixFxSeq: 0,
automixEndedSeq: 0,
currentTime: 0,
duration: 0,
progress: 0,
Comment on lines 182 to +184
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The fields automixFxSeq and automixEndedSeq have been removed from the state initialization, but they are still declared in the StatusState interface (lines 166-167), still used in actions triggerAutomixFx() and endAutomixFx() (lines 308-313), and still watched in PlayerControl.vue (lines 177, 185). This will cause runtime errors when these actions try to increment undefined properties. Either restore these fields to the state initialization or remove all references to them throughout the codebase.

Copilot uses AI. Check for mistakes.
currentTimeOffsetMap: {},
songCoverTheme: {},
pureLyricMode: false,
Expand Down