Skip to content

chore(deps-dev): override adm-zip to 0.6.1 for hugo-extended - #93

Merged
DariuszPorowski merged 1 commit into
mainfrom
kachawla-fix-adm-zip
Oct 28, 2026
Merged

DariuszPorowski merged 1 commit into
mainfrom
kachawla-fix-adm-zip

Conversation

@kachawla

Copy link
Copy Markdown
Member

Summary

Adds an npm overrides entry so hugo-extended uses adm-zip ^0.6.1 instead of 0.5.17, and regenerates the lockfile.

Reason for change

hugo-extended 0.168.0 (the latest release) depends on adm-zip ^0.5.17. The open adm-zip alerts are fixed in 0.6.1, which is outside that range, so Dependabot fails with security_update_not_possible. adm-zip is only used by the hugo-extended install step to unpack the Hugo binary. The override can be removed once hugo-extended allows 0.6.x.

GHSA-vwc7-r8mq-g2x9 lists no patched version, but its affected range is <= 0.6.0, so 0.6.1 is outside it.

Fixes #92

How to test

  1. npm ci. The hugo-extended postinstall should download and install Hugo 0.168.0.
  2. npm ls adm-zip should show adm-zip@0.6.1 under hugo-extended@0.168.0.
  3. npm run hugo:version should print hugo v0.168.0 ... +extended.
  4. npm run build should finish without errors.

File change summary

File Summary of change
package.json Add overrides forcing adm-zip ^0.6.1 for hugo-extended.
package-lock.json Update adm-zip from 0.5.17 to 0.6.1.

hugo-extended 0.168.0 depends on adm-zip ^0.5.17, which has open security alerts fixed in 0.6.1. Dependabot can't update it because the parent range excludes the fix, so pin it with an npm override until hugo-extended widens its range.

Signed-off-by: Karishma Chawla <kachawla@microsoft.com>
@kachawla
kachawla requested review from a team as code owners October 23, 2026 22:30
@kachawla kachawla added the dependencies Pull requests that update a dependency file label Oct 23, 2026
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails
npm/adm-zip 0.6.1 🟢 5.3
Details
CheckScoreReason
Code-Review⚠️ 0Found 1/30 approved changesets -- score normalized to 0
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Maintained🟢 1016 commit(s) and 3 issue activity found in the last 90 days -- score normalized to 10
Binary-Artifacts🟢 10no binaries found in the repo
Packaging⚠️ -1packaging workflow not detected
Security-Policy🟢 10security policy file detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ 3branch protection is not maximal on development and all release branches
SAST🟢 7SAST tool detected but not run on all commits

Scanned Files

  • package-lock.json

@github-actions github-actions Bot added the pr:waiting-for-review A reviewer owns the next action; required approval is pending or unverifiable label Oct 23, 2026
@DariuszPorowski
DariuszPorowski merged commit 4593e97 into main Oct 28, 2026
12 checks passed
@github-actions github-actions Bot removed the pr:waiting-for-review A reviewer owns the next action; required approval is pending or unverifiable label Oct 28, 2026
@DariuszPorowski
DariuszPorowski deleted the kachawla-fix-adm-zip branch October 28, 2026 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dependabot updates are failing

2 participants