Skip to content

fix: write hints-config.mjs to fs#296

Merged
huang-julien merged 1 commit intomainfrom
fix/template
Mar 20, 2026
Merged

fix: write hints-config.mjs to fs#296
huang-julien merged 1 commit intomainfrom
fix/template

Conversation

@huang-julien
Copy link
Member

🔗 Linked issue

fix #295

📚 Description

Forgot to write the config to fs

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 20, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@nuxt/hints@296

commit: b688d78

@coderabbitai
Copy link

coderabbitai bot commented Mar 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2983b698-a183-4b1b-8e5a-c7d04ba8e3f2

📥 Commits

Reviewing files that changed from the base of the PR and between b40565c and b688d78.

📒 Files selected for processing (1)
  • src/module.ts

📝 Walkthrough

Walkthrough

The change modifies src/module.ts to add an explicit write: true parameter to the addTemplate call for hints-config.mjs. This ensures the generated template file is written to disk before the module creates an alias referencing it. Previously, the template was only provided via getContents without being persisted to the file system.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: write hints-config.mjs to fs' directly describes the main change in the PR: ensuring the hints-config.mjs file is written to the filesystem.
Description check ✅ Passed The description references issue #295 and explains the fix: 'Forgot to write the config to fs', which relates to the changeset addressing the missing hints-config.mjs file.
Linked Issues check ✅ Passed The PR successfully addresses issue #295 by ensuring hints-config.mjs is written to the filesystem, which resolves the ENOENT error when the file was missing.
Out of Scope Changes check ✅ Passed The change only modifies the template configuration for hints-config.mjs by adding 'write: true', which is directly scoped to fixing the linked issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/template

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 and usage tips.

@huang-julien huang-julien merged commit 9baf21c into main Mar 20, 2026
7 checks passed
@huang-julien huang-julien deleted the fix/template branch March 20, 2026 17:49
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.

ENOENT: no such file or directory for hints-config.mjs

1 participant