@skillbase/compiler 0.2.5 → 0.2.7

package/dist/index.cjs

@@ -30,8 +30,11 @@ var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: tru
 // src/index.ts
 var index_exports = {};
 __export(index_exports, {
+  SECTION_ORDER: () => SECTION_ORDER,
+  SECTION_TYPE_LABELS: () => SECTION_TYPE_LABELS,
   SkillParseError: () => SkillParseError,
   SoulParseError: () => SoulParseError,
+  autoFixSections: () => autoFixSections,
   buildLegacyBody: () => buildLegacyBody,
   canonicalToParsedSkill: () => canonicalToParsedSkill,
   canonicalToParsedSoul: () => canonicalToParsedSoul,
@@ -42,6 +45,7 @@ __export(index_exports, {
   getCompiler: () => getCompiler,
   getImporter: () => getImporter,
   importFromFormat: () => importFromFormat,
+  inferSectionType: () => inferSectionType,
   inferTags: () => inferTags,
   isVercelFormat: () => isVercelFormat,
   legacyPersonaToSoul: () => legacyPersonaToSoul,
@@ -52,9 +56,12 @@ __export(index_exports, {
   parseSoul: () => parseSoul,
   parsedSkillToCanonical: () => parsedSkillToCanonical,
   parsedSoulToCanonical: () => parsedSoulToCanonical,
+  reorderSections: () => reorderSections,
   resolveContextSlots: () => resolveContextSlots,
   serializeSkill: () => serializeSkill,
   serializeSoul: () => serializeSoul,
+  validateCanonicalOrder: () => validateCanonicalOrder,
+  validateSectionOrder: () => validateSectionOrder,
   validateSkillFrontmatter: () => validateSkillFrontmatter,
   validateSoulFrontmatter: () => validateSoulFrontmatter
 });
@@ -86,6 +93,91 @@ ${yamlStr}
 ${content}`;
 }
 
+// src/section-order.ts
+var SECTION_ORDER = {
+  context: 0,
+  instructions: 1,
+  guidelines: 2,
+  verification: 3,
+  examples: 4,
+  custom: 5
+};
+var SECTION_TYPE_LABELS = {
+  context: "Context",
+  instructions: "Instructions",
+  guidelines: "Guidelines",
+  verification: "Verification",
+  examples: "Examples",
+  custom: "Custom"
+};
+var TYPE_PATTERNS = [
+  {
+    type: "context",
+    keywords: /\b(context|background|overview|about|introduction|who you are|role|persona)\b/i
+  },
+  {
+    type: "examples",
+    keywords: /\b(example|sample|demo|few.?shot|illustration|showcase)\b/i
+  },
+  {
+    type: "guidelines",
+    keywords: /\b(guideline|style|convention|best.?practice|approach|principle|standard|rule|format|tone)\b/i
+  },
+  {
+    type: "verification",
+    keywords: /\b(verif|check|valid|review|self.?check|quality|assert|ensure|confirm|output.?requirement)\b/i
+  },
+  {
+    type: "instructions",
+    keywords: /\b(instruct|task|step|procedure|workflow|process|how to|implement|action|do)\b/i
+  }
+];
+function inferSectionType(label) {
+  if (!label) return "instructions";
+  for (const { type, keywords } of TYPE_PATTERNS) {
+    if (keywords.test(label)) return type;
+  }
+  return "custom";
+}
+function validateSectionOrder(sections) {
+  if (sections.length <= 1) return [];
+  const reordered = reorderSections(sections);
+  const issues = [];
+  for (let i = 0; i < sections.length; i++) {
+    const current = sections[i];
+    const expectedIdx = reordered.indexOf(current);
+    if (expectedIdx !== i) {
+      const typeLabel = SECTION_TYPE_LABELS[current.type] ?? current.type;
+      issues.push({
+        index: i,
+        section: current,
+        expectedIndex: expectedIdx,
+        message: `"${current.label || typeLabel}" is at position ${i + 1}, expected ${expectedIdx + 1}`
+      });
+    }
+  }
+  return issues;
+}
+function reorderSections(sections) {
+  return [...sections].sort((a, b) => {
+    const orderA = SECTION_ORDER[a.type] ?? SECTION_ORDER.custom;
+    const orderB = SECTION_ORDER[b.type] ?? SECTION_ORDER.custom;
+    return orderA - orderB;
+  });
+}
+function autoFixSections(sections) {
+  const typed = sections.map((s) => {
+    if (s.type !== "custom") return s;
+    return { ...s, type: inferSectionType(s.label ?? "") };
+  });
+  return reorderSections(typed);
+}
+function validateCanonicalOrder(canonical) {
+  const sections = canonical.instructions.sections;
+  if (!sections || sections.length <= 1) return [];
+  return validateSectionOrder(sections);
+}
+
 // src/parse/skill-parser.ts
 var REQUIRED_FIELDS = [
   "schema_version",
@@ -159,16 +251,17 @@ function splitBodyIntoSections(body) {
   return { sections };
 }
 function normalizeCanonical(canonical) {
-  const { content } = canonical.instructions;
-  if (!content) return canonical;
-  const { sections } = splitBodyIntoSections(content);
+  const flat = denormalizeCanonical(canonical);
+  const fullContent = flat.instructions.content;
+  if (!fullContent) return canonical;
+  const { sections } = splitBodyIntoSections(fullContent);
   if (sections.length === 0) return canonical;
   return {
     ...canonical,
     instructions: {
       content: "",
       sections: sections.map((s) => ({
-        type: "custom",
+        type: inferSectionType(s.label),
         label: s.label,
         content: s.content
       }))
@@ -179,6 +272,9 @@ function denormalizeCanonical(canonical) {
   const sections = canonical.instructions.sections ?? [];
   if (sections.length === 0) return canonical;
   const parts = [];
+  if (canonical.instructions.content) {
+    parts.push(canonical.instructions.content);
+  }
   for (const s of sections) {
     if (s.label) {
       parts.push(`## ${s.label}
@@ -1279,7 +1375,21 @@ function buildFlatBody(canonical, options) {
       parts.push(`Tone: ${canonical.identity.tone}`, "");
     }
   }
-  parts.push(canonical.rawBody ?? canonical.instructions.content);
+  const sections = canonical.instructions.sections;
+  if (sections && sections.length > 0) {
+    const sorted = reorderSections(sections);
+    for (const s of sorted) {
+      if (s.label) {
+        parts.push(`## ${s.label}
+
+${s.content}`);
+      } else if (s.content) {
+        parts.push(s.content);
+      }
+    }
+  } else {
+    parts.push(canonical.rawBody ?? canonical.instructions.content);
+  }
   if (canonical.constraints) {
     if (canonical.constraints.always && canonical.constraints.always.length > 0) {
       parts.push(
@@ -1319,7 +1429,7 @@ function buildBody(canonical) {
   if (canonical.instructions.content) {
     parts.push(canonical.instructions.content);
   }
-  const sections = canonical.instructions.sections ?? [];
+  const sections = reorderSections(canonical.instructions.sections ?? []);
   for (const s of sections) {
     if (s.label || s.content) {
       if (s.label) {
@@ -1779,8 +1889,11 @@ function getCompiler(target) {
 }
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
+  SECTION_ORDER,
+  SECTION_TYPE_LABELS,
   SkillParseError,
   SoulParseError,
+  autoFixSections,
   buildLegacyBody,
   canonicalToParsedSkill,
   canonicalToParsedSoul,
@@ -1791,6 +1904,7 @@ function getCompiler(target) {
   getCompiler,
   getImporter,
   importFromFormat,
+  inferSectionType,
   inferTags,
   isVercelFormat,
   legacyPersonaToSoul,
@@ -1801,9 +1915,12 @@ function getCompiler(target) {
   parseSoul,
   parsedSkillToCanonical,
   parsedSoulToCanonical,
+  reorderSections,
   resolveContextSlots,
   serializeSkill,
   serializeSoul,
+  validateCanonicalOrder,
+  validateSectionOrder,
   validateSkillFrontmatter,
   validateSoulFrontmatter
 });