Skip to content

feat: add pulse-analytics - #1618

Open
uz1mani wants to merge 1 commit into
nuxt:mainfrom
uz1mani:feat/pulse-analytics
Open

uz1mani wants to merge 1 commit into
nuxt:mainfrom
uz1mani:feat/pulse-analytics

Conversation

@uz1mani

@uz1mani uz1mani commented Sep 16, 2026

Copy link
Copy Markdown

Adds @ciphera-net/pulse-nuxt, a Nuxt 3/4 module for Pulse Analytics. Apache-2.0, zero runtime dependencies.

The module pushes a <script src> into app.head. Nothing of it reaches the client bundle.

npm: https://www.npmjs.com/package/@ciphera-net/pulse-nuxt
Repo: https://github.com/ciphera-net/pulse-nuxt

Entry

Generated with pnpm sync pulse-analytics ciphera-net/pulse-nuxt, then pnpm lint and pnpm build. All green.

I ran two controls first so the green meant something: an invalid category and a missing icon file each make sync fail.

compatibility.nuxt comes from the package's own dist/module.json.

Compatibility

^3.0.0 || ^4.0.0, checked against 3.21.11 and 4.5.2 by building a site on each and reading the tag out of the emitted HTML, then loading that page in Chromium to confirm hydration does not add a second tag. 40 assertions. Harness: scripts/verify-build.mjs in the module repo.

The range is bounded rather than >=3.0.0 because I have not tested a major that does not exist yet.

Icon

icons/pulse-analytics.png, 512x512 RGBA, same as matomo.png and wideangle.png. No SVG mark exists yet; I can send one later.

@github-actions

Copy link
Copy Markdown
Contributor

We've flagged this as a potential contribution without a human behind it. We welcome the thoughtful use of AI tools when contributing, but ask all contributors to follow two core principles:

  1. Never let an LLM speak for you - all comments, issues, and PR descriptions should be written in your own words, reflecting your own understanding.
  2. Never let an LLM think for you - only submit contributions you fully understand and can explain.

Please review these AI-assisted contribution guidelines and update this contribution if needed.

If this was flagged in error, we apologise! 😳 Just let us know. 🙏

@coderabbitai

coderabbitai Bot commented Sep 16, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 182a8d2f-f6eb-47b0-afad-7b0a29584fc8

📥 Commits

Reviewing files that changed from the base of the PR and between 1a1430b and 11d15bc.

⛔ Files ignored due to path filters (1)
  • icons/pulse-analytics.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • modules/pulse-analytics.yml

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

Adds a Nuxt module catalog manifest for pulse-analytics. The manifest includes module metadata, package and repository links, maintainer information, classification, and compatibility with Nuxt 3 and Nuxt 4.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 11d15

The catalog manifest matches the supplied repository contracts and presents no established merge-blocking behavior.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the primary change: adding the pulse-analytics module manifest.
Description check ✅ Passed The description directly explains the added @ciphera-net/pulse-nuxt module, its compatibility, implementation, icon, and validation results.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@uz1mani

uz1mani commented Sep 16, 2026

Copy link
Copy Markdown
Author

Hi, the PR was indeed written using AI but all the input is mine.

uz1mani added a commit to ciphera-net/pulse-nuxt that referenced this pull request Sep 21, 2026
Everything Ciphera-related is a Plane work item since the standing owner rule
of 20-09-2026. This repo's work is PULSE-3, In Progress in cycle W39 Q3, and
the doc attachment is owed at Shipped — which cannot happen while
nuxt/modules#1618 is held by the possible bot label.
@uz1mani
uz1mani force-pushed the feat/pulse-analytics branch from 11d15bc to b39aa33 Compare September 21, 2026 09:14

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant