[mtrezza]

Pull Request

• Report security issues confidentially.
• Any contribution is under this license.

Issue

Resolves Dependabot alerts #238, #233, #211, #204 and #81.

These alerts are for js-yaml in the development dependency tree:

Alert	Severity	Advisory	Vulnerable range	First patched
#238	high	GHSA-2883-xcg3-v3hh	>= 3.0.0, < 3.15.2	3.15.2
#233	high	GHSA-5p4m-2wfm-xmqj	>= 3.0.0, < 3.15.1	3.15.1
#211	high	GHSA-52cp-r559-cp3m	>= 3.0.0, < 3.15.0	3.15.0
#204	medium	GHSA-h67p-54hq-rp68	< 3.15.0	3.15.0
#81	medium	GHSA-mh29-5h37-fv8m	< 3.14.2	3.14.2

The only vulnerable copy on master is node_modules/js-yaml 3.14.1, which only codecov 3.8.3 uses. codecov 3.8.3 pins js-yaml to exactly 3.14.1, so the copy cannot be updated in place. codecov 3.8.3 (July 2021) is the last release of the package. It is deprecated on npm in favor of the new Codecov uploader, and its repository codecov/codecov-node is archived, so no fixed version will be released.

The package is also unused. No npm script, config file or workflow runs it. CI uploads coverage with codecov/codecov-action@v4 in .github/workflows/ci.yml, which does not use this npm package. Instead of pinning around it, this PR removes the dependency.

The removal also drops the development copy of @tootallnate/once 1.1.2. Dependabot alert #179 (@tootallnate/once < 2.0.1) stays open, because the copies of @tootallnate/once 2.0.0 under firebase-admin > @google-cloud/storage are production dependencies and out of scope for this development-only change.

Approach

Changes

• package.json: removes the codecov (3.8.3) devDependency.

• package-lock.json: regenerated from master with npm uninstall codecov --ignore-scripts --package-lock-only. It removes codecov and the 17 packages that only it used:

  • js-yaml 3.14.1, with argparse 1.0.10, sprintf-js 1.0.3 and esprima 4.0.1
  • teeny-request 7.1.1 with its nested uuid 8.3.2, and http-proxy-agent 4.0.1 with @tootallnate/once 1.1.2
  • ignore-walk 3.0.4, with minimatch 3.1.5, brace-expansion 1.1.21, balanced-match 1.0.2 and concat-map 0.0.1
  • urlgrey 1.0.0, with fast-url-parser 1.1.3 and its nested punycode 1.4.1
  • argv 0.0.2

  The entries for agent-base 6.0.2, https-proxy-agent 5.0.0, node-fetch 2.7.0, stream-events 1.0.5, stubs 3.0.0, tr46 0.0.3, webidl-conversions 3.0.1 and whatwg-url 5.0.0 stay at their versions. They change from devOptional to optional, because the remaining packages that need them are in the optional dependency subtree of firebase-admin. No package is added and no version changes. lockfileVersion is unchanged (2).

After this change, the lock file resolves js-yaml only to 4.3.2, which is not affected by these advisories.

Breaking Changes

None. codecov is a development-only tool that is not shipped in the published package and is not invoked by any script or CI job.

Code Changes Required

None. No source, config or workflow files reference the codecov npm package. Coverage upload in CI and the coverage badge in the README are unchanged, because they use the Codecov GitHub Action and service, not this package.

Tasks

No tasks apply: this removes an unused development dependency, with no tests or documentation to add.

Summary by CodeRabbit

• Chores
  • Refreshed project setup and package metadata, and removed an unused development utility.
  • These behind-the-scenes updates do not change the application’s visible features or behavior, and no action is needed from users.

[parse-github-assistant]

🚀 Thanks for opening this pull request! We appreciate your effort in improving the project. Please let us know once your pull request is ready for review.

Tip

• Keep pull requests small. Large PRs will be rejected. Break complex features into smaller, incremental PRs.
• Use Test Driven Development. Write failing tests before implementing functionality. Ensure tests pass.
• Group code into logical blocks. Add a short comment before each block to explain its purpose.
• We offer conceptual guidance. Coding is up to you. PRs must be merge-ready for human review.
• Our review focuses on concept, not quality. PRs with code issues will be rejected. Use an AI agent.
• Human review time is precious. Avoid review ping-pong. Inspect and test your AI-generated code.

Note

Please respond to review comments from AI agents just like you would to comments from a human reviewer. Let the reviewer resolve their own comments, unless they have reviewed and accepted your commit, or agreed with your explanation for why the feedback was incorrect.

Caution

Pull requests must be written using an AI agent with human supervision. Pull requests written entirely by a human will likely be rejected, because of lower code quality, higher review effort and the higher risk of introducing bugs. Please note that AI review comments on this pull request alone do not satisfy this requirement. Our CI and AI review are safeguards, not development tools. If many issues are flagged, rethink your development approach. Invest more effort in planning and design rather than using review cycles to fix low-quality code.

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: CHILL
• Plan: Essentials
• Run ID: 90dce584-c46d-4e92-ad6b-20062dc9eade

Commits

Reviewing files that changed from the base of the PR and between 27ac915 and 8560c07.

Files selected for processing: 2

• package-lock.json
• package.json

Files with no reviewable changes: 1

• package.json

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour.

Walkthrough

The package manifest removes Codecov from development dependencies. The lockfile removes Codecov and its now-unused dependency entries, and changes several package metadata flags from devOptional to optional.

Changes: Dependency cleanup

Layer / File(s)	Summary
Remove Codecov and update lockfile package.json, package-lock.json	The manifest removes Codecov. The lockfile removes Codecov and related dependencies, and updates several packages from devOptional to optional.

Priority: Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Merge Risk: Minimal · up to 8560c

The dependency cleanup is mergeable with no identified functional, security, or availability risk.

Pre-merge checks: 7

Check name	Status	Explanation
Title check		The title uses the required refactor: prefix and begins the subject with a capital letter. It accurately describes removal of the unused codecov dependency.
Description check		The description includes the required Pull Request, Issue, Approach, and Tasks sections. It explains the security issue, implementation, scope, breaking-change status, and task applicability in suffic…
Docstring Coverage		No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within survi…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
Security Check		PASS. The PR changes only package dependencies and the lockfile. It removes codecov 3.8.3 and its vulnerable js-yaml 3.14.1 chain. The reviewed lockfile retains only js-yaml 4.3.2, and common pa…
Engage In Review Feedback		No review feedback required engagement. The supplied review data reports no posted CodeRabbit threads and zero actionable findings in the current review.

Full details: Description check

Explanation

The description includes the required Pull Request, Issue, Approach, and Tasks sections. It explains the security issue, implementation, scope, breaking-change status, and task applicability in sufficient detail.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Full details: Security Check

Explanation

PASS. The PR changes only package dependencies and the lockfile. It removes codecov 3.8.3 and its vulnerable js-yaml 3.14.1 chain. The reviewed lockfile retains only js-yaml 4.3.2, and common package versions, resolutions, and integrity values are unchanged. No executable source or CI workflow changes introduce a security pattern. The existing CI uploader remains codecov/codecov-action@v4.

Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.18%. Comparing base (75d6ad9) to head (8560c07).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #637   +/-   ##
=======================================
  Coverage   99.18%   99.18%           
=======================================
  Files          11       11           
  Lines        1841     1841           
=======================================
  Hits         1826     1826           
  Misses         15       15           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[parseplatformorg]

🎉 This change has been released in version 8.5.9