Merge pull request #442 from Leandro3996/fix/appscript-get-content-endpoint

fix: use getContent() instead of get() for Apps Script file retrieval

Author: taylorwilsdon

gappsscript/apps_script_tools.py

@@ -105,8 +105,10 @@ async def _get_script_project_impl(
     """Internal implementation for get_script_project."""
     logger.info(f"[get_script_project] Email: {user_google_email}, ID: {script_id}")
 
-    project = await asyncio.to_thread(
-        service.projects().get(scriptId=script_id).execute
+    # Get project metadata and content concurrently (independent requests)
+    project, content = await asyncio.gather(
+        asyncio.to_thread(service.projects().get(scriptId=script_id).execute),
+        asyncio.to_thread(service.projects().getContent(scriptId=script_id).execute),
     )
 
     title = project.get("title", "Untitled")
@@ -124,7 +126,7 @@ async def _get_script_project_impl(
         "Files:",
     ]
 
-    files = project.get("files", [])
+    files = content.get("files", [])
     for i, file in enumerate(files, 1):
         file_name = file.get("name", "Untitled")
         file_type = file.get("type", "Unknown")
@@ -172,11 +174,12 @@ async def _get_script_content_impl(
         f"[get_script_content] Email: {user_google_email}, ID: {script_id}, File: {file_name}"
     )
 
-    project = await asyncio.to_thread(
-        service.projects().get(scriptId=script_id).execute
+    # Must use getContent() to retrieve files, not get() which only returns metadata
+    content = await asyncio.to_thread(
+        service.projects().getContent(scriptId=script_id).execute
     )
 
-    files = project.get("files", [])
+    files = content.get("files", [])
     target_file = None
 
     for file in files:

tests/gappsscript/test_apps_script_tools.py

@@ -62,12 +62,19 @@ async def test_list_script_projects():
 async def test_get_script_project():
     """Test retrieving complete project details"""
     mock_service = Mock()
-    mock_response = {
+
+    # projects().get() returns metadata only (no files)
+    mock_metadata_response = {
         "scriptId": "test123",
         "title": "Test Project",
         "creator": {"email": "creator@example.com"},
         "createTime": "2025-01-10T10:00:00Z",
         "updateTime": "2026-01-12T15:30:00Z",
+    }
+
+    # projects().getContent() returns files with source code
+    mock_content_response = {
+        "scriptId": "test123",
         "files": [
             {
                 "name": "Code",
@@ -77,7 +84,8 @@ async def test_get_script_project():
         ],
     }
 
-    mock_service.projects().get().execute.return_value = mock_response
+    mock_service.projects().get().execute.return_value = mock_metadata_response
+    mock_service.projects().getContent().execute.return_value = mock_content_response
 
     result = await _get_script_project_impl(
         service=mock_service, user_google_email="test@example.com", script_id="test123"