Skip to content

Optimize source photos and compress PNG graphics losslessly - #17

Merged
silveltman merged 2 commits into
mainfrom
chore/source-photo-standard
Oct 6, 2026
Merged

silveltman merged 2 commits into
mainfrom
chore/source-photo-standard

Conversation

@silveltman

@silveltman silveltman commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Converts 1 existing source photographs to actual WebP quality 90, with a maximum of 3840px on the longest edge, correct orientation, preserved aspect ratio and no upscaling. Already compliant WebP photographs are unchanged. Photo format is checked from decoded bytes, including PNG photographs with misleading .jpg or .webp extensions. Graphics, screenshots and animations retain suitable formats. Literal references are updated when files are renamed.

Changed source files: 0.33 MB to 0.18 MB, saving 0.15 MB. Git history and original working copies retain the originals; this does not immediately reclaim that much local disk space.

Validation: every converted output decodes as WebP with the recorded dimensions and within the 3840px ceiling; removed filenames and literal references checked; diff whitespace check passed. The latest-commit Check workflow passed. The latest-commit changed-page preview was checked in headless Chrome at 1280px and 412px widths, with no failed loaded image elements. This is a page spot check, not a complete crawl or an actual iPhone test.

Latest-commit Deploy Preview. Verified commit: b1b6c78d041fb06b47e4048795cbdc2698535ba7.

No production changes have been made. No dependency upgrades, hosting/access repairs or Git-history cleanup are included.

Lossless PNG compression

1 PNG assets: 393,148 to 361,364 bytes. Oxipng rewrites compression only. Every changed file was decoded before and after and verified byte-for-byte equal, including alpha. Dimensions, bit depth and rendering metadata (including ICC profiles) are preserved. No lossy quality reduction or format conversion.

Latest commit 9ecbd4c18238efb03eb672c87ecbcf9d7ef9b73f: native Check workflow passed. Exact latest-commit Netlify preview. Native Netlify build passed. One changed page checked at 1280 and 412 pixels with no broken loaded images. This is a spot check, not coverage of every page or lazy image.

Release validation · 6 October 2026

Sil explicitly authorized this merge. Source head 9ecbd4c18238efb03eb672c87ecbcf9d7ef9b73f was reviewed against actual main 0ef8f432de37ff1b124f39e42ba41c67b3dd64d8 in an isolated synthetic integration. Frozen install, pnpm check and pnpm build passed. The CodeRabbit CLI checkpoint completed with zero findings; binary assets excluded by that reviewer were decoded and checked separately. PNG raw pixels remained identical, WebP outputs decode correctly and respect the 3840-pixel source ceiling. Existing GitHub and Netlify checks passed and no unresolved review threads remained.

Squash merge a5241595e3b155403b908e34eab74d1cf6a8d23f has tree bb25b162679eb8bedbf70409c740a9e006f1c96b, identical to the locally validated integration. Netlify production deploy 6ac5097f3dce2c0008353141 is ready and published for that exact merge commit. Mobile and desktop browser checks passed with no broken loaded images or horizontal overflow. This is spot coverage, not a complete crawl or physical iPhone test.

@silveltman
silveltman marked this pull request as ready for review October 2, 2026 17:10
@netlify

netlify Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for silveltman ready!

Name Link
🔨 Latest commit 9ecbd4c
🔍 Latest deploy log https://app.netlify.com/projects/silveltman/deploys/6abffa71a401ef000880b03b
😎 Deploy Preview https://deploy-preview-17--silveltman.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 20befafe-40cb-44eb-9927-e0af4862312e

📥 Commits

Reviewing files that changed from the base of the PR and between a22b6e1 and b1b6c78.

⛔ Files ignored due to path filters (1)
  • src/assets/sil-veltman.jpg is excluded by !**/*.jpg
📒 Files selected for processing (2)
  • src/assets/sil-veltman.webp
  • src/content/layouts/index.yaml

Limit details: You’ve used all 10 included reviews currently available.


📝 Walkthrough

Walkthrough

The layout image source changes from /sil-veltman.jpg to /sil-veltman.webp.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to b1b6c

The layout image points to a valid WebP asset, with no remaining references to the replaced JPEG. No actionable merge-blocking risk is evident.

Architecture Summary

Architecture risk: 🔵 Low · up to b1b6c

The change affects 1 system.

Changed systems: src

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — src (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in src/content/layouts/index.yaml: The image source path changes from /sil-veltman.jpg to /sil-veltman.webp.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly describes the source photo format change and lossless PNG compression stated in the pull request description. It is specific and related to the changeset.
Description check ✅ Passed The description directly explains the WebP conversion, PNG compression, validation steps, file-size savings, and preview checks related to this changeset.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Comment @coderabbitai help to get the list of available commands.

@silveltman silveltman changed the title Standardize source photographs as WebP within 3840 pixels Optimize source photos and compress PNG graphics losslessly Oct 2, 2026
@silveltman
silveltman merged commit a524159 into main Oct 6, 2026
7 checks passed
@silveltman
silveltman deleted the chore/source-photo-standard branch October 6, 2026 14:45
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