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 82.41%. Comparing base (9022fcc) to head (94b7c94).
✅ 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   82.44%   82.41%   -0.04%     
==========================================
  Files          79       79              
  Lines       10142    10151       +9     
  Branches     1165     1164       -1     
==========================================
+ Hits         8362     8366       +4     
- Misses       1579     1584       +5     
  Partials      201      201              

☔ 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?