`cf migrate` fails with `ERR_PNPM_IGNORED_BUILDS` in a pnpm workspace that has not approved workerd

未关闭
  • 哪里不对？
#511 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
4/5
预计耗时
3-5 天
新手友好度
55/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
typescript
领域
build-system, cli

调研方向

Read the cf migrate install path in installProjectDependencies from @cloudflare/codemods, then compare it with the build approval handling added for cf init and @cloudflare/autoconfig. Reproduce with the provided pnpm workspace and check pnpm-workspace.yaml behavior for existing entries, explicit false decisions, and --dry-run. Done means migration approves required builds before installing, or skips installation with clear approval guidance, without leaving a throwing generated config.

由索引模型根据 Issue 内容生成。

描述

Summary of the issue

In a pnpm workspace whose pnpm-workspace.yaml does not approve workerd's build script, cf migrate writes cloudflare.config.ts and then fails when it installs cf. pnpm 12 rejects the install because workerd, a dependency of cf, has an unapproved build script. cf migrate exits with code 1 and writes throw new Error("Migration incomplete. …") into the generated cloudflare.config.ts.

cf init handles this since #92 (fixed in #136), and cloudflare/workers-sdk#16035 (PR cloudflare/workers-sdk#16054) adds the same approvals to @cloudflare/autoconfig. cf migrate installs through installProjectDependencies in @cloudflare/codemods, which neither change covers. @cloudflare/codemods 0.7.3, the version on main, has no build-approval step either.

Expected behaviour

When cf migrate installs with pnpm, it approves the builds that cf needs (workerd) in the active pnpm-workspace.yaml before it installs, as cf init does. It keeps existing entries and explicit false decisions, and writes nothing on --dry-run. If cf migrate cannot approve the builds, it skips the install and says which builds to approve, instead of writing a config that throws.

Version of cf used

1.0.0-beta.27

System information and package versions
  System:
    OS: macOS 27.0
    CPU: (14) arm64 Apple M4 Pro
    Memory: 412.20 MB / 48.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 26.13.0 - ~/.local/share/fnm/node-versions/v26.13.0/installation/bin/node
    npm: 11.20.0 - ~/.local/share/fnm/node-versions/v26.13.0/installation/bin/npm
    pnpm: 12.16.1 - /opt/homebrew/bin/pnpm
    bun: 1.4.4 - /opt/homebrew/bin/bun
    Deno: 2.9.8 - /opt/homebrew/bin/deno
    Watchman: 2026.10.26.00 - /opt/homebrew/bin/watchman
  npmPackages:
    cf: 1.0.0-beta.27 => 1.0.0-beta.27
Reproduction (optional)
  1. Create these files:
    • package.json: { "name": "cf-migrate-pnpm-repro", "private": true }
    • pnpm-workspace.yaml: packages: ["apps/*"] (no allowBuilds)
    • apps/worker/package.json: { "name": "worker", "private": true, "type": "module" }
    • apps/worker/wrangler.jsonc: { "name": "worker", "main": "src/index.js", "compatibility_date": "2026-11-01" }
    • apps/worker/src/index.js: export default { fetch() { return new Response("Hello"); } };
  2. Run pnpm install at the root.
  3. In apps/worker, run pnpm dlx cf@latest migrate.

cf migrate exits with code 1. cloudflare.config.ts contains the throw, and pnpm-workspace.yaml is unchanged.

Workaround: approve workerd first (allowBuilds: { workerd: true } in pnpm-workspace.yaml), then cf migrate completes and writes no throw. Or run cf migrate --no-install and install cf yourself.

Relevant error logs (optional)
Using the Wrangler bundler because @cloudflare/vite-plugin is not declared. Pass --bundler vite to override.
Updated 3 file(s):
├─ cloudflare.config.ts
├─ package.json
└─ ../../pnpm-lock.yaml

Follow-up work:
└─ [required] The generated configuration was written, but `cf` could not be installed automatically. Install `cf@latest` with your package manager before using it. Installation failed: …

devDependencies:
+ cf 1.0.0-beta.27

Error: ERR_PNPM_IGNORED_BUILDS

  × adding a new package
  ╰─▶ Ignored build scripts: [email protected]
  help: Run "pnpm approve-builds" to pick which dependencies should be allowed
        to run scripts.

⚠ Migration requires follow-up work.
主要语言
TypeScript
星标
993
派生
65
平均合并
21 小时 47 分钟
30 天内合并 PR
209

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

cloudflare/cf 的其他 Issue

查看 cloudflare/cf 的全部 Issue

相似的 Issue

更多 TypeScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。