ultimate-jekyll-manager 1.9.24 → 1.9.26

package/bin/ultimate-jekyll

@@ -1,5 +1,9 @@
 #!/usr/bin/env node
-const argv = require('yargs')(process.argv.slice(2)).parseSync();
+// Value-less flags must be declared boolean — otherwise yargs treats the next
+// positional as the flag's VALUE (`mgr test --extended some/target` became
+// extended='some/target' with NO target, silently running EVERYTHING in
+// normal mode). Mirrors the same fix in BEM's CLI.
+const argv = require('yargs')(process.argv.slice(2)).boolean(['extended']).parseSync();
 const cli = new (require('../dist/cli.js'))(argv);
 (async function() {
   'use strict';

package/dist/gulp/tasks/defaults.js

@@ -55,6 +55,11 @@ const FILE_MAP = {
   'src/**/*': {
     overwrite: false,
   },
+  // Consumer-owned after seeding (e.g. test/_init.js fixture hooks) — copy when
+  // missing, never clobber the consumer's version on setup reruns
+  'test/**/*': {
+    overwrite: false,
+  },
   'src/**/*.{html,md,json}': {
     skip: (file) => {
       // Get the name
@@ -604,6 +609,9 @@ function defaultsWatcher(complete) {
 // Default Task
 module.exports = series(defaults, defaultsWatcher);
 
+// Exposed for the build-layer FILE_MAP test (defaults-file-options.test.js)
+module.exports.getFileOptions = getFileOptions;
+
 function getFileOptions(filePath) {
   const defaults = {
     overwrite: true,

package/dist/test/suites/build/defaults-file-options.test.js

@@ -0,0 +1,59 @@
+// getFileOptions from src/gulp/tasks/defaults.js — the FILE_MAP rules that decide
+// which shipped defaults may overwrite consumer files on `npx mgr setup` reruns.
+// Regression here silently clobbers consumer-owned files (test/_init.js was reset
+// to the stub on every setup because no rule matched it and the fall-through
+// default is overwrite: true).
+
+module.exports = {
+  layer: 'build',
+  description: 'defaults FILE_MAP (gulp/tasks/defaults.js getFileOptions)',
+  type: 'group',
+  tests: [
+    {
+      name: 'consumer-owned seeds are copy-once (test/, hooks/, src/)',
+      run: async (ctx) => {
+        const { getFileOptions } = require('../../../gulp/tasks/defaults.js');
+
+        // The regression case: consumer fixture hook must survive setup reruns
+        ctx.expect(getFileOptions('test/_init.js').overwrite).toBe(false);
+        ctx.expect(getFileOptions('test/README.md').overwrite).toBe(false);
+
+        ctx.expect(getFileOptions('hooks/build/pre.js').overwrite).toBe(false);
+        ctx.expect(getFileOptions('src/assets/js/main.js').overwrite).toBe(false);
+        ctx.expect(getFileOptions('src/service-worker.js').overwrite).toBe(false);
+      },
+    },
+    {
+      name: 'framework-owned files always overwrite',
+      run: async (ctx) => {
+        const { getFileOptions } = require('../../../gulp/tasks/defaults.js');
+
+        const workflow = getFileOptions('.github/workflows/build.yml');
+        ctx.expect(workflow.overwrite).toBe(true);
+        ctx.expect(!!workflow.template).toBe(true);
+
+        // Last-match-wins: team images override the earlier src/**/* (false) rule
+        ctx.expect(getFileOptions('src/assets/images/team/rare-ivy/profile.jpg').overwrite).toBe(true);
+      },
+    },
+    {
+      name: 'merge and skip rules resolve as declared',
+      run: async (ctx) => {
+        const { getFileOptions } = require('../../../gulp/tasks/defaults.js');
+
+        ctx.expect(getFileOptions('CLAUDE.md').mergeLines).toBe(true);
+
+        const config = getFileOptions('config/ultimate-jekyll-manager.json');
+        ctx.expect(config.overwrite).toBe(true);
+        ctx.expect(config.merge).toBe(true);
+
+        ctx.expect(getFileOptions('test/.DS_Store').skip).toBe(true);
+
+        // Unmatched files fall through to overwrite: true — any new consumer-owned
+        // default MUST get an explicit rule or setup will clobber it
+        ctx.expect(getFileOptions('dist/robots.txt').overwrite).toBe(true);
+        ctx.expect(getFileOptions('dist/robots.txt').rule).toBeNull();
+      },
+    },
+  ],
+};

package/docs/build-system.md

@@ -22,6 +22,19 @@ serve → build → developmentRebuild
 
 Pure helpers are exposed under [src/gulp/tasks/utils/](../src/gulp/tasks/utils/) (`merge-jekyll-configs`, `_validate-yaml`, `template-transform`, `collectTextNodes`, `dictionary`, `github-cache`, `formatDocument`) — these are the highest-value test targets (zero I/O, callable directly in `build`-layer tests).
 
+## Defaults distribution (`defaults` task)
+
+The `defaults` task copies `dist/defaults/**` from the framework into the consumer project on every build/setup. Per-file behavior is decided by `FILE_MAP` in [defaults.js](../src/gulp/tasks/defaults.js) (`getFileOptions` — **last matching glob wins**), in four classes:
+
+| Class | Behavior | Rules |
+|---|---|---|
+| Copy-once (consumer-owned) | Seeded when missing, NEVER overwritten | `**/*.md`, `hooks/**/*`, `src/**/*`, `test/**/*` (e.g. `test/_init.js`) |
+| Always-overwrite (framework-owned) | Replaced every run | `.github/workflows/build.yml`, `src/assets/images/team/**/*`, and **any file with no matching rule** (fall-through default) |
+| Merge | Combined with the consumer's version | `config/ultimate-jekyll-manager.json` (deep config merge), `CLAUDE.md` / `_.env` / `_.gitignore` (marker-based line merge) |
+| Skip | Never copied | `**/.DS_Store`, `**/__temp/**/*` |
+
+⚠️ The fall-through default is `overwrite: true` — when adding a new consumer-owned file to `src/defaults/`, it MUST get an explicit copy-once rule or setup reruns will clobber the consumer's edits. The rule classes are locked by the build-layer test `defaults-file-options.test.js`.
+
 ## Config flow
 
 Three config files in the consumer project feed the build:

package/docs/test-framework.md

@@ -241,6 +241,8 @@ Same protocol as EM (`__EM_TEST__`) and BXM (`__BXM_TEST__`). One marker per fra
 
 The runner loads an optional `test/_init.js` from **both** test roots — the framework (`<UJM>/test/_init.js`) and the consumer project (`<cwd>/test/_init.js`) — and runs it **once, before any suite** (it is NOT itself run as a test; the `_`-prefix keeps it out of discovery). Mirrors the same hook in BEM/EM/BXM so all four frameworks share one shape.
 
+The stub shipped in `src/defaults/test/_init.js` is **seeded once and never overwritten** — the file is consumer-owned, so setup reruns preserve whatever fixture logic the project puts in it (`test/**/*` is copy-once in the defaults task's `FILE_MAP`; see [build-system.md](build-system.md#defaults-distribution-defaults-task)).
+
 The module **must export a function** — `module.exports = (ctx) => ({ ... })` — called with `{ projectRoot }` and returning the hook object. It may declare:
 
 - `async setup({ projectRoot })` — runs once before the suites, e.g. to scaffold a fixture file the boot layer needs.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ultimate-jekyll-manager",
-  "version": "1.9.24",
+  "version": "1.9.26",
   "description": "Ultimate Jekyll dependency manager",
   "main": "dist/index.js",
   "files": [