Skip to content

fix: harden package manager config#232

Open
zeitlinger wants to merge 3 commits into
mainfrom
codex/security-hardening-package-manager
Open

fix: harden package manager config#232
zeitlinger wants to merge 3 commits into
mainfrom
codex/security-hardening-package-manager

Conversation

@zeitlinger
Copy link
Copy Markdown
Member

@zeitlinger zeitlinger commented May 27, 2026

Summary

Draft canary PR generated from the security-hardening skill prototype in grafana/ai-kit#585.

This applies deterministic safe npm hardening for package roots where npm is unambiguous from the existing lockfile:

  • packageManager: "npm@11.16.0" — exact pin to the npm registry latest dist-tag checked on 2026-05-28
  • ignore-scripts=true
  • save-exact=true
  • allow-git=none
  • min-release-age=3

Validation

  • Ran security_hardening.py fix --modules package-management --safe-only --apply
  • Re-ran security_hardening.py detect --modules package-management
  • Confirmed there are 0 remaining package-management findings in this canary checkout

Notes

This intentionally does not migrate npm to pnpm. pnpm may be a valid stronger hardening path, but it changes lockfiles/install commands and should be a separate owner-reviewed migration.

Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented May 27, 2026

CLA assistant check
All committers have signed the CLA.

@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented May 27, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
@tolzhabayev tolzhabayev moved this from 📬 Triage to 🔬 In review in Grafana Catalog Team May 28, 2026
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
@zeitlinger zeitlinger marked this pull request as ready for review May 28, 2026 12:16
@zeitlinger zeitlinger requested a review from a team as a code owner May 28, 2026 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🔬 In review

Development

Successfully merging this pull request may close these issues.

2 participants