Skip to content

feat: Add cwv-optimizer skill for EDS content ops - #186

Merged
trieloff merged 5 commits into
adobe:mainfrom
focusgts:feat/eds-cwv-optimizer
Jul 7, 2026
Merged

feat: Add cwv-optimizer skill for EDS content ops#186
trieloff merged 5 commits into
adobe:mainfrom
focusgts:feat/eds-cwv-optimizer

Conversation

@focusgts

@focusgts focusgts commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds the cwv-optimizer skill to the EDS content ops plugin.

Diagnoses and fixes Core Web Vitals (LCP, CLS, INP) using EDS-specific patterns: the 100KB budget, E-L-D loading phases, and block JavaScript profiling. Produces specific fixes with before/after projections.

Follows the established format (functional description, External Content Safety, concrete code examples, reference file for progressive disclosure, Apache-2.0). Submitted as a standalone PR per @trieloff's request to keep one skill per PR.

Test plan

  • tessl-review passes (≥80

- RUM Bundler API is path-based (/bundles/{domain}/{year}/{month}/{day})
  with ?domainkey=, not a query-string endpoint with a Bearer header
- CWV readings are cwv-lcp/cwv-cls/cwv-inp checkpoints in bundle.events
- Remove the inaccurate "aem-lib issue adobe#201 / fixed in recent versions"
  claim; createOptimizedPicture() simply does not set width/height

Co-Authored-By: claude-flow <ruv@ruv.net>
- EDS auto-serves content images as responsive WebP; stop advising manual
  format conversion/resize of content images (source dimensions are the lever)
- Identify the LCP element via Chrome DevTools / RUM, not structural guessing
- Eager stylesheet is styles/styles.css (there is no styles/aem.css)
- Pull RUM from bundles.aem.page via @adobe/rum-distiller (not rum.hlx.page +
  hand-rolled parsing)
- Flag the A–F grade banding as a heuristic, not an official Adobe grade

Proactive doc-grounded pass ahead of review on adobe#186.
- Add explicit 'Use when...' clause to description (description: 83% → 100%)
- Prefix bare script/style paths with '/' so tessl does not treat them as
  missing local bundle paths (4 warnings → 0 warnings)
- Remove Related Skills section referencing non-existent skills

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Lars Trieloff <lars@trieloff.net>
@trieloff

trieloff commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

6d103f1 removed the reference to the other skills. Reintroduce them when you open PRs for these skills, please.

@trieloff
trieloff merged commit 9414ba6 into adobe:main Jul 7, 2026
8 of 9 checks passed
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

focusgts added a commit to focusgts/skills that referenced this pull request Jul 8, 2026
Trim the Related Skills list to cwv-optimizer (merged); re-add
performance-budget and experiment-designer when those skills land,
per @trieloff review on adobe#186.
@focusgts

focusgts commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

Thanks Lars, and for raising the Tessl score and merging it. Good call on the cross-skill links. I've trimmed optel-interpreter (#188) to only reference what's already on main, and I'll reintroduce the cross-links as each sibling skill lands. Appreciate you taking the time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants