| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
🦋 Changeset detectedLatest commit: 8f44326 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
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 |
Sorry, something went wrong.
|
Codeowners approval required for this PR:
|
Sorry, something went wrong.
|
All imports are present. The changeset lists only wrangler as patch, which is correct — runtime-types is not a published package that end users depend on directly? Let me verify the changeset matches. The bot mentioned wrangler, vite-plugin, and vitest-pool-workers get bumped (via dependency). That's consistent. Now let me consider the changeset scope. The changeset references runtime-types changes but only bumps wrangler. Per AGENTS.md, changesets should reference the public-facing package. wrangler is the user-facing tool for wrangler types, so this is correct. Let me verify the logic once more for a potential real bug. The comment on line 87-89 still says "Ignore Node.js compatibility as there is currently no mechanism to generate these types dynamically." That's fine. Let me think about whether pushing no_nodejs_compat_v2 alongside no_nodejs_compat could be redundant/conflicting in workerd. Per the resolveNodejsCompat docs (line 80-82): from the default-on date, "each must be turned off separately... with no_nodejs_compat and no_nodejs_compat_v2 respectively." So both are needed and valid. Correct. The implementation is correct and well-tested. Let me do a final check — is there any scenario where a user explicitly passes no_nodejs_compat with a date >= 2026-08-04 (they've disabled node compat)? The filter removes it, then re-adds both opt-outs. Result is the same (node disabled). No regression. And the header retains the original flags for caching. Correct. The code is correct, tests cover the main cases, imports are valid, and the changeset is appropriate. No actionable issues. LGTM |
Sorry, something went wrong.
|
✅ All changesets look good |
Sorry, something went wrong.
|
@cloudflare/autoconfig
npm i https://pkg.pr.new/@cloudflare/autoconfig@15239
npm i https://pkg.pr.new/@cloudflare/build-output-utils@15239
npm i https://pkg.pr.new/@cloudflare/config@15239
npm i https://pkg.pr.new/create-cloudflare@15239
npm i https://pkg.pr.new/@cloudflare/deploy-helpers@15239
npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@15239
npm i https://pkg.pr.new/miniflare@15239
npm i https://pkg.pr.new/@cloudflare/pages-functions@15239
npm i https://pkg.pr.new/@cloudflare/pages-shared@15239
npm i https://pkg.pr.new/@cloudflare/unenv-preset@15239
npm i https://pkg.pr.new/@cloudflare/vite-plugin@15239
npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@15239
npm i https://pkg.pr.new/@cloudflare/workers-auth@15239
npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@15239
npm i https://pkg.pr.new/@cloudflare/workers-utils@15239
npm i https://pkg.pr.new/wrangler@15239 commit: 8f44326 |
Sorry, something went wrong.
There was a problem hiding this comment.
I think this is OK but a couple more code comments would help the future reader understand what is going on.
Sorry, something went wrong.
There was a problem hiding this comment.
Codeowners reviews satisfied
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM 🙂
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Fixes #15215.
Prevent date-enabled Node.js compatibility from adding conflicting globals to generated runtime types
wrangler types now treats Node.js compatibility enabled by a compatibility date the same way as an explicit nodejs_compat flag. Node.js globals continue to come from @types/node instead of being generated as any declarations that override those types.
A picture of a cute animal (not mandatory, but encouraged)