Skip to content

feat: add pnpmPlugin.removePnpmfileCheckSum temporary pnpmfile checksum compatibility for renovate#14

Merged
SoonIter merged 3 commits intomainfrom
wip/entangled-qubit-f32f
Apr 14, 2026
Merged

feat: add pnpmPlugin.removePnpmfileCheckSum temporary pnpmfile checksum compatibility for renovate#14
SoonIter merged 3 commits intomainfrom
wip/entangled-qubit-f32f

Conversation

@SoonIter
Copy link
Copy Markdown
Owner

@SoonIter SoonIter commented Apr 14, 2026

Summary

  • add a temporary pnpmPlugin.removePnpmfileChecksum option to skills.json so pnpm-plugin-skills can remove pnpmfileChecksum from pnpm-lock.yaml in afterAllResolved
  • restore preview release change detection by removing the incorrect predicate-quantifier: 'every' setting from the preview workflow
  • add and update manifest/plugin tests plus pnpm plugin documentation for the new manifest-based configuration

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings April 14, 2026 08:44
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 adds an opt-in compatibility toggle to pnpm-plugin-skills to strip pnpmfileChecksum from the generated pnpm lockfile after resolution, addressing temporary interoperability needs (per the linked Renovate discussion).

Changes:

  • Introduces a temporary pnpmPlugin.removePnpmFileCheckSum config flag consumed by pnpm-plugin-skills.
  • Adds an afterAllResolved hook implementation that deletes pnpmfileChecksum when the flag is enabled.
  • Adds tests for both the opt-in removal behavior and the default no-op behavior.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
packages/pnpm-plugin-skills/src/index.ts Adds config typing and implements the afterAllResolved hook to remove pnpmfileChecksum when enabled.
packages/pnpm-plugin-skills/pnpmfile.mjs Registers the new afterAllResolved hook for pnpm v11 entrypoint.
packages/pnpm-plugin-skills/pnpmfile.cjs Registers the new afterAllResolved hook for pnpm v10 entrypoint.
packages/pnpm-plugin-skills/test/index.test.ts Adds coverage for opt-in checksum removal and default behavior.

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

Comment thread packages/pnpm-plugin-skills/src/index.ts Outdated
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 14, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
skills-package-manager 3482161 Commit Preview URL

Branch Preview URL
Apr 14 2026, 09:10 AM

@SoonIter SoonIter changed the title fix: add temporary pnpmfile checksum compatibility feat: move pnpm plugin config into skills manifest Apr 14, 2026
@SoonIter SoonIter changed the title feat: move pnpm plugin config into skills manifest feat: add pnpmPlugin.removePnpmfileCheckSum temporary pnpmfile checksum compatibility for renovate Apr 14, 2026
@SoonIter SoonIter merged commit 50eb11c into main Apr 14, 2026
6 of 7 checks passed
@SoonIter SoonIter deleted the wip/entangled-qubit-f32f branch April 14, 2026 09:27
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