@terrymooreii/sia 2.3.0 → 2.3.2

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/content.js

@@ -402,10 +402,66 @@ export function getDateFromFilename(filename) {
   return null;
 }
 
+/**
+ * Convert relative image and link paths to absolute paths based on base URL
+ * @param {string} html - HTML content with potential relative paths
+ * @param {string} baseUrl - Base URL for the content item (e.g., '/blog/my-post/')
+ * @returns {string} HTML with absolute paths
+ */
+function fixRelativePaths(html, baseUrl) {
+  if (!html || !baseUrl) return html;
+  
+  // Normalize baseUrl to ensure it ends with a slash
+  const normalizedBaseUrl = baseUrl.endsWith('/') ? baseUrl : baseUrl + '/';
+  
+  // Helper function to convert relative path to absolute
+  const makeAbsolute = (path) => {
+    // Skip if it's already an absolute URL (http://, https://) or absolute path (/)
+    if (/^(https?:|\/|#|mailto:)/.test(path)) {
+      return path;
+    }
+    
+    // Convert relative path to absolute path
+    // Remove leading ./ if present
+    const cleanPath = path.replace(/^\.\//, '');
+    
+    // Combine base URL with path
+    return normalizedBaseUrl + cleanPath;
+  };
+  
+  // Fix relative image paths
+  html = html.replace(
+    /<img\s+([^>]*?)(?:src\s*=\s*(["'])([^"']+)\2)([^>]*)>/gi,
+    (match, beforeAttrs, quote, src, afterAttrs) => {
+      const absolutePath = makeAbsolute(src);
+      
+      // Reconstruct the img tag with the absolute path
+      const before = beforeAttrs ? beforeAttrs.trim() + ' ' : '';
+      const after = afterAttrs ? ' ' + afterAttrs.trim() : '';
+      return `<img ${before}src=${quote}${absolutePath}${quote}${after}>`;
+    }
+  );
+  
+  // Fix relative link paths (but skip anchor links and external URLs)
+  html = html.replace(
+    /<a\s+([^>]*?)(?:href\s*=\s*(["'])([^"']+)\2)([^>]*)>/gi,
+    (match, beforeAttrs, quote, href, afterAttrs) => {
+      const absolutePath = makeAbsolute(href);
+      
+      // Reconstruct the link tag with the absolute path
+      const before = beforeAttrs ? beforeAttrs.trim() + ' ' : '';
+      const after = afterAttrs ? ' ' + afterAttrs.trim() : '';
+      return `<a ${before}href=${quote}${absolutePath}${quote}${after}>`;
+    }
+  );
+  
+  return html;
+}
+
 /**
  * Parse a markdown file with front matter
  */
-export async function parseContent(filePath) {
+export async function parseContent(filePath, options = {}) {
   let content = readFileSync(filePath, 'utf-8');
   
   // Execute beforeParse hook
@@ -546,6 +602,11 @@ export async function loadCollection(config, collectionName) {
         item.url = basePath + permalink;
         item.outputPath = join(config.outputDir, permalink, 'index.html');
         
+        // Fix relative image and link paths in content and excerptHtml
+        // This ensures images and links work on both individual pages and list pages
+        item.content = fixRelativePaths(item.content, item.url);
+        item.excerptHtml = fixRelativePaths(item.excerptHtml, item.url);
+        
         return item;
       } catch (err) {
         console.error(`Error parsing ${filePath}:`, err.message);

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.2",
   "description": "A simple, powerful static site generator with markdown, front matter, and Nunjucks templates",
   "main": "lib/index.js",
   "bin": {