Repository navigation
docs(garage): two study pages cite a pipeline at a path that is three moves old - #571
Merged
Merged
Conversation
… moves old /garage/encoding names the photo pipeline four times and every one of them still says `www/scripts/`: the sentence "the live pipeline (www/scripts/add-photos.sh)" plus three cites of `www/scripts/zenc/`, the Rust wrapper the page's whole corpus study is an argument for. Both are `tools/photos/` now, after www/scripts to scripts/ to tools/photos (#452, #453). Present tense, on a public page, pointing at nothing. /garage/horizon's JXL section priced a third format tier as "another stem format in www/images/". That one is NOT a literal rename: `public/images/` holds the photo DATA surfaces (metadata, exif, alt text, hashes) while the pixel tiers have lived in content-addressed `public/i/` since 2026-07-03, so the sentence names the directory a JXL tier would actually land in. The "120 stems" count is left alone deliberately: it is the site's photo count when the study was written, and updating a measured number without re-running the study invents data. /garage/workers is UNTOUCHED on purpose. It quotes `"main": "www/_worker.js/index.js"` and names `www/_worker.js/counter.js`, but that page is about the Pages-to-Workers migration and the old config IS its subject matter. Correcting it would falsify the page. Both files are hand-authored rather than generated: pipelines/garage/pages.json covers six pages (pqc, octane, useragent, typed-config, hidden-flags, ig-prep) and neither of these is one, so editing src/pages directly is the whole edit. Page bytes move, so both p-dict snapshots go stale until the next roll, which is what that roll is for. Build clean: 51 documents, 3015 refs resolve, 33 understanding payloads byte-equal, no tripwires.
Contributor
Wire-size diff vs merge base
Brotli q11, the quality the edge serves. Every number here is deterministic: Client assetsNo change. 22 files, 77.80 KiB Brotli total. PagesTotal Brotli: 552.47 KiB → 552.50 KiB (+0.03 KiB, +0.0%) across 52 files, 2 changed. No single file moved by 0.13 KiB or more. Plus 2 files under the 0.13 KiB floor, net +0.03 KiB — usually content-hash churn in Worker bundleNo change. 238.20 KiB gzip. Dictionary deltas (
|
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.
The page-copy half of #569, which fixed the comments and deliberately left these.
/garage/encoding
Four cites of the photo pipeline, all still at
www/scripts/:www/scripts/add-photos.sh)"tools/photos/add-photos.shwww/scripts/zenc/x3tools/photos/zenc/zencis the Rust wrapper this page's entire corpus study is an argument for,so the three dead cites are on the sentences doing the most work. The path has
moved twice since (
www/scriptstoscripts/totools/photos/, #452 and#453), and the prose is present tense throughout.
/garage/horizon
The JXL section priced a third format tier as "another stem format in
www/images/". This one is not a literal rename.public/images/holds thephoto DATA surfaces (metadata, exif, alt text, hashes); the pixel tiers have
lived in content-addressed
public/i/since 2026-07-03. So the correctedsentence names the directory a JXL tier would actually land in rather than the
one the old path maps to.
The "120 stems" count beside it is left alone on purpose: that is the site's
photo count when the study was written (158 now), and updating a measured number
without re-running the study invents data.
/garage/workers, untouched on purpose
It quotes
"main": "www/_worker.js/index.js"and nameswww/_worker.js/counter.js. That page is ABOUT the Pages-to-Workers migration,so the old config is its subject matter and correcting it would falsify the
page.
notes
Both edited files are hand-authored, not generated:
pipelines/garage/pages.jsoncovers six pages (pqc, octane, useragent, typed-config, hidden-flags, ig-prep)
and neither of these is one, so editing
src/pages/directly is the whole edit.Page bytes move, so both p-dict snapshots go stale until the next nightly roll.
Checked:
bun run test369/369,bun run pages:check,bun run lint, and afull build (51 documents, 3015 refs resolve, 33 understanding payloads
byte-equal, no tripwires). Confirmed in the SERVED bytes:
.buildcopies of bothpages carry the new paths and zero
www/.