Conversation
Agent-Logs-Url: https://github.com/version-fox/vfox/sessions/abcce404-2e68-4df1-aad4-2d6d0295b4f0 Co-authored-by: bytemain <[email protected]>
Agent-Logs-Url: https://github.com/version-fox/vfox/sessions/abcce404-2e68-4df1-aad4-2d6d0295b4f0 Co-authored-by: bytemain <[email protected]>
Agent-Logs-Url: https://github.com/version-fox/vfox/sessions/abcce404-2e68-4df1-aad4-2d6d0295b4f0 Co-authored-by: bytemain <[email protected]>
Agent-Logs-Url: https://github.com/version-fox/vfox/sessions/abcce404-2e68-4df1-aad4-2d6d0295b4f0 Co-authored-by: bytemain <[email protected]>
Agent-Logs-Url: https://github.com/version-fox/vfox/sessions/abcce404-2e68-4df1-aad4-2d6d0295b4f0 Co-authored-by: bytemain <[email protected]>
|
@aooohan |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #663 +/- ##
==========================================
+ Coverage 25.17% 25.62% +0.45%
==========================================
Files 83 83
Lines 5693 5795 +102
==========================================
+ Hits 1433 1485 +52
- Misses 4082 4115 +33
- Partials 178 195 +17 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Adds .tar.zst / .tzst (zstd-compressed tar) support to the shared Lua archiver decompression pipeline so plugins can decompress SDK distributions packaged with zstd.
Changes:
- Introduces a new
ZstdTarDecompressorusinggithub.com/klauspost/compress/zstdand wires it intoNewDecompressor()via.tar.zst/.tzstsuffix detection. - Adds tests covering decompressor selection, extraction, common-root stripping, and traversal rejection.
- Updates English + Simplified Chinese archiver docs to list zstd formats and updates
go.modto include the zstd dependency as a direct requirement.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| internal/shared/util/decompressor.go | Adds zstd-tar decompressor implementation + file suffix detection. |
| internal/shared/util/decompressor_test.go | Adds tests + helper to generate .tar.zst fixtures. |
| go.mod | Promotes github.com/klauspost/compress to a direct dependency. |
| docs/plugins/library/archiver.md | Documents .tar.zst / .tzst support (EN). |
| docs/zh-hans/plugins/library/archiver.md | Documents .tar.zst / .tzst support (ZH-Hans). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
The Lua archiver library did not recognize or extract zstd-compressed tar archives. Plugins using
archiver.decompress()could not handle SDK distributions packaged as.tar.zstor.tzst.Archive support
.tar.zstand.tzstdetection in the shared decompressor factory.github.com/klauspost/compress/zstd.Extraction behavior
Docs and coverage
Example plugin usage: