Repository navigation
Retire the Jekyll sources and consolidate the static assets - #28
Merged
Merged
Conversation
- Move the tracked image tree from images/ to web/public/images/ and drop the predev/prebuild sync script; assets are served and exported as-is. - Remove the Jekyll presentation layer from main (_layouts/, _includes/, _sass/, assets/, Gemfile, web/styles/globals.scss); the build compiles no SCSS. The full legacy site is preserved on the archive/jekyll-legacy branch and jekyll.yml stays the manual recovery lever. - Update the design guard, Next config, ignore rules, workflow comments and the maintained documentation; check-design now guards web/public/images/. - Record the change and its verification in the design change record. Verified locally: design guard, eleven calendar tests, TypeScript and static export (25 routes, 21 redirects); the 4102 preview serves every referenced image at HTTP 200 from the new location.
sileneer
added a commit
that referenced
this pull request
Oct 11, 2026
This was referenced Oct 11, 2026
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
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.
Retires the second source tree from
mainand physically merges the duplicated image tree, following the owner-requested audit.Changes
images/moves toweb/public/images/as the single tracked copy.web/scripts/sync-assets.mjsand itspredev/prebuildhooks are deleted, and.gitignoreno longer ignores the copy.main:_layouts/,_includes/,_sass/,assets/,Gemfile,web/styles/globals.scss. The build compiles no SCSS (production CSS isclaude.css+site.css), and no maintained source references the removed paths.images/tree,Gemfile, layouts) is archived on thearchive/jekyll-legacybranch, cut frommainata99f355..github/workflows/jekyll.ymlstays manual-only and is documented as the recovery lever dispatched against that branch. (archive/jekyll-legacyis a permanent archive ref, not a change branch.)gh-pagesbranch is deleted (Pages isbuild_type: workflow; recent deployments all come frommain), and the seven remote branches whose PRs (Show team portraits uncropped and shrink portrait payload #21–Consolidate project guidance and reconcile documentation history #27) were already squash-merged were removed. No unmerged branches remained.AGENTS.md,README.md,web/README.md,web/CUTOVER.md,DESIGN.mdand the design change record updated;check-design.mjsnow guardsweb/public/images/...(the pattern previously only matched the oldimages/...paths).Verification (local, Node 22)
npm run build: design guard passed, eleven calendar tests passed, TypeScript + static export OK (25 routes, 21 legacy redirects).out/images/**file set identical toweb/public/images/(25 files);out/.nojekyllpreserved./images/....git -c core.whitespace=cr-at-eol diff --cached --check); README.md keeps its own index line endings (i/crlf) per AGENTS.md.Not included / notes
_config.ymlis intentionally untouched: it is still read at build time for title/logo, and its inert Jekyll-era keys are out of scope here.Owner review needed: the preview is running on port 4102 — please test before merge.