claude/priceless-faraday-314fa0
safeviewing older commitb1d323d · incrementalPR #293reviewed 2026-07-11 03:58 UTC0H · 0M · 1L · 2I- Purpose
- Replace GHA seo-checks.yml gate with advisory SEO lens in Loop C review panel per team decision (Alex): SEO checks should be part of code review, not CI, and web-only.
- Goal
- Move SEO enforcement from a blocking CI gate to an advisory review-panel lens that self-skips on non-web PRs
- Sub-goals
- SG-1: Remove seo-checks.yml and lighthouse-budget.json from GHA
- SG-2: Add 'seo' lens to review.lenses in loops-engine/config.json
- SG-3: Define SEO lens behavior in review-panel/SKILL.md (web-only gate, advisory severity cap)
- SG-4: Update docs (e2e-testing.md, measurement-and-cadence.md, autonomous-operating-model.md, content-pipeline.yml)
- What
- Deleted seo-checks.yml + lighthouse-budget.json; added seo lens to config and SKILL.md; updated docs. Latest commit (b1d323d7) splits SEO lens checklist into sub-bullets and clarifies previewUrlTemplate vs web SEO URL distinction.
- Why
- SEO gate was non-required, ran on all PRs (including platform/domain), and blocked CI. Moving it to the review panel makes it advisory, web-only, and contextual.
- Areas
- .claude/rules+15−9.claude/skills+15−10.github/workflows+1−1docs+13−12e2e/web+2−2scripts+6−2
- Blast
- 12 files, +52/-36 across .claude/, docs/, scripts/, e2e/, .github/ — all documentation/config, no production code
Findings · 3
improvement3
'Note' callout in Deterministic cross-check buries the action instruction
.claude/skills/review-panel/SKILL.md
The bold Note: sentence about the Vercel project distinction should be a separate sub-bullet so the action ('run seo-assert.mjs') and the caveat are independently scannable.
Preview URL note could reference slug-derivation rule
.claude/skills/review-panel/SKILL.md
The note warns not to reuse review.previewUrlTemplate but doesn't say how to derive the slug. A one-phrase pointer ('slug = branch name slugified: / → -, lowercase') would make it self-contained.
Links bullet loses 'broken-link' qualifier
.claude/skills/review-panel/SKILL.md
Original: 'a static, best-effort broken-link pass'; new: 'a static, best-effort pass'. Without 'broken-link', 'pass' reads as a verdict rather than a check type.