@jay-framework/plugin-validator 0.23.0 → 0.23.1

package/dist/index.js

@@ -93,17 +93,13 @@ function collectLocalInterfaces(sourceFile) {
 function collectContractImportedTypes(sourceFile) {
   const contractTypes = /* @__PURE__ */ new Set();
   for (const statement of sourceFile.statements) {
-    if (!ts.isImportDeclaration(statement))
-      continue;
+    if (!ts.isImportDeclaration(statement)) continue;
     const moduleSpecifier = statement.moduleSpecifier;
-    if (!ts.isStringLiteral(moduleSpecifier))
-      continue;
+    if (!ts.isStringLiteral(moduleSpecifier)) continue;
     const modulePath = moduleSpecifier.text;
-    if (!modulePath.includes(".jay-contract"))
-      continue;
+    if (!modulePath.includes(".jay-contract")) continue;
     const importClause = statement.importClause;
-    if (!importClause)
-      continue;
+    if (!importClause) continue;
     const namedBindings = importClause.namedBindings;
     if (namedBindings && ts.isNamedImports(namedBindings)) {
       for (const element of namedBindings.elements) {
@@ -139,8 +135,7 @@ function visitNode(node, result) {
       }
       if (node.arguments.length > 0) {
         const arg = node.arguments[0];
-        if (ts.isIdentifier(arg))
-          ;
+        if (ts.isIdentifier(arg)) ;
       }
     }
   }
@@ -162,10 +157,8 @@ function extractTypeNames(typeNode) {
   return [];
 }
 function checkPropsConsistency(propsTypeName, localInterfaces, contractImportedTypes, contract, contractName, contractPath, sourcePath, errors, warnings) {
-  if (FRAMEWORK_PROP_TYPES.has(propsTypeName))
-    return;
-  if (contractImportedTypes.has(propsTypeName))
-    return;
+  if (FRAMEWORK_PROP_TYPES.has(propsTypeName)) return;
+  if (contractImportedTypes.has(propsTypeName)) return;
   const prefix = `[${contractName}]`;
   const iface = localInterfaces.get(propsTypeName);
   if (!iface) {
@@ -180,8 +173,7 @@ function checkPropsConsistency(propsTypeName, localInterfaces, contractImportedT
     return;
   }
   const ownProperties = iface.properties;
-  if (ownProperties.length === 0)
-    return;
+  if (ownProperties.length === 0) return;
   if (!contract.props || contract.props.length === 0) {
     errors.push({
       type: "contract-invalid",
@@ -225,10 +217,8 @@ function checkParamsConsistency(paramsTypeNames, localInterfaces, contractImport
       suggestion: `Add a params section to the contract (e.g., params: { slug: string })`
     });
     for (const typeName of paramsTypeNames) {
-      if (FRAMEWORK_PROP_TYPES.has(typeName))
-        continue;
-      if (contractImportedTypes.has(typeName))
-        continue;
+      if (FRAMEWORK_PROP_TYPES.has(typeName)) continue;
+      if (contractImportedTypes.has(typeName)) continue;
       const iface = localInterfaces.get(typeName);
       if (iface) {
         const ownProps = iface.properties;
@@ -241,13 +231,10 @@ function checkParamsConsistency(paramsTypeNames, localInterfaces, contractImport
     return;
   }
   for (const typeName of paramsTypeNames) {
-    if (FRAMEWORK_PROP_TYPES.has(typeName))
-      continue;
-    if (contractImportedTypes.has(typeName))
-      continue;
+    if (FRAMEWORK_PROP_TYPES.has(typeName)) continue;
+    if (contractImportedTypes.has(typeName)) continue;
     const iface = localInterfaces.get(typeName);
-    if (!iface)
-      continue;
+    if (!iface) continue;
     const ownProperties = iface.properties;
     const contractParamNames = new Set(contract.params.map((p) => p.name));
     for (const prop of ownProperties) {
@@ -339,16 +326,13 @@ function requiredString(obj, field, itemPath, errors, code) {
 }
 function optionalString(obj, field) {
   const value = obj[field];
-  if (value === void 0)
-    return void 0;
-  if (typeof value !== "string")
-    return void 0;
+  if (value === void 0) return void 0;
+  if (typeof value !== "string") return void 0;
   const trimmed = value.trim();
   return trimmed.length > 0 ? trimmed : void 0;
 }
 function validateInteraction(raw, itemPath, errors) {
-  if (raw === void 0)
-    return void 0;
+  if (raw === void 0) return void 0;
   if (!isRecord$1(raw)) {
     errors.push({
       path: itemPath,
@@ -372,8 +356,7 @@ function validateInteraction(raw, itemPath, errors) {
   };
 }
 function validatePresentation(raw, itemPath, errors) {
-  if (raw === void 0)
-    return void 0;
+  if (raw === void 0) return void 0;
   if (!isRecord$1(raw)) {
     errors.push({
       path: itemPath,
@@ -393,14 +376,12 @@ function validatePresentation(raw, itemPath, errors) {
   }
   if (type === "image") {
     const src = requiredString(raw, "src", itemPath, errors, "presentation-missing-src");
-    if (!src)
-      return void 0;
+    if (!src) return void 0;
     return { type: "image", src };
   }
   if (type === "gif") {
     const src = requiredString(raw, "src", itemPath, errors, "presentation-missing-src");
-    if (!src)
-      return void 0;
+    if (!src) return void 0;
     return {
       type: "gif",
       src,
@@ -445,8 +426,7 @@ function validatePresentation(raw, itemPath, errors) {
 }
 const BROWSE_SIZES = /* @__PURE__ */ new Set(["large", "medium", "small"]);
 function validateBrowse(raw, itemPath, errors) {
-  if (raw === void 0)
-    return void 0;
+  if (raw === void 0) return void 0;
   if (!isRecord$1(raw)) {
     errors.push({
       path: itemPath,
@@ -470,8 +450,7 @@ function validateBrowse(raw, itemPath, errors) {
   return { size: sizeRaw };
 }
 function validateFolderPath(raw, itemPath, errors) {
-  if (raw === void 0)
-    return void 0;
+  if (raw === void 0) return void 0;
   if (!Array.isArray(raw)) {
     errors.push({
       path: `${itemPath}.folderPath`,
@@ -609,8 +588,7 @@ function validateAddMenuCatalogFile(raw, sourcePath) {
   raw.items.forEach((entry, index) => {
     const result = validateAddMenuItem(entry, `${sourcePath}.items[${index}]`);
     errors.push(...result.errors);
-    if (result.item)
-      items.push(result.item);
+    if (result.item) items.push(result.item);
   });
   if (items.length === 0 && errors.length > 0) {
     return { file: null, errors };
@@ -618,11 +596,9 @@ function validateAddMenuCatalogFile(raw, sourcePath) {
   return { file: { items }, errors };
 }
 function normalizeAddMenuPresentation(item) {
-  if (item.presentation)
-    return item.presentation;
+  if (item.presentation) return item.presentation;
   const thumbnail = item.thumbnail?.trim();
-  if (!thumbnail)
-    return void 0;
+  if (!thumbnail) return void 0;
   if (/\.gif$/i.test(thumbnail)) {
     return { type: "gif", src: thumbnail };
   }
@@ -633,15 +609,12 @@ function normalizeAddMenuBrowseSize(item) {
 }
 function hasSingleRootDiv(html) {
   const trimmed = html.trim();
-  if (!trimmed.startsWith("<div"))
-    return false;
+  if (!trimmed.startsWith("<div")) return false;
   const openMatch = trimmed.match(/^<div\b[^>]*>/i);
-  if (!openMatch)
-    return false;
+  if (!openMatch) return false;
   const afterOpen = trimmed.slice(openMatch[0].length);
   const closeIdx = afterOpen.lastIndexOf("</div>");
-  if (closeIdx < 0)
-    return false;
+  if (closeIdx < 0) return false;
   const tail = afterOpen.slice(closeIdx + "</div>".length).trim();
   return tail.length === 0;
 }
@@ -717,8 +690,7 @@ function lintHtmlFragment(item, sourcePath) {
 }
 function lintGifPoster(item, sourcePath) {
   const presentation = normalizeAddMenuPresentation(item);
-  if (presentation?.type !== "gif" || presentation.poster?.trim())
-    return [];
+  if (presentation?.type !== "gif" || presentation.poster?.trim()) return [];
   return [
     catalogWarning(
       "gif-missing-poster",
@@ -729,10 +701,8 @@ function lintGifPoster(item, sourcePath) {
   ];
 }
 function lintBrowseLargeWithoutPresentation(item, sourcePath) {
-  if (normalizeAddMenuBrowseSize(item) !== "large")
-    return [];
-  if (normalizeAddMenuPresentation(item))
-    return [];
+  if (normalizeAddMenuBrowseSize(item) !== "large") return [];
+  if (normalizeAddMenuPresentation(item)) return [];
   return [
     catalogWarning(
       "browse-large-without-presentation",
@@ -805,8 +775,7 @@ function resolveModulePath$1(basePath) {
   return void 0;
 }
 function collectTypeScriptFiles(dir, depth = 0) {
-  if (depth > 4)
-    return [];
+  if (depth > 4) return [];
   const entries = fs.readdirSync(dir, { withFileTypes: true });
   const files = [];
   for (const entry of entries) {
@@ -825,8 +794,7 @@ function resolveHandlerSourceFile(pluginPath, handlerRef, isNpmPackage) {
   }
   const searchRoots = isNpmPackage ? [path.join(pluginPath, "lib"), path.join(pluginPath, "dist")] : [pluginPath];
   for (const root of searchRoots) {
-    if (!fs.existsSync(root))
-      continue;
+    if (!fs.existsSync(root)) continue;
     for (const file of collectTypeScriptFiles(root)) {
       const content = fs.readFileSync(file, "utf-8");
       const definesHandler = new RegExp(
@@ -846,8 +814,7 @@ function resolveHandlerSourceFile(pluginPath, handlerRef, isNpmPackage) {
       if (reExportMatch) {
         const importSpec = reExportMatch[1].replace(/\.js$/, "");
         const resolved = resolveModulePath$1(path.resolve(path.dirname(file), importSpec));
-        if (resolved)
-          return resolved;
+        if (resolved) return resolved;
       }
     }
   }
@@ -857,8 +824,7 @@ function extractBalancedBlock(source, openBraceIndex) {
   let depth = 0;
   for (let index = openBraceIndex; index < source.length; index++) {
     const char = source[index];
-    if (char === "{")
-      depth++;
+    if (char === "{") depth++;
     else if (char === "}") {
       depth--;
       if (depth === 0) {
@@ -878,8 +844,7 @@ function findFunctionBodyOpenBrace(source, searchFrom) {
   if (parenMatch?.index !== void 0) {
     candidates.push(searchFrom + parenMatch.index + parenMatch[0].length - 1);
   }
-  if (candidates.length === 0)
-    return -1;
+  if (candidates.length === 0) return -1;
   return Math.min(...candidates);
 }
 function extractFunctionBody(source, functionName) {
@@ -896,11 +861,9 @@ function extractFunctionBody(source, functionName) {
   ];
   for (const pattern of patterns) {
     const match = pattern.exec(source);
-    if (!match)
-      continue;
+    if (!match) continue;
     const braceIndex = findFunctionBodyOpenBrace(source, match.index);
-    if (braceIndex === -1)
-      continue;
+    if (braceIndex === -1) continue;
     return extractBalancedBlock(source, braceIndex);
   }
   return null;
@@ -913,11 +876,9 @@ function extractDefaultExportFunctionBody(source) {
   ];
   for (const pattern of patterns) {
     const match = pattern.exec(source);
-    if (!match)
-      continue;
+    if (!match) continue;
     const braceIndex = findFunctionBodyOpenBrace(source, match.index);
-    if (braceIndex === -1)
-      continue;
+    if (braceIndex === -1) continue;
     return extractBalancedBlock(source, braceIndex);
   }
   return null;
@@ -927,8 +888,7 @@ function handlerBodyWritesAddMenuCatalog(body) {
 }
 function resolveSetupHandlerFunctionBody(pluginPath, handlerRef, isNpmPackage) {
   const sourceFile = resolveHandlerSourceFile(pluginPath, handlerRef, isNpmPackage);
-  if (!sourceFile)
-    return null;
+  if (!sourceFile) return null;
   const source = fs.readFileSync(sourceFile, "utf-8");
   if (isRelativeHandlerRef(handlerRef)) {
     return extractDefaultExportFunctionBody(source) ?? extractFunctionBody(source, "setup") ?? extractFunctionBody(source, handlerRef);
@@ -936,8 +896,7 @@ function resolveSetupHandlerFunctionBody(pluginPath, handlerRef, isNpmPackage) {
   return extractFunctionBody(source, handlerRef);
 }
 function suggestionForCode(code) {
-  if (!code)
-    return `See ${CONTRIBUTOR_GUIDE} for schema and validation rules`;
+  if (!code) return `See ${CONTRIBUTOR_GUIDE} for schema and validation rules`;
   return ADD_MENU_VALIDATION_SUGGESTIONS[code] ?? `See ${CONTRIBUTOR_GUIDE} for schema and validation rules`;
 }
 function mapSchemaError$1(error, catalogPath) {
@@ -967,8 +926,7 @@ function pluginShipsAddMenuCatalog(context) {
   );
 }
 function validateAddMenuAgentKitHandler(context, result) {
-  if (!pluginShipsAddMenuCatalog(context))
-    return;
+  if (!pluginShipsAddMenuCatalog(context)) return;
   const agentKitHandler = context.manifest.agentkit;
   if (!agentKitHandler) {
     result.warnings.push({
@@ -980,15 +938,13 @@ function validateAddMenuAgentKitHandler(context, result) {
     });
   }
   const setupHandler = typeof context.manifest.setup === "string" ? context.manifest.setup : void 0;
-  if (!setupHandler)
-    return;
+  if (!setupHandler) return;
   const setupBody = resolveSetupHandlerFunctionBody(
     context.pluginPath,
     setupHandler,
     context.isNpmPackage
   );
-  if (!setupBody || !handlerBodyWritesAddMenuCatalog(setupBody))
-    return;
+  if (!setupBody || !handlerBodyWritesAddMenuCatalog(setupBody)) return;
   result.warnings.push({
     type: "add-menu-catalog",
     code: "add-menu-legacy-setup-handler",
@@ -1030,8 +986,7 @@ async function validateAddMenuCatalog(context, result) {
   validateAddMenuAgentKitHandler(context, result);
   for (const relPath of ADD_MENU_CATALOG_REL_PATHS) {
     const catalogPath = path.join(context.pluginPath, relPath);
-    if (!fs.existsSync(catalogPath))
-      continue;
+    if (!fs.existsSync(catalogPath)) continue;
     await validateAddMenuCatalogFileAtPath(catalogPath, relPath, result);
   }
 }
@@ -1655,16 +1610,14 @@ async function validateSchema(context, result) {
 }
 function checkExportExists(exportName, context) {
   const packageJsonPath = path.join(context.pluginPath, "package.json");
-  if (!fs.existsSync(packageJsonPath))
-    return true;
+  if (!fs.existsSync(packageJsonPath)) return true;
   let mainPath;
   try {
     const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, "utf-8"));
     if (packageJson.exports?.["."]) {
       const entry = packageJson.exports["."];
       const entryPath = typeof entry === "string" ? entry : entry.default || entry.import;
-      if (entryPath)
-        mainPath = path.join(context.pluginPath, entryPath);
+      if (entryPath) mainPath = path.join(context.pluginPath, entryPath);
     }
     if (!mainPath && packageJson.main) {
       mainPath = path.join(context.pluginPath, packageJson.main);
@@ -1672,8 +1625,7 @@ function checkExportExists(exportName, context) {
   } catch {
     return true;
   }
-  if (!mainPath || !fs.existsSync(mainPath))
-    return true;
+  if (!mainPath || !fs.existsSync(mainPath)) return true;
   try {
     const content = fs.readFileSync(mainPath, "utf-8");
     const patterns = [
@@ -1733,8 +1685,7 @@ function resolveContractFile(contractSpec, context) {
             const resolvedPath = typeof exportValue === "string" ? exportValue : exportValue.default || exportValue.import || exportValue.require;
             if (resolvedPath) {
               const fullPath = path.join(context.pluginPath, resolvedPath);
-              if (fs.existsSync(fullPath))
-                return fullPath;
+              if (fs.existsSync(fullPath)) return fullPath;
             }
           }
         }
@@ -1743,8 +1694,7 @@ function resolveContractFile(contractSpec, context) {
     }
     for (const dir of ["dist", "lib", ""]) {
       const candidate = path.join(context.pluginPath, dir, contractSpec);
-      if (fs.existsSync(candidate))
-        return candidate;
+      if (fs.existsSync(candidate)) return candidate;
     }
     return void 0;
   } else {
@@ -1832,8 +1782,7 @@ function hasExportModifier(node) {
 function resolveModulePath(basePath) {
   for (const ext of ["", ".ts", ".js", "/index.ts", "/index.js"]) {
     const candidate = basePath + ext;
-    if (fs.existsSync(candidate))
-      return candidate;
+    if (fs.existsSync(candidate)) return candidate;
   }
   return void 0;
 }
@@ -1843,10 +1792,8 @@ function resolveComponentSourcePath(componentName, context) {
   const entryFile = resolveModulePath(entryBase);
   const libEntryFile = !entryFile ? resolveModulePath(path.join(context.pluginPath, "lib", modulePath)) : void 0;
   const sourceEntry = entryFile || libEntryFile;
-  if (!sourceEntry)
-    return void 0;
-  if (!sourceEntry.endsWith(".ts"))
-    return void 0;
+  if (!sourceEntry) return void 0;
+  if (!sourceEntry.endsWith(".ts")) return void 0;
   let sourceCode;
   try {
     sourceCode = fs.readFileSync(sourceEntry, "utf-8");
@@ -1862,12 +1809,9 @@ function resolveComponentSourcePath(componentName, context) {
   );
   const starReexportModules = [];
   for (const statement of sourceFile.statements) {
-    if (!ts.isExportDeclaration(statement))
-      continue;
-    if (!statement.moduleSpecifier)
-      continue;
-    if (!ts.isStringLiteral(statement.moduleSpecifier))
-      continue;
+    if (!ts.isExportDeclaration(statement)) continue;
+    if (!statement.moduleSpecifier) continue;
+    if (!ts.isStringLiteral(statement.moduleSpecifier)) continue;
     const moduleSpec = statement.moduleSpecifier.text;
     const exportClause = statement.exportClause;
     if (!exportClause) {
@@ -1885,12 +1829,10 @@ function resolveComponentSourcePath(componentName, context) {
     }
   }
   for (const moduleSpec of starReexportModules) {
-    if (!moduleSpec.startsWith("."))
-      continue;
+    if (!moduleSpec.startsWith(".")) continue;
     const resolvedBase = path.resolve(path.dirname(sourceEntry), moduleSpec);
     const resolvedPath = resolveModulePath(resolvedBase);
-    if (!resolvedPath || !resolvedPath.endsWith(".ts"))
-      continue;
+    if (!resolvedPath || !resolvedPath.endsWith(".ts")) continue;
     try {
       const modSource = fs.readFileSync(resolvedPath, "utf-8");
       const modFile = ts.createSourceFile(
@@ -1923,16 +1865,12 @@ function resolveContractPath(contract, context) {
 }
 async function checkComponentContractConsistency(contract, context, result) {
   const componentName = contract.component;
-  if (!componentName)
-    return;
+  if (!componentName) return;
   const sourcePath = resolveComponentSourcePath(componentName, context);
-  if (!sourcePath)
-    return;
-  if (!sourcePath.endsWith(".ts"))
-    return;
+  if (!sourcePath) return;
+  if (!sourcePath.endsWith(".ts")) return;
   const contractPath = resolveContractPath(contract, context);
-  if (!contractPath)
-    return;
+  if (!contractPath) return;
   let contractContent;
   try {
     contractContent = await fs.promises.readFile(contractPath, "utf-8");
@@ -1940,8 +1878,7 @@ async function checkComponentContractConsistency(contract, context, result) {
     return;
   }
   const parsed = parseContract(contractContent, path.basename(contractPath));
-  if (parsed.validations.length > 0)
-    return;
+  if (parsed.validations.length > 0) return;
   let sourceCode;
   try {
     sourceCode = await fs.promises.readFile(sourcePath, "utf-8");
@@ -2060,8 +1997,7 @@ async function validatePackageJson(context, result) {
 }
 async function validateDynamicContracts(context, result) {
   const { dynamic_contracts } = context.manifest;
-  if (!dynamic_contracts)
-    return;
+  if (!dynamic_contracts) return;
   const dynamicConfigs = Array.isArray(dynamic_contracts) ? dynamic_contracts : [dynamic_contracts];
   for (const config of dynamicConfigs) {
     const prefix = config.prefix || "(unknown)";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jay-framework/plugin-validator",
-  "version": "0.23.0",
+  "version": "0.23.1",
   "type": "module",
   "license": "Apache-2.0",
   "description": "Validation tool for Jay Stack plugins",
@@ -25,14 +25,14 @@
     "test:watch": "vitest"
   },
   "dependencies": {
-    "@jay-framework/compiler-jay-html": "^0.23.0",
-    "@jay-framework/compiler-shared": "^0.23.0",
-    "@jay-framework/typescript-bridge": "^0.23.0",
+    "@jay-framework/compiler-jay-html": "^0.23.1",
+    "@jay-framework/compiler-shared": "^0.23.1",
+    "@jay-framework/typescript-bridge": "^0.23.1",
     "chalk": "^4.1.2",
     "yaml": "^2.3.4"
   },
   "devDependencies": {
-    "@jay-framework/dev-environment": "^0.23.0",
+    "@jay-framework/dev-environment": "^0.23.1",
     "@types/node": "^22.15.21",
     "rimraf": "^5.0.5",
     "tsup": "^8.0.1",