fix(google-sheets): upgrade to SDK 2.0.0, fix Lambda crash on error paths

[Shubhank-Jonnada]

Summary

Fixes production Lambda crash: `integration-google-sheets` returning "Unhandled" error.

Root cause: Error paths returning `ActionResult(data={"error": ..., "result": False})` fail output schema validation in SDK 2.0.0, causing unhandled exceptions and Lambda crashes.

Fix:

• Convert all error paths to `ActionError(message=...)`
• Remove `"error"` and boolean `"result"` from all output schemas
• Upgrade SDK to `~=2.0.0`
• Fix optional param access: `inputs.get("pageSize")` and `inputs.get("pageToken")` instead of bracket access
• Add 51 pytest unit tests
• Add full integration tests covering all 10 actions

Test Results

Unit Tests (mocked, CI-safe)

Integration	Tests	Coverage	Status
google-sheets	51/51	99%	✅ PASSED

Integration Tests (live Google Sheets/Drive API)

Test Class	Action	Status
`TestListSpreadsheets`	`sheets_list_spreadsheets`	✅ PASSED
`TestGetSpreadsheet`	`sheets_get_spreadsheet`	✅ PASSED
`TestListSheets`	`sheets_list_sheets`	✅ PASSED
`TestReadRange`	`sheets_read_range`	✅ PASSED
`TestWriteRange`	`sheets_write_range`	✅ PASSED
`TestAppendRows`	`sheets_append_rows`	✅ PASSED
`TestFormatRange`	`sheets_format_range`	✅ PASSED
`TestFreeze`	`sheets_freeze`	✅ PASSED
`TestBatchUpdate`	`sheets_batch_update`	✅ PASSED
`TestDuplicateSpreadsheet`	`sheets_duplicate_spreadsheet`	⏭️ SKIPPED
Total	13/14	✅

`duplicate_spreadsheet` skips — token lacks Drive copy permissions for the test file. The action code is correct, it is a permissions constraint on the test token only.

Validation

Check	Result
`validate_integration.py`	✅ Passed (1 pre-existing scope warning)
`check_code.py`	✅ Passed
`ruff check/format`	✅ clean

How to run

```bash

Unit tests

python -m pytest google-sheets/tests/ -m unit -v

Integration tests - read only

GOOGLE_SHEETS_ACCESS_TOKEN= python -m pytest google-sheets/tests/test_google_sheets_integration.py -m "integration and not destructive" -v

Integration tests - destructive

GOOGLE_SHEETS_ACCESS_TOKEN= python -m pytest google-sheets/tests/test_google_sheets_integration.py -m "integration and destructive" -v
```

[github-actions]

🔍 Integration Validation Results

Commit: ba95c8d7596b227077d2a0bea81206458b69adf5 · Merge branch 'master' into fix/google-sheets-sdk-upgrade-v2
Updated: 2026-05-11T23:23:58Z

Changed directories: google-sheets

Check	Result
Structure	⚠️ Passed with warnings
Code	✅ Passed
Tests	✅ Passed
README	✅ Passed
Version	✅ Passed

⚠️ Structure Check output

Validating 1 integration(s)...

============================================================
Integration: google-sheets
============================================================

Warnings (1):
  ⚠️ Potentially unused scopes (please verify): https://www.googleapis.com/auth/spreadsheets

============================================================
SUMMARY
============================================================
Integrations validated: 1
Total errors: 0
Total warnings: 1

⚠️ Validation passed with warnings - please review

✅ Code Check output

[notice] A new release of pip is available: 26.0.1 -> 26.1.1
[notice] To update, run: pip install --upgrade pip
----------------------------------------
Checking: google-sheets
----------------------------------------

📦 Installing dependencies...

🐍 Checking Python syntax...
   ✅ Syntax OK

📥 Checking imports...
   ✅ Imports OK

📄 Checking JSON files...
   ✅ JSON files OK

🔍 Linting with ruff...
   ✅ Lint OK

🎨 Checking formatting with ruff...
   ✅ Formatting OK

🔒 Scanning for security issues with bandit...
   ✅ Security OK

🛡️ Checking dependencies for vulnerabilities with pip-audit...
   ✅ Dependencies OK

🔗 Checking config-code sync...
   ✅ Config-code sync OK

🔄 Checking fetch patterns...
   ✅ Fetch patterns OK

========================================
✅ CODE CHECK PASSED
========================================

✅ Tests Check output

Integration       Tests  Coverage        Status
-----------------------------------------------
google-sheets     57/57      100%      ✅ Passed
-----------------------------------------------
Total             57/57            ✅ All passed

✅ Tests passed: google-sheets

✅ README Check output

========================================
✅ README CHECK PASSED
========================================

✅ Version Check output

✅ google-sheets: 0.1.1 → 2.0.0 (major bump)

========================================
✅ VERSION CHECK PASSED
========================================

[TheRealAgentK]

LGTM and thanks for moving fast on this — the production Lambda crash fix is the priority here.

SDK 2.0.0 migration is clean — context.fetch().data, ActionError(message=...) everywhere, schema envelope fields removed, version bumped. 51 unit tests.

Local CI verified: validate_integration ✅, check_code ✅, ruff ✅, pytest ✅.

Follow-up tracked in #297 (high priority) — please add a test_google_sheets_integration.py in a follow-up PR. The class of bug this PR fixed (schema-validation failures from real API responses) is exactly what mocked unit tests cannot catch, so this gap is worth closing soon. Not blocking this merge given the live impact.

Process nit: branch doesn't follow <type>/<issue#>/<desc> per AGENTS.md. Please follow the convention next time.

[Shubhank-Jonnada]

@TheRealAgentK unit and live integration tests are all done. Here is a summary:

Unit tests: 51/51 passed at 99% coverage across all actions.

Live integration tests: 13/14 passed against the real Google Sheets and Drive API, covering all 10 actions:

• sheets_list_spreadsheets
• sheets_get_spreadsheet
• sheets_list_sheets
• sheets_read_range
• sheets_write_range (destructive)
• sheets_append_rows (destructive)
• sheets_format_range (destructive)
• sheets_freeze (destructive)
• sheets_batch_update (destructive)
• sheets_duplicate_spreadsheet (skipped - token lacks Drive copy permissions on the test file, not a code issue)

Also fixed the optional param access issue flagged by the config-code sync check (pageSize and pageToken now use inputs.get()) and ruff formatting is clean at line-length 120.

[TheRealAgentK]

1. Dead, broken helper — tests/test_google_sheets_integration.py:57-69

def get_first_spreadsheet_id(live_context):
    import asyncio
    async def _get(): ...
    return asyncio.get_event_loop().run_until_complete(_get())

Never called — every test inlines the sheets_list_spreadsheets call. asyncio.get_event_loop() is also deprecated and would conflict with pytest-asyncio's loop if invoked. Either delete it or use it (replace the inline list-call boilerplate at the top of each chained test).

2. Weak assertions in destructive tests — lines 258, 281, 307

assert result.result is not None

result.result is ActionResultData on success or ActionErrorData on failure — always truthy. The test passes even when the action returned an ActionError. Use:

from autohive_integrations_sdk.integration import ResultType
assert result.type == ResultType.ACTION

3. Stale tests/context.py — delete

Pre-SDK-2.0 testbed shim. Unused once conftest is fixed.

4. tests/conftest.py is a no-op

sys.path.insert(0, os.path.dirname(__file__)) puts tests/ on path, not the integration root. The real wiring is the importlib block duplicated across all 4 test files. Replacement:

import os, sys
sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), "..")))

Test files then drop the importlib block and use from google_sheets import google_sheets.

5. inputs.get("pageSize") truthy check — google_sheets.py:57, 59

pageSize=0 is silently dropped. Use is not None. Minor.

---

#3 and #4 are the same issues raised again in #284 / #285 / #306.

[TheRealAgentK]

Approving now - @Shubhank-Jonnada please do another review on your end before merging.