rulesync 0.16.0 → 0.18.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}`
+      );
     }
   }
 }
@@ -622,15 +666,15 @@ async function initCommand() {
 async function createSampleFiles(aiRulesDir) {
   const sampleFiles = [
     {
-      filename: "coding-rules.md",
+      filename: "overview.md",
       content: `---
 root: true
 targets: ["*"]
-description: "General coding standards and best practices"
+description: "Project overview and general development guidelines"
 globs: ["**/*.ts", "**/*.js", "**/*.tsx", "**/*.jsx"]
 ---
 
-# Coding Rules
+# Project Overview
 
 ## General Guidelines
 
@@ -646,54 +690,94 @@ globs: ["**/*.ts", "**/*.js", "**/*.tsx", "**/*.jsx"]
 - Use semicolons
 - Use double quotes for strings
 - Use trailing commas in multi-line objects and arrays
+
+## Architecture Principles
+
+- Organize code by feature, not by file type
+- Keep related files close together
+- Use dependency injection for better testability
+- Implement proper error handling
+- Follow single responsibility principle
 `
     },
     {
-      filename: "naming-conventions.md",
+      filename: "frontend.md",
       content: `---
 root: false
 targets: ["*"]
-description: "Naming conventions for variables, functions, and files"
-globs: ["**/*.ts", "**/*.js"]
+description: "Frontend development rules and best practices"
+globs: ["src/components/**/*.tsx", "src/pages/**/*.tsx", "**/*.css", "**/*.scss"]
 ---
 
-# Naming Conventions
+# Frontend Development Rules
+
+## React Components
 
-## Variables and Functions
-- Use camelCase for variables and functions
-- Use descriptive names that explain the purpose
-- Avoid abbreviations unless they are well-known
+- Use functional components with hooks
+- Follow PascalCase naming for components
+- Use TypeScript interfaces for props
+- Implement proper error boundaries
 
-## Files and Directories
-- Use kebab-case for file names
-- Use PascalCase for component files
-- Use lowercase for directory names
+## Styling
 
-## Constants
-- Use SCREAMING_SNAKE_CASE for constants
-- Group related constants in enums or const objects
+- Use CSS modules or styled-components
+- Follow BEM methodology for CSS classes
+- Prefer flexbox and grid for layouts
+- Use semantic HTML elements
+
+## State Management
+
+- Use React hooks for local state
+- Consider Redux or Zustand for global state
+- Avoid prop drilling with context API
+- Keep state as close to where it's used as possible
+
+## Performance
+
+- Use React.memo for expensive components
+- Implement lazy loading for routes
+- Optimize images and assets
+- Use proper key props in lists
 `
     },
     {
-      filename: "architecture.md",
+      filename: "backend.md",
       content: `---
 root: false
-targets: ["copilot", "cursor"]
-description: "Architectural patterns and project structure guidelines"
-globs: ["src/**/*.ts"]
+targets: ["*"]
+description: "Backend development rules and API guidelines"
+globs: ["src/api/**/*.ts", "src/services/**/*.ts", "src/models/**/*.ts"]
 ---
 
-# Architecture Guidelines
+# Backend Development Rules
 
-## Project Structure
-- Organize code by feature, not by file type
-- Keep related files close together
-- Use index files for clean imports
+## API Design
 
-## Design Patterns
-- Use dependency injection for better testability
-- Implement proper error handling
-- Follow single responsibility principle
+- Follow RESTful conventions
+- Use consistent HTTP status codes
+- Implement proper error handling with meaningful messages
+- Use API versioning when necessary
+
+## Database
+
+- Use proper indexing for performance
+- Implement database migrations
+- Follow naming conventions for tables and columns
+- Use transactions for data consistency
+
+## Security
+
+- Validate all input data
+- Use proper authentication and authorization
+- Implement rate limiting
+- Sanitize database queries to prevent SQL injection
+
+## Code Organization
+
+- Use service layer pattern
+- Implement proper logging
+- Use environment variables for configuration
+- Write comprehensive tests for business logic
 `
     }
   ];

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}`
+      );
     }
   }
 }
@@ -599,15 +643,15 @@ async function initCommand() {
 async function createSampleFiles(aiRulesDir) {
   const sampleFiles = [
     {
-      filename: "coding-rules.md",
+      filename: "overview.md",
       content: `---
 root: true
 targets: ["*"]
-description: "General coding standards and best practices"
+description: "Project overview and general development guidelines"
 globs: ["**/*.ts", "**/*.js", "**/*.tsx", "**/*.jsx"]
 ---
 
-# Coding Rules
+# Project Overview
 
 ## General Guidelines
 
@@ -623,54 +667,94 @@ globs: ["**/*.ts", "**/*.js", "**/*.tsx", "**/*.jsx"]
 - Use semicolons
 - Use double quotes for strings
 - Use trailing commas in multi-line objects and arrays
+
+## Architecture Principles
+
+- Organize code by feature, not by file type
+- Keep related files close together
+- Use dependency injection for better testability
+- Implement proper error handling
+- Follow single responsibility principle
 `
     },
     {
-      filename: "naming-conventions.md",
+      filename: "frontend.md",
       content: `---
 root: false
 targets: ["*"]
-description: "Naming conventions for variables, functions, and files"
-globs: ["**/*.ts", "**/*.js"]
+description: "Frontend development rules and best practices"
+globs: ["src/components/**/*.tsx", "src/pages/**/*.tsx", "**/*.css", "**/*.scss"]
 ---
 
-# Naming Conventions
+# Frontend Development Rules
+
+## React Components
 
-## Variables and Functions
-- Use camelCase for variables and functions
-- Use descriptive names that explain the purpose
-- Avoid abbreviations unless they are well-known
+- Use functional components with hooks
+- Follow PascalCase naming for components
+- Use TypeScript interfaces for props
+- Implement proper error boundaries
 
-## Files and Directories
-- Use kebab-case for file names
-- Use PascalCase for component files
-- Use lowercase for directory names
+## Styling
 
-## Constants
-- Use SCREAMING_SNAKE_CASE for constants
-- Group related constants in enums or const objects
+- Use CSS modules or styled-components
+- Follow BEM methodology for CSS classes
+- Prefer flexbox and grid for layouts
+- Use semantic HTML elements
+
+## State Management
+
+- Use React hooks for local state
+- Consider Redux or Zustand for global state
+- Avoid prop drilling with context API
+- Keep state as close to where it's used as possible
+
+## Performance
+
+- Use React.memo for expensive components
+- Implement lazy loading for routes
+- Optimize images and assets
+- Use proper key props in lists
 `
     },
     {
-      filename: "architecture.md",
+      filename: "backend.md",
       content: `---
 root: false
-targets: ["copilot", "cursor"]
-description: "Architectural patterns and project structure guidelines"
-globs: ["src/**/*.ts"]
+targets: ["*"]
+description: "Backend development rules and API guidelines"
+globs: ["src/api/**/*.ts", "src/services/**/*.ts", "src/models/**/*.ts"]
 ---
 
-# Architecture Guidelines
+# Backend Development Rules
 
-## Project Structure
-- Organize code by feature, not by file type
-- Keep related files close together
-- Use index files for clean imports
+## API Design
 
-## Design Patterns
-- Use dependency injection for better testability
-- Implement proper error handling
-- Follow single responsibility principle
+- Follow RESTful conventions
+- Use consistent HTTP status codes
+- Implement proper error handling with meaningful messages
+- Use API versioning when necessary
+
+## Database
+
+- Use proper indexing for performance
+- Implement database migrations
+- Follow naming conventions for tables and columns
+- Use transactions for data consistency
+
+## Security
+
+- Validate all input data
+- Use proper authentication and authorization
+- Implement rate limiting
+- Sanitize database queries to prevent SQL injection
+
+## Code Organization
+
+- Use service layer pattern
+- Implement proper logging
+- Use environment variables for configuration
+- Write comprehensive tests for business logic
 `
     }
   ];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rulesync",
-  "version": "0.16.0",
+  "version": "0.18.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",