Skip to content

Conversation

@threepointone
Copy link
Contributor

  • remove the changesets cli patch, as well as updating node version, so we don't need to explicitly install newest npm
  • lock mcp sdk version till we figure out how to do breaking changes correctly
  • removes stray permissions block from release.yml

- remove the changesets cli patch, as well as updating node version, so we don't need to explicitly install newest npm
- lock mcp sdk version till we figure out how to do breaking changes correctly
- removes stray permissions block from release.yml
@changeset-bot
Copy link

changeset-bot bot commented Dec 21, 2025

🦋 Changeset detected

Latest commit: ec220b6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
hono-agents Patch
agents Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@claude
Copy link

claude bot commented Dec 21, 2025

Claude Code Review

Dependency update PR - mostly clean. One issue:

scheduler.ts L38: Removed mode: json parameter from generateObject() but eval file (packages/agents/evals/scheduling.eval.ts L16, L336) still has outdated comments referencing it. Clean up those stale references.

Other notes:

  • MCP SDK version lock to 1.23.0 is reasonable
  • Node 24 upgrade looks correct
  • CSS duplicate property removals are good
  • Dependency bumps appear routine

CI still running - verify scheduling functionality works without mode parameter.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 21, 2025

Open in StackBlitz

npm i https://pkg.pr.new/cloudflare/agents@739

commit: ec220b6

@threepointone threepointone merged commit e9b6bb7 into main Dec 21, 2025
5 checks passed
@threepointone threepointone deleted the update-deps branch December 21, 2025 12:30
@github-actions github-actions bot mentioned this pull request Dec 21, 2025
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.

1 participant