@polymorphism-tech/morph-spec 4.10.9 → 4.10.10

package/src/lib/validators/nextjs/next-component-validator.js

@@ -1,183 +1,183 @@
-import { glob } from 'glob';
-import { readFileSync } from 'node:fs';
-import { join } from 'node:path';
-
-/**
- * Next.js Component Validator
- *
- * Validates .tsx files for:
- * 1. Unnecessary 'use client' directives (no interactivity)
- * 2. Missing 'use client' when React hooks are used
- * 3. File naming convention (kebab-case)
- *
- * @module next-component-validator
- */
-
-// ============================================
-// CONSTANTS
-// ============================================
-
-/** Hooks/patterns that require 'use client' */
-const CLIENT_HOOKS = ['useState', 'useEffect', 'useRef', 'useReducer', 'useCallback', 'useMemo', 'useContext'];
-
-/** Event handler patterns that indicate interactivity (hooks are checked via CLIENT_HOOKS) */
-const EVENT_HANDLER_PATTERNS = [
-  /\bonClick\b/,
-  /\bonChange\b/,
-  /\bonSubmit\b/,
-  /\bonFocus\b/,
-  /\bonBlur\b/,
-  /\bonKeyDown\b/,
-];
-
-/** Next.js special files exempt from naming checks */
-const NEXTJS_SPECIAL_FILES = ['page.tsx', 'layout.tsx', 'loading.tsx', 'error.tsx', 'not-found.tsx', 'route.ts', 'middleware.ts'];
-
-// ============================================
-// MAIN VALIDATION FUNCTION
-// ============================================
-
-/**
- * @typedef {Object} ValidationIssue
- * @property {'error' | 'warning' | 'info'} type
- * @property {string} message
- * @property {string} [suggestion]
- * @property {string} file
- * @property {number} [line]
- */
-
-/**
- * Validates a Next.js component file.
- *
- * @param {string} content - File content
- * @param {string} filePath - Relative file path
- * @returns {ValidationIssue[]}
- */
-export function validateNextComponent(content, filePath) {
-  const issues = [];
-
-  // Only validate .tsx and .ts files
-  if (!filePath.endsWith('.tsx') && !filePath.endsWith('.ts')) {
-    return issues;
-  }
-
-  // Skip non-component utility files (no JSX, no hooks)
-  const hasJsx = /(?<![a-zA-Z0-9])<[A-Z][a-zA-Z]*[\s/>]|(?<![a-zA-Z0-9])<[a-z][a-z]+[\s>/]/.test(content);
-  const hasHooks = CLIENT_HOOKS.some(hook => content.includes(hook));
-
-  if (!hasJsx && !hasHooks) {
-    return issues;
-  }
-
-  const hasUseClient = /^'use client';?|^"use client";?/.test(content.trimStart());
-
-  // Check 1: use client present but no interactivity
-  if (hasUseClient) {
-    const hasEventHandlers = EVENT_HANDLER_PATTERNS.some(pattern => pattern.test(content));
-    const hasClientHooks = CLIENT_HOOKS.some(hook => new RegExp(`\\b${hook}\\b`).test(content));
-    if (!hasEventHandlers && !hasClientHooks) {
-      issues.push({
-        type: 'warning',
-        message: "'use client' directive present but no interactivity detected (no hooks, no event handlers)",
-        suggestion: "Remove 'use client' to make this a Server Component, or add interactive behavior",
-        file: filePath,
-      });
-    }
-  }
-
-  // Check 2: hooks used without use client
-  // .ts files (custom hooks, utilities) never render JSX — skip this check
-  if (!hasUseClient) {
-    if (!filePath.endsWith('.tsx')) return issues;
-    const usedHooks = CLIENT_HOOKS.filter(hook => new RegExp(`\\b${hook}\\b`).test(content));
-    if (usedHooks.length > 0) {
-      issues.push({
-        type: 'error',
-        message: `React hook(s) used (${usedHooks.join(', ')}) without 'use client' directive`,
-        suggestion: "Add 'use client' as the first line of the file",
-        file: filePath,
-        line: 1,
-      });
-    }
-  }
-
-  // Check 3: file naming convention (kebab-case)
-  const fileName = filePath.split('/').pop() ?? '';
-  const isSpecialFile = NEXTJS_SPECIAL_FILES.includes(fileName);
-
-  if (!isSpecialFile && fileName.endsWith('.tsx')) {
-    const baseName = fileName.replace('.tsx', '');
-    const hasUpperCase = /[A-Z]/.test(baseName);
-    if (hasUpperCase) {
-      issues.push({
-        type: 'warning',
-        message: `Component file '${fileName}' should use kebab-case naming`,
-        suggestion: `Rename to '${toKebabCase(baseName)}.tsx'`,
-        file: filePath,
-      });
-    }
-  }
-
-  return issues;
-}
-
-// ============================================
-// HELPERS
-// ============================================
-
-function toKebabCase(str) {
-  return str
-    .replace(/([A-Z])/g, '-$1')
-    .toLowerCase()
-    .replace(/^-/, '');
-}
-
-// ============================================
-// PROJECT-LEVEL SCANNER
-// ============================================
-
-/**
- * Validates all Next.js component files in a project directory.
- * Wraps validateNextComponent for use in validation-runner.js.
- *
- * @param {string} projectPath - Root path of the project
- * @param {Object} _options - Validator options (unused, for interface compatibility)
- * @returns {Promise<{ errors: number, warnings: number, issues: Array }>}
- */
-export async function validateNextComponentFiles(projectPath, _options = {}) {
-  const result = { errors: 0, warnings: 0, issues: [] };
-
-  // Find all .tsx and .ts files under src/
-  const srcDir = join(projectPath, 'src');
-  const normalizedSrc = srcDir.replace(/\\/g, '/');
-  const pattern = normalizedSrc + '/**/*.{tsx,ts}';
-  const files = await glob(pattern, { ignore: ['**/node_modules/**', '**/.next/**'] });
-
-  for (const filePath of files) {
-    let content;
-    try {
-      content = readFileSync(filePath, 'utf8');
-    } catch {
-      continue;
-    }
-
-    // Make path relative for cleaner output
-    const normFilePath = filePath.replace(/\\/g, '/');
-    const relPath = normFilePath.replace(projectPath.replace(/\\/g, '/') + '/', '');
-    const fileIssues = validateNextComponent(content, relPath);
-
-    for (const issue of fileIssues) {
-      result.issues.push({
-        level: issue.type,
-        message: issue.message,
-        file: issue.file,
-        line: issue.line,
-        solution: issue.suggestion,
-      });
-      if (issue.type === 'error') result.errors++;
-      else if (issue.type === 'warning') result.warnings++;
-    }
-  }
-
-  return result;
-}
+import { glob } from 'glob';
+import { readFileSync } from 'node:fs';
+import { join } from 'node:path';
+
+/**
+ * Next.js Component Validator
+ *
+ * Validates .tsx files for:
+ * 1. Unnecessary 'use client' directives (no interactivity)
+ * 2. Missing 'use client' when React hooks are used
+ * 3. File naming convention (kebab-case)
+ *
+ * @module next-component-validator
+ */
+
+// ============================================
+// CONSTANTS
+// ============================================
+
+/** Hooks/patterns that require 'use client' */
+const CLIENT_HOOKS = ['useState', 'useEffect', 'useRef', 'useReducer', 'useCallback', 'useMemo', 'useContext'];
+
+/** Event handler patterns that indicate interactivity (hooks are checked via CLIENT_HOOKS) */
+const EVENT_HANDLER_PATTERNS = [
+  /\bonClick\b/,
+  /\bonChange\b/,
+  /\bonSubmit\b/,
+  /\bonFocus\b/,
+  /\bonBlur\b/,
+  /\bonKeyDown\b/,
+];
+
+/** Next.js special files exempt from naming checks */
+const NEXTJS_SPECIAL_FILES = ['page.tsx', 'layout.tsx', 'loading.tsx', 'error.tsx', 'not-found.tsx', 'route.ts', 'middleware.ts'];
+
+// ============================================
+// MAIN VALIDATION FUNCTION
+// ============================================
+
+/**
+ * @typedef {Object} ValidationIssue
+ * @property {'error' | 'warning' | 'info'} type
+ * @property {string} message
+ * @property {string} [suggestion]
+ * @property {string} file
+ * @property {number} [line]
+ */
+
+/**
+ * Validates a Next.js component file.
+ *
+ * @param {string} content - File content
+ * @param {string} filePath - Relative file path
+ * @returns {ValidationIssue[]}
+ */
+export function validateNextComponent(content, filePath) {
+  const issues = [];
+
+  // Only validate .tsx and .ts files
+  if (!filePath.endsWith('.tsx') && !filePath.endsWith('.ts')) {
+    return issues;
+  }
+
+  // Skip non-component utility files (no JSX, no hooks)
+  const hasJsx = /(?<![a-zA-Z0-9])<[A-Z][a-zA-Z]*[\s/>]|(?<![a-zA-Z0-9])<[a-z][a-z]+[\s>/]/.test(content);
+  const hasHooks = CLIENT_HOOKS.some(hook => content.includes(hook));
+
+  if (!hasJsx && !hasHooks) {
+    return issues;
+  }
+
+  const hasUseClient = /^'use client';?|^"use client";?/.test(content.trimStart());
+
+  // Check 1: use client present but no interactivity
+  if (hasUseClient) {
+    const hasEventHandlers = EVENT_HANDLER_PATTERNS.some(pattern => pattern.test(content));
+    const hasClientHooks = CLIENT_HOOKS.some(hook => new RegExp(`\\b${hook}\\b`).test(content));
+    if (!hasEventHandlers && !hasClientHooks) {
+      issues.push({
+        type: 'warning',
+        message: "'use client' directive present but no interactivity detected (no hooks, no event handlers)",
+        suggestion: "Remove 'use client' to make this a Server Component, or add interactive behavior",
+        file: filePath,
+      });
+    }
+  }
+
+  // Check 2: hooks used without use client
+  // .ts files (custom hooks, utilities) never render JSX — skip this check
+  if (!hasUseClient) {
+    if (!filePath.endsWith('.tsx')) return issues;
+    const usedHooks = CLIENT_HOOKS.filter(hook => new RegExp(`\\b${hook}\\b`).test(content));
+    if (usedHooks.length > 0) {
+      issues.push({
+        type: 'error',
+        message: `React hook(s) used (${usedHooks.join(', ')}) without 'use client' directive`,
+        suggestion: "Add 'use client' as the first line of the file",
+        file: filePath,
+        line: 1,
+      });
+    }
+  }
+
+  // Check 3: file naming convention (kebab-case)
+  const fileName = filePath.split('/').pop() ?? '';
+  const isSpecialFile = NEXTJS_SPECIAL_FILES.includes(fileName);
+
+  if (!isSpecialFile && fileName.endsWith('.tsx')) {
+    const baseName = fileName.replace('.tsx', '');
+    const hasUpperCase = /[A-Z]/.test(baseName);
+    if (hasUpperCase) {
+      issues.push({
+        type: 'warning',
+        message: `Component file '${fileName}' should use kebab-case naming`,
+        suggestion: `Rename to '${toKebabCase(baseName)}.tsx'`,
+        file: filePath,
+      });
+    }
+  }
+
+  return issues;
+}
+
+// ============================================
+// HELPERS
+// ============================================
+
+function toKebabCase(str) {
+  return str
+    .replace(/([A-Z])/g, '-$1')
+    .toLowerCase()
+    .replace(/^-/, '');
+}
+
+// ============================================
+// PROJECT-LEVEL SCANNER
+// ============================================
+
+/**
+ * Validates all Next.js component files in a project directory.
+ * Wraps validateNextComponent for use in validation-runner.js.
+ *
+ * @param {string} projectPath - Root path of the project
+ * @param {Object} _options - Validator options (unused, for interface compatibility)
+ * @returns {Promise<{ errors: number, warnings: number, issues: Array }>}
+ */
+export async function validateNextComponentFiles(projectPath, _options = {}) {
+  const result = { errors: 0, warnings: 0, issues: [] };
+
+  // Find all .tsx and .ts files under src/
+  const srcDir = join(projectPath, 'src');
+  const normalizedSrc = srcDir.replace(/\\/g, '/');
+  const pattern = normalizedSrc + '/**/*.{tsx,ts}';
+  const files = await glob(pattern, { ignore: ['**/node_modules/**', '**/.next/**'] });
+
+  for (const filePath of files) {
+    let content;
+    try {
+      content = readFileSync(filePath, 'utf8');
+    } catch {
+      continue;
+    }
+
+    // Make path relative for cleaner output
+    const normFilePath = filePath.replace(/\\/g, '/');
+    const relPath = normFilePath.replace(projectPath.replace(/\\/g, '/') + '/', '');
+    const fileIssues = validateNextComponent(content, relPath);
+
+    for (const issue of fileIssues) {
+      result.issues.push({
+        level: issue.type,
+        message: issue.message,
+        file: issue.file,
+        line: issue.line,
+        solution: issue.suggestion,
+      });
+      if (issue.type === 'error') result.errors++;
+      else if (issue.type === 'warning') result.warnings++;
+    }
+  }
+
+  return result;
+}

package/src/lib/validators/packages/index.js

@@ -1,7 +1,7 @@
-/**
- * Package Validators
- *
- * Package and dependency validation.
- */
-
-export * from './package-validator.js';
+/**
+ * Package Validators
+ *
+ * Package and dependency validation.
+ */
+
+export * from './package-validator.js';