audiometa-python 1.3.0__tar.gz → 1.3.3__tar.gz

{audiometa_python-1.3.0/audiometa_python.egg-info → audiometa_python-1.3.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: audiometa-python
-Version: 1.3.0
+Version: 1.3.3
 Summary: A comprehensive Python library for reading and writing audio metadata across multiple formats
 Author: AudioMeta Python Contributors
 Author-email: Andreas Garcia <garcia.andreas.1991@gmail.com>
@@ -55,6 +55,8 @@ Dynamic: license-file
 
 A powerful, unified Python library for reading and writing audio metadata across multiple formats. AudioMeta supports MP3, FLAC, and WAV audio files, working seamlessly with ID3v1, ID3v2, Vorbis, and RIFF metadata formats through a single, consistent API.
 
+**Try it**: A demo web app is available at [audiometa.themusictree.org](https://audiometa.themusictree.org).
+
 **Author**: [Andreas Garcia](https://github.com/BehindTheMusicTree)
 
 ## ⭐ Show Your Support

{audiometa_python-1.3.0 → audiometa_python-1.3.3}/README.md

@@ -13,6 +13,8 @@
 
 A powerful, unified Python library for reading and writing audio metadata across multiple formats. AudioMeta supports MP3, FLAC, and WAV audio files, working seamlessly with ID3v1, ID3v2, Vorbis, and RIFF metadata formats through a single, consistent API.
 
+**Try it**: A demo web app is available at [audiometa.themusictree.org](https://audiometa.themusictree.org).
+
 **Author**: [Andreas Garcia](https://github.com/BehindTheMusicTree)
 
 ## ⭐ Show Your Support

{audiometa_python-1.3.0 → audiometa_python-1.3.3}/audiometa/manager/_MetadataManager.py

@@ -5,6 +5,7 @@ from typing import TYPE_CHECKING, TypeVar, Union, cast
 from mutagen._file import FileType as MutagenMetadata
 
 from audiometa.exceptions import InvalidMetadataFieldFormatError, InvalidMetadataFieldTypeError
+from audiometa.utils.disc_number_read import parse_disc_number_from_combined_str
 from audiometa.utils.unified_metadata_key import UnifiedMetadataKey
 
 if TYPE_CHECKING:
@@ -698,6 +699,10 @@ class _MetadataManager:
                 return track_str
             return None
 
+        # DISCNUMBER / TPOS-style "n", "n/m", "n-m" (formats using the base read path)
+        if unified_metadata_key == UnifiedMetadataKey.DISC_NUMBER:
+            return parse_disc_number_from_combined_str(str(value[0]))
+
         from typing import get_args, get_origin
 
         origin = get_origin(unified_metadata_key_optional_type)

{audiometa_python-1.3.0 → audiometa_python-1.3.3}/audiometa/manager/_rating_supporting/id3v2/_Id3v2Manager.py

@@ -30,6 +30,7 @@ from mutagen.id3._frames import (
 )
 from mutagen.id3._util import ID3NoHeaderError
 
+from audiometa.utils.disc_number_read import parse_disc_number_from_combined_str, parse_disc_total_from_combined_str
 from audiometa.utils.raw_metadata_sanitizer import sanitize_id3v2_raw_info
 from audiometa.utils.unified_metadata_key import UnifiedMetadataKey
 
@@ -415,12 +416,7 @@ class _Id3v2Manager(_RatingSupportingMetadataManager):
             if tpos_value is None or len(tpos_value) == 0:
                 return None
             tpos_str = str(tpos_value[0])
-            import re
-
-            match = re.match(r"^(\d+)(?:/(\d+))?$", tpos_str)
-            if match:
-                return int(match.group(1))
-            return None
+            return parse_disc_number_from_combined_str(tpos_str)
         if unified_metadata_key == UnifiedMetadataKey.DISC_TOTAL:
             tpos_key = self.Id3TextFrame.DISC_NUMBER
             if tpos_key not in raw_clean_metadata:
@@ -429,12 +425,7 @@ class _Id3v2Manager(_RatingSupportingMetadataManager):
             if tpos_value is None or len(tpos_value) == 0:
                 return None
             tpos_str = str(tpos_value[0])
-            import re
-
-            match = re.match(r"^(\d+)/(\d+)$", tpos_str)
-            if match:
-                return int(match.group(2))
-            return None
+            return parse_disc_total_from_combined_str(tpos_str)
         msg = f"Metadata key not handled: {unified_metadata_key}"
         raise MetadataFieldNotSupportedByMetadataFormatError(msg)
 

{audiometa_python-1.3.0 → audiometa_python-1.3.3}/audiometa/manager/_rating_supporting/vorbis/_VorbisManager.py

@@ -6,6 +6,11 @@ from typing import TYPE_CHECKING, Any, TypeVar, cast
 if TYPE_CHECKING:
     from ...._audio_file import _AudioFile
 from ....exceptions import FileCorruptedError, InvalidRatingValueError, MetadataFieldNotSupportedByMetadataFormatError
+from ....utils.disc_number_read import (
+    parse_disc_number_from_combined_str,
+    parse_disc_total_from_combined_str,
+    parse_explicit_non_negative_disctotal,
+)
 from ....utils.rating_profiles import RatingWriteProfile
 from ....utils.raw_metadata_sanitizer import sanitize_vorbis_raw_info
 from ....utils.tool_path_resolver import get_tool_path
@@ -260,6 +265,49 @@ class _VorbisManager(_RatingSupportingMetadataManager):
         # Cast to RawMetadataDict since RawMetadataKey is str, Enum and string keys work
         self.raw_clean_metadata_uppercase_keys = cast(RawMetadataDict, result_dict)
 
+    def get_unified_metadata_field(self, unified_metadata_key: UnifiedMetadataKey) -> UnifiedMetadataValue:
+        if unified_metadata_key not in (
+            UnifiedMetadataKey.DISC_NUMBER,
+            UnifiedMetadataKey.DISC_TOTAL,
+        ):
+            return super().get_unified_metadata_field(unified_metadata_key)
+
+        if self.raw_clean_metadata_uppercase_keys is None:
+            self._extract_raw_clean_metadata_uppercase_keys_from_file()
+        keys = self.raw_clean_metadata_uppercase_keys
+        if keys is None:
+            return None
+
+        disc_number_key = self.VorbisKey.DISC_NUMBER
+        disc_total_key = self.VorbisKey.DISC_TOTAL
+
+        def _first_string(raw_key: RawMetadataKey) -> str | None:
+            if raw_key not in keys:
+                return None
+            vals = keys[raw_key]
+            if not vals or len(vals) == 0:
+                return None
+            v = vals[0]
+            if v is None or v == "":
+                return None
+            return str(v)
+
+        d = _first_string(disc_number_key)
+        t = _first_string(disc_total_key)
+
+        if unified_metadata_key == UnifiedMetadataKey.DISC_NUMBER:
+            if d is None:
+                return None
+            return parse_disc_number_from_combined_str(d)
+
+        if t is not None:
+            explicit = parse_explicit_non_negative_disctotal(t)
+            if explicit is not None:
+                return explicit
+        if d is None:
+            return None
+        return parse_disc_total_from_combined_str(d)
+
     def _get_raw_rating_by_traktor_or_not(self, raw_clean_metadata: RawMetadataDict) -> tuple[int | None, bool]:
         if self.VorbisKey.RATING in raw_clean_metadata:
             rating_list = raw_clean_metadata[self.VorbisKey.RATING]

audiometa_python-1.3.3/audiometa/test/tests/integration/metadata_field/disc_number/test_reading.py

@@ -0,0 +1,191 @@
+import pytest
+
+from audiometa import get_unified_metadata_field
+from audiometa.test.helpers.id3v2.id3v2_metadata_setter import ID3v2MetadataSetter
+from audiometa.test.helpers.temp_file_with_metadata import temp_file_with_metadata
+from audiometa.test.helpers.vorbis.vorbis_metadata_setter import VorbisMetadataSetter
+from audiometa.utils.unified_metadata_key import UnifiedMetadataKey
+
+
+@pytest.mark.integration
+class TestDiscNumberReading:
+    def test_id3v2_with_total(self):
+        with temp_file_with_metadata({}, "mp3") as test_file:
+            ID3v2MetadataSetter.set_metadata(test_file, {"disc_number": "1/2"})
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 1
+            assert disc_total == 2
+
+    def test_id3v2_without_total(self):
+        with temp_file_with_metadata({}, "mp3") as test_file:
+            ID3v2MetadataSetter.set_metadata(test_file, {"disc_number": "1"})
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 1
+            assert disc_total is None
+
+    def test_id3v2_max_value(self):
+        with temp_file_with_metadata({}, "mp3") as test_file:
+            ID3v2MetadataSetter.set_metadata(test_file, {"disc_number": "99/99"})
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 99
+            assert disc_total == 99
+
+    def test_id3v2_hyphen_separator_read(self):
+        with temp_file_with_metadata({}, "mp3") as test_file:
+            ID3v2MetadataSetter.set_metadata(test_file, {"disc_number": "1-2"})
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 1
+            assert disc_total == 2
+
+    def test_id3v2_invalid_tpos_multiple_slashes_returns_none(self):
+        with temp_file_with_metadata({}, "mp3") as test_file:
+            ID3v2MetadataSetter.set_metadata(test_file, {"disc_number": "1/2/3"})
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number is None
+            assert disc_total is None
+
+    def test_id3v2_read_tpos_above_255_not_clamped(self):
+        with temp_file_with_metadata({}, "mp3") as test_file:
+            ID3v2MetadataSetter.set_metadata(test_file, {"disc_number": "256/300"})
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 256
+            assert disc_total == 300
+
+    def test_vorbis_with_total(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            VorbisMetadataSetter.set_metadata(test_file, {"disc_number": "1", "disc_total": "2"})
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 1
+            assert disc_total == 2
+
+    def test_vorbis_without_total(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            VorbisMetadataSetter.set_metadata(test_file, {"disc_number": "2"})
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 2
+            assert disc_total is None
+
+    def test_vorbis_disc_number_combined_slash_form(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            VorbisMetadataSetter.set_tag(test_file, "DISCNUMBER", "3/5")
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 3
+            assert disc_total == 5
+
+    def test_vorbis_discnumber_combined_slash_via_set_metadata(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            VorbisMetadataSetter.set_metadata(test_file, {"disc_number": "1/2"})
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 1
+            assert disc_total == 2
+
+    def test_vorbis_disc_number_hyphen_form(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            VorbisMetadataSetter.set_tag(test_file, "DISCNUMBER", "1-2")
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 1
+            assert disc_total == 2
+
+    def test_vorbis_disctotal_overrides_embedded_total(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            VorbisMetadataSetter.set_tag(test_file, "DISCNUMBER", "1/3")
+            VorbisMetadataSetter.set_tag(test_file, "DISCTOTAL", "2")
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 1
+            assert disc_total == 2
+
+    def test_vorbis_discnumber_slash_with_matching_disctotal(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            VorbisMetadataSetter.set_metadata(test_file, {"disc_number": "1/2", "disc_total": "2"})
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 1
+            assert disc_total == 2
+
+    def test_vorbis_invalid_disctotal_falls_back_to_discnumber_slash(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            VorbisMetadataSetter.set_tag(test_file, "DISCNUMBER", "1/2")
+            VorbisMetadataSetter.set_tag(test_file, "DISCTOTAL", "bogus")
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 1
+            assert disc_total == 2
+
+    def test_vorbis_negative_disctotal_falls_back_to_discnumber_slash(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            VorbisMetadataSetter.set_tag(test_file, "DISCNUMBER", "1/2")
+            VorbisMetadataSetter.set_tag(test_file, "DISCTOTAL", "-1")
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number == 1
+            assert disc_total == 2
+
+    def test_vorbis_invalid_combined_discnumber(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            VorbisMetadataSetter.set_tag(test_file, "DISCNUMBER", "1/2/3")
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number is None
+            assert disc_total is None
+
+    def test_vorbis_disctotal_only(self):
+        with temp_file_with_metadata({}, "flac") as test_file:
+            VorbisMetadataSetter.set_tag(test_file, "DISCTOTAL", "2")
+            disc_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_NUMBER)
+            disc_total = get_unified_metadata_field(test_file, UnifiedMetadataKey.DISC_TOTAL)
+            assert disc_number is None
+            assert disc_total == 2
+
+    def test_id3v1_not_supported(self):
+        from audiometa.exceptions import MetadataFieldNotSupportedByMetadataFormatError
+        from audiometa.utils.metadata_format import MetadataFormat
+
+        with temp_file_with_metadata({}, "id3v1") as test_file:
+            with pytest.raises(
+                MetadataFieldNotSupportedByMetadataFormatError,
+                match="UnifiedMetadataKey.DISC_NUMBER metadata not supported by ID3v1 format",
+            ):
+                get_unified_metadata_field(
+                    test_file, UnifiedMetadataKey.DISC_NUMBER, metadata_format=MetadataFormat.ID3V1
+                )
+
+            with pytest.raises(
+                MetadataFieldNotSupportedByMetadataFormatError,
+                match="UnifiedMetadataKey.DISC_TOTAL metadata not supported by ID3v1 format",
+            ):
+                get_unified_metadata_field(
+                    test_file, UnifiedMetadataKey.DISC_TOTAL, metadata_format=MetadataFormat.ID3V1
+                )
+
+    def test_riff_not_supported(self):
+        from audiometa.exceptions import MetadataFieldNotSupportedByMetadataFormatError
+        from audiometa.utils.metadata_format import MetadataFormat
+
+        with temp_file_with_metadata({}, "wav") as test_file:
+            with pytest.raises(
+                MetadataFieldNotSupportedByMetadataFormatError,
+                match="UnifiedMetadataKey.DISC_NUMBER metadata not supported by RIFF format",
+            ):
+                get_unified_metadata_field(
+                    test_file, UnifiedMetadataKey.DISC_NUMBER, metadata_format=MetadataFormat.RIFF
+                )
+
+            with pytest.raises(
+                MetadataFieldNotSupportedByMetadataFormatError,
+                match="UnifiedMetadataKey.DISC_TOTAL metadata not supported by RIFF format",
+            ):
+                get_unified_metadata_field(
+                    test_file, UnifiedMetadataKey.DISC_TOTAL, metadata_format=MetadataFormat.RIFF
+                )

{audiometa_python-1.3.0 → audiometa_python-1.3.3}/audiometa/test/tests/integration/metadata_field/track_number/reading/test_edge_cases.py

@@ -7,37 +7,50 @@ from audiometa.utils.unified_metadata_key import UnifiedMetadataKey
 
 @pytest.mark.integration
 class TestTrackNumberReadingEdgeCases:
-    def test_trailing_slash(self):
-        with temp_file_with_metadata({"track_number": "5/"}, "mp3") as test_file:
+    @pytest.mark.parametrize("fmt", ["mp3", "flac"])
+    def test_trailing_slash(self, fmt):
+        with temp_file_with_metadata({"track_number": "5/"}, fmt) as test_file:
             track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
             assert track_number == "5/"
 
-    def test_leading_slash_no_track(self):
-        with temp_file_with_metadata({"track_number": "/12"}, "mp3") as test_file:
+    @pytest.mark.parametrize("fmt", ["mp3", "flac"])
+    def test_leading_slash_no_track(self, fmt):
+        with temp_file_with_metadata({"track_number": "/12"}, fmt) as test_file:
             track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
             assert track_number is None
 
-    def test_non_numeric_values(self):
-        with temp_file_with_metadata({"track_number": "abc/def"}, "mp3") as test_file:
+    @pytest.mark.parametrize("fmt", ["mp3", "flac"])
+    def test_non_numeric_values(self, fmt):
+        with temp_file_with_metadata({"track_number": "abc/def"}, fmt) as test_file:
             track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
             assert track_number is None
 
-    def test_empty_string(self):
-        with temp_file_with_metadata({"track_number": ""}, "mp3") as test_file:
+    @pytest.mark.parametrize(
+        ("fmt", "expected"),
+        [
+            ("mp3", None),
+            ("flac", ""),
+        ],
+    )
+    def test_empty_string(self, fmt, expected):
+        with temp_file_with_metadata({"track_number": ""}, fmt) as test_file:
             track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
-            assert track_number is None
+            assert track_number == expected
 
-    def test_multiple_slashes(self):
-        with temp_file_with_metadata({"track_number": "5/12/15"}, "mp3") as test_file:
+    @pytest.mark.parametrize("fmt", ["mp3", "flac"])
+    def test_multiple_slashes(self, fmt):
+        with temp_file_with_metadata({"track_number": "5/12/15"}, fmt) as test_file:
             track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
             assert track_number is None
 
-    def test_different_separator(self):
-        with temp_file_with_metadata({"track_number": "5-12"}, "mp3") as test_file:
+    @pytest.mark.parametrize("fmt", ["mp3", "flac"])
+    def test_different_separator(self, fmt):
+        with temp_file_with_metadata({"track_number": "5-12"}, fmt) as test_file:
             track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
             assert track_number == "5-12"
 
-    def test_leading_zeros_preserved(self):
-        with temp_file_with_metadata({"track_number": "01"}, "mp3") as test_file:
+    @pytest.mark.parametrize("fmt", ["mp3", "flac"])
+    def test_leading_zeros_preserved(self, fmt):
+        with temp_file_with_metadata({"track_number": "01"}, fmt) as test_file:
             track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
             assert track_number == "01"

audiometa_python-1.3.3/audiometa/test/tests/integration/metadata_field/track_number/reading/test_metadata_format.py

@@ -0,0 +1,76 @@
+import pytest
+
+from audiometa import get_unified_metadata_field, update_metadata
+from audiometa.test.helpers.id3v1.id3v1_metadata_getter import ID3v1MetadataGetter
+from audiometa.test.helpers.temp_file_with_metadata import temp_file_with_metadata
+from audiometa.utils.metadata_format import MetadataFormat
+from audiometa.utils.unified_metadata_key import UnifiedMetadataKey
+
+
+@pytest.mark.integration
+class TestTrackNumberReading:
+    def test_id3v1_1(self):
+        with temp_file_with_metadata({"title": "Test Song", "track": "99"}, "id3v1") as test_file:
+            raw_metadata = ID3v1MetadataGetter.get_raw_metadata(test_file)
+            assert raw_metadata.get("track") == 99
+
+            track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
+            assert track_number == "99"
+
+    def test_id3v2(self):
+        with temp_file_with_metadata({"title": "Test Song", "track_number": "99/99"}, "mp3") as test_file:
+            track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
+            assert track_number == "99/99"
+
+    def test_vorbis(self):
+        with temp_file_with_metadata({"title": "Test Song", "track_number": "99"}, "flac") as test_file:
+            track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
+            assert track_number == "99"
+
+    @pytest.mark.parametrize(
+        ("raw", "expected"),
+        [
+            ("5/12", "5/12"),
+            ("4/11", "4/11"),
+            ("5-12", "5-12"),
+        ],
+    )
+    def test_vorbis_track_number_combined_or_hyphen(self, raw, expected):
+        with temp_file_with_metadata({"title": "Test Song", "track_number": raw}, "flac") as test_file:
+            track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
+            assert track_number == expected
+
+    def test_riff(self):
+        with temp_file_with_metadata({"title": "Test Song", "track_number": None}, "wav") as test_file:
+            track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
+            assert track_number is None
+
+    @pytest.mark.parametrize(
+        ("written", "expected"),
+        [
+            ("5/12", "5/12"),
+            ("4/11", "4/11"),
+            ("5-12", "5-12"),
+        ],
+    )
+    def test_riff_track_number_reads_like_shared_rule_after_unified_write(self, written, expected):
+        with temp_file_with_metadata({}, "wav") as test_file:
+            update_metadata(
+                test_file,
+                {UnifiedMetadataKey.TRACK_NUMBER: written},
+                metadata_format=MetadataFormat.RIFF,
+            )
+            track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
+            assert track_number == expected
+
+    @pytest.mark.parametrize(
+        ("raw", "expected"),
+        [
+            ("5/12", "5/12"),
+            ("5-12", "5-12"),
+        ],
+    )
+    def test_id3v2_track_number_combined_or_hyphen(self, raw, expected):
+        with temp_file_with_metadata({"title": "Test Song", "track_number": raw}, "mp3") as test_file:
+            track_number = get_unified_metadata_field(test_file, UnifiedMetadataKey.TRACK_NUMBER)
+            assert track_number == expected

audiometa_python-1.3.3/audiometa/test/tests/unit/utils/test_disc_number_read.py

@@ -0,0 +1,66 @@
+import pytest
+
+from audiometa.utils.disc_number_read import (
+    parse_disc_number_from_combined_str,
+    parse_disc_total_from_combined_str,
+    parse_explicit_non_negative_disctotal,
+)
+
+
+@pytest.mark.unit
+class TestDiscNumberReadParsing:
+    @pytest.mark.parametrize(
+        ("s", "expected"),
+        [
+            ("1", 1),
+            ("01", 1),
+            ("1/2", 1),
+            ("1-2", 1),
+            ("99/100", 99),
+        ],
+    )
+    def test_parse_disc_number_ok(self, s, expected):
+        assert parse_disc_number_from_combined_str(s) == expected
+
+    @pytest.mark.parametrize(
+        "s",
+        ["", "1/2/3", "x", "1-", "-1", "1/"],
+    )
+    def test_parse_disc_number_none(self, s):
+        assert parse_disc_number_from_combined_str(s) is None
+
+    @pytest.mark.parametrize(
+        ("s", "expected"),
+        [
+            ("1/2", 2),
+            ("1-2", 2),
+            ("0/5", 5),
+        ],
+    )
+    def test_parse_disc_total_from_combined_ok(self, s, expected):
+        assert parse_disc_total_from_combined_str(s) == expected
+
+    @pytest.mark.parametrize(
+        "s",
+        ["1", "1/2/3", "", "x"],
+    )
+    def test_parse_disc_total_from_combined_none(self, s):
+        assert parse_disc_total_from_combined_str(s) is None
+
+    @pytest.mark.parametrize(
+        ("s", "expected"),
+        [
+            ("0", 0),
+            ("2", 2),
+            ("42", 42),
+        ],
+    )
+    def test_parse_explicit_disctotal_ok(self, s, expected):
+        assert parse_explicit_non_negative_disctotal(s) == expected
+
+    @pytest.mark.parametrize(
+        "s",
+        ["", "bogus", "-1", "1.5"],
+    )
+    def test_parse_explicit_disctotal_none(self, s):
+        assert parse_explicit_non_negative_disctotal(s) is None

audiometa_python-1.3.3/audiometa/utils/disc_number_read.py

@@ -0,0 +1,30 @@
+"""Parse combined disc index/total strings (ID3v2 TPOS-style) for unified metadata read."""
+
+import re
+
+_DISC_NUMBER_RE = re.compile(r"^(\d+)(?:[-/](\d+))?$")
+_DISC_TOTAL_FROM_COMBINED_RE = re.compile(r"^(\d+)[-/](\d+)$")
+
+
+def parse_disc_number_from_combined_str(value: str) -> int | None:
+    m = _DISC_NUMBER_RE.match(value)
+    if m:
+        return int(m.group(1))
+    return None
+
+
+def parse_disc_total_from_combined_str(value: str) -> int | None:
+    m = _DISC_TOTAL_FROM_COMBINED_RE.match(value)
+    if m:
+        return int(m.group(2))
+    return None
+
+
+def parse_explicit_non_negative_disctotal(value: str) -> int | None:
+    try:
+        n = int(value)
+    except (ValueError, TypeError):
+        return None
+    if n < 0:
+        return None
+    return n

{audiometa_python-1.3.0 → audiometa_python-1.3.3/audiometa_python.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: audiometa-python
-Version: 1.3.0
+Version: 1.3.3
 Summary: A comprehensive Python library for reading and writing audio metadata across multiple formats
 Author: AudioMeta Python Contributors
 Author-email: Andreas Garcia <garcia.andreas.1991@gmail.com>
@@ -55,6 +55,8 @@ Dynamic: license-file
 
 A powerful, unified Python library for reading and writing audio metadata across multiple formats. AudioMeta supports MP3, FLAC, and WAV audio files, working seamlessly with ID3v1, ID3v2, Vorbis, and RIFF metadata formats through a single, consistent API.
 
+**Try it**: A demo web app is available at [audiometa.themusictree.org](https://audiometa.themusictree.org).
+
 **Author**: [Andreas Garcia](https://github.com/BehindTheMusicTree)
 
 ## ⭐ Show Your Support

{audiometa_python-1.3.0 → audiometa_python-1.3.3}/audiometa_python.egg-info/SOURCES.txt

@@ -389,8 +389,10 @@ audiometa/test/tests/unit/metadata_managers/metadata_field/release_date/test_dat
 audiometa/test/tests/unit/metadata_managers/metadata_field/track_number/__init__.py
 audiometa/test/tests/unit/metadata_managers/metadata_field/track_number/test_track_number_validation.py
 audiometa/test/tests/unit/utils/__init__.py
+audiometa/test/tests/unit/utils/test_disc_number_read.py
 audiometa/test/tests/unit/utils/test_raw_metadata_sanitizer.py
 audiometa/utils/__init__.py
+audiometa/utils/disc_number_read.py
 audiometa/utils/flac_md5_state.py
 audiometa/utils/id3v1_genre_code_map.py
 audiometa/utils/metadata_format.py

{audiometa_python-1.3.0 → audiometa_python-1.3.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "audiometa-python"
-version = "1.3.0"
+version = "1.3.3"
 description = "A comprehensive Python library for reading and writing audio metadata across multiple formats"
 readme = "README.md"
 requires-python = ">=3.12"
@@ -107,6 +107,8 @@ ignore = ["N999", "PLR0911", "PLR0912", "PLR0913", "PLR0914", "PLR0915", "TRY301
 # T201 (print) is acceptable in demo scripts - they are CLI/demo tools, not library code
 "scripts/run_demo_tape.py" = ["T201"]
 "scripts/demo_repl.py" = ["T201"]
+# Release script: CLI tool; relax exception/literal and magic-number rules for clarity
+"scripts/prepare_release.py" = ["TRY003", "EM101", "EM102", "PLR2004", "DTZ011", "RUF005", "T201"]
 
 [tool.mypy]
 python_version = "3.12"