@herb-tools/formatter 0.4.2 → 0.4.3

package/dist/index.esm.js

@@ -129,7 +129,7 @@ class Token {
 }
 
 // NOTE: This file is generated by the templates/template.rb script and should not
-// be modified manually. See /Users/marcoroth/Development/herb-release-6/templates/javascript/packages/core/src/errors.ts.erb
+// be modified manually. See /Users/marcoroth/Development/herb-release-7/templates/javascript/packages/core/src/errors.ts.erb
 class HerbError {
     type;
     message;
@@ -579,7 +579,7 @@ function convertToUTF8(string) {
 }
 
 // NOTE: This file is generated by the templates/template.rb script and should not
-// be modified manually. See /Users/marcoroth/Development/herb-release-6/templates/javascript/packages/core/src/nodes.ts.erb
+// be modified manually. See /Users/marcoroth/Development/herb-release-7/templates/javascript/packages/core/src/nodes.ts.erb
 class Node {
     type;
     location;
@@ -2574,7 +2574,7 @@ function fromSerializedNode(node) {
 }
 
 // NOTE: This file is generated by the templates/template.rb script and should not
-// be modified manually. See /Users/marcoroth/Development/herb-release-6/templates/javascript/packages/core/src/visitor.ts.erb
+// be modified manually. See /Users/marcoroth/Development/herb-release-7/templates/javascript/packages/core/src/visitor.ts.erb
 class Visitor {
     visit(node) {
         if (!node)
@@ -2691,6 +2691,12 @@ class Printer extends Visitor {
     indentLevel = 0;
     inlineMode = false;
     isInComplexNesting = false;
+    static INLINE_ELEMENTS = new Set([
+        'a', 'abbr', 'acronym', 'b', 'bdo', 'big', 'br', 'cite', 'code',
+        'dfn', 'em', 'i', 'img', 'kbd', 'label', 'map', 'object', 'q',
+        'samp', 'small', 'span', 'strong', 'sub', 'sup',
+        'tt', 'var', 'del', 'ins', 'mark', 's', 'u', 'time', 'wbr'
+    ]);
     constructor(source, options) {
         super();
         this.source = source;
@@ -2711,7 +2717,7 @@ class Printer extends Visitor {
         else {
             this.visit(node);
         }
-        return this.lines.filter(Boolean).join("\n");
+        return this.lines.join("\n");
     }
     push(line) {
         this.lines.push(line);
@@ -2726,38 +2732,167 @@ class Printer extends Visitor {
         return " ".repeat(this.indentLevel * this.indentWidth);
     }
     /**
-     * Print an ERB tag (<% %> or <%= %>) with single spaces around inner content.
+     * Format ERB content with proper spacing around the inner content.
+     * Returns empty string if content is empty, otherwise wraps content with single spaces.
      */
-    printERBNode(node) {
+    formatERBContent(content) {
+        return content.trim() ? ` ${content.trim()} ` : "";
+    }
+    /**
+     * Check if a node is an ERB control flow node (if, unless, block, case, while, for)
+     */
+    isERBControlFlow(node) {
+        return node instanceof ERBIfNode || node.type === 'AST_ERB_IF_NODE' ||
+            node instanceof ERBUnlessNode || node.type === 'AST_ERB_UNLESS_NODE' ||
+            node instanceof ERBBlockNode || node.type === 'AST_ERB_BLOCK_NODE' ||
+            node instanceof ERBCaseNode || node.type === 'AST_ERB_CASE_NODE' ||
+            node instanceof ERBCaseMatchNode || node.type === 'AST_ERB_CASE_MATCH_NODE' ||
+            node instanceof ERBWhileNode || node.type === 'AST_ERB_WHILE_NODE' ||
+            node instanceof ERBForNode || node.type === 'AST_ERB_FOR_NODE';
+    }
+    /**
+     * Count total attributes including those inside ERB conditionals
+     */
+    getTotalAttributeCount(attributes, inlineNodes = []) {
+        let totalAttributeCount = attributes.length;
+        inlineNodes.forEach(node => {
+            if (this.isERBControlFlow(node)) {
+                const erbNode = node;
+                if (erbNode.statements) {
+                    totalAttributeCount += erbNode.statements.length;
+                }
+            }
+        });
+        return totalAttributeCount;
+    }
+    /**
+     * Extract HTML attributes from a list of nodes
+     */
+    extractAttributes(nodes) {
+        return nodes.filter((child) => child instanceof HTMLAttributeNode || child.type === 'AST_HTML_ATTRIBUTE_NODE');
+    }
+    /**
+     * Extract inline nodes (non-attribute, non-whitespace) from a list of nodes
+     */
+    extractInlineNodes(nodes) {
+        return nodes.filter(child => !(child instanceof HTMLAttributeNode || child.type === 'AST_HTML_ATTRIBUTE_NODE') &&
+            !(child instanceof WhitespaceNode || child.type === 'AST_WHITESPACE_NODE'));
+    }
+    /**
+     * Render attributes as a space-separated string
+     */
+    renderAttributesString(attributes) {
+        if (attributes.length === 0)
+            return "";
+        return ` ${attributes.map(attr => this.renderAttribute(attr)).join(" ")}`;
+    }
+    /**
+     * Determine if a tag should be rendered inline based on attribute count and other factors
+     */
+    shouldRenderInline(totalAttributeCount, inlineLength, indentLength, maxLineLength = this.maxLineLength, hasComplexERB = false, nestingDepth = 0, inlineNodesLength = 0) {
+        if (hasComplexERB)
+            return false;
+        // Special case: no attributes at all, always inline if it fits
+        if (totalAttributeCount === 0) {
+            return inlineLength + indentLength <= maxLineLength;
+        }
+        const basicInlineCondition = totalAttributeCount <= 3 &&
+            inlineLength + indentLength <= maxLineLength;
+        const erbInlineCondition = inlineNodesLength > 0 && totalAttributeCount <= 3;
+        return basicInlineCondition || erbInlineCondition;
+    }
+    /**
+     * Render multiline attributes for a tag
+     */
+    renderMultilineAttributes(tagName, attributes, inlineNodes = [], allChildren = [], isSelfClosing = false, isVoid = false, hasBodyContent = false) {
         const indent = this.indent();
-        const open = node.tag_opening?.value ?? "";
-        const close = node.tag_closing?.value ?? "";
-        let inner;
-        if (node.tag_opening && node.tag_closing) {
-            const [, openingEnd] = node.tag_opening.range.toArray();
-            const [closingStart] = node.tag_closing.range.toArray();
-            const rawInner = this.source.slice(openingEnd, closingStart);
-            inner = ` ${rawInner.trim()} `;
+        this.push(indent + `<${tagName}`);
+        this.withIndent(() => {
+            // Render children in order, handling both attributes and ERB nodes
+            allChildren.forEach(child => {
+                if (child instanceof HTMLAttributeNode || child.type === 'AST_HTML_ATTRIBUTE_NODE') {
+                    this.push(this.indent() + this.renderAttribute(child));
+                }
+                else if (!(child instanceof WhitespaceNode || child.type === 'AST_WHITESPACE_NODE')) {
+                    this.visit(child);
+                }
+            });
+        });
+        if (isSelfClosing) {
+            this.push(indent + "/>");
+        }
+        else if (isVoid) {
+            this.push(indent + ">");
+        }
+        else if (!hasBodyContent) {
+            this.push(indent + `></${tagName}>`);
         }
         else {
-            const txt = node.content?.value ?? "";
-            inner = txt.trim() ? ` ${txt.trim()} ` : "";
+            this.push(indent + ">");
         }
+    }
+    /**
+     * Print an ERB tag (<% %> or <%= %>) with single spaces around inner content.
+     */
+    printERBNode(node) {
+        const indent = this.inlineMode ? "" : this.indent();
+        const open = node.tag_opening?.value ?? "";
+        const close = node.tag_closing?.value ?? "";
+        const content = node.content?.value ?? "";
+        const inner = this.formatERBContent(content);
         this.push(indent + open + inner + close);
     }
     // --- Visitor methods ---
     visitDocumentNode(node) {
-        node.children.forEach(child => this.visit(child));
+        let lastWasMeaningful = false;
+        let hasHandledSpacing = false;
+        for (let i = 0; i < node.children.length; i++) {
+            const child = node.children[i];
+            if (child instanceof HTMLTextNode || child.type === 'AST_HTML_TEXT_NODE') {
+                const textNode = child;
+                const isWhitespaceOnly = textNode.content.trim() === "";
+                if (isWhitespaceOnly) {
+                    const hasPrevNonWhitespace = i > 0 && this.isNonWhitespaceNode(node.children[i - 1]);
+                    const hasNextNonWhitespace = i < node.children.length - 1 && this.isNonWhitespaceNode(node.children[i + 1]);
+                    const hasMultipleNewlines = textNode.content.includes('\n\n');
+                    if (hasPrevNonWhitespace && hasNextNonWhitespace && hasMultipleNewlines) {
+                        this.push("");
+                        hasHandledSpacing = true;
+                    }
+                    continue;
+                }
+            }
+            if (this.isNonWhitespaceNode(child) && lastWasMeaningful && !hasHandledSpacing) {
+                this.push("");
+            }
+            this.visit(child);
+            if (this.isNonWhitespaceNode(child)) {
+                lastWasMeaningful = true;
+                hasHandledSpacing = false;
+            }
+        }
     }
     visitHTMLElementNode(node) {
         const open = node.open_tag;
         const tagName = open.tag_name?.value ?? "";
         const indent = this.indent();
-        const attributes = open.children.filter((child) => child instanceof HTMLAttributeNode || child.type === 'AST_HTML_ATTRIBUTE_NODE');
-        const inlineNodes = open.children.filter(child => !(child instanceof HTMLAttributeNode || child.type === 'AST_HTML_ATTRIBUTE_NODE') &&
-            !(child instanceof WhitespaceNode || child.type === 'AST_WHITESPACE_NODE'));
-        const children = node.body.filter(child => !(child instanceof WhitespaceNode || child.type === 'AST_WHITESPACE_NODE') &&
-            !((child instanceof HTMLTextNode || child.type === 'AST_HTML_TEXT_NODE') && child?.content.trim() === ""));
+        const attributes = this.extractAttributes(open.children);
+        const inlineNodes = this.extractInlineNodes(open.children);
+        const hasTextFlow = this.isInTextFlowContext(null, node.body);
+        const children = node.body.filter(child => {
+            if (child instanceof WhitespaceNode || child.type === 'AST_WHITESPACE_NODE') {
+                return false;
+            }
+            if (child instanceof HTMLTextNode || child.type === 'AST_HTML_TEXT_NODE') {
+                const content = child.content;
+                if (hasTextFlow && content === " ") {
+                    return true;
+                }
+                return content.trim() !== "";
+            }
+            return true;
+        });
+        const isInlineElement = this.isInlineElement(tagName);
         const hasClosing = open.tag_closing?.value === ">" || open.tag_closing?.value === "/>";
         const isSelfClosing = open.tag_closing?.value === "/>";
         if (!hasClosing) {
@@ -2792,16 +2927,53 @@ class Printer extends Visitor {
                     }
                 }
                 else {
-                    const inlineResult = this.tryRenderInline(children, tagName);
+                    const inlineResult = this.tryRenderInline(children, tagName, 0, false, hasTextFlow);
                     if (inlineResult && (indent.length + inlineResult.length) <= this.maxLineLength) {
                         this.push(indent + inlineResult);
                         return;
                     }
+                    if (hasTextFlow) {
+                        const hasAnyNewlines = children.some(child => {
+                            if (child instanceof HTMLTextNode || child.type === 'AST_HTML_TEXT_NODE') {
+                                return child.content.includes('\n');
+                            }
+                            return false;
+                        });
+                        if (!hasAnyNewlines) {
+                            const fullInlineResult = this.tryRenderInlineFull(node, tagName, attributes, children);
+                            if (fullInlineResult) {
+                                const totalLength = indent.length + fullInlineResult.length;
+                                const maxNesting = this.getMaxNestingDepth(children, 0);
+                                const maxInlineLength = maxNesting <= 1 ? this.maxLineLength : 60;
+                                if (totalLength <= maxInlineLength) {
+                                    this.push(indent + fullInlineResult);
+                                    return;
+                                }
+                            }
+                        }
+                    }
+                }
+            }
+            if (hasTextFlow) {
+                const fullInlineResult = this.tryRenderInlineFull(node, tagName, [], children);
+                if (fullInlineResult) {
+                    const totalLength = indent.length + fullInlineResult.length;
+                    const maxNesting = this.getMaxNestingDepth(children, 0);
+                    const maxInlineLength = maxNesting <= 1 ? this.maxLineLength : 60;
+                    if (totalLength <= maxInlineLength) {
+                        this.push(indent + fullInlineResult);
+                        return;
+                    }
                 }
             }
             this.push(indent + `<${tagName}>`);
             this.withIndent(() => {
-                children.forEach(child => this.visit(child));
+                if (hasTextFlow) {
+                    this.visitTextFlowChildren(children);
+                }
+                else {
+                    children.forEach(child => this.visit(child));
+                }
             });
             if (!node.is_void && !isSelfClosing) {
                 this.push(indent + `</${tagName}>`);
@@ -2828,20 +3000,24 @@ class Printer extends Visitor {
             }
             return;
         }
-        const inline = this.renderInlineOpen(tagName, attributes, isSelfClosing, inlineNodes, open.children);
-        const singleAttribute = attributes[0];
-        singleAttribute &&
-            (singleAttribute.value instanceof HTMLAttributeValueNode || singleAttribute.value?.type === 'AST_HTML_ATTRIBUTE_VALUE_NODE') &&
-            singleAttribute.value?.children.length === 0;
-        const hasERBControlFlow = inlineNodes.some(node => node instanceof ERBIfNode || node.type === 'AST_ERB_IF_NODE' ||
-            node instanceof ERBUnlessNode || node.type === 'AST_ERB_UNLESS_NODE' ||
-            node instanceof ERBBlockNode || node.type === 'AST_ERB_BLOCK_NODE' ||
-            node instanceof ERBCaseNode || node.type === 'AST_ERB_CASE_NODE' ||
-            node instanceof ERBWhileNode || node.type === 'AST_ERB_WHILE_NODE' ||
-            node instanceof ERBForNode || node.type === 'AST_ERB_FOR_NODE');
-        const shouldKeepInline = (attributes.length <= 3 &&
-            inline.length + indent.length <= this.maxLineLength) ||
-            (inlineNodes.length > 0 && !hasERBControlFlow);
+        const hasERBControlFlow = inlineNodes.some(node => this.isERBControlFlow(node)) ||
+            open.children.some(node => this.isERBControlFlow(node));
+        const hasComplexERB = hasERBControlFlow && inlineNodes.some(node => {
+            if (node instanceof ERBIfNode || node.type === 'AST_ERB_IF_NODE') {
+                const erbNode = node;
+                if (erbNode.statements.length > 0 && erbNode.location) {
+                    const startLine = erbNode.location.start.line;
+                    const endLine = erbNode.location.end.line;
+                    return startLine !== endLine;
+                }
+                return false;
+            }
+            return false;
+        });
+        const inline = hasComplexERB ? "" : this.renderInlineOpen(tagName, attributes, isSelfClosing, inlineNodes, open.children);
+        const nestingDepth = this.getMaxNestingDepth(children, 0);
+        const totalAttributeCount = this.getTotalAttributeCount(attributes, inlineNodes);
+        const shouldKeepInline = this.shouldRenderInline(totalAttributeCount, inline.length, indent.length, this.maxLineLength, hasComplexERB, nestingDepth, inlineNodes.length);
         if (shouldKeepInline) {
             if (children.length === 0) {
                 if (isSelfClosing) {
@@ -2851,7 +3027,53 @@ class Printer extends Visitor {
                     this.push(indent + inline);
                 }
                 else {
-                    this.push(indent + inline.replace('>', `></${tagName}>`));
+                    let result = `<${tagName}`;
+                    result += this.renderAttributesString(attributes);
+                    if (inlineNodes.length > 0) {
+                        const currentIndentLevel = this.indentLevel;
+                        this.indentLevel = 0;
+                        const tempLines = this.lines;
+                        this.lines = [];
+                        inlineNodes.forEach(node => {
+                            const wasInlineMode = this.inlineMode;
+                            if (!this.isERBControlFlow(node)) {
+                                this.inlineMode = true;
+                            }
+                            this.visit(node);
+                            this.inlineMode = wasInlineMode;
+                        });
+                        const inlineContent = this.lines.join("");
+                        this.lines = tempLines;
+                        this.indentLevel = currentIndentLevel;
+                        result += inlineContent;
+                    }
+                    result += `></${tagName}>`;
+                    this.push(indent + result);
+                }
+                return;
+            }
+            if (isInlineElement && children.length > 0 && !hasERBControlFlow) {
+                const fullInlineResult = this.tryRenderInlineFull(node, tagName, attributes, children);
+                if (fullInlineResult) {
+                    const totalLength = indent.length + fullInlineResult.length;
+                    if (totalLength <= this.maxLineLength || totalLength <= 120) {
+                        this.push(indent + fullInlineResult);
+                        return;
+                    }
+                }
+            }
+            if (!isInlineElement && children.length > 0 && !hasERBControlFlow) {
+                this.push(indent + inline);
+                this.withIndent(() => {
+                    if (hasTextFlow) {
+                        this.visitTextFlowChildren(children);
+                    }
+                    else {
+                        children.forEach(child => this.visit(child));
+                    }
+                });
+                if (!node.is_void && !isSelfClosing) {
+                    this.push(indent + `</${tagName}>`);
                 }
                 return;
             }
@@ -2862,7 +3084,12 @@ class Printer extends Visitor {
                 this.push(indent + inline);
             }
             this.withIndent(() => {
-                children.forEach(child => this.visit(child));
+                if (hasTextFlow) {
+                    this.visitTextFlowChildren(children);
+                }
+                else {
+                    children.forEach(child => this.visit(child));
+                }
             });
             if (!node.is_void && !isSelfClosing) {
                 this.push(indent + `</${tagName}>`);
@@ -2870,28 +3097,8 @@ class Printer extends Visitor {
             return;
         }
         if (inlineNodes.length > 0 && hasERBControlFlow) {
-            this.push(indent + `<${tagName}`);
-            this.withIndent(() => {
-                open.children.forEach(child => {
-                    if (child instanceof HTMLAttributeNode || child.type === 'AST_HTML_ATTRIBUTE_NODE') {
-                        this.push(this.indent() + this.renderAttribute(child));
-                    }
-                    else if (!(child instanceof WhitespaceNode || child.type === 'AST_WHITESPACE_NODE')) {
-                        this.visit(child);
-                    }
-                });
-            });
-            if (isSelfClosing) {
-                this.push(indent + "/>");
-            }
-            else if (node.is_void) {
-                this.push(indent + ">");
-            }
-            else if (children.length === 0) {
-                this.push(indent + ">" + `</${tagName}>`);
-            }
-            else {
-                this.push(indent + ">");
+            this.renderMultilineAttributes(tagName, attributes, inlineNodes, open.children, isSelfClosing, node.is_void, children.length > 0);
+            if (!isSelfClosing && !node.is_void && children.length > 0) {
                 this.withIndent(() => {
                     children.forEach(child => this.visit(child));
                 });
@@ -2908,25 +3115,40 @@ class Printer extends Visitor {
             }
         }
         else {
-            this.push(indent + `<${tagName}`);
-            this.withIndent(() => {
-                attributes.forEach(attribute => {
-                    this.push(this.indent() + this.renderAttribute(attribute));
-                });
-            });
-            if (isSelfClosing) {
-                this.push(indent + "/>");
-            }
-            else if (node.is_void) {
-                this.push(indent + ">");
+            if (isInlineElement && children.length > 0) {
+                const fullInlineResult = this.tryRenderInlineFull(node, tagName, attributes, children);
+                if (fullInlineResult) {
+                    const totalLength = indent.length + fullInlineResult.length;
+                    if (totalLength <= this.maxLineLength || totalLength <= 120) {
+                        this.push(indent + fullInlineResult);
+                        return;
+                    }
+                }
             }
-            else if (children.length === 0) {
-                this.push(indent + ">" + `</${tagName}>`);
+            if (isInlineElement && children.length === 0) {
+                let result = `<${tagName}`;
+                result += this.renderAttributesString(attributes);
+                if (isSelfClosing) {
+                    result += " />";
+                }
+                else if (node.is_void) {
+                    result += ">";
+                }
+                else {
+                    result += `></${tagName}>`;
+                }
+                this.push(indent + result);
+                return;
             }
-            else {
-                this.push(indent + ">");
+            this.renderMultilineAttributes(tagName, attributes, inlineNodes, open.children, isSelfClosing, node.is_void, children.length > 0);
+            if (!isSelfClosing && !node.is_void && children.length > 0) {
                 this.withIndent(() => {
-                    children.forEach(child => this.visit(child));
+                    if (hasTextFlow) {
+                        this.visitTextFlowChildren(children);
+                    }
+                    else {
+                        children.forEach(child => this.visit(child));
+                    }
                 });
                 this.push(indent + `</${tagName}>`);
             }
@@ -2935,47 +3157,35 @@ class Printer extends Visitor {
     visitHTMLOpenTagNode(node) {
         const tagName = node.tag_name?.value ?? "";
         const indent = this.indent();
-        const attributes = node.children.filter((attribute) => attribute instanceof HTMLAttributeNode || attribute.type === 'AST_HTML_ATTRIBUTE_NODE');
+        const attributes = this.extractAttributes(node.children);
+        const inlineNodes = this.extractInlineNodes(node.children);
         const hasClosing = node.tag_closing?.value === ">";
         if (!hasClosing) {
             this.push(indent + `<${tagName}`);
             return;
         }
-        const inline = this.renderInlineOpen(tagName, attributes, node.is_void);
-        if (attributes.length === 0 || inline.length + indent.length <= this.maxLineLength) {
+        const inline = this.renderInlineOpen(tagName, attributes, node.is_void, inlineNodes, node.children);
+        const totalAttributeCount = this.getTotalAttributeCount(attributes, inlineNodes);
+        const shouldKeepInline = this.shouldRenderInline(totalAttributeCount, inline.length, indent.length, this.maxLineLength, false, 0, inlineNodes.length);
+        if (shouldKeepInline) {
             this.push(indent + inline);
             return;
         }
-        this.push(indent + `<${tagName}`);
-        this.withIndent(() => {
-            attributes.forEach(attribute => {
-                this.push(this.indent() + this.renderAttribute(attribute));
-            });
-        });
-        this.push(indent + (node.is_void ? "/>" : ">"));
+        this.renderMultilineAttributes(tagName, attributes, inlineNodes, node.children, false, node.is_void, false);
     }
     visitHTMLSelfCloseTagNode(node) {
         const tagName = node.tag_name?.value ?? "";
         const indent = this.indent();
-        const attributes = node.attributes.filter((attribute) => attribute instanceof HTMLAttributeNode || attribute.type === 'AST_HTML_ATTRIBUTE_NODE');
-        const inline = this.renderInlineOpen(tagName, attributes, true);
-        const singleAttribute = attributes[0];
-        singleAttribute &&
-            (singleAttribute.value instanceof HTMLAttributeValueNode || singleAttribute.value?.type === 'AST_HTML_ATTRIBUTE_VALUE_NODE') &&
-            singleAttribute.value?.children.length === 0;
-        const shouldKeepInline = attributes.length <= 3 &&
-            inline.length + indent.length <= this.maxLineLength;
+        const attributes = this.extractAttributes(node.attributes);
+        const inlineNodes = this.extractInlineNodes(node.attributes);
+        const inline = this.renderInlineOpen(tagName, attributes, true, inlineNodes, node.attributes);
+        const totalAttributeCount = this.getTotalAttributeCount(attributes, inlineNodes);
+        const shouldKeepInline = this.shouldRenderInline(totalAttributeCount, inline.length, indent.length, this.maxLineLength, false, 0, inlineNodes.length);
         if (shouldKeepInline) {
             this.push(indent + inline);
             return;
         }
-        this.push(indent + `<${tagName}`);
-        this.withIndent(() => {
-            attributes.forEach(attribute => {
-                this.push(this.indent() + this.renderAttribute(attribute));
-            });
-        });
-        this.push(indent + "/>");
+        this.renderMultilineAttributes(tagName, attributes, inlineNodes, node.attributes, true, false, false);
     }
     visitHTMLCloseTagNode(node) {
         const indent = this.indent();
@@ -2985,6 +3195,13 @@ class Printer extends Visitor {
         this.push(indent + open + name + close);
     }
     visitHTMLTextNode(node) {
+        if (this.inlineMode) {
+            const normalizedContent = node.content.replace(/\s+/g, ' ').trim();
+            if (normalizedContent) {
+                this.push(normalizedContent);
+            }
+            return;
+        }
         const indent = this.indent();
         let text = node.content.trim();
         if (!text)
@@ -3118,9 +3335,17 @@ class Printer extends Visitor {
     }
     visitERBInNode(node) {
         this.printERBNode(node);
+        this.withIndent(() => {
+            node.statements.forEach(stmt => this.visit(stmt));
+        });
     }
     visitERBCaseMatchNode(node) {
         this.printERBNode(node);
+        node.conditions.forEach(condition => this.visit(condition));
+        if (node.else_clause)
+            this.visit(node.else_clause);
+        if (node.end_node)
+            this.visit(node.end_node);
     }
     visitERBBlockNode(node) {
         const indent = this.indent();
@@ -3140,22 +3365,18 @@ class Printer extends Visitor {
             const open = node.tag_opening?.value ?? "";
             const content = node.content?.value ?? "";
             const close = node.tag_closing?.value ?? "";
-            this.lines.push(open + content + close);
-            if (node.statements.length > 0) {
+            const inner = this.formatERBContent(content);
+            this.lines.push(open + inner + close);
+            node.statements.forEach((child, _index) => {
                 this.lines.push(" ");
-            }
-            node.statements.forEach((child, index) => {
                 if (child instanceof HTMLAttributeNode || child.type === 'AST_HTML_ATTRIBUTE_NODE') {
                     this.lines.push(this.renderAttribute(child));
                 }
                 else {
                     this.visit(child);
                 }
-                if (index < node.statements.length - 1) {
-                    this.lines.push(" ");
-                }
             });
-            if (node.statements.length > 0) {
+            if (node.statements.length > 0 && node.end_node) {
                 this.lines.push(" ");
             }
             if (node.subsequent) {
@@ -3166,7 +3387,8 @@ class Printer extends Visitor {
                 const endOpen = endNode.tag_opening?.value ?? "";
                 const endContent = endNode.content?.value ?? "";
                 const endClose = endNode.tag_closing?.value ?? "";
-                this.lines.push(endOpen + endContent + endClose);
+                const endInner = this.formatERBContent(endContent);
+                this.lines.push(endOpen + endInner + endClose);
             }
         }
         else {
@@ -3195,7 +3417,6 @@ class Printer extends Visitor {
         });
     }
     visitERBCaseNode(node) {
-        this.indentLevel;
         const indent = this.indent();
         const open = node.tag_opening?.value ?? "";
         const content = node.content?.value ?? "";
@@ -3259,6 +3480,147 @@ class Printer extends Visitor {
             this.visit(node.end_node);
     }
     // --- Utility methods ---
+    isNonWhitespaceNode(node) {
+        if (node instanceof HTMLTextNode || node.type === 'AST_HTML_TEXT_NODE') {
+            return node.content.trim() !== "";
+        }
+        if (node instanceof WhitespaceNode || node.type === 'AST_WHITESPACE_NODE') {
+            return false;
+        }
+        return true;
+    }
+    /**
+     * Check if an element should be treated as inline based on its tag name
+     */
+    isInlineElement(tagName) {
+        return Printer.INLINE_ELEMENTS.has(tagName.toLowerCase());
+    }
+    /**
+     * Check if we're in a text flow context (parent contains mixed text and inline elements)
+     */
+    visitTextFlowChildren(children) {
+        const indent = this.indent();
+        let currentLineContent = "";
+        for (const child of children) {
+            if (child instanceof HTMLTextNode || child.type === 'AST_HTML_TEXT_NODE') {
+                const content = child.content;
+                let processedContent = content.replace(/\s+/g, ' ').trim();
+                if (processedContent) {
+                    const hasLeadingSpace = /^\s/.test(content);
+                    if (currentLineContent && hasLeadingSpace && !currentLineContent.endsWith(' ')) {
+                        currentLineContent += ' ';
+                    }
+                    currentLineContent += processedContent;
+                    const hasTrailingSpace = /\s$/.test(content);
+                    if (hasTrailingSpace && !currentLineContent.endsWith(' ')) {
+                        currentLineContent += ' ';
+                    }
+                    if ((indent.length + currentLineContent.length) > Math.max(this.maxLineLength, 120)) {
+                        this.visitTextFlowChildrenMultiline(children);
+                        return;
+                    }
+                }
+            }
+            else if (child instanceof HTMLElementNode || child.type === 'AST_HTML_ELEMENT_NODE') {
+                const element = child;
+                const openTag = element.open_tag;
+                const childTagName = openTag?.tag_name?.value || '';
+                if (this.isInlineElement(childTagName)) {
+                    const childInline = this.tryRenderInlineFull(element, childTagName, this.extractAttributes(openTag.children), element.body.filter(c => !(c instanceof WhitespaceNode || c.type === 'AST_WHITESPACE_NODE') &&
+                        !((c instanceof HTMLTextNode || c.type === 'AST_HTML_TEXT_NODE') && c?.content.trim() === "")));
+                    if (childInline) {
+                        currentLineContent += childInline;
+                        if ((indent.length + currentLineContent.length) > this.maxLineLength) {
+                            this.visitTextFlowChildrenMultiline(children);
+                            return;
+                        }
+                    }
+                    else {
+                        if (currentLineContent.trim()) {
+                            this.push(indent + currentLineContent.trim());
+                            currentLineContent = "";
+                        }
+                        this.visit(child);
+                    }
+                }
+                else {
+                    if (currentLineContent.trim()) {
+                        this.push(indent + currentLineContent.trim());
+                        currentLineContent = "";
+                    }
+                    this.visit(child);
+                }
+            }
+            else if (child instanceof ERBContentNode || child.type === 'AST_ERB_CONTENT_NODE') {
+                const oldLines = this.lines;
+                const oldInlineMode = this.inlineMode;
+                try {
+                    this.lines = [];
+                    this.inlineMode = true;
+                    this.visit(child);
+                    const erbContent = this.lines.join("");
+                    currentLineContent += erbContent;
+                    if ((indent.length + currentLineContent.length) > Math.max(this.maxLineLength, 120)) {
+                        this.visitTextFlowChildrenMultiline(children);
+                        return;
+                    }
+                }
+                finally {
+                    this.lines = oldLines;
+                    this.inlineMode = oldInlineMode;
+                }
+            }
+            else {
+                if (currentLineContent.trim()) {
+                    this.push(indent + currentLineContent.trim());
+                    currentLineContent = "";
+                }
+                this.visit(child);
+            }
+        }
+        if (currentLineContent.trim()) {
+            const finalLine = indent + currentLineContent.trim();
+            if (finalLine.length > Math.max(this.maxLineLength, 120)) {
+                this.visitTextFlowChildrenMultiline(children);
+                return;
+            }
+            this.push(finalLine);
+        }
+    }
+    visitTextFlowChildrenMultiline(children) {
+        children.forEach(child => this.visit(child));
+    }
+    isInTextFlowContext(parent, children) {
+        const hasTextContent = children.some(child => (child instanceof HTMLTextNode || child.type === 'AST_HTML_TEXT_NODE') &&
+            child.content.trim() !== "");
+        if (!hasTextContent) {
+            return false;
+        }
+        const nonTextChildren = children.filter(child => !(child instanceof HTMLTextNode || child.type === 'AST_HTML_TEXT_NODE'));
+        if (nonTextChildren.length === 0) {
+            return false;
+        }
+        const allInline = nonTextChildren.every(child => {
+            if (child instanceof ERBContentNode || child.type === 'AST_ERB_CONTENT_NODE') {
+                return true;
+            }
+            if (child instanceof HTMLElementNode || child.type === 'AST_HTML_ELEMENT_NODE') {
+                const element = child;
+                const openTag = element.open_tag;
+                const tagName = openTag?.tag_name?.value || '';
+                return this.isInlineElement(tagName);
+            }
+            return false;
+        });
+        if (!allInline) {
+            return false;
+        }
+        const maxNestingDepth = this.getMaxNestingDepth(children, 0);
+        if (maxNestingDepth > 2) {
+            return false;
+        }
+        return true;
+    }
     renderInlineOpen(name, attributes, selfClose, inlineNodes = [], allChildren = []) {
         const parts = attributes.map(attribute => this.renderAttribute(attribute));
         if (inlineNodes.length > 0) {
@@ -3295,7 +3657,9 @@ class Printer extends Visitor {
                 this.lines = [];
                 inlineNodes.forEach(node => {
                     const wasInlineMode = this.inlineMode;
-                    this.inlineMode = true;
+                    if (!this.isERBControlFlow(node)) {
+                        this.inlineMode = true;
+                    }
                     this.visit(node);
                     this.inlineMode = wasInlineMode;
                 });
@@ -3314,33 +3678,128 @@ class Printer extends Visitor {
         const equals = attribute.equals?.value ?? "";
         let value = "";
         if (attribute.value && (attribute.value instanceof HTMLAttributeValueNode || attribute.value?.type === 'AST_HTML_ATTRIBUTE_VALUE_NODE')) {
-            const attrValue = attribute.value;
-            const open_quote = (attrValue.open_quote?.value ?? "");
-            const close_quote = (attrValue.close_quote?.value ?? "");
-            const attribute_value = attrValue.children.map((attr) => {
-                if (attr instanceof HTMLTextNode || attr.type === 'AST_HTML_TEXT_NODE' || attr instanceof LiteralNode || attr.type === 'AST_LITERAL_NODE') {
-                    return attr.content;
+            const attributeValue = attribute.value;
+            let open_quote = attributeValue.open_quote?.value ?? "";
+            let close_quote = attributeValue.close_quote?.value ?? "";
+            let htmlTextContent = "";
+            const content = attributeValue.children.map((child) => {
+                if (child instanceof HTMLTextNode || child.type === 'AST_HTML_TEXT_NODE' || child instanceof LiteralNode || child.type === 'AST_LITERAL_NODE') {
+                    const textContent = child.content;
+                    htmlTextContent += textContent;
+                    return textContent;
                 }
-                else if (attr instanceof ERBContentNode || attr.type === 'AST_ERB_CONTENT_NODE') {
-                    const erbAttr = attr;
-                    return (erbAttr.tag_opening.value + erbAttr.content.value + erbAttr.tag_closing.value);
+                else if (child instanceof ERBContentNode || child.type === 'AST_ERB_CONTENT_NODE') {
+                    const erbAttribute = child;
+                    return erbAttribute.tag_opening.value + erbAttribute.content.value + erbAttribute.tag_closing.value;
                 }
                 return "";
             }).join("");
-            value = open_quote + attribute_value + close_quote;
+            if (open_quote === "" && close_quote === "") {
+                open_quote = '"';
+                close_quote = '"';
+            }
+            else if (open_quote === "'" && close_quote === "'" && !htmlTextContent.includes('"')) {
+                open_quote = '"';
+                close_quote = '"';
+            }
+            value = open_quote + content + close_quote;
         }
         return name + equals + value;
     }
+    /**
+     * Try to render a complete element inline including opening tag, children, and closing tag
+     */
+    tryRenderInlineFull(node, tagName, attributes, children) {
+        let result = `<${tagName}`;
+        result += this.renderAttributesString(attributes);
+        result += ">";
+        const childrenContent = this.tryRenderChildrenInline(children);
+        if (!childrenContent) {
+            return null;
+        }
+        result += childrenContent;
+        result += `</${tagName}>`;
+        return result;
+    }
+    /**
+     * Try to render just the children inline (without tags)
+     */
+    tryRenderChildrenInline(children) {
+        let result = "";
+        for (const child of children) {
+            if (child instanceof HTMLTextNode || child.type === 'AST_HTML_TEXT_NODE') {
+                const content = child.content;
+                const normalizedContent = content.replace(/\s+/g, ' ');
+                const hasLeadingSpace = /^\s/.test(content);
+                const hasTrailingSpace = /\s$/.test(content);
+                const trimmedContent = normalizedContent.trim();
+                if (trimmedContent) {
+                    let finalContent = trimmedContent;
+                    if (hasLeadingSpace && result && !result.endsWith(' ')) {
+                        finalContent = ' ' + finalContent;
+                    }
+                    if (hasTrailingSpace) {
+                        finalContent = finalContent + ' ';
+                    }
+                    result += finalContent;
+                }
+                else if (hasLeadingSpace || hasTrailingSpace) {
+                    if (result && !result.endsWith(' ')) {
+                        result += ' ';
+                    }
+                }
+            }
+            else if (child instanceof HTMLElementNode || child.type === 'AST_HTML_ELEMENT_NODE') {
+                const element = child;
+                const openTag = element.open_tag;
+                const childTagName = openTag?.tag_name?.value || '';
+                if (!this.isInlineElement(childTagName)) {
+                    return null;
+                }
+                const childInline = this.tryRenderInlineFull(element, childTagName, this.extractAttributes(openTag.children), element.body.filter(c => !(c instanceof WhitespaceNode || c.type === 'AST_WHITESPACE_NODE') &&
+                    !((c instanceof HTMLTextNode || c.type === 'AST_HTML_TEXT_NODE') && c?.content.trim() === "")));
+                if (!childInline) {
+                    return null;
+                }
+                result += childInline;
+            }
+            else {
+                const oldLines = this.lines;
+                const oldInlineMode = this.inlineMode;
+                const oldIndentLevel = this.indentLevel;
+                try {
+                    this.lines = [];
+                    this.inlineMode = true;
+                    this.indentLevel = 0;
+                    this.visit(child);
+                    result += this.lines.join("");
+                }
+                finally {
+                    this.lines = oldLines;
+                    this.inlineMode = oldInlineMode;
+                    this.indentLevel = oldIndentLevel;
+                }
+            }
+        }
+        return result.trim();
+    }
     /**
      * Try to render children inline if they are simple enough.
      * Returns the inline string if possible, null otherwise.
      */
-    tryRenderInline(children, tagName, depth = 0) {
-        if (children.length > 10) {
+    tryRenderInline(children, tagName, depth = 0, forceInline = false, hasTextFlow = false) {
+        if (!forceInline && children.length > 10) {
             return null;
         }
         const maxNestingDepth = this.getMaxNestingDepth(children, 0);
-        if (maxNestingDepth > 1) {
+        let maxAllowedDepth = forceInline ? 5 : (tagName && ['h1', 'h2', 'h3', 'h4', 'h5', 'h6', 'div'].includes(tagName) ? 1 : 2);
+        if (hasTextFlow && maxNestingDepth >= 2) {
+            const roughContentLength = this.estimateContentLength(children);
+            if (roughContentLength > 47) {
+                maxAllowedDepth = 1;
+            }
+        }
+        if (!forceInline && maxNestingDepth > maxAllowedDepth) {
             this.isInComplexNesting = true;
             return null;
         }
@@ -3354,8 +3813,9 @@ class Printer extends Visitor {
             else if (child instanceof HTMLElementNode || child.type === 'AST_HTML_ELEMENT_NODE') {
                 const element = child;
                 const openTag = element.open_tag;
-                const attributes = openTag.children.filter((child) => child instanceof HTMLAttributeNode || child.type === 'AST_HTML_ATTRIBUTE_NODE');
-                if (attributes.length > 0) {
+                const elementTagName = openTag?.tag_name?.value || '';
+                const isInlineElement = this.isInlineElement(elementTagName);
+                if (!isInlineElement) {
                     return null;
                 }
             }
@@ -3378,10 +3838,8 @@ class Printer extends Visitor {
                     const element = child;
                     const openTag = element.open_tag;
                     const childTagName = openTag?.tag_name?.value || '';
-                    const attributes = openTag.children.filter((child) => child instanceof HTMLAttributeNode || child.type === 'AST_HTML_ATTRIBUTE_NODE');
-                    const attributesString = attributes.length > 0
-                        ? ' ' + attributes.map(attr => this.renderAttribute(attr)).join(' ')
-                        : '';
+                    const attributes = this.extractAttributes(openTag.children);
+                    const attributesString = this.renderAttributesString(attributes);
                     const elementContent = this.renderElementInline(element);
                     content += `<${childTagName}${attributesString}>${elementContent}</${childTagName}>`;
                 }
@@ -3390,7 +3848,7 @@ class Printer extends Visitor {
                     const open = erbNode.tag_opening?.value ?? "";
                     const erbContent = erbNode.content?.value ?? "";
                     const close = erbNode.tag_closing?.value ?? "";
-                    content += `${open} ${erbContent.trim()} ${close}`;
+                    content += `${open}${this.formatERBContent(erbContent)}${close}`;
                 }
             }
             content = content.replace(/\s+/g, ' ').trim();
@@ -3401,6 +3859,28 @@ class Printer extends Visitor {
             this.inlineMode = oldInlineMode;
         }
     }
+    /**
+     * Estimate the total content length of children nodes for decision making.
+     */
+    estimateContentLength(children) {
+        let length = 0;
+        for (const child of children) {
+            if (child instanceof HTMLTextNode || child.type === 'AST_HTML_TEXT_NODE') {
+                length += child.content.length;
+            }
+            else if (child instanceof HTMLElementNode || child.type === 'AST_HTML_ELEMENT_NODE') {
+                const element = child;
+                const openTag = element.open_tag;
+                const tagName = openTag?.tag_name?.value || '';
+                length += tagName.length + 5; // Rough estimate for tag overhead
+                length += this.estimateContentLength(element.body);
+            }
+            else if (child instanceof ERBContentNode || child.type === 'AST_ERB_CONTENT_NODE') {
+                length += child.content?.value.length || 0;
+            }
+        }
+        return length;
+    }
     /**
      * Calculate the maximum nesting depth in a subtree of nodes.
      */
@@ -3432,10 +3912,8 @@ class Printer extends Visitor {
                 const childElement = child;
                 const openTag = childElement.open_tag;
                 const childTagName = openTag?.tag_name?.value || '';
-                const attributes = openTag.children.filter((child) => child instanceof HTMLAttributeNode || child.type === 'AST_HTML_ATTRIBUTE_NODE');
-                const attributesString = attributes.length > 0
-                    ? ' ' + attributes.map(attr => this.renderAttribute(attr)).join(' ')
-                    : '';
+                const attributes = this.extractAttributes(openTag.children);
+                const attributesString = this.renderAttributesString(attributes);
                 const childContent = this.renderElementInline(childElement);
                 content += `<${childTagName}${attributesString}>${childContent}</${childTagName}>`;
             }
@@ -3444,7 +3922,7 @@ class Printer extends Visitor {
                 const open = erbNode.tag_opening?.value ?? "";
                 const erbContent = erbNode.content?.value ?? "";
                 const close = erbNode.tag_closing?.value ?? "";
-                content += `${open} ${erbContent.trim()} ${close}`;
+                content += `${open}${this.formatERBContent(erbContent)}${close}`;
             }
         }
         return content.replace(/\s+/g, ' ').trim();