@jhits/dashboard 0.0.6 → 0.0.8

package/src/config.ts

@@ -1,26 +1,18 @@
 // packages/jhits-dashboard/src/config.ts
 import type { NextConfig } from "next";
-import { writeFileSync, mkdirSync, existsSync, readFileSync } from "fs";
-import { join } from "path";
+import { writeFileSync, mkdirSync, existsSync, readFileSync } from "node:fs";
+import { join } from "node:path";
 
 /**
  * Automatically creates a catch-all route that handles dashboard routes
- * This runs at build time when withJhitsDashboard() is called
- * No dashboard folder needed - routes are handled via catch-all at [locale] level
  */
 async function ensureDashboardRoutes() {
     try {
-        // Find the host app directory (where next.config.ts is)
         let appDir = process.cwd();
-        const possiblePaths = [
-            appDir,
-            join(appDir, '..'),
-            join(appDir, '..', '..'),
-        ];
+        const possiblePaths = [appDir, join(appDir, '..'), join(appDir, '..', '..')];
 
         for (const basePath of possiblePaths) {
-            const configPath = join(basePath, 'next.config.ts');
-            if (existsSync(configPath)) {
+            if (existsSync(join(basePath, 'next.config.ts'))) {
                 appDir = basePath;
                 break;
             }
@@ -30,17 +22,11 @@ async function ensureDashboardRoutes() {
         const catchAllDir = join(localeDir, '[...path]');
         const catchAllPath = join(catchAllDir, 'page.tsx');
 
-        // Check if catch-all already exists
+        // If it exists, try to inject the dashboard handler
         if (existsSync(catchAllPath)) {
-            // Read existing file and check if it already handles dashboard
-            const fs = await import('fs');
-            const existingContent = fs.readFileSync(catchAllPath, 'utf8');
-            if (existingContent.includes('@jhits/dashboard') || existingContent.includes('DashboardRouter')) {
-                // Already set up, skip
-                return;
-            }
-            // If it exists but doesn't handle dashboard, we need to modify it
-            // Read the file and prepend dashboard handling
+            const existingContent = readFileSync(catchAllPath, 'utf8');
+            if (existingContent.includes('@jhits/dashboard') || existingContent.includes('DashboardRouter')) return;
+
             const dashboardHandler = `
     // Dashboard route handling (auto-added by @jhits/dashboard)
     if (path.length > 0 && path[0] === 'dashboard') {
@@ -53,227 +39,235 @@ async function ensureDashboardRoutes() {
         );
     }
 `;
-            // Insert dashboard handling at the beginning of the function body
             const modifiedContent = existingContent.replace(
                 /(export default async function \w+\([^)]+\) \{[\s\S]*?const resolvedParams = await props\.params;[\s\S]*?const path = resolvedParams\.path \|\| \[\];)/,
                 `$1${dashboardHandler}`
             );
-
-            if (modifiedContent !== existingContent) {
-                fs.writeFileSync(catchAllPath, modifiedContent);
-            } else {
-                // If replacement didn't work, append at a safe location
-                const safeInsertPoint = existingContent.indexOf('const path = resolvedParams.path');
-                if (safeInsertPoint > -1) {
-                    const before = existingContent.substring(0, safeInsertPoint);
-                    const after = existingContent.substring(safeInsertPoint);
-                    const newContent = before + dashboardHandler.trim() + '\n    ' + after;
-                    fs.writeFileSync(catchAllPath, newContent);
-                }
-            }
+            writeFileSync(catchAllPath, modifiedContent);
             return;
         }
 
-        // Create new catch-all route that handles dashboard routes
+        // Create new catch-all if none exists
         mkdirSync(catchAllDir, { recursive: true });
-        writeFileSync(catchAllPath, `// Auto-generated by @jhits/dashboard - do not edit manually
-export { default } from '@jhits/dashboard/catch-all';
-`);
-    } catch {
-        // Silently fail - routes might already exist or be manually created
+        writeFileSync(catchAllPath, `// Auto-generated by @jhits/dashboard - do not edit manually\nexport { default } from '@jhits/dashboard/catch-all';\n`);
+    } catch (e) {
+        console.warn('[@jhits/dashboard] Could not auto-generate dashboard routes:', e);
     }
 }
 
 /**
- * Dynamically find all @jhits/* packages from package.json
- * Reads dependencies and devDependencies to build transpilePackages list
- * 
- * This function finds the app's package.json by looking for next.config.ts
- * in the same directory, ensuring we read from the correct location in monorepos
+ * Finds all @jhits/* packages to ensure they are transpiled
  */
 function findJhitsPackages(): string[] {
     try {
-        // Find the app directory (where next.config.ts is located)
         const appDir = process.cwd();
-        const possiblePaths = [
-            appDir,
-            join(appDir, '..'),
-            join(appDir, '..', '..'),
-        ];
+        const packageJsonPath = join(appDir, 'package.json');
+        if (!existsSync(packageJsonPath)) return ['@jhits'];
 
-        let packageJsonPath: string | null = null;
-
-        for (const basePath of possiblePaths) {
-            const configPath = join(basePath, 'next.config.ts');
-            const pkgPath = join(basePath, 'package.json');
-            if (existsSync(configPath) && existsSync(pkgPath)) {
-                packageJsonPath = pkgPath;
-                break;
-            }
-        }
-
-        // Fallback to process.cwd() if we can't find next.config.ts
-        if (!packageJsonPath) {
-            packageJsonPath = join(process.cwd(), 'package.json');
-        }
-
-        if (!existsSync(packageJsonPath)) {
-            // Fallback to wildcard if package.json not found
-            return ['@jhits'];
-        }
-
-        // Use readFileSync instead of require() to avoid module resolution issues
-        const packageJsonContent = readFileSync(packageJsonPath, 'utf-8');
-        const packageJson = JSON.parse(packageJsonContent);
-
-        const allDeps = {
-            ...(packageJson.dependencies || {}),
-            ...(packageJson.devDependencies || {}),
-        };
+        const packageJson = JSON.parse(readFileSync(packageJsonPath, 'utf-8'));
+        const allDeps = { ...(packageJson.dependencies || {}), ...(packageJson.devDependencies || {}) };
+        return Object.keys(allDeps).filter(pkg => pkg.startsWith('@jhits/'));
+    } catch { return ['@jhits']; }
+}
 
-        // Filter for packages starting with @jhits/
-        const jhitsPackages = Object.keys(allDeps).filter(
-            (pkg) => pkg.startsWith('@jhits/')
-        );
+/**
+ * Detects if packages are installed from tarballs (file: paths) vs workspace
+ * This helps determine if we need special handling for WASM modules
+ */
+function isUsingTarballPackages(): boolean {
+    try {
+        const appDir = process.cwd();
+        const packageJsonPath = join(appDir, 'package.json');
+        if (!existsSync(packageJsonPath)) return false;
 
-        // Always include the base @jhits pattern for any other packages
-        return jhitsPackages.length > 0 ? jhitsPackages : ['@jhits'];
+        const packageJson = JSON.parse(readFileSync(packageJsonPath, 'utf-8'));
+        const allDeps = { ...(packageJson.dependencies || {}), ...(packageJson.devDependencies || {}) };
+        
+        // Check if any @jhits package uses file: protocol (tarball installation)
+        return Object.entries(allDeps).some(([name, version]) => {
+            return name.startsWith('@jhits/') && typeof version === 'string' && version.startsWith('file:');
+        });
     } catch {
-        // Fallback to wildcard on any error
-        console.warn('[withJhitsDashboard] Could not read package.json, using @jhits wildcard');
-        return ['@jhits'];
+        return false;
     }
 }
 
 /**
- * JHITS Dashboard Plugin Configuration
- * 
- * Usage:
- *   import { withJhitsDashboard } from '@jhits/dashboard/config';
- *   export default withJhitsDashboard(nextConfig);
- * 
- * This automatically sets up all dashboard routes - no manual setup needed!
+ * Gets the path to empty.js that works in both workspace and tarball modes
+ * For Turbopack resolveAlias, we need to use package-relative paths, not absolute paths
+ * Turbopack resolves these relative to the package installation location
  */
+function getEmptyFilePath(): string {
+    // Use package-relative path - Turbopack's resolveAlias works better with these
+    // The path is resolved via package.json exports, so it works regardless of installation method
+    return "@jhits/dashboard/src/empty.js";
+}
+
 export function withJhitsDashboard(nextConfig: NextConfig = {}): NextConfig {
-    // Auto-create dashboard routes at build time
-    // NOTE: User management routes are now handled by plugin-users via the unified plugin router
-    // No need to auto-generate /api/users routes anymore
-    if (typeof window === 'undefined') {
-        try {
-            ensureDashboardRoutes();
-            // ensureUserManagementRoutes(); // Disabled - routes are handled by plugin-users via plugin router
-        } catch {
-            // Ignore errors - routes might already exist
-        }
+    // Only run file system modifications on the server during initialization
+    if (typeof window === 'undefined' && process.env.NEXT_PHASE !== 'phase-production-build') {
+        ensureDashboardRoutes().catch(() => { });
+    }
+
+    // Get empty file path - resolve to absolute path for better compatibility
+    // This works in both workspace and tarball modes
+    // Use absolute path for Turbopack resolveAlias as it's more reliable
+    const emptyFile = getEmptyFilePath();
+    // Use package-relative path for the loader - Turbopack can resolve this
+    // This works both in development (monorepo) and when installed as a package
+    const emptyLoaderPath = "@jhits/dashboard/src/empty-loader.js";
+    // Detect tarball installation (currently not used but available for future conditional logic)
+    const usingTarballs = isUsingTarballPackages();
+    
+    // Log for debugging (only in dev mode)
+    if (process.env.NODE_ENV === 'development') {
+        console.log('[@jhits/dashboard] Empty file path:', emptyFile);
+        console.log('[@jhits/dashboard] Using tarball packages:', usingTarballs);
     }
 
     return {
         ...nextConfig,
 
-        // 1. AUTO-TRANSPILE: Dynamically find all @jhits/* packages from package.json
-        // This ensures Turbopack knows exactly which workspace folders to link
         transpilePackages: [
             ...(nextConfig.transpilePackages || []),
             ...findJhitsPackages()
         ],
 
+        turbopack: {
+            ...nextConfig.turbopack,
+            rules: {
+                ...nextConfig.turbopack?.rules,
+                // Handle .md files by treating them as empty JavaScript modules
+                // This prevents "Unknown module type" errors for README.md files
+                // According to Next.js docs: pattern should be *.md with as: *.js
+                // Using our custom empty-loader to return empty module
+                // Loaders can be strings (package names or file paths)
+                '*.md': {
+                    loaders: [emptyLoaderPath],
+                    as: '*.js',
+                },
+                '*.mdx': {
+                    loaders: [emptyLoaderPath],
+                    as: '*.js',
+                },
+            },
+            resolveAlias: {
+                // IMPORTANT: Merge user's resolveAlias FIRST, then apply our aliases
+                // This ensures our aliases take precedence for WASM virtual modules
+                ...nextConfig.turbopack?.resolveAlias,
+                // CRITICAL: We only alias specific toxic libraries for CLIENT bundles.
+                // We DO NOT alias 'fs', 'path', or 'child_process' here because
+                // it breaks the Next.js internal build tools (WASM/SWC).
+                // NOTE: mongodb, bcrypt, and nodemailer are handled by serverExternalPackages
+                // and should NOT be aliased here as it breaks server-side imports.
+                // "mongodb": emptyFile, // REMOVED - breaks server-side imports, use serverExternalPackages instead
+                // "bcrypt": emptyFile, // REMOVED - breaks server-side imports, use serverExternalPackages instead
+                // "nodemailer": emptyFile, // REMOVED - breaks server-side imports, use serverExternalPackages instead
+                // "dns": emptyFile,
+                "server-only": emptyFile,
+                // Shield the auth logic from the client
+                "@jhits/dashboard/lib/auth": emptyFile,
+                // WASM virtual modules used by next-intl-swc-plugin-extractor
+                // These MUST be aliased AFTER merging user's aliases to ensure they take precedence
+                // Required for dev mode (Turbopack) - works in both workspace and tarball mode
+                // The empty.js file is exported in package.json and accessible via @jhits/dashboard/src/empty.js
+                // In prod mode with webpack, these are handled by webpack.resolve.fallback instead
+                "env": emptyFile,
+                "wasi_snapshot_preview1": emptyFile,
+                // Alias specific .md files to empty file (Turbopack doesn't support wildcards in resolveAlias)
+                // Add aliases for known README.md locations in @jhits packages
+                "README.md": emptyFile,
+                "@jhits/dashboard/README.md": emptyFile,
+                "@jhits/plugin-blog/README.md": emptyFile,
+                "@jhits/plugin-content/README.md": emptyFile,
+                "@jhits/plugin-images/README.md": emptyFile,
+                "@jhits/plugin-users/README.md": emptyFile,
+                "@jhits/plugin-website/README.md": emptyFile,
+                "@jhits/plugin-newsletter/README.md": emptyFile,
+                "@jhits/plugin-telemetry/README.md": emptyFile,
+                "@jhits/plugin-dep/README.md": emptyFile,
+                "@jhits/plugin-core/README.md": emptyFile,
+            }
+        },
+
         experimental: {
             ...nextConfig.experimental,
-            // 2. MONOREPO RESOLUTION: This is the secret sauce. 
-            // It allows Next.js to follow symlinks in the pnpm workspace 
-            // even if they aren't explicitly listed.
             externalDir: true,
         },
 
-        // Exclude server-only third-party packages from client bundles
-        // Note: @jhits/* packages should NOT be here - they're in transpilePackages instead
-        // Adding them here would conflict with transpilePackages in Turbopack
         serverExternalPackages: [
             ...(nextConfig.serverExternalPackages || []),
-            'mongodb',
-            'bcrypt',
-            'bcryptjs',
-            'jsonwebtoken',
-            'nodemailer'
+            'mongodb', 'bcrypt', 'bcryptjs', 'jsonwebtoken', 'nodemailer'
         ],
 
         webpack: (config, { isServer }) => {
-            // Ensure Node.js modules are not bundled for client
             if (!isServer) {
+                // Webpack is safer for building "fallbacks" than Turbopack's global alias
                 config.resolve.fallback = {
                     ...config.resolve.fallback,
                     fs: false,
-                    path: false,
-                    'fs/promises': false,
-                    child_process: false,
                     net: false,
                     tls: false,
-                    crypto: false,
-                    stream: false,
-                    util: false,
-                    url: false,
-                    http: false,
-                    https: false,
-                    zlib: false,
+                    child_process: false,
                     os: false,
-                    buffer: false,
-                    process: false,
-                    // WASI modules
+                    // WASM virtual modules used by next-intl-swc-plugin-extractor
+                    // These are needed when packages are installed from tarballs (file: paths)
+                    // Using false tells webpack to ignore these modules (works in prod/webpack mode)
+                    env: false,
                     wasi_snapshot_preview1: false,
                 };
-
-                // Prevent server-only packages from being bundled into client
-                const serverOnlyPackages = [
-                    '@jhits/plugin-dep',
-                    'nodemailer',
-                    'mongodb',
-                    'bcrypt',
-                    'bcryptjs',
-                    'jsonwebtoken',
-                ];
-
-                config.externals = config.externals || [];
-                if (Array.isArray(config.externals)) {
-                    config.externals.push(...serverOnlyPackages);
-                } else if (typeof config.externals === 'object') {
-                    serverOnlyPackages.forEach((pkg) => {
-                        config.externals[pkg] = pkg;
-                    });
-                } else if (typeof config.externals === 'function') {
-                    const originalExternals = config.externals;
-                    config.externals = (
-                        context: string,
-                        request: string,
-                        callback: (err?: Error | null, result?: string) => void
-                    ) => {
-                        if (serverOnlyPackages.includes(request)) {
-                            return callback(null, `commonjs ${request}`);
-                        }
-                        return originalExternals(context, request, callback);
-                    };
-                }
-
-                // Ignore server-only plugin files using IgnorePlugin
-                // This prevents webpack from trying to bundle server-only files
-                try {
-                    // eslint-disable-next-line @typescript-eslint/no-require-imports
-                    const webpack = require('webpack');
-                    config.plugins = config.plugins || [];
-                    config.plugins.push(
-                        new webpack.IgnorePlugin({
-                            resourceRegExp: /^@jhits\/plugin-dep\/src\/(actions|router|index\.server)$/,
-                        })
-                    );
-                } catch {
-                    // webpack might not be available in all contexts (e.g., Turbopack)
-                    // This is fine - the externals and browser field should handle it
-                }
+                
+                // Also add to resolve.alias for webpack (backup for Turbopack resolveAlias)
+                // This ensures WASM modules are handled even if Turbopack's resolveAlias doesn't work
+                config.resolve.alias = {
+                    ...config.resolve.alias,
+                    'env': false,
+                    'wasi_snapshot_preview1': false,
+                };
             }
+            
+            // Ignore .md files (README.md, etc.) from node_modules
+            // This prevents Next.js from trying to process them as modules
+            // Note: webpack is only available when using webpack (not Turbopack)
+            // Use dynamic require to avoid static analysis issues
+            try {
+                // Use Function constructor to create a truly dynamic require that won't be statically analyzed
+                const requireWebpack = new Function('moduleName', 'return require(moduleName)');
+                const webpack = requireWebpack('webpack');
+                config.plugins = config.plugins || [];
+                // Ignore all .md and .mdx files completely
+                config.plugins.push(
+                    new webpack.IgnorePlugin({
+                        resourceRegExp: /\.md$/,
+                    }),
+                    new webpack.IgnorePlugin({
+                        resourceRegExp: /\.mdx$/,
+                    })
+                );
+            } catch {
+                // webpack not available (likely using Turbopack) - skip webpack-specific config
+                // This is expected when using Turbopack, so we silently continue
+            }
+            
+            // Add resolve alias for .md files to prevent module resolution errors
+            config.resolve.alias = {
+                ...config.resolve.alias,
+                'README.md': false,
+            };
+            
+            // Add a rule to handle .md files as empty modules if they somehow get imported
+            // This is a fallback in case IgnorePlugin doesn't catch everything
+            config.module = config.module || {};
+            config.module.rules = config.module.rules || [];
+            // Insert at the beginning to catch .md files before other rules
+            config.module.rules.unshift({
+                test: /\.mdx?$/,
+                use: {
+                    loader: require.resolve('./empty-loader.js'),
+                },
+            });
+            
             return config;
         },
-
-        // Routes are automatically created and managed by the plugin
-        // The dashboard folder is completely transparent - you never need to touch it
-        // Just use withJhitsDashboard() and everything works!
     };
 }

package/src/empty-loader.js

@@ -0,0 +1,4 @@
+// Simple loader that returns an empty module for .md files
+module.exports = function(source) {
+    return 'module.exports = {};';
+};

package/src/empty.js

@@ -0,0 +1,5 @@
+// Empty module - used as an alias for modules that should be ignored
+// This is used for WASM virtual modules and other modules that shouldn't be bundled
+// Supports both CommonJS and ES modules for maximum compatibility
+module.exports = {};
+export default {};

package/src/index.client.tsx

@@ -0,0 +1,16 @@
+// Main export file for the dashboard plugin - CLIENT-SAFE ONLY
+// This file should NEVER import or export server-only code
+
+'use client';
+
+// Client-safe components only
+export { PluginRegistry } from './lib/plugin-registry';
+export { PluginNotFound } from './components/PluginNotFound';
+export { WebsiteProvider, useWebsite } from './lib/website-context';
+export type { WebsiteInfo } from './lib/website-context';
+export { Providers, AuthGuard } from './components/Providers';
+
+// Client components
+export { default as Sidebar } from './components/dashboard/Sidebar';
+export { default as Topbar } from './components/dashboard/Topbar';
+

package/src/index.server.tsx

@@ -0,0 +1,66 @@
+// Server-only exports for the dashboard plugin
+// This file contains all server-side functionality
+
+import 'server-only';
+
+// Router components - server-only
+import DashboardLayout from './app/[locale]/dashboard/layout';
+import DashboardHome from './app/[locale]/dashboard/page';
+import DashboardPreferences from './app/[locale]/dashboard/preferences/page';
+import DashboardProfile from './app/[locale]/dashboard/profile/page';
+import DashboardPluginRoute from './app/[locale]/dashboard/[...pluginRoute]/page';
+
+export async function DashboardRouter({
+    path,
+    params
+}: {
+    path: string[];
+    params: Promise<{ locale: string }>;
+}) {
+    const resolvedParams = await params;
+    const locale = resolvedParams.locale;
+
+    // Get the actual route (first segment of path)
+    const route = path.length === 0 ? 'home' : path[0];
+
+    // Handle different dashboard routes
+    switch (route) {
+        case 'preferences':
+            return <DashboardPreferences />;
+        case 'profile':
+            return <DashboardProfile />;
+        case 'home':
+        case '':
+            return <DashboardHome />;
+        default:
+            // This is a plugin route - pass the full path as pluginRoute
+            return <DashboardPluginRoute params={Promise.resolve({
+                locale,
+                pluginRoute: path
+            })} />;
+    }
+}
+
+// Layout wrapper
+export function DashboardRouterLayout({ children }: { children: React.ReactNode }) {
+    return <DashboardLayout>{children}</DashboardLayout>;
+}
+
+// Server-only components
+export { default as DashboardCatchAll } from './components/DashboardCatchAll';
+
+// Server-only config and utilities
+export { withJhitsDashboard } from './config';
+export { getDashboardMessages } from './i18n/translations';
+export { PLATFORM_MODULES } from './lib/modules-config';
+
+// Server-only lib exports (for advanced usage)
+export { authOptions } from './lib/auth';
+export { default as clientPromise } from './lib/mongodb';
+
+// API router exports
+export { handlePluginApi, type PluginRouterConfig } from './api/pluginRouter';
+export { handleDashboardApi, createNextRequestFromRequest } from './api/masterRouter';
+
+// Metadata generation
+export { generateDashboardMetadata } from './lib/generate-dashboard-metadata';

package/src/index.tsx

@@ -1,69 +1,16 @@
 // Main export file for the dashboard plugin
-// This exports all the dashboard components and pages that can be used by host apps
-
-// Router components - inlined to avoid module resolution issues
-import DashboardLayout from './app/[locale]/dashboard/layout';
-import DashboardHome from './app/[locale]/dashboard/page';
-import DashboardPreferences from './app/[locale]/dashboard/preferences/page';
-import DashboardProfile from './app/[locale]/dashboard/profile/page';
-import DashboardPluginRoute from './app/[locale]/dashboard/[...pluginRoute]/page';
-
-export async function DashboardRouter({ 
-    path, 
-    params 
-}: { 
-    path: string[];
-    params: Promise<{ locale: string }>;
-}) {
-    const resolvedParams = await params;
-    const locale = resolvedParams.locale;
-    
-    // Get the actual route (first segment of path)
-    const route = path.length === 0 ? 'home' : path[0];
-    
-    // Handle different dashboard routes
-    switch (route) {
-        case 'preferences':
-            return <DashboardPreferences />;
-        case 'profile':
-            return <DashboardProfile />;
-        case 'home':
-        case '':
-            return <DashboardHome />;
-        default:
-            // This is a plugin route - pass the full path as pluginRoute
-            return <DashboardPluginRoute params={Promise.resolve({ 
-                locale, 
-                pluginRoute: path 
-            })} />;
-    }
-}
-
-// Layout wrapper
-export function DashboardRouterLayout({ children }: { children: React.ReactNode }) {
-    return <DashboardLayout>{children}</DashboardLayout>;
-}
-
-// Components (for advanced usage)
-export { default as Sidebar } from './components/dashboard/Sidebar';
-export { default as Topbar } from './components/dashboard/Topbar';
-export { PluginNotFound } from './components/PluginNotFound';
-export { Providers, AuthGuard } from './components/Providers';
-
-// Plugin system
-export { PluginRegistry } from './lib/plugin-registry';
-export { PLATFORM_MODULES } from './lib/modules-config';
-
-// Config
-export { withJhitsDashboard } from './config';
-
-// Translations
-export { getDashboardMessages } from './i18n/translations';
-
-// Catch-all route handler (for client app integration)
-export { default as DashboardCatchAll } from './components/DashboardCatchAll';
-
-// Website context (for accessing website info in dashboard components)
-export { WebsiteProvider, useWebsite } from './lib/website-context';
-export type { WebsiteInfo } from './lib/website-context';
+// Re-exports client-safe code by default
+// For server-only code, import from '@jhits/dashboard/server'
+
+// Re-export client-safe code
+export * from './index.client';
+
+// Note: Server-only exports are available via '@jhits/dashboard/server'
+// This includes:
+// - DashboardRouter, DashboardRouterLayout
+// - DashboardCatchAll
+// - withJhitsDashboard
+// - getDashboardMessages
+// - PLATFORM_MODULES
+// - authOptions, clientPromise