@rune-kit/rune 2.1.1 → 2.2.1

package/compiler/emitter.js

@@ -8,7 +8,7 @@
 import { readdir, readFile, mkdir, writeFile } from 'node:fs/promises';
 import { existsSync } from 'node:fs';
 import path from 'node:path';
-import { parseSkill, parsePack } from './parser.js';
+import { parseSkill, parsePack, extractCrossRefs, extractToolRefs } from './parser.js';
 import { transformSkill } from './transformer.js';
 
 /**
@@ -124,15 +124,29 @@ export async function buildAll({ runeRoot, outputRoot, adapter, disabledSkills =
 
       const { header, body, footer } = transformSkill(parsed, adapter);
       const output = [header, body, footer].filter(Boolean).join('\n');
-      const fileName = outputFileName(parsed.name, adapter);
-      const outputPath = path.join(outputDir, fileName);
+
+      let outputPath;
+      let displayName;
+
+      if (adapter.useSkillDirectories) {
+        // Directory-per-skill: .codex/skills/rune-{name}/SKILL.md
+        const dirName = `${adapter.skillPrefix}${parsed.name}`;
+        const skillDir = path.join(outputDir, dirName);
+        await mkdir(skillDir, { recursive: true });
+        outputPath = path.join(skillDir, adapter.skillFileName || 'SKILL.md');
+        displayName = `${dirName}/${adapter.skillFileName || 'SKILL.md'}`;
+      } else {
+        const fileName = outputFileName(parsed.name, adapter);
+        outputPath = path.join(outputDir, fileName);
+        displayName = fileName;
+      }
 
       await writeFile(outputPath, output, 'utf-8');
 
       stats.skillCount++;
       stats.crossRefsResolved += parsed.crossRefs.length;
       stats.toolRefsResolved += parsed.toolRefs.length;
-      stats.files.push(fileName);
+      stats.files.push(displayName);
     } catch (err) {
       stats.errors.push({ file: skillPath, error: err.message });
     }
@@ -143,16 +157,56 @@ export async function buildAll({ runeRoot, outputRoot, adapter, disabledSkills =
     try {
       const content = await readFile(packPath, 'utf-8');
       const parsed = parsePack(content, packPath);
+      const packName = path.basename(path.dirname(packPath));
+      const packDir = path.dirname(packPath);
+
+      // For split packs, load individual skill files and concatenate into body
+      if (parsed.isSplit && parsed.skillManifest.length > 0) {
+        const skillBodies = [];
+        for (const skill of parsed.skillManifest) {
+          const skillPath = path.join(packDir, skill.file);
+          if (existsSync(skillPath)) {
+            const skillContent = await readFile(skillPath, 'utf-8');
+            // Strip frontmatter from skill file — we only need the body
+            const skillBodyMatch = skillContent.replace(/\r\n/g, '\n').match(/^---\n[\s\S]*?\n---\n?([\s\S]*)$/);
+            const skillBody = skillBodyMatch ? skillBodyMatch[1].trim() : skillContent.trim();
+            skillBodies.push(skillBody);
+          } else {
+            stats.errors.push({ file: skillPath, error: `Skill file not found (listed in ${packPath} manifest)` });
+          }
+        }
+        // Concatenate: index body + all skill bodies
+        parsed.body = parsed.body + '\n\n' + skillBodies.join('\n\n---\n\n');
+        // Re-extract refs from the full concatenated body
+        parsed.crossRefs = extractCrossRefs(parsed.body);
+        parsed.toolRefs = extractToolRefs(parsed.body);
+      }
+
+      // Normalize pack name for headers (ext-trading instead of @rune/trading)
+      parsed.name = `ext-${packName}`;
+
       const { header, body, footer } = transformSkill(parsed, adapter);
       const output = [header, body, footer].filter(Boolean).join('\n');
-      const packName = path.basename(path.dirname(packPath));
-      const fileName = outputFileName(`ext-${packName}`, adapter);
-      const outputPath = path.join(outputDir, fileName);
+
+      let outputPath;
+      let displayName;
+
+      if (adapter.useSkillDirectories) {
+        const dirName = `${adapter.skillPrefix}ext-${packName}`;
+        const outPackDir = path.join(outputDir, dirName);
+        await mkdir(outPackDir, { recursive: true });
+        outputPath = path.join(outPackDir, adapter.skillFileName || 'SKILL.md');
+        displayName = `${dirName}/${adapter.skillFileName || 'SKILL.md'}`;
+      } else {
+        const fileName = outputFileName(`ext-${packName}`, adapter);
+        outputPath = path.join(outputDir, fileName);
+        displayName = fileName;
+      }
 
       await writeFile(outputPath, output, 'utf-8');
 
       stats.packCount++;
-      stats.files.push(fileName);
+      stats.files.push(displayName);
     } catch (err) {
       stats.errors.push({ file: packPath, error: err.message });
     }
@@ -221,12 +275,12 @@ function generateIndex(stats, adapter) {
     '## Core Skills',
     '',
     ...stats.files
-      .filter(f => !f.includes('ext-') && !f.includes('index'))
+      .filter(f => !f.match(/[-/]ext-/) && !f.includes('index'))
       .map(f => `- ${f}`),
     '',
   ];
 
-  const extFiles = stats.files.filter(f => f.includes('ext-'));
+  const extFiles = stats.files.filter(f => f.match(/[-/]ext-/));
   if (extFiles.length > 0) {
     lines.push('## Extension Packs', '', ...extFiles.map(f => `- ${f}`), '');
   }

package/compiler/parser.js

@@ -65,7 +65,7 @@ function parseFrontmatter(content) {
 /**
  * Extract all rune:<name> cross-references from body
  */
-function extractCrossRefs(body) {
+export function extractCrossRefs(body) {
   const refs = [];
   const lines = body.split('\n');
 
@@ -90,7 +90,7 @@ function extractCrossRefs(body) {
 /**
  * Extract all Claude Code tool references from body
  */
-function extractToolRefs(body) {
+export function extractToolRefs(body) {
   const refs = [];
   const lines = body.split('\n');
 
@@ -188,15 +188,25 @@ export function parseSkill(content, filePath = '') {
 
 /**
  * Parse a PACK.md extension file (same format, slightly different metadata)
+ * Supports both monolith format (all skills inline) and split format (skills in separate files)
+ *
+ * Split format detection: metadata.format === "split" OR metadata.skills array present
  */
 export function parsePack(content, filePath = '') {
   const { frontmatter, body } = parseFrontmatter(content);
+  const metadata = frontmatter.metadata || {};
+
+  // Detect split format
+  const isSplit = metadata.format === 'split' || Array.isArray(metadata.skills);
 
   return {
     name: frontmatter.name || '',
     description: frontmatter.description || '',
-    version: frontmatter.version || '1.0.0',
+    version: metadata.version || frontmatter.version || '1.0.0',
     layer: 'L4',
+    group: 'extension',
+    isSplit,
+    skillManifest: isSplit ? parseSkillManifest(metadata.skills || []) : [],
     body,
     crossRefs: extractCrossRefs(body),
     toolRefs: extractToolRefs(body),
@@ -206,3 +216,32 @@ export function parsePack(content, filePath = '') {
     frontmatter,
   };
 }
+
+/**
+ * Parse the skills manifest from split PACK.md frontmatter
+ *
+ * @param {Array} skills - array of skill entries from frontmatter metadata.skills
+ * @returns {Array<{name: string, file: string, model: string, description: string}>}
+ */
+function parseSkillManifest(skills) {
+  if (!Array.isArray(skills)) return [];
+
+  return skills.map(skill => {
+    // Handle both string format ("skill-name") and object format ({name, file, model, description})
+    if (typeof skill === 'string') {
+      return {
+        name: skill,
+        file: `skills/${skill}.md`,
+        model: 'sonnet',
+        description: '',
+      };
+    }
+
+    return {
+      name: skill.name || '',
+      file: skill.file || `skills/${skill.name}.md`,
+      model: skill.model || 'sonnet',
+      description: skill.description || '',
+    };
+  });
+}

package/compiler/transformer.js

@@ -2,7 +2,7 @@
  * Transform Pipeline
  *
  * Applies all transforms in order to convert a parsed skill into platform-specific output.
- * Pipeline: frontmatter → cross-refs → tool-names → subagents → hooks → branding
+ * Pipeline: frontmatter → cross-refs → tool-names → subagents → compliance → hooks → branding
  */
 
 import { transformCrossReferences } from './transforms/cross-references.js';
@@ -10,6 +10,7 @@ import { transformToolNames } from './transforms/tool-names.js';
 import { transformFrontmatter } from './transforms/frontmatter.js';
 import { transformSubagents } from './transforms/subagents.js';
 import { generateHookConstraints } from './transforms/hooks.js';
+import { transformCompliance } from './transforms/compliance.js';
 import { addBranding } from './transforms/branding.js';
 
 /**
@@ -40,16 +41,19 @@ export function transformSkill(parsedSkill, adapter) {
   // 3. Convert subagent/parallel instructions to sequential
   body = transformSubagents(body, adapter);
 
-  // 4. Platform-specific post-processing
+  // 4. Inject compliance preamble (enforcement for non-Claude platforms)
+  body = transformCompliance(body, adapter);
+
+  // 5. Platform-specific post-processing
   body = adapter.postProcess(body);
 
-  // 5. Generate header (platform-specific frontmatter/preamble)
+  // 6. Generate header (platform-specific frontmatter/preamble)
   const header = adapter.generateHeader(parsedSkill);
 
-  // 6. Generate hook constraints section
+  // 7. Generate hook constraints section
   const hookConstraints = generateHookConstraints(parsedSkill, adapter);
 
-  // 7. Inject hook constraints after the first heading
+  // 8. Inject hook constraints after the first heading
   if (hookConstraints) {
     const firstHeadingEnd = body.indexOf('\n## ');
     if (firstHeadingEnd !== -1) {
@@ -62,7 +66,7 @@ export function transformSkill(parsedSkill, adapter) {
     }
   }
 
-  // 8. Generate footer (branding + CTA)
+  // 9. Generate footer (branding + CTA)
   const footer = adapter.generateFooter();
 
   return { header, body, footer };

package/compiler/transforms/branding.js

@@ -7,7 +7,7 @@
 const DEFAULT_FOOTER = [
   '',
   '---',
-  '> **Rune Skill Mesh** — 49 skills, 170+ connections',
+  '> **Rune Skill Mesh** — 58 skills, 200+ connections',
   '> Source: https://github.com/rune-kit/rune',
   '> For the full experience with subagents, hooks, adaptive routing, and mesh analytics — use Rune as a Claude Code plugin.',
 ].join('\n');

package/compiler/transforms/compliance.js

@@ -0,0 +1,40 @@
+/**
+ * Compliance Transform
+ *
+ * Injects a compliance preamble into every compiled skill file.
+ * This distributes enforcement to ALL platforms (Cursor, Windsurf, etc.)
+ * so the AI is reminded of routing discipline even if skill-router
+ * itself is not loaded in context.
+ *
+ * Only applies to non-Claude platforms (Claude has native plugin enforcement).
+ */
+
+const COMPLIANCE_PREAMBLE = [
+  '',
+  '> **RUNE COMPLIANCE**: Before ANY code response, you MUST:',
+  '> 1. Classify this request (CODE_CHANGE | QUESTION | DEBUG | REVIEW | EXPLORE)',
+  '> 2. Route through the correct Rune skill (see skill-router routing table)',
+  "> 3. Follow the skill's workflow — do NOT freelance or skip steps",
+  '> Violation: writing code without skill routing = incorrect behavior.',
+  '',
+].join('\n');
+
+/**
+ * Inject compliance preamble after the first heading in a skill body.
+ *
+ * @param {string} body - skill body text
+ * @param {object} adapter - platform adapter
+ * @returns {string} body with compliance preamble injected
+ */
+export function transformCompliance(body, adapter) {
+  if (adapter.name === 'claude') return body;
+
+  // Insert after first heading (# skill-name), before first ## section
+  const firstH2 = body.indexOf('\n## ');
+  if (firstH2 !== -1) {
+    return body.slice(0, firstH2) + COMPLIANCE_PREAMBLE + body.slice(firstH2);
+  }
+
+  // Fallback: append at end if no ## sections found
+  return body + COMPLIANCE_PREAMBLE;
+}