Skip to content

Handle compound WXYC library format strings#49

Merged
jakebromberg merged 1 commit intomainfrom
fix/compound-library-formats
Mar 14, 2026
Merged

Handle compound WXYC library format strings#49
jakebromberg merged 1 commit intomainfrom
fix/compound-library-formats

Conversation

@jakebromberg
Copy link
Member

Summary

  • Parse compound library format strings (cd x 2, vinyl - 12", vinyl - LP x 3) that previously normalized to None, bypassing format filtering in verify_cache.py
  • Add _LIB_QUANTITY_SUFFIX_RE regex to strip quantity suffixes and vinyl subformat extraction logic to normalize_library_format
  • Add 12 parametrized test cases covering all production compound format patterns

Closes #48

Test plan

  • 12 new parametrized test cases pass for compound formats
  • All 51 format normalization tests pass
  • Full unit suite (483 tests) passes
  • ruff check and format clean

…_format

Strip quantity suffixes (x 2, x 3 box) and extract vinyl subformats (vinyl - 7", vinyl - LP x 2) so ~5,400 production library entries normalize to correct format categories instead of None.

Closes #48
@jakebromberg jakebromberg merged commit 290001a into main Mar 14, 2026
3 checks passed
@jakebromberg jakebromberg deleted the fix/compound-library-formats branch March 14, 2026 13:57
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.

Handle compound WXYC library format strings in normalize_library_format

1 participant