Add new commands for patch remove, list and GC - #2
Merged
Mikola Lysenko (mikolalysenko) merged 2 commits intoNov 14, 2025
Merged
Conversation
- Add list command to display patches from local manifest with detailed information - Add remove command to remove patches by PURL or UUID - Add gc (garbage collection) command to clean up unused blob files - Support --json output for list command - Support --dry-run for gc command 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove command now automatically cleans up unused blobs after updating manifest - Aligns with existing behavior in download command - Ensures orphaned blobs are cleaned up immediately 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
4 tasks done
Mikola Lysenko (mikolalysenko)
pushed a commit
that referenced
this pull request
Oct 4, 2026
Vendoring on a pnpm 7/8 lock writes the project's absolute path into pnpm-lock.yaml. When that path contained " #" or ": " (for example "My Project #2"), it was written unquoted, so YAML cut it short or rejected the line and every frozen install failed while vendor reported success. The specifier is now spelled the way pnpm itself writes it: plain when safe, single-quoted otherwise. Re-running vendor heals a lock an older release left broken, and revert still restores the original bytes. Fixes #754 Assisted-by: Claude Code:claude-opus-5-5
Mikola Lysenko (mikolalysenko)
added a commit
that referenced
this pull request
Oct 5, 2026
* Start fix for #754 Assisted-by: Claude Code:claude-opus-5-5 * Quote pnpm 7/8 vendored specifier when needed Vendoring on a pnpm 7/8 lock writes the project's absolute path into pnpm-lock.yaml. When that path contained " #" or ": " (for example "My Project #2"), it was written unquoted, so YAML cut it short or rejected the line and every frozen install failed while vendor reported success. The specifier is now spelled the way pnpm itself writes it: plain when safe, single-quoted otherwise. Re-running vendor heals a lock an older release left broken, and revert still restores the original bytes. Fixes #754 Assisted-by: Claude Code:claude-opus-5-5 * Run real pnpm 7/8 vendoring under # and : paths Adds end-to-end legs that vendor with real pnpm 7.33.5 and 8.15.9 in project directories named "hash #x" and "colon: x", then require the same-path frozen offline install to land the patched bytes. Without the quoting fix both fail with ERR_PNPM_OUTDATED_LOCKFILE. Refs #754 Assisted-by: Claude Code:claude-opus-5-5 * Skip the ": " test dir on Windows Windows does not allow ":" in a directory name, so the #754 tests that create a "colon: x" project dir would fail there before testing anything. Those cases now run on Linux and macOS only; the " #" case still runs everywhere. Assisted-by: Claude Code:claude-opus-5-5 * Drop unrelated cargo fmt --all churn from the #754 fix The quoting commit reformatted 130 files outside the fix, which buries the real change and conflicts with every other open PR touching those files. Restore them to main; the fix itself (pnpm_lock_legacy.rs, formats/pnpm/lines.rs, e2e_vendor_pnpm_build.rs) is unchanged. Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.