Skip to content

fix: set npm open-pull-requests-limit to 0 for security-only policy - #211

Merged
don-petry merged 29 commits into
mainfrom
claude/issue-173-20260419-2033
May 21, 2026
Merged

fix: set npm open-pull-requests-limit to 0 for security-only policy#211
don-petry merged 29 commits into
mainfrom
claude/issue-173-20260419-2033

Conversation

@don-petry

Copy link
Copy Markdown
Collaborator

Summary

  • Sets open-pull-requests-limit for the npm ecosystem from 10 to 0 in .github/dependabot.yml
  • Per dependabot policy, application ecosystems should use 0 to suppress routine version-update PRs while still allowing security-alert-triggered PRs (which bypass the limit)
  • The github-actions ecosystem correctly retains 10

Closes #173

Generated with Claude Code

Closes #173

Co-authored-by: don-petry <don-petry@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 19, 2026 20:34
@coderabbitai

coderabbitai Bot commented Apr 19, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@don-petry has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 42 minutes and 54 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ea0c4268-ffaa-4e3e-aef6-4336cdf818c9

📥 Commits

Reviewing files that changed from the base of the PR and between 98cc446 and edfb4ee.

📒 Files selected for processing (1)
  • .github/dependabot.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/issue-173-20260419-2033

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.

@don-petry

Copy link
Copy Markdown
Collaborator Author

@don-petry — PR is ready for review. Single-line config change: npm open-pull-requests-limit set from 100 in .github/dependabot.yml, bringing the repo into compliance with the dependabot security-only policy.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates Dependabot configuration to comply with the organization’s “security-only” policy for application ecosystems by preventing routine npm version-update PRs while still allowing security-alert PRs.

Changes:

  • Set open-pull-requests-limit: 0 for the npm ecosystem in .github/dependabot.yml.
  • Kept the github-actions ecosystem limit at 10 as-is.

@github-actions
github-actions Bot requested a review from a team as a code owner May 4, 2026 20:39

@donpetry-bot donpetry-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated review — APPROVED ✓

Risk: LOW
Reviewed commit: 262a47f8a9d38274ff807f0fca0cb364cb0d8154
Review mode: triage-approved (single reviewer)

Summary

Single-line change to .github/dependabot.yml setting the npm ecosystem's open-pull-requests-limit from 10 to 0, bringing the repo into compliance with the org's dependabot security-only policy. The github-actions ecosystem correctly retains its limit of 10. Confirms the triage tier's low-risk assessment.

Linked issue analysis

Closes #173 (Compliance: wrong-limit-npm), an automated compliance-audit finding that explicitly requests open-pull-requests-limit: 0 for the npm ecosystem with a reference to the standards doc. The PR applies exactly that remediation — nothing more, nothing less. With limit 0, routine version-update PRs are suppressed while security-alert-triggered PRs still bypass the limit, matching the policy's intent.

Findings

No issues found.

  • Diff is a single value change on one line; no other config keys (schedule, labels, directory, ecosystem list) were touched.
  • Scope is appropriately narrow — github-actions ecosystem entry is untouched, consistent with the policy distinguishing application vs. CI ecosystems.
  • PR description accurately reflects the change and cites the governing standard.
  • No unresolved review threads or unanswered human-reviewer questions; Copilot reviewer left a descriptive (non-blocking) overview.

CI status

All required checks green:

  • CodeQL Analyze (actions) — SUCCESS
  • CodeQL Analyze (javascript-typescript) — SUCCESS
  • CodeQL Analyze (python) — SUCCESS
  • CodeQL (rollup) — SUCCESS
  • CodeRabbit — SUCCESS
  • SonarQube Cloud quality gate — passed (0 new issues)

Reviewed automatically by the PR-review agent (single-reviewer mode: opus 4.7). Reply if you need a human review.

@sonarqubecloud

Copy link
Copy Markdown

@don-petry
don-petry merged commit f31d0b7 into main May 21, 2026
21 of 22 checks passed
@don-petry
don-petry deleted the claude/issue-173-20260419-2033 branch May 21, 2026 08:16
don-petry added a commit that referenced this pull request Aug 4, 2026
…211)

Closes #173

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: don-petry <don-petry@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

Compliance: wrong-limit-npm

3 participants