millicentokaa/acbu-backend

[miracleokaa]

Summary

Added engine-strict=true to .npmrc to enforce Node version compatibility during pnpm install. Without this flag, installation succeeds on incompatible Node versions, deferring failures to runtime with cryptic errors from unsupported syntax features. This change ensures early detection of version mismatches before deployment. Additionally verified that tsconfig.build.json already has sourceMap: false to prevent shipping source maps to production (#469), and confirmed that seed files contain no hardcoded user IDs that could collide with production data (#467).

Scope

• Backend API behavior
• Build/CI only
• Docs only
• Other

Validation

• pnpm run build
• pnpm test
• pnpm lint

Links

Closes #475
Closes #469
Closes #467

Summary by CodeRabbit

• Chores
  • Strengthened Node.js version enforcement during install to help prevent unsupported environments from being used.

[drips-wave]

@miracleokaa Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 806267df-d964-4090-8aa5-4eefadc44778

📥 Commits

Reviewing files that changed from the base of the PR and between d6fa28a and 5d49dfb.

📒 Files selected for processing (1)

• .npmrc

---

📝 Walkthrough

Walkthrough

Adds engine-strict=true to .npmrc, causing npm/pnpm to treat Node.js engine version mismatches as hard errors during install rather than warnings.

Changes

Engine Strict Enforcement

Layer / File(s)	Summary
Enable engine-strict in npm config .npmrc	Adds engine-strict=true so installs fail immediately on incompatible Node.js versions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A bunny checked the engines tight,
"Wrong Node? We stop—not run all night!"
One line added, strict and true,
No more cryptic errors to pursue.
🐇✅

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	The change only addresses #475; it does not modify tsconfig.build.json or prisma/seed.ts for #469 and #467.	Include the missing fixes for #469 and #467, or remove those issues from the linked scope if they are not part of this PR.
Title check	❓ Inconclusive	The title is just the repository name and does not describe the change.	Use a short descriptive title such as 'Enforce Node version compatibility via engine-strict in .npmrc'.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description follows the required sections and includes summary, scope, validation, and links.
Out of Scope Changes check	✅ Passed	The only code change matches the stated .npmrc objective and introduces no unrelated edits.
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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}