agile-context-engineering 0.5.0 → 0.5.1

package/hooks/ace-check-update.js

@@ -7,17 +7,24 @@ const path = require('path');
 const os = require('os');
 const { spawn } = require('child_process');
 
+// Use CLAUDE_PLUGIN_DATA for persistent cache (survives plugin updates)
+// Use CLAUDE_PLUGIN_ROOT to find the bundled VERSION file
+const pluginData = process.env.CLAUDE_PLUGIN_DATA;
+const pluginRoot = process.env.CLAUDE_PLUGIN_ROOT;
+
+// Fallback for legacy standalone installs
 const homeDir = os.homedir();
 const cwd = process.cwd();
-const cacheDir = path.join(homeDir, '.claude', 'cache');
+
+const cacheDir = pluginData || path.join(homeDir, '.claude', 'cache');
 const cacheFile = path.join(cacheDir, 'ace-update-check.json');
 
-// VERSION file locations (check project first, then global)
-// New plugin structure: shared/VERSION; legacy: agile-context-engineering/VERSION
-const projectVersionFile = path.join(cwd, '.claude', 'shared', 'VERSION');
-const globalVersionFile = path.join(homeDir, '.claude', 'shared', 'VERSION');
-const legacyProjectVersionFile = path.join(cwd, '.claude', 'agile-context-engineering', 'VERSION');
-const legacyGlobalVersionFile = path.join(homeDir, '.claude', 'agile-context-engineering', 'VERSION');
+// VERSION file: prefer plugin root, then legacy locations
+const versionFiles = [
+  pluginRoot ? path.join(pluginRoot, 'shared', 'VERSION') : null,
+  path.join(cwd, '.claude', 'shared', 'VERSION'),
+  path.join(homeDir, '.claude', 'shared', 'VERSION'),
+].filter(Boolean);
 
 // Ensure cache directory exists
 if (!fs.existsSync(cacheDir)) {
@@ -30,14 +37,8 @@ const child = spawn(process.execPath, ['-e', `
   const { execSync } = require('child_process');
 
   const cacheFile = ${JSON.stringify(cacheFile)};
-  const versionFiles = [
-    ${JSON.stringify(projectVersionFile)},
-    ${JSON.stringify(globalVersionFile)},
-    ${JSON.stringify(legacyProjectVersionFile)},
-    ${JSON.stringify(legacyGlobalVersionFile)},
-  ];
+  const versionFiles = ${JSON.stringify(versionFiles)};
 
-  // Check new paths first, then legacy
   let installed = '0.0.0';
   try {
     for (const vf of versionFiles) {

package/hooks/ace-statusline.js

@@ -5,6 +5,7 @@
 const fs = require('fs');
 const path = require('path');
 const os = require('os');
+const { execSync } = require('child_process');
 
 // Read JSON from stdin
 let input = '';
@@ -43,7 +44,7 @@ process.stdin.on('end', () => {
       }
     }
 
-    // Current task from todos
+    // Current task from todos (Claude Code internal directory — always ~/.claude/)
     let task = '';
     const homeDir = os.homedir();
     const todosDir = path.join(homeDir, '.claude', 'todos');
@@ -64,24 +65,41 @@ process.stdin.on('end', () => {
       } catch (e) {}
     }
 
-    // ACE update available?
+    // ACE update available? Check plugin data dir first, then legacy location
     let aceUpdate = '';
-    const cacheFile = path.join(homeDir, '.claude', 'cache', 'ace-update-check.json');
-    if (fs.existsSync(cacheFile)) {
-      try {
-        const cache = JSON.parse(fs.readFileSync(cacheFile, 'utf8'));
-        if (cache.update_available) {
-          aceUpdate = '\x1b[33m\u2B06 /ace:update\x1b[0m \u2502 ';
-        }
-      } catch (e) {}
+    const pluginData = process.env.CLAUDE_PLUGIN_DATA;
+    const cacheLocations = [
+      pluginData ? path.join(pluginData, 'ace-update-check.json') : null,
+      path.join(homeDir, '.claude', 'cache', 'ace-update-check.json'),
+    ].filter(Boolean);
+
+    for (const cacheFile of cacheLocations) {
+      if (fs.existsSync(cacheFile)) {
+        try {
+          const cache = JSON.parse(fs.readFileSync(cacheFile, 'utf8'));
+          if (cache.update_available) {
+            aceUpdate = '\x1b[33m\u2B06 /ace:update\x1b[0m \u2502 ';
+          }
+          break;
+        } catch (e) {}
+      }
     }
 
+    // Git branch
+    let branch = '';
+    try {
+      const raw = execSync('git rev-parse --abbrev-ref HEAD', { cwd: dir, encoding: 'utf8', timeout: 2000, stdio: ['pipe', 'pipe', 'pipe'], windowsHide: true }).trim();
+      if (raw && raw !== 'HEAD') {
+        branch = ` \x1b[36m\u2387 ${raw}\x1b[0m`;
+      }
+    } catch {}
+
     // Output
     const dirname = path.basename(dir);
     if (task) {
-      process.stdout.write(`${aceUpdate}\x1b[2m${model}\x1b[0m \u2502 \x1b[1m${task}\x1b[0m \u2502 \x1b[2m${dirname}\x1b[0m${ctx}`);
+      process.stdout.write(`${aceUpdate}\x1b[2m${model}\x1b[0m \u2502 \x1b[1m${task}\x1b[0m \u2502 \x1b[2m${dirname}\x1b[0m${branch}${ctx}`);
     } else {
-      process.stdout.write(`${aceUpdate}\x1b[2m${model}\x1b[0m \u2502 \x1b[2m${dirname}\x1b[0m${ctx}`);
+      process.stdout.write(`${aceUpdate}\x1b[2m${model}\x1b[0m \u2502 \x1b[2m${dirname}\x1b[0m${branch}${ctx}`);
     }
   } catch (e) {
     // Silent fail - don't break statusline on parse errors

package/hooks/hooks.json

@@ -0,0 +1,14 @@
+{
+  "hooks": {
+    "SessionStart": [
+      {
+        "hooks": [
+          {
+            "type": "command",
+            "command": "node \"${CLAUDE_PLUGIN_ROOT}/hooks/ace-check-update.js\""
+          }
+        ]
+      }
+    ]
+  }
+}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "agile-context-engineering",
-  "version": "0.5.0",
-  "description": "ACE - Agile Context Engineering: A spec-driven development system for Claude Code and Crush (formerly OpenCode) with Agile workflows",
+  "version": "0.5.1",
+  "description": "ACE - Agile Context Engineering: a spec-driven development system for Claude Code, Codex, and Crush with Agile workflows",
   "main": "index.js",
   "bin": {
     "agile-context-engineering": "bin/install.js",
@@ -21,6 +21,7 @@
   },
   "keywords": [
     "claude-code",
+    "codex",
     "crush",
     "opencode",
     "anthropic",

package/shared/lib/ace-core.js

@@ -270,6 +270,57 @@ function parseKeyValueArgs(args) {
   return result;
 }
 
+/**
+ * Parse raw skill init args into a structured object.
+ * Handles key=value params, flags (--name), and positional args.
+ * ALL skill script.js files MUST use this for argument parsing.
+ *
+ * Returns: { story: '...', text: '...', _flags: ['--agent-teams-off'], _positional: '...' }
+ */
+function parseSkillArgs(rawArgs) {
+  const args = rawArgs.filter(a => a !== '--raw');
+  const flags = args.filter(a => a.startsWith('--'));
+  const params = args.filter(a => !a.startsWith('--'));
+  const hasKeyValue = params.some(a => a.includes('='));
+
+  const result = { _flags: flags };
+
+  if (hasKeyValue) {
+    Object.assign(result, parseKeyValueArgs(params));
+  } else if (params.length > 0) {
+    result._positional = params.join(' ');
+  }
+
+  return result;
+}
+
+/**
+ * Shared CLI dispatch for all skill scripts.
+ * Parses process.argv, extracts --raw, parses key=value args, and routes to handlers.
+ * ALL skill script.js files MUST use this — no manual arg parsing.
+ *
+ * Handlers receive (cwd, raw, args, parsed):
+ *   cwd    — process.cwd()
+ *   raw    — boolean, --raw flag present
+ *   args   — raw arg array (after command name, --raw stripped)
+ *   parsed — parseSkillArgs result: { story: '...', _flags: [...], _positional: '...' }
+ */
+function runSkillScript(handlers) {
+  const cwd = process.cwd();
+  const allArgs = process.argv.slice(2);
+  const raw = allArgs.includes('--raw');
+  const cmd = allArgs[0];
+  const restArgs = allArgs.slice(1).filter(a => a !== '--raw');
+  const parsed = parseSkillArgs(allArgs.slice(1));
+
+  const handler = handlers[cmd];
+  if (!handler) {
+    error(`Unknown command: ${cmd}\nAvailable: ${Object.keys(handlers).join(', ')}`);
+    return;
+  }
+  handler(cwd, raw, restArgs, parsed);
+}
+
 /**
  * Run a shell command and return trimmed stdout. Returns null on failure.
  */
@@ -304,5 +355,7 @@ module.exports = {
   loadSettings,
   writeSettings,
   parseKeyValueArgs,
+  parseSkillArgs,
+  runSkillScript,
   execCommand,
 };