airbyte-source-github 1.8.29.dev202506231149__tar.gz → 1.8.29.dev202506231501__tar.gz

{airbyte_source_github-1.8.29.dev202506231149 → airbyte_source_github-1.8.29.dev202506231501}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: airbyte-source-github
-Version: 1.8.29.dev202506231149
+Version: 1.8.29.dev202506231501
 Summary: Source implementation for GitHub.
 License: MIT
 Author: Airbyte

{airbyte_source_github-1.8.29.dev202506231149 → airbyte_source_github-1.8.29.dev202506231501}/pyproject.toml

@@ -5,7 +5,7 @@ requires = [
 build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
-version = "1.8.29.dev202506231149"
+version = "1.8.29.dev202506231501"
 name = "airbyte-source-github"
 description = "Source implementation for GitHub."
 authors = [

{airbyte_source_github-1.8.29.dev202506231149 → airbyte_source_github-1.8.29.dev202506231501}/source_github/errors_handlers.py

@@ -43,6 +43,13 @@ GITHUB_DEFAULT_ERROR_MAPPING = DEFAULT_ERROR_MAPPING | {
 }
 
 
+def is_conflict_with_empty_repository(response_or_exception: Optional[Union[requests.Response, Exception]] = None) -> bool:
+    if isinstance(response_or_exception, requests.Response) and response_or_exception.status_code == requests.codes.CONFLICT:
+        response_data = response_or_exception.json()
+        return response_data.get("message") == "Git Repository is empty."
+    return False
+
+
 class GithubStreamABCErrorHandler(HttpStatusErrorHandler):
     def __init__(self, stream: HttpStream, **kwargs):  # type: ignore # noqa
         self.stream = stream
@@ -88,6 +95,13 @@ class GithubStreamABCErrorHandler(HttpStatusErrorHandler):
                     error_message=f"Response status code: {response_or_exception.status_code}. Retrying...",
                 )
 
+            if is_conflict_with_empty_repository(response_or_exception=response_or_exception):
+                return ErrorResolution(
+                    response_action=ResponseAction.IGNORE,
+                    failure_type=FailureType.config_error,
+                    error_message="Git Repository is empty.",
+                )
+
         return super().interpret_response(response_or_exception)
 
 
@@ -135,18 +149,3 @@ class GitHubGraphQLErrorHandler(GithubStreamABCErrorHandler):
                 )
 
         return super().interpret_response(response_or_exception)
-
-
-class CommitsErrorHandler(GithubStreamABCErrorHandler):
-    def interpret_response(self, response_or_exception: Optional[Union[requests.Response, Exception]] = None) -> ErrorResolution:
-        if isinstance(response_or_exception, requests.Response):
-            if response_or_exception.status_code == requests.codes.CONFLICT:
-                response_data = response_or_exception.json()
-                if response_data.get("message") == "Git Repository is empty.":
-                    return ErrorResolution(
-                        response_action=ResponseAction.IGNORE,
-                        failure_type=FailureType.config_error,
-                        error_message="Git Repository is empty.",
-                    )
-
-        return super().interpret_response(response_or_exception)

{airbyte_source_github-1.8.29.dev202506231149 → airbyte_source_github-1.8.29.dev202506231501}/source_github/streams.py

@@ -26,10 +26,10 @@ from . import constants
 from .backoff_strategies import ContributorActivityBackoffStrategy, GithubStreamABCBackoffStrategy
 from .errors_handlers import (
     GITHUB_DEFAULT_ERROR_MAPPING,
-    CommitsErrorHandler,
     ContributorActivityErrorHandler,
     GitHubGraphQLErrorHandler,
     GithubStreamABCErrorHandler,
+    is_conflict_with_empty_repository,
 )
 from .graphql import (
     CursorStorage,
@@ -101,10 +101,6 @@ class GithubStreamABC(HttpStream, ABC):
         stream_slice: Mapping[str, Any] = None,
         next_page_token: Mapping[str, Any] = None,
     ) -> Iterable[Mapping]:
-        if response.status_code == 409:
-            response_data = response.json()
-            if response_data.get("message") == "Git Repository is empty.":
-                return
         for record in response.json():  # GitHub puts records in an array.
             yield self.transform(record=record, stream_slice=stream_slice)
 
@@ -222,6 +218,22 @@ class GithubStream(GithubStreamABC):
         record["repository"] = stream_slice["repository"]
         return record
 
+    def parse_response(
+        self,
+        response: requests.Response,
+        stream_state: Mapping[str, Any],
+        stream_slice: Mapping[str, Any] = None,
+        next_page_token: Mapping[str, Any] = None,
+    ) -> Iterable[Mapping]:
+        if is_conflict_with_empty_repository(response):
+            return
+        yield from super().parse_response(
+            response=response,
+            stream_state=stream_state,
+            stream_slice=stream_slice,
+            next_page_token=next_page_token,
+        )
+
 
 class SemiIncrementalMixin(CheckpointMixin):
     """
@@ -668,11 +680,6 @@ class Commits(IncrementalMixin, GithubStream):
         self.branches_stream = Branches(**kwargs)
         self.repositories_stream = RepositoryStats(**kwargs)
 
-    def get_error_handler(self) -> Optional[ErrorHandler]:
-        return CommitsErrorHandler(
-            logger=self.logger, max_retries=self.max_retries, error_mapping=GITHUB_DEFAULT_ERROR_MAPPING, stream=self
-        )
-
     def request_params(self, stream_state: Mapping[str, Any], stream_slice: Mapping[str, Any] = None, **kwargs) -> MutableMapping[str, Any]:
         params = super(IncrementalMixin, self).request_params(stream_state=stream_state, stream_slice=stream_slice, **kwargs)
         since = self.get_starting_point(stream_state=stream_state, stream_slice=stream_slice)