@jay-framework/jay-stack-cli 0.18.1 → 0.18.3

package/dist/index.js

@@ -10,7 +10,7 @@ import YAML from "yaml";
 import { getLogger, setDevLogger, createDevLogger } from "@jay-framework/logger";
 import { parseJayFile, JAY_IMPORT_RESOLVER, generateElementDefinitionFile, ContractTagType, parseContract, generateElementFile, generateServerElementFile, htmlElementTagNameMap } from "@jay-framework/compiler-jay-html";
 import { JAY_CONTRACT_EXTENSION, JAY_EXTENSION, resolvePluginManifest, LOCAL_PLUGIN_PATH, JayAtomicType, JayEnumType, loadPluginManifest, RuntimeMode, GenerateTarget } from "@jay-framework/compiler-shared";
-import { listContracts, materializeContracts, scanPlugins as scanPlugins$1 } from "@jay-framework/stack-server-runtime";
+import { scanPlugins as scanPlugins$1, listContracts, materializeContracts } from "@jay-framework/stack-server-runtime";
 import { listContracts as listContracts2, materializeContracts as materializeContracts2 } from "@jay-framework/stack-server-runtime";
 import { Command } from "commander";
 import chalk from "chalk";
@@ -3358,6 +3358,46 @@ async function validateSchema(context, result) {
       });
     }
   }
+  if (manifest.validators) {
+    if (!Array.isArray(manifest.validators)) {
+      result.errors.push({
+        type: "schema",
+        message: 'Field "validators" must be an array',
+        location: "plugin.yaml"
+      });
+    } else {
+      manifest.validators.forEach((validator, index) => {
+        if (!validator.name) {
+          result.errors.push({
+            type: "schema",
+            message: `Validator at index ${index} is missing "name" field`,
+            location: "plugin.yaml"
+          });
+        }
+        if (!validator.handler) {
+          result.errors.push({
+            type: "schema",
+            message: `Validator "${validator.name || index}" is missing "handler" field`,
+            location: "plugin.yaml",
+            suggestion: "Specify the relative path to the validator handler module"
+          });
+        }
+        if (validator.handler && !context.isNpmPackage) {
+          const handlerPath = path.join(context.pluginPath, validator.handler);
+          const extensions = ["", ".ts", ".js", "/index.ts", "/index.js"];
+          const found = extensions.some((ext) => fs.existsSync(handlerPath + ext));
+          if (!found) {
+            result.errors.push({
+              type: "file-missing",
+              message: `Validator "${validator.name}" handler not found: ${validator.handler}`,
+              location: "plugin.yaml validators",
+              suggestion: `Create the validator handler at ${handlerPath}.ts`
+            });
+          }
+        }
+      });
+    }
+  }
 }
 function resolveContractFile(contractSpec, context) {
   if (context.isNpmPackage) {
@@ -4226,6 +4266,88 @@ function checkHeadlessInstanceProps(jayHtml, file) {
   walkElement(jayHtml.body);
   return warnings;
 }
+async function runPluginValidators(projectRoot, parsedFiles, errors, warnings) {
+  const scannedPlugins = await scanPlugins$1({ projectRoot });
+  for (const [, plugin] of scannedPlugins) {
+    if (!plugin.manifest.validators)
+      continue;
+    for (const validatorDef of plugin.manifest.validators) {
+      let validatorFn;
+      try {
+        const handlerPath = path.resolve(plugin.pluginPath, validatorDef.handler);
+        const handlerModule = await import(handlerPath);
+        validatorFn = handlerModule.validate;
+        if (typeof validatorFn !== "function") {
+          errors.push({
+            file: `plugin:${plugin.name}`,
+            message: `Validator "${validatorDef.name}" handler does not export a "validate" function`,
+            stage: "plugin"
+          });
+          continue;
+        }
+      } catch (loadErr) {
+        errors.push({
+          file: `plugin:${plugin.name}`,
+          message: `Failed to load validator "${validatorDef.name}": ${loadErr.message}`,
+          stage: "plugin"
+        });
+        continue;
+      }
+      const source = `${plugin.name}/${validatorDef.name}`;
+      for (const { relativePath, parsed } of parsedFiles) {
+        const ctx = {
+          filePath: relativePath,
+          body: parsed.body,
+          contract: parsed.contract ? {
+            name: parsed.contract.name,
+            tags: parsed.contract.tags,
+            props: parsed.contract.props,
+            params: parsed.contract.params
+          } : void 0,
+          headlessImports: parsed.headlessImports.map((imp) => ({
+            key: imp.key,
+            contractName: imp.contractName,
+            contract: imp.contract ? {
+              name: imp.contract.name,
+              tags: imp.contract.tags,
+              props: imp.contract.props,
+              params: imp.contract.params
+            } : void 0
+          })),
+          projectRoot
+        };
+        try {
+          const findings = await validatorFn(ctx);
+          for (const finding of findings) {
+            if (finding.severity === "error") {
+              errors.push({
+                file: relativePath,
+                message: finding.message,
+                stage: "plugin",
+                source,
+                suggestion: finding.suggestion
+              });
+            } else {
+              warnings.push({
+                file: relativePath,
+                message: finding.message,
+                source,
+                suggestion: finding.suggestion
+              });
+            }
+          }
+        } catch (runErr) {
+          errors.push({
+            file: relativePath,
+            message: `Validator "${source}" threw: ${runErr.message}`,
+            stage: "plugin",
+            source
+          });
+        }
+      }
+    }
+  }
+}
 async function validateJayFiles(options = {}) {
   const config = loadConfig();
   const resolvedConfig = getConfigWithDefaults(config);
@@ -4234,6 +4356,7 @@ async function validateJayFiles(options = {}) {
   const errors = [];
   const warnings = [];
   const coverage = [];
+  const parsedFiles = [];
   const jayHtmlFiles = await findJayFiles(scanDir);
   const contractFiles = await findContractFiles(scanDir);
   if (options.verbose) {
@@ -4299,6 +4422,7 @@ async function validateJayFiles(options = {}) {
         }
         continue;
       }
+      parsedFiles.push({ relativePath, parsed: parsedFile.val });
       const routeParamWarnings = checkRouteParams(parsedFile.val, jayFile, scanDir, content);
       for (const msg of routeParamWarnings) {
         warnings.push({ file: relativePath, message: msg });
@@ -4363,6 +4487,7 @@ async function validateJayFiles(options = {}) {
       }
     }
   }
+  await runPluginValidators(projectRoot, parsedFiles, errors, warnings);
   return {
     valid: errors.length === 0,
     jayHtmlFilesScanned: jayHtmlFiles.length,
@@ -4389,8 +4514,12 @@ function printJayValidationResult(result, options) {
     logger.important(chalk.red("❌ Jay Stack validation failed\n"));
     logger.important("Errors:");
     for (const error of result.errors) {
+      const prefix = error.source ? `[${error.source}] ` : "";
       logger.important(chalk.red(`  ❌ ${error.file}`));
-      logger.important(chalk.gray(`     ${error.message}`));
+      logger.important(chalk.gray(`     ${prefix}${error.message}`));
+      if (error.suggestion) {
+        logger.important(chalk.blue(`     Suggestion: ${error.suggestion}`));
+      }
       logger.important("");
     }
     const validFiles = result.jayHtmlFilesScanned + result.contractFilesScanned - result.errors.length;
@@ -4402,8 +4531,12 @@ function printJayValidationResult(result, options) {
     logger.important("");
     logger.important(chalk.yellow("Warnings:"));
     for (const warning of result.warnings) {
+      const prefix = warning.source ? `[${warning.source}] ` : "";
       logger.important(chalk.yellow(`  ⚠ ${warning.file}`));
-      logger.important(chalk.gray(`    ${warning.message}`));
+      logger.important(chalk.gray(`    ${prefix}${warning.message}`));
+      if (warning.suggestion) {
+        logger.important(chalk.blue(`    Suggestion: ${warning.suggestion}`));
+      }
       logger.important("");
     }
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jay-framework/jay-stack-cli",
-  "version": "0.18.1",
+  "version": "0.18.3",
   "license": "Apache-2.0",
   "type": "module",
   "main": "dist/index.js",
@@ -24,15 +24,15 @@
     "test:watch": "vitest"
   },
   "dependencies": {
-    "@jay-framework/compiler-jay-html": "^0.18.1",
-    "@jay-framework/compiler-shared": "^0.18.1",
-    "@jay-framework/dev-server": "^0.18.1",
-    "@jay-framework/editor-server": "^0.18.1",
-    "@jay-framework/fullstack-component": "^0.18.1",
-    "@jay-framework/logger": "^0.18.1",
-    "@jay-framework/plugin-validator": "^0.18.1",
-    "@jay-framework/production-server": "^0.18.1",
-    "@jay-framework/stack-server-runtime": "^0.18.1",
+    "@jay-framework/compiler-jay-html": "^0.18.3",
+    "@jay-framework/compiler-shared": "^0.18.3",
+    "@jay-framework/dev-server": "^0.18.3",
+    "@jay-framework/editor-server": "^0.18.3",
+    "@jay-framework/fullstack-component": "^0.18.3",
+    "@jay-framework/logger": "^0.18.3",
+    "@jay-framework/plugin-validator": "^0.18.3",
+    "@jay-framework/production-server": "^0.18.3",
+    "@jay-framework/stack-server-runtime": "^0.18.3",
     "chalk": "^4.1.2",
     "commander": "^14.0.0",
     "express": "^5.0.1",
@@ -43,7 +43,7 @@
     "yaml": "^2.3.4"
   },
   "devDependencies": {
-    "@jay-framework/dev-environment": "^0.18.1",
+    "@jay-framework/dev-environment": "^0.18.3",
     "@types/express": "^5.0.2",
     "@types/node": "^22.15.21",
     "nodemon": "^3.0.3",