rootless-config 1.4.5 → 1.4.6

package/package.json

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

package/src/cli/commands/setup.js

@@ -1,21 +1,62 @@
-/*-------- rootless setup — one-command full root cleanup: init + migrate + prepare --------*/
+/*-------- rootless setup — one-command full root cleanup: init + normalize + migrate --------*/
 
 import path from 'node:path'
+import { readdir, rename } from 'node:fs/promises'
 import { createLogger } from '../../utils/logger.js'
+import { fileExists, ensureDir } from '../../utils/fsUtils.js'
+import { isPatchable } from '../../core/scriptPatcher.js'
 import initCmd from './init.js'
 import migrateCmd from './migrate.js'
-import prepareCmd from './prepare.js'
+
+/**
+ * Fix files that ended up in the wrong subfolder from a previous run.
+ * Non-patchable files (web assets, docs, scripts) belong in assets/, not configs/.
+ */
+async function normalizeContainer(containerPath, logger) {
+  const configsDir = path.join(containerPath, 'configs')
+  const assetsDir  = path.join(containerPath, 'assets')
+  if (!(await fileExists(configsDir))) return
+
+  const entries = await readdir(configsDir, { withFileTypes: true })
+  for (const e of entries) {
+    if (!e.isFile()) continue
+    if (!isPatchable(e.name)) {
+      // Not a tool config — belongs in assets/
+      await ensureDir(assetsDir)
+      await rename(
+        path.join(configsDir, e.name),
+        path.join(assetsDir, e.name)
+      )
+      logger.debug(`Normalized: .root/configs/${e.name} → .root/assets/${e.name}`)
+    }
+  }
+}
+
+/**
+ * Count total files managed across all .root subdirectories.
+ */
+async function countRootFiles(containerPath) {
+  let total = 0
+  for (const sub of ['env', 'assets', 'configs']) {
+    const dir = path.join(containerPath, sub)
+    if (!(await fileExists(dir))) continue
+    const entries = await readdir(dir, { withFileTypes: true })
+    total += entries.filter(e => e.isFile()).length
+  }
+  return total
+}
 
 export default {
   name: 'setup',
-  description: 'Full automated root cleanup: init .root, migrate all configs, prepare (copy + patch scripts)',
+  description: 'Full automated root cleanup: move ALL files into .root container (run prepare to deploy)',
 
   async handler(args) {
     const logger = createLogger({ verbose: args.verbose ?? false })
     const projectRoot = args.cwd ? path.resolve(args.cwd) : process.cwd()
+    const containerPath = path.join(projectRoot, '.root')
 
     logger.info('─────────────────────────────────────────')
-    logger.info(' rootless setup — cleaning your project root')
+    logger.info(' rootless setup — moving everything to .root/')
     logger.info('─────────────────────────────────────────')
     logger.info('')
 
@@ -24,31 +65,36 @@ export default {
     await initCmd.handler({ ...args, yes: true, cwd: projectRoot })
     logger.info('')
 
-    // ── Step 2: migrate ───────────────────────────────────────────────────────
-    logger.info('[2/3] Migrating config files from root…')
-    await migrateCmd.handler({ ...args, yes: true, cwd: projectRoot })
+    // ── Step 2: normalize existing .root structure (fix misrouted files) ──────
+    logger.info('[2/3] Normalizing .root structure…')
+    await normalizeContainer(containerPath, logger)
+    logger.success('Structure normalized')
     logger.info('')
 
-    // ── Step 3: prepare ───────────────────────────────────────────────────────
-    logger.info('[3/3] Preparing root (copying mandatory files + patching scripts)…')
-    await prepareCmd.handler({ ...args, yes: true })
+    // ── Step 3: migrate ALL files from root → .root/ ──────────────────────────
+    logger.info('[3/3] Migrating files from root to .root/…')
+    await migrateCmd.handler({ ...args, yes: true, cwd: projectRoot })
     logger.info('')
 
+    const total = await countRootFiles(containerPath)
+
     logger.info('─────────────────────────────────────────')
-    logger.success('Setup complete! Your project root is clean.')
-    logger.info('')
-    logger.info('What happened:')
-    logger.info('  • .root/ container created')
-    logger.info('  • Config files moved to .root/configs/')
-    logger.info('  • .env* files moved to .root/env/')
-    logger.info('  • Web assets moved to .root/assets/')
-    logger.info('  • package.json scripts patched with --config flags')
-    logger.info('  • Mandatory files copied back to root (prepare)')
-    logger.info('')
-    logger.info('Next steps:')
-    logger.info('  rootless status   — see what\'s managed')
-    logger.info('  rootless doctor   — verify everything is wired correctly')
-    logger.info('  rootless watch    — auto-sync on .root/ changes')
+    logger.success(`Setup complete! ${total} files now managed in .root/`)
+    logger.info('')
+    logger.info('Your project root is now clean.')
+    logger.info('')
+    logger.info('  .root/env/     — .env files')
+    logger.info('  .root/assets/  — web assets, HTML, images, scripts')
+    logger.info('  .root/configs/ — tool configs (vite, eslint, jest…)')
+    logger.info('')
+    logger.info('⚠  Files were REMOVED from root.')
+    logger.info('   Run the following before deploying or starting the project:')
+    logger.info('')
+    logger.info('   rootless prepare   — copies files back to root')
+    logger.info('')
+    logger.info('Other commands:')
+    logger.info('  rootless status    — show what is managed')
+    logger.info('  rootless watch     — auto-sync .root/ → root on changes')
     logger.info('─────────────────────────────────────────')
   },
 }