fix: Update paths in build scripts and enhance error handling in validators

Author: Coderrob

.github/workflows/catalogs-build.yml

@@ -16,7 +16,7 @@ jobs:
           node-version: '20'
       - run: npm ci
       - run: npm run build
-      - run: node tools/ci-scripts/build-catalogs.js
+      - run: node scripts/ci-scripts/build-catalogs.js
       - uses: stefanzweifel/git-auto-commit-action@v5
         with:
           commit_message: 'Update catalogs'

.github/workflows/docs-validate.yml

@@ -14,5 +14,5 @@ jobs:
           node-version: '20'
       - run: npm ci
       - run: npm run build
-      - run: node tools/ci-scripts/build-catalogs.js
+      - run: node scripts/ci-scripts/build-catalogs.js
       - run: npm run cli ref audit

reference/standards/catalogs/registry.json

@@ -1,6 +1,6 @@
 {
   "tech:typescript/frameworks/[email protected]": {
-    "path": "tech\\typescript\\frameworks\\express@5\\guide.md",
+    "path": "reference\\tech\\typescript\\frameworks\\express@5\\guide.md",
     "status": "active",
     "sha": "placeholder",
     "aliases": [],

scripts/ci-scripts/build-catalogs.js

@@ -5,9 +5,9 @@ const path = require('path');
 const yaml = require('js-yaml');
 
 const ROOT = path.resolve(__dirname, '..', '..');
-const STANDARDS_DIR = path.join(ROOT, 'standards');
-const TECH_DIR = path.join(ROOT, 'tech');
-const TEMPLATES_DIR = path.join(ROOT, 'templates');
+const STANDARDS_DIR = path.join(ROOT, 'reference', 'standards');
+const TECH_DIR = path.join(ROOT, 'reference', 'tech');
+const TEMPLATES_DIR = path.join(ROOT, 'docs', 'templates');
 const REGISTRY_PATH = path.join(STANDARDS_DIR, 'catalogs', 'registry.json');
 const ALIASES_PATH = path.join(STANDARDS_DIR, 'catalogs', 'aliases.json');
 

src/validators/ajv.validator.ts

@@ -25,8 +25,14 @@ export class AjvValidator {
    * Compiles the validation schema for use in validation.
    */
   compile(): void {
-    const schema = this.loader.load();
-    this.validateFn = this.ajv.compile(schema as AnySchema);
+    try {
+      const schema = this.loader.load();
+      this.validateFn = this.ajv.compile(schema as AnySchema);
+    } catch (error) {
+      const schemaPath = this.loader.getSchemaPath();
+      const errorMessage = error instanceof Error ? error.message : String(error);
+      throw new Error(`Failed to compile validator for schema '${schemaPath}': ${errorMessage}`);
+    }
   }
 
   /**

src/validators/schema.loader.ts

@@ -30,4 +30,12 @@ export class SchemaLoader {
     }
     return parseJsonFile(this.schemaPath, fileManager);
   }
+
+  /**
+   * Gets the path to the schema file.
+   * @returns The schema file path.
+   */
+  getSchemaPath(): string {
+    return this.schemaPath;
+  }
 }

src/validators/validator.ts

@@ -29,9 +29,12 @@ export function validateTasks(tasks: ITask[]): IValidationResult {
   const validator = new AjvValidator(loader);
   const errors: string[] = [];
   for (let i = 0; i < tasks.length; i++) {
-    // Array access with controlled index is safe
-    // eslint-disable-next-line security/detect-object-injection
-    const res = validator.validate(tasks[i]);
+    const task = tasks.at(i);
+    if (!task) {
+      errors.push(`Task[${i}] is undefined`);
+      continue;
+    }
+    const res = validator.validate(task);
     if (!res.isValid) {
       const msg = (res.errors || [])
         .map((e: unknown) => {