bmad-plus 0.22.0 → 0.23.1

package/tools/cli/lib/update-dispatch.js

@@ -24,6 +24,14 @@ function refusal(reason, detail) {
   });
 }
 
+/**
+ * One refusal for an `--auto` request, wherever it is evaluated: a policy that
+ * is not in `auto` mode never reads as the generic `approval-required` status.
+ */
+function autoRefusal(reason) {
+  return refusal(reason === 'approval-required' ? 'automatic-updates-not-authorized' : reason);
+}
+
 function requirePolicy(policy, targetVersion, auto) {
   if (policy.registry !== OFFICIAL_REGISTRY) throw refusal('unsupported-registry');
   if (policy.mode === 'off') throw refusal('updates-disabled');
@@ -31,7 +39,7 @@ function requirePolicy(policy, targetVersion, auto) {
     throw refusal('prerelease-not-allowed');
   if (auto) {
     const authorization = authorizeTarget(policy, targetVersion);
-    if (!authorization.allowed) throw refusal(authorization.reason);
+    if (!authorization.allowed) throw autoRefusal(authorization.reason);
   }
 }
 
@@ -179,4 +187,4 @@ async function runLatestUpdate({ projectDir, auto = false, yes = false }, inject
   };
 }
 
-module.exports = { runLatestUpdate };
+module.exports = { runLatestUpdate, autoRefusal };

package/tools/cli/lib/update-transaction.js

@@ -78,11 +78,15 @@ function collectManagedFiles({ projectDir, packageRoot = DEFAULT_PACKAGE_ROOT, p
       pack: PACKS[id],
     });
     for (const entry of entries.files)
-      files.set(relativeFile(projectDir, entry.target), entry.source);
+      files.set(relativeFile(projectDir, entry.target), {
+        source: entry.source,
+        root: entry.root || packageRoot,
+      });
   }
-  for (const file of MODULE_FILES) files.set(file, path.join(bmadSrc, path.basename(file)));
-  return [...files].map(([file, source]) => {
-    safeTarget(packageRoot, relativeFile(packageRoot, source));
+  for (const file of MODULE_FILES)
+    files.set(file, { source: path.join(bmadSrc, path.basename(file)), root: packageRoot });
+  return [...files].map(([file, { source, root }]) => {
+    safeTarget(root, relativeFile(root, source));
     return { file, content: fs.readFileSync(source), kind: 'pack' };
   });
 }
@@ -260,8 +264,7 @@ function planUpdate({
   for (const { file } of files) safeTarget(projectDir, file);
   if (auto && (legacy || conflicts.length))
     throw new Error(
-      'Automatic update refused: ' +
-        (legacy ? 'legacy ownership inventory' : 'local file conflicts')
+      'Update refused: ' + (legacy ? 'legacy ownership inventory' : 'local file conflicts') + '.'
     );
   return {
     projectDir,