@featurevisor/core 2.10.0 → 2.12.0

package/src/linter/lintProject.ts

@@ -1,11 +1,14 @@
 // for use in node only
 import * as path from "path";
 
+import type { Schema } from "@featurevisor/types";
+
 import { getAttributeZodSchema } from "./attributeSchema";
 import { getConditionsZodSchema } from "./conditionSchema";
 import { getSegmentZodSchema } from "./segmentSchema";
 import { getGroupZodSchema } from "./groupSchema";
 import { getFeatureZodSchema } from "./featureSchema";
+import { getSchemaZodSchema } from "./schema";
 import { getTestsZodSchema } from "./testSchema";
 
 import { checkForCircularDependencyInRequired } from "./checkCircularDependency";
@@ -54,6 +57,8 @@ export async function lintProject(
       fullPath = path.join(projectConfig.featuresDirectoryPath, fileName);
     } else if (type === "group") {
       fullPath = path.join(projectConfig.groupsDirectoryPath, fileName);
+    } else if (type === "schema") {
+      fullPath = path.join(projectConfig.schemasDirectoryPath, fileName);
     } else if (type === "test") {
       fullPath = path.join(projectConfig.testsDirectoryPath, fileName);
     } else {
@@ -209,6 +214,17 @@ export async function lintProject(
     }
   }
 
+  // List schemas and load parsed schemas for feature variable validation (schema references)
+  const schemas = await datasource.listSchemas();
+  const schemasByKey: Record<string, Schema> = {};
+  for (const key of schemas) {
+    try {
+      schemasByKey[key] = await datasource.readSchema(key);
+    } catch {
+      // Schema file may be invalid; skip for feature variable resolution
+    }
+  }
+
   // lint features
   const features = await datasource.listFeatures();
   const featureZodSchema = getFeatureZodSchema(
@@ -217,6 +233,8 @@ export async function lintProject(
     flattenedAttributes as [string, ...string[]],
     segments as [string, ...string[]],
     features as [string, ...string[]],
+    schemas,
+    schemasByKey,
   );
 
   if (!options.entityType || options.entityType === "feature") {
@@ -374,6 +392,68 @@ export async function lintProject(
 
   // @TODO: feature cannot exist in multiple groups
 
+  // lint schemas (schemas and schemasByKey already loaded above for feature linting)
+  const schemaZodSchema = getSchemaZodSchema(schemas);
+
+  if (!options.entityType || options.entityType === "schema") {
+    const filteredKeys = !keyPattern ? schemas : schemas.filter((key) => keyPattern.test(key));
+
+    if (filteredKeys.length > 0) {
+      console.log(`Linting ${filteredKeys.length} schemas...\n`);
+    }
+
+    for (const key of filteredKeys) {
+      const fullPath = getFullPathFromKey("schema", key);
+
+      if (!ENTITY_NAME_REGEX.test(key)) {
+        console.log(CLI_FORMAT_BOLD_UNDERLINE, fullPath);
+
+        if (options.authors) {
+          const authors = await getAuthorsOfEntity(datasource, "schema", key);
+          console.log(`     Authors: ${authors.join(", ")}\n`);
+        }
+
+        console.log(CLI_FORMAT_RED, `  => Error: Invalid name: "${key}"`);
+        console.log(CLI_FORMAT_RED, `     ${ENTITY_NAME_REGEX_ERROR}`);
+        console.log("");
+        hasError = true;
+      }
+
+      try {
+        const parsed = await datasource.readSchema(key);
+
+        const result = schemaZodSchema.safeParse(parsed);
+
+        if (!result.success) {
+          console.log(CLI_FORMAT_BOLD_UNDERLINE, fullPath);
+
+          if (options.authors) {
+            const authors = await getAuthorsOfEntity(datasource, "schema", key);
+            console.log(`     Authors: ${authors.join(", ")}\n`);
+          }
+
+          if ("error" in result) {
+            printZodError(result.error);
+          }
+
+          hasError = true;
+        }
+      } catch (e) {
+        console.log(CLI_FORMAT_BOLD_UNDERLINE, fullPath);
+
+        if (options.authors) {
+          const authors = await getAuthorsOfEntity(datasource, "schema", key);
+          console.log(`     Authors: ${authors.join(", ")}\n`);
+        }
+
+        console.log("");
+        console.log(e);
+
+        hasError = true;
+      }
+    }
+  }
+
   // lint tests
   const tests = await datasource.listTests();
 
@@ -487,6 +567,10 @@ export const lintPlugin: Plugin = {
       command: "lint --entityType=group",
       description: "lint only groups",
     },
+    {
+      command: "lint --entityType=schema",
+      description: "lint only schemas",
+    },
     {
       command: "lint --entityType=test",
       description: "lint only tests",