Merge pull request #484 from sethladd/add-chat-spaces-readonly-scope

Add chat.spaces.readonly scope for listing spaces

Author: taylorwilsdon

.github/workflows/ruff.yml

@@ -15,7 +15,8 @@ jobs:
     steps:
     - uses: actions/checkout@v6
       with:
-        ref: ${{ github.head_ref || github.ref }}
+        ref: ${{ github.event.pull_request.head.ref || github.ref }}
+        repository: ${{ github.event.pull_request.head.repo.full_name || github.repository }}
         token: ${{ secrets.GITHUB_TOKEN }}
     - uses: actions/setup-python@v6
       with:
@@ -30,14 +31,14 @@ jobs:
         uv run ruff check --fix
         uv run ruff format
     - name: Commit and push fixes
-      if: github.event_name == 'pull_request'
+      if: github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository
       run: |
         git diff --quiet && exit 0
         git config user.name "github-actions[bot]"
         git config user.email "github-actions[bot]@users.noreply.github.com"
         git add -A
         git commit -m "style: auto-fix ruff lint and format"
-        git push || echo "::warning::Could not push auto-fixes — fork PRs require 'Allow edits from maintainers' enabled"
+        git push
     - name: Validate
       run: |
         uv run ruff check

auth/scopes.py

@@ -41,6 +41,7 @@
 CHAT_READONLY_SCOPE = "https://www.googleapis.com/auth/chat.messages.readonly"
 CHAT_WRITE_SCOPE = "https://www.googleapis.com/auth/chat.messages"
 CHAT_SPACES_SCOPE = "https://www.googleapis.com/auth/chat.spaces"
+CHAT_SPACES_READONLY_SCOPE = "https://www.googleapis.com/auth/chat.spaces.readonly"
 
 # Google Sheets API scopes
 SHEETS_READONLY_SCOPE = "https://www.googleapis.com/auth/spreadsheets.readonly"
@@ -98,6 +99,7 @@
     TASKS_SCOPE: {TASKS_READONLY_SCOPE},
     CONTACTS_SCOPE: {CONTACTS_READONLY_SCOPE},
     CHAT_WRITE_SCOPE: {CHAT_READONLY_SCOPE},
+    CHAT_SPACES_SCOPE: {CHAT_SPACES_READONLY_SCOPE},
     FORMS_BODY_SCOPE: {FORMS_BODY_READONLY_SCOPE},
     SCRIPT_PROJECTS_SCOPE: {SCRIPT_PROJECTS_READONLY_SCOPE},
     SCRIPT_DEPLOYMENTS_SCOPE: {SCRIPT_DEPLOYMENTS_READONLY_SCOPE},
@@ -150,7 +152,12 @@ def has_required_scopes(available_scopes, required_scopes):
     GMAIL_SETTINGS_BASIC_SCOPE,
 ]
 
-CHAT_SCOPES = [CHAT_READONLY_SCOPE, CHAT_WRITE_SCOPE, CHAT_SPACES_SCOPE]
+CHAT_SCOPES = [
+    CHAT_READONLY_SCOPE,
+    CHAT_WRITE_SCOPE,
+    CHAT_SPACES_SCOPE,
+    CHAT_SPACES_READONLY_SCOPE,
+]
 
 SHEETS_SCOPES = [SHEETS_READONLY_SCOPE, SHEETS_WRITE_SCOPE, DRIVE_READONLY_SCOPE]
 
@@ -201,7 +208,7 @@ def has_required_scopes(available_scopes, required_scopes):
     "calendar": [CALENDAR_READONLY_SCOPE],
     "docs": [DOCS_READONLY_SCOPE, DRIVE_READONLY_SCOPE],
     "sheets": [SHEETS_READONLY_SCOPE, DRIVE_READONLY_SCOPE],
-    "chat": [CHAT_READONLY_SCOPE],
+    "chat": [CHAT_READONLY_SCOPE, CHAT_SPACES_READONLY_SCOPE],
     "forms": [FORMS_BODY_READONLY_SCOPE, FORMS_RESPONSES_READONLY_SCOPE],
     "slides": [SLIDES_READONLY_SCOPE],
     "tasks": [TASKS_READONLY_SCOPE],

auth/service_decorator.py

@@ -39,6 +39,7 @@
     CHAT_READONLY_SCOPE,
     CHAT_WRITE_SCOPE,
     CHAT_SPACES_SCOPE,
+    CHAT_SPACES_READONLY_SCOPE,
     FORMS_BODY_SCOPE,
     FORMS_BODY_READONLY_SCOPE,
     FORMS_RESPONSES_READONLY_SCOPE,
@@ -440,6 +441,7 @@ def _remove_user_email_arg_from_docstring(docstring: str) -> str:
     "chat_read": CHAT_READONLY_SCOPE,
     "chat_write": CHAT_WRITE_SCOPE,
     "chat_spaces": CHAT_SPACES_SCOPE,
+    "chat_spaces_readonly": CHAT_SPACES_READONLY_SCOPE,
     # Forms scopes
     "forms": FORMS_BODY_SCOPE,
     "forms_read": FORMS_BODY_READONLY_SCOPE,

gchat/chat_tools.py

@@ -99,7 +99,7 @@ def _extract_rich_links(msg: dict) -> List[str]:
 
 
 @server.tool()
-@require_google_service("chat", "chat_read")
+@require_google_service("chat", "chat_spaces_readonly")
 @handle_http_errors("list_spaces", service_type="chat")
 async def list_spaces(
     service,