bertui 1.0.2 → 1.1.0

package/index.js

@@ -32,5 +32,5 @@ export default {
     buildCSS,
     copyCSS,
     program,
-    version: "0.3.9"
+    version: "1.0.3"
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bertui",
-  "version": "1.0.2",
+  "version": "1.1.0",
   "description": "Lightning-fast React dev server powered by Bun and Elysia",
   "type": "module",
   "main": "./index.js",
@@ -50,7 +50,7 @@
   },
   "peerDependencies": {
     "react": "^18.0.0 || ^19.0.0",
-    "react-dom": "^18.0.0 || ^19.0.0",
+    "react-dom": "^19.2.3",
     "bun": ">=1.0.0"
   },
   "engines": {

package/src/build/server-island-validator.js

@@ -0,0 +1,156 @@
+// bertui/src/build/server-island-validator.js
+// Fixed validation for Server Islands - no false positives!
+
+import logger from '../logger/logger.js';
+
+/**
+ * Validates that a Server Island component follows all rules
+ * @param {string} sourceCode - The component source code
+ * @param {string} filePath - Path to the file (for error messages)
+ * @returns {{ valid: boolean, errors: string[] }}
+ */
+export function validateServerIsland(sourceCode, filePath) {
+  const errors = [];
+  
+  // Rule 1: No React hooks (FIXED: only match actual function calls)
+  const hookPatterns = [
+    'useState',
+    'useEffect',
+    'useContext',
+    'useReducer',
+    'useCallback',
+    'useMemo',
+    'useRef',
+    'useImperativeHandle',
+    'useLayoutEffect',
+    'useDebugValue',
+    'useId',
+    'useDeferredValue',
+    'useTransition',
+    'useSyncExternalStore'
+  ];
+  
+  for (const hook of hookPatterns) {
+    // FIXED: Only match hooks as function calls, not in text/comments
+    // Looks for: useState( or const [x] = useState(
+    const regex = new RegExp(`\\b${hook}\\s*\\(`, 'g');
+    if (regex.test(sourceCode)) {
+      errors.push(`❌ Uses React hook: ${hook}`);
+    }
+  }
+  
+  // Rule 2: No bertui/router imports
+  if (sourceCode.includes('from \'bertui/router\'') || 
+      sourceCode.includes('from "bertui/router"')) {
+    errors.push('❌ Imports from \'bertui/router\' (use <a> tags instead of Link)');
+  }
+  
+  // Rule 3: No browser APIs (FIXED: only match actual usage, not in strings/comments)
+  const browserAPIs = [
+    { pattern: 'window\\.', name: 'window' },
+    { pattern: 'document\\.', name: 'document' },
+    { pattern: 'localStorage\\.', name: 'localStorage' },
+    { pattern: 'sessionStorage\\.', name: 'sessionStorage' },
+    { pattern: 'navigator\\.', name: 'navigator' },
+    { pattern: 'location\\.', name: 'location' },
+    { pattern: 'history\\.', name: 'history' },
+    { pattern: '(?<!//.*|/\\*.*|\\*)\\bfetch\\s*\\(', name: 'fetch' },
+    { pattern: '\\.addEventListener\\s*\\(', name: 'addEventListener' },
+    { pattern: '\\.removeEventListener\\s*\\(', name: 'removeEventListener' },
+    { pattern: '\\bsetTimeout\\s*\\(', name: 'setTimeout' },
+    { pattern: '\\bsetInterval\\s*\\(', name: 'setInterval' },
+    { pattern: '\\brequestAnimationFrame\\s*\\(', name: 'requestAnimationFrame' }
+  ];
+  
+  for (const api of browserAPIs) {
+    const regex = new RegExp(api.pattern, 'g');
+    if (regex.test(sourceCode)) {
+      if (api.name === 'console') {
+        logger.warn(`⚠️  ${filePath} uses console.log (will not work in static HTML)`);
+      } else {
+        errors.push(`❌ Uses browser API: ${api.name}`);
+      }
+    }
+  }
+  
+  // Rule 4: No event handlers (these won't work without JS)
+  const eventHandlers = [
+    'onClick=',
+    'onChange=',
+    'onSubmit=',
+    'onInput=',
+    'onFocus=',
+    'onBlur=',
+    'onMouseEnter=',
+    'onMouseLeave=',
+    'onKeyDown=',
+    'onKeyUp=',
+    'onScroll='
+  ];
+  
+  for (const handler of eventHandlers) {
+    if (sourceCode.includes(handler)) {
+      errors.push(`❌ Uses event handler: ${handler.replace('=', '')} (Server Islands are static HTML)`);
+    }
+  }
+  
+  // Rule 5: Check for dynamic imports
+  if (/import\s*\(/.test(sourceCode)) {
+    errors.push('❌ Uses dynamic import() (not supported in Server Islands)');
+  }
+  
+  // Rule 6: Check for async/await (usually indicates API calls)
+  if (/async\s+function|async\s*\(|async\s+\w+\s*\(/.test(sourceCode)) {
+    errors.push('❌ Uses async/await (Server Islands must be synchronous)');
+  }
+  
+  const valid = errors.length === 0;
+  
+  return { valid, errors };
+}
+
+/**
+ * Display validation errors in a clear format
+ */
+export function displayValidationErrors(filePath, errors) {
+  logger.error(`\n🏝️  Server Island validation failed: ${filePath}`);
+  logger.error('\nViolations:');
+  errors.forEach(error => logger.error(`  ${error}`));
+  logger.error('\n📖 Server Island Rules:');
+  logger.error('  ✅ Pure static JSX only');
+  logger.error('  ❌ No React hooks (useState, useEffect, etc.)');
+  logger.error('  ❌ No Link component (use <a> tags)');
+  logger.error('  ❌ No browser APIs (window, document, fetch)');
+  logger.error('  ❌ No event handlers (onClick, onChange, etc.)');
+  logger.error('\n💡 Tip: Remove the "export const render = \\"server\\"" line');
+  logger.error('   if you need these features (page will be client-only).\n');
+}
+
+/**
+ * Extract and validate all Server Islands in a project
+ */
+export async function validateAllServerIslands(routes) {
+  const serverIslands = [];
+  const validationResults = [];
+  
+  for (const route of routes) {
+    const sourceCode = await Bun.file(route.path).text();
+    const isServerIsland = sourceCode.includes('export const render = "server"');
+    
+    if (isServerIsland) {
+      const validation = validateServerIsland(sourceCode, route.path);
+      
+      validationResults.push({
+        route: route.route,
+        path: route.path,
+        ...validation
+      });
+      
+      if (validation.valid) {
+        serverIslands.push(route);
+      }
+    }
+  }
+  
+  return { serverIslands, validationResults };
+}