feat: add notform module - #1518
Conversation
- add notform icon - add notform module metadata
|
Note Reviews pausedIt 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 Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughAdds a new Estimated code review effort: 1 (Trivial) | ~5 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
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
📒 Files selected for processing (1)
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 |
There was a problem hiding this comment.
🗄️ 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
🔗 Linked issue
NotForm.#1503📚 Description
This PR adds the
notform-nuxtNuxt module to the modules registry.