Skip to content

🐛 fix(deps): encode pnpm integrity hash - #45

Merged
xdanger merged 1 commit into
mainfrom
fix/pnpm-11-20-0-hash
Aug 3, 2026
Merged

🐛 fix(deps): encode pnpm integrity hash#45
xdanger merged 1 commit into
mainfrom
fix/pnpm-11-20-0-hash

Conversation

@xdanger

@xdanger xdanger commented Aug 3, 2026

Copy link
Copy Markdown
Member

Summary

  • replace the SRI base64 digest introduced by the previous pnpm bump with Corepack's required SHA-512 hexadecimal digest
  • keep pnpm at 11.20.0; this changes only integrity metadata

Validation

  • Corepack resolves and verifies pnpm 11.20.0
  • frozen install
  • full repository lint suite

Risk

Low. This is a metadata-only repair with no dependency or lockfile change.

- 🐛 encode the Corepack hash as SHA-512 hex

- 🔧 keep the mise pin aligned where configured
@claude

claude Bot commented Aug 3, 2026

Copy link
Copy Markdown

Approved

No new issues found. This is a straightforward metadata fix — replacing the base64 SRI digest with Corepack's expected SHA-512 hex digest. The hash is the correct length (128 hex chars) and the version remains pinned at 11.20.0.

Findings: None.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR repairs the packageManager integrity metadata for pnpm@11.20.0 by switching from an SRI-style base64 digest to the SHA-512 hex digest format required by Corepack, without changing the pnpm version or dependencies.

Changes:

  • Update packageManager to use +sha512.<128-hex> integrity metadata for pnpm@11.20.0.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@xdanger
xdanger merged commit 5e8cc08 into main Aug 3, 2026
5 checks passed
@xdanger
xdanger deleted the fix/pnpm-11-20-0-hash branch August 3, 2026 19:05
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.

2 participants