Skip to content

Conversation

@chris-durbin
Copy link
Contributor

@chris-durbin chris-durbin commented Jan 23, 2026

Jira Issue ID

HARMONY-2242

Description

Fix build failure from audit again.

Local Test Steps

npm run better-audit passes

PR Acceptance Checklist

  • Acceptance criteria met
  • Tests added/updated (if needed) and passing
  • Documentation updated (if needed)
  • Harmony in a Box tested (if changes made to microservices or new dependencies added)

Summary by CodeRabbit

  • Chores
    • Updated npm audit script configuration to enable parallel execution with streaming output and moderate severity threshold filtering
    • Cleared security audit baseline configuration

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 23, 2026

📝 Walkthrough

Walkthrough

The PR removes an npm security audit exception entry from the .nsprc configuration file and updates the package.json audit script to include the moderate severity level flag with parallel execution.

Changes

Cohort / File(s) Summary
NPM Audit Configuration
.nsprc
Removes deprecated security audit exception entry (1112432), reducing the .nsprc file from a JSON object with metadata to an empty object
NPM Scripts
package.json
Updates the better-audit-output script to add -l moderate flag to better-npm-audit and enables parallel streaming execution

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Suggested labels

hacktoberfest-accepted

Suggested reviewers

  • indiejames
  • ygliuvt

Poem

🐰 A config swept clean, exceptions dismissed,
Audit flags raised with moderate gist,
Security tightened with each careful edit,
Small tweaks that make the codebase credit! 🌟

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the Jira issue (HARMONY-2242) and summarizes the main change: fixing a build failure from audit.
Description check ✅ Passed The description includes all required template sections: Jira Issue ID, Description, Local Test Steps, and PR Acceptance Checklist, though the description text is minimal.
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.


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.

@chris-durbin chris-durbin merged commit 66092f6 into main Jan 26, 2026
6 checks passed
@chris-durbin chris-durbin deleted the harmony-2242 branch January 26, 2026 13:39
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.

3 participants