Skip to content

Replace runzip with unzipper for file extraction#5

Closed
Copilot wants to merge 5 commits intomainfrom
copilot/replace-runzip-with-unzipper-again
Closed

Replace runzip with unzipper for file extraction#5
Copilot wants to merge 5 commits intomainfrom
copilot/replace-runzip-with-unzipper-again

Conversation

Copy link

Copilot AI commented Dec 11, 2025

  • Understand the issue: replace runzip shell command with unzipper npm package
  • Identify the location: scripts/fetch-data.ts
  • Install unzipper package (v0.12.3) and @types/unzipper (v0.10.11) type definitions
  • Update extractModulePropsFromZip function to use unzipper.Open.url() with native fetch instead of shell command
  • Remove unused imports (exec, promisify from child_process and util)
  • Update bun.lock with unzipper and all transitive dependencies
  • Run code review
  • Run security scan - no vulnerabilities found
Original prompt

Replace runzip with unzipper for file extraction


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits December 11, 2025 13:54
…tion

Co-authored-by: aviraxp <18079988+aviraxp@users.noreply.github.com>
Co-authored-by: aviraxp <18079988+aviraxp@users.noreply.github.com>
Copilot AI changed the title [WIP] Replace runzip with unzipper for file extraction Replace runzip with unzipper for file extraction Dec 11, 2025
Copilot AI requested a review from aviraxp December 11, 2025 14:01
Remove package-lock.json from .gitignore
@aviraxp
Copy link
Contributor

aviraxp commented Dec 11, 2025

@copilot Run bun install --frozen-lockfile

bun install v1.3.4 (5eb2145b)
Resolving dependencies
Resolved, downloaded and extracted [68]
error: lockfile had changes, but lockfile is frozen
note: try re-running without --frozen-lockfile and commit the updated lockfile

Co-authored-by: aviraxp <18079988+aviraxp@users.noreply.github.com>
Copy link
Author

Copilot AI commented Dec 11, 2025

@copilot Run bun install --frozen-lockfile

bun install v1.3.4 (5eb2145b)...

Updated bun.lock with unzipper and all its transitive dependencies in commit 0f12e36. The lockfile should now pass bun install --frozen-lockfile.

@aviraxp aviraxp closed this Dec 12, 2025
@aviraxp aviraxp deleted the copilot/replace-runzip-with-unzipper-again branch December 21, 2025 13:17
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.

2 participants