godpowers 3.11.0 → 3.13.1

package/lib/installer-args.js

@@ -25,9 +25,74 @@ const COMMANDS = new Set([
   'import-ledger'
 ]);
 
-function parseArgs(argv, cwd = process.cwd()) {
-  const args = argv.slice(2);
-  const opts = {
+// Per-command positional slots, filled in order into the first empty slot.
+const POSITIONALS = {
+  state: ['stateAction'],
+  verify: ['verifyCommand'],
+  route: ['routePrompt'],
+  memory: ['memoryAction', 'memoryKey', 'memoryValue'],
+  lesson: ['lessonAction', 'lessonText'],
+  outcome: ['outcomeAction', 'outcomeSlug']
+};
+
+// Boolean flags (and their aliases) -> opts field set to true.
+const BOOL_FLAGS = {
+  '--json': 'json',
+  '--brief': 'brief',
+  '--full': 'full',
+  '--apply': 'apply',
+  '--dry-run': 'dryRun',
+  '--attest': 'attest',
+  '--peek': 'peek',
+  '--all': 'all',
+  '-g': 'global',
+  '--global': 'global',
+  '-l': 'local',
+  '--local': 'local',
+  '-h': 'help',
+  '--help': 'help',
+  '-u': 'uninstall',
+  '--uninstall': 'uninstall'
+};
+
+// Value flags -> opts field. Each accepts both "--flag value" and "--flag=value".
+const VALUE_FLAGS = {
+  '--tier': 'tier',
+  '--step': 'step',
+  '--status': 'status',
+  '--substep': 'substep',
+  '--claim': 'claim',
+  '--timeout': 'timeout',
+  '--evidence': 'evidence',
+  '--since': 'since',
+  '--action': 'reflectAction',
+  '--outcome': 'outcome',
+  '--observation': 'observation',
+  '--root-cause': 'rootCause',
+  '--next': 'nextAction',
+  '--lesson': 'lesson',
+  '--category': 'category',
+  '--tags': 'tags',
+  '--scope': 'scope',
+  '--goal': 'outcomeGoal',
+  '--verify': 'outcomeVerify',
+  '--budget': 'budget',
+  '--reason': 'reason',
+  '--from': 'importFrom',
+  '--profile': 'profile',
+  '--project': 'project',
+  '--name': 'extensionName',
+  '--output': 'extensionOutput',
+  '--skill': 'extensionSkill',
+  '--agent': 'extensionAgent',
+  '--workflow': 'extensionWorkflow'
+};
+
+// Value flags whose value is resolved to an absolute path.
+const PATH_FLAGS = new Set(['--project', '--output']);
+
+function defaultOpts(cwd) {
+  return {
     command: null,
     project: cwd,
     json: false,
@@ -80,301 +145,86 @@ function parseArgs(argv, cwd = process.cwd()) {
     all: false,
     help: false,
     uninstall: false,
-    profile: 'core',
+    profile: 'core'
   };
+}
 
-  for (let i = 0; i < args.length; i++) {
-    const arg = args[i];
-    if (COMMANDS.has(arg)) {
-      opts.command = arg;
-      continue;
-    }
-    if (opts.command === 'state' && !opts.stateAction && !arg.startsWith('-')) {
-      opts.stateAction = arg;
-      continue;
-    }
-    if (opts.command === 'verify' && opts.verifyCommand === null && !arg.startsWith('-')) {
-      opts.verifyCommand = arg;
-      continue;
-    }
-    if (opts.command === 'route' && opts.routePrompt === null && !arg.startsWith('-')) {
-      opts.routePrompt = arg;
-      continue;
+function setValue(opts, flag, value) {
+  opts[VALUE_FLAGS[flag]] = PATH_FLAGS.has(flag) ? path.resolve(value) : value;
+}
+
+// Fill the first empty positional slot for the active command. Returns true when
+// the arg was consumed as a positional.
+function consumePositional(opts, arg) {
+  if (arg.startsWith('-')) return false;
+  const slots = POSITIONALS[opts.command];
+  if (!slots) return false;
+  for (const slot of slots) {
+    if (opts[slot] === null) {
+      opts[slot] = arg;
+      return true;
     }
-    if (opts.command === 'memory' && !arg.startsWith('-')) {
-      if (opts.memoryAction === null) { opts.memoryAction = arg; continue; }
-      if (opts.memoryKey === null) { opts.memoryKey = arg; continue; }
-      if (opts.memoryValue === null) { opts.memoryValue = arg; continue; }
+  }
+  return false;
+}
+
+// Handle one flag at args[i]. Returns the index to continue from (advanced past
+// a consumed value for space-form flags).
+function consumeFlag(opts, args, i) {
+  const arg = args[i];
+
+  if (arg in BOOL_FLAGS) {
+    opts[BOOL_FLAGS[arg]] = true;
+    return i;
+  }
+  if (arg === '--minimal') {
+    opts.profile = 'core';
+    return i;
+  }
+  if (arg === '--runtime') {
+    if (args[i + 1]) {
+      opts.runtimes.push(args[i + 1]);
+      return i + 1;
     }
-    if (opts.command === 'lesson' && !arg.startsWith('-')) {
-      if (opts.lessonAction === null) { opts.lessonAction = arg; continue; }
-      if (opts.lessonText === null) { opts.lessonText = arg; continue; }
+    return i;
+  }
+  if (arg.startsWith('--runtime=')) {
+    opts.runtimes.push(arg.slice('--runtime='.length));
+    return i;
+  }
+  if (arg in VALUE_FLAGS) {
+    if (args[i + 1]) {
+      setValue(opts, arg, args[i + 1]);
+      return i + 1;
     }
-    if (opts.command === 'outcome' && !arg.startsWith('-')) {
-      if (opts.outcomeAction === null) { opts.outcomeAction = arg; continue; }
-      if (opts.outcomeSlug === null) { opts.outcomeSlug = arg; continue; }
+    return i;
+  }
+  const eq = arg.indexOf('=');
+  if (arg.startsWith('--') && eq > 0) {
+    const name = arg.slice(0, eq);
+    if (name in VALUE_FLAGS) {
+      setValue(opts, name, arg.slice(eq + 1));
+      return i;
     }
+  }
+  if (arg.startsWith('--') && RUNTIMES[arg.slice(2)]) {
+    opts.runtimes.push(arg.slice(2));
+  }
+  return i;
+}
+
+function parseArgs(argv, cwd = process.cwd()) {
+  const args = argv.slice(2);
+  const opts = defaultOpts(cwd);
 
-    switch (arg) {
-      case '--json':
-        opts.json = true;
-        break;
-      case '--brief':
-        opts.brief = true;
-        break;
-      case '--full':
-        opts.full = true;
-        break;
-      case '--apply':
-        opts.apply = true;
-        break;
-      case '--dry-run':
-        opts.dryRun = true;
-        break;
-      case '--runtime':
-        if (args[i + 1]) {
-          opts.runtimes.push(args[i + 1]);
-          i++;
-        }
-        break;
-      case '--tier':
-        if (args[i + 1]) {
-          opts.tier = args[i + 1];
-          i++;
-        }
-        break;
-      case '--step':
-        if (args[i + 1]) {
-          opts.step = args[i + 1];
-          i++;
-        }
-        break;
-      case '--status':
-        if (args[i + 1]) {
-          opts.status = args[i + 1];
-          i++;
-        }
-        break;
-      case '--substep':
-        if (args[i + 1]) {
-          opts.substep = args[i + 1];
-          i++;
-        }
-        break;
-      case '--claim':
-        if (args[i + 1]) {
-          opts.claim = args[i + 1];
-          i++;
-        }
-        break;
-      case '--timeout':
-        if (args[i + 1]) {
-          opts.timeout = args[i + 1];
-          i++;
-        }
-        break;
-      case '--evidence':
-        if (args[i + 1]) {
-          opts.evidence = args[i + 1];
-          i++;
-        }
-        break;
-      case '--attest':
-        opts.attest = true;
-        break;
-      case '--peek':
-        opts.peek = true;
-        break;
-      case '--since':
-        if (args[i + 1]) {
-          opts.since = args[i + 1];
-          i++;
-        }
-        break;
-      case '--action':
-        if (args[i + 1]) {
-          opts.reflectAction = args[i + 1];
-          i++;
-        }
-        break;
-      case '--outcome':
-        if (args[i + 1]) {
-          opts.outcome = args[i + 1];
-          i++;
-        }
-        break;
-      case '--observation':
-        if (args[i + 1]) {
-          opts.observation = args[i + 1];
-          i++;
-        }
-        break;
-      case '--root-cause':
-        if (args[i + 1]) {
-          opts.rootCause = args[i + 1];
-          i++;
-        }
-        break;
-      case '--next':
-        if (args[i + 1]) {
-          opts.nextAction = args[i + 1];
-          i++;
-        }
-        break;
-      case '--lesson':
-        if (args[i + 1]) {
-          opts.lesson = args[i + 1];
-          i++;
-        }
-        break;
-      case '--category':
-        if (args[i + 1]) {
-          opts.category = args[i + 1];
-          i++;
-        }
-        break;
-      case '--tags':
-        if (args[i + 1]) {
-          opts.tags = args[i + 1];
-          i++;
-        }
-        break;
-      case '--scope':
-        if (args[i + 1]) {
-          opts.scope = args[i + 1];
-          i++;
-        }
-        break;
-      case '--goal':
-        if (args[i + 1]) {
-          opts.outcomeGoal = args[i + 1];
-          i++;
-        }
-        break;
-      case '--verify':
-        if (args[i + 1]) {
-          opts.outcomeVerify = args[i + 1];
-          i++;
-        }
-        break;
-      case '--budget':
-        if (args[i + 1]) {
-          opts.budget = args[i + 1];
-          i++;
-        }
-        break;
-      case '--reason':
-        if (args[i + 1]) {
-          opts.reason = args[i + 1];
-          i++;
-        }
-        break;
-      case '--from':
-        if (args[i + 1]) {
-          opts.importFrom = args[i + 1];
-          i++;
-        }
-        break;
-      case '--project':
-        if (args[i + 1]) {
-          opts.project = path.resolve(args[i + 1]);
-          i++;
-        }
-        break;
-      case '-g':
-      case '--global':
-        opts.global = true;
-        break;
-      case '-l':
-      case '--local':
-        opts.local = true;
-        break;
-      case '--all':
-        opts.all = true;
-        break;
-      case '--minimal':
-        opts.profile = 'core';
-        break;
-      case '--profile':
-        if (args[i + 1]) {
-          opts.profile = args[i + 1];
-          i++;
-        }
-        break;
-      case '-h':
-      case '--help':
-        opts.help = true;
-        break;
-      case '-u':
-      case '--uninstall':
-        opts.uninstall = true;
-        break;
-      default:
-        if (arg.startsWith('--project=')) {
-          opts.project = path.resolve(arg.slice('--project='.length));
-        } else if (arg.startsWith('--name=')) {
-          opts.extensionName = arg.slice('--name='.length);
-        } else if (arg.startsWith('--output=')) {
-          opts.extensionOutput = path.resolve(arg.slice('--output='.length));
-        } else if (arg.startsWith('--skill=')) {
-          opts.extensionSkill = arg.slice('--skill='.length);
-        } else if (arg.startsWith('--agent=')) {
-          opts.extensionAgent = arg.slice('--agent='.length);
-        } else if (arg.startsWith('--workflow=')) {
-          opts.extensionWorkflow = arg.slice('--workflow='.length);
-        } else if (arg.startsWith('--runtime=')) {
-          opts.runtimes.push(arg.slice('--runtime='.length));
-        } else if (arg.startsWith('--tier=')) {
-          opts.tier = arg.slice('--tier='.length);
-        } else if (arg.startsWith('--step=')) {
-          opts.step = arg.slice('--step='.length);
-        } else if (arg.startsWith('--status=')) {
-          opts.status = arg.slice('--status='.length);
-        } else if (arg.startsWith('--substep=')) {
-          opts.substep = arg.slice('--substep='.length);
-        } else if (arg.startsWith('--claim=')) {
-          opts.claim = arg.slice('--claim='.length);
-        } else if (arg.startsWith('--timeout=')) {
-          opts.timeout = arg.slice('--timeout='.length);
-        } else if (arg.startsWith('--evidence=')) {
-          opts.evidence = arg.slice('--evidence='.length);
-        } else if (arg.startsWith('--since=')) {
-          opts.since = arg.slice('--since='.length);
-        } else if (arg.startsWith('--action=')) {
-          opts.reflectAction = arg.slice('--action='.length);
-        } else if (arg.startsWith('--outcome=')) {
-          opts.outcome = arg.slice('--outcome='.length);
-        } else if (arg.startsWith('--observation=')) {
-          opts.observation = arg.slice('--observation='.length);
-        } else if (arg.startsWith('--root-cause=')) {
-          opts.rootCause = arg.slice('--root-cause='.length);
-        } else if (arg.startsWith('--next=')) {
-          opts.nextAction = arg.slice('--next='.length);
-        } else if (arg.startsWith('--lesson=')) {
-          opts.lesson = arg.slice('--lesson='.length);
-        } else if (arg.startsWith('--category=')) {
-          opts.category = arg.slice('--category='.length);
-        } else if (arg.startsWith('--tags=')) {
-          opts.tags = arg.slice('--tags='.length);
-        } else if (arg.startsWith('--scope=')) {
-          opts.scope = arg.slice('--scope='.length);
-        } else if (arg.startsWith('--goal=')) {
-          opts.outcomeGoal = arg.slice('--goal='.length);
-        } else if (arg.startsWith('--verify=')) {
-          opts.outcomeVerify = arg.slice('--verify='.length);
-        } else if (arg.startsWith('--budget=')) {
-          opts.budget = arg.slice('--budget='.length);
-        } else if (arg.startsWith('--reason=')) {
-          opts.reason = arg.slice('--reason='.length);
-        } else if (arg.startsWith('--from=')) {
-          opts.importFrom = arg.slice('--from='.length);
-        } else if (arg.startsWith('--profile=')) {
-          opts.profile = arg.slice('--profile='.length);
-        } else if (arg.startsWith('--') && RUNTIMES[arg.slice(2)]) {
-          opts.runtimes.push(arg.slice(2));
-        }
-        break;
+  for (let i = 0; i < args.length; i++) {
+    const arg = args[i];
+    if (COMMANDS.has(arg)) {
+      opts.command = arg;
+      continue;
     }
+    if (consumePositional(opts, arg)) continue;
+    i = consumeFlag(opts, args, i);
   }
 
   return opts;

package/lib/installer-core.js

@@ -6,6 +6,7 @@ const { resolveRuntime } = require('./installer-runtimes');
 const { selectedSkillNames, normalizeProfiles } = require('./install-profiles');
 const identity = require('./package-identity');
 const frontmatter = require('./frontmatter');
+const { log, success, error } = require('./cli-log');
 
 const VERSION = identity.PACKAGE_VERSION;
 
@@ -21,18 +22,6 @@ const VERSION = identity.PACKAGE_VERSION;
  * @property {number} agents Number of specialist agent files.
  */
 
-function log(msg) {
-  console.log(`  ${msg}`);
-}
-
-function success(msg) {
-  console.log(`  \x1b[32m+\x1b[0m ${msg}`);
-}
-
-function error(msg) {
-  console.error(`  \x1b[31mx\x1b[0m ${msg}`);
-}
-
 function installSkillFile(srcFile, skillsDest, runtimeKey, targetName = null) {
   const baseName = targetName || path.basename(srcFile, '.md');
   if (runtimeKey === 'codex') {

package/lib/planning-systems.js

@@ -11,6 +11,7 @@ const path = require('path');
 const crypto = require('crypto');
 
 const state = require('./state');
+const { exists } = require('./sync-fs');
 
 const MAX_FILE_BYTES = 80 * 1024;
 const MAX_SYSTEM_FILES = 80;
@@ -95,10 +96,6 @@ function rel(projectRoot, absPath) {
   return path.relative(projectRoot, absPath).split(path.sep).join('/');
 }
 
-function exists(projectRoot, relPath) {
-  return fs.existsSync(path.join(projectRoot, relPath));
-}
-
 function ensureDir(filePath) {
   fs.mkdirSync(path.dirname(filePath), { recursive: true });
 }

package/lib/recipe-coverage-sync.js

@@ -9,6 +9,7 @@ const fs = require('fs');
 const path = require('path');
 
 const recipes = require('./recipes');
+const { read, write } = require('./sync-fs');
 
 const LOG_PATH = '.godpowers/surface/RECIPE-COVERAGE-SYNC.md';
 
@@ -40,17 +41,6 @@ const REQUIRED_COVERAGE = [
   }
 ];
 
-function read(projectRoot, relPath) {
-  const file = path.join(projectRoot, relPath);
-  if (!fs.existsSync(file)) return '';
-  return fs.readFileSync(file, 'utf8');
-}
-
-function write(projectRoot, relPath, content) {
-  const file = path.join(projectRoot, relPath);
-  fs.mkdirSync(path.dirname(file), { recursive: true });
-  fs.writeFileSync(file, content);
-}
 
 function addCheck(checks, id, status, relPath, message, opts = {}) {
   checks.push({

package/lib/release-surface-sync.js

@@ -9,6 +9,8 @@
 const fs = require('fs');
 const path = require('path');
 
+const { read, write, readJson } = require('./sync-fs');
+
 const LOG_PATH = '.godpowers/surface/RELEASE-SURFACE-SYNC.md';
 
 const REQUIRED_PACKAGE_GUARDS = [
@@ -37,26 +39,6 @@ const REQUIRED_RELEASE_TESTS = [
   'scripts/test-install-smoke.js'
 ];
 
-function read(projectRoot, relPath) {
-  const file = path.join(projectRoot, relPath);
-  if (!fs.existsSync(file)) return '';
-  return fs.readFileSync(file, 'utf8');
-}
-
-function write(projectRoot, relPath, content) {
-  const file = path.join(projectRoot, relPath);
-  fs.mkdirSync(path.dirname(file), { recursive: true });
-  fs.writeFileSync(file, content);
-}
-
-function readJson(projectRoot, relPath) {
-  try {
-    return JSON.parse(read(projectRoot, relPath));
-  } catch (err) {
-    return null;
-  }
-}
-
 function releaseGateText(projectRoot, pkg) {
   return [
     JSON.stringify((pkg && pkg.scripts) || {}),

package/lib/repo-doc-sync.js

@@ -10,25 +10,10 @@ const path = require('path');
 const crypto = require('crypto');
 
 const pillars = require('./pillars');
+const { read, write, exists } = require('./sync-fs');
 
 const LOG_PATH = '.godpowers/docs/REPO-DOC-SYNC.md';
 
-function read(projectRoot, relPath) {
-  const file = path.join(projectRoot, relPath);
-  if (!fs.existsSync(file)) return '';
-  return fs.readFileSync(file, 'utf8');
-}
-
-function write(projectRoot, relPath, text) {
-  const file = path.join(projectRoot, relPath);
-  fs.mkdirSync(path.dirname(file), { recursive: true });
-  fs.writeFileSync(file, text);
-}
-
-function exists(projectRoot, relPath) {
-  return fs.existsSync(path.join(projectRoot, relPath));
-}
-
 function countFiles(projectRoot, dir, pattern) {
   const full = path.join(projectRoot, dir);
   if (!fs.existsSync(full)) return 0;

package/lib/repo-surface-sync.js

@@ -11,6 +11,7 @@ const fs = require('fs');
 const path = require('path');
 
 const { parseSimpleYaml } = require('./intent');
+const { read, write, exists, readJson } = require('./sync-fs');
 const extensions = require('./extensions');
 const repoDocSync = require('./repo-doc-sync');
 const routeQualitySync = require('./route-quality-sync');
@@ -55,22 +56,6 @@ function rel(projectRoot, absPath) {
   return path.relative(projectRoot, absPath).split(path.sep).join('/');
 }
 
-function exists(projectRoot, relPath) {
-  return fs.existsSync(path.join(projectRoot, relPath));
-}
-
-function read(projectRoot, relPath) {
-  const file = path.join(projectRoot, relPath);
-  if (!fs.existsSync(file)) return '';
-  return fs.readFileSync(file, 'utf8');
-}
-
-function write(projectRoot, relPath, content) {
-  const file = path.join(projectRoot, relPath);
-  fs.mkdirSync(path.dirname(file), { recursive: true });
-  fs.writeFileSync(file, content);
-}
-
 function listFiles(projectRoot, relDir, pattern) {
   const dir = path.join(projectRoot, relDir);
   if (!fs.existsSync(dir)) return [];
@@ -80,14 +65,6 @@ function listFiles(projectRoot, relDir, pattern) {
     .map((name) => `${relDir}/${name}`.replace(/\\/g, '/'));
 }
 
-function readJson(projectRoot, relPath) {
-  try {
-    return JSON.parse(read(projectRoot, relPath));
-  } catch (err) {
-    return null;
-  }
-}
-
 function releaseGateText(projectRoot, pkg) {
   return [
     JSON.stringify((pkg && pkg.scripts) || {}),

package/lib/requirements.js

@@ -32,6 +32,7 @@ const path = require('path');
 const linkage = require('./linkage');
 const state = require('./state');
 const atomic = require('./atomic-write');
+const textUtil = require('./text-util');
 
 const PRD_PATH = '.godpowers/prd/PRD.md';
 const ROADMAP_PATH = '.godpowers/roadmap/ROADMAP.md';
@@ -58,11 +59,7 @@ function pad2(n) {
 }
 
 function slugify(text) {
-  return String(text)
-    .toLowerCase()
-    .replace(/[^a-z0-9]+/g, '-')
-    .replace(/^-+|-+$/g, '')
-    .slice(0, 40) || 'increment';
+  return textUtil.slugify(text, 'increment');
 }
 
 // ============================================================================

package/lib/route-quality-sync.js

@@ -10,6 +10,7 @@ const fs = require('fs');
 const path = require('path');
 
 const { parseSimpleYaml } = require('./intent');
+const { read, write } = require('./sync-fs');
 
 const LOG_PATH = '.godpowers/surface/ROUTE-QUALITY-SYNC.md';
 const CONTEXTUAL_NEXT_VALUES = new Set([
@@ -100,18 +101,6 @@ const TIER_GATE_COMMANDS = new Set([
   '/god-harden'
 ]);
 
-function read(projectRoot, relPath) {
-  const file = path.join(projectRoot, relPath);
-  if (!fs.existsSync(file)) return '';
-  return fs.readFileSync(file, 'utf8');
-}
-
-function write(projectRoot, relPath, content) {
-  const file = path.join(projectRoot, relPath);
-  fs.mkdirSync(path.dirname(file), { recursive: true });
-  fs.writeFileSync(file, content);
-}
-
 function listFiles(projectRoot, relDir, pattern) {
   const dir = path.join(projectRoot, relDir);
   if (!fs.existsSync(dir)) return [];