@laitszkin/apollo-toolkit 3.14.0 → 3.14.2

package/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 All notable changes to this repository are documented in this file.
 
+## [v3.14.2] - 2026-05-15
+
+### Fixed
+
+- Support flags before verb in architecture atlas dispatch, preventing silent default to `open` when `--spec` precedes the verb (e.g., `apltk architecture --spec docs/plans/xxx feature add ...`).
+
+## [v3.14.1] - 2026-05-15
+
+### Fixed
+
+- Replace non-existent `cli.main()` with `cli.dispatch()` in architecture handler, fixing `apltk architecture --help` crash.
+- Correct `repoRoot()` path resolution in `validate-openai-agent-config` tool (3 levels up from `dist/lib/tools/` instead of 2) so CI validation finds skill directories.
+- Add missing `npm run build` step to publish CI workflow so tests importing from `dist/` resolve correctly.
+- Replace removed Python validation scripts (`validate_skill_frontmatter.py`, `validate_openai_agent_config.py`) in CI workflow with their TypeScript CLI equivalents.
+
 ## [v3.14.0] - 2026-05-15
 
 ### Added

package/dist/lib/tools/architecture.js

@@ -5,23 +5,12 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.architectureHandler = architectureHandler;
 const node_path_1 = __importDefault(require("node:path"));
-const LEGACY_VERBS = new Set(['open', 'diff']);
 function architectureHandler(args, context) {
     const sourceRoot = context.sourceRoot || node_path_1.default.resolve(__dirname, '..', '..', '..');
     // Delegate to the existing atlas CLI (still in JS)
     const cliPath = node_path_1.default.join(sourceRoot, 'init-project-html', 'lib', 'atlas', 'cli.js');
     try {
         const cli = require(cliPath);
-        const verb = args[0];
-        if (!verb || verb.startsWith('-') || LEGACY_VERBS.has(verb)) {
-            // Sync execution for legacy verbs
-            const code = cli.main(args, {
-                stdout: context.stdout || process.stdout,
-                stderr: context.stderr || process.stderr,
-            });
-            return Promise.resolve(code);
-        }
-        // Async dispatch for declarative verbs
         return cli.dispatch(args, {
             stdout: context.stdout || process.stdout,
             stderr: context.stderr || process.stderr,

package/dist/lib/tools/validate-openai-agent-config.js

@@ -14,8 +14,14 @@ const INTERFACE_ALLOWED_KEYS = new Set([
     'icon_small', 'icon_large', 'brand_color',
 ]);
 const HEX_COLOR_PATTERN = /^#[0-9A-Fa-f]{6}$/;
-function repoRoot() {
-    return node_path_1.default.resolve(__dirname, '..', '..');
+function repoRoot(context) {
+    if (context?.sourceRoot)
+        return context.sourceRoot;
+    // __dirname is dist/lib/tools/; need to go up 3 levels to project root
+    const fromDirname = node_path_1.default.resolve(__dirname, '..', '..', '..');
+    if (node_fs_1.default.existsSync(node_path_1.default.join(fromDirname, 'package.json')))
+        return fromDirname;
+    return node_path_1.default.resolve(__dirname, '..', '..', '..');
 }
 function iterSkillDirs(root) {
     return node_fs_1.default.readdirSync(root)
@@ -162,7 +168,7 @@ function validateSkill(skillDir) {
     return errors;
 }
 function validateOpenaiAgentConfigHandler(args, context) {
-    const root = repoRoot();
+    const root = repoRoot(context);
     const skillDirs = iterSkillDirs(root);
     if (!skillDirs.length) {
         context.stdout?.write('No top-level skill directories found.\n');

package/init-project-html/lib/atlas/cli.js

@@ -968,6 +968,26 @@ function splitList(value) {
   return String(value).split(',').map((s) => s.trim()).filter(Boolean);
 }
 
+function findFirstPositional(args) {
+  const booleanFlags = new Set(['no-render', 'no-open', 'help', 'force']);
+  let i = 0;
+  while (i < args.length) {
+    const token = args[i];
+    if (token === '--') return i + 1 < args.length ? i + 1 : -1;
+    if (token.startsWith('--')) {
+      const eq = token.indexOf('=');
+      if (eq !== -1) { i++; continue; }
+      const name = token.slice(2);
+      if (booleanFlags.has(name)) { i++; continue; }
+      if (i + 1 < args.length && !args[i + 1].startsWith('-')) { i += 2; } else { i++; }
+      continue;
+    }
+    if (token === '-h') { i++; continue; }
+    return i;
+  }
+  return -1;
+}
+
 function parseFlags(args) {
   const positional = [];
   const flags = Object.create(null);
@@ -1920,14 +1940,20 @@ async function dispatch(argv, io = { stdout: process.stdout, stderr: process.std
   const args = [...argv];
   let verb = 'open';
   let explicitVerb = false;
-  if (args.length > 0 && !args[0].startsWith('-')) {
-    verb = args.shift();
+  const verbIdx = findFirstPositional(args);
+  if (verbIdx >= 0) {
+    verb = args[verbIdx];
     explicitVerb = true;
+    args.splice(verbIdx, 1);
   }
   let subverb = null;
   const multiVerbs = new Set(['feature', 'submodule', 'function', 'variable', 'dataflow', 'error', 'edge', 'meta', 'actor']);
-  if (multiVerbs.has(verb) && args.length > 0 && !args[0].startsWith('-')) {
-    subverb = args.shift();
+  if (multiVerbs.has(verb)) {
+    const subverbIdx = findFirstPositional(args);
+    if (subverbIdx >= 0) {
+      subverb = args[subverbIdx];
+      args.splice(subverbIdx, 1);
+    }
   }
   const { flags } = parseFlags(args);
 

package/lib/tools/architecture.ts

@@ -1,8 +1,6 @@
 import path from 'node:path';
 import type { ToolContext } from '../types';
 
-const LEGACY_VERBS = new Set(['open', 'diff']);
-
 export function architectureHandler(args: string[], context: ToolContext): Promise<number> {
   const sourceRoot = context.sourceRoot || path.resolve(__dirname, '..', '..', '..');
 
@@ -11,18 +9,6 @@ export function architectureHandler(args: string[], context: ToolContext): Promi
 
   try {
     const cli = require(cliPath);
-
-    const verb = args[0];
-    if (!verb || verb.startsWith('-') || LEGACY_VERBS.has(verb)) {
-      // Sync execution for legacy verbs
-      const code = cli.main(args, {
-        stdout: context.stdout || process.stdout,
-        stderr: context.stderr || process.stderr,
-      });
-      return Promise.resolve(code);
-    }
-
-    // Async dispatch for declarative verbs
     return cli.dispatch(args, {
       stdout: context.stdout || process.stdout,
       stderr: context.stderr || process.stderr,

package/lib/tools/validate-openai-agent-config.ts

@@ -11,8 +11,12 @@ const INTERFACE_ALLOWED_KEYS = new Set([
 ]);
 const HEX_COLOR_PATTERN = /^#[0-9A-Fa-f]{6}$/;
 
-function repoRoot(): string {
-  return path.resolve(__dirname, '..', '..');
+function repoRoot(context?: ToolContext): string {
+  if (context?.sourceRoot) return context.sourceRoot;
+  // __dirname is dist/lib/tools/; need to go up 3 levels to project root
+  const fromDirname = path.resolve(__dirname, '..', '..', '..');
+  if (fs.existsSync(path.join(fromDirname, 'package.json'))) return fromDirname;
+  return path.resolve(__dirname, '..', '..', '..');
 }
 
 function iterSkillDirs(root: string): string[] {
@@ -183,7 +187,7 @@ function validateSkill(skillDir: string): string[] {
 }
 
 export function validateOpenaiAgentConfigHandler(args: string[], context: ToolContext): Promise<number> {
-  const root = repoRoot();
+  const root = repoRoot(context);
   const skillDirs = iterSkillDirs(root);
 
   if (!skillDirs.length) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@laitszkin/apollo-toolkit",
-  "version": "3.14.0",
+  "version": "3.14.2",
   "description": "Apollo Toolkit npm installer for managed skill copying across Codex, OpenClaw, and Trae.",
   "license": "MIT",
   "author": "LaiTszKin",