-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
docs: add AI-assisted contribution guidelines #32725
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@nuxt/kit
nuxt
@nuxt/rspack-builder
@nuxt/schema
@nuxt/vite-builder
@nuxt/webpack-builder
commit: |
84e8870
to
fc4e510
Compare
WalkthroughA new section regarding AI-assisted contributions was added to both the pull request template and the contribution guidelines. In the pull request template, contributors are now instructed to ensure that any AI-assisted work reflects their own understanding and is written in their own voice. The contribution guidelines expand on this by outlining principles for using AI tools: contributors should not let AI generate communication on their behalf or submit code they do not fully understand. The documentation also invites feedback on improving AI usage in contributions. No changes were made to code or exported entities. Estimated code review effort1 (~5 minutes) 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
✨ Finishing Touches🧪 Generate 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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
.github/PULL_REQUEST_TEMPLATE.md (1)
9-13
: Consider mirroring the two-principle structure used in the contribution guide
The template currently reminds authors to write in their own voice, but it omits the complementary “never let an LLM think for you” point that appears in the guidelines. Adding a second sentence or bullet here would keep messaging consistent across documents and avoid confusion.docs/5.community/4.contribution.md (1)
83-98
: Define the “LLM” acronym on first use
Readers unfamiliar with the term may be left guessing. A quick “large language model (LLM)” after the first occurrence improves clarity without adding noise.-#### Never let an LLM speak for you +#### Never let a large language model (LLM) speak for you
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
.github/PULL_REQUEST_TEMPLATE.md
(1 hunks)docs/5.community/4.contribution.md
(1 hunks)
🧠 Learnings (3)
📓 Common learnings
Learnt from: Tofandel
PR: nuxt/nuxt#0
File: :0-0
Timestamp: 2024-11-11T12:34:22.648Z
Learning: Ensure that AI-generated summaries accurately reflect the key changes in the PR, focusing on notable changes such as the removal of unused imports and variables starting with underscores.
.github/PULL_REQUEST_TEMPLATE.md (1)
Learnt from: Tofandel
PR: nuxt/nuxt#0
File: :0-0
Timestamp: 2024-11-11T12:34:22.648Z
Learning: Ensure that AI-generated summaries accurately reflect the key changes in the PR, focusing on notable changes such as the removal of unused imports and variables starting with underscores.
docs/5.community/4.contribution.md (1)
Learnt from: Tofandel
PR: nuxt/nuxt#0
File: :0-0
Timestamp: 2024-11-11T12:34:22.648Z
Learning: Ensure that AI-generated summaries accurately reflect the key changes in the PR, focusing on notable changes such as the removal of unused imports and variables starting with underscores.
🪛 LanguageTool
docs/5.community/4.contribution.md
[style] ~95-~95: Consider using a less common alternative to make your writing sound more unique and professional.
Context: ... #### Never let an LLM think for you * Feel free to use AI tools to generate code or explor...
(FEEL_FREE_TO_STYLE_ME)
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: Tofandel
PR: nuxt/nuxt#0
File: :0-0
Timestamp: 2024-11-11T12:34:22.648Z
Learning: Ensure that AI-generated summaries accurately reflect the key changes in the PR, focusing on notable changes such as the removal of unused imports and variables starting with underscores.
.github/PULL_REQUEST_TEMPLATE.md (1)
Learnt from: Tofandel
PR: nuxt/nuxt#0
File: :0-0
Timestamp: 2024-11-11T12:34:22.648Z
Learning: Ensure that AI-generated summaries accurately reflect the key changes in the PR, focusing on notable changes such as the removal of unused imports and variables starting with underscores.
docs/5.community/4.contribution.md (1)
Learnt from: Tofandel
PR: nuxt/nuxt#0
File: :0-0
Timestamp: 2024-11-11T12:34:22.648Z
Learning: Ensure that AI-generated summaries accurately reflect the key changes in the PR, focusing on notable changes such as the removal of unused imports and variables starting with underscores.
🪛 LanguageTool
docs/5.community/4.contribution.md
[style] ~95-~95: Consider using a less common alternative to make your writing sound more unique and professional.
Context: ... #### Never let an LLM think for you * Feel free to use AI tools to generate code or explor...
(FEEL_FREE_TO_STYLE_ME)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (20)
- GitHub Check: test-fixtures (windows-latest, dev, vite, default, manifest-on, json, 20)
- GitHub Check: test-fixtures (windows-latest, built, webpack, async, manifest-on, json, 20)
- GitHub Check: test-fixtures (windows-latest, built, vite, default, manifest-off, json, 20)
- GitHub Check: test-fixtures (windows-latest, built, webpack, default, manifest-on, json, 20)
- GitHub Check: test-fixtures (windows-latest, built, vite, async, manifest-off, json, 20)
- GitHub Check: test-fixtures (windows-latest, built, vite, default, manifest-on, json, 20)
- GitHub Check: test-fixtures (windows-latest, built, vite, async, manifest-on, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, built, rspack, default, manifest-on, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, built, webpack, default, manifest-on, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, built, webpack, async, manifest-on, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, built, vite, async, manifest-on, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, dev, vite, async, manifest-off, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, built, vite, default, manifest-on, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, built, vite, async, manifest-on, js, 20)
- GitHub Check: test-fixtures (ubuntu-latest, dev, vite, async, manifest-on, js, 20)
- GitHub Check: test-fixtures (ubuntu-latest, dev, vite, default, manifest-on, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, dev, vite, default, manifest-off, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, dev, vite, async, manifest-on, json, 20)
- GitHub Check: test-benchmark
- GitHub Check: typecheck (windows-latest, bundler)
CodSpeed Performance ReportMerging #32725 will not alter performanceComparing Summary
Footnotes |
Co-authored-by: Daniel Roe <daniel@roe.dev>
add reference to original blog post Co-authored-by: Daniel Roe <daniel@roe.dev>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (3)
docs/5.community/4.contribution.md (3)
85-90
: Spell out “LLM” on first use and promote these to a titled sub-section.Readers unfamiliar with the acronym may stumble, and the two principles currently sit outside the document’s heading hierarchy. A small tweak improves clarity and navigation:
-#### Never let an LLM speak for you +### 🤖 AI-Assisted Contributions + +#### Never let a Large Language Model (LLM) speak for you(The existing bullets under this heading can remain unchanged.)
91-96
: Mirror the acronym expansion in the second heading for consistency.-#### Never let an LLM think for you +#### Never let a Large Language Model (LLM) think for you
97-98
: Minor grammar improvement for smoother reading.-Our aim is ensuring quality and maintaining the joy of collaborating and communicating with real people. +Our aim is to ensure quality while preserving the joy of collaborating and communicating with real people.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/5.community/4.contribution.md
(1 hunks)
🪛 LanguageTool
docs/5.community/4.contribution.md
[style] ~93-~93: Consider using a less common alternative to make your writing sound more unique and professional.
Context: ... #### Never let an LLM think for you * Feel free to use AI tools to generate code or explor...
(FEEL_FREE_TO_STYLE_ME)
🧰 Additional context used
🪛 LanguageTool
docs/5.community/4.contribution.md
[style] ~93-~93: Consider using a less common alternative to make your writing sound more unique and professional.
Context: ... #### Never let an LLM think for you * Feel free to use AI tools to generate code or explor...
(FEEL_FREE_TO_STYLE_ME)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (20)
- GitHub Check: test-fixtures (windows-latest, built, webpack, async, manifest-on, json, 20)
- GitHub Check: test-fixtures (windows-latest, built, vite, default, manifest-on, json, 20)
- GitHub Check: test-fixtures (windows-latest, built, rspack, async, manifest-on, json, 20)
- GitHub Check: test-fixtures (windows-latest, built, webpack, default, manifest-on, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, dev, vite, default, manifest-off, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, built, vite, async, manifest-on, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, built, webpack, default, manifest-on, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, built, rspack, async, manifest-on, json, 20)
- GitHub Check: test-fixtures (windows-latest, dev, vite, async, manifest-on, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, built, vite, async, manifest-on, js, 20)
- GitHub Check: test-fixtures (windows-latest, dev, vite, async, manifest-off, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, dev, vite, async, manifest-on, json, 20)
- GitHub Check: test-fixtures (ubuntu-latest, dev, vite, async, manifest-on, js, 20)
- GitHub Check: test-size
- GitHub Check: typecheck (windows-latest, bundler)
- GitHub Check: test-benchmark
- GitHub Check: release-pr
- GitHub Check: typecheck (ubuntu-latest, bundler)
- GitHub Check: code
- GitHub Check: lint-docs
🔇 Additional comments (1)
docs/5.community/4.contribution.md (1)
83-84
: Incorporated previous suggestions – looks good.The introductory sentence now matches the maintainer’s earlier suggestion and reads well in context.
2be1167
to
e335b93
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perfect!
I love this! ❤️ human > perfection |
🔗 Linked issue
resolves #32719
📚 Description
This PR adds guidelines for AI-assisted contributions to help maintain quality and authenticity in the Nuxt community.
As AI tools become more prevalent in development workflows, these guidelines help: