Skip to content

docs(site): add self-hosted page tracker via Cloudflare Worker, drop GoatCounter #9430

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jdx merged 4 commits into from
Apr 27, 2026
Merged

docs(site): add self-hosted page tracker via Cloudflare Worker, drop GoatCounter #9430

Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
e046d21
docs(site): switch analytics to Plausible via Cloudflare Worker proxy
jdx Apr 27, 2026
5bb438c
docs(site): use data-domain/data-api attributes and drop inline init
jdx Apr 27, 2026
75f0bc6
docs(site): restore GTM and set domain to mise.en.dev
jdx Apr 27, 2026
bf02c04
docs(site): use shrill.en.dev as proxy host
jdx Apr 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions docs/.vitepress/config.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -359,7 +359,6 @@ export default withMermaid(
rel: "stylesheet",
},
],
// Analytics

@cursor cursor Bot Apr 27, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

GTM scripts not removed despite stated intent

High Severity

The PR intended to replace Google Tag Manager (GTM) and GoatCounter with Plausible. While GoatCounter was removed, the GTM scripts (loader and inline gtag() initialization) remain in the head array. This results in dual analytics tracking, with both GTM and Plausible firing.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit bf02c04. Configure here.

[
"script",
{
Expand All @@ -378,9 +377,10 @@ export default withMermaid(
[
"script",
{
"data-goatcounter": "https://jdx.goatcounter.com/count",
async: "",
src: "//gc.zgo.at/count.js",
defer: "",
"data-domain": "mise.en.dev",
Comment thread
greptile-apps[bot] marked this conversation as resolved.
"data-api": "https://shrill-1.en.dev/f5f1/event",
src: "https://shrill-1.en.dev/shrill/script.js",
},
Comment on lines +382 to 384

@greptile-apps greptile-apps Bot Apr 27, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 Proxy hostname mismatch vs PR description

The PR description says the Cloudflare Worker is reachable at shrill-1.en.dev, but the actual script src and data-api both point to shrill.en.dev (no -1). If the worker is only deployed at shrill-1.en.dev, the script will 404 and no pageviews will be recorded.

Fix in Claude Code

],
// OpenGraph
Expand Down
Loading