Skip to content

test(perf): record route-profile media dimensions [Codex] - #1379

Open
jderochervlk wants to merge 1 commit into
test/route-family-performance-baselinesfrom
test/route-profile-media-dimensions
Open

jderochervlk wants to merge 1 commit into
test/route-family-performance-baselinesfrom
test/route-profile-media-dimensions

Conversation

@jderochervlk

@jderochervlk jderochervlk commented Sep 22, 2026 •

Copy link
Copy Markdown
Collaborator

Part of #1371.

This follow-up completes the route-profile report’s media evidence without changing page behavior.

Each built profile now retains one entry per image and video with its declared source, width, and height. Missing or invalid dimensions are represented as null, alongside the existing aggregate missing-dimension counts.

The coverage verifies declared image dimensions, absent video dimensions, real production-build output, and the existing aggregate asset contract.

@jderochervlk
jderochervlk added this pull request to stack #1380 September 22, 2026 11:55
@jderochervlk jderochervlk changed the title test(perf): record route-profile media dimensions test(perf): record route-profile media dimensions [Codex] Sep 22, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-22T13:31:34.963369Z e585faa New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions

github-actions Bot commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Lighthouse route profiles

Compared with target branch test/route-family-performance-baselines at commit b77606d. Every score is the median of the deployed preview runs for that route.

Route Performance Accessibility Best practices SEO
/ 84 -> 83 (-1) 80 -> 80 (0) 100 -> 100 (0) 58 -> 58 (0)
/docs/manual/introduction 77 -> 76 (-1) 78 -> 78 (0) 100 -> 100 (0) 50 -> 50 (0)
/docs/manual/api/stdlib/array 74 -> 76 (+2) 77 -> 77 (0) 100 -> 100 (0) 50 -> 50 (0)
/blog 78 -> 77 (-1) 80 -> 80 (0) 100 -> 100 (0) 50 -> 50 (0)
/blog/reactive-analysis 83 -> 85 (+2) 78 -> 78 (0) 100 -> 100 (0) 50 -> 50 (0)
/community/overview 83 -> 88 (+5) 85 -> 85 (0) 100 -> 100 (0) 58 -> 58 (0)
/packages 76 -> 74 (-2) 86 -> 86 (0) 96 -> 96 (0) 45 -> 45 (0)
/syntax-lookup 85 -> 83 (-2) 83 -> 83 (0) 100 -> 100 (0) 58 -> 58 (0)
/try 26 -> 24 (-2) 86 -> 86 (0) 96 -> 96 (0) 92 -> 92 (0)
/brand 87 -> 87 (0) 83 -> 83 (0) 100 -> 100 (0) 54 -> 54 (0)
/__route-profile-not-found 94 -> 94 (0) 82 -> 82 (0) 100 -> 100 (0) 42 -> 42 (0)

Download the full Lighthouse reports and baselines

Commit d186002

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
guide e585faa Commit Preview URL

Branch Preview URL
Sep 22 2026, 01:29 PM

Expose per-image and per-video declared dimensions in the built-site report.
@github-actions

Copy link
Copy Markdown
Contributor

Docs preview

This branch was successfully deployed

2 active deployments
preview — e585faa2 Deployed Sep 22, 2026 by github-actions[bot]
guide-preview — e585faa2 Deployed Sep 22, 2026 by github-actions[bot]
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