Changeset 62871 for trunk

Timestamp:

07/28/2026 04:33:51 PM (6 weeks ago)

Author:

lancewillett

Message:

Build/Test Tools: Copy local environment configuration synchronously.

On a fresh checkout, start.js copied .env.example asynchronously and
then immediately loaded .env, so the read could win the race and leave
configuration unset for that process. Copy synchronously, guarded by an
existsSync check, so .env exists before dotenv.config() runs and an
existing file is left untouched. Dropping the swallow-all callback also
lets a real copy error surface instead of being logged as "already
exists".

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

Props jonsurrell, lucasbustamante, mukesh27, adrianmoldovanwp.
See #65716.

File:

• trunk/tools/local-env/scripts/start.js (modified) (1 diff)

trunk/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() );