Skip to content

feat: add notform module#1518

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

feat: add notform module#1518
favorodera wants to merge 12 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

📝 Walkthrough

Walkthrough

Adds a YAML metadata entry for the notform Nuxt module, including package identifiers, links, maintainer information, categorization, and Nuxt 4 compatibility. Updates the npm identifier in the nuxt-charts metadata from nuxt-charts to nuxt-charts-legacy.

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

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning The nuxt-charts package identifier change is unrelated to the NotForm listing request. Remove the nuxt-charts rename unless it is required by the issue, or explain its necessity in the PR.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding the NotForm module.
Description check ✅ Passed The description matches the PR by stating it adds the notform-nuxt module to the registry.
Linked Issues check ✅ Passed The new notform metadata matches issue #1503 by listing the module, repo, npm package, and Nuxt 4 compatibility.
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