Skip to content

Conversation

@threepointone
Copy link
Contributor

of note, removes a direct dep of ai-chat on the ai sdk

of note, removes a direct dep of ai-chat on the ai sdk
@changeset-bot
Copy link

changeset-bot bot commented Jan 7, 2026

🦋 Changeset detected

Latest commit: 5132320

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

This PR includes changesets to release 4 packages
Name Type
hono-agents Patch
@cloudflare/codemode Patch
@cloudflare/ai-chat 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 Jan 7, 2026

Claude Code Review

Issue: The ai package removal from packages/ai-chat/package.json dependencies is incomplete.

In packages/ai-chat/package.json:63-65, the PR removes the ai dependency but it is still listed in devDependencies (line 23) and also bumped to ^6.0.15.

Question: Is this intentional? The PR description says it removes a direct dep of ai-chat on the ai sdk, but:

  • If ai should be removed entirely, it should also be removed from devDependencies
  • If ai should only be a peer/dev dependency, this is correct

Please clarify the intended dependency relationship for the ai package in @cloudflare/ai-chat.


Otherwise: Dependency updates look standard. Changeset is present. No other issues identified.

@threepointone
Copy link
Contributor Author

yes it's a peer dep

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 7, 2026

Open in StackBlitz

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

commit: 5132320

@threepointone threepointone merged commit 87dc96d into main Jan 8, 2026
5 checks passed
@threepointone threepointone deleted the update-deps branch January 8, 2026 13:05
@github-actions github-actions bot mentioned this pull request Jan 8, 2026
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