Skip to content

fix: bump fast-uri to 3.1.4 to patch SNYK-JS-FASTURI-18021349#227

Merged
radovanjorgic merged 1 commit into
mainfrom
rado/ISS-349245
Jul 22, 2026
Merged

fix: bump fast-uri to 3.1.4 to patch SNYK-JS-FASTURI-18021349#227
radovanjorgic merged 1 commit into
mainfrom
rado/ISS-349245

Conversation

@radovanjorgic

@radovanjorgic radovanjorgic commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR patches a high-severity Snyk finding in the transitive dev dependency fast-uri.

It bumps fast-uri 3.1.3 → 3.1.4 as a lockfile-only change (dev dependency, pulled in via ajv > fast-uri under the @microsoft/api-extractor toolchain), fixing the Interpretation Conflict vulnerability SNYK-JS-FASTURI-18021349. The bump is reachable within the parent ajv's ^3.0.1 range, so no package.json change is needed. Resolved against registry.npmjs.org (the internal mirror lagged on 3.1.4); no internal-mirror URLs remain in the lockfile. Verified with a clean npm ci, npm run build, npm test, and a re-run of snyk test (SNYK-JS-FASTURI-18021349 no longer present).

Remaining dev-only findings with no clean fix (tracked separately, not addressed here):

  • SNYK-JS-INFLIGHT-6095116inflight@1.0.6 (ISS-349246): abandoned package, no patched release; pulled in transitively via jest > glob@7. Would need an overrides entry or an upstream jest/glob upgrade — larger than a vuln-patch warrants.
  • SNYK-JS-JSYAML-17342520js-yaml@3.15.0 (ISS-349247): fix only in 4.2.0, unreachable because the parent @istanbuljs/load-nyc-config@1.1.0 pins js-yaml@^3.13.1 (cross-major).

Both are dev/test tooling only and are not shipped in the published package.

Note: npm run lint currently reports 10 pre-existing prettier errors in src/uploader/uploader.helpers.test.ts — these are present on clean main and unrelated to this lockfile-only change, so they are not touched here.

Connected Issues

Checklist

  • Tests added/updated and ran with npm run test OR no tests needed.
  • Ran backwards compatibility tests with npm run test:backwards-compatibility.
  • Code formatted and checked with npm run lint.
  • Tested airdrop-template linked to this PR.
  • Documentation updated and provided a link to PR / new docs OR no docs needed.

Transitive lockfile-only bump of fast-uri 3.1.3 -> 3.1.4 (dev dependency,
via ajv > fast-uri under the api-extractor toolchain), patching the
high-severity Interpretation Conflict SNYK-JS-FASTURI-18021349. Reachable
within the parent ajv's ^3.0.1 range. Resolved against public npmjs.org
(the internal mirror lagged 3.1.4). Confirmed via `npm ci` + `snyk test`.

Two remaining dev-only findings have no clean fix and are tracked separately:
- SNYK-JS-INFLIGHT-6095116 (inflight@1.0.6, ISS-349246): abandoned package,
  no patched release; pulled in via jest > glob@7.
- SNYK-JS-JSYAML-17342520 (js-yaml@3.15.0, ISS-349247): fix only in 4.2.0,
  unreachable — parent @istanbuljs/load-nyc-config pins js-yaml@^3.13.1.
@radovanjorgic
radovanjorgic marked this pull request as ready for review July 22, 2026 09:07
@radovanjorgic
radovanjorgic requested review from a team and gasperzgonec as code owners July 22, 2026 09:07
@radovanjorgic
radovanjorgic merged commit a6f7d51 into main Jul 22, 2026
9 checks passed
@radovanjorgic
radovanjorgic deleted the rado/ISS-349245 branch July 22, 2026 09:09
radovanjorgic added a commit that referenced this pull request Jul 22, 2026
…gration docs

Forward-port three main commits onto the refactored v2 code:
- #225: attachment loader report emits plural 'attachments' item_type
  (loading-adapter.ts, both CREATED and FAILED branches; supportedItemTypes
  stays singular as the stats-file routing key)
- #223: axios 1.17.0 -> 1.18.1 (lockfile-only; 10 Snyk fixes)
- #227: fast-uri 3.1.3 -> 3.1.4 (lockfile-only, transitive; SNYK-JS-FASTURI-18021349)

Correct MIGRATION.md and the migrate-v2 skill against the actual v2 source
(from the doc/skill verification audit): fix the npm install version (@beta /
newest 2.x, not @2.0.0 which 404s), broaden the WorkerAdapter detect to catch
value-context construction, add dispositions for the removed dist/state/state
State class and createAdapterState, document the ProcessTaskInterface and
TaskAdapterInterface generic flip, correct the mappers *Response count and the
ExternalSystemAttachmentIteratorFunction symbol name, and remove stale
"MIGRATION.md is wrong" callouts.
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