Thanks for the report @kesmik. Seems like we're not fully addressing all the variants of action that can be passed. I'll take a quick look at this and re-release after merge if necessary.

@JohnVillalovos @igorp-collabora this was my fear with typing overloads, I think we have to provide all the variations of preceding arguments (str or None) as well as the action which in this case is Callable[[Buffer], int] for f.write.

There's probably more elegant ways to overload hopefully at this point, as long as it's an option for us in the versions we support.

Edit: sorry ok it's easier than this, I think. WIll have a PR soon.

hi, @nejch thanks for update if you guys are busy, I can create a pull request, I am just not sure what is the proper way to fix it. I would just use in overload: [Callable[..., Any]], but not sure if it is ok.

action= can take any callable that accepts a single bytes argument. The return is not used at all and can be set to Any.

I need to test if Callable[[bytes], Any] would accept the f.write method or if a protocol would be needed.

action= can take any callable that accepts a single bytes argument. The return is not used at all and can be set to Any.

I need to test if Callable[[bytes], int] would accept the f.write method or if a protocol would be needed.

Thanks @igorp-collabora I was also just looking at this and looks like that would work, though maybe the new Buffer protocol from 3.12 would be better in the long run. I was wondering why we had that specific signature from #2211. I've added this in #3106 for now.

@nejch @JohnVillalovos using Callable[[bytes], Any] fixes this issue.

I created this typing file:

from gitlab import Gitlab
import tempfile


def test(api: Gitlab, prj_id: int, suffix: str, path: str, ref: str) -> None:
    with tempfile.NamedTemporaryFile(delete=False, suffix=suffix) as f:
        project = api.projects.get(prj_id, lazy=True)
        project.files.raw(
            file_path=path,
            ref=ref,
            streamed=True,
            action=f.write,
        )


def test2(api: Gitlab, prj_id: int, suffix: str, path: str, ref: str) -> None:
    with open("test.file", "wb") as f:
        project = api.projects.get(prj_id, lazy=True)
        project.files.raw(
            file_path=path,
            ref=ref,
            streamed=True,
            action=f.write,
        )

Before it would raise the same type checking errors.

After this patch there are no errors:

diff --git a/gitlab/v4/objects/files.py b/gitlab/v4/objects/files.py
index ce2193c2..e1f7b229 100644
--- a/gitlab/v4/objects/files.py
+++ b/gitlab/v4/objects/files.py
@@ -308,7 +308,7 @@ class ProjectFileManager(CreateMixin, UpdateMixin, DeleteMixin, RESTManager):
         file_path: str,
         ref: Optional[str] = None,
         streamed: Literal[True] = True,
-        action: Optional[Callable[[bytes], None]] = None,
+        action: Optional[Callable[[bytes], Any]] = None,
         chunk_size: int = 1024,
         *,
         iterator: Literal[False] = False,

Success: no issues found in 1 source file

I triggered a new release in case this is affecting version bumps out in the wild.