Skip to content

docs: pin source links to full commit hashes#63

Merged
rubenhensen merged 1 commit into
mainfrom
dobby/pin-source-link-hashes
May 7, 2026
Merged

docs: pin source links to full commit hashes#63
rubenhensen merged 1 commit into
mainfrom
dobby/pin-source-link-hashes

Conversation

@dobby-coder
Copy link
Copy Markdown
Contributor

@dobby-coder dobby-coder Bot commented May 7, 2026

Closes #57.

Three source links in docs/repos/ violated the full-40-char-commit-hash convention:

  • docs/repos/cryptify.md:89 — repointed from /blob/main/src/main.rs to commit 2a6dac195ef4efae8758084aaacab04bc9c94206 (latest commit touching that file).
  • docs/repos/pg-sveltekit.md:124 — expanded short hash 2b29c1ba18 to the full SHA 2b29c1ba18be39d89380a9dd8dd4d3370e17e0b2. Snippet content unchanged; verified against source.
  • docs/repos/pg-sveltekit.md:159 — same hash expansion. While verifying, the in-doc handleSend snippet was missing two lines that exist at +page.svelte L36-L65 (errorMessage = ''; reset and a console.error(...) call). Added them so the snippet faithfully matches the pinned range.

Verification

  • curl -sI returns 200 for all three pinned URLs.
  • Fetched +page.server.ts and +page.svelte at the full SHA — content matches the docs snippets line-for-line after the small +page.svelte correction.
  • Stayed at the pre-consolidation hash for the pg-sveltekit examples per repo memory: snippets cannot simply be repinned across the d6c7f01 consolidation because the API form changed (encryptAndSend wrapper era vs. current pg.open({uuid}).decrypt(...)). A full rewrite against the current API is a separate, larger piece of work.

Reviewer quickstart

```
git fetch origin && git checkout dobby/pin-source-link-hashes && npm install && npm run docs:build
```
Local build succeeded in 24s.

- cryptify.md: pin src/main.rs link to commit 2a6dac1 instead of /blob/main/
- pg-sveltekit.md: expand short hashes (2b29c1ba18) to full 40-char SHAs
- pg-sveltekit.md: align +page.svelte handleSend snippet with source at L36-L65

Closes #57
@dobby-coder dobby-coder Bot requested a review from rubenhensen May 7, 2026 15:55
@rubenhensen rubenhensen merged commit 508182d into main May 7, 2026
1 check passed
@dobby-coder dobby-coder Bot deleted the dobby/pin-source-link-hashes branch May 7, 2026 20:02
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.

docs: pin three source links to full commit hashes (cryptify.md, pg-sveltekit.md)

1 participant