Skip to content

fix: package STT health-check audio - #187

Merged
zouyonghe merged 1 commit into
AstrBotDevs:mainfrom
zouyonghe:fix/issue-186-package-stt-audio
Oct 18, 2026
Merged

zouyonghe merged 1 commit into
AstrBotDevs:mainfrom
zouyonghe:fix/issue-186-package-stt-audio

Conversation

@zouyonghe

@zouyonghe zouyonghe commented Oct 17, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Include samples/stt_health_check.wav in required packaged backend sources.
  • Add a regression test to guard against omitting the STT health-check audio.

Fixes #186.

Verification

  • pnpm run test:prepare-resources (138 passed)
  • cargo test --manifest-path src-tauri/Cargo.toml --locked -- --test-threads=1 (162 passed)
  • make lint

Note: make test had one unrelated Rust test failure under parallel execution; the failing test passes in isolation and the serial full Rust suite passes.

Summary by Sourcery

Include the STT health-check audio in backend packages and protect its inclusion with a regression test.

Bug Fixes:

  • Ensure the STT health-check audio is included in packaged backend sources.

Tests:

  • Add a regression test verifying that the packaged backend includes the STT health-check audio.

Include the health-check WAV used by STTProvider.test() in packaged AstrBot sources. This prevents provider tests from failing with a missing file in desktop builds. Fixes AstrBotDevs#186.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Approved.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

@zouyonghe
zouyonghe merged commit 6cf79c3 into AstrBotDevs:main Oct 18, 2026
4 checks passed
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.

[Bug] 疑似打包时没有加入stt测试语音导致测试报错

1 participant