-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Update dependencies across the repository #11204
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…sed ws/MSW to drop the request host from the URL, breaking MSW
…ortunately cloudflare's types are pretty messy, so this is a bit of a messy change. However, in the spirit of dogfooding I think it's worth it—these warts will go away in time
…asset-handler tests. This package is no longer maintained or supported, and so I only made the minimal changes required to make it run, rather than the _actual_ right thing to do (migrate to vitest-pool-workers so that the tests run in a real environment
🦋 Changeset detectedLatest commit: fa078ee The changes in this PR will be included in the next version bump. 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 |
create-cloudflare
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-editor-shared
@cloudflare/workers-utils
wrangler
commit: |
|
Failed to automatically backport this PR's changes to Wrangler v3. Please manually create a PR targeting the Depending on your changes, running Notes:
|
| } | ||
|
|
||
| await engineStub.receiveEvent({ | ||
| const newStub = env.ENGINE.get(env.ENGINE.idFromName("MOCK-INSTANCE-ID")); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need to get a new stub for the same instance here?
| "dependencies": { | ||
| "react": "^19.0.0", | ||
| "react-dom": "^19.0.0" | ||
| "react": "~19.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK so 19.2 is broken but most of these are currently on 19.1.xx.
Shouldn't these all be ~19.1.0 then?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I saw you fixed this in a later commit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After further digging, it turned out to be 19.1.1 that's the first broken version
| node_version: 22 | ||
| # Browser rendering is disabled here because of https://pptr.dev/troubleshooting#issues-with-apparmor-on-ubuntu | ||
| filter: '--filter="./packages/*" --filter="./fixtures/*" --filter="./packages/vite-plugin-cloudflare/playground" --filter="!./packages/kv-asset-handler" --filter="!./fixtures/browser-rendering"' | ||
| filter: '--filter="./packages/*" --filter="./fixtures/*" --filter="./packages/vite-plugin-cloudflare/playground" --filter="!./fixtures/browser-rendering"' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kv-asset-handler no longer fails on node 22!
| test.skip("getAssetFromKV non ASCII path support", async (t) => { | ||
| mockRequestScope(); | ||
| const event = getEvent(new Request("https://blah.com/测试.html")); | ||
| const res = await getAssetFromKV(event); | ||
|
|
||
| if (res) { | ||
| t.is(await res.text(), "My filename is non-ascii"); | ||
| } else { | ||
| t.fail("Response was undefined"); | ||
| } | ||
| }); | ||
|
|
||
| test("getAssetFromKV supports browser percent encoded URLs", async (t) => { | ||
| mockRequestScope(); | ||
| const event = getEvent( | ||
| new Request("https://example.com/%not-really-percent-encoded.html") | ||
| ); | ||
| const res = await getAssetFromKV(event); | ||
|
|
||
| if (res) { | ||
| t.is(await res.text(), "browser percent encoded"); | ||
| } else { | ||
| t.fail("Response was undefined"); | ||
| } | ||
| }); | ||
|
|
||
| test("getAssetFromKV supports user percent encoded URLs", async (t) => { | ||
| mockRequestScope(); | ||
| const event = getEvent(new Request("https://blah.com/%2F.html")); | ||
| const res = await getAssetFromKV(event); | ||
|
|
||
| if (res) { | ||
| t.is(await res.text(), "user percent encoded"); | ||
| } else { | ||
| t.fail("Response was undefined"); | ||
| } | ||
| }); | ||
|
|
||
| test("getAssetFromKV only decode URL when necessary", async (t) => { | ||
| mockRequestScope(); | ||
| const event1 = getEvent( | ||
| new Request("https://blah.com/%E4%BD%A0%E5%A5%BD.html") | ||
| ); | ||
| const event2 = getEvent(new Request("https://blah.com/你好.html")); | ||
| const res1 = await getAssetFromKV(event1); | ||
| const res2 = await getAssetFromKV(event2); | ||
|
|
||
| if (res1 && res2) { | ||
| t.is(await res1.text(), "Im important"); | ||
| t.is(await res2.text(), "Im important"); | ||
| } else { | ||
| t.fail("Response was undefined"); | ||
| } | ||
| }); | ||
|
|
||
| test("getAssetFromKV Support for user decode url path", async (t) => { | ||
| test.skip("getAssetFromKV Support for user decode url path", async (t) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These probably should be made to work, but the complexity of fixing them doesn't seem worth it, given we no longer support the package.
| "dependencies": { | ||
| "react": "^19.0.0", | ||
| "react-dom": "^19.0.0" | ||
| "react": "~19.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After further digging, it turned out to be 19.1.1 that's the first broken version
|
This keeps going stale. I'm going to try and land the individual commits as individual PRs |
A mix of outdated dependencies, fixing peer dependency warnings, running
pnpm dedupe. See individual commits for details