Bug: copySeoFiles.mjs Fails to Copy SEO Files to Correct dist Paths for Applications #7748

Author: tobiu

buildScripts/copySeoFiles.mjs

@@ -18,10 +18,11 @@ const SEO_FILES = ['robots.txt', 'llm.txt', 'sitemap.xml'];
 /**
  * Recursively finds application root directories by looking for index.html.
  * Excludes 'examples' directory.
- * @param {string} currentDir
- * @returns {string[]} Array of absolute paths to app roots.
+ * @param {string} currentDir - The directory to start searching from.
+ * @param {string} baseAppDir - The base 'apps' directory to calculate relative paths from.
+ * @returns {Array<{appRootPath: string, appRelativePath: string}>} Array of objects with absolute and relative paths.
  */
-function findAppRoots(currentDir) {
+function findAppRoots(currentDir, baseAppDir) {
     let appRoots  = [];
     const entries = fs.readdirSync(currentDir, { withFileTypes: true });
 
@@ -31,10 +32,11 @@ function findAppRoots(currentDir) {
         if (entry.isDirectory()) {
             // Check if current directory contains index.html
             if (fs.existsSync(path.join(fullPath, 'index.html'))) {
-                appRoots.push(fullPath);
+                const appRelativePath = path.relative(baseAppDir, fullPath);
+                appRoots.push({ appRootPath: fullPath, appRelativePath: appRelativePath });
             }
             // Recurse into subdirectories
-            appRoots = appRoots.concat(findAppRoots(fullPath));
+            appRoots = appRoots.concat(findAppRoots(fullPath, baseAppDir));
         }
     }
     return appRoots;
@@ -43,22 +45,19 @@ function findAppRoots(currentDir) {
 /**
  * Copies SEO files from an app root to its corresponding dist folders.
  * @param {string} appRootPath - Absolute path to the app's root directory.
+ * @param {string} appRelativePath - Relative path of the app from the base 'apps' directory.
  * @param {string} env - The build environment ('all', 'dev', 'prod').
  */
-function copySeoFilesForApp(appRootPath, env) {
-    const appName = path.basename(appRootPath);
-    console.log(chalk.blue(`Copying SEO files for app: ${appName}`));
+function copySeoFilesForApp(appRootPath, appRelativePath, env) {
+    console.log(chalk.blue(`Copying SEO files for app: ${appRelativePath}`));
 
     const targetEnvs = [];
-    if (env === 'all' || env === 'dev') {
-        targetEnvs.push('development');
-    }
-    if (env === 'all' || env === 'prod') {
-        targetEnvs.push('production');
-    }
+    if (env === 'all' || env === 'dev')  {targetEnvs.push('development');}
+    if (env === 'all' || env === 'esm')  {targetEnvs.push('esm');}
+    if (env === 'all' || env === 'prod') {targetEnvs.push('production');}
 
     for (const targetEnv of targetEnvs) {
-        const targetDistDir = path.join(DIST_DIR, targetEnv, appName);
+        const targetDistDir = path.join(DIST_DIR, targetEnv, 'apps', appRelativePath);
         if (!fs.existsSync(targetDistDir)) {
             console.warn(chalk.yellow(`  Warning: Target directory ${targetDistDir} does not exist. Skipping SEO file copy.`));
             continue;
@@ -94,9 +93,9 @@ console.log(chalk.green(programName));
 
 // --- Start SEO file copying logic ---
 console.log(chalk.blue('Copying SEO files for applications...'));
-const appRoots = findAppRoots(APP_DIR);
-for (const appRoot of appRoots) {
-    copySeoFilesForApp(appRoot, env);
+const appData = findAppRoots(APP_DIR, APP_DIR); // Pass APP_DIR as baseAppDir
+for (const app of appData) {
+    copySeoFilesForApp(app.appRootPath, app.appRelativePath, env);
 }
 // --- End SEO file copying logic ---