From 3c8ba8274e4e71bbb012bc3d030aae58322792fe Mon Sep 17 00:00:00 2001
From: Patrick Adelino <patrick.adeelino@gmail.com>
Date: Wed, 24 Jun 2026 17:42:47 -0300
Subject: [PATCH] Fix GitHub files content decoding

---
 .../handlers/github_handler/github_tables.py  | 41 +++++++++++++++++--
 1 file changed, 37 insertions(+), 4 deletions(-)

diff --git a/mindsdb/integrations/handlers/github_handler/github_tables.py b/mindsdb/integrations/handlers/github_handler/github_tables.py
index df7cde350f3..f17833f3e61 100644
--- a/mindsdb/integrations/handlers/github_handler/github_tables.py
+++ b/mindsdb/integrations/handlers/github_handler/github_tables.py
@@ -1115,12 +1115,27 @@ def get_columns(self) -> List[str]:
 
 class GithubFilesTable(APIResource):
 
-    def get_path(self, repo, path, file_matches=None, file_not_matches=None, limit=None):
+    def get_path(
+        self,
+        repo,
+        path,
+        file_matches=None,
+        file_not_matches=None,
+        limit=None,
+        include_content=True,
+    ):
 
         res = []
         for item in list(repo.get_contents(path)):
             if item.type == "dir":
-                subres = self.get_path(repo, item.path, file_matches, file_not_matches, limit)
+                subres = self.get_path(
+                    repo,
+                    item.path,
+                    file_matches,
+                    file_not_matches,
+                    limit,
+                    include_content,
+                )
                 res.extend(subres)
                 limit -= len(subres)
             else:
@@ -1141,8 +1156,17 @@ def get_path(self, repo, path, file_matches=None, file_not_matches=None, limit=N
                     file = {
                         'path': item.path,
                         'name': item.name,
-                        'content': item.decoded_content,
                     }
+                    if include_content:
+                        try:
+                            file['content'] = item.decoded_content
+                        except AssertionError as exc:
+                            logger.warning(
+                                "Could not decode GitHub file content for %s: %s",
+                                item.path,
+                                exc,
+                            )
+                            file['content'] = None
                     res.append(file)
                     limit -= 1
             if limit <= 0:
@@ -1187,9 +1211,18 @@ def list(self,
         if len(file_not_matches) == 0:
             file_not_matches = None
 
+        include_content = not targets or 'content' in targets
+
         all_res = []
         for repo in repos:
-            res = self.get_path(repo, path, file_matches, file_not_matches, limit - len(all_res))
+            res = self.get_path(
+                repo,
+                path,
+                file_matches,
+                file_not_matches,
+                limit - len(all_res),
+                include_content,
+            )
             for item in res:
                 item["repository"] = repo.full_name
             all_res.extend(res)