Add version logging to error reports

Author: ArtOfCode-

app/assets/stylesheets/forms.scss

@@ -31,6 +31,7 @@ select.form-element {
   & > .actions {
     display: flex;
     align-items: flex-end;
+    margin: 4px 0;
   }
 }
 

app/controllers/admin_controller.rb

@@ -9,13 +9,25 @@ class AdminController < ApplicationController
   def index; end
 
   def error_reports
-    @reports = if params[:uuid].present?
-                 ErrorLog.where(uuid: params[:uuid])
-               elsif current_user.is_global_admin
-                 ErrorLog.all
-               else
-                 ErrorLog.where(community: RequestContext.community)
-               end.newest_first.paginate(page: params[:page], per_page: 50)
+    base_scope = if current_user.is_global_admin
+                   ErrorLog.all
+                 else
+                   ErrorLog.where(community: RequestContext.community)
+                 end
+
+    if params[:uuid].present?
+      base_scope = base_scope.where(uuid: params[:uuid])
+    end
+
+    if params[:version].present? && params[:version] == 'current'
+      sha, _date = helpers.current_commit
+      base_scope = base_scope.where(version: sha)
+    elsif params[:version].present?
+      base_scope = base_scope.where(version: params[:version])
+    end
+
+    @reports = base_scope.newest_first.paginate(page: params[:page], per_page: 50)
+    render layout: 'without_sidebar'
   end
 
   def privileges

app/controllers/errors_controller.rb

@@ -20,10 +20,11 @@ def error
     end
 
     if @exception.present? && [422, 500].include?(@status)
+      sha, _date = helpers.current_commit
       @log = ErrorLog.create(community: RequestContext.community, user: current_user, klass: @exception&.class,
                              message: @exception&.message, backtrace: @exception&.backtrace&.join("\n"),
                              request_uri: request.original_url, host: request.raw_host_with_port,
-                             uuid: SecureRandom.uuid, user_agent: request.user_agent)
+                             uuid: SecureRandom.uuid, user_agent: request.user_agent, version: sha)
     end
 
     render views[@status] || 'errors/error', formats: :html, status: @status, layout: 'without_sidebar'

app/views/admin/error_reports.html.erb

@@ -1,11 +1,35 @@
 <h1><%= t 'admin.tools.error_reports' %></h1>
 <p class="is-lead"><%= pluralize(@reports.count, t('g.error')) %> <%= t 'g.logged' %></p>
 
-<%= form_tag admin_error_reports_path, method: :get, class: 'has-margin-bottom-4' do %>
-  <%= label_tag :uuid, t('admin.error_search_uuid') %>
-  <%= text_field_tag :uuid, params[:uuid], class: 'form-element', placeholder: 'xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx' %>
-  <%= submit_tag t('g.search').capitalize, class: 'button is-filled' %>
-<% end %>
+<div class="flex-row">
+  <%= form_tag admin_error_reports_path, method: :get, class: 'has-margin-bottom-4 primary' do %>
+    <div class="form-group-horizontal">
+      <div class="form-group">
+        <%= label_tag :uuid, t('admin.error_search_uuid') %>
+        <%= text_field_tag :uuid, params[:uuid], class: 'form-element', placeholder: 'xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx' %>
+      </div>
+      <div class="actions">
+        <%= submit_tag t('g.search').capitalize, class: 'button is-filled' %>
+      </div>
+    </div>
+  <% end %>
+
+  <%= form_tag admin_error_reports_path, method: :get, class: 'has-margin-bottom-4 primary' do %>
+    <div class="form-group-horizontal">
+      <div class="form-group">
+        <%= label_tag :uuid, t('admin.error_search_version') %>
+        <%= text_field_tag :uuid, params[:uuid], class: 'form-element', placeholder: '(full SHA)' %>
+      </div>
+      <div class="actions">
+        <div class="button-list">
+          <button type="submit" class="button is-filled"><%= t('g.search').capitalize %></button>
+          <%= link_to 'Current', query_url(version: 'current'), class: 'button is-outlined' %>
+        </div>
+      </div>
+    </div>
+  <% end %>
+</div>
+
 
 <% @reports.each do |report| %>
   <%= render 'error_report', report: report %>

config/locales/strings/en.admin.yml

@@ -16,6 +16,7 @@ en:
       organisation-wide changes like privacy policy or TOS changes. Format in a way that will be legible when Markdown
       is removed.
     error_search_uuid: 'Search for an error UUID'
+    error_search_version: 'Errors from version (commit)'
     privileges_blurb: >
       Here you can define the reputation required to gain each available privilege. Click on a value to edit it.
     email_query_blurb: >

db/migrate/20250920103432_add_version_to_error_logs.rb

@@ -0,0 +1,5 @@
+class AddVersionToErrorLogs < ActiveRecord::Migration[7.2]
+  def change
+    add_column :error_logs, :version, :string
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[7.2].define(version: 2025_09_05_231140) do
+ActiveRecord::Schema[7.2].define(version: 2025_09_20_103432) do
   create_table "abilities", charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
     t.bigint "community_id"
     t.string "name"
@@ -253,6 +253,7 @@
     t.datetime "updated_at", precision: nil, null: false
     t.string "uuid"
     t.string "user_agent"
+    t.string "version"
     t.index ["community_id"], name: "index_error_logs_on_community_id"
     t.index ["user_id"], name: "index_error_logs_on_user_id"
   end