@operor/skills 0.1.0

package/test/config.test.ts

@@ -0,0 +1,162 @@
+import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
+import {
+  resolveEnvVars,
+  resolveEnvRecord,
+  loadSkillsConfig,
+  saveSkillsConfig,
+} from '../src/config.js';
+import type { SkillsConfig } from '../src/config.js';
+import { mkdtempSync, rmSync, readFileSync, writeFileSync } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+
+// ─── resolveEnvVars ──────────────────────────────────────────────────────────
+
+describe('resolveEnvVars', () => {
+  beforeEach(() => {
+    vi.stubEnv('TEST_KEY', 'hello');
+    vi.stubEnv('OTHER_KEY', 'world');
+  });
+
+  afterEach(() => {
+    vi.unstubAllEnvs();
+  });
+
+  it('should resolve a single env var', () => {
+    expect(resolveEnvVars('${TEST_KEY}')).toBe('hello');
+  });
+
+  it('should resolve multiple env vars', () => {
+    expect(resolveEnvVars('${TEST_KEY}-${OTHER_KEY}')).toBe('hello-world');
+  });
+
+  it('should return empty string for missing env var', () => {
+    expect(resolveEnvVars('${MISSING_VAR}')).toBe('');
+  });
+
+  it('should leave plain strings unchanged', () => {
+    expect(resolveEnvVars('no-vars-here')).toBe('no-vars-here');
+  });
+});
+
+describe('resolveEnvRecord', () => {
+  beforeEach(() => {
+    vi.stubEnv('API_KEY', 'sk_test_123');
+  });
+
+  afterEach(() => {
+    vi.unstubAllEnvs();
+  });
+
+  it('should resolve all values in a record', () => {
+    const result = resolveEnvRecord({
+      KEY: '${API_KEY}',
+      PLAIN: 'literal',
+    });
+    expect(result).toEqual({
+      KEY: 'sk_test_123',
+      PLAIN: 'literal',
+    });
+  });
+});
+
+// ─── loadSkillsConfig / saveSkillsConfig ─────────────────────────────────────
+
+describe('loadSkillsConfig', () => {
+  let tmpDir: string;
+
+  beforeEach(() => {
+    tmpDir = mkdtempSync(join(tmpdir(), 'skills-config-'));
+  });
+
+  afterEach(() => {
+    rmSync(tmpDir, { recursive: true, force: true });
+  });
+
+  it('should return empty config when mcp.json does not exist', () => {
+    const config = loadSkillsConfig(tmpDir);
+    expect(config.skills).toEqual([]);
+  });
+
+  it('should load skills from mcp.json', () => {
+    const data: SkillsConfig = {
+      skills: [
+        {
+          name: 'shopify',
+          transport: 'stdio',
+          command: 'npx',
+          args: ['-y', 'shopify-mcp'],
+          env: { SHOPIFY_TOKEN: '${SHOPIFY_TOKEN}' },
+          enabled: true,
+        },
+      ],
+    };
+    writeFileSync(join(tmpDir, 'mcp.json'), JSON.stringify(data));
+
+    const config = loadSkillsConfig(tmpDir);
+    expect(config.skills).toHaveLength(1);
+    expect(config.skills[0].name).toBe('shopify');
+    expect(config.skills[0].transport).toBe('stdio');
+  });
+
+  it('should throw on invalid JSON', () => {
+    writeFileSync(join(tmpDir, 'mcp.json'), 'not json{{{');
+    expect(() => loadSkillsConfig(tmpDir)).toThrow('Invalid JSON');
+  });
+
+  it('should return empty skills for non-object JSON', () => {
+    writeFileSync(join(tmpDir, 'mcp.json'), '"just a string"');
+    const config = loadSkillsConfig(tmpDir);
+    expect(config.skills).toEqual([]);
+  });
+
+  it('should return empty skills when skills key is missing', () => {
+    writeFileSync(join(tmpDir, 'mcp.json'), '{}');
+    const config = loadSkillsConfig(tmpDir);
+    expect(config.skills).toEqual([]);
+  });
+});
+
+describe('saveSkillsConfig', () => {
+  let tmpDir: string;
+
+  beforeEach(() => {
+    tmpDir = mkdtempSync(join(tmpdir(), 'skills-config-'));
+  });
+
+  afterEach(() => {
+    rmSync(tmpDir, { recursive: true, force: true });
+  });
+
+  it('should write mcp.json', () => {
+    const config: SkillsConfig = {
+      skills: [
+        { name: 'stripe', transport: 'stdio', command: 'npx', args: ['-y', '@stripe/mcp'] },
+      ],
+    };
+
+    saveSkillsConfig(config, tmpDir);
+
+    const raw = readFileSync(join(tmpDir, 'mcp.json'), 'utf-8');
+    const parsed = JSON.parse(raw);
+    expect(parsed.skills).toHaveLength(1);
+    expect(parsed.skills[0].name).toBe('stripe');
+  });
+
+  it('should be round-trippable with loadSkillsConfig', () => {
+    const config: SkillsConfig = {
+      skills: [
+        { name: 'a', transport: 'stdio', command: 'npx', args: ['-y', 'a-mcp'], enabled: true },
+        { name: 'b', transport: 'http', url: 'https://b.example.com/mcp', enabled: false },
+      ],
+    };
+
+    saveSkillsConfig(config, tmpDir);
+    const loaded = loadSkillsConfig(tmpDir);
+
+    expect(loaded.skills).toHaveLength(2);
+    expect(loaded.skills[0].name).toBe('a');
+    expect(loaded.skills[1].name).toBe('b');
+    expect(loaded.skills[1].url).toBe('https://b.example.com/mcp');
+  });
+});

package/tsconfig.json

@@ -0,0 +1,9 @@
+{
+  "extends": "../../tsconfig.json",
+  "compilerOptions": {
+    "outDir": "./dist",
+    "rootDir": "./src"
+  },
+  "include": ["src/**/*"],
+  "exclude": ["node_modules", "dist"]
+}

package/tsdown.config.ts

@@ -0,0 +1,10 @@
+import { defineConfig } from 'tsdown';
+
+export default defineConfig({
+  entry: ['src/index.ts'],
+  format: ['esm'],
+  dts: true,
+  clean: true,
+  sourcemap: true,
+  outExtensions: () => ({ js: '.js', dts: '.d.ts' }),
+});

package/vitest.config.ts

@@ -0,0 +1,3 @@
+import { defineConfig } from 'vitest/config';
+
+export default defineConfig({});