PR Custody

[codex] fix googleads customer id overrides

@singhbharath03draftchecks n/achecks…codex/int-2031-googleads-customer-id → master12 files · +360 −96updated 1w ago

▸Description · 1 noise

Summary

Fixes INT-2031 by adding an optional customer_id parameter across Google Ads tools so MCC users can target sub-accounts per tool call.

What changed

Added a shared Google Ads customer ID helper for normalization, metadata fallback, and MCC login-customer-id header handling.
Updated all Google Ads request models to expose optional customer_id.
Routed customer-scoped API calls through the per-call customer ID when provided, with the existing connection customer ID as the fallback.
Added regression tests covering schema exposure, sub-account URL routing, MCC login header behavior, and fallback behavior.

Validation

uv run pytest tests/test_apps/test_googleads/test_customer_id_override.py -q -> 18 passed
uv run --with ruff ruff check apps/googleads/actions tests/test_apps/test_googleads --config config/ruff.toml -> passed
uv run python -m compileall -q apps/googleads/actions tests/test_apps/test_googleads -> passed
git diff --check -> passed

▸ 1 bot/status comment hidden

@linear-code[bot]1w ago

Problem

Manager (MCC) accounts in Google Ads have sub-accounts with different customer IDs. Currently, our Google Ads tools only query the manager account because we don't expose a customer_id field on the tools. This breaks the MCC use case — users with manager accounts cannot query data for their sub-accounts.

Customer report: Chris Powers from overit.com (Plain thread T-10771, Priority: High). The server can return a list of sub-accounts, but cannot query them individually.

Proposed Solution

Add a customer_id parameter to all Google Ads tools:

If customer_id is passed in the tool call, use it (send as the login-customer-id header / path param)
If customer_id is NOT passed, fall back to the customer_id stored in the connection metadata (current behavior)

This allows users to:

List sub-accounts under their MCC
Query any specific sub-account by passing its customer_id

Affected Tools

All Google Ads tools (e.g., GOOGLEADS_SEARCH_STREAM_GAQL, etc.)

Filed via Zen on behalf of Pranai (pranai@usefulagents.com).

loading diff…

@linear-code[bot]1w ago

Problem

Customer report: Chris Powers from overit.com (Plain thread T-10771, Priority: High). The server can return a list of sub-accounts, but cannot query them individually.

Proposed Solution

Add a customer_id parameter to all Google Ads tools:

If customer_id is passed in the tool call, use it (send as the login-customer-id header / path param)
If customer_id is NOT passed, fall back to the customer_id stored in the connection metadata (current behavior)

This allows users to:

List sub-accounts under their MCC
Query any specific sub-account by passing its customer_id

Affected Tools

All Google Ads tools (e.g., GOOGLEADS_SEARCH_STREAM_GAQL, etc.)

Filed via Zen on behalf of Pranai (pranai@usefulagents.com).