Skip to content

chore: add plausible analytics#451

Merged
jdx merged 2 commits intomainfrom
claude/plausible-analytics
Apr 27, 2026
Merged

chore: add plausible analytics#451
jdx merged 2 commits intomainfrom
claude/plausible-analytics

Conversation

@jdx
Copy link
Copy Markdown
Owner

@jdx jdx commented Apr 27, 2026

Summary

  • Adds Plausible Analytics to the docs site via the shrill.en.dev Cloudflare worker proxy

🤖 Generated with Claude Code


Note

Low Risk
Low risk docs-only change that injects a third-party analytics script; primary risk is unintended tracking/script-loading issues in the docs build/runtime.

Overview
Adds a deferred analytics <script> tag to the docs site (docs/.vitepress/config.mjs) head configuration, configured with data-domain=fnox.jdx.dev and a shrill.en.dev worker data-api endpoint.

Reviewed by Cursor Bugbot for commit c0d303a. Bugbot is set up for automated code reviews on this repo. Configure here.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 27, 2026

Greptile Summary

Injects a single deferred <script> tag for Plausible Analytics into the VitePress head config, routing events through the shrill.en.dev Cloudflare worker proxy. The change is minimal and self-contained — data-domain correctly references the site domain, and the proxy URLs appear intentional for ad-blocker bypass.

Confidence Score: 5/5

Safe to merge — single-line config change with no logic, security, or correctness issues.

The only changed file is a VitePress config that adds a well-formed, deferred analytics script tag. The previously flagged data-domain value has been corrected to fnox.jdx.dev. No P0 or P1 findings.

No files require special attention.

Important Files Changed

Filename Overview
docs/.vitepress/config.mjs Adds a deferred Plausible Analytics script tag routed through the shrill.en.dev Cloudflare worker proxy with the correct site domain (fnox.jdx.dev).

Sequence Diagram

sequenceDiagram
    participant Browser
    participant shrill.en.dev (Proxy)
    participant Plausible Analytics

    Browser->>shrill.en.dev (Proxy): GET /shrill/script.js (deferred)
    shrill.en.dev (Proxy)-->>Browser: analytics script
    Browser->>shrill.en.dev (Proxy): POST /f5f1/event (pageview data, domain=fnox.jdx.dev)
    shrill.en.dev (Proxy)->>Plausible Analytics: Forward event
    Plausible Analytics-->>shrill.en.dev (Proxy): 202 Accepted
    shrill.en.dev (Proxy)-->>Browser: 202 Accepted
Loading

Reviews (2): Last reviewed commit: "fix: use site domain for plausible data-..." | Re-trigger Greptile

Comment thread docs/.vitepress/config.mjs Outdated
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the VitePress configuration in docs/.vitepress/config.mjs to include a new script tag for analytics. The script is configured with deferral and specific data attributes for domain and API tracking. I have no feedback to provide.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@jdx jdx enabled auto-merge (squash) April 27, 2026 16:12
@jdx jdx merged commit c8f00a9 into main Apr 27, 2026
16 checks passed
@jdx jdx deleted the claude/plausible-analytics branch April 27, 2026 16:33
NorthIsUp pushed a commit to NorthIsUp/fnox that referenced this pull request Apr 28, 2026
## Summary

- Adds Plausible Analytics to the docs site via the shrill.en.dev
Cloudflare worker proxy

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Low risk docs-only change that injects a third-party analytics script;
primary risk is unintended tracking/script-loading issues in the docs
build/runtime.
> 
> **Overview**
> Adds a deferred analytics `<script>` tag to the docs site
(`docs/.vitepress/config.mjs`) head configuration, configured with
`data-domain=fnox.jdx.dev` and a `shrill.en.dev` worker `data-api`
endpoint.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
c0d303a. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
jdx pushed a commit that referenced this pull request May 2, 2026
### 🚜 Refactor

- extract providers and core types into fnox-core crate by
[@jdx](https://github.com/jdx) in
[#458](#458)

### 📚 Documentation

- prefix star count with ★ glyph and populate it on deploy by
[@jdx](https://github.com/jdx) in
[#447](#447)
- add favicons and web app manifest by [@jdx](https://github.com/jdx) in
[#448](#448)

### 🔍 Other Changes

- **(docs)** remove shrill.en.dev analytics script by
[@jdx](https://github.com/jdx) in
[#457](#457)
- **(release)** add musl Linux targets for Alpine compatibility by
[@jdx](https://github.com/jdx) in
[#452](#452)
- add plausible analytics by [@jdx](https://github.com/jdx) in
[#451](#451)
- bump hk to 1.44.3 by [@jdx](https://github.com/jdx) in
[#454](#454)

### 📦️ Dependency Updates

- update autofix-ci/action action to v1.3.4 by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#455](#455)
- update apple-actions/import-codesign-certs action to v7 by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#456](#456)
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