@terrymooreii/sia 2.3.0 → 2.3.1

package/lib/build.js

@@ -236,6 +236,7 @@ export async function build(options = {}) {
   // Load plugins
   let plugins = [];
   try {
+    console.log('🔌 Loading plugins...');
     plugins = await loadPlugins(config);
     
     // Register plugin hooks

package/lib/plugins.js

@@ -45,13 +45,21 @@ export function validatePlugin(plugin, pluginPath) {
 export async function loadPlugin(pluginPath, config) {
   try {
     // Support both .js and .mjs files
+    // Add cache busting query parameter to ensure fresh reloads during dev
+    // This prevents Node.js from using cached versions of plugins during hot reload
+    const cacheBuster = `?t=${Date.now()}`;
     let module;
     
-    if (pluginPath.endsWith('.mjs')) {
-      module = await import(`file://${pluginPath}`);
-    } else {
-      module = await import(pluginPath);
-    }
+    // Normalize path to use file:// protocol for consistent cache busting
+    // All plugin paths from discovery are absolute, so we can safely use file://
+    const normalizedPath = pluginPath.startsWith('file://') 
+      ? pluginPath 
+      : pluginPath.startsWith('/') || /^[A-Za-z]:/.test(pluginPath) // Unix absolute or Windows drive
+        ? `file://${pluginPath}`
+        : pluginPath; // Relative path (shouldn't happen, but handle it)
+    
+    // Import with cache busting to ensure fresh reloads during dev
+    module = await import(`${normalizedPath}${cacheBuster}`);
     
     // Support both default export and named export
     const plugin = module.default || module;
@@ -86,18 +94,31 @@ export function discoverLocalPlugins(rootDir) {
   
   try {
     const items = readdirSync(pluginsDir);
+    console.log(`🔍 Scanning _plugins directory: found ${items.length} item(s)`);
     
     for (const item of items) {
       const itemPath = join(pluginsDir, item);
       const stat = statSync(itemPath);
       
-      // Only process .js and .mjs files
-      if (stat.isFile() && (item.endsWith('.js') || item.endsWith('.mjs'))) {
+      // Check extension case-insensitively
+      const lowerItem = item.toLowerCase();
+      const isJsFile = lowerItem.endsWith('.js') || lowerItem.endsWith('.mjs');
+      
+      // Debug: log what we found
+      if (stat.isFile()) {
+        console.log(`  📄 Found file: ${item} (${isJsFile ? 'plugin candidate' : 'skipped - not .js/.mjs'})`);
+      } else if (stat.isDirectory()) {
+        console.log(`  📁 Found directory: ${item} (skipped - plugins must be files)`);
+      }
+      
+      // Only process .js and .mjs files (case-insensitive)
+      if (stat.isFile() && isJsFile) {
         plugins.push({
           type: 'local',
           path: itemPath,
-          name: item.replace(/\.(js|mjs)$/, '')
+          name: item.replace(/\.(js|mjs)$/i, '') // Case-insensitive replacement
         });
+        console.log(`  ✓ Added plugin: ${item}`);
       }
     }
   } catch (err) {
@@ -250,9 +271,15 @@ export async function discoverPlugins(config) {
   
   // Filter by explicit plugin list if provided
   const explicitPlugins = config.plugins?.plugins;
-  if (explicitPlugins && Array.isArray(explicitPlugins)) {
+  if (explicitPlugins && Array.isArray(explicitPlugins) && explicitPlugins.length > 0) {
+    console.log(`🔍 Filtering plugins: only loading ${explicitPlugins.join(', ')}`);
     const explicitSet = new Set(explicitPlugins);
-    return discovered.filter(p => explicitSet.has(p.name));
+    const filtered = discovered.filter(p => explicitSet.has(p.name));
+    const filteredOut = discovered.filter(p => !explicitSet.has(p.name));
+    if (filteredOut.length > 0) {
+      console.log(`  ⚠️  Filtered out ${filteredOut.length} plugin(s): ${filteredOut.map(p => p.name).join(', ')}`);
+    }
+    return filtered;
   }
   
   return discovered;
@@ -262,9 +289,24 @@ export async function discoverPlugins(config) {
  * Load all discovered plugins
  */
 export async function loadPlugins(config) {
+  // Check if plugins are disabled
+  if (config.plugins?.enabled === false) {
+    console.log('🔌 Plugins are disabled in config');
+    return [];
+  }
+  
+  const rootDir = config.rootDir || process.cwd();
   const discovered = await discoverPlugins(config);
   
   if (discovered.length === 0) {
+    // Log that we checked but found nothing (helps with debugging)
+    const pluginsDir = join(rootDir, '_plugins');
+    const hasPluginsDir = existsSync(pluginsDir);
+    if (hasPluginsDir) {
+      console.log('🔌 No plugins discovered (directory exists but no valid plugins found)');
+    } else {
+      console.log('🔌 No plugins directory found, skipping plugin discovery');
+    }
     return [];
   }
   

package/lib/server.js

@@ -166,7 +166,8 @@ function setupWatcher(config, wss) {
     config.includesDir,
     join(config.rootDir, '_config.yml'),
     join(config.rootDir, '_config.json'),
-    join(config.rootDir, 'styles')
+    join(config.rootDir, 'styles'),
+    join(config.rootDir, '_plugins')
   ].filter(p => existsSync(p));
   
   const watcher = chokidar.watch(watchPaths, {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@terrymooreii/sia",
-  "version": "2.3.0",
+  "version": "2.3.1",
   "description": "A simple, powerful static site generator with markdown, front matter, and Nunjucks templates",
   "main": "lib/index.js",
   "bin": {