Repository navigation
Conversation
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 34 minutes. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (4)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Review fixes pushed in 3bb875f. The packaging-source finding is addressed and its thread has a reply/resolution. Full The updated changes are ready for another review; the latest review checks are still running. |
Problem
The existing packaging command requires PowerShell, so contributors cannot build a review artifact with the repository's Node toolchain alone on macOS or Linux.
Change
Add an opt-in
pnpm package:portablecommand using only Node and Git. It emits deterministic ZIP/XPI extension artifacts, a committed review-source archive, and SHA-256 checksums under dist. Use the project's existing release version, require clean committed source, reject symlinks and unsupported ZIP64 sizes, and write forward-slash UTF-8 archive paths.The existing PowerShell command, release workflow, and ZIP/source checksum filenames are preserved. There is no fork-specific metadata requirement, signing, publication, installation, or native companion change.
Validation
pnpm testpasses on this branch.tests/test-portable-packaging.mjscompares every extension byte against the review source and verifies ignored files and real CRLF checkouts cannot change any artifact. It runs innpm testandnpm run test:package. Both pass on Linux ARM64; Windows execution has not been directly validated. Both artifacts use one canonical committed Git tree, including committed file modes.Generated by GPT 6.1 Sol in Codex, using T3 Code.

Confidence Score: 5/5
Safe to merge based on the reviewed changes. No outstanding blocking findings remain.Summary
Adds opt-in
pnpm package:portablepackaging with Node and Git.No new issues were found. The earlier archive mismatch is also fixed: both archives use the same frozen
HEADtree.Reviews (3) · Last reviewed commit: "Disable inherited signing for packaging ..." · Reviewed by Greptile