Commit ce3501e
committed
chore(moonshine): Address remaining PR #259 review nitpicks
This commit addresses the remaining low-priority review comments from PR #259.
Test Improvements:
- Adds `serial_test` to isolate environment variable tests in `moonshine_e2e.rs`.
- Adds a test for `verify_sample_rate()` in `moonshine.rs`.
- Adds a test for buffer overflow protection in `moonshine.rs`.
- Refactors `load_test_audio()` to validate sample rate and channels and removes duplicated test code.
Code Cleanup:
- Removes the unused `active_config` field from `MoonshinePlugin`.
Script Improvements:
- Uses `set -euo pipefail` in `install-moonshine-deps.sh` and `verify-stt-setup.sh`.1 parent ccd2ca5 commit ce3501e
File tree
17 files changed
+1225
-31
lines changed- crates
- app
- src/stt
- coldvox-stt
- src/plugins
- tests
- common
- coldvox-text-injection/src
- injectors
- scripts
17 files changed
+1225
-31
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
| 14 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
8 | 18 | | |
9 | 19 | | |
10 | 20 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
68 | 68 | | |
69 | 69 | | |
70 | 70 | | |
| 71 | + | |
71 | 72 | | |
72 | 73 | | |
73 | 74 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
579 | 579 | | |
580 | 580 | | |
581 | 581 | | |
| 582 | + | |
| 583 | + | |
| 584 | + | |
| 585 | + | |
| 586 | + | |
| 587 | + | |
| 588 | + | |
582 | 589 | | |
583 | 590 | | |
584 | 591 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
22 | 31 | | |
23 | 32 | | |
24 | 33 | | |
25 | 34 | | |
| 35 | + | |
26 | 36 | | |
27 | 37 | | |
28 | 38 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
11 | 14 | | |
12 | 15 | | |
13 | 16 | | |
| |||
27 | 30 | | |
28 | 31 | | |
29 | 32 | | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
0 commit comments