rootless-config 1.1.0 → 1.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rootless-config",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "description": "Store project config files outside the project root, auto-deploy them where tools expect them.",
   "type": "module",
   "bin": {

package/src/cli/commands/migrate.js

@@ -9,12 +9,18 @@ import { patchPackageScripts } from '../../core/scriptPatcher.js'
 
 const CONFIG_PATTERN = /\.(config|rc)\.(js|ts|mjs|cjs|json)$|\.eslintrc$|\.babelrc$|tsconfig.*\.json$/
 const ENV_PATTERN = /^\.env/
+// Dot-files and special files that must be in root — auto-detected
+const DOTFILE_PATTERN = /^\.(nvmrc|node-version|npmrc|yarnrc|yarnrc\.yml|pnpmfile\.cjs|editorconfig|gitignore|gitattributes|dockerignore|browserslistrc|prettierignore|eslintignore|stylelintignore|czrc|cz\.json|commitlintrc.*|lintstagedrc.*|releaserc.*)$|^(Dockerfile.*|Procfile|browserslist)$/
 
 async function findMigratableFiles(projectRoot) {
   const entries = await readdir(projectRoot, { withFileTypes: true })
   return entries
-    .filter(e => e.isFile() && (CONFIG_PATTERN.test(e.name) || ENV_PATTERN.test(e.name)))
-    .filter(e => e.name !== 'package.json') // never migrate package.json
+    .filter(e => e.isFile() && (
+      CONFIG_PATTERN.test(e.name) ||
+      ENV_PATTERN.test(e.name) ||
+      DOTFILE_PATTERN.test(e.name)
+    ))
+    .filter(e => e.name !== 'package.json')
     .map(e => e.name)
     .sort()
 }

package/src/core/cliWrapper.js

@@ -13,7 +13,7 @@ import { readManifest, removeFromManifest } from './generatedManifest.js'
 import { removeHash } from './fileHashCache.js'
 import { resolveProjectRoot, resolveContainerPath, setContainerOverride } from './pathResolver.js'
 import { fileExists, removeFile } from '../utils/fsUtils.js'
-import { patchPackageScripts } from './scriptPatcher.js'
+import { patchPackageScripts, isRootRequired } from './scriptPatcher.js'
 import { ValidationError } from '../types/errors.js'
 
 async function loadConfig(containerPath) {
@@ -50,28 +50,54 @@ async function preparePipeline(options = {}) {
   const containerPath = await resolveContainerPath()
   const config = ctx.container.manifest ?? {}
   const graph = await getGraph(config)
-  const isCleanMode = (config.mode ?? 'proxy') === 'clean'
+  const isCleanMode = (config.mode ?? 'clean') === 'clean'
 
   const projectRoot = await resolveProjectRoot()
 
   if (isCleanMode) {
-    // Clean mode: only copy .env files to root, patch package.json scripts for everything else
-    const envDir = path.join(containerPath, 'env')
+    const envDir     = path.join(containerPath, 'env')
     const configsDir = path.join(containerPath, 'configs')
+    const assetsDir  = path.join(containerPath, 'assets')
 
-    const envEntries = []
+    const copyEntries = []
+
+    // 1. env/* — all .env files always go to root
     if (await fileExists(envDir)) {
-      const envFiles = await readdir(envDir, { withFileTypes: true })
-      for (const f of envFiles) {
+      for (const f of await readdir(envDir, { withFileTypes: true })) {
         if (!f.isFile()) continue
         const absPath = path.join(envDir, f.name)
         const plugin = ctx.pluginRegistry.resolve(absPath)[0] ?? null
-        envEntries.push({ source: absPath, target: path.join(projectRoot, f.name), mode: 'copy', plugin })
+        copyEntries.push({ source: absPath, target: path.join(projectRoot, f.name), mode: 'copy', plugin })
+      }
+    }
+
+    // 2. assets/* — favicon, robots.txt, manifests etc. always go to root
+    if (await fileExists(assetsDir)) {
+      for (const f of await readdir(assetsDir, { withFileTypes: true })) {
+        if (!f.isFile()) continue
+        const absPath = path.join(assetsDir, f.name)
+        const plugin = ctx.pluginRegistry.resolve(absPath)[0] ?? null
+        copyEntries.push({ source: absPath, target: path.join(projectRoot, f.name), mode: 'copy', plugin })
+      }
+    }
+
+    // 3. configs/* — split by whether the file can be redirected via CLI flags
+    //    isRootRequired(name) = true  → copy to root (tool can't be redirected: .nvmrc, .editorconfig, etc.)
+    //    isRootRequired(name) = false → skip copy, only patch scripts below
+    if (await fileExists(configsDir)) {
+      for (const f of await readdir(configsDir, { withFileTypes: true })) {
+        if (!f.isFile()) continue
+        if (isRootRequired(f.name)) {
+          const absPath = path.join(configsDir, f.name)
+          const plugin = ctx.pluginRegistry.resolve(absPath)[0] ?? null
+          copyEntries.push({ source: absPath, target: path.join(projectRoot, f.name), mode: 'copy', plugin })
+        }
       }
     }
 
-    const results = await generateFiles(envEntries, options, ctx.logger)
+    const results = await generateFiles(copyEntries, options, ctx.logger)
 
+    // 4. Patch package.json scripts for all patchable tool configs (vite, eslint, jest, pm2, etc.)
     const patched = await patchPackageScripts(projectRoot, configsDir, ctx.logger)
     if (patched) ctx.logger.success('package.json scripts patched with --config flags')
     else ctx.logger.debug('No script patching needed')
@@ -80,7 +106,7 @@ async function preparePipeline(options = {}) {
     return results
   }
 
-  // Proxy mode (default): generate proxy/copy for every file in .root/
+  // Proxy mode: generate proxy/copy for every file in .root/
   const entries = await buildGenerationEntries(containerPath, graph, ctx)
   const mapped = entries.map(e => ({
     ...e,

package/src/core/scriptPatcher.js

@@ -5,23 +5,110 @@ import { readdir } from 'node:fs/promises'
 import { readJsonFile, writeJsonFile, fileExists } from '../utils/fsUtils.js'
 
 /**
- * Known tools: what config files they use and which flag points to a config.
+ * Tools that support a --config (or similar) flag.
+ * These files can be REMOVED from root — scripts get patched instead.
  * Ordered by specificity (longer command first to avoid partial matches).
  */
 const TOOL_CONFIGS = [
-  { cmd: 'vite build',  flag: '--config', files: ['vite.config.js', 'vite.config.ts', 'vite.config.mjs'] },
-  { cmd: 'vite',        flag: '--config', files: ['vite.config.js', 'vite.config.ts', 'vite.config.mjs'] },
-  { cmd: 'vitest',      flag: '--config', files: ['vitest.config.js', 'vitest.config.ts', 'vitest.config.mjs'] },
-  { cmd: 'eslint',      flag: '--config', files: ['eslint.config.js', 'eslint.config.mjs', 'eslint.config.cjs'] },
-  { cmd: 'prettier',    flag: '--config', files: ['prettier.config.js', 'prettier.config.cjs', '.prettierrc.js'] },
-  { cmd: 'jest',        flag: '--config', files: ['jest.config.js', 'jest.config.ts', 'jest.config.mjs', 'jest.config.cjs'] },
-  { cmd: 'webpack',     flag: '--config', files: ['webpack.config.js', 'webpack.config.ts', 'webpack.config.mjs'] },
-  { cmd: 'rollup',      flag: '--config', files: ['rollup.config.js', 'rollup.config.ts', 'rollup.config.mjs'] },
-  { cmd: 'tsup',        flag: '--config', files: ['tsup.config.js', 'tsup.config.ts'] },
-  { cmd: 'tsc',         flag: '--project', files: ['tsconfig.json', 'tsconfig.build.json'] },
-  { cmd: 'nodemon',     flag: '--config', files: ['nodemon.json', '.nodemonrc', 'nodemon.config.js'] },
+  { cmd: 'vite build',       flag: '--config',  files: ['vite.config.js', 'vite.config.ts', 'vite.config.mjs'] },
+  { cmd: 'vite preview',     flag: '--config',  files: ['vite.config.js', 'vite.config.ts', 'vite.config.mjs'] },
+  { cmd: 'vite',             flag: '--config',  files: ['vite.config.js', 'vite.config.ts', 'vite.config.mjs'] },
+  { cmd: 'vitest run',       flag: '--config',  files: ['vitest.config.js', 'vitest.config.ts', 'vitest.config.mjs'] },
+  { cmd: 'vitest',           flag: '--config',  files: ['vitest.config.js', 'vitest.config.ts', 'vitest.config.mjs'] },
+  { cmd: 'eslint',           flag: '--config',  files: ['eslint.config.js', 'eslint.config.mjs', 'eslint.config.cjs'] },
+  { cmd: 'prettier',         flag: '--config',  files: ['prettier.config.js', 'prettier.config.cjs', '.prettierrc.js', 'prettier.config.mjs'] },
+  { cmd: 'jest',             flag: '--config',  files: ['jest.config.js', 'jest.config.ts', 'jest.config.mjs', 'jest.config.cjs'] },
+  { cmd: 'webpack serve',    flag: '--config',  files: ['webpack.config.js', 'webpack.config.ts', 'webpack.config.mjs'] },
+  { cmd: 'webpack',          flag: '--config',  files: ['webpack.config.js', 'webpack.config.ts', 'webpack.config.mjs'] },
+  { cmd: 'rollup',           flag: '--config',  files: ['rollup.config.js', 'rollup.config.ts', 'rollup.config.mjs'] },
+  { cmd: 'tsup',             flag: '--config',  files: ['tsup.config.js', 'tsup.config.ts', 'tsup.config.mjs'] },
+  { cmd: 'tsc',              flag: '--project', files: ['tsconfig.json', 'tsconfig.build.json', 'tsconfig.prod.json'] },
+  { cmd: 'nodemon',          flag: '--config',  files: ['nodemon.json', '.nodemonrc', 'nodemon.config.js'] },
+  { cmd: 'stylelint',        flag: '--config',  files: ['stylelint.config.js', 'stylelint.config.mjs', 'stylelint.config.cjs', '.stylelintrc.js', '.stylelintrc.json'] },
+  { cmd: 'next build',       flag: '--',        files: [] }, // next doesn't support --config; handled separately
+  { cmd: 'tailwindcss',      flag: '--config',  files: ['tailwind.config.js', 'tailwind.config.ts', 'tailwind.config.mjs'] },
+  { cmd: 'postcss',          flag: '--config',  files: ['postcss.config.js', 'postcss.config.mjs', 'postcss.config.cjs'] },
+  { cmd: 'babel',            flag: '--config-file', files: ['babel.config.js', 'babel.config.mjs', 'babel.config.cjs', 'babel.config.json'] },
+  { cmd: 'mocha',            flag: '--config',  files: ['.mocharc.js', '.mocharc.cjs', '.mocharc.yml', '.mocharc.json', 'mocha.config.js'] },
+  { cmd: 'nyc',              flag: '--config',  files: ['.nycrc', '.nycrc.json', 'nyc.config.js', 'nyc.config.cjs'] },
+  { cmd: 'cypress run',      flag: '--config-file', files: ['cypress.config.js', 'cypress.config.ts', 'cypress.config.mjs'] },
+  { cmd: 'cypress open',     flag: '--config-file', files: ['cypress.config.js', 'cypress.config.ts', 'cypress.config.mjs'] },
+  { cmd: 'playwright test',  flag: '--config',  files: ['playwright.config.js', 'playwright.config.ts', 'playwright.config.mjs'] },
 ]
 
+/**
+ * Tools that use the config file as a POSITIONAL argument (not a --flag).
+ * These files can also be removed from root — the argument in the script is patched.
+ */
+const POSITIONAL_TOOLS = [
+  { cmd: 'pm2 start',   files: ['ecosystem.config.js', 'ecosystem.config.cjs', 'ecosystem.config.mjs'] },
+  { cmd: 'pm2 restart', files: ['ecosystem.config.js', 'ecosystem.config.cjs', 'ecosystem.config.mjs'] },
+  { cmd: 'pm2 reload',  files: ['ecosystem.config.js', 'ecosystem.config.cjs', 'ecosystem.config.mjs'] },
+]
+
+/**
+ * Files that MUST be physically present in the project root.
+ * These cannot be redirected via CLI flags — tools discover them by convention.
+ * Everything in .root/configs/ NOT matching TOOL_CONFIGS or POSITIONAL_TOOLS falls into this category,
+ * but we keep an explicit list for clarity and migrate detection.
+ */
+const ROOT_REQUIRED_FILES = new Set([
+  // Node.js version managers
+  '.nvmrc', '.node-version',
+  // Package managers
+  '.npmrc', '.yarnrc', '.yarnrc.yml', '.pnpmfile.cjs',
+  // Editor
+  '.editorconfig',
+  // Git (root discovery via git, these are auto-found)
+  '.gitignore', '.gitattributes', '.gitmodules',
+  // Docker
+  'Dockerfile', '.dockerignore',
+  // Process / deployment
+  'Procfile',
+  // Browser targets
+  '.browserslistrc', 'browserslist',
+  // Lint ignore files (eslint/prettier look for these automatically)
+  '.eslintignore', '.prettierignore', '.stylelintignore',
+  // Commitlint / lint-staged / release (auto-discovery only)
+  'commitlint.config.js', 'commitlint.config.cjs', 'commitlint.config.mjs',
+  '.commitlintrc', '.commitlintrc.js', '.commitlintrc.json',
+  '.lintstagedrc', '.lintstagedrc.js', '.lintstagedrc.json', '.lintstagedrc.cjs',
+  'lint-staged.config.js', 'lint-staged.config.mjs', 'lint-staged.config.cjs',
+  '.releaserc', '.releaserc.js', '.releaserc.json', '.releaserc.yml',
+  'release.config.js', 'release.config.cjs',
+  // Semantic tooling
+  '.czrc', '.cz.json',
+  // HTTPS / certs
+  '.htaccess',
+  // Web assets
+  'robots.txt', 'sitemap.xml', 'favicon.ico', 'manifest.json', 'site.webmanifest',
+])
+
+/**
+ * Returns true if the file can be redirected away from root via CLI flags.
+ * Files where this returns false MUST be copied to root.
+ */
+function isPatchable(filename) {
+  return (
+    TOOL_CONFIGS.some(t => t.files.includes(filename)) ||
+    POSITIONAL_TOOLS.some(t => t.files.includes(filename))
+  )
+}
+
+/**
+ * Returns true if the file must always be physically present in root.
+ */
+function isRootRequired(filename) {
+  if (ROOT_REQUIRED_FILES.has(filename)) return true
+  // .env* patterns
+  if (/^\.env/.test(filename)) return true
+  // Dockerfile variants: Dockerfile.dev, Dockerfile.prod etc.
+  if (/^Dockerfile/.test(filename)) return true
+  // Not patchable and not in ROOT_REQUIRED_FILES → still copy to be safe
+  if (!isPatchable(filename)) return true
+  return false
+}
+
 /**
  * Build a map of { cmd → relative config path } based on what actually exists in configsDir.
  * configsDir is the absolute path to .root/configs/
@@ -36,14 +123,28 @@ async function buildConfigMap(configsDir, projectRoot) {
   }
 
   const map = {}
+
+  // Flag-based tools
   for (const tool of TOOL_CONFIGS) {
-    if (map[tool.cmd]) continue // already mapped (longer cmd took priority)
+    if (!tool.files.length) continue
+    if (map[tool.cmd]) continue
+    const found = tool.files.find(f => files.includes(f))
+    if (!found) continue
+    const abs = path.join(configsDir, found)
+    const rel = path.relative(projectRoot, abs).replace(/\\/g, '/')
+    map[tool.cmd] = { path: rel.startsWith('.') ? rel : `./${rel}`, flag: tool.flag, positional: false }
+  }
+
+  // Positional tools (pm2 etc.)
+  for (const tool of POSITIONAL_TOOLS) {
+    if (map[tool.cmd]) continue
     const found = tool.files.find(f => files.includes(f))
     if (!found) continue
     const abs = path.join(configsDir, found)
     const rel = path.relative(projectRoot, abs).replace(/\\/g, '/')
-    map[tool.cmd] = rel.startsWith('.') ? rel : `./${rel}`
+    map[tool.cmd] = { path: rel.startsWith('.') ? rel : `./${rel}`, flag: null, positional: true, originalFile: found }
   }
+
   return map
 }
 
@@ -52,20 +153,27 @@ async function buildConfigMap(configsDir, projectRoot) {
  * Handles semicolons, &&, || (multi-command scripts).
  */
 function patchScriptString(script, configMap) {
-  // Process longer commands first to avoid partial matches (vite build before vite)
+  // Sort by command length descending to avoid partial matches
   const sorted = Object.entries(configMap).sort((a, b) => b[0].length - a[0].length)
 
   let result = script
-  for (const [cmd, configPath] of sorted) {
-    const tool = TOOL_CONFIGS.find(t => t.cmd === cmd)
-    if (!tool) continue
-    const flag = tool.flag
-
-    // Match the command not already followed by its flag
-    // Word boundary on left, ensure flag not already present after the cmd
-    const escapedCmd = cmd.replace(/[.*+?^${}()|[\]\\]/g, '\\$&').replace(/\s+/g, '\\s+')
-    const regex = new RegExp(`((?:^|[;&|\\s]))(${escapedCmd})(?!\\s+${flag.replace(/-/g, '\\-')})`, 'g')
-    result = result.replace(regex, (_, pre, match) => `${pre}${match} ${flag} ${configPath}`)
+  for (const [cmd, info] of sorted) {
+    if (info.positional) {
+      // Replace filename argument directly: pm2 start ecosystem.config.cjs → pm2 start .root/...
+      for (const file of (POSITIONAL_TOOLS.find(t => t.cmd === cmd)?.files ?? [])) {
+        const escapedFile = file.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
+        const escapedCmd = cmd.replace(/[.*+?^${}()|[\]\\]/g, '\\$&').replace(/\s+/g, '\\s+')
+        const regex = new RegExp(`(${escapedCmd})\\s+${escapedFile}`, 'g')
+        result = result.replace(regex, `$1 ${info.path}`)
+      }
+    } else {
+      // Flag-based: insert --config <path> after command if not already present
+      const flag = info.flag
+      const escapedCmd = cmd.replace(/[.*+?^${}()|[\]\\]/g, '\\$&').replace(/\s+/g, '\\s+')
+      const escapedFlag = flag.replace(/-/g, '\\-')
+      const regex = new RegExp(`((?:^|[;&|\\s]))(${escapedCmd})(?!\\s+${escapedFlag})`, 'g')
+      result = result.replace(regex, (_, pre, match) => `${pre}${match} ${flag} ${info.path}`)
+    }
   }
   return result
 }
@@ -114,7 +222,7 @@ async function addPrepareHook(projectRoot) {
   if (!pkg.scripts) pkg.scripts = {}
 
   const existing = pkg.scripts.prepare ?? ''
-  if (existing.includes('rootless prepare')) return existing // already set
+  if (existing.includes('rootless prepare')) return existing
 
   pkg.scripts.prepare = existing
     ? `rootless prepare --yes && ${existing}`
@@ -124,4 +232,4 @@ async function addPrepareHook(projectRoot) {
   return pkg.scripts.prepare
 }
 
-export { patchPackageScripts, addPrepareHook, buildConfigMap, patchScriptString }
+export { patchPackageScripts, addPrepareHook, buildConfigMap, patchScriptString, isPatchable, isRootRequired }