rootless-config 1.4.4 → 1.4.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rootless-config",
-  "version": "1.4.4",
+  "version": "1.4.5",
   "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

@@ -5,20 +5,38 @@ import { readdir, unlink } from 'node:fs/promises'
 import { createLogger } from '../../utils/logger.js'
 import { fileExists, ensureDir, atomicWrite, copyFile, readJsonFile } from '../../utils/fsUtils.js'
 import { confirm } from '../../utils/prompt.js'
-import { patchPackageScripts, isRootRequired, isPatchable, isWebAsset, NEVER_MIGRATE } from '../../core/scriptPatcher.js'
+import { patchPackageScripts, isPatchable, NEVER_MIGRATE } from '../../core/scriptPatcher.js'
 
-// A file is migratable when rootless knows what to do with it:
-//  - isRootRequired → must be in root (will be copied there by prepare)
-//  - isPatchable    → can be redirected via --config flag in package.json scripts
-// Anything that's NEVER_MIGRATE (lock files, package.json) is excluded.
+// Files that are OS/editor/VCS noise — never migrate, never touch
+const SYSTEM_FILES = new Set([
+  '.DS_Store', 'Thumbs.db', 'desktop.ini', '.git', '.svn',
+])
 
+/**
+ * Determine which .root sub-folder a file should land in:
+ *  env/     — .env* files (always copied to root by prepare)
+ *  configs/ — tool configs that support --config flags (scripts get patched)
+ *  assets/  — everything else (always copied to root by prepare)
+ */
+function getDestSubdir(filename) {
+  if (/^\.env/.test(filename)) return 'env'
+  if (isPatchable(filename)) return 'configs'
+  return 'assets'
+}
+
+/**
+ * Returns ALL files in projectRoot that should be migrated.
+ * Excludes: directories, package manager lock files, OS noise.
+ */
 async function findMigratableFiles(projectRoot) {
   const entries = await readdir(projectRoot, { withFileTypes: true })
   return entries
     .filter(e => {
       if (!e.isFile()) return false
       if (NEVER_MIGRATE.has(e.name)) return false
-      return isRootRequired(e.name) || isPatchable(e.name)
+      if (SYSTEM_FILES.has(e.name)) return false
+      if (e.name === '.root') return false
+      return true  // migrate EVERYTHING else
     })
     .map(e => e.name)
     .sort()
@@ -66,8 +84,7 @@ export default {
 
     for (const name of candidates) {
       const src = path.join(projectRoot, name)
-      const isEnv = /^\.env/.test(name)
-      const destSubdir = isEnv ? 'env' : (isWebAsset(name) ? 'assets' : 'configs')
+      const destSubdir = getDestSubdir(name)
       const destDir = path.join(containerPath, destSubdir)
 
       await ensureDir(destDir)