Skip to content

fix(producer): dither silent test WAVs below plugin compression guard#2043

Open
km061597 wants to merge 1 commit into
heygen-com:mainfrom
km061597:claude/compression-ratio-error-icr0ah
Open

fix(producer): dither silent test WAVs below plugin compression guard#2043
km061597 wants to merge 1 commit into
heygen-com:mainfrom
km061597:claude/compression-ratio-error-icr0ah

Conversation

@km061597

@km061597 km061597 commented Jul 8, 2026

Copy link
Copy Markdown

Two near-silent PCM fixtures compressed at 259:1 and 69:1, tripping the Claude Code plugin install guard ("Suspicious compression ratio ... max 50:1") and blocking the plugin from loading.

Fill their data sections with deterministic ±16 (-66 dBFS) dither — inaudible but incompressible — dropping both to ~2:1. Headers are untouched, so the fixtures still decode as the same-length audio tracks the render tests expect.

Extend the compression-ratio guard test to cover all three bundled WAVs so a pure-silence fixture can't regress the install payload again.

Claude-Session: https://claude.ai/code/session_01Q6Wcqhgc9MgjJVWH8U7WvQ

What

Brief description of the change.

Why

Why is this change needed?

How

How was this implemented? Any notable design decisions?

Test plan

How was this tested?

  • Unit tests added/updated
  • Manual testing performed
  • Documentation updated (if applicable)

Two near-silent PCM fixtures compressed at 259:1 and 69:1, tripping the
Claude Code plugin install guard ("Suspicious compression ratio ... max
50:1") and blocking the plugin from loading.

Fill their data sections with deterministic ±16 (-66 dBFS) dither —
inaudible but incompressible — dropping both to ~2:1. Headers are
untouched, so the fixtures still decode as the same-length audio tracks
the render tests expect.

Extend the compression-ratio guard test to cover all three bundled WAVs
so a pure-silence fixture can't regress the install payload again.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Q6Wcqhgc9MgjJVWH8U7WvQ
@miguel-heygen

Copy link
Copy Markdown
Collaborator

@km061597 why claude is the only author of the commit, include yourself too!

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.

3 participants