From 3dc66fc05eaddfb80ef8e56fbc5e2d8dda90f8ee Mon Sep 17 00:00:00 2001
From: Ariel Juodziukynas <arieljuod@gmail.com>
Date: Thu, 28 May 2026 16:49:40 -0300
Subject: [PATCH 1/2] IIRR-34: Render puzzles's code with code-style

---
 Gemfile                                   |  2 ++
 Gemfile.lock                              |  2 ++
 app/helpers/application_helper.rb         |  3 +++
 app/views/puzzles/_puzzles_table.html.erb |  4 ++--
 app/views/puzzles/update.turbo_stream.erb |  4 ++--
 db/seeds.rb                               | 29 +++++++++++++----------
 test/fixtures/puzzles.yml                 |  4 ++--
 test/helpers/markdown_helper_test.rb      | 27 +++++++++++++++++++++
 8 files changed, 57 insertions(+), 18 deletions(-)
 create mode 100644 test/helpers/markdown_helper_test.rb

diff --git a/Gemfile b/Gemfile
index 3acef3a..156ae5c 100644
--- a/Gemfile
+++ b/Gemfile
@@ -22,6 +22,8 @@ gem "stimulus-rails"
 # Build JSON APIs with ease [https://github.com/rails/jbuilder]
 gem "jbuilder"
 
+gem "redcarpet"
+
 # Use Active Model has_secure_password [https://guides.rubyonrails.org/active_model_basics.html#securepassword]
 # gem "bcrypt", "~> 3.1.7"
 
diff --git a/Gemfile.lock b/Gemfile.lock
index 66d6c35..62b9f0f 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -381,6 +381,7 @@ GEM
     rdoc (6.14.1)
       erb
       psych (>= 4.0.0)
+    redcarpet (3.6.1)
     redis-client (0.24.0)
       connection_pool
     regexp_parser (2.10.0)
@@ -590,6 +591,7 @@ DEPENDENCIES
   propshaft
   puma (>= 5.0)
   rails (~> 8.0.2)
+  redcarpet
   rubocop-rails-omakase
   selenium-webdriver
   sentry-rails
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index de6be79..ce36e2a 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -1,2 +1,5 @@
 module ApplicationHelper
+  def markdown(text)
+    RedcarpetCompat.new(text, :fenced_code).to_html.html_safe
+  end
 end
diff --git a/app/views/puzzles/_puzzles_table.html.erb b/app/views/puzzles/_puzzles_table.html.erb
index a880c95..eeca005 100644
--- a/app/views/puzzles/_puzzles_table.html.erb
+++ b/app/views/puzzles/_puzzles_table.html.erb
@@ -15,9 +15,9 @@
   <tbody>
     <% puzzles.each do |puzzle| %>
       <tr id="<%= dom_id(puzzle) %>">
-        <td><%= simple_format(puzzle.question) %></td>
+        <td><%= markdown(puzzle.question) %></td>
         <td><%= puzzle.answer %></td>
-        <td><%= puzzle.explanation %></td>
+        <td><%= markdown(puzzle.explanation) %></td>
         <td>
           <% if puzzle.link.present? %>
             <%= link_to 'View', puzzle.link, target: '_blank' %>
diff --git a/app/views/puzzles/update.turbo_stream.erb b/app/views/puzzles/update.turbo_stream.erb
index 8f26218..e9dd213 100644
--- a/app/views/puzzles/update.turbo_stream.erb
+++ b/app/views/puzzles/update.turbo_stream.erb
@@ -1,9 +1,9 @@
 <turbo-stream action="replace" target="<%= dom_id(@puzzle) %>">
   <template>
     <tr id="<%= dom_id(@puzzle) %>">
-      <td><%= simple_format(@puzzle.question) %></td>
+      <td><%= markdown(@puzzle.question) %></td>
       <td><%= @puzzle.answer %></td>
-      <td><%= @puzzle.explanation %></td>
+      <td><%= markdown(@puzzle.explanation) %></td>
       <td>
         <% if @puzzle.link.present? %>
           <%= link_to 'View', @puzzle.link, target: '_blank' %>
diff --git a/db/seeds.rb b/db/seeds.rb
index 1df51e8..39a5bb6 100644
--- a/db/seeds.rb
+++ b/db/seeds.rb
@@ -6,32 +6,32 @@
 # rates > 80 do not. Entries without `state` default to :pending.
 puzzles = [
   {
-    question: "Ruby or Rails provided this method? Array.new(5) { |i| i * 2 }",
+    question: "Ruby or Rails provided this method? `Array.new(5) { |i| i * 2 }`",
     answer: :ruby,
     explanation: "`Array.new` is a core Ruby method that creates a new array with a specified size and optional block for initialization. This is part of Ruby’s core library."
   },
   {
-    question: "Ruby or Rails provided this method? File.open('log.txt', 'w') { |file| file.write('Hello, World!') }",
+    question: "Ruby or Rails provided this method? `File.open('log.txt', 'w') { |file| file.write('Hello, World!') }`",
     answer: :ruby,
     explanation: "`File.open` is a core Ruby method for opening files. It’s part of Ruby's standard library for file handling, not part of Rails."
   },
   {
-    question: "Ruby or Rails provided this method? render json: @user",
+    question: "Ruby or Rails provided this method? `render json: @user`",
     answer: :rails,
     explanation: "`render json:` is a Rails method used to render a JSON response from a controller action."
   },
   {
-    question: "Ruby or Rails provided this method? link_to 'Home', root_path",
+    question: "Ruby or Rails provided this method? `link_to 'Home', root_path`",
     answer: :rails,
     explanation: "`link_to` is a Rails helper method that generates HTML links. `root_path` is a Rails path helper."
   },
   {
-    question: "Ruby or Rails provided this method? params[:id]",
+    question: "Ruby or Rails provided this method? `params[:id]`",
     answer: :rails,
     explanation: "`params[:id]` is used in Rails to fetch query parameters or URL parameters in controller actions."
   },
   {
-    question: "Ruby or Rails provided this method? before_action :authenticate_user!",
+    question: "Ruby or Rails provided this method? `before_action :authenticate_user!`",
     answer: :rails,
     explanation: "`before_action` is a Rails callback defined in `ActionController::Callbacks`. It runs specified methods before controller actions.",
     state: :archived,
@@ -39,7 +39,7 @@
     success_rate: 20
   },
   {
-    question: "Ruby or Rails provided this method? 42.times { puts 'hello' }",
+    question: "Ruby or Rails provided this method? `42.times { puts 'hello' }`",
     answer: :ruby,
     explanation: "`Integer#times` is a core Ruby method that iterates a block a specified number of times.",
     state: :archived,
@@ -47,7 +47,7 @@
     success_rate: 40
   },
   {
-    question: "Ruby or Rails provided this method? User.where(active: true).order(:name)",
+    question: "Ruby or Rails provided this method? `User.where(active: true).order(:name)`",
     answer: :rails,
     explanation: "`where` and `order` are ActiveRecord query methods provided by Rails to build SQL queries.",
     state: :archived,
@@ -55,7 +55,7 @@
     success_rate: 50
   },
   {
-    question: "Ruby or Rails provided this method? 'hello world'.split(' ')",
+    question: "Ruby or Rails provided this method? `'hello world'.split(' ')`",
     answer: :ruby,
     explanation: "`String#split` is a core Ruby method that divides a string into an array based on a delimiter.",
     state: :archived,
@@ -63,7 +63,7 @@
     success_rate: 70
   },
   {
-    question: "Ruby or Rails provided this method? flash[:notice] = 'Saved!'",
+    question: "Ruby or Rails provided this method? `flash[:notice] = 'Saved!'`",
     answer: :rails,
     explanation: "`flash` is a Rails feature provided by `ActionDispatch::Flash` for passing messages between requests.",
     state: :archived,
@@ -71,7 +71,7 @@
     success_rate: 80
   },
   {
-    question: "Ruby or Rails provided this method? [1, 2, 3].reduce(:+)",
+    question: "Ruby or Rails provided this method? `[1, 2, 3].reduce(:+)`",
     answer: :ruby,
     explanation: "`Enumerable#reduce` (also `inject`) is a core Ruby method that combines elements using a binary operation.",
     state: :archived,
@@ -79,12 +79,17 @@
     success_rate: 90
   },
   {
-    question: "Ruby or Rails provided this method? validates :email, presence: true, uniqueness: true",
+    question: "Ruby or Rails provided this method? `validates :email, presence: true, uniqueness: true`",
     answer: :rails,
     explanation: "`validates` is an ActiveModel/ActiveRecord method from Rails that adds validation rules to models.",
     state: :archived,
     sent_at: 1.day.ago,
     success_rate: 100
+  },
+  {
+    question: "Ruby or Rails provided this method?\n\n```\nclass SomeClass\n  def method\n    whatever\n  end\nend\n```",
+    answer: :rails,
+    explanation: "Not really important, just to show it renders blocks"
   }
 ]
 
diff --git a/test/fixtures/puzzles.yml b/test/fixtures/puzzles.yml
index 45588a1..86e15e8 100644
--- a/test/fixtures/puzzles.yml
+++ b/test/fixtures/puzzles.yml
@@ -1,5 +1,5 @@
 one:
-  question: "Ruby or Rails provided this method? render json: @user"
+  question: "Ruby or Rails provided this method? `render json: @user`"
   answer: "rails"
   explanation: "This is a test puzzle"
   link: "https://example.com"
@@ -7,7 +7,7 @@ one:
   suggested_by: "test_user"
 
 two:
-  question: "Ruby or Rails provided this method? puts 'Hello, World!'"
+  question: "Ruby or Rails provided this method? `puts 'Hello, World!'`"
   answer: "ruby"
   explanation: "This is a test puzzle 2"
   link: ""
diff --git a/test/helpers/markdown_helper_test.rb b/test/helpers/markdown_helper_test.rb
new file mode 100644
index 0000000..a099411
--- /dev/null
+++ b/test/helpers/markdown_helper_test.rb
@@ -0,0 +1,27 @@
+require "test_helper"
+
+class MarkdownHelperTest < ActiveJob::TestCase
+  include ApplicationHelper
+
+  test "renders `code` with code tag" do
+    puzzle_content = "Some content with `some code` in it"
+    as_html = markdown(puzzle_content)
+
+    assert_includes as_html, "<code>some code</code>"
+  end
+
+  test "renders ```code``` with code block" do
+    puzzle_content = <<~CONTENT
+    Some content with
+
+    ```
+    class ACodeBlock
+    end
+    ```
+    in it
+    CONTENT
+    as_html = markdown(puzzle_content)
+
+    assert_includes as_html, "<pre><code>class ACodeBlock"
+  end
+end

From 825a427045ddda550aaaac08e7e8e963afa7f76c Mon Sep 17 00:00:00 2001
From: Ariel Juodziukynas <arieljuod@gmail.com>
Date: Thu, 28 May 2026 19:26:26 -0300
Subject: [PATCH 2/2] IIRR-34: Address PR feedback

---
 app/helpers/application_helper.rb    | 4 +++-
 test/helpers/markdown_helper_test.rb | 2 +-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index ce36e2a..3acfde9 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -1,5 +1,7 @@
 module ApplicationHelper
   def markdown(text)
-    RedcarpetCompat.new(text, :fenced_code).to_html.html_safe
+    renderer = Redcarpet::Render::HTML.new(escape_html: true)
+    markdown = Redcarpet::Markdown.new(renderer, fenced_code_blocks: true)
+    markdown.render(text.to_s).html_safe
   end
 end
diff --git a/test/helpers/markdown_helper_test.rb b/test/helpers/markdown_helper_test.rb
index a099411..bc0f8cb 100644
--- a/test/helpers/markdown_helper_test.rb
+++ b/test/helpers/markdown_helper_test.rb
@@ -1,6 +1,6 @@
 require "test_helper"
 
-class MarkdownHelperTest < ActiveJob::TestCase
+class MarkdownHelperTest < ActionView::TestCase
   include ApplicationHelper
 
   test "renders `code` with code tag" do