The seo-connect target ran connect.py without sourcing ~/.claude/.env, so GOOGLE_OAUTH_CLIENT_ID/SECRET (documented to live there) never reached os.environ — connect.py aborted telling the user to set what they had set. Mirror fetch.sh's sourcing; add a regression lock.
seo-data — GSC + CrUX data layer for /seo FULL audits
Small, isolated engine that gives the /seo skill real Google data instead of
guesses: Search Console (queries, positions, indexation) and CrUX
(Core Web Vitals field data — real users, not lab simulation). It knows
nothing about SEO scoring; it only turns Google APIs into normalized JSON.
The seo-analyzer agent consumes that JSON in STEP 4 (Core Web Vitals) and
the new "Performance GSC" subsection; the /seo skill selects the account
and property in STEP 0 of a FULL audit (not needed for LOCAL).
Multi-account by design: the token store is keyed by a user-chosen label, and
every call takes --account/--property explicitly. Two audits running at
the same time (two sites, two sessions) never share mutable state — nothing
is written to disk during an audit, only at make seo-connect.
Setup
One-time per Google account:
make seo-connect
This creates ~/.claude/.venv-seo-data/ (isolated venv, deps pinned in
requirements.txt), installs google-auth, google-auth-oauthlib,
requests, then runs connect.py: it opens a browser for OAuth consent and
asks for a label (e.g. client-a) to key the account — pick a name, not
an email, since the store never stores or requests the account's email.
Before running it, set these 3 keys in ~/.claude/.env (the canonical
vault; link.sh only symlinks the repo's .env to it and warns with a
cp .env.example .env hint if it's missing — it never creates the vault
itself):
GOOGLE_OAUTH_CLIENT_ID=<your-client-id>.apps.googleusercontent.com
GOOGLE_OAUTH_CLIENT_SECRET=<your-client-secret>
CRUX_API_KEY=<your-crux-api-key>
GOOGLE_OAUTH_CLIENT_ID/GOOGLE_OAUTH_CLIENT_SECRET— OAuth2 "Desktop app" credentials from the Google Cloud Console (APIs & Services → Credentials). Shared across every account you connect; the OAuth scope requested ishttps://www.googleapis.com/auth/webmasters.readonlyonly — read-only Search Console, nothing can be modified or deleted via this token.CRUX_API_KEY— a Chrome UX Report API key (restrict it to CrUX + PageSpeed in the Console). Get one at https://developer.chrome.com/docs/crux/api. No OAuth involved: CrUX is public field data, gated by API key only, independent of any connected account.
make seo-connect is idempotent and rerunnable — connecting a second
account just runs it again with a different label; reusing an existing
label prompts to overwrite.
fetch.sh contract
lib/seo-data/fetch.sh is the one stable entrypoint analyzers call. It
sources ~/.claude/.env, prefers the isolated venv (falls back to system
python3 for stdlib-only paths), dispatches to google_seo.py or
tokenstore.py, and never prints a secret to stdout or stderr.
fetch.sh accounts
→ {"status":"ok","accounts":[{"label":"…","properties":[…],"granted_at":"…"}]} # [] if none connected
fetch.sh crux --url https://ex.com [--strategy mobile|desktop]
→ {"status":"ok","source":"crux","lcp_p75_ms":…,"inp_p75_ms":…,"cls_p75":…} # a missing metric omits its key
→ {"status":"degraded","reason":"no_crux_key"|"no_field_data"|"rate_limited"}
# a 404 on page-level data retries at origin-level before degrading
fetch.sh queries --account client-a --property sc-domain:ex.com [--days 90] [--dim query|page]
→ {"status":"ok","source":"gsc","dimension":"query","rows":[{"key":"…","clicks":…,"impressions":…,"ctr":…,"position":…}]}
→ {"status":"degraded","reason":"no_credentials"|"token_revoked"|"network_error"|"rate_limited"}
fetch.sh inspect --account client-a --property … --url https://ex.com/page
→ {"status":"ok","source":"gsc","indexed":true,"coverage":"…","last_crawl":"…"}
→ {"status":"degraded","reason":"…"}
Rules that hold for every subcommand:
- JSON always on stdout, never empty. Even an unexpected error (HTTP
403/5xx, timeout, DNS failure) prints
{"status":"degraded","reason":"unexpected_error"}— never a raw traceback. statusis"ok"or"degraded"on exit 0;"error"on exit 2. Analyzers branch on this field;"error"only shows up on bad usage,reasonis informational otherwise.- Exit code 0 on
okand ondegraded. The engine never fails the process just because Google data isn't available — that's a normal, expected outcome the analyzer handles by falling back. Exit code 2 is reserved for bad usage: unknown subcommand, missing required flag, invalid argument — those paths emit{"status":"error",...}instead. --storeis accepted uniformly by every subcommand for consistentfetch.shdispatch, even thoughcruxignores it (CrUX needs no account).- Never prints a secret. No env var, refresh token, or access token ever reaches stdout or stderr, including in error paths.
Two env vars exist for testing, never for normal use:
SEO_DATA_ENV_FILE overrides which env file is sourced (tests point it at
/dev/null so a real ~/.claude/.env on the machine can never leak into a
test run), and SEO_DATA_DEBUG=1 re-enables stderr for local debugging
(stderr is suppressed by default so library warnings can't leak a secret
into an agent's context).
Token store
~/.claude/seo-data/tokens.json — refresh tokens, keyed by the label chosen
at make seo-connect, one entry per connected account:
{
"version": 1,
"accounts": {
"client-a": {
"refresh_token": "<opaque>",
"scopes": ["https://www.googleapis.com/auth/webmasters.readonly"],
"granted_at": "2026-07-09T12:00:00+00:00",
"properties": ["sc-domain:site-a.com", "https://www.site-a.com/"]
}
}
}
Security posture:
- File
0600, directory0700.tokenstore.save_accountre-asserts both permissions on every write. - Written only at
connecttime, atomically.tmp→fsync→os.replace(atomic rename), under an exclusivefcntllock, so two simultaneousmake seo-connectruns can't corrupt the file. Audits never write to this file — access tokens are exchanged in memory and never persisted, so two audits running concurrently never contend on it. - Keyed by label, not email. Identifying accounts by email would
require widening the OAuth scope just for identification; the label the
user picks at connect time is sufficient and keeps the scope at
webmasters.readonlyonly (least privilege). - Refresh tokens are redacted from
list.fetch.sh accounts(andtokenstore.py list) return label, properties, andgranted_atonly — therefresh_tokenfield is intentionally never included in that output. - Allowlisted in gitleaks. The store lives under
~/.claude/, outside this repo, so it's never committed directly — butmake scan-secretsalso sweeps~/.claudefor stray copies of secrets..gitleaks.tomlhas an explicit[allowlist].pathsentry for(^|/)\.claude/seo-data/tokens\.json$, the same treatment~/.claude/.envalready gets, so a legitimate local secret store doesn't drown real findings in false positives. - Also gitignored (
.venv-seo-data/andseo-data/tokens.jsonin.gitignore) as a second, belt-and-suspenders guard in case a relative path ever put either under the repo tree.
Graceful degradation
Missing API key, no connected account, or a revoked/expired token is a normal outcome, not a failure:
- No
CRUX_API_KEY→cruxreturns{"status":"degraded","reason":"no_crux_key"}. - No account connected, or the store has no refresh token for the given
--account→queries/inspectreturn{"status":"degraded","reason":"no_credentials"}. - Refresh token revoked at Google's end →
{"status":"degraded","reason":"token_revoked"}(a transient network blip during refresh is classified"network_error"instead, so a flaky connection never forces the user back through OAuth). - Rate limited (HTTP 429) on any Google API →
{"status":"degraded","reason":"rate_limited"}.
In every case: exit code 0, valid JSON on stdout, no crash. The /seo
FULL audit continues on the anonymous PageSpeed API (lab data) instead of
CrUX field data, and the report surfaces the fix as a user action:
make seo-connect. doctor.sh also flags both non-fatally as WARN: a
missing CRUX_API_KEY warns on its own, while no connected Google account
is the one that names make seo-connect.
Testing
make test
# or, to run only this engine's suite:
bash lib/seo-data/seo-data.test.sh
The suite is network-free: google_seo.py reads fixtures from
lib/seo-data/fixtures/ (crux_mobile.json, gsc_queries.json,
gsc_inspect.json) whenever SEO_DATA_MOCK_DIR is set, instead of calling
Google's APIs. Degradation paths run with real env vars unset (env -u CRUX_API_KEY, env -u SEO_DATA_MOCK_DIR) to exercise the no-key/no-creds
branches deterministically. Every fetch.sh invocation in the tests also
sets SEO_DATA_ENV_FILE=/dev/null so a machine with a live
~/.claude/.env never lets real credentials leak into a test run.