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.87%. Comparing base (9ee8595) to head (db00985).
✅ 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.90%   82.87%   -0.04%     
==========================================
  Files          79       79              
  Lines       10402    10411       +9     
  Branches     1197     1196       -1     
==========================================
+ Hits         8624     8628       +4     
- Misses       1577     1582       +5     
  Partials      201      201              

☔ View full report in Codecov by Harness.
📢 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?