Skip to content

fix(docs): restore canonical Umami analytics - #815

Merged
jeremi merged 4 commits into
mainfrom
fix/docs-umami-analytics
Aug 23, 2026
Merged

fix(docs): restore canonical Umami analytics#815
jeremi merged 4 commits into
mainfrom
fix/docs-umami-analytics

Conversation

@jeremi

@jeremi jeremi commented Aug 23, 2026

Copy link
Copy Markdown
Member

Summary

  • restore Umami on the canonical released documentation with the Registry Docs website ID stored as a reproducible source input
  • keep analytics disabled on /dev and immutable /v/<version> archives, and fail release archive builds when that boundary is violated
  • add an accessible Privacy page describing the docs analytics events, hosting data, retention, and visitor rights

Why

The hermetic release archive builder clears deployment-provided PUBLIC_UMAMI_* values. RegistryHead.astro previously 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:

  • website ID: 0a8aa090-83c5-4207-8c90-9fcc1e50bb78
  • script: https://stats.registrystack.org/script.js
  • domain: docs.registrystack.org

Verification

  • npm test: 450 tests passed
  • npm run check: passed, including all tutorial dry runs, accessibility, SEO, LLM output, and 18,829 internal links and assets
  • DOCS_DOCSET=v0.25.0 npm run build:archive: passed against the real release path
  • release artifact verification: canonical root contains exactly the expected tracker; versioned archive contains no analytics
  • final privacy edit: content, Markdown, Vale, and Astro build checks passed

The 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.org when the next Registry Stack release archive containing this fix is published.

Signed-off-by: Jeremi Joslin <jeremi@joslin.fr>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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".

Comment thread docs/site/src/lib/analytics.mjs
Comment thread docs/site/src/content/docs/privacy.mdx Outdated
Comment thread docs/site/src/content/docs/privacy.mdx Outdated
Signed-off-by: Jeremi Joslin <jeremi@joslin.fr>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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".

Comment thread docs/site/src/content/docs/privacy.mdx Outdated
jeremi added 2 commits August 23, 2026 20:27
Signed-off-by: Jeremi Joslin <jeremi@joslin.fr>
Signed-off-by: Jeremi Joslin <jeremi@joslin.fr>
@jeremi
jeremi merged commit 873b221 into main Aug 23, 2026
34 checks passed
@jeremi
jeremi deleted the fix/docs-umami-analytics branch August 23, 2026 18:34
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.

1 participant