Skip to content

Commit be0c628

Browse files
jspspikemrbbot
andauthored
Bump miniflare version to 20231023 (#4274)
* Bump miniflare version to 20231023 * chore: bump `miniflare` to `3.20231025.0` * Try to improve reliability of `pages-plugin-mounted-on-root-app` test --------- Co-authored-by: bcoll <[email protected]>
1 parent b404ab7 commit be0c628

File tree

7 files changed

+53
-64
lines changed

7 files changed

+53
-64
lines changed

.changeset/good-beers-design.md

+9
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
---
2+
"wrangler": patch
3+
---
4+
5+
chore: bump `miniflare` to [`3.20231025.0`](https://github.com/cloudflare/miniflare/releases/tag/v3.20231025.0)
6+
7+
This change enables Node-like `console.log()`ing in local mode. Objects with
8+
lots of properties, and instances of internal classes like `Request`, `Headers`,
9+
`ReadableStream`, etc will now be logged with much more detail.

fixtures/pages-plugin-mounted-on-root-app/tests/index.test.ts

+8-28
Original file line numberDiff line numberDiff line change
@@ -1,43 +1,23 @@
1-
import { fork } from "child_process";
21
import * as path from "path";
32
import { fetch } from "undici";
43
import { describe, it, beforeAll, afterAll } from "vitest";
5-
import type { ChildProcess } from "child_process";
4+
import { runWranglerPagesDev } from "../../shared/src/run-wrangler-long-lived";
65

76
describe("Pages Functions", () => {
8-
let wranglerProcess: ChildProcess;
97
let ip: string;
108
let port: number;
9+
let stop: (() => Promise<unknown>) | undefined;
1110

1211
beforeAll(async () => {
13-
await new Promise((resolve) => {
14-
wranglerProcess = fork(
15-
path.join("..", "..", "packages", "wrangler", "bin", "wrangler.js"),
16-
["pages", "dev", "public", "--port=0"],
17-
{
18-
stdio: ["ignore", "ignore", "ignore", "ipc"],
19-
cwd: path.resolve(__dirname, ".."),
20-
}
21-
).on("message", (message) => {
22-
const parsedMessage = JSON.parse(message.toString());
23-
ip = parsedMessage.ip;
24-
port = parsedMessage.port;
25-
resolve(null);
26-
});
27-
});
12+
({ ip, port, stop } = await runWranglerPagesDev(
13+
path.resolve(__dirname, ".."),
14+
"public",
15+
["--port=0"]
16+
));
2817
});
2918

3019
afterAll(async () => {
31-
await new Promise((resolve, reject) => {
32-
wranglerProcess.once("exit", (code) => {
33-
if (!code) {
34-
resolve(code);
35-
} else {
36-
reject(code);
37-
}
38-
});
39-
wranglerProcess.kill("SIGTERM");
40-
});
20+
await stop?.();
4121
});
4222

4323
it("mounts a plugin correctly at root", async ({ expect }) => {

fixtures/pages-ws-app/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
},
1717
"devDependencies": {
1818
"@cloudflare/workers-tsconfig": "workspace:*",
19-
"miniflare": "3.20231016.0",
19+
"miniflare": "3.20231025.0",
2020
"wrangler": "workspace:*",
2121
"ws": "^8.8.0"
2222
},

packages/pages-shared/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
"test:ci": "vitest run"
1919
},
2020
"dependencies": {
21-
"miniflare": "3.20231016.0"
21+
"miniflare": "3.20231025.0"
2222
},
2323
"devDependencies": {
2424
"@cloudflare/workers-tsconfig": "workspace:*",

packages/wrangler/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@
107107
"blake3-wasm": "^2.1.5",
108108
"chokidar": "^3.5.3",
109109
"esbuild": "0.17.19",
110-
"miniflare": "3.20231016.0",
110+
"miniflare": "3.20231025.0",
111111
"nanoid": "^3.3.3",
112112
"path-to-regexp": "^6.2.0",
113113
"resolve.exports": "^2.0.2",

packages/wrangler/src/dev/local.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ export function Local(props: LocalProps) {
133133
useInspector({
134134
inspectorUrl,
135135
port: props.inspectorPort,
136-
logToTerminal: true,
136+
logToTerminal: false,
137137
sourceMapPath: props.sourceMapPath,
138138
name: props.name,
139139
sourceMapMetadata: props.bundle?.sourceMapMetadata,

pnpm-lock.yaml

+32-32
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)