aleman 1.3.1 → 1.4.0

package/ChangeLog

@@ -1,3 +1,15 @@
+2025.09.04, v1.4.0
+
+feature:
+- 2749907 aleman: menu: select: jsx-operator
+- 43294ec aleman: menu: jsx-operator
+
+2025.09.04, v1.3.2
+
+feature:
+- a94f8d9 aleman: menu: jsx-operator: add
+- 6a18625 aleman: menu: operator: add
+
 2025.09.04, v1.3.1
 
 feature:

package/menu/rules/build-menu/index.js

@@ -1,7 +1,10 @@
-import {template, operator} from 'putout';
+import {template} from 'putout';
 import {checkDataName} from '../check-data-name.js';
+import {
+    appendAttributeValue,
+    setAttributeValue,
+} from '../jsx-operator.js';
 
-const {setLiteralValue} = operator;
 const {entries} = Object;
 
 const noop = () => {};
@@ -80,37 +83,16 @@ export const traverse = ({options, push}) => ({
 });
 
 function setSubmenu(menuItem) {
-    const {attributes} = menuItem.openingElement;
-    
-    for (const attr of attributes) {
-        if (attr.name.name === 'className') {
-            setLiteralValue(attr.value, `${attr.value.value} menu-submenu`);
-            break;
-        }
-    }
+    appendAttributeValue(menuItem, 'className', 'menu-submenu');
 }
 
 function setDataMenuPath(key, name, menuItem) {
-    const {attributes} = menuItem.openingElement;
     const dataMenuPath = name ? `${name}.${key}` : key;
-    
-    for (const attr of attributes) {
-        if (attr.name.name === 'data-menu-path') {
-            setLiteralValue(attr.value, dataMenuPath);
-            break;
-        }
-    }
+    setAttributeValue(menuItem, 'data-menu-path', dataMenuPath);
 }
 
 function setIcon(name, menuItem) {
-    const {attributes} = menuItem.openingElement;
-    
-    for (const attr of attributes) {
-        if (attr.name.name === 'className') {
-            setLiteralValue(attr.value, `${attr.value.value} icon ${getIconName(name)}`);
-            break;
-        }
-    }
+    appendAttributeValue(menuItem, 'className', `icon ${getIconName(name)}`);
 }
 
 function getIconName(name) {

package/menu/rules/check-data-name.js

@@ -1,10 +1,7 @@
-export function checkDataName(path, dataName = 'menu') {
-    const {attributes} = path.node;
+import {getAttributeValue} from './jsx-operator.js';
+
+export function checkDataName(path, value = 'menu') {
+    const attribute = getAttributeValue(path, 'data-name');
     
-    for (const {name, value} of attributes) {
-        if (name.name === 'data-name')
-            return value.value === dataName;
-    }
-    
-    return false;
+    return attribute === value;
 }

package/menu/rules/hide-submenu/index.js

@@ -1,4 +1,6 @@
 import {operator} from 'putout';
+import {checkDataName} from '../check-data-name.js';
+import {getAttributePath} from '../jsx-operator.js';
 
 const {setLiteralValue} = operator;
 
@@ -23,31 +25,17 @@ export const traverse = ({push, options}) => ({
         if (showSubmenu)
             return;
         
-        const attributes = path.get('attributes');
         const openingElementPath = path.parentPath.parentPath.get('openingElement');
         
         if (!checkDataName(openingElementPath, name))
             return false;
         
-        for (const attr of attributes) {
-            const {name, value} = attr.node;
-            
-            if (name.name !== 'className')
-                continue;
-            
-            if (value.value.includes('menu-submenu-show'))
-                push(attr);
-        }
+        const attributePath = getAttributePath(path, 'className');
+        
+        if (!attributePath)
+            return;
+        
+        if (attributePath.node.value.value.includes('menu-submenu-show'))
+            push(attributePath);
     },
 });
-
-function checkDataName(path, dataName = 'menu') {
-    const {attributes} = path.node;
-    
-    for (const {name, value} of attributes) {
-        if (name.name === 'data-name')
-            return value.value === dataName;
-    }
-    
-    return false;
-}

package/menu/rules/jsx-operator.js

@@ -0,0 +1,78 @@
+import {operator} from 'putout';
+
+const {setLiteralValue} = operator;
+
+export function getAttributeValue(path, attributeName) {
+    const {attributes} = path.node;
+    
+    for (const {name, value} of attributes) {
+        if (name.name === attributeName)
+            return value.value;
+    }
+    
+    return '';
+}
+
+export function getAttributeNode(node, name) {
+    let result = null;
+    const {attributes} = node.openingElement;
+    
+    for (const attr of attributes) {
+        if (attr.name.name === name) {
+            result = attr;
+            break;
+        }
+    }
+    
+    return result;
+}
+
+export function getAttributePath(path, name) {
+    let result = null;
+    const attributes = path.get('attributes');
+    
+    for (const attr of attributes) {
+        if (attr.node.name.name === name) {
+            result = attr;
+            break;
+        }
+    }
+    
+    return result;
+}
+
+export function appendAttributeValue(path, name, value) {
+    const node = path.node || path;
+    const attributeNode = getAttributeNode(node, name);
+    
+    if (!attributeNode)
+        return;
+    
+    if (attributeNode.value.value.includes(value))
+        return;
+    
+    setLiteralValue(attributeNode.value, `${attributeNode.value.value} ${value}`);
+}
+
+export function setAttributeValue(node, name, value) {
+    const attributeNode = getAttributeNode(node, name);
+    
+    if (attributeNode)
+        setLiteralValue(attributeNode.value, value);
+}
+
+export function removeAttributeValue(path, name, attributeValue) {
+    if (!path)
+        return;
+    
+    const node = path.node || path;
+    const classAttribute = getAttributeNode(node, name);
+    
+    if (!classAttribute)
+        return;
+    
+    const {value} = classAttribute.value;
+    
+    if (value.includes(attributeValue))
+        setLiteralValue(classAttribute.value, value.replace(RegExp(`\\s?${attributeValue}`), ''));
+}

package/menu/rules/select/index.js

@@ -1,5 +1,10 @@
 import {operator, types} from 'putout';
 import {checkDataName} from '../check-data-name.js';
+import {
+    appendAttributeValue,
+    getAttributePath,
+    removeAttributeValue,
+} from '../jsx-operator.js';
 
 const {isJSXElement} = types;
 const {setLiteralValue} = operator;
@@ -53,75 +58,30 @@ export const traverse = ({options, push}) => ({
         if (!current)
             return;
         
-        for (const attr of current.get('openingElement.attributes')) {
-            const {name} = attr.node;
-            
-            if (name.name !== 'className')
-                continue;
-            
-            push({
-                path: attr,
-                current,
-                prev,
-                next,
-                showSubmenu,
-            });
-            break;
-        }
+        const currentOpeningElement = current.get('openingElement');
+        const attributePath = getAttributePath(currentOpeningElement, 'className');
+        
+        push({
+            path: attributePath,
+            current,
+            prev,
+            next,
+            showSubmenu,
+        });
     },
 });
 
 function unselect(path) {
-    if (!path)
-        return;
-    
-    for (const attr of path.get('openingElement.attributes')) {
-        const {name, value} = attr.node;
-        
-        if (name.name !== 'className')
-            continue;
-        
-        if (value.value.includes('menu-item-selected'))
-            setLiteralValue(value, value.value.replace(' menu-item-selected', ''));
-    }
+    removeAttributeValue(path, 'className', 'menu-item-selected');
 }
 
 function addShowSubmenu(path, {showSubmenu}) {
-    for (const attr of path.get('openingElement.attributes')) {
-        const {name, value} = attr.node;
-        
-        if (name.name !== 'className')
-            continue;
-        
-        const currentValue = value.value;
-        
-        if (showSubmenu && !currentValue.includes('menu-submenu-show')) {
-            setLiteralValue(value, `${currentValue} menu-submenu-show`);
-            break;
-        }
-        
-        if (!showSubmenu && currentValue.includes('menu-submenu-show')) {
-            setLiteralValue(value, currentValue.replace('menu-submenu-show', ''));
-            break;
-        }
-    }
+    if (showSubmenu)
+        return appendAttributeValue(path, 'className', 'menu-submenu-show');
+    
+    removeAttributeValue(path, 'className', 'menu-submenu-show');
 }
 
 function removeShowSubmenu(path) {
-    if (!path)
-        return;
-    
-    for (const attr of path.get('openingElement.attributes')) {
-        const {name, value} = attr.node;
-        
-        if (name.name !== 'className')
-            continue;
-        
-        const currentValue = value.value;
-        
-        if (currentValue.includes('menu-submenu-show')) {
-            setLiteralValue(value, currentValue.replace(' menu-submenu-show', ''));
-            break;
-        }
-    }
+    removeAttributeValue(path, 'className', 'menu-submenu-show');
 }

package/menu/rules/submenu/fixture/li.js

@@ -0,0 +1,3 @@
+<li data-menu-path="Upload" data-name="menu-item" className="menu-item icon icon-view">
+    <label data-menu-path="Upload">Upload</label>
+</li>

package/menu/rules/submenu/fixture/not-menu.js

@@ -0,0 +1,17 @@
+// second selected
+<ul data-name="menu" class="menu menu-hidden">
+    <li data-menu-path="Upload" data-name="menu-item" className="menu-item icon icon-view">
+        <label data-menu-path="Upload">Upload</label>
+    </li>
+    <li data-menu-path="New" data-name="menu-item" className="menu-item icon icon-edit menu-item-selected">
+        <label data-menu-path="New">New</label>
+            <ul data-name="world" class="menu menu-hidden">
+                <li data-menu-path="New.File" data-name="menu-item" className="menu-item icon icon-view">
+                    <label data-menu-path="New.File">File</label>
+                </li>
+                <li data-menu-path="New.Directory" data-name="menu-item" className="menu-item icon icon-edit">
+                    <label data-menu-path="New.Directory">Directory</label>
+                </li>
+            </ul>
+    </li>
+</ul>;

package/menu/rules/submenu/index.js

@@ -1,5 +1,10 @@
 import {operator, types} from 'putout';
 import {checkDataName} from '../check-data-name.js';
+import {
+    getAttributePath,
+    getAttributeValue,
+    removeAttributeValue,
+} from '../jsx-operator.js';
 
 const {isJSXElement} = types;
 const {setLiteralValue} = operator;
@@ -29,6 +34,9 @@ export const traverse = ({options, push}) => ({
         if (path.node.name.name !== 'li')
             return;
         
+        if (!isJSXElement(path.parentPath.parentPath))
+            return;
+        
         const {insideSubmenu = true, submenuIndex = 1} = options;
         const parentMenu = path.parentPath.parentPath.parentPath;
         
@@ -38,9 +46,6 @@ export const traverse = ({options, push}) => ({
         if (!isParentSelected(parentMenu))
             return;
         
-        if (!isJSXElement(path.parentPath.parentPath))
-            return;
-        
         const openingElementPath = path.parentPath.parentPath.get('openingElement');
         
         if (!checkDataName(openingElementPath))
@@ -58,7 +63,8 @@ export const traverse = ({options, push}) => ({
         if (!current)
             return;
         
-        const [currentClassPath] = getClassPath(current);
+        const currentOpeningElementPath = current.get('openingElement');
+        const currentClassPath = getAttributePath(currentOpeningElementPath, 'className');
         
         push({
             path: currentClassPath,
@@ -70,46 +76,15 @@ export const traverse = ({options, push}) => ({
 });
 
 function isParentSelected(path) {
-    const attributes = path.get('openingElement.attributes');
+    const openingElement = path.get('openingElement');
+    const classAttributeValue = getAttributeValue(openingElement, 'className');
     
-    for (const attr of attributes) {
-        const {name, value} = attr.node;
-        
-        if (name.name === 'className')
-            return value.value.includes('menu-item-selected');
-    }
-    
-    return false;
+    return classAttributeValue.includes('menu-item-selected');
 }
 
 function unselect(path) {
     if (!path)
         return;
     
-    for (const attr of path.get('openingElement.attributes')) {
-        const {name, value} = attr.node;
-        
-        if (name.name !== 'className')
-            continue;
-        
-        if (value.value.includes('menu-item-selected'))
-            setLiteralValue(value, value.value.replace(' menu-item-selected', ''));
-    }
-}
-
-function getClassPath(path) {
-    if (!path)
-        return [null, ''];
-    
-    for (const attr of path.get('openingElement.attributes')) {
-        const {name, value} = attr.node;
-        
-        if (name.name !== 'className')
-            continue;
-        
-        return [
-            attr,
-            value.value,
-        ];
-    }
+    removeAttributeValue(path, 'className', 'menu-item-selected');
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "aleman",
-  "version": "1.3.1",
+  "version": "1.4.0",
   "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout-based framework for web",