Skip to content

Remove unused puppeteer dependency from Cloudflare Worker project#202

Merged
boyphongsakorn merged 3 commits intodependabot/npm_and_yarn/npm_and_yarn-e50b0cc2fffrom
copilot/sub-pr-200
Nov 8, 2025
Merged

Remove unused puppeteer dependency from Cloudflare Worker project#202
boyphongsakorn merged 3 commits intodependabot/npm_and_yarn/npm_and_yarn-e50b0cc2fffrom
copilot/sub-pr-200

Conversation

Copy link
Contributor

Copilot AI commented Nov 8, 2025

This PR addresses a dependency audit update that bumped tar-fs and ws versions. A question was raised about puppeteer's presence in the dependency list.

Analysis

Verified that puppeteer is listed in package.json but:

  • Not imported or used in any source files (index.js, worker/*.js)
  • Incompatible with Cloudflare Workers (webworker target) - requires Node.js runtime and headless browser
  • Project only uses cheerio for HTML parsing and node-fetch for HTTP requests

Changes Made

  • Removed puppeteer from package.json dependencies
  • Updated package-lock.json to reflect the removal
  • Regenerated pnpm-lock.yaml to remove all puppeteer references
  • Verified build still works successfully without the dependency

The removal of this unused dependency reduces the project's dependency footprint and eliminates a package that is incompatible with the Cloudflare Workers environment.


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

@github-actions
Copy link

github-actions bot commented Nov 8, 2025

ถ้าคุณต้องการช่วยเราในการแก้ไขบัค หรือ เพิ่มฟังก์ชั่น ได้โปรดส่งคำขอไปยังที่ https://github.com/quad-b/lottsanook-nodejs สำหรับ NodeJS หรือ https://github.com/quad-b/lottsanook สำหรับ PHP | This repository does not accept pull requests, if your want to pull requests for NodeJS,JS code please pull requests to https://github.com/quad-b/lottsanook-nodejs

@github-actions github-actions bot closed this Nov 8, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Nov 8, 2025
@boyphongsakorn boyphongsakorn reopened this Nov 8, 2025
Copilot AI changed the title [WIP] Bump npm_and_yarn group with updates for tar-fs Clarify puppeteer dependency status in Cloudflare Worker project Nov 8, 2025
Copilot AI requested a review from boyphongsakorn November 8, 2025 16:57
@boyphongsakorn boyphongsakorn marked this pull request as ready for review November 8, 2025 16:58
@boyphongsakorn
Copy link
Collaborator

@copilot remove puppeteer in package

Co-authored-by: boyphongsakorn <3013842+boyphongsakorn@users.noreply.github.com>
Copilot AI changed the title Clarify puppeteer dependency status in Cloudflare Worker project Remove unused puppeteer dependency from Cloudflare Worker project Nov 8, 2025
@boyphongsakorn
Copy link
Collaborator

@copilot in pnpm-lock.yaml is not remove

Co-authored-by: boyphongsakorn <3013842+boyphongsakorn@users.noreply.github.com>
@boyphongsakorn boyphongsakorn merged commit 83b6846 into dependabot/npm_and_yarn/npm_and_yarn-e50b0cc2ff Nov 8, 2025
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants