Format with Syntax Tree

Author: kddnewton

.rubocop.yml

@@ -49,6 +49,9 @@ Style/GuardClause:
 Style/IdenticalConditionalBranches:
   Enabled: false
 
+Style/IfInsideElse:
+  Enabled: false
+
 Style/KeywordParametersOrder:
   Enabled: false
 

Rakefile

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
-require 'bundler/gem_tasks'
-require 'rake/testtask'
+require "bundler/gem_tasks"
+require "rake/testtask"
 
 Rake::TestTask.new(:test) do |t|
-  t.libs << 'test'
-  t.libs << 'lib'
-  t.test_files = FileList['test/**/*_test.rb']
+  t.libs << "test"
+  t.libs << "lib"
+  t.test_files = FileList["test/**/*_test.rb"]
 end
 
 task default: :test

lib/syntax_tree/cli.rb

@@ -83,9 +83,7 @@ def run(handler, filepath, source)
         warning = "[#{Color.yellow("warn")}] #{filepath}"
         formatted = handler.format(source)
 
-        if formatted != handler.format(formatted)
-          raise NonIdempotentFormatError
-        end
+        raise NonIdempotentFormatError if formatted != handler.format(formatted)
       rescue StandardError
         warn(warning)
         raise
@@ -300,13 +298,15 @@ def run(argv)
       def each_file(arguments)
         if $stdin.tty?
           arguments.each do |pattern|
-            Dir.glob(pattern).each do |filepath|
-              next unless File.file?(filepath)
-
-              handler = HANDLERS[File.extname(filepath)]
-              source = handler.read(filepath)
-              yield handler, filepath, source
-            end
+            Dir
+              .glob(pattern)
+              .each do |filepath|
+                next unless File.file?(filepath)
+
+                handler = HANDLERS[File.extname(filepath)]
+                source = handler.read(filepath)
+                yield handler, filepath, source
+              end
           end
         else
           yield HANDLERS[".rb"], :stdin, $stdin.read

lib/syntax_tree/formatter.rb

@@ -41,11 +41,12 @@ def format(node, stackable: true)
 
         # If the node has a stree-ignore comment right before it, then we're
         # going to just print out the node as it was seen in the source.
-        if leading.last&.ignore?
-          doc = text(source[node.location.start_char...node.location.end_char])
-        else
-          doc = node.format(self)
-        end
+        doc =
+          if leading.last&.ignore?
+            text(source[node.location.start_char...node.location.end_char])
+          else
+            node.format(self)
+          end
 
         # Print all comments that were found after the node.
         trailing.each do |comment|

lib/syntax_tree/language_server.rb

@@ -34,17 +34,37 @@ def run
         in { method: "shutdown" }
           store.clear
           return
-        in { method: "textDocument/didChange", params: { textDocument: { uri: }, contentChanges: [{ text: }, *] } }
+        in {
+             method: "textDocument/didChange",
+             params: { textDocument: { uri: }, contentChanges: [{ text: }, *] }
+           }
           store[uri] = text
-        in { method: "textDocument/didOpen", params: { textDocument: { uri:, text: } } }
+        in {
+             method: "textDocument/didOpen",
+             params: { textDocument: { uri:, text: } }
+           }
           store[uri] = text
-        in { method: "textDocument/didClose", params: { textDocument: { uri: } } }
+        in {
+             method: "textDocument/didClose", params: { textDocument: { uri: } }
+           }
           store.delete(uri)
-        in { method: "textDocument/formatting", id:, params: { textDocument: { uri: } } }
+        in {
+             method: "textDocument/formatting",
+             id:,
+             params: { textDocument: { uri: } }
+           }
           write(id: id, result: [format(store[uri])])
-        in { method: "textDocument/inlayHints", id:, params: { textDocument: { uri: } } }
+        in {
+             method: "textDocument/inlayHints",
+             id:,
+             params: { textDocument: { uri: } }
+           }
           write(id: id, result: inlay_hints(store[uri]))
-        in { method: "syntaxTree/visualizing", id:, params: { textDocument: { uri: } } }
+        in {
+             method: "syntaxTree/visualizing",
+             id:,
+             params: { textDocument: { uri: } }
+           }
           output = []
           PP.pp(SyntaxTree.parse(store[uri]), output)
           write(id: id, result: output.join)
@@ -61,15 +81,24 @@ def run
     def capabilities
       {
         documentFormattingProvider: true,
-        textDocumentSync: { change: 1, openClose: true }
+        textDocumentSync: {
+          change: 1,
+          openClose: true
+        }
       }
     end
 
     def format(source)
       {
         range: {
-          start: { line: 0, character: 0 },
-          end: { line: source.lines.size + 1, character: 0 }
+          start: {
+            line: 0,
+            character: 0
+          },
+          end: {
+            line: source.lines.size + 1,
+            character: 0
+          }
         },
         newText: SyntaxTree.format(source)
       }