fix(docs): restore canonical Umami analytics - #815
Conversation
Signed-off-by: Jeremi Joslin <jeremi@joslin.fr>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2d7dc36b91
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Signed-off-by: Jeremi Joslin <jeremi@joslin.fr>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4abb89f529
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Signed-off-by: Jeremi Joslin <jeremi@joslin.fr>
Signed-off-by: Jeremi Joslin <jeremi@joslin.fr>
Summary
/devand immutable/v/<version>archives, and fail release archive builds when that boundary is violatedWhy
The hermetic release archive builder clears deployment-provided
PUBLIC_UMAMI_*values.RegistryHead.astropreviously required the website ID from that environment, so the canonical archive was published without the tracker even though the public website remained correctly configured.This change preserves hermetic release builds while binding the canonical docs tree to the existing Umami property:
0a8aa090-83c5-4207-8c90-9fcc1e50bb78https://stats.registrystack.org/script.jsdocs.registrystack.orgVerification
npm test: 450 tests passednpm run check: passed, including all tutorial dry runs, accessibility, SEO, LLM output, and 18,829 internal links and assetsDOCS_DOCSET=v0.25.0 npm run build:archive: passed against the real release pathThe existing unrelated Astro deprecation hint, three Vale suggestions, and OpenAPI lint warnings remain non-blocking.
Review and rollout
The Privacy page is Tier C under the docs guidance and requires maintainer sign-off before merge. Its Umami details were checked against the official FAQ and metric definitions.
Because canonical docs are promoted from immutable release archives, merging this PR does not rewrite the currently published archive. Tracking resumes on
docs.registrystack.orgwhen the next Registry Stack release archive containing this fix is published.