Skip to content

feat: add notform module - #1518

Open
favorodera wants to merge 19 commits into
nuxt:mainfrom
favorodera:feature/add-notform
Open

feat: add notform module#1518
favorodera wants to merge 19 commits into
nuxt:mainfrom
favorodera:feature/add-notform

Conversation

@favorodera

Copy link
Copy Markdown

🔗 Linked issue

📚 Description

This PR adds the notform-nuxt Nuxt module to the modules registry.

@coderabbitai

coderabbitai Bot commented Jun 21, 2026

Copy link
Copy Markdown

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

Adds a new notform Nuxt module metadata entry with package, repository, documentation, maintainer, and Nuxt 4 compatibility details. Adds Bluesky profile fields to nuxt-auto-form and nuxt-open-fetch. Updates vitalizer to require Nuxt 4 or newer.

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

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning The Bluesky metadata additions and the vitalizer compatibility change are unrelated to the NotForm listing requested by issue #1503. Remove the unrelated changes to modules/nuxt-auto-form.yml, modules/nuxt-open-fetch.yml, and modules/vitalizer.yml.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: adding the NotForm Nuxt module.
Description check ✅ Passed The description directly explains that the PR adds the notform-nuxt module to the Nuxt modules registry.
Linked Issues check ✅ Passed The PR adds the requested NotForm module metadata and declares Nuxt 4 compatibility as required by issue #1503.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ 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.

@coderabbitai coderabbitai 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@modules/nuxt-charts.yml`:
- Line 4: Update the npm package identifier in the nuxt-charts module metadata
from nuxt-charts-legacy to the published nuxt-charts package, keeping the
existing module configuration unchanged.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e565d7b3-8e83-4adc-ad58-c7076aa9c943

📥 Commits

Reviewing files that changed from the base of the PR and between 7372bcd and 09554d0.

📒 Files selected for processing (1)
  • modules/nuxt-charts.yml

Comment thread modules/nuxt-charts.yml
description: Nuxt module for vue-chrts
repo: dennisadriaans/vue-chrts#main/packages/nuxt-charts
npm: nuxt-charts
npm: nuxt-charts-legacy

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

npm view nuxt-charts-legacy name version dist-tags --json
npm view nuxt-charts-legacy --json | jq -e '.versions | length > 0'

Repository: nuxt/modules

Length of output: 1142


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== nuxt-charts-legacy registry search (public npm API via web request if available) =="
if command -v curl >/dev/null 2>&1; then
  curl -fsSL "https://registry.npmjs.org/-/v1/search?text=nuxt-charts-legacy&size=20&from=0&quality=0.05&popularity=0.05&maintenance=0.1&text=nuxt-charts-legacy&text=nuxt-charts" 2>/dev/null | jq '{total, hits: [.objects[]? | {package_name: .package.name, version: .package.version}]}' || true
fi

echo
echo "== inspect modules/nuxt-charts.yml and related repo usage =="
cat -n modules/nuxt-charts.yml
rg -n "nuxt-charts-legacy|nuxt-charts" . --glob '!**/.git/**' --glob '!**/node_modules/**' --glob '!**/dist/**' --glob '!**/build/**'

Repository: nuxt/modules

Length of output: 1210


Use the published npm package instead of nuxt-charts-legacy.

nuxt-charts-legacy is unavailable as an npm package in both the repository registry and public npm registry, while the module metadata points to nuxt-charts elsewhere. Registry sync will fail to discover versions/fetch module.json unless this is changed to the actual published package name.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@modules/nuxt-charts.yml` at line 4, Update the npm package identifier in the
nuxt-charts module metadata from nuxt-charts-legacy to the published nuxt-charts
package, keeping the existing module configuration unchanged.

Source: MCP tools

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.

[Module Listing Request]: NotForm.

1 participant