Skip to content

fix(mongodb-downloader): replace decompress with maintained @xhmikosr/decompress - #927

Merged
lerouxb merged 1 commit into
mainfrom
move-off-decompress
Oct 14, 2026
Merged

lerouxb merged 1 commit into
mainfrom
move-off-decompress

Conversation

@lerouxb

@lerouxb lerouxb commented Oct 14, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI balanced review requested due to automatic review settings October 14, 2026 11:24
@lerouxb
lerouxb requested review from a team as code owners October 14, 2026 11:24
@lerouxb
lerouxb requested a review from paula-stacho October 14, 2026 11:24
@@ -0,0 +1,22 @@
declare module '@xhmikosr/decompress' {

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@xhmikosr/decompress@11.1.4 ships no types; excluding the declaration produces TS7016 and an implicit-any error in the filter callback.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The new declaration incorrectly types runtime Date values as strings.

0 open findings

What changed in this PR

Replaces the unmaintained decompression dependency used for MongoDB ZIP artifacts.

Changes:

  • Migrates to @xhmikosr/decompress.
  • Adds local TypeScript declarations.
  • Adds ZIP extraction regression tests.
File Description
packages/​mongodb-downloader/​src/​index.ts Updates the decompressor import.
packages/​mongodb-downloader/​src/​index.spec.ts Tests ZIP extraction behavior.
packages/​mongodb-downloader/​src/​decompress.d.ts Declares the dependency API.
packages/​mongodb-downloader/​package.json Replaces dependencies and typings.
package-lock.json Updates resolved dependency versions.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@lerouxb
lerouxb enabled auto-merge (squash) October 14, 2026 11:40
@lerouxb
lerouxb disabled auto-merge October 14, 2026 12:06
@lerouxb
lerouxb merged commit 8a549fe into main Oct 14, 2026
20 checks passed
@lerouxb
lerouxb deleted the move-off-decompress branch October 14, 2026 12:07
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.

3 participants