heraspec 0.1.14 → 0.1.15

package/bin/heraspec.js

@@ -214,95 +214,145 @@ var init_skills_template_map = __esm({
     SKILL_TEMPLATE_MAP = {
       // Cross-cutting skills
       "ui-ux": {
-        templateFileName: "ui-ux-skill.md",
+        templateFileName: "ui-ux/skill.md",
         isCrossCutting: true,
-        resourceDirs: ["scripts", "templates", "data"]
+        resourceDirs: ["ui-ux/scripts", "ui-ux/templates", "ui-ux/data"]
       },
       "design-system": {
-        templateFileName: "design-system-skill.md",
+        templateFileName: "design-system/skill.md",
         isCrossCutting: true,
-        resourceDirs: ["scripts", "data", "knowledge/design-systems"]
+        resourceDirs: ["ui-ux/scripts", "ui-ux/data", "design-system/knowledge/design-systems"]
       },
       "documents": {
-        templateFileName: "documents-skill.md",
+        templateFileName: "documents/skill.md",
         isCrossCutting: true,
         resourceDirs: ["documents/templates"]
       },
       "content-optimization": {
-        templateFileName: "content-optimization-skill.md",
+        templateFileName: "content-optimization/skill.md",
         isCrossCutting: true
       },
       "unit-test": {
-        templateFileName: "unit-test-skill.md",
+        templateFileName: "unit-test/skill.md",
         isCrossCutting: true
       },
       "integration-test": {
-        templateFileName: "integration-test-skill.md",
+        templateFileName: "integration-test/skill.md",
         isCrossCutting: true
       },
       "e2e-test": {
-        templateFileName: "e2e-test-skill.md",
+        templateFileName: "e2e-test/skill.md",
         isCrossCutting: true
       },
       "suggestion": {
-        templateFileName: "suggestion-skill.md",
+        templateFileName: "suggestion/skill.md",
         isCrossCutting: true
       },
       "sourcecode-analyzer": {
-        templateFileName: "sourcecode-analyzer-skill.md",
+        templateFileName: "sourcecode-analyzer/skill.md",
         isCrossCutting: true,
-        viFileName: "sourcecode-analyzer-skill.vi.md"
+        viFileName: "sourcecode-analyzer/skill.vi.md"
       },
       "project-memory": {
-        templateFileName: "project-memory-skill.md",
-        viFileName: "project-memory-skill.vi.md",
+        templateFileName: "project-memory/skill.md",
+        viFileName: "project-memory/skill.vi.md",
         isCrossCutting: true
       },
       "smart-explore": {
-        templateFileName: "smart-explore-skill.md",
+        templateFileName: "smart-explore/skill.md",
         isCrossCutting: true
       },
       "deploy-documentation": {
-        templateFileName: "deploy-documentation-skill.md",
+        templateFileName: "deploy-documentation/skill.md",
         isCrossCutting: true,
-        resourceDirs: ["templates"]
+        resourceDirs: ["ui-ux/templates"]
+      },
+      "code-review": {
+        templateFileName: "code-review/skill.md",
+        viFileName: "code-review/skill.vi.md",
+        isCrossCutting: true
+      },
+      "debug": {
+        templateFileName: "debug/skill.md",
+        viFileName: "debug/skill.vi.md",
+        isCrossCutting: true
+      },
+      "system-design": {
+        templateFileName: "system-design/skill.md",
+        viFileName: "system-design/skill.vi.md",
+        isCrossCutting: true
+      },
+      "tech-debt": {
+        templateFileName: "tech-debt/skill.md",
+        viFileName: "tech-debt/skill.vi.md",
+        isCrossCutting: true
+      },
+      "spec-writer": {
+        templateFileName: "spec-writer/skill.md",
+        viFileName: "spec-writer/skill.vi.md",
+        isCrossCutting: true
+      },
+      "seo-audit": {
+        templateFileName: "seo-audit/skill.md",
+        viFileName: "seo-audit/skill.vi.md",
+        isCrossCutting: true
+      },
+      "campaign-plan": {
+        templateFileName: "campaign-plan/skill.md",
+        viFileName: "campaign-plan/skill.vi.md",
+        isCrossCutting: true
+      },
+      "content-creation": {
+        templateFileName: "content-creation/skill.md",
+        viFileName: "content-creation/skill.vi.md",
+        isCrossCutting: true
+      },
+      "email-sequence": {
+        templateFileName: "email-sequence/skill.md",
+        viFileName: "email-sequence/skill.vi.md",
+        isCrossCutting: true
+      },
+      "sql-queries": {
+        templateFileName: "sql-queries/skill.md",
+        viFileName: "sql-queries/skill.vi.md",
+        isCrossCutting: true
       },
       // Perfex module skills
       "perfex-module:module-codebase": {
-        templateFileName: "module-codebase-skill.md",
+        templateFileName: "perfex-module/module-codebase/skill.md",
         isCrossCutting: false,
         projectType: "perfex-module"
       },
       "wordpress:ux-element": {
-        templateFileName: "ux-element-skill.md",
+        templateFileName: "wordpress/ux-element/skill.md",
         isCrossCutting: false,
         projectType: "wordpress",
-        resourceDirs: ["ux-element/templates"]
+        resourceDirs: ["wordpress/ux-element/templates"]
       },
       "wordpress:plugin-standard": {
-        templateFileName: "wordpress-plugin-standard-skill.md",
+        templateFileName: "wordpress/plugin-standard/skill.md",
         isCrossCutting: false,
         projectType: "wordpress",
-        resourceDirs: ["wordpress-plugin-standard/templates"]
+        resourceDirs: ["wordpress/plugin-standard/templates"]
       },
       "wordpress:plugin-check": {
-        templateFileName: "wordpress-plugin-check-skill.md",
+        templateFileName: "wordpress/plugin-check/skill.md",
         isCrossCutting: false,
         projectType: "wordpress"
       },
       "wordpress:plugin-directory": {
-        templateFileName: "plugin-directory-skill.md",
+        templateFileName: "wordpress/plugin-directory/skill.md",
         isCrossCutting: false,
         projectType: "wordpress"
       },
       // WordPress Plugin specific (matches PROJECT_TYPES)
       "wordpress-plugin:plugin-check": {
-        templateFileName: "wordpress-plugin-check-skill.md",
+        templateFileName: "wordpress/plugin-check/skill.md",
         isCrossCutting: false,
         projectType: "wordpress-plugin"
       },
       "wordpress-plugin:plugin-directory": {
-        templateFileName: "plugin-directory-skill.md",
+        templateFileName: "wordpress/plugin-directory/skill.md",
         isCrossCutting: false,
         projectType: "wordpress-plugin"
       }
@@ -3076,7 +3126,7 @@ var init_skill = __esm({
           if (templateInfo.resourceDirs) {
             for (const resourceDir of templateInfo.resourceDirs) {
               const srcResourceDir = path7.join(coreTemplatesDir, resourceDir);
-              const destResourceDir = path7.join(destPath, resourceDir);
+              const destResourceDir = path7.join(destPath, path7.basename(resourceDir));
               if (await FileSystemUtils.fileExists(srcResourceDir)) {
                 await FileSystemUtils.copyDirectory(srcResourceDir, destResourceDir);
               }
@@ -3162,6 +3212,7 @@ This document defines the workflow for AI agents working with HeraSpec.
 - **SKILL PREREQUISITE**: If a task maps to a skill (e.g., "Generate documentation"), you **MUST** verify the skill folder exists in \`heraspec/skills/\`. If missing:
   - **Preferred**: Proactively install it via \`heraspec skill add <name>\` (e.g., \`heraspec skill add documents\`) if you have terminal access.
   - **Fallback**: If you cannot install it, **STOP** and ask the user to add it. **DO NOT** attempt manual generation without the skill.
+- **IGNORE VIETNAMESE SKILL FILES**: When reading and executing skill workflows, you **MUST ONLY** look at the English version of the skill instructions (e.g., \`skill.md\`). You **MUST completely ignore** all Vietnamese versions (e.g., \`skill.vi.md\` or files ending with \`*.vi.md\`). These Vietnamese files are for reference/development purposes by the user only.
 - **USER CONFIRMATION**: For destructive actions or public releases, always request explicit User approval first.
 
 ### Restricted Commands (REQUIRE USER CONFIRMATION)
@@ -3252,6 +3303,7 @@ The following commands are classified by risk level. You **MUST NOT** execute th
    - Understand purpose, steps, inputs, outputs
    - Follow tone, rules, and limitations
    - Check available templates and scripts
+   - **Ignore Vietnamese Skill Files**: Only read the English \`skill.md\` instructions. Completely ignore any \`*.vi.md\` translation files.
 
 4. **Use skill resources**:
    - Run scripts from \`scripts/\` folder if needed
@@ -3458,6 +3510,7 @@ If \\\`heraspec/skills/smart-explore/\\\` exists, prefer token-efficient explora
      > heraspec skill add documents"
    - **DO NOT** create a task plan or attempt manual generation without the skill language.
 3. **Execute After Install**: Only after the skill folder is created (or installed), read the \`skill.md\` and proceed with the task.
+4. **Ignore Vietnamese Skill Files**: You MUST only read the English \`skill.md\` file inside the skill folder. Completely ignore all \`*.vi.md\` files (e.g., \`skill.vi.md\`), as they are strictly for reference purposes by human developers.
 
 **Example prompt for Documents skill:**
 If you need to generate multi-format documentation, use this prompt:
@@ -3824,20 +3877,86 @@ ${section.content.trimEnd()}`);
     }
   }
   /**
-   * Generic section replacer: replace content from marker to next ## header
+   * Find the index of the next top-level H2 section in the markdown content.
+   * Tracks fenced code blocks to ignore H2 headers inside code blocks.
    */
-  replaceSection(content, sectionMarker, newSection) {
+  findSectionEndIndex(content, startIndex) {
+    const lines = content.substring(startIndex).split(/\r?\n/);
+    let offset = startIndex;
+    let inCodeBlock = false;
+    for (let i = 0; i < lines.length; i++) {
+      const line = lines[i];
+      const lineLengthWithNewline = line.length + (content.charAt(offset + line.length) === "\r" ? 2 : 1);
+      if (i === 0) {
+        offset += lineLengthWithNewline;
+        continue;
+      }
+      if (line.trim().startsWith("```")) {
+        inCodeBlock = !inCodeBlock;
+      }
+      if (!inCodeBlock && line.startsWith("## ")) {
+        return offset;
+      }
+      offset += lineLengthWithNewline;
+    }
+    return content.length;
+  }
+  /**
+   * Finds the start index of the next standard section in the sequence.
+   * This is used to replace the entire section cleanly, even if there are internal H2s or duplicates.
+   */
+  findNextSectionIndex(content, startIndex, currentHeader) {
+    const sectionsOrder = [
+      ["## Universal Safety Rules"],
+      ["## Core Workflow"],
+      ["## Skills System", "## Skills system", "### Skills System", "### skills system"],
+      ["## Proactive Memory-Aware Development", "## Memory-Aware Development"]
+    ];
+    let currentGroupIndex = -1;
+    for (let i = 0; i < sectionsOrder.length; i++) {
+      if (sectionsOrder[i].some((h) => currentHeader.toLowerCase().trim() === h.toLowerCase().trim())) {
+        currentGroupIndex = i;
+        break;
+      }
+    }
+    if (currentGroupIndex === -1) {
+      return this.findSectionEndIndex(content, startIndex);
+    }
+    let nextSectionIndex = -1;
+    for (let i = currentGroupIndex + 1; i < sectionsOrder.length; i++) {
+      const variations = sectionsOrder[i];
+      for (const variation of variations) {
+        const idx = content.indexOf(variation, startIndex + currentHeader.length);
+        if (idx !== -1) {
+          if (nextSectionIndex === -1 || idx < nextSectionIndex) {
+            nextSectionIndex = idx;
+          }
+        }
+      }
+    }
+    if (nextSectionIndex !== -1) {
+      return nextSectionIndex;
+    }
+    return this.findSectionEndIndex(content, startIndex);
+  }
+  /**
+   * Generic section replacer: replace content from marker to next H2 header or endMarker
+   */
+  replaceSection(content, sectionMarker, newSection, endMarker) {
     const startIndex = content.indexOf(sectionMarker);
     if (startIndex === -1) return content;
-    let endIndex = content.indexOf("\n## ", startIndex + sectionMarker.length);
+    let endIndex = -1;
+    if (endMarker) {
+      endIndex = content.indexOf(endMarker, startIndex + sectionMarker.length);
+    }
     if (endIndex === -1) {
-      endIndex = content.length;
+      endIndex = this.findNextSectionIndex(content, startIndex, sectionMarker);
     }
     const before = content.substring(0, startIndex).trimEnd();
     const after = content.substring(endIndex);
     return before + "\n\n" + newSection + (after.trimStart().startsWith("\n") ? "" : "\n\n") + after;
   }
-  replaceSkillsSection(existingContent, newSkillsSection) {
+  replaceSkillsSection(existingContent, newSkillsSection, endMarker) {
     const startMarkers = ["## Skills System", "## Skills system", "### Skills System", "### Skills system"];
     let startIndex = -1;
     let foundMarker = "";
@@ -3851,28 +3970,19 @@ ${section.content.trimEnd()}`);
     if (startIndex === -1) {
       return this.appendSkillsSection(existingContent, newSkillsSection);
     }
-    let endIndex = existingContent.indexOf("\n## ", startIndex + foundMarker.length);
-    if (endIndex === -1) {
-      endIndex = existingContent.indexOf("\n### ", startIndex + foundMarker.length);
+    let endIndex = -1;
+    if (endMarker) {
+      endIndex = existingContent.indexOf(endMarker, startIndex + foundMarker.length);
     }
     if (endIndex === -1) {
-      endIndex = existingContent.length;
+      endIndex = this.findNextSectionIndex(existingContent, startIndex, foundMarker);
     }
     const before = existingContent.substring(0, startIndex).trimEnd();
     const after = existingContent.substring(endIndex);
     return before + "\n\n" + newSkillsSection + (after.trimStart().startsWith("\n") ? "" : "\n\n") + after;
   }
   replaceSafetyRules(existingContent, newSafetySection) {
-    const marker = "## Universal Safety Rules";
-    const startIndex = existingContent.indexOf(marker);
-    if (startIndex === -1) return existingContent;
-    let endIndex = existingContent.indexOf("\n## ", startIndex + marker.length);
-    if (endIndex === -1) {
-      endIndex = existingContent.length;
-    }
-    const before = existingContent.substring(0, startIndex).trimEnd();
-    const after = existingContent.substring(endIndex);
-    return before + "\n\n" + newSafetySection + (after.trimStart().startsWith("\n") ? "" : "\n\n") + after;
+    return this.replaceSection(existingContent, "## Universal Safety Rules", newSafetySection, "## Core Workflow");
   }
   appendSkillsSection(existingContent, skillsSection) {
     const rulesMarker = "\n## Rules\n";
@@ -3966,7 +4076,7 @@ ${section.content.trimEnd()}`);
       if (templateInfo.resourceDirs) {
         for (const resourceDir of templateInfo.resourceDirs) {
           const srcResourceDir = path8.join(coreTemplatesDir, resourceDir);
-          const destResourceDir = path8.join(skillPath, resourceDir);
+          const destResourceDir = path8.join(skillPath, path8.basename(resourceDir));
           if (await FileSystemUtils.fileExists(srcResourceDir)) {
             if (await FileSystemUtils.fileExists(destResourceDir)) {
               await FileSystemUtils.removeDirectory(destResourceDir, true);
@@ -4267,15 +4377,12 @@ python3 heraspec/skills/ui-ux/scripts/search.py "pricing plans" --domain pages
 1. Copy skill from HeraSpec core:
    \`\`\`bash
    # Copy UI/UX skill
-   cp -r /path/to/HeraSpec/src/core/templates/skills/ui-ux-skill.md heraspec/skills/ui-ux/
-   cp -r /path/to/HeraSpec/src/core/templates/skills/scripts heraspec/skills/ui-ux/
-   cp -r /path/to/HeraSpec/src/core/templates/skills/data heraspec/skills/ui-ux/
-   cp -r /path/to/HeraSpec/src/core/templates/skills/templates heraspec/skills/ui-ux/
+   cp -r /path/to/HeraSpec/src/core/templates/skills/ui-ux heraspec/skills/
    \`\`\`
 
 2. Or use \`heraspec skill add ui-ux\` (if available)
 
-3. Read \`heraspec/skills/ui-ux/ui-ux-skill.md\` for complete documentation
+3. Read \`heraspec/skills/ui-ux/skill.md\` for complete documentation
 
 ### Flatsome UX Element Skill
 
@@ -4425,7 +4532,7 @@ python3 heraspec/skills/ui-ux/scripts/search.py "product-detail single-product"
 ## \u{1F4DA} Detailed Documentation
 
 After copying UI/UX skill to your project, see:
-- \`heraspec/skills/ui-ux/ui-ux-skill.md\` - Complete skill documentation
+- \`heraspec/skills/ui-ux/skill.md\` - Complete skill documentation
 - \`heraspec/skills/ui-ux/templates/example-prompt-full-theme.md\` - Detailed prompt examples
 - \`heraspec/skills/ui-ux/templates/prompt-template-full-theme.md\` - Copy-paste templates
 
@@ -4441,10 +4548,7 @@ After copying UI/UX skill to your project, see:
 
 1. Copy UI/UX skill to project:
    \`\`\`bash
-   cp -r /path/to/HeraSpec/src/core/templates/skills/ui-ux-skill.md heraspec/skills/ui-ux/
-   cp -r /path/to/HeraSpec/src/core/templates/skills/scripts heraspec/skills/ui-ux/
-   cp -r /path/to/HeraSpec/src/core/templates/skills/data heraspec/skills/ui-ux/
-   cp -r /path/to/HeraSpec/src/core/templates/skills/templates heraspec/skills/ui-ux/
+   cp -r /path/to/HeraSpec/src/core/templates/skills/ui-ux heraspec/skills/
    \`\`\`
 
 2. Use prompt template from above
@@ -4471,32 +4575,45 @@ After copying UI/UX skill to your project, see:
   async updateReadmeFile(readmePath) {
     const existingContent = await FileSystemUtils.readFile(readmePath);
     const heraspecSection = this.getHeraSpecReadmeSection();
-    const sectionMarkers = [
-      "## HeraSpec",
-      "## HeraSpec Development",
-      "### HeraSpec",
-      "### HeraSpec Development",
-      "<!-- HeraSpec Section -->"
-    ];
-    let hasHeraSpecSection = false;
     let sectionStartIndex = -1;
     let sectionEndIndex = -1;
-    for (const marker of sectionMarkers) {
-      const index = existingContent.indexOf(marker);
-      if (index !== -1) {
-        hasHeraSpecSection = true;
-        sectionStartIndex = index;
-        sectionEndIndex = existingContent.indexOf("\n## ", index + marker.length);
-        if (sectionEndIndex === -1) {
-          sectionEndIndex = existingContent.indexOf("\n### ", index + marker.length);
-        }
-        if (sectionEndIndex === -1) {
-          sectionEndIndex = existingContent.length;
+    const commentMarker = "<!-- HeraSpec Section -->";
+    const commentIndex = existingContent.indexOf(commentMarker);
+    if (commentIndex !== -1) {
+      sectionStartIndex = commentIndex;
+      const firstHeaderIndex = existingContent.indexOf("## ", commentIndex + commentMarker.length);
+      if (firstHeaderIndex !== -1) {
+        let nextHeaderIndex = existingContent.indexOf("\n## ", firstHeaderIndex + 3);
+        if (nextHeaderIndex === -1) {
+          nextHeaderIndex = existingContent.indexOf("\n### ", firstHeaderIndex + 3);
+        }
+        sectionEndIndex = nextHeaderIndex !== -1 ? nextHeaderIndex : existingContent.length;
+      } else {
+        sectionEndIndex = existingContent.length;
+      }
+    } else {
+      const sectionMarkers = [
+        "## HeraSpec Development",
+        "## HeraSpec",
+        "### HeraSpec Development",
+        "### HeraSpec"
+      ];
+      for (const marker of sectionMarkers) {
+        const index = existingContent.indexOf(marker);
+        if (index !== -1) {
+          sectionStartIndex = index;
+          sectionEndIndex = existingContent.indexOf("\n## ", index + marker.length);
+          if (sectionEndIndex === -1) {
+            sectionEndIndex = existingContent.indexOf("\n### ", index + marker.length);
+          }
+          if (sectionEndIndex === -1) {
+            sectionEndIndex = existingContent.length;
+          }
+          break;
         }
-        break;
       }
     }
-    if (hasHeraSpecSection && sectionStartIndex !== -1) {
+    if (sectionStartIndex !== -1) {
       const before = existingContent.substring(0, sectionStartIndex).trimEnd();
       const after = existingContent.substring(sectionEndIndex);
       const updatedContent = before + "\n\n" + heraspecSection + (after.trimStart().startsWith("\n") ? "" : "\n\n") + after;
@@ -4876,7 +4993,6 @@ This will archive "${changeName}" and merge delta specs into source specs.`);
       const archiveDir = path10.join(
         ".",
         HERASPEC_DIR_NAME,
-        CHANGES_DIR_NAME,
         ARCHIVES_DIR_NAME
       );
       await FileSystemUtils.createDirectory(archiveDir);
@@ -5200,7 +5316,6 @@ var RestoreCommand = class {
     const archivesDir = path12.join(
       ".",
       HERASPEC_DIR_NAME,
-      CHANGES_DIR_NAME,
       ARCHIVES_DIR_NAME
     );
     if (!await FileSystemUtils.fileExists(archivesDir)) {