Skip to content

Revert "fix(google-analytics): fix input validation and align output schemas with other Google MCPs (#475)"#517

Merged
yuriassuncx merged 1 commit into
mainfrom
revert-ga-475
Jul 9, 2026
Merged

Revert "fix(google-analytics): fix input validation and align output schemas with other Google MCPs (#475)"#517
yuriassuncx merged 1 commit into
mainfrom
revert-ga-475

Conversation

@yuriassuncx

@yuriassuncx yuriassuncx commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • CI checks pass
  • Manual smoke test of google-analytics tools if needed before merge

🤖 Generated with Claude Code


Summary by cubic

Reverts #475 for google-analytics. Restores the pre-#475 schemas and tool behavior, including response-wrapped outputs and stricter native input types.

  • Refactors

    • Bring back response wrapper: outputs now return { response: ... } for accounts, property, ads links, annotations, runReport, runRealtimeReport, and runFunnelReport.
    • Remove JSON-string preprocessing in tool inputs; arrays/objects/booleans must be passed as native values.
    • Restore custom dimensions/metrics shape: { dimensions: { customDimensions: [...] }, metrics: { customMetrics: [...] } }.
    • Revert schema fields to prior state (e.g., remove canEdit on property summaries, remove annotationDateRange, rename quota field to tokensPerProjectPerDay, drop dataLossFromOtherRow and emptyReason from response metadata).
  • Migration

    • Read results from the response key.
    • Stop sending JSON-encoded strings in inputs; send native JSON values.
    • Access custom dimensions and metrics under dimensions.customDimensions and metrics.customMetrics.

Written for commit b40c703. Summary will update on new commits.

Review in cubic

…schemas with other Google MCPs (#475)"

This reverts commit 6f3612c.
@yuriassuncx yuriassuncx enabled auto-merge July 9, 2026 17:13
@yuriassuncx yuriassuncx merged commit 843251b into main Jul 9, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants