Changeset 63422

Timestamp:

09/01/2026 05:50:10 PM (11 hours ago)

Author:

lancewillett

Message:

Build/Test Tools: Correct local environment failure handling.

Merges [62871] and [63416] to the 7.0 branch.

Developed in: ​https://github.com/WordPress/wordpress-develop/pull/13352

Reviewed by: johnbillion.
Props jonsurrell, lucasbustamante, mukesh27, adrianmoldovanwp.
See #65745.

Location:

branches/7.0

Files:

• . (modified) (1 prop)
• tools/local-env/scripts/docker.js (modified) (1 diff)
• tools/local-env/scripts/start.js (modified) (3 diffs)

branches/7.0/tools/local-env/scripts/docker.js

@@ -39 +39 @@
 );
 
-process.exit( returns.status );
+if ( returns.error ) {
+        console.error( `Could not run Docker Compose. ${ returns.error.message }` );
+} else if ( returns.signal && returns.signal !== 'SIGINT' ) {
+        console.error( `Docker Compose was terminated by ${ returns.signal }.` );
+}
+
+// `status` is null when Docker could not be spawned at all, or was killed by a signal. SIGINT is
+// how a long-running command such as `env:logs` is normally ended, so it is not a failure worth an
+// npm error block. Every other signal means the command was killed before it finished.
+process.exit( returns.signal === 'SIGINT' ? 0 : ( returns.status ?? 1 ) );

branches/7.0/tools/local-env/scripts/start.js

@@ -5 +5 @@
 const { execSync, spawnSync } = require( 'child_process' );
 const local_env_utils = require( './utils' );
-const { constants, copyFile } = require( 'node:fs' );
+const { copyFileSync, existsSync } = require( 'node:fs' );
 
 // Copy the default .env file when one is not present.
-copyFile( '.env.example', '.env', constants.COPYFILE_EXCL, () => {
-        console.log( '.env file already exists. .env.example was not copied.' );
-});
+if ( ! existsSync( '.env' ) ) {
+        copyFileSync( '.env.example', '.env' );
+}
 
 dotenvExpand.expand( dotenv.config() );
@@ -33 +33 @@
 }
 
-spawnSync(
+const up = spawnSync(
         'docker',
         [
@@ -45 +45 @@
         { stdio: 'inherit' }
 );
+
+// No signal is exempt here, unlike in `docker.js`: `env:start` runs `composer update -W` next, and
+// that must not run against containers that never came up.
+if ( up.status !== 0 ) {
+        const reason = up.signal ? `It was terminated by ${ up.signal }.` : up.error?.message ?? '';
+
+        console.error( `Could not start the Docker containers. ${ reason }`.trim() );
+
+        // `status` is null when Docker could not be spawned at all, or was killed by a signal.
+        process.exit( up.status ?? 1 );
+}
 
 // If Docker Toolbox is being used, we need to manually forward LOCAL_PORT to the Docker VM.