fix: handle CRLF line endings and Windows path separators in skill/plugin loading

- Fix YAML frontmatter regex in skillParser, skillManager, subagentParser
  to accept \r\n (CRLF) line endings. On Windows, git checkout with
  core.autocrlf=true converts LF→CRLF, causing all SKILL.md frontmatter
  parsing to fail silently — plugin skills were skipped and never
  registered as slash commands.
- Fix skillParser skill type detection to recognize Windows backslash
  paths (\.wave\skills) in addition to forward slash paths.
- Fix fileWatcher path comparison to normalize backslashes before
  matching, so file watch events are correctly dispatched on Windows.
- Fix commandPathResolver to split on both / and \ when generating
  command IDs from file paths.

Author: lewis617

packages/agent-sdk/src/managers/skillManager.ts

@@ -442,7 +442,9 @@ export class SkillManager extends EventEmitter {
     const skillPath = `Base directory for this skill: ${skill.skillPath}\n\n`;
 
     // Extract content after frontmatter
-    const contentMatch = skill.content.match(/^---\n[\s\S]*?\n---\n([\s\S]*)$/);
+    const contentMatch = skill.content.match(
+      /^---\r?\n[\s\S]*?\r?\n---\r?\n([\s\S]*)$/,
+    );
     let mainContent = contentMatch ? contentMatch[1].trim() : skill.content;
 
     // 1. Substitute parameters ($1, $ARGUMENTS, etc.)

packages/agent-sdk/src/services/fileWatcher.ts

@@ -281,12 +281,11 @@ export class FileWatcherService extends EventEmitter {
 
     // Notify all watchers that match the path or are parents of the path
     for (const [watchedPath, entry] of this.watchers.entries()) {
+      const normalizedFilePath = filePath.replace(/\\/g, "/");
+      const normalizedWatchedPath = watchedPath.replace(/\\/g, "/");
       if (
-        filePath === watchedPath ||
-        filePath.startsWith(watchedPath + "/") ||
-        // Handle cases where the watched path might be a file and we get an event for it
-        // (already covered by filePath === watchedPath)
-        false
+        normalizedFilePath === normalizedWatchedPath ||
+        normalizedFilePath.startsWith(normalizedWatchedPath + "/")
       ) {
         entry.lastEvent = event.timestamp;
 

packages/agent-sdk/src/utils/commandPathResolver.ts

@@ -26,7 +26,9 @@ export function generateCommandId(filePath: string, rootDir: string): string {
     throw new Error("Command filename cannot be empty");
   }
 
-  const segments = relativePath.split("/").filter((segment) => segment !== "");
+  const segments = relativePath
+    .split(/[\\/]/)
+    .filter((segment) => segment !== "");
 
   // Remove .md extension from the last segment
   const lastSegment = segments[segments.length - 1];

packages/agent-sdk/src/utils/skillParser.ts

@@ -35,7 +35,7 @@ export function parseSkillFile(
     result.content = content;
 
     // Parse YAML frontmatter
-    const frontmatterMatch = content.match(/^---\n([\s\S]*?)\n---/);
+    const frontmatterMatch = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
     if (!frontmatterMatch) {
       result.validationErrors.push("Missing YAML frontmatter");
       return result;
@@ -53,9 +53,11 @@ export function parseSkillFile(
 
     // Determine skill type and path
     const skillPath = basePath || dirname(filePath);
-    const skillType = skillPath.includes("/.wave/skills")
-      ? "project"
-      : "personal";
+    const skillType =
+      skillPath.includes("/.wave/skills") ||
+      skillPath.includes("\\.wave\\skills")
+        ? "project"
+        : "personal";
 
     // Extract allowed tools
     let allowedTools: string[] | undefined;

packages/agent-sdk/src/utils/subagentParser.ts

@@ -30,7 +30,8 @@ function parseFrontmatter(content: string): {
   frontmatter: SubagentFrontmatter;
   body: string;
 } {
-  const frontmatterRegex = /^---\s*\n([\s\S]*?)\n---\s*\n([\s\S]*)$/;
+  const frontmatterRegex =
+    /^---[ \t]*\r?\n([\s\S]*?)\r?\n---[ \t]*\r?\n([\s\S]*)$/;
   const match = content.match(frontmatterRegex);
 
   if (!match) {