w_syntax_tree-erb 0.9.0 → 0.9.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4b1234ed5afef9a96ebefb415733354ac530ffa9bb35b718195d36be9f9fed7c
-  data.tar.gz: 03c589bb628753e874f05aa6ea504f0d3d14e2057a7843d919abfd31f9fc66e9
+  metadata.gz: 9ebb5ab1bbcd632cf3e6b4dffa3eee540b6feff33ac8d0e7bec89de2b2171ebe
+  data.tar.gz: 3a11a343fb6ebd9b83e715e71e4dd615e997dc73e500b1b4b7f841055edc01db
 SHA512:
-  metadata.gz: cb5dd5cad343aabf4eaa8bd8cb177eeabb673cf68b8a016f36c13ee5ff2fbbb277a313800849cee940013355e430edca26d5d8541d566588f7902c41816d20a9
-  data.tar.gz: 3a147354e9fbef1bf89b604033c57f3746f1e17e69d050e7b4b41ffc409005000f4a7e5a12085e2704da71c4de4e16691795f6fe4600ba04de1106f5179c15b6
+  metadata.gz: e5440a04f66cc2117e4be78d6736afeee2aa28f2baf76208d4de2a4225a0b5d81597d4f834d9e228f31f1a8e7a7f4814a90e321cda5972d177c2ee05691ad086
+  data.tar.gz: 67eac9bcb1faac4736047b2870f80b8173073c74d5c74fe4c2900d0a917780c86405b6b7968abd40efff4d55efc2a8ddf7fabce8e0434260734ddc611c2f0312

data/CHANGELOG.md

@@ -6,11 +6,21 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) a
 
 ## [Unreleased]
 
-## [0.0.1] - 2023-05-25
+## [0.9.2] - 2023-06-30
+
+- Handle whitespace in HTML-strings using ERB-tags
+
+## [0.9.1] - 2023-06-28
+
+- Handle formatting of multi-line ERB-tags with more than one statement.
+
+## [0.9.0] - 2023-06-22
 
 ### Added
 
-- 🎉 First version based on syntax_tree-xml 🎉
+- 🎉 First version based on syntax_tree-xml 🎉.
+- Can format a lot of .html.erb-syntax and works as a plugin to syntax_tree.
+- This is still early and there are a lot of different weird syntaxes out there.
 
-[unreleased]: https://github.com/davidwessman/syntax_tree-erb/compare/v0.1.0...HEAD
-[0.0.1]: https://github.com/davidwessman/syntax_tree-erb/compare/b280a...v0.1.0
+[unreleased]: https://github.com/davidwessman/syntax_tree-erb/compare/v0.9.0...HEAD
+[0.9.0]: https://github.com/davidwessman/syntax_tree-erb/compare/419727a73af94057ca0980733e69ac8b4d52fdf4...v0.9.0

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    syntax_tree-erb (0.0.1)
+    w_syntax_tree-erb (0.9.2)
       prettier_print (>= 1.2.0)
       syntax_tree (>= 6.1.1)
 
@@ -9,7 +9,7 @@ GEM
   remote: https://rubygems.org/
   specs:
     docile (1.4.0)
-    minitest (5.18.0)
+    minitest (5.18.1)
     prettier_print (1.2.1)
     rake (13.0.6)
     simplecov (0.22.0)
@@ -32,7 +32,7 @@ DEPENDENCIES
   minitest
   rake
   simplecov
-  syntax_tree-erb!
+  w_syntax_tree-erb!
 
 BUNDLED WITH
    2.4.1

data/README.md

@@ -33,6 +33,8 @@ Add this line to your application's Gemfile:
 gem "w_syntax_tree-erb", "~> 0.9", require: false
 ```
 
+> I added the `w_` prefix to avoid conflicts if there will ever be an official `syntax_tree-erb` gem.
+
 ## Usage
 
 ```sh

data/lib/syntax_tree/erb/format.rb

@@ -11,7 +11,11 @@ module SyntaxTree
 
       # Visit a Token node.
       def visit_token(node)
-        q.text(node.value.strip)
+        if %i[text whitespace].include?(node.type)
+          q.text(node.value)
+        else
+          q.text(node.value.strip)
+        end
       end
 
       # Visit a Document node.
@@ -80,8 +84,7 @@ module SyntaxTree
           q.text(" ")
           visit(node.keyword)
         end
-
-        visit(node.content)
+        node.content.nil? ? q.text(" ") : visit(node.content)
 
         visit(node.closing_tag)
       end
@@ -104,29 +107,44 @@ module SyntaxTree
       end
 
       def visit_erb_content(node)
-        rows =
-          if node.value.is_a?(String)
-            node.value.split("\n")
-          else
-            formatter =
-              SyntaxTree::Formatter.new("", [], erb_print_width(node.value))
-            formatter.format(node.value.statements)
-            formatter.flush
-            formatter.output.join.split("\n")
-          end
+        if node.value.is_a?(String)
+          output_rows(node.value.split("\n"))
+        else
+          child_nodes = node.value&.statements&.child_nodes || []
 
-        if rows.size > 1
-          q.group do
+          if child_nodes.size == 1
             q.text(" ")
-            q.seplist(rows, -> { q.breakable(" ") }) { |row| q.text(row) }
+            q.seplist(child_nodes, -> { q.breakable("") }) do |child_node|
+              format_statement(child_node)
+            end
             q.text(" ")
+          elsif child_nodes.size > 1
+            q.indent do
+              q.breakable("")
+              q.seplist(child_nodes, -> { q.breakable("") }) do |child_node|
+                format_statement(child_node)
+              end
+            end
+            q.breakable
           end
+        end
+      end
+
+      def format_statement(statement)
+        formatter =
+          SyntaxTree::Formatter.new("", [], erb_print_width(statement))
+        formatter.format(statement)
+        formatter.flush
+        rows = formatter.output.join.split("\n")
+
+        output_rows(formatter.output.join.split("\n"))
+      end
+
+      def output_rows(rows)
+        if rows.size > 1
+          q.seplist(rows, -> { q.breakable("") }) { |row| q.text(row) }
         elsif rows.size == 1
-          q.text(" ")
           q.text(rows.first)
-          q.text(" ")
-        else
-          q.text(" ")
         end
       end
 
@@ -203,15 +221,10 @@ module SyntaxTree
         end
       end
 
-      def erb_print_width(syntax_tree)
-        statements = syntax_tree.statements.body
+      def erb_print_width(node)
         # Set the width to maximum if we have an IfNode or IfOp,
         # we cannot format them purely with SyntaxTree because the ERB-syntax will be unparseable.
-        if statements.any? { |node| check_for_if_statement(node) }
-          999_999
-        else
-          SyntaxTree::ERB::MAX_WIDTH
-        end
+        check_for_if_statement(node) ? 999_999 : SyntaxTree::ERB::MAX_WIDTH
       end
 
       def check_for_if_statement(node)

data/lib/syntax_tree/erb/nodes.rb

@@ -220,8 +220,19 @@ module SyntaxTree
 
       def initialize(opening_tag:, keyword:, content:, closing_tag:, location:)
         @opening_tag = opening_tag
-        @keyword = keyword
-        @content = ErbContent.new(value: content.map(&:value).join) if content
+        # prune whitespace from keyword
+        @keyword =
+          if keyword
+            Token.new(
+              type: keyword.type,
+              value: keyword.value.strip,
+              location: keyword.location
+            )
+          end
+        # Set content to nil if it is empty
+        content ||= []
+        content = content.map(&:value).join if content.is_a?(Array)
+        @content = ErbContent.new(value: content) unless content.strip.empty?
         @closing_tag = closing_tag
         @location = location
       end

data/lib/syntax_tree/erb/pretty_print.rb

@@ -108,12 +108,12 @@ module SyntaxTree
       end
 
       def visit_erb_end(node)
-        q.text("erb_end")
+        q.pp("erb_end")
       end
 
       # Visit an ErbContent node.
       def visit_erb_content(node)
-        q.text(node.value)
+        q.pp(node.value)
       end
 
       # Visit an Attribute node.
@@ -132,7 +132,7 @@ module SyntaxTree
       end
 
       def visit_erb_close(node)
-        visit_node("erb_close", node)
+        visit(node.closing)
       end
 
       def visit_erb_do_close(node)

data/lib/syntax_tree/erb/version.rb

@@ -2,6 +2,6 @@
 
 module SyntaxTree
   module ERB
-    VERSION = "0.9.0"
+    VERSION = "0.9.2"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: w_syntax_tree-erb
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.9.2
 platform: ruby
 authors:
 - Kevin Newton
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-06-22 00:00:00.000000000 Z
+date: 2023-06-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: prettier_print