Call find_or_create directly instead of User.for_login
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LG4iASZnHb1yEM1JAASiWn
diff --git a/lib/rank_king/models.rb b/lib/rank_king/models.rb
index 7abee23..e60bc98 100644
--- a/lib/rank_king/models.rb
+++ b/lib/rank_king/models.rb
@@ -9,8 +9,6 @@ module RankKing
# @rbs skip
class User < Sequel::Model(DB[:users])
one_to_many :lists, class: "RankKing::List"
-
- def self.for_login(login) = find_or_create(login:)
end
# @rbs skip
diff --git a/lib/rank_king/web.rb b/lib/rank_king/web.rb
index 48fda45..6d54c01 100644
--- a/lib/rank_king/web.rb
+++ b/lib/rank_king/web.rb
@@ -21,7 +21,7 @@ module RankKing
route do |r|
check_csrf!
- user = User.for_login(ProxyAuth.login(r.env))
+ user = User.find_or_create(login: ProxyAuth.login(r.env))
r.root do
lists = List.where(user_id: [user.id, nil]).order(:id).all
diff --git a/sig/rank_king/models.rbs b/sig/rank_king/models.rbs
index 7497348..6a9691b 100644
--- a/sig/rank_king/models.rbs
+++ b/sig/rank_king/models.rbs
@@ -5,7 +5,6 @@ module RankKing
class User < Sequel::Model
def login: () -> String
def lists: () -> Array[List]
- def self.for_login: (String login) -> User
end
class List < Sequel::Model