@kenjura/ursa 0.32.0 → 0.33.0

package/CHANGELOG.md

@@ -1,3 +1,8 @@
+# 0.33
+2025-12-10
+
+- Fixed the broke-ass menu
+
 # 0.32
 2025-12-10
 

package/meta/menu.js

@@ -257,6 +257,22 @@ document.addEventListener('DOMContentLoaded', () => {
                         caret.addEventListener('click', toggleExpand);
                     }
                     
+                    // If the link points to the current page, clicking it should toggle instead of navigate
+                    if (link) {
+                        link.addEventListener('click', (e) => {
+                            const linkHref = link.getAttribute('href');
+                            const currentHref = window.location.pathname;
+                            const normalizedLinkHref = linkHref.replace(/\/index\.html$/, '').replace(/\.html$/, '');
+                            const normalizedCurrentHref = currentHref.replace(/\/index\.html$/, '').replace(/\.html$/, '');
+                            
+                            if (normalizedLinkHref === normalizedCurrentHref) {
+                                // Already on this page - toggle instead of navigate
+                                toggleExpand(e);
+                            }
+                            // Otherwise, let the default navigation happen
+                        });
+                    }
+                    
                     row.addEventListener('click', (e) => {
                         if (!e.target.closest('a')) {
                             toggleExpand(e);

package/package.json

@@ -2,7 +2,7 @@
   "name": "@kenjura/ursa",
   "author": "Andrew London <andrew@kenjura.com>",
   "type": "module",
-  "version": "0.32.0",
+  "version": "0.33.0",
   "description": "static site generator from MD/wikitext/YML",
   "main": "lib/index.js",
   "bin": {

package/src/helper/automenu.js

@@ -137,10 +137,13 @@ function buildMenuData(tree, source, validPaths, parentPath = '') {
     let rawHref = null;
     if (hasChildren) {
       if (hasIndexFile(item.path)) {
-        rawHref = `/${relativePath}/index.html`;
+        // Construct proper path - relativePath already starts with /
+        const cleanPath = relativePath.startsWith('/') ? relativePath : '/' + relativePath;
+        rawHref = `${cleanPath}/index.html`.replace(/\/\//g, '/');
       }
     } else {
-      rawHref = `/${relativePath.replace(ext, '')}`;
+      const cleanPath = relativePath.startsWith('/') ? relativePath : '/' + relativePath;
+      rawHref = cleanPath.replace(ext, '');
     }
     
     // Resolve the href and check if target exists
@@ -173,7 +176,9 @@ function buildMenuData(tree, source, validPaths, parentPath = '') {
 }
 
 export async function getAutomenu(source, validPaths) {
-  const tree = dirTree(source);
+  const tree = dirTree(source, {
+    exclude: /[\/\\]\.|node_modules/,  // Exclude hidden folders (starting with .) and node_modules
+  });
   const menuData = buildMenuData(tree, source, validPaths);
   
   // Add home item with resolved href
@@ -205,10 +210,9 @@ function renderMenuLevel(items, level) {
     const hasChildrenClass = item.hasChildren ? ' has-children' : '';
     const hasChildrenIndicator = item.hasChildren ? '<span class="menu-more">⋯</span>' : '';
     const inactiveClass = item.inactive ? ' inactive' : '';
-    const debugText = item.debug ? ` [DEBUG: ${item.debug}]` : '';
     
     const labelHtml = item.href
-      ? `<a href="${item.href}" class="menu-label${inactiveClass}">${item.label}${debugText}</a>`
+      ? `<a href="${item.href}" class="menu-label${inactiveClass}">${item.label}</a>`
       : `<span class="menu-label">${item.label}</span>`;
     
     return `

package/src/jobs/generate.js

@@ -103,13 +103,14 @@ export async function generate({
 
   // read all articles, process them, copy them to build
   const articleExtensions = /\.(md|txt|yml)/;
+  const hiddenOrSystemDirs = /[\/\\]\.(?!\.)|[\/\\]node_modules[\/\\]/;  // Matches hidden folders (starting with .) or node_modules
   const allSourceFilenamesThatAreArticles = allSourceFilenames.filter(
-    (filename) => filename.match(articleExtensions)
+    (filename) => filename.match(articleExtensions) && !filename.match(hiddenOrSystemDirs)
   );
-  const allSourceFilenamesThatAreDirectories = await filterAsync(
+  const allSourceFilenamesThatAreDirectories = (await filterAsync(
     allSourceFilenames,
     (filename) => isDirectory(filename)
-  );
+  )).filter((filename) => !filename.match(hiddenOrSystemDirs));
 
   // Build set of valid internal paths for link validation (must be before menu)
   const validPaths = buildValidPaths(allSourceFilenamesThatAreArticles, source);