Conversation
Non-major: - hono 4.13.4, @hono/node-server 2.1.1, @hono/standard-validator 0.4.0 - jose 6.2.10, uuid 14.0.2, viem 2.55.19 - @noble/curves 2.3.0, multiformats 14.0.5 - @inquirer/prompts 8.6.0, figlet 1.11.4, wrap-ansi 10.0.1, yoctocolors 2.2.0 - knip 6.32.2, oxfmt 0.65.0, oxlint 1.80.0, tsdown 0.22.14, tsx 4.23.12, turbo 2.10.12, vitest 4.1.11, mintlify 4.2.824 Major: - typescript 7.0.2 and oxlint-tsgolint 7.0.2001 (oxlint 1.80 requires it) - @solana/kit 8.0.0 and @solana/codecs-strings 8.0.0 (@solana-program/token 0.16 requires kit 8) - @types/node 26.3.0 - @changesets/cli 3.0.1, @changesets/changelog-github 1.0.0 - ai 7.0.79, @ai-sdk/anthropic 4.0.42, @ai-sdk/openai 4.0.47, @ai-sdk/valibot 3.0.30 (identity demo only) Left alone: - @a2a-js/sdk 1.0 is an A2A protocol major that changes the Message shape and adds gRPC/protobuf peers; ack-id publishes a ^0.3.0 peer. - did-resolver 5 conflicts with did-jwt and did-jwt-vc, which still depend on ^4.1.0 and produce incompatible Resolvable types. Also disables the @scarf/scarf telemetry build script pulled in by mintlify. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
oxlint 1.80 reports no-unnecessary-type-assertion for casts whose receiver already accepts the source type. Remove the seven flagged casts; no behavior change. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Remove every override the current tree no longer needs (body-parser, path-to-regexp, picomatch, js-yaml, read-yaml-file, ws, uuid, tar, rollup, postcss, jsondiffpatch, adm-zip) and the matching minimumReleaseAgeExclude entries. The dependency bump resolves those advisories on its own. Keep or add only what pnpm audit still requires: - esbuild@<0.24.3 -> ^0.25.0 (drizzle-kit's deprecated @esbuild-kit) - vite@7 -> ^7.3.6 (7.3.5 pins esbuild ^0.27, GHSA low) - qs@6 -> ^6.15.2 (express 4 via mintlify) - sharp@<0.35.0 -> ^0.35.0 (libvips CVEs via mintlify; never built here) - puppeteer@24 -> ^25.0.0 (drops extract-zip, which has no patched release; never built here) pnpm audit: 0 vulnerabilities (was 35). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
ai 7 scopes GenerateTextResult.response to the final step and deprecates it. result.response.messages would drop tool-call and tool-result messages from earlier steps in the identity demo's multi-step loop. result.responseMessages carries the accumulated messages of all steps, matching the ai 6 behavior. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
sharp 0.33.5 still ships an install script; only 0.35 drops it and loads prebuilt @img/sharp-* binaries, so the override does run at docs build time (verified with mintlify dev). The changeset $schema now points at @changesets/config 4.0.0, which @changesets/cli 3.0.1 installs. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (14)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. WalkthroughThe pull request refreshes workspace and package dependencies, updates vulnerability and build policies, aligns demos with newer SDK and type behavior, and adds Changesets metadata for dependency-only package releases. ChangesDependency refresh and compatibility
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This PR updates dependencies, narrows package overrides, and adjusts the identity demo for the new AI response API; the reported checks and audits pass, so no actionable merge-blocking risk remains beyond normal review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 5 files. (9 skipped: 9 unsupported.) ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Changes
pnpm auditgoes from 35 advisories to 0pnpm-workspace.yamlcut from 18 entries to 5, each scoped and commentedminimumReleaseAgeExcluderemoved; no longer neededascasts removed that oxlint 1.80 flags as unnecessaryresult.responseMessages(ai 7 change)packageManager@agentcommercekit/keysand@agentcommercekit/ack-idProblem
The dependency tree had drifted: 35 audit advisories (1 critical, 19 high), most of them under mintlify, and an override block that had grown by accretion. Several of those overrides pinned versions the tree no longer pulls in, and two (
vite,esbuild) were global instead of scoped to the vulnerable line.Solution
Bump everything the full check accepts, then rebuild the override set from zero: remove every override, reinstall, audit, and add back only what still fails. Five remain:
esbuild@<0.24.3@esbuild-kitpins esbuild 0.18vite@7qs@6sharp@<0.35.0puppeteer@24extract-ziphas no patched release; puppeteer 25 drops itTwo majors were left alone on purpose:
@a2a-js/sdk1.0 is an A2A protocol major (newMessageshape, gRPC/protobuf peers) and would changeack-id's published^0.3.0peer range.did-resolver5 conflicts withdid-jwtanddid-jwt-vc, which still depend on^4.1.0and produce incompatibleResolvabletypes.Two things worth knowing about the majors that did land:
ai7 scopesresult.responseto the final step. The identity demo usedresult.response.messagesto collect the full multi-step history, which would silently drop earlier tool-call messages. It now usesresult.responseMessages, which keeps the ai 6 semantics.typescript7 is the native compiler and exposes no JS API. Every repo command (tsdown build, oxlint type checking, knip, changesets,mintlify dev) works, but@mintlify/mdx/serverwould throw on import if the docs ever add twoslash blocks.Testing
pnpm run check(build + lint + format + test): 29/29 tasks pass, 0 lint warningspnpm audit: no known vulnerabilitiespnpm install --frozen-lockfilewith both pnpm 11.8.0 and 11.24.0: lockfile stablepnpm exec changeset status: clean@solana/kit7 and 8 release notes; none of the removed APIs are usedPre-existing, not addressed here:
mintlify validatewarns on the@mintlify/componentsimport indocs/ack-pay/payment-service.mdx, and.changeset/config.jsoncarries an inert"tag": falsekey.AI usage: Claude Code (Claude Fable 5) performed the dependency bumps, override analysis, source edits, and verification, and drove a multi-agent panel review (codex, claude, opencode). The author directed the work and reviewed the result.
🤖 Generated with Claude Code
Summary by CodeRabbit
Bug Fixes
Maintenance