chore(deps): simplify fast-uri dependency and update Ajv resolver - #895
Merged
mldangelo-oai merged 2 commits intoSep 11, 2026
Merged
Conversation
Bumps [fast-uri](https://github.com/fastify/fast-uri) from 3.1.6 to 4.1.4. - [Release notes](https://github.com/fastify/fast-uri/releases) - [Commits](fastify/fast-uri@v3.1.6...v4.1.4) --- updated-dependencies: - dependency-name: fast-uri dependency-version: 4.1.4 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
mldangelo-oai
approved these changes
Sep 11, 2026
mldangelo-oai
left a comment
Collaborator
There was a problem hiding this comment.
Reviewed the dependency consumers and upstream source. Removing the unused direct pin while updating Ajv's supported v3 resolver leaves one fast-uri version in the SDK lockfile. All current-head CI checks pass. Type, build and package checks also pass with the TypeScript 7 upgrades already on main.
mldangelo-oai
deleted the
dependabot/npm_and_yarn/sdk/typescript/fast-uri-4.1.4
branch
September 11, 2026 20:28
Merged
3 tasks
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.
Summary
Update the URI resolver actually used by the SDK's schema validators and remove an unused direct dependency.
Changes
fast-uridependency: SDK source and build scripts do not import it.fast-uri3.1.6 to 3.1.7, within their declared^3.0.1range.Testing
pnpm run types,pnpm run format, andpnpm run build:cipassed.pnpm pack --pack-destination ../../distpassed;check-package.mjsvalidated all 488 archive entries.fast-uri3.1.7 and resolves relative external schema references correctly.Risk and rollout
The change retains Ajv's supported dependency major and the SDK's existing Node support. URI parsing gains the published port and bracket fixes. Normal dependency installation applies the lockfile update; no configuration migration is required. Existing schema, credential, path, and scan-integrity protections remain in place.
Public disclosure review