chore(repo): remove populate-storage scripts, consolidate into nx-release

Remove the now-redundant populate-storage.js and run-populate-storage.mjs
scripts. The release logic is inlined directly in global-setup.ts, and
the populate-local-registry-storage task is simplified to an orchestration
point that delegates to nx-release via dependsOn.

When running e2e tests outside of Nx (e.g. Jest directly), global-setup
now auto-starts verdaccio if it's not already running.

Author: FrozenPandaz

e2e/utils/global-setup.ts

@@ -1,10 +1,9 @@
 import { Config } from '@jest/types';
 import { existsSync, removeSync } from 'fs-extra';
 import * as isCI from 'is-ci';
-import { exec, execSync } from 'node:child_process';
+import { ChildProcess, exec, execSync, spawn } from 'node:child_process';
 import { join } from 'node:path';
 import { registerTsConfigPaths } from '../../packages/nx/src/plugins/js/utils/register';
-import { runLocalRelease } from '../../scripts/local-registry/populate-storage';
 
 export default async function (globalConfig: Config.ConfigGlobals) {
   try {
@@ -25,14 +24,23 @@ export default async function (globalConfig: Config.ConfigGlobals) {
     const registry = `http://${listenAddress}:${port}`;
     const authToken = 'secretVerdaccioToken';
 
-    while (true) {
-      await new Promise((resolve) => setTimeout(resolve, 250));
-      try {
-        await assertLocalRegistryIsRunning(registry);
-        break;
-      } catch {
-        console.log(`Waiting for Local registry to start on ${registry}...`);
-      }
+    // When running outside of Nx (e.g. Jest directly), start verdaccio ourselves
+    let verdaccioProcess: ChildProcess | undefined;
+    if (requiresLocalRelease && !(await isLocalRegistryRunning(registry))) {
+      console.log(
+        `Local registry not detected at ${registry}, starting verdaccio...`
+      );
+      verdaccioProcess = spawn(
+        'npx',
+        [
+          'verdaccio',
+          '--config',
+          '.verdaccio/config.yml',
+          '--listen',
+          `${listenAddress}:${port}`,
+        ],
+        { stdio: 'ignore', detached: true }
+      );
     }
 
     process.env.npm_config_registry = registry;
@@ -54,6 +62,11 @@ export default async function (globalConfig: Config.ConfigGlobals) {
     global.e2eTeardown = () => {
       // Clean up environment variable instead of npm config command
       delete process.env[`npm_config_//${listenAddress}:${port}/:_authToken`];
+      // Kill verdaccio if we started it
+      if (verdaccioProcess) {
+        verdaccioProcess.kill();
+        verdaccioProcess = undefined;
+      }
     };
 
     /**
@@ -77,8 +90,26 @@ export default async function (globalConfig: Config.ConfigGlobals) {
       if (requiresLocalRelease) {
         console.log('Publishing packages to local registry');
         const publishVersion = process.env.PUBLISHED_VERSION ?? 'major';
-        // Always show full release logs on CI, they should only happen once via e2e-ci
-        await runLocalRelease(publishVersion, isCI || isVerbose);
+        const verbose = isCI || isVerbose;
+        const releaseCommand = `pnpm nx-release --local ${publishVersion}`;
+        console.log(`> ${releaseCommand}`);
+        await new Promise<void>((resolve, reject) => {
+          const child = exec(releaseCommand, {
+            maxBuffer: 1024 * 1000000,
+            windowsHide: false,
+          });
+          if (verbose) {
+            child.stdout?.pipe(process.stdout);
+            child.stderr?.pipe(process.stderr);
+          }
+          child.on('exit', (code) => {
+            if (code === 0) {
+              resolve();
+            } else {
+              reject(new Error(`Local release failed with exit code ${code}`));
+            }
+          });
+        });
       }
     }
   } catch (err) {
@@ -112,9 +143,11 @@ function getPublishedVersion(): Promise<string | undefined> {
   });
 }
 
-async function assertLocalRegistryIsRunning(url) {
-  const response = await fetch(url);
-  if (!response.ok) {
-    throw new Error(`HTTP error! status: ${response.status}`);
+async function isLocalRegistryRunning(url: string): Promise<boolean> {
+  try {
+    const response = await fetch(url);
+    return response.ok;
+  } catch {
+    return false;
   }
 }

project.json

@@ -22,17 +22,10 @@
           "input": "production",
           "projects": ["tag:maven:dev.nx.maven"]
         },
-        "{workspaceRoot}/scripts/local-registry",
         "native"
       ],
-      "dependsOn": [
-        "local-registry",
-        {
-          "target": "build",
-          "projects": ["tag:npm:public"]
-        }
-      ],
-      "command": "node ./scripts/local-registry/run-populate-storage.mjs",
+      "dependsOn": ["local-registry", "nx-release"],
+      "command": "echo 'Registry storage populated via nx-release dependency'",
       "outputs": ["{workspaceRoot}/dist/local-registry/storage"]
     },
     "nx-release": {

scripts/local-registry/populate-storage.js

@@ -218,6 +218,10 @@ const VALID_AUTHORS_FOR_LATEST = [
 
   hackFixForDevkitPeerDependencies();
 
+  if (options.local) {
+    await waitForLocalRegistry();
+  }
+
   // Run with dynamic output-style so that we have more minimal logs by default but still always see errors
   let publishCommand = `pnpm nx release publish --registry=${getRegistry()} --tag=${distTag} --output-style=dynamic --parallel=8`;
   if (options.dryRun) {
@@ -400,10 +404,6 @@ function parseArgs() {
             'Registry is still set to localhost! Run "pnpm local-registry disable" or pass --force'
           );
         }
-      } else {
-        if (!args.force && !registryIsLocalhost) {
-          throw new Error('--local was passed and registry is not localhost');
-        }
       }
 
       return true;
@@ -476,6 +476,34 @@ function determineDistTag(
   return distTag;
 }
 
+function waitForLocalRegistry(): Promise<void> {
+  const registryUrl = getRegistry().toString();
+  console.log(`Waiting for local registry at ${registryUrl}...`);
+  return new Promise<void>((resolve, reject) => {
+    const timeout = setTimeout(() => {
+      clearInterval(interval);
+      reject(
+        new Error(
+          `Local registry at ${registryUrl} did not become available within 60 seconds`
+        )
+      );
+    }, 60_000);
+    const interval = setInterval(async () => {
+      try {
+        const response = await fetch(registryUrl);
+        if (response.ok) {
+          clearInterval(interval);
+          clearTimeout(timeout);
+          console.log('Local registry is ready.');
+          resolve();
+        }
+      } catch {
+        // Registry not up yet
+      }
+    }, 50);
+  });
+}
+
 //TODO(@Coly010): Remove this after fixing up the release peer dep handling
 function hackFixForDevkitPeerDependencies() {
   const { readFileSync, writeFileSync } = require('fs');