fix(music): playing an unknown key leaves the current track playing but untracked

未关闭 适合新手
#498 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
82/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
typescript
领域
backend, testing

调研方向

阅读 modules/music/src/music.library.ts 第 61–74 行,追踪 play() 如何处理缺失的键和当前曲目。添加一个单元测试,在另一首曲目正在播放时播放一个未知键,然后验证之后播放有效键时,旧曲目不再影响播放。测试确认旧曲目不会继续与新曲目同时播放后,即为完成。

由索引模型根据 Issue 内容生成。

描述

bug good first issue modules:music
Which package is this bug report for?

music

Issue description

play("typo") sets _current = null without pausing the track that is playing, then throws. The next play("b") calls null?.pause(), so the old track keeps playing alongside the new one.

Where:

Evidence:

if (!element) {
  this._current = null;
  throw new NfNotFound(key);
}

Suggested fix: Throw before touching _current (or pause it first). Add a unit test.

Found during a code audit of main at be72076.

Code sample

No response

Versions
  • Engine main at be72076
  • pnpm 12.0.0, Node.js 26, Bun 1.3.14
Issue priority

Medium (should be fixed soon)

I have tested this issue on a development release

be72076

主要语言
TypeScript
星标
7
派生
2
平均合并
1 天 17 小时
30 天内合并 PR
17

环境准备

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

NanoForge-dev/Engine 的其他 Issue

查看 NanoForge-dev/Engine 的全部 Issue

相似的 Issue

更多 TypeScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。