Revert "Revert "Resolving TypeError, during version unpacking ""

[auvipy]

Reverts #2225

[codecov]

Codecov Report

❌ Patch coverage is 90.47619% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.85%. Comparing base (4c9fb4b) to head (4ce1c95).
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
kombu/utils/text.py	90.47%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2333      +/-   ##
==========================================
- Coverage   81.11%   80.85%   -0.26%     
==========================================
  Files          77       77              
  Lines        9705     9689      -16     
  Branches     1195     1194       -1     
==========================================
- Hits         7872     7834      -38     
- Misses       1628     1636       +8     
- Partials      205      219      +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Copilot]

Pull Request Overview

This PR reverts the previous revert of version unpacking logic (#2225), reintroducing an updated regex-based parsing approach and extending test coverage.

• Added new test cases for complex version strings in test_utils.py.
• Refactored version_string_as_tuple to use a fullmatch regex and updated _unpack_version signature.
• Cleaned up formatting in escape_regex, fmatch_iter, and fmatch_best.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
t/unit/utils/test_utils.py	Reformatted @pytest.mark.parametrize and added tests for additional version patterns
kombu/utils/text.py	Introduced regex parsing in version_string_as_tuple, updated helper signatures, and adjusted utility formatting

[Copilot]

Using sorted() to find the best match sorts the entire iterable, which is inefficient if only the max is needed. Consider using max(fmatch_iter(...), key=lambda x: x[0], default=(None, None))[1] to avoid constructing a full list.

[auvipy]

and this @couzhei

[Copilot]

The _splitmicro helper is no longer referenced by version_string_as_tuple and appears unused. Consider removing it to eliminate dead code.

[auvipy]

@couzhei what is your take on this?