bmad-plus 0.17.0 → 0.17.1

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

@@ -8,14 +8,19 @@ const { safeAdapterPath } = require('./installed-adapters');
 const POLICY_FILE = '_bmad/update-policy.json';
 const OFFICIAL_REGISTRY = 'https://registry.npmjs.org/';
 const DEFAULT_POLICY = Object.freeze({
-  mode: 'notify', channel: 'latest', allowedRange: null,
-  allowPrerelease: false, registry: OFFICIAL_REGISTRY,
+  mode: 'notify',
+  channel: 'latest',
+  allowedRange: null,
+  allowPrerelease: false,
+  registry: OFFICIAL_REGISTRY,
 });
 
 function isExactVersion(version) {
-  return typeof version === 'string' &&
+  return (
+    typeof version === 'string' &&
     /^\d+\.\d+\.\d+(?:-[0-9A-Za-z.-]+)?(?:\+[0-9A-Za-z.-]+)?$/.test(version) &&
-    semver.valid(version) !== null;
+    semver.valid(version) !== null
+  );
 }
 
 function validatePolicy(value) {
@@ -27,13 +32,21 @@ function validatePolicy(value) {
   }
   const policy = { ...DEFAULT_POLICY, ...value };
   delete policy.schemaVersion;
-  if (Object.keys(policy).some(key => !Object.hasOwn(DEFAULT_POLICY, key)) ||
-      !['off', 'notify', 'auto'].includes(policy.mode) ||
-      typeof policy.channel !== 'string' || !/^[a-z][a-z0-9_-]{0,31}$/.test(policy.channel) ||
-      typeof policy.allowPrerelease !== 'boolean' || policy.registry !== OFFICIAL_REGISTRY ||
-      (policy.allowedRange !== null && (typeof policy.allowedRange !== 'string' ||
-        !policy.allowedRange.trim() || !semver.validRange(policy.allowedRange)))) {
-    throw new Error('Invalid update policy: check mode, channel, range, prerelease flag and official registry.');
+  if (
+    Object.keys(policy).some((key) => !Object.hasOwn(DEFAULT_POLICY, key)) ||
+    !['off', 'notify', 'auto'].includes(policy.mode) ||
+    typeof policy.channel !== 'string' ||
+    !/^[a-z][a-z0-9_-]{0,31}$/.test(policy.channel) ||
+    typeof policy.allowPrerelease !== 'boolean' ||
+    policy.registry !== OFFICIAL_REGISTRY ||
+    (policy.allowedRange !== null &&
+      (typeof policy.allowedRange !== 'string' ||
+        !policy.allowedRange.trim() ||
+        !semver.validRange(policy.allowedRange)))
+  ) {
+    throw new Error(
+      'Invalid update policy: check mode, channel, range, prerelease flag and official registry.'
+    );
   }
   if (policy.mode === 'auto' && policy.allowedRange === null) {
     throw new Error('Automatic updates require an explicit allowedRange.');
@@ -58,11 +71,18 @@ function writeProjectJson(projectDir, relativeFile, data) {
   const temporaryName = `${relativeFile}.${process.pid}.${crypto.randomUUID()}.tmp`;
   const temporary = safeAdapterPath(projectDir, temporaryName);
   try {
-    fs.writeFileSync(temporary, JSON.stringify(data, null, 2) + '\n', { encoding: 'utf8', flag: 'wx' });
+    fs.writeFileSync(temporary, JSON.stringify(data, null, 2) + '\n', {
+      encoding: 'utf8',
+      flag: 'wx',
+    });
     safeAdapterPath(projectDir, relativeFile);
     fs.renameSync(temporary, file);
   } finally {
-    try { fs.unlinkSync(temporary); } catch { /* Preserve the original write error if cleanup fails. */ }
+    try {
+      fs.unlinkSync(temporary);
+    } catch {
+      /* Preserve the original write error if cleanup fails. */
+    }
   }
 }
 
@@ -74,17 +94,35 @@ function writeUpdatePolicy(projectDir, value) {
 
 function authorizeTarget(value, version) {
   let policy;
-  try { policy = validatePolicy(value); } catch { return { allowed: false, reason: 'invalid-policy' }; }
+  try {
+    policy = validatePolicy(value);
+  } catch {
+    return { allowed: false, reason: 'invalid-policy' };
+  }
   if (!isExactVersion(version)) return { allowed: false, reason: 'invalid-target-version' };
-  if (policy.mode !== 'auto') return { allowed: false, reason: policy.mode === 'off' ? 'updates-disabled' : 'approval-required' };
-  if (!policy.allowPrerelease && semver.prerelease(version)) return { allowed: false, reason: 'prerelease-not-allowed' };
-  if (!semver.satisfies(version, policy.allowedRange, { includePrerelease: policy.allowPrerelease })) {
+  if (policy.mode !== 'auto')
+    return {
+      allowed: false,
+      reason: policy.mode === 'off' ? 'updates-disabled' : 'approval-required',
+    };
+  if (!policy.allowPrerelease && semver.prerelease(version))
+    return { allowed: false, reason: 'prerelease-not-allowed' };
+  if (
+    !semver.satisfies(version, policy.allowedRange, { includePrerelease: policy.allowPrerelease })
+  ) {
     return { allowed: false, reason: 'outside-allowed-range' };
   }
   return { allowed: true, reason: 'authorized-policy' };
 }
 
 module.exports = {
-  DEFAULT_POLICY, POLICY_FILE, OFFICIAL_REGISTRY, isExactVersion,
-  validatePolicy, readUpdatePolicy, writeUpdatePolicy, authorizeTarget, writeProjectJson,
+  DEFAULT_POLICY,
+  POLICY_FILE,
+  OFFICIAL_REGISTRY,
+  isExactVersion,
+  validatePolicy,
+  readUpdatePolicy,
+  writeUpdatePolicy,
+  authorizeTarget,
+  writeProjectJson,
 };