Skip to content

[RUN-4470]Remove fs malware placeholder package from devDependencies#1808

Merged
jayas006 merged 3 commits into
4.0.xfrom
fix/remove-fs-malware-package
May 20, 2026
Merged

[RUN-4470]Remove fs malware placeholder package from devDependencies#1808
jayas006 merged 3 commits into
4.0.xfrom
fix/remove-fs-malware-package

Conversation

@jayas006
Copy link
Copy Markdown
Contributor

@jayas006 jayas006 commented May 20, 2026

Fixes failed docs publish and CI by removing the malicious npm fs tombstone and resolving Snyk high/critical findings in the VuePress toolchain.

Publish / npm ci: Removed fs@0.0.1-security from devDependencies. That package is npm’s malware placeholder, not Node’s built-in fs, and PagerDuty’s registry blocks it with 403.
Snyk: Upgraded direct dependencies (axios, glob, lodash) and pinned safe transitive versions via package.json overrides (ajv, fast-uri, brace-expansion, follow-redirects, picomatch, qs).
VuePress scripts (notes.mjs, pr-feed.mjs, etc.) continue to use Node’s built-in fs via import fs from 'fs' — no npm fs package is required.

@jayas006 jayas006 requested review from a team and Copilot May 20, 2026 15:32
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the fs malware-placeholder npm package (0.0.1-security) from this documentation site’s Node/VuePress toolchain dependencies to avoid installing a non-functional / security placeholder module.

Changes:

  • Removed fs: ^0.0.1-security from devDependencies in package.json.
  • Removed the corresponding fs entry from package-lock.json (including the node_modules/fs package record).

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Drops the fs placeholder package from devDependencies.
package-lock.json Updates the lockfile to remove the resolved fs placeholder package entry.

@jayas006 jayas006 changed the title Remove fs malware placeholder package from devDependencies [RUN-4470]Remove fs malware placeholder package from devDependencies May 20, 2026
@jayas006 jayas006 added this to the 5.20.1 milestone May 20, 2026
@jayas006 jayas006 force-pushed the fix/remove-fs-malware-package branch from 60eae9f to ec1c13f Compare May 20, 2026 15:56
@jayas006 jayas006 merged commit aa3867e into 4.0.x May 20, 2026
3 checks passed
@jayas006 jayas006 deleted the fix/remove-fs-malware-package branch May 20, 2026 16:01
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