Quieter builds (#6058)

This patch cleans up warnings we were seeing when doing a full build. Specifically:

- fixtures/remix-pages-app had a bunch of warnings about impending features that it should be upgraded to, so I did that. (tbh this one needs a full upgrade of packages, but we'll get to that later when we're upgrading across the codebase)
- updated `@microsoft/api-extractor` so it didn't complain that it didn't match the `typescript` version (that we'd recently upgraded)
- it also silenced a bunch of warnings when exporting types from `wrangler`. We'll need to fix those, but we'll do that when we work on unstable_dev etc.
- workers-playground was complaining about the size of the bundle being generated, so I increased the limit on it

.changeset/moody-papayas-obey.md

@@ -0,0 +1,13 @@
+---
+"miniflare": patch
+"wrangler": patch
+---
+
+Quieter builds
+
+This patch cleans up warnings we were seeing when doing a full build. Specifically:
+
+- fixtures/remix-pages-app had a bunch of warnings about impending features that it should be upgraded to, so I did that. (tbh this one needs a full upgrade of packages, but we'll get to that later when we're upgrading across the codebase)
+- updated `@microsoft/api-extractor` so it didn't complain that it didn't match the `typescript` version (that we'd recently upgraded)
+- it also silenced a bunch of warnings when exporting types from `wrangler`. We'll need to fix those, but we'll do that when we work on unstable_dev etc.
+- workers-playground was complaining about the size of the bundle being generated, so I increased the limit on it

fixtures/remix-pages-app/app/entry.client.tsx

@@ -1,4 +1,4 @@
 import { RemixBrowser } from "@remix-run/react";
-import { hydrate } from "react-dom";
+import { hydrateRoot } from "react-dom/client";
 
-hydrate(<RemixBrowser />, document);
+hydrateRoot(document, <RemixBrowser />);

fixtures/remix-pages-app/app/root.tsx

@@ -6,13 +6,14 @@ import {
 	Scripts,
 	ScrollRestoration,
 } from "@remix-run/react";
-import type { MetaFunction } from "@remix-run/cloudflare";
 
-export const meta: MetaFunction = () => ({
-	charset: "utf-8",
-	title: "New Remix App",
-	viewport: "width=device-width,initial-scale=1",
-});
+export const meta = () => {
+	return [
+		{ charset: "utf-8" },
+		{ title: "New Remix App" },
+		{ viewport: "width=device-width,initial-scale=1" },
+	];
+};
 
 export default function App() {
 	return (

fixtures/remix-pages-app/remix.config.js

@@ -1,9 +1,23 @@
 /** @type {import('@remix-run/dev').AppConfig} */
 module.exports = {
-	serverBuildTarget: "cloudflare-pages",
+	publicPath: "/build/",
+	serverBuildPath: "functions/[[path]].js",
+	serverConditions: ["worker"],
+	serverMainFields: ["workerd", "browser", "module", "main"],
+	serverModuleFormat: "esm",
+	serverPlatform: "neutral",
+	serverDependenciesToBundle: "all",
+	serverMinify: true,
 	server: "./server.js",
 	devServerBroadcastDelay: 1000,
 	ignoredRouteFiles: ["**/.*"],
+	future: {
+		v2_errorBoundary: true,
+		v2_normalizeFormMethod: true,
+		v2_meta: true,
+		v2_headers: true,
+		v2_routeConvention: true,
+	},
 	// appDirectory: "app",
 	// assetsBuildDirectory: "public/build",
 	// serverBuildPath: "functions/[[path]].js",

packages/miniflare/package.json

@@ -58,7 +58,7 @@
 		"@ava/typescript": "^4.0.0",
 		"@cloudflare/kv-asset-handler": "workspace:*",
 		"@cloudflare/workers-types": "^4.20240605.0",
-		"@microsoft/api-extractor": "^7.36.3",
+		"@microsoft/api-extractor": "^7.47.0",
 		"@types/debug": "^4.1.7",
 		"@types/estree": "^1.0.0",
 		"@types/glob-to-regexp": "^0.4.1",

packages/workers-playground/vite.config.ts

@@ -19,5 +19,8 @@ export default defineConfig(({ mode }) => {
 
 		appType: "spa",
 		base: "/playground",
+		build: {
+			chunkSizeWarningLimit: 1000,
+		},
 	};
 });

packages/wrangler/api-extractor.json

@@ -345,6 +345,10 @@
 			"default": {
 				"logLevel": "none"
 				// "addToApiReportFile": false
+			},
+			"ae-forgotten-export": {
+				"logLevel": "none",
+				"addToApiReportFile": true
 			}
 
 			// "ae-extra-release-tag": {

packages/wrangler/package.json

@@ -94,7 +94,7 @@
 		"@cloudflare/workers-types": "^4.20240605.0",
 		"@cspotcode/source-map-support": "0.8.1",
 		"@iarna/toml": "^3.0.0",
-		"@microsoft/api-extractor": "^7.28.3",
+		"@microsoft/api-extractor": "^7.47.0",
 		"@sentry/node": "^7.86.0",
 		"@sentry/types": "^7.86.0",
 		"@sentry/utils": "^7.86.0",

packages/wrangler/src/api/startDevWorker/DevEnv.ts

@@ -10,9 +10,6 @@ import type { Controller, RuntimeController } from "./BaseController";
 import type { ErrorEvent } from "./events";
 import type { DevWorker, StartDevWorkerOptions } from "./types";
 
-/**
- * @internal
- */
 export class DevEnv extends EventEmitter {
 	config: ConfigController;
 	bundler: BundlerController;