imprv: vfunc and AsyncReadyCallback signatures (#308)

- Split `Gio.AsyncReadyCallback` into two private aliases:
  `_AsyncReadyCallback` for single user_data and
  `_AsyncReadyVarArgsCallback` for variadic user_data. Public
  async functions and methods now use the var-args form, while
  vfuncs and `simple_async_report_gerror_in_idle` use the single
  form.
- Treat vfuncs on objects/interfaces as methods so they receive
  `self`, keep their destroy/closure args visible, and emit them
  as positional-only (trailing `/`) since PyGObject's C
  marshaller invokes them positionally.

Author: bryanforbes

src/gi-stubs/repository/Adw.pyi

@@ -1043,7 +1043,7 @@ class AlertDialog(Dialog):
         self,
         parent: _Gtk4.Widget | None,
         cancellable: Gio.Cancellable | None,
-        callback: Gio.AsyncReadyCallback[AlertDialog, Unpack[_DataTs]] | None,
+        callback: Gio._AsyncReadyVarArgsCallback[AlertDialog, Unpack[_DataTs]] | None,
         *user_data: Unpack[_DataTs],
     ) -> None: ...
     @overload
@@ -1052,10 +1052,10 @@ class AlertDialog(Dialog):
         parent: _Gtk4.Widget | None = None,
         cancellable: Gio.Cancellable | None = None,
         *,
-        callback: Gio.AsyncReadyCallback[AlertDialog] | None,
+        callback: Gio._AsyncReadyVarArgsCallback[AlertDialog] | None,
     ) -> None: ...
     def choose_finish(self, result: Gio.AsyncResult) -> str: ...
-    def do_response(self, response: str) -> None: ...
+    def do_response(self, response: str, /) -> None: ...
     def get_body(self) -> str: ...
     def get_body_use_markup(self) -> bool: ...
     def get_close_response(self) -> str: ...
@@ -5725,18 +5725,18 @@ class MessageDialog(_Gtk4.Window):
     def choose(
         self,
         cancellable: Gio.Cancellable | None,
-        callback: Gio.AsyncReadyCallback[MessageDialog, Unpack[_DataTs]] | None,
+        callback: Gio._AsyncReadyVarArgsCallback[MessageDialog, Unpack[_DataTs]] | None,
         *user_data: Unpack[_DataTs],
     ) -> None: ...
     @overload
     def choose(
         self,
         cancellable: Gio.Cancellable | None = None,
         *,
-        callback: Gio.AsyncReadyCallback[MessageDialog] | None,
+        callback: Gio._AsyncReadyVarArgsCallback[MessageDialog] | None,
     ) -> None: ...
     def choose_finish(self, result: Gio.AsyncResult) -> str: ...
-    def do_response(self, response: str) -> None: ...
+    def do_response(self, response: str, /) -> None: ...
     def get_body(self) -> str: ...
     def get_body_use_markup(self) -> bool: ...
     def get_close_response(self) -> str: ...

src/gi-stubs/repository/Atk.pyi

@@ -424,9 +424,9 @@ class Hyperlink(GObject.Object, Action):
     def parent(self) -> GObject.Object: ...
     def do_get_end_index(self) -> int: ...
     def do_get_n_anchors(self) -> int: ...
-    def do_get_object(self, i: int) -> Object: ...
+    def do_get_object(self, i: int, /) -> Object: ...
     def do_get_start_index(self) -> int: ...
-    def do_get_uri(self, i: int) -> str: ...
+    def do_get_uri(self, i: int, /) -> str: ...
     def do_is_selected_link(self) -> bool: ...
     def do_is_valid(self) -> bool: ...
     def do_link_activated(self) -> None: ...
@@ -925,11 +925,11 @@ class Object(GObject.Object):
     def add_relationship(
         self, relationship: _RelationTypeValueType, target: Object
     ) -> bool: ...
-    def do_active_descendant_changed(self, child: int | Any | None) -> None: ...
+    def do_active_descendant_changed(self, child: int | Any | None, /) -> None: ...
     def do_children_changed(
-        self, change_index: int, changed_child: int | Any | None
+        self, change_index: int, changed_child: int | Any | None, /
     ) -> None: ...
-    def do_focus_event(self, focus_in: bool) -> None: ...
+    def do_focus_event(self, focus_in: bool, /) -> None: ...
     def do_get_attributes(self) -> list[int]: ...
     def do_get_description(self) -> str: ...
     def do_get_index_in_parent(self) -> int: ...
@@ -940,16 +940,16 @@ class Object(GObject.Object):
     def do_get_object_locale(self) -> str: ...
     def do_get_parent(self) -> Object: ...
     def do_get_role(self) -> Role: ...
-    def do_initialize(self, data: int | Any | None) -> None: ...
-    def do_property_change(self, values: PropertyValues) -> None: ...
+    def do_initialize(self, data: int | Any | None, /) -> None: ...
+    def do_property_change(self, values: PropertyValues, /) -> None: ...
     def do_ref_relation_set(self) -> RelationSet: ...
     def do_ref_state_set(self) -> StateSet: ...
-    def do_remove_property_change_handler(self, handler_id: int) -> None: ...
-    def do_set_description(self, description: str) -> None: ...
-    def do_set_name(self, name: str) -> None: ...
-    def do_set_parent(self, parent: Object) -> None: ...
-    def do_set_role(self, role: _RoleValueType) -> None: ...
-    def do_state_change(self, name: str, state_set: bool) -> None: ...
+    def do_remove_property_change_handler(self, handler_id: int, /) -> None: ...
+    def do_set_description(self, description: str, /) -> None: ...
+    def do_set_name(self, name: str, /) -> None: ...
+    def do_set_parent(self, parent: Object, /) -> None: ...
+    def do_set_role(self, role: _RoleValueType, /) -> None: ...
+    def do_state_change(self, name: str, state_set: bool, /) -> None: ...
     def do_visible_data_changed(self) -> None: ...
     def get_accessible_id(self) -> str: ...
     def get_attributes(self) -> list[int]: ...
@@ -1509,7 +1509,7 @@ class Socket(Object, Component):
         accessible_table_summary: Object | None = ...,
         accessible_value: float = ...,
     ) -> None: ...
-    def do_embed(self, plug_id: str) -> None: ...
+    def do_embed(self, plug_id: str, /) -> None: ...
     def embed(self, plug_id: str) -> None: ...
     def is_occupied(self) -> bool: ...
     @classmethod

src/gi-stubs/repository/GObject.pyi

@@ -1484,10 +1484,10 @@ class ParamSpec(_gi.Fundamental):
     def do_set_property(
         self, pspec, value
     ): ...  # FIXME: Override is missing typing annotation
-    def do_value_is_valid(self, value: Any) -> bool: ...
-    def do_value_set_default(self, value: Any) -> None: ...
-    def do_value_validate(self, value: Any) -> bool: ...
-    def do_values_cmp(self, value1: Any, value2: Any) -> int: ...
+    def do_value_is_valid(self, value: Any, /) -> bool: ...
+    def do_value_set_default(self, value: Any, /) -> None: ...
+    def do_value_validate(self, value: Any, /) -> bool: ...
+    def do_values_cmp(self, value1: Any, value2: Any, /) -> int: ...
     def get_blurb(self) -> str | None: ...
     def get_default_value(self) -> Any: ...
     def get_name(self) -> str: ...

src/gi-stubs/repository/GdkPixbuf.pyi

@@ -192,7 +192,7 @@ class Pixbuf(GObject.Object, Gio.Icon, Gio.LoadableIcon):
     def get_file_info_async(
         filename: str,
         cancellable: Gio.Cancellable | None,
-        callback: Gio.AsyncReadyCallback[None, Unpack[_DataTs]] | None,
+        callback: Gio._AsyncReadyVarArgsCallback[None, Unpack[_DataTs]] | None,
         *user_data: Unpack[_DataTs],
     ) -> None: ...
     @overload
@@ -201,7 +201,7 @@ class Pixbuf(GObject.Object, Gio.Icon, Gio.LoadableIcon):
         filename: str,
         cancellable: Gio.Cancellable | None = None,
         *,
-        callback: Gio.AsyncReadyCallback[None] | None,
+        callback: Gio._AsyncReadyVarArgsCallback[None] | None,
     ) -> None: ...
     @staticmethod
     def get_file_info_finish(
@@ -285,7 +285,7 @@ class Pixbuf(GObject.Object, Gio.Icon, Gio.LoadableIcon):
     def new_from_stream_async(
         stream: Gio.InputStream,
         cancellable: Gio.Cancellable | None,
-        callback: Gio.AsyncReadyCallback[None, Unpack[_DataTs]] | None,
+        callback: Gio._AsyncReadyVarArgsCallback[None, Unpack[_DataTs]] | None,
         *user_data: Unpack[_DataTs],
     ) -> None: ...
     @overload
@@ -294,7 +294,7 @@ class Pixbuf(GObject.Object, Gio.Icon, Gio.LoadableIcon):
         stream: Gio.InputStream,
         cancellable: Gio.Cancellable | None = None,
         *,
-        callback: Gio.AsyncReadyCallback[None] | None,
+        callback: Gio._AsyncReadyVarArgsCallback[None] | None,
     ) -> None: ...
     @classmethod
     def new_from_stream_at_scale(
@@ -312,10 +312,7 @@ class Pixbuf(GObject.Object, Gio.Icon, Gio.LoadableIcon):
         height: int,
         preserve_aspect_ratio: bool,
         cancellable: Gio.Cancellable | None = None,
-        callback: Callable[
-            [GObject.Object | None, Gio.AsyncResult, Unpack[_DataTs]], None
-        ]
-        | None = None,
+        callback: Gio._AsyncReadyVarArgsCallback[None, Unpack[_DataTs]] | None = None,
         *user_data: Unpack[_DataTs],
     ) -> None: ...
     @classmethod
@@ -363,10 +360,7 @@ class Pixbuf(GObject.Object, Gio.Icon, Gio.LoadableIcon):
         option_keys: Sequence[str] | None = None,
         option_values: Sequence[str] | None = None,
         cancellable: Gio.Cancellable | None = None,
-        callback: Callable[
-            [GObject.Object | None, Gio.AsyncResult, Unpack[_DataTs]], None
-        ]
-        | None = None,
+        callback: Gio._AsyncReadyVarArgsCallback[Pixbuf, Unpack[_DataTs]] | None = None,
         *user_data: Unpack[_DataTs],
     ) -> None: ...
     def savev(
@@ -413,8 +407,10 @@ class PixbufAnimation(GObject.Object):
     """
     @property
     def parent_instance(self) -> GObject.Object: ...
-    def do_get_iter(self, start_time: GLib.TimeVal | None) -> PixbufAnimationIter: ...
-    def do_get_size(self, width: int, height: int) -> None: ...
+    def do_get_iter(
+        self, start_time: GLib.TimeVal | None, /
+    ) -> PixbufAnimationIter: ...
+    def do_get_size(self, width: int, height: int, /) -> None: ...
     def do_get_static_image(self) -> Pixbuf: ...
     def do_is_static_image(self) -> bool: ...
     def get_height(self) -> int: ...
@@ -442,7 +438,7 @@ class PixbufAnimation(GObject.Object):
     def new_from_stream_async(
         stream: Gio.InputStream,
         cancellable: Gio.Cancellable | None,
-        callback: Gio.AsyncReadyCallback[None, Unpack[_DataTs]] | None,
+        callback: Gio._AsyncReadyVarArgsCallback[None, Unpack[_DataTs]] | None,
         *user_data: Unpack[_DataTs],
     ) -> None: ...
     @overload
@@ -451,7 +447,7 @@ class PixbufAnimation(GObject.Object):
         stream: Gio.InputStream,
         cancellable: Gio.Cancellable | None = None,
         *,
-        callback: Gio.AsyncReadyCallback[None] | None,
+        callback: Gio._AsyncReadyVarArgsCallback[None] | None,
     ) -> None: ...
     @classmethod
     def new_from_stream_finish(
@@ -495,7 +491,7 @@ class PixbufAnimationIter(GObject.Object):
     @property
     def parent_instance(self) -> GObject.Object: ...
     def advance(self, current_time: GLib.TimeVal | None = None) -> bool: ...
-    def do_advance(self, current_time: GLib.TimeVal | None) -> bool: ...
+    def do_advance(self, current_time: GLib.TimeVal | None, /) -> bool: ...
     def do_get_delay_time(self) -> int: ...
     def do_get_pixbuf(self) -> Pixbuf: ...
     def do_on_currently_loading_frame(self) -> bool: ...
@@ -581,9 +577,9 @@ class PixbufLoader(GObject.Object):
     def priv(self) -> int: ...
     def close(self) -> bool: ...
     def do_area_prepared(self) -> None: ...
-    def do_area_updated(self, x: int, y: int, width: int, height: int) -> None: ...
+    def do_area_updated(self, x: int, y: int, width: int, height: int, /) -> None: ...
     def do_closed(self) -> None: ...
-    def do_size_prepared(self, width: int, height: int) -> None: ...
+    def do_size_prepared(self, width: int, height: int, /) -> None: ...
     def get_animation(self) -> PixbufAnimation | None: ...
     def get_format(self) -> PixbufFormat | None: ...
     def get_pixbuf(self) -> Pixbuf | None: ...