Treat tables and definition lists as block elements

Their tags were missing from block_element?, so tables and definition
lists - most of the CSS and HTML reference content - collapsed into runs
like "NameTypefooString".

lib/mcp/server.rb

@@ -233,7 +233,9 @@ def self.html_to_text(html)
 
     def self.block_element?(tag_name)
       return false unless tag_name
-      %w(p div h1 h2 h3 h4 h5 h6 ul ol li blockquote pre br).include?(tag_name.downcase)
+      %w(p div h1 h2 h3 h4 h5 h6 ul ol li dl dt dd
+         table caption thead tbody tfoot tr th td
+         blockquote pre br).include?(tag_name.downcase)
     end
 
     def self.search_docset(app_settings, slug, query, args = {})

test/files/docs/mcp_fixture/db.json

@@ -1 +1 @@
-{"array/push":"<h1>Array#push</h1> <p>Appends &amp; returns the array.</p>","array/pop":"<h1>Array#pop</h1> <p>Removes the last element.</p>"}
+{"array/push":"<h1>Array#push</h1> <p>Appends &amp; returns the array.</p>","array/pop":"<h1>Array#pop</h1> <p>Removes the last element.</p>","array/table":"<h1>Attributes</h1> <table><tr><th>Name</th><th>Type</th></tr><tr><td>foo</td><td>String</td></tr></table> <dl><dt>bar</dt><dd>A thing.</dd></dl>"}

test/mcp_test.rb

@@ -177,6 +177,15 @@ def rpc(method, params = nil, id: 1)
       assert_includes result['content'].first['text'], 'Removes the last element.'
     end
 
+    it 'separates table cells and definition lists in the extracted text' do
+      args = { 'slug' => 'mcp_fixture', 'path' => 'array/table' }
+      result = rpc('tools/call', { 'name' => 'devdocs_get_page', 'arguments' => args })['result']
+      text = result['content'].first['text']
+      refute_includes text, 'NameType'
+      refute_includes text, 'fooString'
+      refute_includes text, 'barA thing.'
+    end
+
     it 'returns error for invalid slug in search (path traversal protection)' do
       args = { 'slug' => '../../../etc/passwd', 'query' => 'test' }
       response = rpc('tools/call', { 'name' => 'devdocs_search', 'arguments' => args })