This is port of The Errbit to SQL by Claude code by biow0lf · Pull Request #2832 · errbit/errbit

biow0lf · 2026-05-13T21:04:33Z

No description provided.

+    end
+
+    def reset_password(new_password, new_password_confirmation)
+      self.password = new_password


+
+    def reset_password(new_password, new_password_confirmation)
+      self.password = new_password
+      self.password_confirmation = new_password_confirmation


Copilot

Pull request overview

This PR begins an ActiveRecord/SQLite-based SQL port alongside the existing Mongoid implementation. It adds new namespaced Errbit models, SQL schema/migrations, database configuration, and model specs/factories for apps, users, and watchers.

Changes:

Enables ActiveRecord and Devise ActiveRecord integration with SQLite configuration.
Adds SQL-backed Errbit::App, Errbit::User, and Errbit::Watcher models plus migrations/schema.
Adds factories, specs, and test support for the new ActiveRecord models.

Reviewed changes

Copilot reviewed 21 out of 22 changed files in this pull request and generated 14 comments.

Show a summary per file

File	Description
`Gemfile`	Adds SQLite and ActiveRecord test helper dependencies.
`Gemfile.lock`	Locks added SQL/test dependencies.
`config/application.rb`	Enables ActiveRecord railtie.
`config/database.yml`	Adds SQLite database configuration for all environments.
`config/initializers/devise.rb`	Loads Devise ActiveRecord ORM support.
`app/models/errbit.rb`	Adds table name prefix for namespaced SQL models.
`app/models/errbit/application_record.rb`	Adds ActiveRecord base class.
`app/models/errbit/app.rb`	Adds SQL-backed app model subset.
`app/models/errbit/user.rb`	Adds SQL-backed user model subset.
`app/models/errbit/watcher.rb`	Adds SQL-backed watcher model.
`db/migrate/20260513192604_create_errbit_users.rb`	Creates SQL users table.
`db/migrate/20260513204143_create_errbit_apps.rb`	Creates SQL apps table.
`db/migrate/20260513204638_create_errbit_watchers.rb`	Creates SQL watchers table.
`db/schema.rb`	Captures generated ActiveRecord schema.
`spec/factories/errbit/app_factory.rb`	Adds SQL app factory.
`spec/factories/errbit/user_factory.rb`	Adds SQL user factory.
`spec/factories/errbit/watcher_factory.rb`	Adds SQL watcher factories.
`spec/models/errbit/app_spec.rb`	Adds SQL app model specs.
`spec/models/errbit/user_spec.rb`	Adds SQL user model specs.
`spec/models/errbit/watcher_spec.rb`	Adds SQL watcher model specs.
`spec/support/active_record.rb`	Adds ActiveRecord database cleanup hooks.
`spec/support/shoulda-matchers.rb`	Adds Shoulda Matchers configuration.

Comments suppressed due to low confidence (5)

app/models/errbit/user.rb:54

watching? calls apps, but this ActiveRecord model does not define an apps association (the only association to apps is through watchers, and no has_many :watchers/has_many :apps, through: exists), so calling this method will raise NoMethodError instead of returning whether the user watches the app.

    def watching?(app)
      apps.all.include?(app)
    end

app/models/errbit/app.rb:4

These new ActiveRecord models live under the Errbit:: namespace, but the application still routes and controllers use the existing top-level App, User, and Watcher classes (for example devise_for :users maps to User, and controllers query App). As written, this SQL model is not used by the production code path, so the port will continue reading/writing Mongoid models instead of these tables.

module Errbit
  class App < ApplicationRecord

app/models/errbit/app.rb:28

The SQL App model only implements the instance watched_by? helper, but the existing deletion flow calls the class-level App.watched_by(user) scope to find apps before removing a user. If this model replaces the Mongoid App, that code path will raise NoMethodError and user deletion will not clean up watcher rows.

    def watched_by?(user)
      watchers.where(errbit_user_id: user.id).exists?
    end

app/models/errbit/app.rb:4

The existing apps index sorts apps in Ruby (app_scope.to_a.sort), which relies on the Mongoid App including Comparable and defining <=>. This ActiveRecord port omits that comparison method, so wiring it into the existing controller would make the apps index fail with an object comparison error.

module Errbit
  class App < ApplicationRecord

app/models/errbit/app.rb:4

This App port does not define the problems association that the existing controllers and views call (app.problems for app pages, feeds, counts, and problem searches). If this replaces the Mongoid model, those core app/error pages will raise instead of showing errors.

module Errbit
  class App < ApplicationRecord

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

+# frozen_string_literal: true
+
+RSpec.configure do |config|
+  config.before(:suite) do


+  <<: *default
+  database: storage/production.sqlite3


+      t.string :name
+      t.string :api_key


+      ## Token authenticatable
+      t.string :authentication_token
+
+      t.string :name


+      t.references :errbit_user, foreign_key: true
+      t.string :email


+      t.string :current_app_version
+      t.boolean :notify_all_users, null: false, default: false
+      t.boolean :notify_on_errs, null: false, default: true
+      t.text :email_at_notices


+    def email_choosen
+      email.blank? ? "chosen" : ""
+    end
+


+    attr_accessor :_watcher_type
+


+    attr_accessor :_watcher_type
+
+    def watcher_type
+      @_watcher_type ||= email.present? ? "email" : "user"


+      self.class.validators_on(:password).map { |v| v.validate_each(self, :password, password) }
+      return false if errors.any?
+


-    app.update(app_params)

-    if app.save
+    if app.update(app_params)


 class HealthController < ActionController::Base # rubocop:disable Rails/ApplicationController
  def api_key_tester
-    app = App.where(api_key: params[:api_key]).first
+    app = Errbit::App.where(api_key: params[:api_key]).first


Copilot

Pull request overview

Copilot reviewed 194 out of 195 changed files in this pull request and generated 8 comments.

+    def watching?(app)
+      apps.all.include?(app)
+    end


+      def backtrace_lines(problem)
+        notice = Errbit::NoticeDecorator.new(problem.notices.last)
+        return unless notice
+


+
+              <br>
+
+              <p><%= @comment.body.to_s.gsub("\n", "<br>").html_safe %></p>


+    expect(described_class.table_name).to eq("errbit_watchers")
+  end
+
+  context "validations" do


+production:
+  <<: *default
+  database: storage/production.sqlite3


+RSpec.describe "Users", type: :system do
+  describe "#index" do
+    # current_user = create(:errbit_user, name: "Ihor Zubkov")
+    #
+    # sign_in(current_user)
+    #
+    # visit users_path
+  end


+      t.boolean :error_class, null: false, default: true
+      t.boolean :message, null: false, default: true
+      t.integer :backtrace_lines, default: -1
+      t.boolean :component, null: false, default: true
+      t.boolean :action, null: false, default: true
+      t.boolean :environment_name, null: false, default: true


Copilot

Pull request overview

Copilot reviewed 202 out of 203 changed files in this pull request and generated 8 comments.

+      def backtrace_lines(problem)
+        notice = Errbit::NoticeDecorator.new(problem.notices.last)
+        return unless notice
+
+        backtrace = notice.backtrace
+        return unless backtrace
+


+    def self.run
+      count = Errbit::Problem.count
+      puts format(LOG_START, count)
+
+      Errbit::Problem.find_each.with_index do |problem, index|
+        problem.recache
+        problem.destroy if problem.notices_count == 0
+
+        next unless (index + 1) % LOG_EVERY == 0
+
+        puts format(LOG_ITR, index * 100 / count, count - index)
+      end


+    def self.run
+      count = Errbit::Notice.count
+      puts format(LOG_START, count)
+
+      Errbit::Notice.find_each.with_index do |notice, index|
+        refingerprint(notice)
+
+        next unless (index + 1) % LOG_EVERY == 0
+
+        puts format(LOG_ITR, index * 100 / count, count - index)
+      end


+      puts "Finished generating notice fingerprints"
+      puts "Destroying orphaned err records"
+
+      Errbit::Err.find_each { |err| err.destroy if err.notices.empty? }
+


  desc "Remove notices in batch"
-  task :notices_delete, [:problem_id] => [:environment] do
+  task :notices_delete, [:problem_id] => [:environment] do |_task, args|
    BATCH_SIZE = 1000
    if args[:problem_id]
-      item_count = Problem.find(args[:problem_id]).notices.count
+      item_count = Errbit::Problem.find(args[:problem_id]).notices.count
      removed_count = 0
      puts "Notices to remove: #{item_count}"
      while item_count > 0
-        Problem.find(args[:problem_id]).notices.limit(BATCH_SIZE).each do |notice|
-          notice.remove
+        Errbit::Problem.find(args[:problem_id]).notices.limit(BATCH_SIZE).each do |notice|
+          notice.destroy
          removed_count += 1
        end
        item_count -= BATCH_SIZE


+              <p class="heading">COMMENT:</p>
+
+              <br>
+
+              <p><%= @comment.body.to_s.gsub("\n", "<br>").html_safe %></p>
+            </div>


+RSpec.describe "Users", type: :system do
+  describe "#index" do
+    # current_user = create(:errbit_user, name: "Ihor Zubkov")
+    #
+    # sign_in(current_user)
+    #
+    # visit users_path
+  end


+    def initialize(*problems)
+      problems = problems.flatten.uniq
+      @merged_problem = problems[0]
+      @child_problems = problems[1..]
+      fail ArgumentError, "need almost 2 uniq different problems" if @child_problems.empty?
+    end


Copilot

Pull request overview

Copilot reviewed 202 out of 203 changed files in this pull request and generated 7 comments.

+    def watching?(app)
+      apps.all.include?(app)
+    end


+      def backtrace_lines(problem)
+        notice = Errbit::NoticeDecorator.new(problem.notices.last)
+        return unless notice
+
+        backtrace = notice.backtrace
+        return unless backtrace
+
+        output = +""
+        backtrace.lines[0..4].each { |line| output << backtrace_line(line) }
+        "```#{output}```"
+      end


+production:
+  <<: *default
+  database: storage/production.sqlite3


+RSpec.describe "Users", type: :system do
+  describe "#index" do
+    # current_user = create(:errbit_user, name: "Ihor Zubkov")
+    #
+    # sign_in(current_user)
+    #
+    # visit users_path
+  end


+      it "saves a new problem for the app" do
+        expect {
          post "/notifier_api/v2/notices", params: {data: xml}
          expect(response).to be_successful
-        end.to change(errbit_app.problems, :count).by(1)
+        }.to change { errbit_app.problems.count }.by(1)


+      expect(f_1).not_to eq(f_2)
+    end
+
+    context "two backtraces with the same first two lines" do


+    FIELDS = [
+      :name, :username, :email, :password, :github_login, :per_page, :time_zone,
+      :password, :password_confirmation
+    ].freeze


biow0lf self-assigned this May 13, 2026

Copilot AI review requested due to automatic review settings May 13, 2026 21:04

Copilot started reviewing on behalf of biow0lf May 13, 2026 21:05 View session

github-advanced-security Bot found potential problems May 13, 2026

View reviewed changes

Copilot AI reviewed May 13, 2026

View reviewed changes

github-advanced-security Bot found potential problems May 15, 2026

View reviewed changes

Comment thread app/controllers/apps_controller.rb

app.update(app_params)

if app.save

if app.update(app_params)

github-advanced-security Bot found potential problems May 15, 2026

View reviewed changes

Comment thread app/controllers/health_controller.rb

class HealthController < ActionController::Base # rubocop:disable Rails/ApplicationController

def api_key_tester

app = App.where(api_key: params[:api_key]).first

app = Errbit::App.where(api_key: params[:api_key]).first

biow0lf force-pushed the claude-code-port-to-sql branch 2 times, most recently from 3e04589 to 2bea878 Compare May 19, 2026 09:57

biow0lf requested a review from Copilot May 19, 2026 09:58

Copilot started reviewing on behalf of biow0lf May 19, 2026 09:59 View session

Copilot AI reviewed May 19, 2026

View reviewed changes

biow0lf force-pushed the claude-code-port-to-sql branch 2 times, most recently from c63ad6c to 26da3b5 Compare May 20, 2026 02:52

biow0lf requested a review from Copilot May 20, 2026 02:52

Copilot started reviewing on behalf of biow0lf May 20, 2026 02:53 View session

Copilot AI reviewed May 20, 2026

View reviewed changes

biow0lf force-pushed the claude-code-port-to-sql branch 2 times, most recently from cb45f31 to 08c3a4c Compare May 23, 2026 19:48

biow0lf requested a review from Copilot May 23, 2026 19:49

Copilot started reviewing on behalf of biow0lf May 23, 2026 19:49 View session

Copilot AI reviewed May 23, 2026

View reviewed changes

biow0lf force-pushed the claude-code-port-to-sql branch from 08c3a4c to 8e1f898 Compare May 25, 2026 11:15

biow0lf force-pushed the claude-code-port-to-sql branch 2 times, most recently from 40c0729 to 45e0c13 Compare June 4, 2026 16:00

biow0lf force-pushed the claude-code-port-to-sql branch from 45e0c13 to 10e6fb9 Compare July 5, 2026 21:28

biow0lf added 4 commits July 5, 2026 23:34

Start porting with Claude code

bc7f6b9

Update

09dfe15

Update

d633ab2

Update

35781b7

biow0lf added 28 commits July 5, 2026 23:34

Update

1a4fd0e

Update

a90b202

Update

92e241e

Update

c0ffe18

Update

d9e1c1b

Update

ef8cba3

Update

bcf773f

Update

74753ae

Update

5a92c9a

Update

49f930d

Update

5e4d380

Update

c5106d2

Update

127761f

Update

ac7cb62

Update

5f3acd7

Update

8ed7674

Update

acf0bc8

Update

069d0d3

Fix rubocop warnings

fdd238f

Update

358e380

Warnings

6a3437f

Update

708b75b

Update

a80998a

Update

e425081

More tests

0601437

More tests

7f8621e

Update sqlite3 gem from 2.9.4 to 2.9.5

7a8e4c3

Update shoulda-matchers gem from 7.0.1 to 8.0.1

0ddb650

biow0lf force-pushed the claude-code-port-to-sql branch from 10e6fb9 to 0ddb650 Compare July 5, 2026 21:34

mikelolasagasti mentioned this pull request Aug 20, 2026

Port Errbit from MongoDB to SQLite #3092

Open

Navigation Menu

This is port of The Errbit to SQL by Claude code - #2832

This is port of The Errbit to SQL by Claude code#2832
biow0lf wants to merge 65 commits into
mainfrom
claude-code-port-to-sql

biow0lf commented May 13, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

This branch has not been deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		self.class.validators_on(:password).map { \|v\| v.validate_each(self, :password, password) }
		return false if errors.any?


		<br>

		<p><%= @comment.body.to_s.gsub("\n", "<br>").html_safe %></p>

Conversation

biow0lf commented May 13, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

This branch has not been deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants