fix(deps): patch fast-uri and humanfs security alerts - #80
Merged
Merged
Conversation
John Kennedy (jkennedyvz)
marked this pull request as ready for review
September 9, 2026 06:43
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Patch the two vulnerable transitive dependencies behind all five currently open Dependabot alerts. No application code changes or alert dismissals.
fast-uri: 3.1.5 → 3.1.6, including the existing npm override. Used throughelectron-store→conf→ AJV.@humanfs/node: 0.16.7 → 0.16.8, used by ESLint. Includes required@humanfs/core0.19.2 and@humanfs/types0.15.0 dependencies.package.jsonandpackage-lock.jsonchange. Lockfile regenerated with targeted npm update, not manually edited.Related Issue
Dependency presence is verified; this does not assert that the application exposes the advisories' full exploit paths. Available narrow patches are preferable to speculative non-exploitability dismissals. Alerts remain open until merged and rescanned.
Type of Change
Checklist
Test Plan
Validated on Linux, Node 24.18.1 / npm 11.16.0. CI uses Node 20; remote CI results are separate.
sfw npm ci --ignore-scripts --no-audit --no-fundsfw npm audit --json: zero vulnerabilitiesnpm run lint: zero errors, three formatting warnings in unchanged renderer filesnpm run typechecknpm run build: succeeds, with a dynamic/static-import warning in unchanged DB codegit diff --checkAdditional Notes
Install scripts were disabled locally; Electron GUI startup was not tested. Existing metadata/config warnings: npm reports an unknown
node-linkersetting;npm lsreports ESLint 10 outside the React plugins' peer ranges. These dependency versions and source files are unchanged. CI and CONTRIBUTING use npm and the repository tracks only package-lock.json, despite its pnpm packageManager field; that inconsistency is left out of scope. The existing .gitignore lacks common secret-file patterns; no secret files were staged and no unrelated ignore changes are included.