deps: periodic dependency update#836
Conversation
|
The latest updates on your projects. Learn more about Vercel 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.
|
There was a problem hiding this comment.
Arcjet Review — 🟡 Medium Risk
Decision: Reviewers Assigned
Rationale: Periodic dependency update bumping ~40 packages. The vast majority are patch/minor bumps to well-known packages (React, Astro, Nuxt, NestJS, Hono, AI SDK, etc.) and look low-risk. However there is one major version bump — Next.js 15.5.15 → 16.2.9 — that warrants human verification, and several pre-1.0 packages (@astrojs/starlight 0.38→0.39, astro-embed 0.12→0.13, starlight-links-validator 0.23→0.24, @expressive-code/* 0.41→0.42) where minor bumps can include breaking changes. The accompanying code changes in src/components/TOC.tsx and src/content.config.ts correctly accommodate a schema change (ajToc becoming optional) with a sensible ?? [] fallback. Pinned exact versions and no new dependencies introduced. No secrets, no security issues found. Escalating because of the Next.js major bump and the multiple 0.x package bumps; no specific reviewers are configured, so a maintainer should pick this up.
Summary of Changes
Periodic dependency update: ~40 packages bumped in root package.json plus three snippet package.jsons. Notable: Next.js 15→16 (major), Astro 6.1→6.3, @sveltejs/kit 2.57→2.65, ai 6.0.158→6.0.182, @astrojs/starlight 0.38→0.39, nuxt 4.4.2→4.4.8, arcjet snippets 1.3.1→1.5.0. Code adjustments in TOC.tsx and content.config.ts make the ajToc schema field optional (likely required by starlight 0.39 changes) and update the TypeScript types with NonNullable<...> plus a ?? [] fallback. No new dependencies added.
Escalation Triggers
- Dependency Changes: package.json files modified in root and three snippets directories; ~40 dependency version bumps including a Next.js major version bump (15 → 16).
Review Focus Areas
- Has the Next.js 15 → 16 upgrade been validated against the project's Next.js usage (snippets/examples, App Router behavior, peer-deps with React 19)?
Major version bumps can include breaking changes to routing, caching defaults, middleware, and build output. Confirm the build still passes and any next-specific snippets render correctly. - Did @astrojs/starlight 0.38.3 → 0.39.2 introduce the schema change that required making
ajTocoptional, and have you verified the docs build (astro check+ build) passes end-to-end?
Starlight is pre-1.0; 0.x → 0.x+1 jumps commonly include breaking changes. The TOC.tsx/content.config.ts edits in this PR suggest a known incompatibility, so it's worth confirming nothing else regressed. - Were the AI SDK bumps (@ai-sdk/openai 3.0.52→3.0.63, @ai-sdk/react 3.0.160→3.0.184, ai 6.0.158→6.0.182) verified together — any peer-dep mismatches?
The three @ai-sdk packages must stay version-compatible; mismatched versions can cause subtle runtime errors. - When
astroEntry.data.ajTocis undefined and[]is substituted, does the rest of the TOC render gracefully (no empty- , no a11y issues, no broken sticky behavior)?
- Do the snippet upgrades (@arcjet/* 1.3.1 → 1.5.0 and astro 6.1.5 → 6.3.2) still compile under
astro check?
Snippets are user-facing example code; if they break, doc readers will copy broken examples.
The new fallback silently hides the TOC for entries without one — confirm that's the intended UX and that callers don't depend on TOC always being non-empty.Notes
Diff is ~150 lines, well below the 1000-line threshold. All version bumps use exact (pinned) versions, no postinstall scripts introduced. No secrets, no auth/CI/infra/migration changes detected.
Path filtering: 1 file excluded by ignore paths. 6 of 7 files included in review.
Review: 5fed7432 | Model:
anthropic/claude-opus-4-7| Powered by Arcjet Review - Do the snippet upgrades (@arcjet/* 1.3.1 → 1.5.0 and astro 6.1.5 → 6.3.2) still compile under
Remaining security alerts are not easily fixable (parent package has them pinned exactly)