@solidactions/cli 1.11.0 → 1.12.0

package/dist/commands/skill-push.js

@@ -0,0 +1,174 @@
+"use strict";
+/**
+ * solidactions skill push <dir>
+ *
+ * Pushes a local skill folder into the crews library via the crews MCP server.
+ * Idempotent upsert: create, or update on name collision.
+ */
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.parseSkillFile = parseSkillFile;
+exports.readTopLevelReferences = readTopLevelReferences;
+exports.skillPushWithConfig = skillPushWithConfig;
+exports.skillPush = skillPush;
+const fs_1 = __importDefault(require("fs"));
+const path_1 = __importDefault(require("path"));
+const chalk_1 = __importDefault(require("chalk"));
+const js_yaml_1 = __importDefault(require("js-yaml"));
+const api_1 = require("../utils/api");
+const mcp_1 = require("../utils/mcp");
+/**
+ * Parse a SKILL.md file that begins with a YAML frontmatter block.
+ *
+ * Returns { name, description, properties, body } where:
+ *   - name and description come from the frontmatter
+ *   - properties is the rest of the frontmatter (excluding name and description)
+ *   - body is everything after the closing --- delimiter
+ *
+ * Throws with a descriptive message if the file is malformed or missing
+ * required fields.
+ */
+function parseSkillFile(content) {
+    if (!content.startsWith('---')) {
+        throw new Error('SKILL.md must begin with a YAML frontmatter block (---).');
+    }
+    // Find the closing --- delimiter (must be on its own line after the opening ---)
+    const afterOpen = content.slice(3);
+    const closeIdx = afterOpen.indexOf('\n---');
+    if (closeIdx === -1) {
+        throw new Error('SKILL.md frontmatter is not closed (missing closing ---).');
+    }
+    const yamlBlock = afterOpen.slice(0, closeIdx);
+    // Body is everything after the closing ---\n
+    const body = afterOpen.slice(closeIdx + 4).replace(/^\n/, '');
+    let fm;
+    try {
+        fm = js_yaml_1.default.load(yamlBlock);
+    }
+    catch (e) {
+        throw new Error(`Failed to parse SKILL.md frontmatter YAML: ${e.message}`);
+    }
+    if (!fm || typeof fm !== 'object') {
+        throw new Error('SKILL.md frontmatter is empty or not a YAML object.');
+    }
+    if (!fm.name || typeof fm.name !== 'string') {
+        throw new Error('SKILL.md frontmatter must contain a "name" field (string).');
+    }
+    if (!fm.description || typeof fm.description !== 'string') {
+        throw new Error('SKILL.md frontmatter must contain a "description" field (string).');
+    }
+    const { name, description, ...rest } = fm;
+    // Remove 'type' if present — the server sets it
+    const { type: _type, ...properties } = rest;
+    return { name, description, properties, body };
+}
+/**
+ * Read top-level (non-recursive) files in dir, excluding SKILL.md.
+ * Returns a map of filename → utf8 contents.
+ */
+function readTopLevelReferences(dir) {
+    const entries = fs_1.default.readdirSync(dir, { withFileTypes: true });
+    const references = {};
+    for (const entry of entries) {
+        if (!entry.isFile())
+            continue;
+        if (entry.name === 'SKILL.md')
+            continue;
+        const filePath = path_1.default.join(dir, entry.name);
+        references[entry.name] = fs_1.default.readFileSync(filePath, 'utf8');
+    }
+    return references;
+}
+/**
+ * Core implementation — accepts an injected config so tests can point at a
+ * stub server without touching the filesystem config.
+ */
+async function skillPushWithConfig(dir, options, config) {
+    const absDir = path_1.default.resolve(dir);
+    // 1. Check directory exists
+    if (!fs_1.default.existsSync(absDir) || !fs_1.default.statSync(absDir).isDirectory()) {
+        process.stderr.write(chalk_1.default.red(`error: "${dir}" is not a directory.\n`));
+        process.exit(1);
+    }
+    // 2. Read SKILL.md
+    const skillMdPath = path_1.default.join(absDir, 'SKILL.md');
+    if (!fs_1.default.existsSync(skillMdPath)) {
+        process.stderr.write(chalk_1.default.red(`error: "${skillMdPath}" not found. The skill directory must contain a SKILL.md file.\n`));
+        process.exit(1);
+    }
+    const skillMdContent = fs_1.default.readFileSync(skillMdPath, 'utf8');
+    // 3. Parse frontmatter + body
+    let parsed;
+    try {
+        parsed = parseSkillFile(skillMdContent);
+    }
+    catch (e) {
+        process.stderr.write(chalk_1.default.red(`error: ${e.message}\n`));
+        process.exit(1);
+    }
+    const { name, description, properties, body } = parsed;
+    // 4. Read sibling reference files
+    const references = readTopLevelReferences(absDir);
+    // 5. Compose an idempotent upsert: try create; on name_collision, edit.
+    const isRole = !!options.role;
+    const tool = isRole ? 'roles' : 'skills';
+    const createArgs = isRole
+        ? { action: 'create_skill', role: options.role, name, description, body, properties, references }
+        : { action: 'create', name, description, body, properties, references };
+    let result;
+    try {
+        result = await (0, mcp_1.callCrewsTool)(config, tool, createArgs);
+    }
+    catch (e) {
+        process.stderr.write(chalk_1.default.red(`error: MCP request failed — ${e.message}\n`));
+        process.exit(1);
+    }
+    let didUpdate = false;
+    // 6. On name collision, switch to the edit path (the upsert). properties → properties_patch.
+    if (!result.ok && result.data?.code === 'name_collision') {
+        const editArgs = isRole
+            ? { action: 'edit_skill', role: options.role, name, description, body, properties_patch: properties, references }
+            : { action: 'edit', identifier: name, description, body, properties_patch: properties, references };
+        try {
+            result = await (0, mcp_1.callCrewsTool)(config, tool, editArgs);
+        }
+        catch (e) {
+            process.stderr.write(chalk_1.default.red(`error: MCP request failed — ${e.message}\n`));
+            process.exit(1);
+        }
+        didUpdate = true;
+    }
+    // 7. Handle result.
+    if (!result.ok) {
+        const errData = result.data;
+        const errCode = errData?.code ?? 'unknown_error';
+        const errMsg = errData?.message ?? 'MCP returned an error with no message';
+        process.stderr.write(chalk_1.default.red(`${errCode}: ${errMsg}\n`));
+        process.exit(1);
+    }
+    if (options.json) {
+        console.log(JSON.stringify(result.data));
+        process.exit(0);
+    }
+    const data = result.data;
+    if (didUpdate) {
+        // edit shape: {version_id, body_blob_sha}
+        console.log(chalk_1.default.green(`updated skill '${name}' (version ${data.version_id ?? '?'})`));
+    }
+    else {
+        // create shape: {skill_doc_id, folder_id, reference_doc_ids}
+        const skillDocId = data.skill_doc_id ?? data.doc_id ?? data.id ?? '?';
+        const refCount = Object.keys(data.reference_doc_ids ?? {}).length;
+        console.log(chalk_1.default.green(`created skill '${name}' (doc ${skillDocId}, ${refCount} refs)`));
+    }
+    process.exit(0);
+}
+/**
+ * Entry point called from index.ts.
+ */
+async function skillPush(dir, options) {
+    const config = await (0, api_1.requireConfigWithWorkspace)();
+    await skillPushWithConfig(dir, options, config);
+}

package/dist/index.js

@@ -32,6 +32,7 @@ const oauth_actions_search_1 = require("./commands/oauth-actions-search");
 const oauth_actions_list_1 = require("./commands/oauth-actions-list");
 const oauth_actions_show_1 = require("./commands/oauth-actions-show");
 const workspaces_1 = require("./commands/workspaces");
+const skill_push_1 = require("./commands/skill-push");
 const config_1 = require("./utils/config");
 // eslint-disable-next-line @typescript-eslint/no-var-requires
 const pkg = require('../package.json');
@@ -391,6 +392,19 @@ ai
     .option('--all', 'Install all available examples')
     .option('--overwrite', 'Overwrite existing examples without warning')
     .action((names, options) => { (0, ai_examples_1.aiExamples)(names, options); });
+// =============================================================================
+// skill <subcommand>  (SOP surface — flat top-level noun per cli#34)
+// =============================================================================
+const skill = program.command('skill').description('Manage agent skills (crews SOP surface)');
+skill
+    .command('push')
+    .description('Push a local skill folder into the library (create, or update if it already exists)')
+    .argument('<dir>', 'Path to the skill directory (must contain SKILL.md)')
+    .option('--role <name>', 'Scope the skill to a role instead of the shared library')
+    .option('--json', 'Output result as JSON')
+    .action(async (dir, options) => {
+    await (0, skill_push_1.skillPush)(dir, options);
+});
 program.parseAsync().catch((err) => {
     console.error(chalk_1.default.red(err.message ?? String(err)));
     process.exit(1);

package/dist/utils/mcp.js

@@ -0,0 +1,117 @@
+"use strict";
+/**
+ * Minimal MCP client for the SolidActions crews MCP server.
+ *
+ * Sends a single stateless JSON-RPC tools/call POST.  No initialize handshake
+ * required — the streamable-HTTP transport at /mcp/crews accepts single
+ * stateless POSTs (confirmed live).
+ */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.callCrewsTool = callCrewsTool;
+const http = __importStar(require("http"));
+const https = __importStar(require("https"));
+const url_1 = require("url");
+const api_1 = require("./api");
+/**
+ * Call a single MCP tool on /mcp/crews.
+ *
+ * Returns { ok: true, data: <success shape> } or { ok: false, data: { code, message } }.
+ */
+async function callCrewsTool(config, toolName, args) {
+    const baseHeaders = (0, api_1.getApiHeaders)(config, 'application/json');
+    // Override Accept to include text/event-stream for streamable-HTTP transport
+    const headers = {
+        ...baseHeaders,
+        'Accept': 'application/json, text/event-stream',
+    };
+    const body = JSON.stringify({
+        jsonrpc: '2.0',
+        id: 1,
+        method: 'tools/call',
+        params: {
+            name: toolName,
+            arguments: args,
+        },
+    });
+    const parsed = new url_1.URL(`${config.host}/mcp/crews`);
+    const isHttps = parsed.protocol === 'https:';
+    const transport = isHttps ? https : http;
+    const responseData = await new Promise((resolve, reject) => {
+        const options = {
+            hostname: parsed.hostname,
+            port: parsed.port || (isHttps ? 443 : 80),
+            path: parsed.pathname + (parsed.search || ''),
+            method: 'POST',
+            headers: {
+                ...headers,
+                'Content-Length': Buffer.byteLength(body),
+            },
+        };
+        const req = transport.request(options, (res) => {
+            let raw = '';
+            res.on('data', (chunk) => { raw += chunk; });
+            res.on('end', () => {
+                if (res.statusCode && res.statusCode >= 400) {
+                    reject(new Error(`MCP request failed with HTTP ${res.statusCode}: ${raw}`));
+                }
+                else {
+                    resolve(raw);
+                }
+            });
+        });
+        req.on('error', reject);
+        req.write(body);
+        req.end();
+    });
+    let parsed2;
+    try {
+        parsed2 = JSON.parse(responseData);
+    }
+    catch {
+        throw new Error(`MCP server returned non-JSON response: ${responseData}`);
+    }
+    const result = parsed2?.result;
+    const isError = result?.isError === true;
+    const textContent = result?.content?.[0]?.text ?? '{}';
+    let toolData;
+    try {
+        toolData = JSON.parse(textContent);
+    }
+    catch {
+        throw new Error(`MCP tool result content is not valid JSON: ${textContent}`);
+    }
+    return { ok: !isError, data: toolData };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@solidactions/cli",
-  "version": "1.11.0",
+  "version": "1.12.0",
   "description": "SolidActions CLI - Deploy and manage workflow automation",
   "main": "dist/index.js",
   "bin": {