rulesync 0.16.0 → 0.17.0

package/dist/index.js

@@ -368,14 +368,20 @@ var import_gray_matter = __toESM(require("gray-matter"));
 async function parseRulesFromDirectory(aiRulesDir) {
   const ruleFiles = await findFiles(aiRulesDir);
   const rules = [];
+  const errors = [];
   for (const filepath of ruleFiles) {
     try {
       const rule = await parseRuleFile(filepath);
       rules.push(rule);
     } catch (error) {
-      console.warn(`Failed to parse rule file ${filepath}:`, error);
+      const errorMessage = error instanceof Error ? error.message : String(error);
+      errors.push(`Failed to parse rule file ${filepath}: ${errorMessage}`);
     }
   }
+  if (errors.length > 0) {
+    throw new Error(`Validation errors found:
+${errors.join("\n")}`);
+  }
   return rules;
 }
 async function parseRuleFile(filepath) {
@@ -393,14 +399,36 @@ async function parseRuleFile(filepath) {
 }
 function validateFrontmatter(data, filepath) {
   if (!data || typeof data !== "object") {
-    throw new Error(`Invalid frontmatter in ${filepath}: must be an object`);
+    if (!data) {
+      throw new Error(
+        `Missing frontmatter in ${filepath}: file must contain YAML frontmatter with required fields (root, targets, description, globs)`
+      );
+    }
+    throw new Error(`Invalid frontmatter in ${filepath}: frontmatter must be a valid YAML object`);
   }
   const obj = data;
+  if (Object.keys(obj).length === 0) {
+    throw new Error(
+      `Missing frontmatter in ${filepath}: file must contain YAML frontmatter with required fields (root, targets, description, globs)`
+    );
+  }
+  if (obj.root === void 0) {
+    throw new Error(`Missing required field "root" in ${filepath}: must be true or false`);
+  }
   if (typeof obj.root !== "boolean") {
-    throw new Error(`Invalid root in ${filepath}: must be a boolean`);
+    throw new Error(
+      `Invalid "root" field in ${filepath}: must be a boolean (true or false), got ${typeof obj.root}`
+    );
+  }
+  if (obj.targets === void 0) {
+    throw new Error(
+      `Missing required field "targets" in ${filepath}: must be an array like ["*"] or ["copilot", "cursor"]`
+    );
   }
   if (!Array.isArray(obj.targets)) {
-    throw new Error(`Invalid targets in ${filepath}: must be an array`);
+    throw new Error(
+      `Invalid "targets" field in ${filepath}: must be an array, got ${typeof obj.targets}`
+    );
   }
   const validTargets = ["copilot", "cursor", "cline", "claude", "roo", "*"];
   for (const target of obj.targets) {
@@ -410,15 +438,31 @@ function validateFrontmatter(data, filepath) {
       );
     }
   }
+  if (obj.description === void 0) {
+    throw new Error(
+      `Missing required field "description" in ${filepath}: must be a descriptive string`
+    );
+  }
   if (!obj.description || typeof obj.description !== "string") {
-    throw new Error(`Invalid description in ${filepath}: must be a non-empty string`);
+    throw new Error(
+      `Invalid "description" field in ${filepath}: must be a non-empty string, got ${typeof obj.description}`
+    );
+  }
+  if (obj.globs === void 0) {
+    throw new Error(
+      `Missing required field "globs" in ${filepath}: must be an array of file patterns like ["**/*.ts"]`
+    );
   }
   if (!Array.isArray(obj.globs)) {
-    throw new Error(`Invalid globs in ${filepath}: must be an array`);
+    throw new Error(
+      `Invalid "globs" field in ${filepath}: must be an array, got ${typeof obj.globs}`
+    );
   }
   for (const glob of obj.globs) {
     if (typeof glob !== "string") {
-      throw new Error(`Invalid glob in ${filepath}: all globs must be strings`);
+      throw new Error(
+        `Invalid glob pattern in ${filepath}: all globs must be strings, got ${typeof glob}`
+      );
     }
   }
 }

package/dist/index.mjs

@@ -345,14 +345,20 @@ import matter from "gray-matter";
 async function parseRulesFromDirectory(aiRulesDir) {
   const ruleFiles = await findFiles(aiRulesDir);
   const rules = [];
+  const errors = [];
   for (const filepath of ruleFiles) {
     try {
       const rule = await parseRuleFile(filepath);
       rules.push(rule);
     } catch (error) {
-      console.warn(`Failed to parse rule file ${filepath}:`, error);
+      const errorMessage = error instanceof Error ? error.message : String(error);
+      errors.push(`Failed to parse rule file ${filepath}: ${errorMessage}`);
     }
   }
+  if (errors.length > 0) {
+    throw new Error(`Validation errors found:
+${errors.join("\n")}`);
+  }
   return rules;
 }
 async function parseRuleFile(filepath) {
@@ -370,14 +376,36 @@ async function parseRuleFile(filepath) {
 }
 function validateFrontmatter(data, filepath) {
   if (!data || typeof data !== "object") {
-    throw new Error(`Invalid frontmatter in ${filepath}: must be an object`);
+    if (!data) {
+      throw new Error(
+        `Missing frontmatter in ${filepath}: file must contain YAML frontmatter with required fields (root, targets, description, globs)`
+      );
+    }
+    throw new Error(`Invalid frontmatter in ${filepath}: frontmatter must be a valid YAML object`);
   }
   const obj = data;
+  if (Object.keys(obj).length === 0) {
+    throw new Error(
+      `Missing frontmatter in ${filepath}: file must contain YAML frontmatter with required fields (root, targets, description, globs)`
+    );
+  }
+  if (obj.root === void 0) {
+    throw new Error(`Missing required field "root" in ${filepath}: must be true or false`);
+  }
   if (typeof obj.root !== "boolean") {
-    throw new Error(`Invalid root in ${filepath}: must be a boolean`);
+    throw new Error(
+      `Invalid "root" field in ${filepath}: must be a boolean (true or false), got ${typeof obj.root}`
+    );
+  }
+  if (obj.targets === void 0) {
+    throw new Error(
+      `Missing required field "targets" in ${filepath}: must be an array like ["*"] or ["copilot", "cursor"]`
+    );
   }
   if (!Array.isArray(obj.targets)) {
-    throw new Error(`Invalid targets in ${filepath}: must be an array`);
+    throw new Error(
+      `Invalid "targets" field in ${filepath}: must be an array, got ${typeof obj.targets}`
+    );
   }
   const validTargets = ["copilot", "cursor", "cline", "claude", "roo", "*"];
   for (const target of obj.targets) {
@@ -387,15 +415,31 @@ function validateFrontmatter(data, filepath) {
       );
     }
   }
+  if (obj.description === void 0) {
+    throw new Error(
+      `Missing required field "description" in ${filepath}: must be a descriptive string`
+    );
+  }
   if (!obj.description || typeof obj.description !== "string") {
-    throw new Error(`Invalid description in ${filepath}: must be a non-empty string`);
+    throw new Error(
+      `Invalid "description" field in ${filepath}: must be a non-empty string, got ${typeof obj.description}`
+    );
+  }
+  if (obj.globs === void 0) {
+    throw new Error(
+      `Missing required field "globs" in ${filepath}: must be an array of file patterns like ["**/*.ts"]`
+    );
   }
   if (!Array.isArray(obj.globs)) {
-    throw new Error(`Invalid globs in ${filepath}: must be an array`);
+    throw new Error(
+      `Invalid "globs" field in ${filepath}: must be an array, got ${typeof obj.globs}`
+    );
   }
   for (const glob of obj.globs) {
     if (typeof glob !== "string") {
-      throw new Error(`Invalid glob in ${filepath}: all globs must be strings`);
+      throw new Error(
+        `Invalid glob pattern in ${filepath}: all globs must be strings, got ${typeof glob}`
+      );
     }
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rulesync",
-  "version": "0.16.0",
+  "version": "0.17.0",
   "description": "Unified AI rules management CLI tool that generates configuration files for various AI development tools",
   "main": "dist/index.js",
   "module": "dist/index.mjs",