npm - skyloom - Versions diffs - 1.13.5 → 1.13.7 - Mend

skyloom 1.13.5 → 1.13.7

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (195) hide show

package/.github/workflows/ci.yml +36 -36
package/README.md +220 -159
package/config/providers.yaml +39 -39
package/config/skills/api_integrator/SKILL.md +15 -15
package/config/skills/arch_designer/SKILL.md +13 -13
package/config/skills/ci_cd_manager/SKILL.md +14 -14
package/config/skills/code_analysis/SKILL.md +13 -13
package/config/skills/code_generator/SKILL.md +12 -12
package/config/skills/code_reviewer/SKILL.md +13 -13
package/config/skills/content_writer/SKILL.md +14 -14
package/config/skills/data_transformer/SKILL.md +15 -15
package/config/skills/document_analysis/SKILL.md +13 -13
package/config/skills/emotional_companion/SKILL.md +15 -15
package/config/skills/performance_checker/SKILL.md +14 -14
package/config/skills/security_auditor/SKILL.md +14 -14
package/config/skills/self_evolve/SKILL.md +13 -13
package/config/skills/sys_operator/SKILL.md +15 -15
package/config/skills/task_planner/SKILL.md +14 -14
package/config/skills/web_research/SKILL.md +14 -14
package/config/skills/workflow_designer/SKILL.md +13 -13
package/dist/agents/dew.js +52 -52
package/dist/agents/fair.js +84 -84
package/dist/agents/fog.js +30 -30
package/dist/agents/frost.js +32 -32
package/dist/agents/rain.js +32 -32
package/dist/agents/snow.js +68 -68
package/dist/cli/commands_md.d.ts +41 -0
package/dist/cli/commands_md.d.ts.map +1 -0
package/dist/cli/commands_md.js +140 -0
package/dist/cli/commands_md.js.map +1 -0
package/dist/cli/input_macros.d.ts +28 -0
package/dist/cli/input_macros.d.ts.map +1 -0
package/dist/cli/input_macros.js +120 -0
package/dist/cli/input_macros.js.map +1 -0
package/dist/cli/loom.d.ts +220 -0
package/dist/cli/loom.d.ts.map +1 -0
package/dist/cli/loom.js +1094 -0
package/dist/cli/loom.js.map +1 -0
package/dist/cli/loom_chat.d.ts +20 -0
package/dist/cli/loom_chat.d.ts.map +1 -0
package/dist/cli/loom_chat.js +685 -0
package/dist/cli/loom_chat.js.map +1 -0
package/dist/cli/main.js +310 -14
package/dist/cli/main.js.map +1 -1
package/dist/cli/tui.d.ts.map +1 -1
package/dist/cli/tui.js +7 -1
package/dist/cli/tui.js.map +1 -1
package/dist/core/agent/guard.d.ts +45 -0
package/dist/core/agent/guard.d.ts.map +1 -0
package/dist/core/agent/guard.js +113 -0
package/dist/core/agent/guard.js.map +1 -0
package/dist/core/agent.d.ts +17 -0
package/dist/core/agent.d.ts.map +1 -1
package/dist/core/agent.js +182 -93
package/dist/core/agent.js.map +1 -1
package/dist/core/factory.d.ts.map +1 -1
package/dist/core/factory.js +34 -2
package/dist/core/factory.js.map +1 -1
package/dist/core/file_checkpoint.d.ts +57 -0
package/dist/core/file_checkpoint.d.ts.map +1 -0
package/dist/core/file_checkpoint.js +162 -0
package/dist/core/file_checkpoint.js.map +1 -0
package/dist/core/hooks.d.ts +43 -0
package/dist/core/hooks.d.ts.map +1 -0
package/dist/core/hooks.js +110 -0
package/dist/core/hooks.js.map +1 -0
package/dist/core/llm.d.ts.map +1 -1
package/dist/core/llm.js +15 -9
package/dist/core/llm.js.map +1 -1
package/dist/core/longdoc.js +5 -5
package/dist/core/mcp.d.ts +16 -0
package/dist/core/mcp.d.ts.map +1 -1
package/dist/core/mcp.js +55 -0
package/dist/core/mcp.js.map +1 -1
package/dist/core/model_config.d.ts +40 -0
package/dist/core/model_config.d.ts.map +1 -0
package/dist/core/model_config.js +191 -0
package/dist/core/model_config.js.map +1 -0
package/dist/core/skill.d.ts +7 -0
package/dist/core/skill.d.ts.map +1 -1
package/dist/core/skill.js +47 -0
package/dist/core/skill.js.map +1 -1
package/dist/core/skymd.d.ts +39 -0
package/dist/core/skymd.d.ts.map +1 -0
package/dist/core/skymd.js +177 -0
package/dist/core/skymd.js.map +1 -0
package/dist/core/tool.d.ts +12 -0
package/dist/core/tool.d.ts.map +1 -1
package/dist/core/tool.js +30 -0
package/dist/core/tool.js.map +1 -1
package/dist/core/verify.d.ts +27 -0
package/dist/core/verify.d.ts.map +1 -0
package/dist/core/verify.js +62 -0
package/dist/core/verify.js.map +1 -0
package/dist/skills/loader.d.ts +22 -2
package/dist/skills/loader.d.ts.map +1 -1
package/dist/skills/loader.js +45 -15
package/dist/skills/loader.js.map +1 -1
package/dist/tools/builtin.d.ts.map +1 -1
package/dist/tools/builtin.js +13 -3
package/dist/tools/builtin.js.map +1 -1
package/dist/tools/model_tool.d.ts +11 -0
package/dist/tools/model_tool.d.ts.map +1 -0
package/dist/tools/model_tool.js +71 -0
package/dist/tools/model_tool.js.map +1 -0
package/dist/tools/todo.d.ts +30 -0
package/dist/tools/todo.d.ts.map +1 -0
package/dist/tools/todo.js +78 -0
package/dist/tools/todo.js.map +1 -0
package/docs/AESTHETIC_DESIGN.md +152 -144
package/docs/OPTIMIZATION_PLAN.md +178 -178
package/package.json +1 -1
package/scripts/install.js +48 -48
package/scripts/link.js +10 -10
package/setup.bat +79 -79
package/skill-test-ty2fOA/test.md +10 -10
package/src/agents/dew.ts +70 -70
package/src/agents/fair.ts +102 -102
package/src/agents/fog.ts +48 -48
package/src/agents/frost.ts +50 -50
package/src/agents/rain.ts +50 -50
package/src/agents/snow.ts +239 -239
package/src/cli/commands_md.ts +112 -0
package/src/cli/input_macros.ts +83 -0
package/src/cli/loom.ts +982 -0
package/src/cli/loom_chat.ts +598 -0
package/src/cli/main.ts +255 -9
package/src/cli/mode.ts +58 -58
package/src/cli/tui.ts +228 -222
package/src/core/agent/guard.ts +134 -0
package/src/core/agent/task.ts +100 -100
package/src/core/agent.ts +177 -95
package/src/core/arbitrate.ts +162 -162
package/src/core/catalog.ts +178 -178
package/src/core/checkpoint.ts +94 -94
package/src/core/estimate.ts +104 -104
package/src/core/evolve.ts +191 -191
package/src/core/factory.ts +31 -2
package/src/core/file_checkpoint.ts +136 -0
package/src/core/filter.ts +103 -103
package/src/core/graph.ts +156 -156
package/src/core/hooks.ts +126 -0
package/src/core/icons.ts +53 -53
package/src/core/index.ts +37 -37
package/src/core/learn.ts +146 -146
package/src/core/llm.ts +15 -9
package/src/core/longdoc.ts +155 -155
package/src/core/mcp.ts +48 -0
package/src/core/mcp_server.ts +176 -176
package/src/core/model_config.ts +157 -0
package/src/core/profile.ts +255 -255
package/src/core/router.ts +124 -124
package/src/core/sandbox.ts +142 -142
package/src/core/security.ts +243 -243
package/src/core/skill.ts +42 -0
package/src/core/skymd.ts +143 -0
package/src/core/theme.ts +65 -65
package/src/core/tool.ts +30 -0
package/src/core/tool_router.ts +193 -193
package/src/core/vector.ts +152 -152
package/src/core/verify.ts +71 -0
package/src/core/workspace.ts +150 -150
package/src/plugins/loader.ts +66 -66
package/src/skills/loader.ts +45 -16
package/src/sql.js.d.ts +29 -29
package/src/tools/builtin.ts +13 -3
package/src/tools/computer.ts +269 -269
package/src/tools/delegate.ts +49 -49
package/src/tools/model_tool.ts +74 -0
package/src/tools/todo.ts +76 -0
package/src/web/tts.ts +93 -93
package/tests/agent.test.ts +159 -159
package/tests/agent_helpers.test.ts +48 -48
package/tests/bus.test.ts +121 -121
package/tests/catalog.test.ts +86 -86
package/tests/checkpoint_commands.test.ts +124 -0
package/tests/claude_compat.test.ts +110 -0
package/tests/config.test.ts +41 -41
package/tests/guard.test.ts +75 -0
package/tests/icons.test.ts +45 -45
package/tests/loom.test.ts +248 -0
package/tests/memory.test.ts +170 -170
package/tests/model_config.test.ts +109 -0
package/tests/router.test.ts +86 -86
package/tests/schemas.test.ts +51 -51
package/tests/semantic.test.ts +83 -83
package/tests/setup.ts +10 -10
package/tests/skill.test.ts +172 -172
package/tests/skymd.test.ts +146 -0
package/tests/task.test.ts +60 -60
package/tests/todo_toolstats.test.ts +94 -0
package/tests/tool.test.ts +108 -108
package/tests/tool_router.test.ts +71 -71
package/tests/tui.test.ts +67 -67
package/vitest.config.ts +17 -17

package/tests/schemas.test.ts CHANGED Viewed

@@ -1,51 +1,51 @@
-/**
- * Tests for structured output schema validation.
- */
-import { describe, it, expect } from 'vitest';
-import { validateTaskPlan, TaskPlanSchema, parseSchema, SchemaValidationError } from '../src/core/schemas';
-describe('validateTaskPlan', () => {
-  it('parses valid JSON plan', () => {
-    const data = JSON.parse('{"goal": "build app", "steps": [{"id": "1", "description": "design", "agent": "fog"}]}');
-    const plan = validateTaskPlan(data);
-    expect(plan.goal).toBe('build app');
-    expect(plan.steps).toHaveLength(1);
-    expect(plan.steps[0].agent).toBe('fog');
-  });
-  it('passes through agent name', () => {
-    const data = JSON.parse('{"goal": "x", "steps": [{"id": "1", "description": "a", "agent": "unknown"}]}');
-    const plan = validateTaskPlan(data);
-    // Schema-level validation just passes through; agent fallback is in SnowAgent
-    expect(plan.steps[0].agent).toBe('unknown');
-  });
-  it('empty steps', () => {
-    const data = JSON.parse('{"goal": "x", "steps": []}');
-    const plan = validateTaskPlan(data);
-    expect(plan.steps).toHaveLength(0);
-  });
-  it('parses depends_on', () => {
-    const data = JSON.parse('{"goal": "x", "steps": [{"id": "1", "description": "a", "depends_on": ["0"]}]}');
-    const plan = validateTaskPlan(data);
-    expect(plan.steps[0].depends_on).toEqual(['0']);
-  });
-  it('default fields', () => {
-    const data = JSON.parse('{"goal": "x", "steps": [{"id": "1", "description": "a"}]}');
-    const plan = validateTaskPlan(data);
-    expect(plan.steps[0].agent).toBe('rain');
-    expect(plan.steps[0].depends_on).toEqual([]);
-  });
-});
-describe('SchemaValidationError', () => {
-  it('raises on empty', () => {
-    expect(() => parseSchema('', TaskPlanSchema)).toThrow(SchemaValidationError);
-  });
-  it('raises on garbage', () => {
-    expect(() => parseSchema('<html>garbage</html>', TaskPlanSchema)).toThrow(SchemaValidationError);
-  });
-});
+/**
+ * Tests for structured output schema validation.
+ */
+import { describe, it, expect } from 'vitest';
+import { validateTaskPlan, TaskPlanSchema, parseSchema, SchemaValidationError } from '../src/core/schemas';
+describe('validateTaskPlan', () => {
+  it('parses valid JSON plan', () => {
+    const data = JSON.parse('{"goal": "build app", "steps": [{"id": "1", "description": "design", "agent": "fog"}]}');
+    const plan = validateTaskPlan(data);
+    expect(plan.goal).toBe('build app');
+    expect(plan.steps).toHaveLength(1);
+    expect(plan.steps[0].agent).toBe('fog');
+  });
+  it('passes through agent name', () => {
+    const data = JSON.parse('{"goal": "x", "steps": [{"id": "1", "description": "a", "agent": "unknown"}]}');
+    const plan = validateTaskPlan(data);
+    // Schema-level validation just passes through; agent fallback is in SnowAgent
+    expect(plan.steps[0].agent).toBe('unknown');
+  });
+  it('empty steps', () => {
+    const data = JSON.parse('{"goal": "x", "steps": []}');
+    const plan = validateTaskPlan(data);
+    expect(plan.steps).toHaveLength(0);
+  });
+  it('parses depends_on', () => {
+    const data = JSON.parse('{"goal": "x", "steps": [{"id": "1", "description": "a", "depends_on": ["0"]}]}');
+    const plan = validateTaskPlan(data);
+    expect(plan.steps[0].depends_on).toEqual(['0']);
+  });
+  it('default fields', () => {
+    const data = JSON.parse('{"goal": "x", "steps": [{"id": "1", "description": "a"}]}');
+    const plan = validateTaskPlan(data);
+    expect(plan.steps[0].agent).toBe('rain');
+    expect(plan.steps[0].depends_on).toEqual([]);
+  });
+});
+describe('SchemaValidationError', () => {
+  it('raises on empty', () => {
+    expect(() => parseSchema('', TaskPlanSchema)).toThrow(SchemaValidationError);
+  });
+  it('raises on garbage', () => {
+    expect(() => parseSchema('<html>garbage</html>', TaskPlanSchema)).toThrow(SchemaValidationError);
+  });
+});

package/tests/semantic.test.ts CHANGED Viewed

@@ -1,83 +1,83 @@
-/**
- * Tests for lightweight semantic retrieval.
- */
-import { describe, it, expect } from 'vitest';
-import { SemanticScorer, getScorer } from '../src/core/semantic';
-describe('SemanticScorer', () => {
-  const scorer = new SemanticScorer();
-  it('identical strings score 1.0', () => {
-    expect(scorer.similarity('hello world', 'hello world')).toBe(1.0);
-  });
-  it('completely different strings score 0.0', () => {
-    expect(scorer.similarity('aaaaa', 'bbbbb')).toBe(0.0);
-  });
-  it('partial overlap scores between 0 and 1', () => {
-    const score = scorer.similarity('deploy to server', 'deployment script');
-    expect(score).toBeGreaterThan(0.0);
-    expect(score).toBeLessThan(1.0);
-  });
-  it('CJK similarity works', () => {
-    const score = scorer.similarity('部署', '部署命令');
-    expect(score).toBeGreaterThan(0.0);
-  });
-  it('empty strings return 0', () => {
-    expect(scorer.similarity('', 'hello')).toBe(0.0);
-    expect(scorer.similarity('hello', '')).toBe(0.0);
-    expect(scorer.similarity('', '')).toBe(0.0);
-  });
-  it('case insensitive', () => {
-    expect(scorer.similarity('Hello World', 'hello world')).toBe(1.0);
-  });
-  it('mixed language overlap', () => {
-    const score = scorer.similarity('pnpm install', 'pnpm 安装');
-    expect(score).toBeGreaterThan(0.0);
-  });
-  it('code identifiers match partially', () => {
-    const score = scorer.similarity('snake_case_var', 'snakeCaseVar');
-    expect(score).toBeGreaterThan(0.0);
-  });
-  it('rank returns ordered results', () => {
-    const candidates = [
-      { key: 'k1', value: 'deploy to production' },
-      { key: 'k2', value: 'install dependencies' },
-      { key: 'k3', value: 'rollback version' },
-    ];
-    const ranked = scorer.rank('deploy', candidates, 'value', 2);
-    expect(ranked.length).toBeLessThanOrEqual(2);
-    expect(ranked[0][1].key).toBe('k1');
-  });
-  it('rank filters below minScore', () => {
-    const candidates = [
-      { key: 'k1', value: 'completely unrelated text here' },
-    ];
-    const ranked = scorer.rank('zzzzz', candidates, 'value', 1, 0.5);
-    expect(ranked).toHaveLength(0);
-  });
-  it('rank uses key field boost', () => {
-    const candidates = [
-      { key: 'deploy_command', value: 'npm run build' },
-    ];
-    const ranked = scorer.rank('deploy', candidates, 'value', 1);
-    expect(ranked).toHaveLength(1);
-  });
-});
-describe('getScorer singleton', () => {
-  it('returns the same instance', () => {
-    const s1 = getScorer();
-    const s2 = getScorer();
-    expect(s1).toBe(s2);
-  });
-});
+/**
+ * Tests for lightweight semantic retrieval.
+ */
+import { describe, it, expect } from 'vitest';
+import { SemanticScorer, getScorer } from '../src/core/semantic';
+describe('SemanticScorer', () => {
+  const scorer = new SemanticScorer();
+  it('identical strings score 1.0', () => {
+    expect(scorer.similarity('hello world', 'hello world')).toBe(1.0);
+  });
+  it('completely different strings score 0.0', () => {
+    expect(scorer.similarity('aaaaa', 'bbbbb')).toBe(0.0);
+  });
+  it('partial overlap scores between 0 and 1', () => {
+    const score = scorer.similarity('deploy to server', 'deployment script');
+    expect(score).toBeGreaterThan(0.0);
+    expect(score).toBeLessThan(1.0);
+  });
+  it('CJK similarity works', () => {
+    const score = scorer.similarity('部署', '部署命令');
+    expect(score).toBeGreaterThan(0.0);
+  });
+  it('empty strings return 0', () => {
+    expect(scorer.similarity('', 'hello')).toBe(0.0);
+    expect(scorer.similarity('hello', '')).toBe(0.0);
+    expect(scorer.similarity('', '')).toBe(0.0);
+  });
+  it('case insensitive', () => {
+    expect(scorer.similarity('Hello World', 'hello world')).toBe(1.0);
+  });
+  it('mixed language overlap', () => {
+    const score = scorer.similarity('pnpm install', 'pnpm 安装');
+    expect(score).toBeGreaterThan(0.0);
+  });
+  it('code identifiers match partially', () => {
+    const score = scorer.similarity('snake_case_var', 'snakeCaseVar');
+    expect(score).toBeGreaterThan(0.0);
+  });
+  it('rank returns ordered results', () => {
+    const candidates = [
+      { key: 'k1', value: 'deploy to production' },
+      { key: 'k2', value: 'install dependencies' },
+      { key: 'k3', value: 'rollback version' },
+    ];
+    const ranked = scorer.rank('deploy', candidates, 'value', 2);
+    expect(ranked.length).toBeLessThanOrEqual(2);
+    expect(ranked[0][1].key).toBe('k1');
+  });
+  it('rank filters below minScore', () => {
+    const candidates = [
+      { key: 'k1', value: 'completely unrelated text here' },
+    ];
+    const ranked = scorer.rank('zzzzz', candidates, 'value', 1, 0.5);
+    expect(ranked).toHaveLength(0);
+  });
+  it('rank uses key field boost', () => {
+    const candidates = [
+      { key: 'deploy_command', value: 'npm run build' },
+    ];
+    const ranked = scorer.rank('deploy', candidates, 'value', 1);
+    expect(ranked).toHaveLength(1);
+  });
+});
+describe('getScorer singleton', () => {
+  it('returns the same instance', () => {
+    const s1 = getScorer();
+    const s2 = getScorer();
+    expect(s1).toBe(s2);
+  });
+});

package/tests/setup.ts CHANGED Viewed

@@ -1,10 +1,10 @@
-/**
- * Test setup — global fixtures and mocks for Skyloom tests.
- */
-import { vi, beforeEach } from 'vitest';
-import { MessageBus } from '../src/core/bus';
-// Clear tool state before each test
-beforeEach(() => {
-  // No-op — individual tests handle their own setup
-});
+/**
+ * Test setup — global fixtures and mocks for Skyloom tests.
+ */
+import { vi, beforeEach } from 'vitest';
+import { MessageBus } from '../src/core/bus';
+// Clear tool state before each test
+beforeEach(() => {
+  // No-op — individual tests handle their own setup
+});

package/tests/skill.test.ts CHANGED Viewed

@@ -1,172 +1,172 @@
-/**
- * Tests for skill system.
- */
-import { describe, it, expect } from 'vitest';
-import * as fs from 'fs';
-import * as path from 'path';
-describe('Skill', () => {
-  it('creates a skill with model override', async () => {
-    const { Skill } = await import('../src/core/skill');
-    const skill = new Skill({
-      name: 'test_skill',
-      description: 'A test skill',
-      model: 'claude-opus-4-7',
-    });
-    expect(skill.model).toBe('claude-opus-4-7');
-    expect(skill.temperature).toBeNull();
-    expect(skill.maxTokens).toBeNull();
-  });
-  it('creates a skill with temperature override', async () => {
-    const { Skill } = await import('../src/core/skill');
-    const skill = new Skill({ name: 'test_skill', description: 'A test skill', temperature: 0.3 });
-    expect(skill.temperature).toBe(0.3);
-  });
-  it('creates a skill with max_tokens override', async () => {
-    const { Skill } = await import('../src/core/skill');
-    const skill = new Skill({ name: 'test_skill', description: 'A test skill', maxTokens: 32000 });
-    expect(skill.maxTokens).toBe(32000);
-  });
-  it('default has no overrides', async () => {
-    const { Skill } = await import('../src/core/skill');
-    const skill = new Skill({ name: 'test', description: 'test' });
-    expect(skill.model).toBeNull();
-    expect(skill.temperature).toBeNull();
-    expect(skill.maxTokens).toBeNull();
-  });
-});
-describe('Skill.fromMarkdown', () => {
-  it('loads skill with YAML frontmatter', async () => {
-    const { Skill } = await import('../src/core/skill');
-    const tmpDir = fs.mkdtempSync('skill-test-');
-    const mdPath = path.join(tmpDir, 'test.md');
-    fs.writeFileSync(mdPath, `---
-name: test_skill
-description: A test
-model: claude-sonnet-4-6
-temperature: 0.5
-max_tokens: 8192
----
-## Test Skill
-This is a test skill.
-`, 'utf-8');
-    const skill = Skill.fromMarkdown(mdPath);
-    expect(skill).not.toBeNull();
-    if (skill) {
-      expect(skill.model).toBe('claude-sonnet-4-6');
-      expect(skill.temperature).toBe(0.5);
-      expect(skill.maxTokens).toBe(8192);
-      expect(skill.systemPrompt).toContain('This is a test skill');
-    }
-    fs.rmSync(tmpDir, { recursive: true, force: true });
-  });
-  it('supports string path', async () => {
-    const { Skill } = await import('../src/core/skill');
-    const tmpDir = fs.mkdtempSync('skill-test-');
-    const mdPath = path.join(tmpDir, 'x.md');
-    fs.writeFileSync(mdPath, '---\nname: x\ndescription: x\n---\n\nBody.', 'utf-8');
-    const s = Skill.fromMarkdown(mdPath);
-    expect(s).not.toBeNull();
-    if (s) {
-      expect(s.name).toBe('x');
-      expect(typeof s.sourcePath).toBe('string');
-    }
-    fs.rmSync(tmpDir, { recursive: true, force: true });
-  });
-  it('derives triggers from quoted descriptions', async () => {
-    const { Skill } = await import('../src/core/skill');
-    const tmpDir = fs.mkdtempSync('skill-test-');
-    const mdPath = path.join(tmpDir, 'pptx.md');
-    fs.writeFileSync(mdPath, `---
-name: pptx
-description: |-
-  Use this skill any time a .pptx file is involved.
-  Trigger whenever the user mentions "deck," "slides," or "presentation."
-  If a .pptx file needs to be opened, use it.
----
-Body.`, 'utf-8');
-    const s = Skill.fromMarkdown(mdPath);
-    expect(s).not.toBeNull();
-    if (s) {
-      const triggersLower = s.triggers.map(t => t.toLowerCase());
-      expect(triggersLower).toContain('deck');
-      expect(triggersLower).toContain('slides');
-      expect(triggersLower).toContain('presentation');
-      expect(triggersLower).toContain('.pptx');
-    }
-    fs.rmSync(tmpDir, { recursive: true, force: true });
-  });
-  it('small body loaded in full', async () => {
-    const { Skill } = await import('../src/core/skill');
-    const tmpDir = fs.mkdtempSync('skill-test-');
-    const mdPath = path.join(tmpDir, 'small.md');
-    fs.writeFileSync(mdPath, '---\nname: small\ndescription: x\n---\n\n# Small Skill\n\nThis fits inline easily.', 'utf-8');
-    const s = Skill.fromMarkdown(mdPath);
-    expect(s).not.toBeNull();
-    if (s) {
-      expect(s.bodyTruncated).toBe(false);
-      expect(s.systemPrompt).toContain('This fits inline easily');
-    }
-    fs.rmSync(tmpDir, { recursive: true, force: true });
-  });
-  it('large body truncated to head', async () => {
-    const { Skill } = await import('../src/core/skill');
-    const tmpDir = fs.mkdtempSync('skill-test-');
-    const mdPath = path.join(tmpDir, 'big.md');
-    let body = '# Big Skill\n\n## Quick Reference\nFirst section content.\n\n';
-    body += '## Detailed Guide\n';
-    body += 'X'.repeat(5000);
-    fs.writeFileSync(mdPath, `---\nname: big\ndescription: x\n---\n\n${body}`, 'utf-8');
-    const s = Skill.fromMarkdown(mdPath);
-    expect(s).not.toBeNull();
-    if (s) {
-      expect(s.bodyTruncated).toBe(true);
-      expect(s.systemPrompt).toContain('Big Skill');
-      expect(s.systemPrompt).toContain('Quick Reference');
-      expect(s.systemPrompt).not.toContain('Detailed Guide');
-    }
-    fs.rmSync(tmpDir, { recursive: true, force: true });
-  });
-});
-describe('SkillRegistry', () => {
-  it('registers and retrieves skills', async () => {
-    const { Skill, SkillRegistry } = await import('../src/core/skill');
-    const reg = new SkillRegistry();
-    const skill = new Skill({ name: 'test', description: 'Test' });
-    reg.register(skill);
-    expect(reg.get('test')).toBe(skill);
-    expect(reg.get('missing')).toBeUndefined();
-  });
-  it('lists names', async () => {
-    const { Skill, SkillRegistry } = await import('../src/core/skill');
-    const reg = new SkillRegistry();
-    reg.register(new Skill({ name: 'a', description: 'A' }));
-    reg.register(new Skill({ name: 'b', description: 'B' }));
-    expect(reg.listNames()).toEqual(['a', 'b']);
-  });
-  it('merges registries', async () => {
-    const { Skill, SkillRegistry } = await import('../src/core/skill');
-    const r1 = new SkillRegistry();
-    r1.register(new Skill({ name: 'a', description: 'A' }));
-    const r2 = new SkillRegistry();
-    r2.register(new Skill({ name: 'b', description: 'B' }));
-    r1.merge(r2);
-    expect(r1.get('b')).toBeDefined();
-  });
-});
+/**
+ * Tests for skill system.
+ */
+import { describe, it, expect } from 'vitest';
+import * as fs from 'fs';
+import * as path from 'path';
+describe('Skill', () => {
+  it('creates a skill with model override', async () => {
+    const { Skill } = await import('../src/core/skill');
+    const skill = new Skill({
+      name: 'test_skill',
+      description: 'A test skill',
+      model: 'claude-opus-4-7',
+    });
+    expect(skill.model).toBe('claude-opus-4-7');
+    expect(skill.temperature).toBeNull();
+    expect(skill.maxTokens).toBeNull();
+  });
+  it('creates a skill with temperature override', async () => {
+    const { Skill } = await import('../src/core/skill');
+    const skill = new Skill({ name: 'test_skill', description: 'A test skill', temperature: 0.3 });
+    expect(skill.temperature).toBe(0.3);
+  });
+  it('creates a skill with max_tokens override', async () => {
+    const { Skill } = await import('../src/core/skill');
+    const skill = new Skill({ name: 'test_skill', description: 'A test skill', maxTokens: 32000 });
+    expect(skill.maxTokens).toBe(32000);
+  });
+  it('default has no overrides', async () => {
+    const { Skill } = await import('../src/core/skill');
+    const skill = new Skill({ name: 'test', description: 'test' });
+    expect(skill.model).toBeNull();
+    expect(skill.temperature).toBeNull();
+    expect(skill.maxTokens).toBeNull();
+  });
+});
+describe('Skill.fromMarkdown', () => {
+  it('loads skill with YAML frontmatter', async () => {
+    const { Skill } = await import('../src/core/skill');
+    const tmpDir = fs.mkdtempSync('skill-test-');
+    const mdPath = path.join(tmpDir, 'test.md');
+    fs.writeFileSync(mdPath, `---
+name: test_skill
+description: A test
+model: claude-sonnet-4-6
+temperature: 0.5
+max_tokens: 8192
+---
+## Test Skill
+This is a test skill.
+`, 'utf-8');
+    const skill = Skill.fromMarkdown(mdPath);
+    expect(skill).not.toBeNull();
+    if (skill) {
+      expect(skill.model).toBe('claude-sonnet-4-6');
+      expect(skill.temperature).toBe(0.5);
+      expect(skill.maxTokens).toBe(8192);
+      expect(skill.systemPrompt).toContain('This is a test skill');
+    }
+    fs.rmSync(tmpDir, { recursive: true, force: true });
+  });
+  it('supports string path', async () => {
+    const { Skill } = await import('../src/core/skill');
+    const tmpDir = fs.mkdtempSync('skill-test-');
+    const mdPath = path.join(tmpDir, 'x.md');
+    fs.writeFileSync(mdPath, '---\nname: x\ndescription: x\n---\n\nBody.', 'utf-8');
+    const s = Skill.fromMarkdown(mdPath);
+    expect(s).not.toBeNull();
+    if (s) {
+      expect(s.name).toBe('x');
+      expect(typeof s.sourcePath).toBe('string');
+    }
+    fs.rmSync(tmpDir, { recursive: true, force: true });
+  });
+  it('derives triggers from quoted descriptions', async () => {
+    const { Skill } = await import('../src/core/skill');
+    const tmpDir = fs.mkdtempSync('skill-test-');
+    const mdPath = path.join(tmpDir, 'pptx.md');
+    fs.writeFileSync(mdPath, `---
+name: pptx
+description: |-
+  Use this skill any time a .pptx file is involved.
+  Trigger whenever the user mentions "deck," "slides," or "presentation."
+  If a .pptx file needs to be opened, use it.
+---
+Body.`, 'utf-8');
+    const s = Skill.fromMarkdown(mdPath);
+    expect(s).not.toBeNull();
+    if (s) {
+      const triggersLower = s.triggers.map(t => t.toLowerCase());
+      expect(triggersLower).toContain('deck');
+      expect(triggersLower).toContain('slides');
+      expect(triggersLower).toContain('presentation');
+      expect(triggersLower).toContain('.pptx');
+    }
+    fs.rmSync(tmpDir, { recursive: true, force: true });
+  });
+  it('small body loaded in full', async () => {
+    const { Skill } = await import('../src/core/skill');
+    const tmpDir = fs.mkdtempSync('skill-test-');
+    const mdPath = path.join(tmpDir, 'small.md');
+    fs.writeFileSync(mdPath, '---\nname: small\ndescription: x\n---\n\n# Small Skill\n\nThis fits inline easily.', 'utf-8');
+    const s = Skill.fromMarkdown(mdPath);
+    expect(s).not.toBeNull();
+    if (s) {
+      expect(s.bodyTruncated).toBe(false);
+      expect(s.systemPrompt).toContain('This fits inline easily');
+    }
+    fs.rmSync(tmpDir, { recursive: true, force: true });
+  });
+  it('large body truncated to head', async () => {
+    const { Skill } = await import('../src/core/skill');
+    const tmpDir = fs.mkdtempSync('skill-test-');
+    const mdPath = path.join(tmpDir, 'big.md');
+    let body = '# Big Skill\n\n## Quick Reference\nFirst section content.\n\n';
+    body += '## Detailed Guide\n';
+    body += 'X'.repeat(5000);
+    fs.writeFileSync(mdPath, `---\nname: big\ndescription: x\n---\n\n${body}`, 'utf-8');
+    const s = Skill.fromMarkdown(mdPath);
+    expect(s).not.toBeNull();
+    if (s) {
+      expect(s.bodyTruncated).toBe(true);
+      expect(s.systemPrompt).toContain('Big Skill');
+      expect(s.systemPrompt).toContain('Quick Reference');
+      expect(s.systemPrompt).not.toContain('Detailed Guide');
+    }
+    fs.rmSync(tmpDir, { recursive: true, force: true });
+  });
+});
+describe('SkillRegistry', () => {
+  it('registers and retrieves skills', async () => {
+    const { Skill, SkillRegistry } = await import('../src/core/skill');
+    const reg = new SkillRegistry();
+    const skill = new Skill({ name: 'test', description: 'Test' });
+    reg.register(skill);
+    expect(reg.get('test')).toBe(skill);
+    expect(reg.get('missing')).toBeUndefined();
+  });
+  it('lists names', async () => {
+    const { Skill, SkillRegistry } = await import('../src/core/skill');
+    const reg = new SkillRegistry();
+    reg.register(new Skill({ name: 'a', description: 'A' }));
+    reg.register(new Skill({ name: 'b', description: 'B' }));
+    expect(reg.listNames()).toEqual(['a', 'b']);
+  });
+  it('merges registries', async () => {
+    const { Skill, SkillRegistry } = await import('../src/core/skill');
+    const r1 = new SkillRegistry();
+    r1.register(new Skill({ name: 'a', description: 'A' }));
+    const r2 = new SkillRegistry();
+    r2.register(new Skill({ name: 'b', description: 'B' }));
+    r1.merge(r2);
+    expect(r1.get('b')).toBeDefined();
+  });
+});