kushi-agents 5.0.4 → 5.2.0

package/src/parallel-refresh.test.mjs

@@ -0,0 +1,50 @@
+// kushi v5.2.0 — parallel refresh tests.
+// Validates worker pool mechanics, failure aggregation, and --sequential flag.
+
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import fs from 'node:fs';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const repoRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..');
+
+test('parallel: refresh-project SKILL.md documents parallel dispatch', () => {
+  const skillFile = path.join(repoRoot, 'plugin', 'skills', 'refresh-project', 'SKILL.md');
+  const content = fs.readFileSync(skillFile, 'utf8');
+  // Must reference parallel dispatch
+  assert.match(content, /parallel/i, 'refresh-project must mention parallel dispatch');
+  assert.match(content, /sequential/i, 'refresh-project must mention --sequential fallback');
+});
+
+test('parallel: parallel-execution doctrine exists and has key sections', () => {
+  const doctrine = path.join(repoRoot, 'plugin', 'instructions', 'parallel-execution.instructions.md');
+  assert.ok(fs.existsSync(doctrine), 'parallel-execution.instructions.md must exist');
+  const content = fs.readFileSync(doctrine, 'utf8');
+  assert.match(content, /max_workers/, 'must document max_workers config');
+  assert.match(content, /throttle/, 'must document throttling');
+  assert.match(content, /error aggregation/i, 'must document error aggregation');
+  assert.match(content, /deterministic/i, 'must document deterministic ordering');
+});
+
+test('parallel: config supports max_workers setting', () => {
+  const doctrine = path.join(repoRoot, 'plugin', 'instructions', 'parallel-execution.instructions.md');
+  const content = fs.readFileSync(doctrine, 'utf8');
+  // Verify default value documented
+  assert.match(content, /default.*4|4.*default/i, 'default max_workers should be 4');
+  // Verify sequential = max_workers 1
+  assert.match(content, /sequential.*1|1.*sequential/i, '--sequential means max_workers=1');
+});
+
+test('parallel: failures in one worker do not halt others', () => {
+  // Structural test: verify the doctrine explicitly states this
+  const doctrine = path.join(repoRoot, 'plugin', 'instructions', 'parallel-execution.instructions.md');
+  const content = fs.readFileSync(doctrine, 'utf8');
+  assert.match(content, /failure.*NOT.*halt|independent/i, 'must state workers are independent');
+});
+
+test('parallel: refresh-project references parallel-execution doctrine', () => {
+  const skillFile = path.join(repoRoot, 'plugin', 'skills', 'refresh-project', 'SKILL.md');
+  const content = fs.readFileSync(skillFile, 'utf8');
+  assert.match(content, /parallel-execution\.instructions\.md/, 'must cite parallel doctrine');
+});

package/src/schema-evolve.test.mjs

@@ -0,0 +1,78 @@
+// kushi v5.2.0 — schema-evolve skill tests.
+// Validates rule capture and persistence behavior.
+
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import fs from 'node:fs';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const repoRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..');
+
+test('schema-evolve: SKILL.md exists with correct structure', () => {
+  const skillFile = path.join(repoRoot, 'plugin', 'skills', 'schema-evolve', 'SKILL.md');
+  assert.ok(fs.existsSync(skillFile), 'schema-evolve/SKILL.md must exist');
+  const content = fs.readFileSync(skillFile, 'utf8');
+  assert.match(content, /name:\s*"schema-evolve"/, 'must declare name');
+  assert.match(content, /USE WHEN/, 'description must start with USE WHEN');
+  assert.match(content, /CLAUDE\.md/, 'must reference CLAUDE.md as storage');
+  assert.match(content, /Step checklist/, 'must have step checklist (orchestrator pattern)');
+});
+
+test('schema-evolve: doctrine exists and defines rule format', () => {
+  const doctrine = path.join(repoRoot, 'plugin', 'instructions', 'schema-evolve.instructions.md');
+  assert.ok(fs.existsSync(doctrine), 'schema-evolve.instructions.md must exist');
+  const content = fs.readFileSync(doctrine, 'utf8');
+  assert.match(content, /## Rule:/, 'must document rule heading format');
+  assert.match(content, /Added/, 'must document timestamp field');
+  assert.match(content, /Scope/, 'must document scope field');
+  assert.match(content, /CLAUDE\.md/, 'must specify CLAUDE.md as storage location');
+});
+
+test('schema-evolve: rules stored in CLAUDE.md with proper structure', () => {
+  // Create a fixture with a CLAUDE.md and verify format
+  const fixture = path.join(repoRoot, '.testtmp', `schema-evolve-${Date.now()}`);
+  const stateDir = path.join(fixture, 'Evidence', 'tester', 'State');
+  fs.mkdirSync(stateDir, { recursive: true });
+
+  try {
+    // Simulate what schema-evolve would write
+    const claudeContent = `---
+kushi_state_page: true
+---
+
+# CLAUDE.md — Project Conventions
+
+Rules captured via \`schema-evolve\` skill. Read by build-state, ask-project, refresh-project at run start.
+
+## Rule: Use HCA abbreviation
+
+- **Added**: 2026-05-29T14:30:00Z
+- **Scope**: project
+- **Source**: user (natural language)
+
+Always use 'HCA' not 'Healthcare Accelerator' in summaries.
+`;
+    const claudePath = path.join(stateDir, 'CLAUDE.md');
+    fs.writeFileSync(claudePath, claudeContent);
+
+    // Verify structure
+    const written = fs.readFileSync(claudePath, 'utf8');
+    assert.match(written, /kushi_state_page:\s*true/, 'must have frontmatter');
+    assert.match(written, /## Rule:/, 'must have rule heading');
+    assert.match(written, /\*\*Added\*\*:/, 'must have Added timestamp');
+    assert.match(written, /\*\*Scope\*\*:/, 'must have Scope field');
+    assert.match(written, /\*\*Source\*\*:/, 'must have Source field');
+  } finally {
+    fs.rmSync(fixture, { recursive: true, force: true });
+  }
+});
+
+test('schema-evolve: hard doctrine override detection documented', () => {
+  const skillFile = path.join(repoRoot, 'plugin', 'skills', 'schema-evolve', 'SKILL.md');
+  const content = fs.readFileSync(skillFile, 'utf8');
+  // Must document that WorkIQ-only, CSC, verbatim-by-default cannot be overridden
+  assert.match(content, /WorkIQ-only/, 'must mention WorkIQ-only as non-overridable');
+  assert.match(content, /verbatim-by-default/, 'must mention verbatim-by-default');
+  assert.match(content, /CSC/, 'must mention CSC format');
+});

package/src/teach.test.mjs

@@ -0,0 +1,45 @@
+// kushi v5.2.0 — teach skill tests.
+// Validates topic matching and unknown-topic handling.
+
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import fs from 'node:fs';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const repoRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..');
+
+test('teach: SKILL.md exists and has correct structure', () => {
+  const skillFile = path.join(repoRoot, 'plugin', 'skills', 'teach', 'SKILL.md');
+  assert.ok(fs.existsSync(skillFile), 'teach/SKILL.md must exist');
+  const content = fs.readFileSync(skillFile, 'utf8');
+  // Check frontmatter
+  assert.match(content, /name:\s*"teach"/, 'must declare name: teach');
+  assert.match(content, /USE WHEN/, 'description must start with USE WHEN');
+  // Check topic mapping table
+  assert.match(content, /contradictions.*living-wiki/i, 'contradictions topic must map to living-wiki');
+  assert.match(content, /parallel.*parallel-execution/i, 'parallel topic must map to doctrine');
+  assert.match(content, /otel.*otel\.instructions/i, 'otel topic must map to doctrine');
+});
+
+test('teach: topic mapping covers all v5.2.0 features', () => {
+  const skillFile = path.join(repoRoot, 'plugin', 'skills', 'teach', 'SKILL.md');
+  const content = fs.readFileSync(skillFile, 'utf8');
+  // Each new v5.2.0 feature should be teachable
+  const requiredTopics = ['hooks', 'parallel', 'otel', 'schema'];
+  for (const topic of requiredTopics) {
+    assert.match(content, new RegExp(topic, 'i'), `teach must cover topic: ${topic}`);
+  }
+});
+
+test('teach: references existing doctrine files', () => {
+  const skillFile = path.join(repoRoot, 'plugin', 'skills', 'teach', 'SKILL.md');
+  const content = fs.readFileSync(skillFile, 'utf8');
+  // Extract doctrine file references
+  const doctrineRefs = content.match(/[\w-]+\.instructions\.md/g) || [];
+  // Each referenced doctrine should exist
+  for (const ref of doctrineRefs) {
+    const docPath = path.join(repoRoot, 'plugin', 'instructions', ref);
+    assert.ok(fs.existsSync(docPath), `referenced doctrine must exist: ${ref}`);
+  }
+});