Skip to content

[codex] add Google Ads UI domain skill#174

Draft
klaymansean wants to merge 1 commit intobrowser-use:mainfrom
klaymansean:codex/google-ads-domain-skill
Draft

[codex] add Google Ads UI domain skill#174
klaymansean wants to merge 1 commit intobrowser-use:mainfrom
klaymansean:codex/google-ads-domain-skill

Conversation

@klaymansean
Copy link
Copy Markdown

@klaymansean klaymansean commented Apr 23, 2026

What

  • add domain-skills/google-ads/ui.md
  • document durable Google Ads UI patterns for deep links, ad edits, asset associations, conversions, Google tag settings, and change-history verification
  • capture the key persistence trap where some Ads asset forms visually accept input but silently no-op unless the edit path triggers real user-like input state

Why

Google Ads is an Angular-heavy SPA with generated classes, route-driven state, and several misleading success paths. The main failure mode we hit repeatedly was a save flow that looked successful in the UI but never landed in Change history. This playbook gives the next agent the stable anchors and verification path needed to avoid rediscovering that behavior.

Impact

  • faster Ads navigation via deep links instead of rail-clicking
  • safer verification by treating Change history as the source of truth
  • better distinction between ad-editor fields, which tolerate native setter plus events, and asset forms, which often require real text entry plus blur to persist

Validation

  • distilled from repeated live Browser Harness use against the Google Ads web UI
  • normalized into repo-native domain-skills/<site>/<topic>.md format
  • checked against current upstream browser-harness conventions in domain-skills/

Notes

  • this PR is documentation-only
  • no runtime code changes

Summary by cubic

Add domain-skills/google-ads/ui.md, a field-tested playbook for operating the Google Ads UI. It standardizes deep links, stable selectors, safe text entry, and Change history verification to prevent silent no-ops in asset forms.

  • New Features
    • Deep link patterns for campaigns, ads, assets, conversions, and change history; guidance on ocid and authuser=0.
    • Stable selector strategy and table reading; fix for the Ads status-filter trap that hides rows.
    • Reliable edit patterns: ad editor via setter+events; asset forms (e.g., sitelinks) require real typing+blur; verify via Change history.
    • Coverage for conversions, Google tag settings, auth prompts, popup cleanup, and RTL layouts.

Written for commit 1dbe8d8. Summary will update on new commits.

@browser-harness-review
Copy link
Copy Markdown

✅ Skill review passed

Reviewed 1 file(s) — no findings.

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.

1 participant