@almyty/skills 1.0.5 → 1.0.7

package/dist/__tests__/client.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/__tests__/client.test.js

@@ -0,0 +1,145 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { parseRef } from '../client.js';
+import { generateMetaSkill } from '../meta-skill.js';
+import { loadConfig } from '../config.js';
+// ---------------------------------------------------------------------------
+// parseRef()
+// ---------------------------------------------------------------------------
+describe('parseRef', () => {
+    it('parses @org/gateway as a gateway ref', () => {
+        const result = parseRef('@org/gateway');
+        expect(result).toEqual({
+            type: 'gateway',
+            orgSlug: 'org',
+            gatewaySlug: 'gateway',
+            raw: '@org/gateway',
+        });
+    });
+    it('parses @org/gateway/skill as a skill ref', () => {
+        const result = parseRef('@org/gateway/skill');
+        expect(result).toEqual({
+            type: 'skill',
+            orgSlug: 'org',
+            gatewaySlug: 'gateway',
+            skillName: 'skill',
+            raw: '@org/gateway/skill',
+        });
+    });
+    it('parses a UUID string as a uuid ref', () => {
+        const uuid = 'a1b2c3d4-e5f6-7890-abcd-ef1234567890';
+        const result = parseRef(uuid);
+        expect(result).toEqual({
+            type: 'uuid',
+            uuid,
+            raw: uuid,
+        });
+    });
+    it('parses uppercase UUID correctly', () => {
+        const uuid = 'A1B2C3D4-E5F6-7890-ABCD-EF1234567890';
+        const result = parseRef(uuid);
+        expect(result.type).toBe('uuid');
+        expect(result.uuid).toBe(uuid);
+    });
+    it('returns search for a random string', () => {
+        const result = parseRef('some-random-query');
+        expect(result).toEqual({
+            type: 'search',
+            raw: 'some-random-query',
+        });
+    });
+    it('returns search for an empty string', () => {
+        const result = parseRef('');
+        expect(result).toEqual({
+            type: 'search',
+            raw: '',
+        });
+    });
+    it('returns search for @ with only one segment', () => {
+        const result = parseRef('@org');
+        expect(result).toEqual({
+            type: 'search',
+            raw: '@org',
+        });
+    });
+    it('returns search for @ with more than three segments', () => {
+        const result = parseRef('@org/gw/skill/extra');
+        expect(result).toEqual({
+            type: 'search',
+            raw: '@org/gw/skill/extra',
+        });
+    });
+});
+// ---------------------------------------------------------------------------
+// generateMetaSkill()
+// ---------------------------------------------------------------------------
+describe('generateMetaSkill', () => {
+    it('returns a SkillFile with name "skills" and fileName "almyty-skills"', () => {
+        const skill = generateMetaSkill();
+        expect(skill.name).toBe('skills');
+        expect(skill.fileName).toBe('almyty-skills');
+    });
+    it('content includes npx @almyty/skills commands', () => {
+        const { content } = generateMetaSkill();
+        expect(content).toContain('npx @almyty/skills');
+    });
+    it('content references daemon command', () => {
+        const { content } = generateMetaSkill();
+        expect(content).toContain('daemon');
+    });
+    it('content references search command', () => {
+        const { content } = generateMetaSkill();
+        expect(content).toContain('search');
+    });
+    it('content references install command', () => {
+        const { content } = generateMetaSkill();
+        expect(content).toContain('install');
+    });
+    it('content references run command', () => {
+        const { content } = generateMetaSkill();
+        expect(content).toContain('run');
+    });
+    it('content includes SKILL.md front-matter', () => {
+        const { content } = generateMetaSkill();
+        expect(content).toContain('---');
+        expect(content).toContain('name: almyty-skills');
+        expect(content).toContain('description:');
+    });
+});
+// ---------------------------------------------------------------------------
+// loadConfig()
+// ---------------------------------------------------------------------------
+describe('loadConfig', () => {
+    const originalEnv = process.env;
+    beforeEach(() => {
+        // Reset env between tests
+        process.env = { ...originalEnv };
+        delete process.env.APIFAI_SKILLS_DIR;
+    });
+    afterEach(() => {
+        process.env = originalEnv;
+        vi.restoreAllMocks();
+    });
+    it('returns empty config when no .almytyrc exists and no env var', () => {
+        // Use a temp directory that definitely has no .almytyrc
+        const config = loadConfig('/tmp/nonexistent-dir-' + Date.now());
+        expect(config).toEqual({});
+    });
+    it('uses APIFAI_SKILLS_DIR env var when set', () => {
+        process.env.APIFAI_SKILLS_DIR = '/custom/skills/dir';
+        const config = loadConfig();
+        expect(config).toEqual({ skillsDir: '/custom/skills/dir' });
+    });
+    it('env var takes precedence over .almytyrc files', () => {
+        process.env.APIFAI_SKILLS_DIR = '/env/override';
+        // Even if we pass a projectDir, env should take priority
+        const config = loadConfig('/some/project');
+        expect(config.skillsDir).toBe('/env/override');
+    });
+    it('returns empty config for a directory with no rc file and no home rc', () => {
+        // Use a deeply nested temp path that certainly has no .almytyrc anywhere
+        // and also won't match ~/.almytyrc (already handled by the missing file)
+        const fakePath = `/tmp/almyty-test-${Date.now()}/deeply/nested/project`;
+        const config = loadConfig(fakePath);
+        expect(config).toEqual({});
+    });
+});

package/dist/auth.d.ts

@@ -1,20 +1,6 @@
 /**
- * Credential resolver for @almyty/skills.
- *
- * Authentication itself lives in the dedicated @almyty/auth package
- * (`npx @almyty/auth login`). This module just READS the shared
- * credentials store and surfaces them to the rest of the skills CLI.
- *
- * Lookup order:
- *   1. ALMYTY_TOKEN environment variable      (CI / scripts)
- *   2. ~/.almyty/credentials.json              (interactive use, written
- *                                               by `npx @almyty/auth login`)
+ * Thin re-export from the shared @almyty/client credential resolver.
+ * Keeps `./auth.js` imports working across the skills-cli codebase.
  */
-/**
- * Resolve token: env var > stored credentials. Exits with a clear hint
- * if no credentials are available.
- */
-export declare function resolveAuth(): {
-    url: string;
-    token: string;
-};
+export { resolveCredentials, resolveCredentialsOrExit, resolveCredentialsOrExit as resolveAuth, loadCredentials, CREDENTIALS_FILE, } from '@almyty/client';
+export type { StoredCredentials } from '@almyty/client';

package/dist/auth.js

@@ -1,57 +1,5 @@
 /**
- * Credential resolver for @almyty/skills.
- *
- * Authentication itself lives in the dedicated @almyty/auth package
- * (`npx @almyty/auth login`). This module just READS the shared
- * credentials store and surfaces them to the rest of the skills CLI.
- *
- * Lookup order:
- *   1. ALMYTY_TOKEN environment variable      (CI / scripts)
- *   2. ~/.almyty/credentials.json              (interactive use, written
- *                                               by `npx @almyty/auth login`)
+ * Thin re-export from the shared @almyty/client credential resolver.
+ * Keeps `./auth.js` imports working across the skills-cli codebase.
  */
-import { readFileSync, existsSync } from 'fs';
-import { homedir } from 'os';
-import { join } from 'path';
-const CREDENTIALS_FILE = join(homedir(), '.almyty', 'credentials.json');
-function loadCredentials() {
-    try {
-        if (!existsSync(CREDENTIALS_FILE))
-            return null;
-        return JSON.parse(readFileSync(CREDENTIALS_FILE, 'utf-8'));
-    }
-    catch {
-        return null;
-    }
-}
-/**
- * Resolve token: env var > stored credentials. Exits with a clear hint
- * if no credentials are available.
- */
-export function resolveAuth() {
-    const envToken = process.env.ALMYTY_TOKEN;
-    // Read URL from: env > config file > default
-    let configUrl;
-    try {
-        const { readFileSync, existsSync } = require('fs');
-        const { join } = require('path');
-        const { homedir } = require('os');
-        const configPath = join(homedir(), '.almyty', 'config.json');
-        if (existsSync(configPath)) {
-            configUrl = JSON.parse(readFileSync(configPath, 'utf-8')).apiUrl;
-        }
-    }
-    catch { }
-    const envUrl = process.env.ALMYTY_URL || configUrl || 'https://api.almyty.com';
-    if (envToken) {
-        return { url: envUrl, token: envToken };
-    }
-    const creds = loadCredentials();
-    if (creds?.token) {
-        return { url: creds.url, token: creds.token };
-    }
-    console.error('Not authenticated. Run one of:');
-    console.error('  npx @almyty/auth login        # browser-based login');
-    console.error('  export ALMYTY_TOKEN=<token>   # for CI');
-    process.exit(1);
-}
+export { resolveCredentials, resolveCredentialsOrExit, resolveCredentialsOrExit as resolveAuth, loadCredentials, CREDENTIALS_FILE, } from '@almyty/client';

package/dist/client.js

@@ -4,14 +4,14 @@ export function parseRef(ref) {
     const uuidRegex = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
     if (uuidRegex.test(ref))
         return { type: 'uuid', uuid: ref, raw: ref };
-    if (ref.startsWith('@')) {
-        const parts = ref.slice(1).split('/');
-        if (parts.length === 3) {
-            return { type: 'skill', orgSlug: parts[0], gatewaySlug: parts[1], skillName: parts[2], raw: ref };
-        }
-        if (parts.length === 2) {
-            return { type: 'gateway', orgSlug: parts[0], gatewaySlug: parts[1], raw: ref };
-        }
+    // Accept both @org/gateway and org/gateway (@ is optional)
+    const normalized = ref.startsWith('@') ? ref.slice(1) : ref;
+    const parts = normalized.split('/');
+    if (parts.length === 3) {
+        return { type: 'skill', orgSlug: parts[0], gatewaySlug: parts[1], skillName: parts[2], raw: ref };
+    }
+    if (parts.length === 2) {
+        return { type: 'gateway', orgSlug: parts[0], gatewaySlug: parts[1], raw: ref };
     }
     return { type: 'search', raw: ref };
 }
@@ -70,7 +70,7 @@ export class AlmytyClient {
         return data?.data?.skills || [];
     }
     async fetchGateway(gatewayIdOrRef) {
-        if (gatewayIdOrRef.startsWith('@')) {
+        if (gatewayIdOrRef.includes('/')) {
             const parsed = parseRef(gatewayIdOrRef);
             if (parsed.orgSlug && parsed.gatewaySlug) {
                 return this.resolveGateway(parsed.orgSlug, parsed.gatewaySlug);
@@ -133,7 +133,7 @@ export class AlmytyClient {
         return this.post(`/gateways/${gatewayId}/skills/${resolvedId}/execute`, { parameters });
     }
     async resolveRef(ref) {
-        if (ref.startsWith('@')) {
+        if (ref.includes('/')) {
             const parsed = parseRef(ref);
             if (parsed.orgSlug && parsed.gatewaySlug) {
                 const gw = await this.resolveGateway(parsed.orgSlug, parsed.gatewaySlug);

package/dist/index.js

@@ -28,8 +28,8 @@ Authentication:
   are shared across every almyty CLI.
 
 References:
-  @org/gateway                   All skills from a gateway
-  @org/gateway/skill             A specific skill
+  org/gateway                   All skills from a gateway
+  org/gateway/skill             A specific skill
   skill-name                     Search by name
   <uuid>                         Direct ID reference
 
@@ -48,9 +48,9 @@ Options:
 
 Examples:
   npx @almyty/skills daemon
-  npx @almyty/skills install @myorg/petstore/get-pet
+  npx @almyty/skills install myorg/petstore/get-pet
   npx @almyty/skills search "weather"
-  npx @almyty/skills run @myorg/petstore/get-pet --petId 123
+  npx @almyty/skills run myorg/petstore/get-pet --petId 123
 `);
 }
 function parseArgs(argv) {
@@ -76,7 +76,7 @@ function parseArgs(argv) {
         else if (arg === '--version' || arg === '-v') {
             result.flags.version = true;
         }
-        else if (arg.startsWith('@')) {
+        else if (arg.includes('/')) {
             result.ref = arg;
         }
         else if (arg.startsWith('--')) {
@@ -302,7 +302,7 @@ async function main() {
                         const desc = r.toolDescription ? ` — ${r.toolDescription}` : '';
                         console.log(`  ${label}${desc}`);
                     }
-                    console.log(`\nBe more specific: npx @almyty/skills install @org/gateway/skill`);
+                    console.log(`\nBe more specific: npx @almyty/skills install org/gateway/skill`);
                     return;
                 }
             }
@@ -351,7 +351,7 @@ async function main() {
                 toolId = results[0].toolId;
             }
             else {
-                console.error('Error: run requires a skill reference (@org/gateway/skill or skill-name)');
+                console.error('Error: run requires a skill reference (org/gateway/skill or skill-name)');
                 process.exit(1);
             }
             const params = parseRunParams(args);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@almyty/skills",
-  "version": "1.0.5",
+  "version": "1.0.7",
   "description": "Install API skills into AI coding agents — Claude Code, Cursor, Windsurf, Copilot, Codex",
   "type": "module",
   "main": "dist/index.js",
@@ -11,7 +11,7 @@
     "dist"
   ],
   "scripts": {
-    "build": "tsc",
+    "build": "tsc && chmod +x dist/index.js",
     "dev": "tsx src/index.ts",
     "test": "vitest run",
     "test:watch": "vitest",
@@ -31,6 +31,9 @@
   ],
   "author": "almyty",
   "license": "BSL-1.1",
+  "dependencies": {
+    "@almyty/client": "^0.1.0"
+  },
   "devDependencies": {
     "@types/node": "^25.4.0",
     "tsx": "^4.7.0",