fix: audio test hardening

.github/pull_request_template.md

@@ -4,8 +4,8 @@
   - Read docs/CONTRIBUTING.md and docs/RULES.md.
   - Keep the diff to the change. No drive-by renames, reformatting, or
     refactors of code the change does not touch.
-  - Run `bun run check`, `cargo check --manifest-path src-tauri/Cargo.toml`,
-    and `bun run format` — all must pass and leave the tree clean.
+  - Run `bun run test`, `bun run check`, and `bun run format` —
+    all must pass and leave the tree clean.
   - If you changed a `#[derive(TS)]` type in Rust, run `bun run generate` and
     commit the regenerated files in src/lib/modules/pipeline/generated/.
 -->
@@ -21,8 +21,8 @@ _Explain the design choice. What alternatives were considered and rejected? If t
 ## Checklist
 
 - [ ] Diff is limited to the change — no unrelated edits
+- [ ] `bun run test` passes
 - [ ] `bun run check` passes
-- [ ] `cargo check --manifest-path src-tauri/Cargo.toml` passes
 - [ ] `bun run format` leaves the tree clean
 - [ ] Generated TS types are committed with the Rust change (if any)
 - [ ] No new dependency without a reason in the PR description

.github/workflows/ci.yml

@@ -55,6 +55,9 @@ jobs:
       - name: Install JS deps
         run: bun install --frozen-lockfile
 
+      - name: Frontend tests
+        run: bun run test:frontend
+
       # Each OS backend needs its native headers to compile. Linux installs
       # PipeWire and the GTK stack explicitly; macOS and Windows runners ship
       # their SDKs (CoreAudio / WASAPI + MSVC) preinstalled.
@@ -93,8 +96,6 @@ jobs:
           fi
           if [ "$RUNNER_OS" = "Windows" ]; then
             cargo test --no-run
-          elif [ "$RUNNER_OS" = "Linux" ]; then
-            cargo test -- --skip audio::playback::tests::playback_pulls_samples
           else
             cargo test
           fi

AGENTS.md

@@ -33,7 +33,7 @@ Without these, one chunk takes ~16 ms and the worker stalls.
 ## When in doubt
 
 - Read the current code, not earlier explanations.
-- RT path change → `cargo check`.
+- RT path change → `bun run check`.
 - Svelte change → `bun run check`.
 - New node or shared UI component → reuse existing Storybook components first, add its stories under `src/stories/` in the same commit (see [docs/UI.md](docs/UI.md)).
 - Rust `#[derive(TS)]` change → `bun run generate`, commit the generated files with the Rust change.

CONTRIBUTING.md

@@ -57,12 +57,12 @@ If a platform genuinely cannot support what you are adding, return a real error
 ## Before you open the PR
 
 ```bash
-bun run check                                  # svelte-check + tsc
-cargo check --manifest-path src-tauri/Cargo.toml
+bun run test                                   # frontend + Rust tests in parallel
+bun run check                                  # frontend + Rust checks in parallel
 bun run format                                 # Prettier + rustfmt
 ```
 
-Both checks must pass, and `format` must leave the tree clean. A new node or shared component ships with its Storybook stories in the same PR; see [docs/UI.md](docs/UI.md). If you changed a `#[derive(TS)]` type in Rust, run `bun run generate` and commit the regenerated files in `src/lib/modules/pipeline/generated/` together with the Rust change — never by hand, and never on their own.
+All checks must pass, and `format` must leave the tree clean. A new node or shared component ships with its Storybook stories in the same PR; see [docs/UI.md](docs/UI.md). If you changed a `#[derive(TS)]` type in Rust, run `bun run generate` and commit the regenerated files in `src/lib/modules/pipeline/generated/` together with the Rust change — never by hand, and never on their own.
 
 Anything persisted to disk — pipeline JSON, `virtual-devices.json`, the macOS driver plist — has to keep loading for people upgrading. Add a `#[serde(default)]` or a versioned migration; do not silently change the meaning of an existing field.
 

docs/DEVELOPMENT.md

@@ -38,10 +38,10 @@ bun run tauri dev
 ## Useful commands
 
 ```bash
-bun run check                          # svelte-check + tsc
+bun run test                           # frontend + Rust tests in parallel
+bun run check                          # frontend + Rust checks in parallel
 bun run generate                       # regenerate TypeScript types from Rust (ts-rs)
 bun run storybook                      # Storybook on :6006, nodes and shared components with mocked engine
-cargo check --manifest-path src-tauri/Cargo.toml
 bun run tauri build --bundles app      # local .app build (macOS)
 bun run tauri build --bundles appimage # local AppImage build (Linux)
 bun run tauri build --bundles nsis     # local installer build (Windows)

docs/RULES.md

@@ -45,7 +45,7 @@ type(scope): subject
 ## When in doubt
 
 - Read the current code, not earlier explanations.
-- RT path change → `cargo check`.
+- RT path change → `bun run check`.
 - Svelte change → `bun run check`.
 - New node or shared component → add its stories in the same commit, see [UI.md](UI.md).
 - Rust `#[derive(TS)]` change → `bun run generate`, commit the generated files with the Rust change.

package.json

@@ -7,7 +7,10 @@
 		"dev": "vite dev",
 		"build": "vite build",
 		"preview": "vite preview",
-		"check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json",
+		"test": "bun scripts/test-all.ts",
+		"test:frontend": "bun test tests",
+		"check": "bun scripts/check-all.ts",
+		"check:frontend": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json",
 		"check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch",
 		"tauri": "tauri",
 		"format": "prettier --write \"**/*.{ts,tsx,md,svelte,json}\" && cargo fmt --manifest-path src-tauri/Cargo.toml",

scripts/check-all.ts

@@ -0,0 +1,29 @@
+const tasks = [
+	{
+		name: 'frontend',
+		command: ['bun', 'run', 'check:frontend']
+	},
+	{
+		name: 'backend',
+		command: ['cargo', 'check', '--manifest-path', 'src-tauri/Cargo.toml']
+	}
+] as const;
+
+const children = tasks.map((task) => ({
+	...task,
+	process: Bun.spawn(task.command, {
+		stdin: 'inherit',
+		stdout: 'inherit',
+		stderr: 'inherit'
+	})
+}));
+
+const results = await Promise.all(children.map(async ({ name, process }) => ({ name, exitCode: await process.exited })));
+const failed = results.filter(({ exitCode }) => exitCode !== 0);
+
+if (failed.length > 0) {
+	for (const { name, exitCode } of failed) {
+		console.error(`${name} check failed with exit code ${exitCode}`);
+	}
+	process.exit(1);
+}

scripts/test-all.ts

@@ -0,0 +1,29 @@
+const tasks = [
+	{
+		name: 'frontend',
+		command: ['bun', 'run', 'test:frontend']
+	},
+	{
+		name: 'backend',
+		command: ['cargo', 'test', '--manifest-path', 'src-tauri/Cargo.toml']
+	}
+] as const;
+
+const children = tasks.map((task) => ({
+	...task,
+	process: Bun.spawn(task.command, {
+		stdin: 'inherit',
+		stdout: 'inherit',
+		stderr: 'inherit'
+	})
+}));
+
+const results = await Promise.all(children.map(async ({ name, process }) => ({ name, exitCode: await process.exited })));
+const failed = results.filter(({ exitCode }) => exitCode !== 0);
+
+if (failed.length > 0) {
+	for (const { name, exitCode } of failed) {
+		console.error(`${name} tests failed with exit code ${exitCode}`);
+	}
+	process.exit(1);
+}

src-tauri/Cargo.toml

@@ -59,6 +59,10 @@ opt-level = 3
 [profile.dev.package.deep_filter]
 opt-level = 3
 
+[dev-dependencies]
+proptest = "1"
+tauri = { version = "2", features = ["test"] }
+
 [dependencies]
 tauri = { version = "2", features = ["unstable"] }
 tauri-plugin-opener = "2"

src-tauri/proptest-regressions/audio/effects/compressor.txt

@@ -0,0 +1,7 @@
+# Seeds for failure cases proptest has generated in the past. It is
+# automatically read and these particular cases re-run before any
+# novel cases are generated.
+#
+# It is recommended to check this file in to source control so that
+# everyone who runs the test benefits from these saved cases.
+cc 23a029b70867959f62fcbeb4c4f6e04ea8fb2e8dbeaef82c1212fd1deaec5e00 # shrinks to input = [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0], poison_pos = 0, makeup = 0.0