@herb-tools/formatter 0.8.0 → 0.8.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@herb-tools/formatter",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "description": "Auto-formatter for HTML+ERB templates with intelligent indentation, line wrapping, and ERB-aware pretty-printing.",
   "license": "MIT",
   "homepage": "https://herb-tools.dev",
@@ -35,10 +35,10 @@
     }
   },
   "dependencies": {
-    "@herb-tools/config": "0.8.0",
-    "@herb-tools/core": "0.8.0",
-    "@herb-tools/printer": "0.8.0",
-    "@herb-tools/rewriter": "0.8.0"
+    "@herb-tools/config": "0.8.1",
+    "@herb-tools/core": "0.8.1",
+    "@herb-tools/printer": "0.8.1",
+    "@herb-tools/rewriter": "0.8.1"
   },
   "files": [
     "package.json",

package/src/format-helpers.ts

@@ -194,7 +194,9 @@ export function isERBTag(text: string): boolean {
  * Check if a string ends with an ERB tag
  */
 export function endsWithERBTag(text: string): boolean {
-  return /%>$/.test(text.trim())
+  const trimmed = text.trim()
+
+  return /%>$/.test(trimmed) || /%>\S+$/.test(trimmed)
 }
 
 /**

package/src/format-printer.ts

@@ -1862,7 +1862,7 @@ export class FormatPrinter extends Printer {
     const firstWord = words[0]
     const firstChar = firstWord[0]
 
-    if (!/[a-zA-Z0-9.!?:;]/.test(firstChar)) {
+    if (/\s/.test(firstChar)) {
       return false
     }
 
@@ -1879,6 +1879,11 @@ export class FormatPrinter extends Printer {
         unit: { content: remainingText, type: 'text', isAtomic: false, breaksFlow: false },
         node: textNode
       })
+    } else if (endsWithWhitespace(textNode.content)) {
+      result.push({
+        unit: { content: ' ', type: 'text', isAtomic: false, breaksFlow: false },
+        node: textNode
+      })
     }
 
     return true
@@ -1955,10 +1960,8 @@ export class FormatPrinter extends Printer {
       const hasWhitespace = this.hasWhitespaceBeforeNode(children, lastProcessedIndex, index, child)
       const lastUnit = result[result.length - 1]
       const lastIsAtomic = lastUnit.unit.isAtomic && (lastUnit.unit.type === 'erb' || lastUnit.unit.type === 'inline')
-      const trimmed = child.content.trim()
-      const startsWithClosingPunct = trimmed.length > 0 && /^[.!?:;]/.test(trimmed)
 
-      if (lastIsAtomic && (!hasWhitespace || startsWithClosingPunct) && this.tryMergeTextAfterAtomic(result, child)) {
+      if (lastIsAtomic && !hasWhitespace && this.tryMergeTextAfterAtomic(result, child)) {
         return
       }
     }