Python 3.13 better error messages breaks verisonizer

[andrewthetechie]

https://github.com/alexschimpf/fastapi-versionizer/blob/main/fastapi_versionizer/versionizer.py#L343-L347

Python 3.13 better error messages break this indexing.

Example TypeError now

"APIRouter.add_api_route() got an unexpected keyword argument 'response_field'. Did you mean 'response_model'?"

[andrewthetechie]

# Compile the regex once so it can be reused efficiently.
UNEXPECTED_KEYWORD_RE = re.compile(r"got an unexpected keyword argument '(\w+)'")

@staticmethod
def _add_route_to_router(
    route: Union[APIRoute, APIWebSocketRoute],
    router: APIRouter,
    version: Tuple[int, int]
) -> None:
    kwargs = dict(route.__dict__)

    deprecated_in_version = getattr(route.endpoint, '_deprecate_in_version', None)
    if deprecated_in_version is not None:
        deprecated_in_major, deprecated_in_minor = deprecated_in_version
        if (
            version[0] >= deprecated_in_major or
            (version[0] == deprecated_in_major and version[1] >= deprecated_in_minor)
        ):
            kwargs['deprecated'] = True

    # Retry loop to remove offending keyword arguments.
    for _ in range(10000):
        try:
            if isinstance(route, APIRoute):
                return router.add_api_route(**kwargs)
            elif isinstance(route, APIWebSocketRoute):
                return router.add_api_websocket_route(**kwargs)
        except TypeError as e:
            e_str = str(e)
            match = UNEXPECTED_KEYWORD_RE.search(e_str)
            if match:
                unexpected_key = match.group(1)
                kwargs.pop(unexpected_key, None)
            else:
                raise e

    raise Exception('Failed to add route')

Fixes the issue with regex

[OlegZv]

This was solved in #63. I will close this when the new version is released (very soon)

[RomeoAva]

Commenting to stay subscribed and receive updates on any changes related to this issue (hopefully released soon 🤞). Thank you for your work

[looztra]

any news on a release that includes the fix?

[OlegZv]

Hey @looztra! It's 99% there. Working with @alexschimpf on the solution

[alexschimpf]

@OlegZv @looztra Sorry for the massive delay, folks. Been super busy. I've updated the Github PAT, so hopefully the workflows start working again.

[OlegZv]

This should be closed now with the https://github.com/alexschimpf/fastapi-versionizer/releases/tag/v4.0.2 release.