Merge branch 'remove/throwonerror-flag' into add/error-report-modal

.github/workflows/build-website.yml

@@ -2,6 +2,9 @@ name: Deploy to playground.wordpress.net
 
 on:
     workflow_dispatch:
+    # Deploy the website every day at 9am UTC
+    schedule:
+        - cron: '0 9 * * *'
 
 jobs:
     build_and_deploy:

packages/php-wasm/universal/src/lib/base-php.ts

@@ -276,7 +276,7 @@ export abstract class BasePHP implements IsomorphicLocalPHP {
 			}
 
 			const response = await this.#handleRequest();
-			if (request.throwOnError && response.exitCode !== 0) {
+			if (response.exitCode !== 0) {
 				const output = {
 					stdout: response.text,
 					stderr: response.errors,

packages/php-wasm/universal/src/lib/php-request-handler.ts

@@ -119,7 +119,7 @@ export class PHPRequestHandler implements RequestHandler {
 			request.url.startsWith('http://') ||
 			request.url.startsWith('https://');
 		const requestedUrl = new URL(
-			// Remove the hash part of the URL as it's meant for the server.
+			// Remove the hash part of the URL as it's not meant for the server.
 			request.url.split('#')[0],
 			isAbsolute ? undefined : DEFAULT_BASE_URL
 		);

packages/php-wasm/universal/src/lib/universal-php.ts

@@ -543,12 +543,6 @@ export interface PHPRunOptions {
 	 * The code snippet to eval instead of a php file.
 	 */
 	code?: string;
-
-	/**
-	 * Whether to throw an error if the PHP process exits with a non-zero code
-	 * or outputs to stderr.
-	 */
-	throwOnError?: boolean;
 }
 
 /**

packages/playground/blueprints/public/blueprint-schema.json

@@ -1411,10 +1411,6 @@
 				"code": {
 					"type": "string",
 					"description": "The code snippet to eval instead of a php file."
-				},
-				"throwOnError": {
-					"type": "boolean",
-					"description": "Whether to throw an error if the PHP process exits with a non-zero code or outputs to stderr."
 				}
 			},
 			"additionalProperties": false

packages/playground/blueprints/public/schema-readme.md

@@ -5,5 +5,5 @@ Whenever the types are modified, the schema needs to be rebuilt using
 `nx build playground-blueprints` and then committed to the repository.
 
 Unfortunately, it is not auto-rebuilt in `npm run dev` mode as the
-`dts-bundle-generator` utility we use for type rollyps does not support
+`dts-bundle-generator` utility we use for type rollups does not support
 watching for changes.

packages/playground/blueprints/src/lib/steps/activate-plugin.ts

@@ -35,7 +35,6 @@ export const activatePlugin: StepHandler<ActivatePluginStep> = async (
 
 	const docroot = await playground.documentRoot;
 	await playground.run({
-		throwOnError: true,
 		code: `<?php
 define( 'WP_ADMIN', true );
 require_once( ${phpVar(docroot)}. "/wp-load.php" );

packages/playground/blueprints/src/lib/steps/activate-theme.ts

@@ -34,7 +34,6 @@ export const activateTheme: StepHandler<ActivateThemeStep> = async (
 	progress?.tracker.setCaption(`Activating ${themeFolderName}`);
 	const docroot = await playground.documentRoot;
 	await playground.run({
-		throwOnError: true,
 		code: `<?php
 define( 'WP_ADMIN', true );
 require_once( ${phpVar(docroot)}. "/wp-load.php" );

packages/playground/blueprints/src/lib/steps/define-wp-config-consts.spec.ts

@@ -164,7 +164,7 @@ describe('rewriteDefineCalls', () => {
 define('WP_DEBUG', true);
 
 // The third define() argument is also supported:
-@define('SAVEQUERIES', false, true); 
+@define('SAVEQUERIES', false, true);
 
 // Expression
 define(true ? 'WP_DEBUG_LOG' : 'WP_DEBUG_LOG', 123);
@@ -230,9 +230,10 @@ describe('defineBeforeRun', () => {
 			SITE_URL: 'http://test.url',
 		};
 		await defineBeforeRun(php, constants);
-		const response = await php.run({
-			code: `<?php echo json_encode(['SITE_URL' => SITE_URL]);`,
-		});
-		expect(response.errors).toContain('PHP Fatal error:');
+		await expect(
+			php.run({
+				code: `<?php echo json_encode(['SITE_URL' => SITE_URL]);`,
+			})
+		).rejects.toThrow('PHP.run() failed with exit code');
 	});
 });

packages/playground/blueprints/src/lib/steps/define-wp-config-consts.ts

@@ -95,7 +95,6 @@ export async function rewriteDefineCalls(
 		consts,
 	});
 	await playground.run({
-		throwOnError: true,
 		code: `${rewriteWpConfigToDefineConstants}
 	$wp_config_path = '/tmp/code.php';
 	$wp_config = file_get_contents($wp_config_path);

packages/playground/blueprints/src/lib/steps/enable-multisite.ts

@@ -63,7 +63,6 @@ export const enableMultisite: StepHandler<EnableMultisiteStep> = async (
 
 	// Deactivate all the plugins as required by the multisite installation.
 	const result = await playground.run({
-		throwOnError: true,
 		code: `<?php
 define( 'WP_ADMIN', true );
 require_once(${phpVar(docroot)} . "/wp-load.php");

packages/playground/blueprints/src/lib/steps/import-wordpress-files.ts

@@ -119,7 +119,6 @@ export const importWordPressFiles: StepHandler<
 		joinPaths(documentRoot, 'wp-admin', 'upgrade.php')
 	);
 	await playground.run({
-		throwOnError: true,
 		code: `<?php
             $_GET['step'] = 'upgrade_db';
             require ${upgradePhp};

packages/playground/blueprints/src/lib/steps/run-php.ts

@@ -27,5 +27,5 @@ export const runPHP: StepHandler<RunPHPStep, Promise<PHPResponse>> = async (
 	playground,
 	{ code }
 ) => {
-	return await playground.run({ code, throwOnError: true });
+	return await playground.run({ code });
 };

packages/playground/blueprints/src/lib/steps/site-data.ts

@@ -34,7 +34,6 @@ export const setSiteOptions: StepHandler<SetSiteOptionsStep> = async (
 ) => {
 	const docroot = await php.documentRoot;
 	await php.run({
-		throwOnError: true,
 		code: `<?php
 		include ${phpVar(docroot)} . '/wp-load.php';
 		$site_options = ${phpVar(options)};
@@ -81,7 +80,6 @@ export const updateUserMeta: StepHandler<UpdateUserMetaStep> = async (
 ) => {
 	const docroot = await php.documentRoot;
 	await php.run({
-		throwOnError: true,
 		code: `<?php
 		include ${phpVar(docroot)} . '/wp-load.php';
 		$meta = ${phpVar(meta)};

packages/playground/blueprints/src/lib/utils/run-php-with-zip-functions.ts

@@ -6,7 +6,6 @@ export async function runPhpWithZipFunctions(
 	code: string
 ) {
 	return await playground.run({
-		throwOnError: true,
 		code: zipFunctions + code,
 	});
 }

packages/playground/remote/src/lib/worker-thread.ts

@@ -403,15 +403,13 @@ try {
 
 					if (args.includes('-r')) {
 						result = await childPHP.run({
-							throwOnError: true,
 							code: `${cliBootstrapScript} ${
 								args[args.indexOf('-r') + 1]
 							}`,
 							env: options.env,
 						});
 					} else if (args[1] === 'wp-cli.phar') {
 						result = await childPHP.run({
-							throwOnError: true,
 							code: `${cliBootstrapScript} require( "/wordpress/wp-cli.phar" );`,
 							env: {
 								...options.env,
@@ -423,7 +421,6 @@ try {
 						});
 					} else {
 						result = await childPHP.run({
-							throwOnError: true,
 							scriptPath: args[1],
 							env: options.env,
 						});

packages/playground/website/demos/php-blueprints.ts

@@ -79,11 +79,11 @@ try {
 		 *
 		 * > echo file_get_contents('http://localhost:5400/website-server/');
 		 * > <b>Warning</b>:  PHP Request Startup: Failed to open stream: Operation timed out in <b>php-wasm run script</b> on line <b>13</b><br />
-		 * 
+		 *
 		 * The check is therefore disabled for now.
 		 */
 		require '/wordpress/blueprints.phar';
-		
+
 		$blueprint = BlueprintBuilder::create()
 			// This isn't a WordPress zip file since wordpress.org
 			// doesn't expose the right CORS headers. It is a HTTPS-hosted
@@ -105,7 +105,7 @@ try {
 				'https://downloads.wordpress.org/plugin/hello-dolly.zip',
 				// When the regular UrlDataSource is used, the second
 				// downloaded zip file always errors with:
-				// > Failed to open stream: Operation timed out 
+				// > Failed to open stream: Operation timed out
 				'https://downloads.wordpress.org/plugin/classic-editor.zip',
 				'https://downloads.wordpress.org/plugin/gutenberg.17.7.0.zip',
 			] )
@@ -120,26 +120,26 @@ try {
 			->withFile( 'wordpress.txt', 'Data' )
 			->toBlueprint()
 		;
-		
+
 		echo "Running the following Blueprint:\n";
 		echo json_encode($blueprint, JSON_PRETTY_PRINT)."\n\n";
-		
+
 		$subscriber = new class implements EventSubscriberInterface {
 			public static function getSubscribedEvents() {
 				return [
 					ProgressEvent::class => 'onProgress',
 					DoneEvent::class     => 'onDone',
 				];
 			}
-		
+
 			public function onProgress( ProgressEvent $event ) {
 				post_message_to_js(json_encode([
 					'type'    => 'progress',
 					'caption'  => $event->caption,
 					'progress' => $event->progress,
 				]));
 			}
-		
+
 			public function onDone( DoneEvent $event ) {
 				post_message_to_js(json_encode([
 					'type'    => 'progress',
@@ -148,7 +148,7 @@ try {
 			}
 		};
 
-		
+
 		$results = run_blueprint(
 			$blueprint,
 			[
@@ -158,13 +158,12 @@ try {
 				'progressType'       => 'steps',
 			]
 		);
-		
+
 		echo "Blueprint execution finished!\n";
 		echo "Contents of /wordpress/wp-content/plugins:";
 		print_r(glob('/wordpress/wp-content/plugins/*'));
-		
+
 		`,
-		throwOnError: true,
 	});
 
 	outputDiv.textContent += result.text;

packages/playground/wordpress/public/wp-nightly/wp-admin/css/about-rtl.css

@@ -514,6 +514,10 @@
 	display: none !important;
 }
 
+.about__container code {
+	font-size: inherit;
+}
+
 .about__section {
 	font-size: 1.125rem;
 	line-height: 1.55;

packages/playground/wordpress/public/wp-nightly/wp-admin/css/about.css

@@ -513,6 +513,10 @@
 	display: none !important;
 }
 
+.about__container code {
+	font-size: inherit;
+}
+
 .about__section {
 	font-size: 1.125rem;
 	line-height: 1.55;

packages/playground/wordpress/src/wordpress/get-wordpress-module-details.ts

@@ -24,7 +24,7 @@ export function getWordPressModuleDetails(wpVersion: string = "6.4"): { size: nu
 		case 'nightly':
 			/** @ts-ignore */
 			return {
-				size: 5127808,
+				size: 5127917,
 				url: url_nightly,
 			};