refactor: simplify scope validation helpers in google workspace scripts

Fix double file read bug in google_api.py _missing_scopes(), consolidate
redundant _normalize_scope_values into callers, merge duplicate except blocks.

skills/productivity/google-workspace/scripts/google_api.py

@@ -44,25 +44,15 @@ SCOPES = [
 ]
 
 
-def _load_token_payload() -> dict:
-    try:
-        return json.loads(TOKEN_PATH.read_text())
-    except Exception:
-        return {}
-
-
-def _normalize_scope_values(values) -> set[str]:
-    if not values:
-        return set()
-    if isinstance(values, str):
-        values = values.split()
-    return {str(value).strip() for value in values if str(value).strip()}
-
-
 def _missing_scopes() -> list[str]:
-    granted = _normalize_scope_values(_load_token_payload().get("scopes") or _load_token_payload().get("scope"))
+    try:
-    if not granted:
+        payload = json.loads(TOKEN_PATH.read_text())
+    except Exception:
         return []
+    raw = payload.get("scopes") or payload.get("scope")
+    if not raw:
+        return []
+    granted = {s.strip() for s in (raw.split() if isinstance(raw, str) else raw) if s.strip()}
     return sorted(scope for scope in SCOPES if scope not in granted)
 
 

skills/productivity/google-workspace/scripts/setup.py

@@ -56,24 +56,15 @@ REDIRECT_URI = "http://localhost:1"
 def _load_token_payload(path: Path = TOKEN_PATH) -> dict:
     try:
         return json.loads(path.read_text())
-    except FileNotFoundError:
-        return {}
     except Exception:
         return {}
 
 
-def _normalize_scope_values(values) -> set[str]:
-    if not values:
-        return set()
-    if isinstance(values, str):
-        values = values.split()
-    return {str(value).strip() for value in values if str(value).strip()}
-
-
 def _missing_scopes_from_payload(payload: dict) -> list[str]:
-    granted = _normalize_scope_values(payload.get("scopes") or payload.get("scope"))
+    raw = payload.get("scopes") or payload.get("scope")
-    if not granted:
+    if not raw:
         return []
+    granted = {s.strip() for s in (raw.split() if isinstance(raw, str) else raw) if s.strip()}
     return sorted(scope for scope in SCOPES if scope not in granted)