Rescue exceptions in docstring parsing (castwide#830)

Author: castwide

lib/solargraph/source.rb

@@ -491,6 +491,10 @@ def parse_docstring comments
         # HACK: Pass a dummy code object to the parser for plugins that
         # expect it not to be nil
         YARD::Docstring.parser.parse(comments, YARD::CodeObjects::Base.new(:root, 'stub'))
+      rescue StandardError => e
+        Solargraph.logger.info "YARD failed to parse docstring: [#{e.class}] #{e.message}"
+        Solargraph.logger.debug "Unparsed comment: #{comments}"
+        YARD::Docstring.parser
       end
     end
   end

spec/source_spec.rb

@@ -317,4 +317,10 @@ class Foo
     )
     expect(source.string_ranges.length).to eq(4)
   end
+
+  it 'handles errors in docstrings' do
+    # YARD has a known problem with empty @overload tags
+    comments = "@overload\n@return [String]"
+    expect { Solargraph::Source.parse_docstring(comments) }.not_to raise_error
+  end
 end