fixes

app.py

@@ -58,8 +58,8 @@ class GitHubMCPServer:
                 "error": f"Failed to fetch repository information: {str(e)}"
             }
     
-    def get_file_content(self, owner: str, repo: str, path: str) -> dict:
-        """Get content of a specific file"""
+    def get_file_content(self, owner: str, repo: str, path: str) -> str:
+        """Get content of a specific file - returns just the file content as string"""
         try:
             url = f"https://api.github.com/repos/{owner}/{repo}/contents/{path}"
             response = requests.get(url, headers=self.headers)
@@ -70,82 +70,30 @@ class GitHubMCPServer:
                     # Decode base64 content
                     try:
                         content = base64.b64decode(data["content"]).decode('utf-8')
-                        file_extension = path.split('.')[-1].lower() if '.' in path else 'unknown'
-                        
-                        return {
-                            "success": True,
-                            "file_path": path,
-                            "file_name": path.split('/')[-1],
-                            "file_extension": file_extension,
-                            "content": content,
-                            "content_length": len(content),
-                            "line_count": len(content.split('\n')),
-                            "sha": data["sha"],
-                            "download_url": data.get("download_url", ""),
-                            "is_binary": False
-                        }
+                        return content
                     except UnicodeDecodeError:
-                        return {
-                            "success": False,
-                            "error": "File contains binary data that cannot be decoded as text",
-                            "file_path": path,
-                            "is_binary": True
-                        }
+                        return f"ERROR: File '{path}' contains binary data that cannot be decoded as text"
                 else:
-                    return {
-                        "success": False,
-                        "error": f"Path '{path}' is not a file or content is not available",
-                        "item_type": data.get("type", "unknown")
-                    }
+                    return f"ERROR: Path '{path}' is not a file or content is not available"
             else:
-                return {
-                    "success": False,
-                    "error": f"File not found or inaccessible (HTTP {response.status_code})",
-                    "file_path": path,
-                    "status_code": response.status_code
-                }
+                return f"ERROR: File '{path}' not found or inaccessible (HTTP {response.status_code})"
             
         except Exception as e:
-            return {
-                "success": False,
-                "error": f"Failed to fetch file content: {str(e)}",
-                "file_path": path
-            }
+            return f"ERROR: Failed to fetch file content for '{path}': {str(e)}"
     
-    def scan_repository(self, owner: str, repo: str, extensions: str = ".py,.js,.ts,.php,.java") -> dict:
-        """Scan repository for code files"""
+    def scan_repository(self, owner: str, repo: str, extensions: str = ".py,.js,.ts,.php,.java") -> list:
+        """Scan repository for code files - returns simple list of file paths"""
         try:
             ext_list = [ext.strip() for ext in extensions.split(",") if ext.strip()]
             all_files = []
             self._scan_directory_sync(owner, repo, "", ext_list, all_files)
             
-            # Group files by extension for better organization
-            files_by_extension = {}
-            for file_info in all_files:
-                ext = file_info.get('name', '').split('.')[-1].lower() if '.' in file_info.get('name', '') else 'no_extension'
-                if ext not in files_by_extension:
-                    files_by_extension[ext] = []
-                files_by_extension[ext].append(file_info)
-            
-            return {
-                "success": True,
-                "total_files_found": len(all_files),
-                "extensions_scanned": ext_list,
-                "files_by_extension": files_by_extension,
-                "all_files": all_files[:100],  # Limit to first 100 files
-                "scan_summary": {
-                    "total_files": len(all_files),
-                    "extensions_found": list(files_by_extension.keys()),
-                    "largest_file": max(all_files, key=lambda x: x.get('size', 0)) if all_files else None
-                }
-            }
+            # Return simple list of file paths for easier processing by CodeAgent
+            file_paths = [file_info.get('path', '') for file_info in all_files[:50]]  # Limit to 50 files
+            return file_paths
             
         except Exception as e:
-            return {
-                "success": False,
-                "error": f"Failed to scan repository: {str(e)}",
-                "total_files_found": 0
-            }
+            return [f"ERROR: Failed to scan repository: {str(e)}"]
     
     def _scan_directory_sync(self, owner: str, repo: str, path: str, extensions: List[str], all_files: List[Dict]):
         """Recursively scan directory for files"""