reverse_markdown 2.1.1 → 3.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cffa2a16f5ad4187cb23a2e6a1015121b943f1c83a688308cf0a480ec15d3c49
-  data.tar.gz: 6941ad7c07005ff4d5c115fd894ee2a0cf40a09367833268f4c55c018f30263a
+  metadata.gz: d374ce991c236ccd344ef88d17531e3dc845bc4eb76015f5237f2646be1b3b66
+  data.tar.gz: 3bb3e85c76d512647fe7c70f983b22113775c2ab02bc7410dec85cfd06c46cb8
 SHA512:
-  metadata.gz: 620ce03beae7238accdc6836e025dc7b19fe308a7bcbcd7143dc7472d09e7d25758a3f6c2080d5d75e60fee662bc87ab200f177b5b6f41fa5958bde4b569a49e
-  data.tar.gz: ea69f20a1e70e4f41d6024502af362351ccaf6ec8936b4dcadc2414202e994cdf8ddf08ebbd829c0722c373bbe7e87a223c3c913bf6d845be9308ef418f3be75
+  metadata.gz: 1c860d94f4b07ae28cd2553e86c6538ebcaecd861a25bb695879ed2f69b6da6459c2ff9884d7c86127c18dac6217ddd3b41ee8e069eaa012e04b9d6d280cebd4
+  data.tar.gz: 1025d7a735a38f59d2e921d5e7fb29e41b007a238eca7052bba16b319316695a50dc15f1729b47f2c27887244000b0c5a2b3d386be1765ee0c37578f5be2978d

data/.github/workflows/ci.yml

@@ -0,0 +1,35 @@
+name: CI
+
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby-version: [ '2.7', '3.0', '3.1', '3.2', '3.3', '4.0', 'jruby-9.4' ]
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby-version }}
+          bundler-cache: true
+
+      - name: Install dependencies
+        run: bundle install
+
+      - name: Run tests
+        run: bundle exec rspec
+
+      - name: Run tests
+        run: bundle exec rspec

data/.tool-versions

@@ -0,0 +1 @@
+ruby 3.3.5

data/CHANGELOG.md

@@ -1,6 +1,22 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
+## 3.0.2 - January 2026
+- Add support for ruby 4.0
+- Some cleanup of test cases, thanks @joelhawksley, see #107
+- Fix whitespace collapsing between inline elements, see #34
+- Fix whitespace around links, see #91
+- Split emphasis markers at paragraph breaks, merge heading lines, see #95
+
+## 3.0.1 - December 2025
+- Use https instead of http
+
+## 3.0.0 - October 2024
+- BREAKING: Dropped support for ruby 2.6.0 and lower
+- Bugfix for missing newline behind `ol`, thanks @Kevinrob, see #104
+- Add support for `iframe` tags, thanks @gagandeepsinghj, see #102
+- Support for frozen string literals, thanks @pat, see #105
+
 ## 2.1.1 - October 2021
 - Fixes unintentional newline characters within lists with paragraphs, thanks @diogoosorio, see #93
 - Lets \n to be present in <pre> tag. solves #77 #78, thanks @shivabhusal

data/Gemfile

@@ -1,4 +1,4 @@
-source "http://rubygems.org"
+source "https://rubygems.org"
 
 # Specify your gem's dependencies in reverse_markdown.gemspec
 gemspec

data/README.md

@@ -2,16 +2,22 @@
 
 Transform html into markdown. Useful for example if you want to import html into your markdown based application.
 
-[![Build Status](https://secure.travis-ci.org/xijo/reverse_markdown.svg?branch=master)](https://travis-ci.org/xijo/reverse_markdown) [![Gem Version](https://badge.fury.io/rb/reverse_markdown.svg)](http://badge.fury.io/rb/reverse_markdown) [![Code Climate](https://codeclimate.com/github/xijo/reverse_markdown.svg)](https://codeclimate.com/github/xijo/reverse_markdown) [![Code Climate](https://codeclimate.com/github/xijo/reverse_markdown/coverage.png)](https://codeclimate.com/github/xijo/reverse_markdown)
+![Build Status](https://github.com/xijo/reverse_markdown/actions/workflows/ci.yml/badge.svg) [![Gem Version](https://badge.fury.io/rb/reverse_markdown.svg)](http://badge.fury.io/rb/reverse_markdown) [![Code Climate](https://codeclimate.com/github/xijo/reverse_markdown.svg)](https://codeclimate.com/github/xijo/reverse_markdown) [![Test Coverage](https://api.codeclimate.com/v1/badges/6ce481ba7ae6f57dc4d3/test_coverage)](https://codeclimate.com/github/xijo/reverse_markdown/test_coverage)
 
 ## Changelog
 
 See [Change Log](CHANGELOG.md)
 
+## Limitations
+
+A perfect HTML to Markdown conversion is not possible. HTML is far more expressive than Markdown - it supports tables with merged cells, arbitrary nesting, inline styles, and countless other features that have no Markdown equivalent.
+
+This gem aims to provide good enough defaults for most common cases. It handles standard content well but does not attempt to solve every edge case. If you have highly specific conversion needs, you can [write custom converters](https://github.com/xijo/reverse_markdown/wiki/Write-your-own-converter) to handle them.
+
 ## Requirements
 
 1. [Nokogiri](http://nokogiri.org/)
-2. Ruby 2.0.0 or higher
+2. Ruby 2.7.0 or higher
 
 ## Installation
 

data/lib/reverse_markdown/converters/base.rb

@@ -2,7 +2,7 @@ module ReverseMarkdown
   module Converters
     class Base
       def treat_children(node, state)
-        node.children.inject('') do |memo, child|
+        node.children.inject(+'') do |memo, child|
           memo << treat(child, state)
         end
       end
@@ -15,10 +15,34 @@ module ReverseMarkdown
         string.gsub(/(?<!\\)[*_]/, '*' => '\*', '_' => '\_')
       end
 
+      # Wrap content with markers (e.g., ** or _), splitting at paragraph breaks
+      # so markers don't span across breaks (which breaks markdown rendering)
+      def wrap_with_markers(content, marker)
+        # Split on paragraph breaks, preserving the breaks
+        segments = content.split(/(\s*\n\s*\n\s*)/)
+
+        segments.map.with_index do |segment, i|
+          if i.odd?  # This is a break segment (captured delimiter)
+            segment
+          elsif segment.strip.empty?
+            segment
+          else
+            # Wrap with markers, preserving border whitespace
+            leading = segment[/\A\s*/]
+            trailing = segment[/\s*\z/]
+            "#{leading}#{marker}#{segment.strip}#{marker}#{trailing}"
+          end
+        end.join
+      end
+
       def extract_title(node)
         title = escape_keychars(node['title'].to_s)
         title.empty? ? '' : %[ "#{title}"]
       end
+
+      def extract_src(node)
+        node['src'].to_s.empty? ? '' : node['src'].to_s
+      end
     end
   end
 end

data/lib/reverse_markdown/converters/blockquote.rb

@@ -4,7 +4,7 @@ module ReverseMarkdown
       def convert(node, state = {})
         content = treat_children(node, state).strip
         content = ReverseMarkdown.cleaner.remove_newlines(content)
-        "\n\n> " << content.lines.to_a.join('> ') << "\n\n"
+        +"\n\n> " << content.lines.to_a.join('> ') << "\n\n"
       end
     end
 

data/lib/reverse_markdown/converters/div.rb

@@ -2,7 +2,7 @@ module ReverseMarkdown
   module Converters
     class Div < Base
       def convert(node, state = {})
-        "\n" << treat_children(node, state) << "\n"
+        +"\n" << treat_children(node, state) << "\n"
       end
     end
 

data/lib/reverse_markdown/converters/em.rb

@@ -6,7 +6,7 @@ module ReverseMarkdown
         if content.strip.empty? || state[:already_italic]
           content
         else
-          "#{content[/^\s*/]}_#{content.strip}_#{content[/\s*$/]}"
+          wrap_with_markers(content, '_')
         end
       end
     end

data/lib/reverse_markdown/converters/figcaption.rb

@@ -5,7 +5,7 @@ module ReverseMarkdown
         if node.text.strip.empty?
           ""
         else
-          "\n" << "_#{node.text.strip}_" << "\n"
+          +"\n" << "_#{node.text.strip}_" << "\n"
         end
       end
     end

data/lib/reverse_markdown/converters/h.rb

@@ -3,7 +3,10 @@ module ReverseMarkdown
     class H < Base
       def convert(node, state = {})
         prefix = '#' * node.name[/\d/].to_i
-        ["\n", prefix, ' ', treat_children(node, state), "\n"].join
+        content = treat_children(node, state).strip
+        # Merge lines into one (markdown headings can't span multiple lines)
+        content = content.split(/\s*\n\s*/).join(' ')
+        "\n#{prefix} #{content}\n"
       end
     end
 

data/lib/reverse_markdown/converters/iframe.rb

@@ -0,0 +1,11 @@
+module ReverseMarkdown
+  module Converters
+    class Iframe < Base
+      def convert(node, state = {})
+        extract_src(node)
+      end
+    end
+
+    register :iframe, Iframe.new
+  end
+end

data/lib/reverse_markdown/converters/ol.rb

@@ -3,7 +3,7 @@ module ReverseMarkdown
     class Ol < Base
       def convert(node, state = {})
         ol_count = state.fetch(:ol_count, 0) + 1
-        "\n" << treat_children(node, state.merge(ol_count: ol_count))
+        +"\n" << treat_children(node, state.merge(ol_count: ol_count)) << "\n"
       end
     end
 

data/lib/reverse_markdown/converters/p.rb

@@ -2,7 +2,7 @@ module ReverseMarkdown
   module Converters
     class P < Base
       def convert(node, state = {})
-        "\n\n" << treat_children(node, state).strip << "\n\n"
+        +"\n\n" << treat_children(node, state).strip << "\n\n"
       end
     end
 

data/lib/reverse_markdown/converters/pre.rb

@@ -4,9 +4,9 @@ module ReverseMarkdown
       def convert(node, state = {})
         content = treat_children(node, state)
         if ReverseMarkdown.config.github_flavored
-          "\n```#{language(node)}\n" << content.strip << "\n```\n"
+          +"\n```#{language(node)}\n" << content.strip << "\n```\n"
         else
-          "\n\n    " << content.lines.to_a.join("    ") << "\n\n"
+          +"\n\n    " << content.lines.to_a.join("    ") << "\n\n"
         end
       end
 

data/lib/reverse_markdown/converters/strong.rb

@@ -6,7 +6,7 @@ module ReverseMarkdown
         if content.strip.empty? || state[:already_strong]
           content
         else
-          "#{content[/^\s*/]}**#{content.strip}**#{content[/\s*$/]}"
+          wrap_with_markers(content, '**')
         end
       end
     end

data/lib/reverse_markdown/converters/table.rb

@@ -2,7 +2,7 @@ module ReverseMarkdown
   module Converters
     class Table < Base
       def convert(node, state = {})
-        "\n\n" << treat_children(node, state) << "\n"
+        +"\n\n" << treat_children(node, state) << "\n"
       end
     end
 

data/lib/reverse_markdown/converters/text.rb

@@ -11,12 +11,20 @@ module ReverseMarkdown
 
       private
 
+      INLINE_ELEMENTS = [:a, :abbr, :b, :bdi, :bdo, :cite, :code, :data, :del,
+                          :dfn, :em, :i, :ins, :kbd, :mark, :q, :rp, :rt, :ruby,
+                          :s, :samp, :small, :span, :strong, :sub, :sup, :time,
+                          :u, :var, :wbr, :font, :tt].freeze
+
       def treat_empty(node)
         parent = node.parent.name.to_sym
         if [:ol, :ul].include?(parent)  # Otherwise the identation is broken
           ''
         elsif node.text == ' '          # Regular whitespace text node
           ' '
+        elsif INLINE_ELEMENTS.include?(parent) && node.text =~ /\n/
+          # Preserve newlines between inline elements as space (HTML whitespace collapsing)
+          ' '
         else
           ''
         end
@@ -25,7 +33,7 @@ module ReverseMarkdown
       def treat_text(node)
         text = node.text
         text = preserve_nbsp(text)
-        text = remove_border_newlines(text)
+        text = remove_border_newlines(text, node)
         text = remove_inner_newlines(text)
         text = escape_keychars(text)
 
@@ -43,8 +51,39 @@ module ReverseMarkdown
         text.gsub(/[<>]/, '>' => '\>', '<' => '\<')
       end
 
-      def remove_border_newlines(text)
-        text.gsub(/\A\n+/, '').gsub(/\n+\z/, '')
+      def remove_border_newlines(text, node)
+        # Convert leading newlines to space if there's preceding inline content
+        result = if has_adjacent_inline_content?(node, :previous)
+          text.gsub(/\A\n+/, ' ')
+        else
+          text.gsub(/\A\n+/, '')
+        end
+
+        # Convert trailing newlines to space if there's following inline content
+        if has_adjacent_inline_content?(node, :next)
+          result.gsub(/\n+\z/, ' ')
+        else
+          result.gsub(/\n+\z/, '')
+        end
+      end
+
+      def has_adjacent_inline_content?(node, direction)
+        sibling = direction == :next ? node.next_sibling : node.previous_sibling
+        while sibling
+          if sibling.text?
+            return true unless sibling.text.strip.empty?
+          elsif INLINE_ELEMENTS.include?(sibling.name.to_sym)
+            return true
+          else
+            return false
+          end
+          sibling = direction == :next ? sibling.next_sibling : sibling.previous_sibling
+        end
+
+        parent = node.parent
+        return false unless INLINE_ELEMENTS.include?(parent.name.to_sym)
+
+        has_adjacent_inline_content?(parent, direction)
       end
 
       def remove_inner_newlines(text)

data/lib/reverse_markdown/version.rb

@@ -1,3 +1,3 @@
 module ReverseMarkdown
-  VERSION = '2.1.1'
+  VERSION = '3.0.2'
 end

data/lib/reverse_markdown.rb

@@ -31,6 +31,7 @@ require 'reverse_markdown/converters/table'
 require 'reverse_markdown/converters/td'
 require 'reverse_markdown/converters/text'
 require 'reverse_markdown/converters/tr'
+require 'reverse_markdown/converters/iframe'
 
 module ReverseMarkdown
 

data/reverse_markdown.gemspec

@@ -23,6 +23,5 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'simplecov'
   s.add_development_dependency 'rake'
   s.add_development_dependency 'kramdown'
-  s.add_development_dependency 'byebug'
-  s.add_development_dependency 'codeclimate-test-reporter'
+  s.add_development_dependency 'debug' unless RUBY_ENGINE == 'jruby'
 end

data/spec/assets/anchors.html

@@ -9,7 +9,7 @@
     Even with stripped elements inbetween: !<span><a href="http://still.not.an.image.foobar.com">there</a></span> should be an extra space.
 
     ignore <a href="foo.html">   </a> anchor tags with no link text
-    not ignore <a href="foo.html"><img src="image.png" alt="An Image" /></a> anchor tags with images
+    not ignore <a href="foo.html"><img src="image.png" alt="An Image"></a> anchor tags with images
     pass through the text of <a href="#content">internal jumplinks</a> without treating them as links
     pass through the text of <a id="content">anchor tags with no href</a> without treating them as links
 

data/spec/assets/basic.html

@@ -11,7 +11,7 @@
     <em>em tag content</em>
     before <em></em> and after empty em tags
     before <em> </em> and after em tags containing whitespace
-    before <em> <em> <br /> </em> </em> and after em tags containing whitespace
+    before <em> <em> <br> </em> </em> and after em tags containing whitespace
     <em><em>double em tags</em></em>
     <p><em><em>double em tags in p tag</em></em></p>
     a<em> em with leading and trailing </em>whitespace
@@ -22,7 +22,7 @@
     <strong>strong tag content</strong>
     before <strong></strong> and after empty strong tags
     before <strong> </strong> and after strong tags containing whitespace
-    before <strong> <strong> <br /> </strong> </strong> and after strong tags containing whitespace
+    before <strong> <strong> <br> </strong> </strong> and after strong tags containing whitespace
     <strong><strong>double strong tags</strong></strong>
     <p><strong><strong>double strong tags in p tag</strong></strong></p>
     before
@@ -39,10 +39,10 @@
     <b>b tag content</b>
     <i>i tag content</i>
 
-    br tags become double space followed by newline<br/>
+    br tags become double space followed by newline<br>
 
     before hr
-    <hr/>
+    <hr>
     after hr
 
     <div>section 1</div>

data/spec/assets/code.html

@@ -19,4 +19,3 @@ end tell
 
   </body>
 </html>
-

data/spec/assets/from_the_wild.html

@@ -1,14 +1,14 @@
 <p>
   <strong>
     <strong>
-      .<br />
+      .<br>
     </strong>
     *** intentcast
   </strong>
   : logo design
   <strong>
     <strong>
-      <br />
+      <br>
     </strong>
   </strong>
   <strong>

data/spec/assets/html_fragment.html

@@ -1,3 +1,3 @@
 naked text 1
 <p>paragraph text</p>
-naked text 2
+naked text 2

data/spec/assets/iframe.html

@@ -0,0 +1,4 @@
+<h1>Welcome to My Page</h1>
+<p>This is a sample paragraph before the iframe.</p>
+<iframe src="https://www.example.com" width="600" height="400"></iframe>
+<p>This is a sample paragraph after the iframe.</p>

data/spec/assets/lists.html

@@ -35,7 +35,7 @@
     <ul>
       <li class="toclevel-1 tocsection-1"><a href="Basic_concepts"><span class="tocnumber">1</span> <span class="toctext">Basic concepts</span></a></li>
       <li class="toclevel-1 tocsection-2"><a href="History_of_the_idea"><span class="tocnumber">2</span> <span class="toctext">History of the idea</span></a></li>
-      <li class="toclevel-1 tocsection-3"><a href="Intelligence_explosion"><span class="tocnumber">3</span> <span class="toctext">Intelligence explosion</span></a>
+      <li class="toclevel-1 tocsection-3"><a href="Intelligence_explosion"><span class="tocnumber">3</span> <span class="toctext">Intelligence explosion</span></a></li>
     </ul>
 
     <ul>
@@ -86,7 +86,7 @@
           <li>bravo alpha</li>
           <li>bravo bravo
             <ul>
-              <li>bravo bravo alpha</i>
+              <li>bravo bravo alpha</li>
             </ul>
           </li>
         </ul>
@@ -95,5 +95,10 @@
       <li>delta</li>
     </ul>
 
+    <ul>
+      <li>item followed with a text</li>
+    </ul>
+    text after the list
+
   </body>
 </html>

data/spec/assets/paragraphs.html

@@ -21,4 +21,4 @@
       <strong><em>Combination:&nbsp;</em></strong>
     </p>
   </body>
-</html>
+</html>

data/spec/components/basic_spec.rb

@@ -40,4 +40,32 @@ describe ReverseMarkdown do
   it { is_expected.to match /before hr \n\* \* \*\n after hr/ }
 
   it { is_expected.to match /section 1\n ?\nsection 2/ }
+
+  describe 'whitespace handling between inline elements' do
+    it 'preserves whitespace (including newlines) between spans' do
+      input = "<span>Hello\n</span><span>World</span>"
+      result = ReverseMarkdown.convert(input)
+      expect(result).to eq "Hello World"
+    end
+
+    it 'preserves whitespace between inline elements in paragraphs' do
+      input = "<p><span>Hello\n</span><span>World</span></p>"
+      result = ReverseMarkdown.convert(input)
+      expect(result).to eq "Hello World\n\n"
+    end
+
+    it 'preserves whitespace between nested inline elements' do
+      # The text "A" is nested inside <span> inside <em>, but <em> has a following sibling
+      # This requires traversing up through parent nodes to find following content
+      input = "<p><em><span>A\n</span></em><span>B</span></p>"
+      result = ReverseMarkdown.convert(input)
+      expect(result).to eq "_A_ B\n\n"
+    end
+
+    it 'preserves whitespace surrounding links' do
+      # Issue #91: newlines around inline elements should become spaces
+      result = ReverseMarkdown.convert("a\n<a href='1'>link</a>\nis good")
+      expect(result.strip).to eq "a [link](1) is good"
+    end
+  end
 end

data/spec/components/from_the_wild_spec.rb

@@ -1,17 +1,15 @@
 require 'spec_helper'
 
 describe ReverseMarkdown do
-
   let(:input)    { File.read('spec/assets/from_the_wild.html') }
   let(:document) { Nokogiri::HTML(input) }
   subject { ReverseMarkdown.convert(input) }
 
   it "should make sense of strong-crazy markup (as seen in the wild)" do
-    expect(subject).to include "**.  \n \\*\\*\\* intentcast** : logo design   \n **.**\n\n"
+    expect(subject).to include "**.  \n  \\*\\*\\* intentcast** : logo design     \n    **.**\n\n"
   end
 
   it "should not over escape * or _" do
     expect(subject).to include '[![](example.com/foo_bar.png) I\_AM\_HELPFUL](example.com/foo_bar)'
   end
-
 end

data/spec/components/html_fragment_spec.rb

@@ -1,11 +1,9 @@
 require 'spec_helper'
 
 describe ReverseMarkdown do
-
   let(:input)    { File.read('spec/assets/html_fragment.html') }
   let(:document) { Nokogiri::HTML(input) }
   subject { ReverseMarkdown.convert(input) }
 
   it { is_expected.to eq("naked text 1\n\nparagraph text\n\nnaked text 2") }
 end
-

data/spec/components/iframe_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe ReverseMarkdown do
+  let(:input)    { File.read('spec/assets/iframe.html') }
+  let(:document) { Nokogiri::HTML(input) }
+  subject { ReverseMarkdown.convert(input) }
+
+  it do
+    expected = <<~MD
+      # Welcome to My Page
+
+      This is a sample paragraph before the iframe.
+
+      https://www.example.com
+
+      This is a sample paragraph after the iframe.
+
+    MD
+
+    expect(subject).to eq expected
+  end
+end

data/spec/components/lists_spec.rb

@@ -65,4 +65,7 @@ describe ReverseMarkdown do
     it { is_expected.to match /\n- delta\n/ }
   end
 
+  context "text following list should have a new line separator" do
+    it { is_expected.to match /\n- item followed with a text\n\n text after the list/ }
+  end
 end

data/spec/lib/reverse_markdown/converters/em_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe ReverseMarkdown::Converters::Em do
+  let(:converter) { ReverseMarkdown::Converters::Em.new }
+
+  it 'returns an empty string if the node is empty' do
+    input = node_for('<em></em>')
+    expect(converter.convert(input)).to eq ''
+  end
+
+  it 'returns just the content if the em tag is nested in another em' do
+    input = node_for('<em><em>foo</em></em>')
+    expect(converter.convert(input.children.first, already_italic: true)).to eq 'foo'
+  end
+
+  it 'moves border whitespaces outside of the delimiters tag' do
+    input = node_for("<em> \n foo </em>")
+    expect(converter.convert(input)).to eq " _foo_ "
+  end
+
+  it 'splits markers at paragraph breaks' do
+    # Issue #95: <br><br> inside em creates a paragraph break
+    # Markers must be split so markdown renders correctly
+    result = ReverseMarkdown.convert('<em>hello<br><br>world</em>')
+    expect(result).to include('_hello_')
+    expect(result).to include('_world_')
+  end
+end

data/spec/lib/reverse_markdown/converters/h_spec.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+
+describe ReverseMarkdown::Converters::H do
+  let(:converter) { ReverseMarkdown::Converters::H.new }
+
+  it 'merges line breaks into single line' do
+    # Markdown headings can't span multiple lines, so merge them
+    result = ReverseMarkdown.convert('<h1>foo<br>bar</h1>')
+    expect(result.strip).to eq '# foo bar'
+  end
+
+  it 'handles multiple line breaks' do
+    result = ReverseMarkdown.convert('<h2>a<br>b<br>c</h2>')
+    expect(result.strip).to eq '## a b c'
+  end
+end

data/spec/lib/reverse_markdown/converters/strong_spec.rb

@@ -17,4 +17,12 @@ describe ReverseMarkdown::Converters::Strong do
     input = node_for("<strong> \n foo </strong>")
     expect(converter.convert(input)).to eq " **foo** "
   end
+
+  it 'splits markers at paragraph breaks' do
+    # Issue #95: <br><br> inside strong creates a paragraph break
+    # Markers must be split so markdown renders correctly
+    result = ReverseMarkdown.convert('<strong>hello<br><br>world</strong>')
+    expect(result).to include('**hello**')
+    expect(result).to include('**world**')
+  end
 end

data/spec/lib/reverse_markdown/converters/text_spec.rb

@@ -22,7 +22,7 @@ describe ReverseMarkdown::Converters::Text do
     expect(result).to eq 'foo bar'
   end
 
-  it 'removes trailing newlines' do
+  it 'removes trailing newlines when no following content' do
     input = node_for("<p>foo bar\n\n</p>")
     result = converter.convert(input)
     expect(result).to eq 'foo bar'

data/spec/lib/reverse_markdown_spec.rb

@@ -36,7 +36,8 @@ describe ReverseMarkdown do
 
     describe 'force_encoding option', jruby: :exclude do
       it 'raises invalid byte sequence in UTF-8 exception' do
-        expect { ReverseMarkdown.convert("hi \255") }.to raise_error(ArgumentError)
+        # Older versions of ruby used to raise ArgumentError here. Remove when we drop support for 3.1.
+        expect { ReverseMarkdown.convert("hi \255") }.to raise_error { [Encoding::CompatibilityError, ArgumentError].include?(_1.class) }
       end
 
       it 'handles invalid byte sequence if option is set' do

data/spec/spec_helper.rb

@@ -1,5 +1,5 @@
 require 'simplecov'
-# require 'byebug'
+require 'debug' unless RUBY_ENGINE == 'jruby'
 
 SimpleCov.profiles.define 'gem' do
   add_filter '/spec/'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: reverse_markdown
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 3.0.2
 platform: ruby
 authors:
 - Johannes Opper
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-31 00:00:00.000000000 Z
+date: 2026-01-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -81,21 +81,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: byebug
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: codeclimate-test-reporter
+  name: debug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -117,9 +103,10 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ci.yml"
 - ".gitignore"
 - ".rspec"
-- ".travis.yml"
+- ".tool-versions"
 - CHANGELOG.md
 - Gemfile
 - LICENSE
@@ -145,6 +132,7 @@ files:
 - lib/reverse_markdown/converters/figure.rb
 - lib/reverse_markdown/converters/h.rb
 - lib/reverse_markdown/converters/hr.rb
+- lib/reverse_markdown/converters/iframe.rb
 - lib/reverse_markdown/converters/ignore.rb
 - lib/reverse_markdown/converters/img.rb
 - lib/reverse_markdown/converters/li.rb
@@ -167,6 +155,7 @@ files:
 - spec/assets/from_the_wild.html
 - spec/assets/full_example.html
 - spec/assets/html_fragment.html
+- spec/assets/iframe.html
 - spec/assets/lists.html
 - spec/assets/minimum.html
 - spec/assets/paragraphs.html
@@ -179,6 +168,7 @@ files:
 - spec/components/escapables_spec.rb
 - spec/components/from_the_wild_spec.rb
 - spec/components/html_fragment_spec.rb
+- spec/components/iframe_spec.rb
 - spec/components/lists_spec.rb
 - spec/components/paragraphs_spec.rb
 - spec/components/quotation_spec.rb
@@ -191,7 +181,9 @@ files:
 - spec/lib/reverse_markdown/converters/br_spec.rb
 - spec/lib/reverse_markdown/converters/del_spec.rb
 - spec/lib/reverse_markdown/converters/details_spec.rb
+- spec/lib/reverse_markdown/converters/em_spec.rb
 - spec/lib/reverse_markdown/converters/figure_spec.rb
+- spec/lib/reverse_markdown/converters/h_spec.rb
 - spec/lib/reverse_markdown/converters/li_spec.rb
 - spec/lib/reverse_markdown/converters/pre_spec.rb
 - spec/lib/reverse_markdown/converters/strong_spec.rb
@@ -203,7 +195,7 @@ homepage: http://github.com/xijo/reverse_markdown
 licenses:
 - WTFPL
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -218,8 +210,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
-signing_key: 
+rubygems_version: 3.5.16
+signing_key:
 specification_version: 4
 summary: Convert html code into markdown.
 test_files:
@@ -230,6 +222,7 @@ test_files:
 - spec/assets/from_the_wild.html
 - spec/assets/full_example.html
 - spec/assets/html_fragment.html
+- spec/assets/iframe.html
 - spec/assets/lists.html
 - spec/assets/minimum.html
 - spec/assets/paragraphs.html
@@ -242,6 +235,7 @@ test_files:
 - spec/components/escapables_spec.rb
 - spec/components/from_the_wild_spec.rb
 - spec/components/html_fragment_spec.rb
+- spec/components/iframe_spec.rb
 - spec/components/lists_spec.rb
 - spec/components/paragraphs_spec.rb
 - spec/components/quotation_spec.rb
@@ -254,7 +248,9 @@ test_files:
 - spec/lib/reverse_markdown/converters/br_spec.rb
 - spec/lib/reverse_markdown/converters/del_spec.rb
 - spec/lib/reverse_markdown/converters/details_spec.rb
+- spec/lib/reverse_markdown/converters/em_spec.rb
 - spec/lib/reverse_markdown/converters/figure_spec.rb
+- spec/lib/reverse_markdown/converters/h_spec.rb
 - spec/lib/reverse_markdown/converters/li_spec.rb
 - spec/lib/reverse_markdown/converters/pre_spec.rb
 - spec/lib/reverse_markdown/converters/strong_spec.rb

data/.travis.yml

@@ -1,18 +0,0 @@
-language: ruby
-cache: bundler
-
-rvm:
-  - 2.0
-  - 2.1
-  - 2.2
-  - 2.3
-  - 2.4
-  - 2.5
-  - 2.6
-  - 2.7
-  - jruby-9.2.8.0
-
-notifications:
-  disabled: false
-  recipients:
-    - xijo@pm.me