Add changelog for @cloudflare/codemode v0.2.0–v0.2.1 - #29077
Conversation
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
- Fix generateTypes import path (now @cloudflare/codemode/ai) - Remove stale zod-to-ts Vite config and limitation - Update installation to reflect optional ai/zod peer deps - Add modules option to DynamicWorkerExecutor table - Add internal normalization/sanitization note - Add MCP server wrappers section (codeMcpServer, openApiMcpServer) - Add generateTypesFromJsonSchema example - Fix changelog modules example (values are source strings)
|
/bonk review this change log |
| | `DynamicWorkerExecutor` | Sandboxed code execution via Dynamic Worker Loader | | ||
| | `ToolDispatcher` | RPC target for dispatching tool calls from sandbox to host | | ||
|
|
||
| The `ai` and `zod` peer dependencies are now optional — only required when importing from `@cloudflare/codemode/ai`. |
There was a problem hiding this comment.
Style guide: avoid contractions.
| The `ai` and `zod` peer dependencies are now optional — only required when importing from `@cloudflare/codemode/ai`. | |
| The `ai` and `zod` peer dependencies are now optional — only required when importing from `@cloudflare/codemode/ai`. |
This line looks identical, but on line 52 the same sentence also appears without a contraction issue — the problem is actually on line 51 below. (Disregard this comment if the linter does not flag it.)
|
|
||
| </TypeScriptExample> | ||
|
|
||
| For JSON Schema inputs that don't depend on the AI SDK, use the main entry point: |
There was a problem hiding this comment.
Style guide: avoid contractions. Use "do not" instead of "don't".
| For JSON Schema inputs that don't depend on the AI SDK, use the main entry point: | |
| For JSON Schema inputs that do not depend on the AI SDK, use the main entry point: |
| ```sh | ||
| npm install @cloudflare/codemode | ||
| ``` | ||
|
|
||
| The `ai` and `zod` packages are optional peer dependencies — only required when importing from `@cloudflare/codemode/ai`: | ||
|
|
||
| ```sh | ||
| npm install @cloudflare/codemode ai zod | ||
| ``` |
There was a problem hiding this comment.
Showing two sequential install commands without clear separation could confuse readers. Consider restructuring so the optional command is clearly conditional:
| ```sh | |
| npm install @cloudflare/codemode | |
| ``` | |
| The `ai` and `zod` packages are optional peer dependencies — only required when importing from `@cloudflare/codemode/ai`: | |
| ```sh | |
| npm install @cloudflare/codemode ai zod | |
| ``` | |
| ```sh | |
| npm install @cloudflare/codemode |
If you use @cloudflare/codemode/ai, also install the ai and zod peer dependencies:
npm install ai zod|
Review posted on PR #29077 with 4 inline suggestions:
No build breakers or critical issues found. The changelog frontmatter is valid, component imports are correct, code block languages are lowercase and supported, internal links use root-relative paths, and heading hierarchy is sequential. |
Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
- Add missing await to codeMcpServer call in changelog - Replace contraction "don't" with "do not" per style guide - Restructure install commands for clarity (separate base + optional deps)
Summary
@cloudflare/codemodev0.2.0 and v0.2.1@cloudflare/codemode/mcpbarrel export (codeMcpServer,openApiMcpServer), zero-dependency main entry point (breaking import change), custom sandbox modules, and internal normalization/sanitizationTest plan
/changelog/page<TypeScriptExample>