chore: slim public/ to served assets only, OG points at light PNG - #6
Merged
Conversation
…ding-recommended PNG Brand assets now have a canonical home: github.com/claricle/branding. This repo keeps only the subset the site actually serves from public/. Removed from public/ (all preserved in claricle/branding): - logo-claricle.pdf (designer source) - logo-claricle.svg (medium-saturation variant, not referenced by code) - logo-claricle.png (834x834 medium-saturation, was the OG image) - logo-claricle_icon-dark.png (834x834, not referenced — SVG variant is) Switched OG image to /logo-claricle_icon-light.png per the branding README's guidance: social preview cards render on white, and the light variant's saturated colors are correct there. Updated: - BaseLayout.astro: ogImage path - test/site-sanity.spec.ts: replaced the medium-SVG color test with a check that the 834x834 OG PNG exists with correct dimensions - test/build.spec.ts: dist assertion now checks for the OG PNG name - CLAUDE.md and README.md: brand-assets sections rewritten to point at claricle/branding as canonical
The 200x200 PNG at the repo root was the original designer source. It
now lives in claricle/branding as the canonical copy; no point keeping
a duplicate here. The site never served it (only /public/ files are
served) and it was only kept per the never-delete-source rule, which
the user explicitly waived for this duplication.
Dropped the matching spec ("exposes the canonical logo PNG at the repo
root") and updated CLAUDE.md to point at claricle/branding instead.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Brand assets now have a canonical home at
claricle/branding. This PR slimsclaricle.github.io/public/down to only the files the site actually serves.Removed from
public/(all preserved in branding)logo-claricle.pdf(designer source)logo-claricle.svg(medium-saturation variant, not referenced by code)logo-claricle.png(834x834 medium-saturation, was the OG image)logo-claricle_icon-dark.png(834x834, not referenced — only the SVG variant is)Switched OG image
BaseLayoutnow references/logo-claricle_icon-light.pngfor Open Graph, matching the branding README's guidance: social preview cards render on white, the light variant's saturated colors read correctly there.What stays in
public/logo-claricle_icon-light.svg+logo-claricle_icon-dark.svg(theme-aware logo)logo-claricle_icon-light.png(OG image)favicon.svg,favicon-96x96.png,favicon.ico,apple-touch-icon.png)site.webmanifestDocs
CLAUDE.mdandREADME.mdrewritten to point atclaricle/brandingas the source of truth.Note on root
logo-claricle.pngThe 200×200 canonical PNG at the repo root is untouched — preserved per the "never delete source files" rule. It's now duplicated in
claricle/branding. If you want me to remove the root copy in a follow-up, say the word.Test plan
curl -I https://claricle.github.io/logo-claricle_icon-light.pngreturns 200curl -I https://claricle.github.io/logo-claricle.pngreturns 404 (file removed)