openuispec 0.2.16 → 0.2.17

package/check/audit.ts

@@ -10,7 +10,7 @@
  *   openuispec check --target web --audit --format json
  */
 
-import { readFileSync } from "node:fs";
+import { existsSync, readFileSync } from "node:fs";
 import { join, resolve } from "node:path";
 import YAML from "yaml";
 
@@ -31,6 +31,16 @@ export interface AuditResult {
 }
 
 const AI_DEFAULT_FONTS = new Set(["Inter", "Roboto", "Arial", "Open Sans"]);
+const REQUIRED_TOKEN_FILES = [
+  "color.yaml",
+  "typography.yaml",
+  "spacing.yaml",
+  "elevation.yaml",
+  "motion.yaml",
+  "layout.yaml",
+  "themes.yaml",
+  "icons.yaml",
+];
 
 function readYaml(path: string): any {
   try {
@@ -40,8 +50,36 @@ function readYaml(path: string): any {
   }
 }
 
+function readYamlForAudit(path: string, domain: string, findings: AuditFinding[]): any {
+  try {
+    return YAML.parse(readFileSync(path, "utf-8"));
+  } catch (err: any) {
+    if (err?.code === "ENOENT") return null;
+    findings.push({
+      domain,
+      rule: "unreadable_file",
+      severity: "error",
+      message: `Could not parse "${path}". Fix malformed YAML before relying on this audit.`,
+    });
+    return null;
+  }
+}
+
+function checkRequiredTokenFiles(tokensDir: string, findings: AuditFinding[]): void {
+  for (const filename of REQUIRED_TOKEN_FILES) {
+    if (!existsSync(join(tokensDir, filename))) {
+      findings.push({
+        domain: "tokens",
+        rule: "missing_file",
+        severity: "error",
+        message: `Required token file "${filename}" is missing.`,
+      });
+    }
+  }
+}
+
 function checkTypography(tokensDir: string, findings: AuditFinding[]): void {
-  const doc = readYaml(join(tokensDir, "typography.yaml"));
+  const doc = readYamlForAudit(join(tokensDir, "typography.yaml"), "typography", findings);
   if (!doc?.typography) return;
 
   // Font diversity: primary must NOT be a common AI default
@@ -68,7 +106,7 @@ function checkTypography(tokensDir: string, findings: AuditFinding[]): void {
 }
 
 function checkColor(tokensDir: string, findings: AuditFinding[]): void {
-  const doc = readYaml(join(tokensDir, "color.yaml"));
+  const doc = readYamlForAudit(join(tokensDir, "color.yaml"), "color", findings);
   if (!doc?.color) return;
 
   // Pure black/white check
@@ -103,8 +141,9 @@ function checkColor(tokensDir: string, findings: AuditFinding[]): void {
 
   // Theme coverage: check themes.yaml for both light + dark
   {
-    const themes = readYaml(join(tokensDir, "themes.yaml"));
-    const themeKeys = Object.keys(themes?.themes ?? {});
+    const themes = readYamlForAudit(join(tokensDir, "themes.yaml"), "color", findings);
+    if (!themes?.themes) return;
+    const themeKeys = Object.keys(themes.themes);
     const hasLight = themeKeys.some((k) => k.includes("light"));
     const hasDark = themeKeys.some((k) => k.includes("dark"));
     if (!hasLight || !hasDark) {
@@ -119,7 +158,7 @@ function checkColor(tokensDir: string, findings: AuditFinding[]): void {
 }
 
 function checkSpacing(tokensDir: string, findings: AuditFinding[]): void {
-  const doc = readYaml(join(tokensDir, "spacing.yaml"));
+  const doc = readYamlForAudit(join(tokensDir, "spacing.yaml"), "spacing", findings);
   if (!doc?.spacing) return;
 
   // Scale usage: at least 4 distinct values
@@ -156,7 +195,7 @@ function checkSpacing(tokensDir: string, findings: AuditFinding[]): void {
 }
 
 function checkMotion(tokensDir: string, findings: AuditFinding[]): void {
-  const doc = readYaml(join(tokensDir, "motion.yaml"));
+  const doc = readYamlForAudit(join(tokensDir, "motion.yaml"), "motion", findings);
   if (!doc?.motion) return;
 
   // Duration variety: at least 2 distinct durations
@@ -185,7 +224,7 @@ function checkMotion(tokensDir: string, findings: AuditFinding[]): void {
 function checkContracts(contractsDir: string, findings: AuditFinding[]): void {
   // All collections have empty_state in must_handle or variants
   {
-    const doc = readYaml(join(contractsDir, "collection.yaml"));
+    const doc = readYamlForAudit(join(contractsDir, "collection.yaml"), "contracts", findings);
     const collection = doc ? doc[Object.keys(doc)[0]] : null;
     if (collection) {
       const mustHandle: string[] = collection.generation?.must_handle ?? [];
@@ -211,6 +250,7 @@ export function buildAuditResult(projectDir: string, threshold: number = 0): Aud
   const effectiveThreshold = threshold > 0 ? threshold : (manifest?.generation_guidance?.audit_threshold ?? 0);
 
   const findings: AuditFinding[] = [];
+  checkRequiredTokenFiles(tokensDir, findings);
   checkTypography(tokensDir, findings);
   checkColor(tokensDir, findings);
   checkSpacing(tokensDir, findings);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openuispec",
-  "version": "0.2.16",
+  "version": "0.2.17",
   "license": "MIT",
   "type": "module",
   "description": "A semantic UI specification format for AI-native, platform-native app development",

package/prepare/index.ts

@@ -666,8 +666,8 @@ function generationRules(target: string, outputDir: string, manifest: Record<str
 }
 
 function matchesTargetPlatform(item: string, target: string): boolean {
-  const tagMatch = item.match(/^\[([a-z]+)\]/);
-  return !tagMatch || tagMatch[1] === target;
+  const tagMatch = item.match(/^\[([a-z]+)\]/i);
+  return !tagMatch || tagMatch[1].toLowerCase() === target;
 }
 
 function complexityRule(complexity: string): string {
@@ -710,10 +710,10 @@ function buildAntiPatterns(
 
   // Contract-specific must_avoid
   const contract_specific: Record<string, string[]> = {};
-  try {
-    const contractsDir = resolve(projectDir, manifest.includes?.contracts ?? './contracts/');
-    if (existsSync(contractsDir)) {
-      for (const file of readdirSync(contractsDir).filter((f) => f.endsWith('.yaml') && !f.startsWith('x_'))) {
+  const contractsDir = resolve(projectDir, manifest.includes?.contracts ?? './contracts/');
+  if (existsSync(contractsDir)) {
+    for (const file of readdirSync(contractsDir).filter((f) => f.endsWith('.yaml') && !f.startsWith('x_'))) {
+      try {
         const content = YAML.parse(readFileSync(join(contractsDir, file), 'utf-8'));
         const contractName = Object.keys(content)[0];
         const mustAvoid: string[] = content[contractName]?.generation?.must_avoid ?? [];
@@ -721,9 +721,11 @@ function buildAntiPatterns(
           const filtered = mustAvoid.filter((item: string) => matchesTargetPlatform(item, target));
           if (filtered.length > 0) contract_specific[contractName] = filtered;
         }
+      } catch {
+        continue;
       }
     }
-  } catch { /* skip on error */ }
+  }
 
   // Project-specific avoid from design section
   const project_specific: string[] = [];
@@ -1386,6 +1388,8 @@ function buildUpdatePrepareResult(cwd: string, target: string, includeContents:
   const sharedLayerConfigs = sharedLayersForTarget(projectDir, target);
   const codeRoots = suggestCodeRoots(target, outputDir, projectDir, sharedLayerConfigs);
   const manifest = readManifest(projectDir);
+  const antiPatterns = buildAntiPatterns(manifest, projectDir, target);
+  const designContext = buildDesignContext(manifest);
   const sharedLayerInfos = buildSharedLayerInfos(projectDir, target, sharedLayerConfigs);
   const platformDef = readPlatformDefinition(projectDir, manifest, target);
   const platformConfig = buildPlatformConfig(target, platformDef);
@@ -1429,6 +1433,8 @@ function buildUpdatePrepareResult(cwd: string, target: string, includeContents:
     items,
     ...(sharedLayerInfos.length > 0 ? { shared_layers: sharedLayerInfos } : {}),
     ...(includeContents ? { spec_contents: readAllSpecContents(projectDir) } : {}),
+    ...(antiPatterns ? { anti_patterns: antiPatterns } : {}),
+    ...(designContext ? { design_context: designContext } : {}),
     next_steps: nextSteps,
   };
 }

package/schema/validate.ts

@@ -399,6 +399,16 @@ function buildAjv(): AjvInstance {
 }
 
 const BASE = "https://openuispec.rsteam.uz/schema/";
+const TOKEN_FILE_SCHEMAS: Record<string, string> = {
+  "color.yaml": "color.schema.json",
+  "typography.yaml": "typography.schema.json",
+  "spacing.yaml": "spacing.schema.json",
+  "elevation.yaml": "elevation.schema.json",
+  "motion.yaml": "motion.schema.json",
+  "layout.yaml": "layout.schema.json",
+  "themes.yaml": "themes.schema.json",
+  "icons.yaml": "icons.schema.json",
+};
 
 // ── validate one file ────────────────────────────────────────────────
 
@@ -536,20 +546,13 @@ const GROUPS: Record<string, ValidationGroup> = {
     run(ajv, projectDir, includes) {
       let errors = 0;
       const tokensDir = resolveInclude(projectDir, includes.tokens);
-      const tokenMap: Record<string, string> = {
-        "color.yaml": "color.schema.json",
-        "typography.yaml": "typography.schema.json",
-        "spacing.yaml": "spacing.schema.json",
-        "elevation.yaml": "elevation.schema.json",
-        "motion.yaml": "motion.schema.json",
-        "layout.yaml": "layout.schema.json",
-        "themes.yaml": "themes.schema.json",
-        "icons.yaml": "icons.schema.json",
-      };
-      for (const [data, schema] of Object.entries(tokenMap)) {
+      for (const [data, schema] of Object.entries(TOKEN_FILE_SCHEMAS)) {
         const filePath = join(tokensDir, data);
         if (existsSync(filePath)) {
           errors += validateFile(ajv, filePath, `${BASE}tokens/${schema}`);
+        } else {
+          console.log(`  FAIL  ${data} (required token file is missing)`);
+          errors += 1;
         }
       }
       return errors;
@@ -557,20 +560,16 @@ const GROUPS: Record<string, ValidationGroup> = {
     collectJson(ajv, projectDir, includes, groupKey) {
       const errors: JsonError[] = [];
       const tokensDir = resolveInclude(projectDir, includes.tokens);
-      const tokenMap: Record<string, string> = {
-        "color.yaml": "color.schema.json",
-        "typography.yaml": "typography.schema.json",
-        "spacing.yaml": "spacing.schema.json",
-        "elevation.yaml": "elevation.schema.json",
-        "motion.yaml": "motion.schema.json",
-        "layout.yaml": "layout.schema.json",
-        "themes.yaml": "themes.schema.json",
-        "icons.yaml": "icons.schema.json",
-      };
-      for (const [data, schema] of Object.entries(tokenMap)) {
+      for (const [data, schema] of Object.entries(TOKEN_FILE_SCHEMAS)) {
         const filePath = join(tokensDir, data);
         if (existsSync(filePath)) {
           errors.push(...collectValidateFile(ajv, filePath, `${BASE}tokens/${schema}`));
+        } else {
+          errors.push({
+            file: data,
+            path: "(root)",
+            message: "required token file is missing",
+          });
         }
       }
       return { group: groupKey, errors };