deps: periodic dependency update - #214
Conversation
There was a problem hiding this comment.
Arcjet Review — 🟡 Medium Risk
Decision: Approved
Rationale: This PR is a periodic dependency/security maintenance update. Changes are limited to package.json/pyproject.toml version bumps and npm overrides entries that pin transitive dependencies past known GHSA advisories, plus a documentation update in OVERRIDES.md that clearly explains each pin and its removal condition. No application code is touched. The dependency-changes trigger fires, but each override maps to a specific published advisory and the reasoning is documented per pin. Version bumps are minor/patch updates to well-known packages (fastify 5.12.0→5.12.3, fastapi 0.138→0.141, uvicorn 0.49→0.52, @langchain/openai 1.2.8→1.5.6, @types/node, npm-check-updates, gunicorn). Reviewer confidence is high enough to approve at Medium risk; the risk that lingers is dependency-update-inherent (transitive graph changes not visible without lockfiles).
Summary of Changes
Periodic security/dependency maintenance across the example apps: minor version bumps for fastify, fastapi/uvicorn, gunicorn, @langchain/openai, @types/node, npm-check-updates, and new npm overrides pinning qs, nanoid, undici, uuid, adm-zip, sharp, and @opentelemetry/core past a set of documented GHSA advisories. OVERRIDES.md is updated with a table explaining each new pin and its removal condition.
Escalation Triggers
- Dependency Changes: Modifies package.json in 12 example projects plus the root, and two pyproject.toml files, bumping direct dependency versions and adding npm
overridesfor transitive dependencies.
Review Focus Areas
- Confirm that @workflow/core, @workflow/world-local, and @workflow/world-vercel still function correctly with the forced nanoid 5.1.16 and undici 7.29.0 versions (major/minor bumps of transitives via
overridescan occasionally introduce API breakage).
npmoverridesbypass the parent package's declared version constraints, so any assumption the parent made about the pinned version is worth spot-checking. - Verify @opentelemetry/core 2.11.0 is compatible with the Google ADK OpenTelemetry exporters, and that uuid 11.x's API surface is compatible with gaxios' usage.
The OVERRIDES.md note says ADK pins core 2.1.0; forcing 2.11.0 is a substantial version jump for a shared instrumentation library. - Confirm sharp 0.35.4 works with Rampart's Transformers 4.2.0 at runtime (OVERRIDES.md notes Transformers requires sharp 0.34.x).
Overriding to a version the parent explicitly does not allow can cause runtime failures; a smoke run of the guard example is worth doing before merge.
Notes
The commit log mentions browserslist and undici updates that would live in lockfiles; those files are not present in the filtered diff, which is expected and not a description accuracy issue.
Path filtering: 21 files excluded by ignore paths. 16 of 37 files included in review.
Review: e4994679 | Model: anthropic/claude-opus-4-7 | Powered by Arcjet Review
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
Ignoring alerts on:
|
|
@SocketSecurity ignore pypi/gunicorn@26.2.0 npm/fastify@5.12.3 pypi/fastapi@0.141.1 Gunicorn: unchanged, unused Django configuration example. Fastify: unchanged maintainer build script generating validators from a hardcoded schema. FastAPI: upstream docs tooling, excluded from the wheel and unused here. |
Periodic dependency and security update