Update based on PR feedback

Author: WillTaylorDev

packages/wrangler/src/__tests__/deploy.test.ts

@@ -4446,88 +4446,6 @@ addEventListener('fetch', event => {});`
 			`);
 		});
 
-		it("should warn when using smart placement with assets-first", async () => {
-			const assets = [
-				{ filePath: ".assetsignore", content: "*.bak\nsub-dir" },
-				{ filePath: "file-1.txt", content: "Content of file-1" },
-				{ filePath: "file-2.bak", content: "Content of file-2" },
-				{ filePath: "file-3.txt", content: "Content of file-3" },
-				{ filePath: "sub-dir/file-4.bak", content: "Content of file-4" },
-				{ filePath: "sub-dir/file-5.txt", content: "Content of file-5" },
-			];
-			writeAssets(assets, "assets");
-			writeWranglerConfig({
-				assets: {
-					directory: "assets",
-					experimental_serve_directly: true,
-				},
-				placement: {
-					mode: "smart",
-				},
-			});
-			const bodies: AssetManifest[] = [];
-			await mockAUSRequest(bodies);
-			mockSubDomainRequest();
-			mockUploadWorkerRequest({
-				expectedAssets: {
-					jwt: "<<aus-completion-token>>",
-					config: {
-						serve_directly: true,
-					},
-				},
-				expectedType: "none",
-			});
-
-			await runWrangler("deploy");
-
-			expect(std.warn).toMatchInlineSnapshot(`
-				"▲ [WARNING] Using assets with smart placement turned on may result in poor performance.
-
-				"
-			`);
-		});
-
-		it("should warn when using smart placement with assets-first", async () => {
-			const assets = [
-				{ filePath: ".assetsignore", content: "*.bak\nsub-dir" },
-				{ filePath: "file-1.txt", content: "Content of file-1" },
-				{ filePath: "file-2.bak", content: "Content of file-2" },
-				{ filePath: "file-3.txt", content: "Content of file-3" },
-				{ filePath: "sub-dir/file-4.bak", content: "Content of file-4" },
-				{ filePath: "sub-dir/file-5.txt", content: "Content of file-5" },
-			];
-			writeAssets(assets, "assets");
-			writeWranglerConfig({
-				assets: {
-					directory: "assets",
-					experimental_serve_directly: true,
-				},
-				placement: {
-					mode: "smart",
-				},
-			});
-			const bodies: AssetManifest[] = [];
-			await mockAUSRequest(bodies);
-			mockSubDomainRequest();
-			mockUploadWorkerRequest({
-				expectedAssets: {
-					jwt: "<<aus-completion-token>>",
-					config: {
-						serve_directly: true,
-					},
-				},
-				expectedType: "none",
-			});
-
-			await runWrangler("deploy");
-
-			expect(std.warn).toMatchInlineSnapshot(`
-				"▲ [WARNING] Using assets with smart placement turned on may result in poor performance.
-
-				"
-			`);
-		});
-
 		it("should warn if experimental_serve_directly=false but no binding is provided", async () => {
 			const assets = [
 				{ filePath: ".assetsignore", content: "*.bak\nsub-dir" },
@@ -4562,13 +4480,18 @@ addEventListener('fetch', event => {});`
 			await runWrangler("deploy");
 
 			expect(std.warn).toMatchInlineSnapshot(`
-				"▲ [WARNING] experimental_serve_directly=false but no assets.binding provided.
+				"▲ [WARNING] Assets are not accessible, as the [assets.experimental_serve_directly] configuration key is set to \`false\`, but no [assets.binding] configuration is provided.
+
+				  ❯❯ Add an assets binding to your assets configuration: BINDING = \\"ASSETS\\", or
+				  ❯❯ Set experimental_serve_directly=false in your assets configuration
+
+				  Read more: https://developers.cloudflare.com/workers/static-assets/binding/#binding
 
 				"
 			`);
 		});
 
-		it("should error if an experimental_serve_directly is false without providing a user Worker", async () => {
+		it("should error if experimental_serve_directly is false and no user Worker is provided", async () => {
 			writeWranglerConfig({
 				assets: {
 					directory: "xyz",

packages/wrangler/src/__tests__/dev.test.ts

@@ -1720,13 +1720,18 @@ describe.sequential("wrangler dev", () => {
 			await runWranglerUntilConfig("dev");
 
 			expect(std.warn).toMatchInlineSnapshot(`
-				"▲ [WARNING] experimental_serve_directly=false but no assets.binding provided.
+				"▲ [WARNING] Assets are not accessible, as the [assets.experimental_serve_directly] configuration key is set to \`false\`, but no [assets.binding] configuration is provided.
+
+				  ❯❯ Add an assets binding to your assets configuration: BINDING = \\"ASSETS\\", or
+				  ❯❯ Set experimental_serve_directly=false in your assets configuration
+
+				  Read more: https://developers.cloudflare.com/workers/static-assets/binding/#binding
 
 				"
 			`);
 		});
 
-		it("should error if an experimental_serve_directly is false without providing a user Worker", async () => {
+		it("should error if experimental_serve_directly is false and no user Worker is provided", async () => {
 			writeWranglerConfig({
 				assets: { directory: "assets", experimental_serve_directly: false },
 			});

packages/wrangler/src/assets.ts

@@ -444,17 +444,7 @@ export function validateAssetsArgsAndConfig(
 		);
 	}
 
-	// Smart placement turned on when using assets
-	if (
-		config?.placement?.mode === "smart" &&
-		config?.assets?.experimental_serve_directly
-	) {
-		logger.warn(
-			"Using assets with smart placement turned on may result in poor performance."
-		);
-	}
-
-	// User worker ahead of assets, but no assets binding provided
+	// User Worker ahead of assets, but no assets binding provided
 	if (
 		"legacy" in args
 			? args.assets?.assetConfig?.serve_directly === false &&
@@ -463,7 +453,10 @@ export function validateAssetsArgsAndConfig(
 				!config?.assets?.binding
 	) {
 		logger.warn(
-			"experimental_serve_directly=false but no assets.binding provided."
+			"Assets are not accessible, as the [assets.experimental_serve_directly] configuration key is set to `false`, but no [assets.binding] configuration is provided.\n" +
+				'❯❯ Add an assets binding to your assets configuration: BINDING = "ASSETS", or\n' +
+				"❯❯ Set experimental_serve_directly=false in your assets configuration\n\n" +
+				"Read more: https://developers.cloudflare.com/workers/static-assets/binding/#binding"
 		);
 	}