GeezerVisonThe World Wide Web, restored to 800 pixels of glory. |
| Home|About|Gaming cabinet|Archives|How it works|Starter sites |
GeezerVison is showing github.com — Add new commands for patch remove, list and GC by mikolalysenko · Pull Request #2 · SocketDev/socket-patch 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 <[email protected]> - 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 <[email protected]> Uh oh!
There was an error while loading. Please reload this page.
4 tasks done
Mikola Lysenko (mikolalysenko)
pushed a commit
that referenced
this pull request
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
* 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 <[email protected]> --------- Co-authored-by: Claude <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
ReviewersNo reviewsUh oh!
There was an error while loading. Please reload this page. AssigneesNo one assignedLabels
None yet
ProjectsMilestoneNo milestoneDevelopmentSuccessfully merging this pull request may close these issues. Uh oh!
There was an error while loading. Please reload this page. 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. |
| You are visitor 01402503 — GeezerVison does not store the pages you look up. Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required. |