fbe 0.48.3 → 0.48.4

data/lib/fbe/octo.rb

@@ -42,10 +42,10 @@ class Fbe::OffQuota < StandardError; end
 # @param [Hash] global Hash of global options
 # @param [Loog] loog Logging facility
 # @return [Hash] Usually returns a JSON, as it comes from the GitHub API
-def Fbe.octo(options: $options, global: $global, loog: $loog)
-  raise 'The $global is not set' if global.nil?
-  raise 'The $options is not set' if options.nil?
-  raise 'The $loog is not set' if loog.nil?
+def Fbe.octo(options: $options, global: $global, loog: $loog) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
+  raise(Fbe::Error, 'The $global is not set') if global.nil?
+  raise(Fbe::Error, 'The $options is not set') if options.nil?
+  raise(Fbe::Error, 'The $loog is not set') if loog.nil?
   global[:octo] ||=
     begin
       loog.info("Fbe version is #{Fbe::VERSION}")
@@ -73,12 +73,7 @@ def Fbe.octo(options: $options, global: $global, loog: $loog)
         end
         o.auto_paginate = true
         o.per_page = 100
-        o.connection_options = {
-          request: {
-            open_timeout: 15,
-            timeout: 15
-          }
-        }
+        o.connection_options = { request: { open_timeout: 15, timeout: 15 } }
         stack =
           Faraday::RackBuilder.new do |builder|
             builder.use(
@@ -96,27 +91,21 @@ def Fbe.octo(options: $options, global: $global, loog: $loog)
             builder.use(Fbe::Middleware::RateLimit)
             builder.use(Fbe::Middleware::Trace, trace, ignores: [:fresh])
             if options.sqlite_cache
-              maxsize = Filesize.from(options.sqlite_cache_maxsize || '100M').to_i
-              maxvsize = Filesize.from(options.sqlite_cache_maxvsize || '100K').to_i
-              cache_min_age = options.sqlite_cache_min_age&.to_i
+              maxsize = Integer(Filesize.from(options.sqlite_cache_maxsize || '100M'))
+              maxvsize = Integer(Filesize.from(options.sqlite_cache_maxvsize || '100K'))
+              minage = options.sqlite_cache_min_age.nil? ? nil : Integer(options.sqlite_cache_min_age.to_s, 10)
               store = Fbe::Middleware::SqliteStore.new(
-                options.sqlite_cache, Fbe::VERSION, loog:, maxsize:, maxvsize:, ttl: 24, cache_min_age:
+                options.sqlite_cache, Fbe::VERSION, loog:, maxsize:, maxvsize:, ttl: 24, cache_min_age: minage
               )
               loog.info(
                 "Using HTTP cache in SQLite file: #{store.path} (" \
                 "#{File.exist?(store.path) ? Filesize.from(File.size(store.path).to_s).pretty : 'file is absent'}, " \
                 "max size: #{Filesize.from(maxsize.to_s).pretty}, max vsize: #{Filesize.from(maxvsize.to_s).pretty})"
               )
-              builder.use(
-                Faraday::HttpCache,
-                store:, serializer: JSON, shared_cache: false, logger: Loog::NULL
-              )
+              builder.use(Faraday::HttpCache, store:, serializer: JSON, shared_cache: false, logger: Loog::NULL)
             else
               loog.info("No HTTP cache in SQLite file, because 'sqlite_cache' option is not provided")
-              builder.use(
-                Faraday::HttpCache,
-                serializer: Marshal, shared_cache: false, logger: Loog::NULL
-              )
+              builder.use(Faraday::HttpCache, serializer: Marshal, shared_cache: false, logger: Loog::NULL)
             end
             builder.adapter(Faraday.default_adapter)
           end
@@ -133,7 +122,7 @@ def Fbe.octo(options: $options, global: $global, loog: $loog)
         o = Fbe::FakeOctokit.new
       end
       o =
-        decoor(o, loog:, trace:) do
+        decoor(o, loog:, trace:) do # rubocop:disable Metrics/BlockLength
           def print_trace!(all: false, max: 5)
             if @trace.empty?
               @loog.debug('GitHub API trace is empty')
@@ -165,8 +154,7 @@ def Fbe.octo(options: $options, global: $global, loog: $loog)
               @trace.clear
             end
           end
-
-          def off_quota?(threshold: 50)
+          def off_quota?(threshold: 50) # rubocop:disable Layout/EmptyLineBetweenDefs
             left = @origin.rate_limit!.remaining
             if left < threshold
               @loog.info("Too much GitHub API quota consumed already (#{left} < #{threshold})")
@@ -176,34 +164,30 @@ def Fbe.octo(options: $options, global: $global, loog: $loog)
               false
             end
           end
-
-          def user_name_by_id(id)
-            raise 'The ID of the user is nil' if id.nil?
-            raise 'The ID of the user must be an Integer' unless id.is_a?(Integer)
+          def user_name_by_id(id) # rubocop:disable Layout/EmptyLineBetweenDefs
+            raise(Fbe::Error, 'The ID of the user is nil') if id.nil?
+            raise(Fbe::Error, 'The ID of the user must be an Integer') unless id.is_a?(Integer)
             json = @origin.user(id)
             name = json[:login].downcase
             @loog.debug("GitHub user ##{id} has a name: @#{name}")
             name
           end
-
-          def repo_id_by_name(name)
-            raise 'The name of the repo is nil' if name.nil?
+          def repo_id_by_name(name) # rubocop:disable Layout/EmptyLineBetweenDefs
+            raise(Fbe::Error, 'The name of the repo is nil') if name.nil?
             json = @origin.repository(name)
             id = json[:id]
-            raise "Repository #{name} not found" if id.nil?
+            raise(Fbe::Error, "Repository #{name} not found") if id.nil?
             @loog.debug("GitHub repository #{name.inspect} has an ID: ##{id}")
             id
           end
-
-          def repo_name_by_id(id)
-            raise 'The ID of the repo is nil' if id.nil?
-            raise 'The ID of the repo must be an Integer' unless id.is_a?(Integer)
+          def repo_name_by_id(id) # rubocop:disable Layout/EmptyLineBetweenDefs
+            raise(Fbe::Error, 'The ID of the repo is nil') if id.nil?
+            raise(Fbe::Error, 'The ID of the repo must be an Integer') unless id.is_a?(Integer)
             json = @origin.repository(id)
             name = json[:full_name].downcase
             @loog.debug("GitHub repository ##{id} has a name: #{name}")
             name
           end
-
           # Disable auto pagination for octokit client called in block
           #
           # @yield [octo] Give octokit client with disabled auto pagination
@@ -214,10 +198,10 @@ def Fbe.octo(options: $options, global: $global, loog: $loog)
           #      Fbe.octo.with_disable_auto_paginate do |octo|
           #        octo.list_issue('zerocracy/fbe', per_page: 1).first
           #      end
-          def with_disable_auto_paginate
+          def with_disable_auto_paginate # rubocop:disable Layout/EmptyLineBetweenDefs
             ap = @origin.auto_paginate
             @origin.auto_paginate = false
-            yield self if block_given?
+            yield(self) if block_given?
           ensure
             @origin.auto_paginate = ap
           end
@@ -225,7 +209,7 @@ def Fbe.octo(options: $options, global: $global, loog: $loog)
       o =
         intercepted(o) do |e, m, _args, _r|
           if e == :before && m != :off_quota? && m != :print_trace! && m != :rate_limit && o.off_quota?
-            raise Fbe::OffQuota, "We are off-quota (remaining: #{o.rate_limit.remaining}), can't do #{m}()"
+            raise(Fbe::OffQuota, "We are off-quota (remaining: #{o.rate_limit.remaining}), can't do #{m}()")
           end
         end
       o
@@ -250,7 +234,7 @@ end
 #
 # @note All methods return static or pseudo-random data
 # @note No actual API calls are made
-class Fbe::FakeOctokit
+class Fbe::FakeOctokit # rubocop:disable Metrics/ClassLength, Style/OneClassPerFile
   # Generates a random time in the past.
   #
   # @return [Time] A random time within the last 10,000 seconds
@@ -304,6 +288,21 @@ class Fbe::FakeOctokit
     ]
   end
 
+  # Lists organization repositories, including private ones when the token has access.
+  #
+  # @param [String] _org The organization name (ignored in mock)
+  # @param [Hash] _options Filters such as +type:+ (e.g. 'all', 'private', 'public')
+  # @return [Array<Hash>] Array of repository hashes
+  # @example
+  #   client.organization_repositories('zerocracy', type: 'all')
+  #   # => [{:id=>123, :full_name=>"yegor256/judges", ...}, ...]
+  def organization_repositories(_org, _options = {})
+    [
+      repository('yegor256/judges'),
+      repository('yegor256/factbase')
+    ]
+  end
+
   # Gets repository invitations for the authenticated user.
   #
   # @param [Hash] _options Additional options (not used in mock)
@@ -482,7 +481,7 @@ class Fbe::FakeOctokit
   #   fake_client.accept_repository_invitation(1) #=> true
   # rubocop:disable Naming/PredicateMethod
   def accept_repository_invitation(id)
-    raise Octokit::NotFound if id == 404_000
+    raise(Octokit::NotFound) if id == 404_000
     true
   end
   # rubocop:enable Naming/PredicateMethod
@@ -509,7 +508,7 @@ class Fbe::FakeOctokit
   #   fake_client.user(526_301) #=> {:id=>444, :login=>"yegor256", :type=>"User"}
   #   fake_client.user('octocat') #=> {:id=>444, :login=>nil, :type=>"User"}
   def user(uid)
-    raise Octokit::NotFound if [404_001, 404_002].include?(uid)
+    raise(Octokit::NotFound) if [404_001, 404_002].include?(uid)
     login = (uid == 526_301 ? 'yegor256' : 'torvalds') if uid.is_a?(Integer)
     {
       id: 444,
@@ -609,13 +608,13 @@ class Fbe::FakeOctokit
   #   client.repository('octocat/Hello-World')
   #   # => {:id=>1296269, :full_name=>"octocat/Hello-World", ...}
   def repository(name)
-    raise Octokit::NotFound if [404_123, 404_124].include?(name)
-    full_name = name.is_a?(Integer) ? 'yegor256/test' : name
-    full_name = 'zerocracy/baza' if name == 1439
-    full_name = 'foo/bazz' if name == 810
+    raise(Octokit::NotFound) if [404_123, 404_124].include?(name)
+    repo = name.is_a?(Integer) ? 'yegor256/test' : name
+    repo = 'zerocracy/baza' if name == 1439
+    repo = 'foo/bazz' if name == 810
     {
       id: name_to_number(name),
-      full_name:,
+      full_name: repo,
       default_branch: 'master',
       private: false,
       owner: { login: name.to_s.split('/')[0], id: 526_301, site_admin: false },
@@ -1179,7 +1178,7 @@ class Fbe::FakeOctokit
     ]
   end
 
-  def repository_events(repo, _options = {})
+  def repository_events(repo, _options = {}) # rubocop:disable Metrics/MethodLength
     [
       {
         id: '123',
@@ -1508,7 +1507,7 @@ class Fbe::FakeOctokit
     ]
   end
 
-  def check_runs_for_ref(repo, sha)
+  def check_runs_for_ref(repo, sha) # rubocop:disable Metrics/MethodLength
     data = {
       'zerocracy/baza' => {
         total_count: 7,

data/lib/fbe/overwrite.rb

@@ -31,9 +31,9 @@ require_relative 'fb'
 #   user = fb.query('(eq login "john")').first
 #   Fbe.overwrite(user, status: 'active', role: 'admin')
 #   # All properties preserved, 'status' and 'role' are updated
-def Fbe.overwrite(fact, property_or_hash, values = nil, fb: Fbe.fb, fid: '_id')
-  raise 'The fact is nil' if fact.nil?
-  raise 'The fb is nil' if fb.nil?
+def Fbe.overwrite(fact, property_or_hash, values = nil, fb: Fbe.fb, fid: '_id') # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+  raise(Fbe::Error, 'The fact is nil') if fact.nil?
+  raise(Fbe::Error, 'The fb is nil') if fb.nil?
   if property_or_hash.is_a?(Hash)
     before = {}
     fact.all_properties.each do |k|
@@ -41,7 +41,7 @@ def Fbe.overwrite(fact, property_or_hash, values = nil, fb: Fbe.fb, fid: '_id')
     end
     modified = false
     property_or_hash.each do |k, vv|
-      raise "The value for #{k} is nil" if vv.nil?
+      raise(Fbe::Error, "The value for #{k} is nil") if vv.nil?
       vv = [vv] unless vv.is_a?(Array)
       next if before[k.to_s] == vv
       before[k.to_s] = vv
@@ -49,27 +49,27 @@ def Fbe.overwrite(fact, property_or_hash, values = nil, fb: Fbe.fb, fid: '_id')
     end
     return fact unless modified
     id = fact[fid]&.first
-    raise "There is no #{fid} in the fact, cannot use Fbe.overwrite" if id.nil?
-    raise "No facts by #{fid} = #{id}" if fb.query("(eq #{fid} #{id})").delete!.zero?
+    raise(Fbe::Error, "There is no #{fid} in the fact, cannot use Fbe.overwrite") if id.nil?
+    raise(Fbe::Error, "No facts by #{fid} = #{id}") if fb.query("(eq #{fid} #{id})").delete!.zero?
     fb.txn do |fbt|
       n = fbt.insert
       before.each do |k, vv|
         next unless n[k].nil?
         vv.each do |v|
-          n.send(:"#{k}=", v)
+          n.public_send(:"#{k}=", v)
         end
       end
     end
     return
   end
   property = property_or_hash
-  raise "The property is not a String but #{property.class} (#{property})" unless property.is_a?(String)
-  raise 'The values is nil' if values.nil?
+  raise(Fbe::Error, "The property is not a String but #{property.class} (#{property})") unless property.is_a?(String)
+  raise(Fbe::Error, 'The values is nil') if values.nil?
   values = [values] unless values.is_a?(Array)
   return fact if !fact[property].nil? && fact[property].one? && values.one? && fact[property].first == values.first
   if fact[property].nil?
     values.each do |v|
-      fact.send(:"#{property}=", v)
+      fact.public_send(:"#{property}=", v)
     end
     return
   end
@@ -78,15 +78,15 @@ def Fbe.overwrite(fact, property_or_hash, values = nil, fb: Fbe.fb, fid: '_id')
     before[k.to_s] = fact[k]
   end
   id = fact[fid]&.first
-  raise "There is no #{fid} in the fact, cannot use Fbe.overwrite" if id.nil?
-  raise "No facts by #{fid} = #{id}" if fb.query("(eq #{fid} #{id})").delete!.zero?
+  raise(Fbe::Error, "There is no #{fid} in the fact, cannot use Fbe.overwrite") if id.nil?
+  raise(Fbe::Error, "No facts by #{fid} = #{id}") if fb.query("(eq #{fid} #{id})").delete!.zero?
   fb.txn do |fbt|
     n = fbt.insert
     before[property.to_s] = values
     before.each do |k, vv|
       next unless n[k].nil?
       vv.each do |v|
-        n.send(:"#{k}=", v)
+        n.public_send(:"#{k}=", v)
       end
     end
   end

data/lib/fbe/pmp.rb

@@ -42,12 +42,11 @@ require_relative 'fb'
 #
 #   # Get deadline from time area
 #   deadline = Fbe.pmp.time.deadline
-def Fbe.pmp(fb: Fbe.fb, global: $global, options: $options, loog: $loog)
+def Fbe.pmp(fb: Fbe.fb, global: $global, options: $options, loog: $loog) # rubocop:disable Metrics/AbcSize
   xml = Nokogiri::XML(File.read(File.join(__dir__, '../../assets/pmp.xml')))
   pmpv =
     Class.new(SimpleDelegator) do
-      attr_reader :default, :type, :memo, :value
-
+      attr_reader :default, :type, :memo, :value # rubocop:disable Layout/EmptyLinesAroundAttributeAccessor
       def initialize(value, default, type, memo)
         super(value)
         @value = value
@@ -63,7 +62,7 @@ def Fbe.pmp(fb: Fbe.fb, global: $global, options: $options, loog: $loog)
     others do |*args1|
       area = args1.first.to_s
       node = xml.at_xpath("/pmp/area[@name='#{area}']")
-      raise "Unknown area #{area.inspect}" if node.nil?
+      raise Fbe::Error, "Unknown area #{area.inspect}" if node.nil?
       Class.new do
         define_method(:properties) do
           node.xpath('p/name').map(&:text)
@@ -82,8 +81,8 @@ def Fbe.pmp(fb: Fbe.fb, global: $global, options: $options, loog: $loog)
             memo = prop.at_xpath('memo').text
             default =
               case type
-              when 'int' then default.to_i
-              when 'float' then default.to_f
+              when 'int' then Integer(default, 10)
+              when 'float' then Float(default)
               when 'bool' then default == 'true'
               else default
               end
@@ -91,8 +90,8 @@ def Fbe.pmp(fb: Fbe.fb, global: $global, options: $options, loog: $loog)
           result ||= default
           result =
             case type
-            when 'int' then result.to_i
-            when 'float' then result.to_f
+            when 'int' then Integer(Float(result).truncate)
+            when 'float' then Float(result)
             when 'bool' then result == 'true'
             else result
             end

data/lib/fbe/regularly.rb

@@ -30,11 +30,11 @@ require_relative 'fb'
 #     # PMP might have: days_between_cleanups=3, cleanup_history_days=30
 #   end
 def Fbe.regularly(area, p_every_days, p_since_days = nil, fb: Fbe.fb, judge: $judge, loog: $loog, &)
-  raise 'The area is nil' if area.nil?
-  raise 'The p_every_days is nil' if p_every_days.nil?
-  raise 'The fb is nil' if fb.nil?
-  raise 'The $judge is not set' if judge.nil?
-  raise 'The $loog is not set' if loog.nil?
+  raise(Fbe::Error, 'The area is nil') if area.nil?
+  raise(Fbe::Error, 'The p_every_days is nil') if p_every_days.nil?
+  raise(Fbe::Error, 'The fb is nil') if fb.nil?
+  raise(Fbe::Error, 'The $judge is not set') if judge.nil?
+  raise(Fbe::Error, 'The $loog is not set') if loog.nil?
   pmp = fb.query("(and (eq what 'pmp') (eq area '#{area}') (exists #{p_every_days}))").each.first
   interval = pmp.nil? ? 7 : pmp[p_every_days].first
   recent = fb.query(
@@ -59,7 +59,7 @@ def Fbe.regularly(area, p_every_days, p_since_days = nil, fb: Fbe.fb, judge: $ju
       since = Time.now - (days * 24 * 60 * 60)
       f.since = since
     end
-    yield f
+    yield(f)
   end
   nil
 end

data/lib/fbe/repeatedly.rb

@@ -32,11 +32,11 @@ require_relative 'overwrite'
 #     # PMP might have: hours_between_checks=6
 #   end
 def Fbe.repeatedly(area, p_every_hours, fb: Fbe.fb, judge: $judge, loog: $loog, &)
-  raise 'The area is nil' if area.nil?
-  raise 'The p_every_hours is nil' if p_every_hours.nil?
-  raise 'The fb is nil' if fb.nil?
-  raise 'The $judge is not set' if judge.nil?
-  raise 'The $loog is not set' if loog.nil?
+  raise(Fbe::Error, 'The area is nil') if area.nil?
+  raise(Fbe::Error, 'The p_every_hours is nil') if p_every_hours.nil?
+  raise(Fbe::Error, 'The fb is nil') if fb.nil?
+  raise(Fbe::Error, 'The $judge is not set') if judge.nil?
+  raise(Fbe::Error, 'The $loog is not set') if loog.nil?
   pmp = fb.query("(and (eq what 'pmp') (eq area '#{area}') (exists #{p_every_hours}))").each.first
   hours = pmp.nil? ? 24 : pmp[p_every_hours].first
   recent = fb.query(
@@ -54,6 +54,6 @@ def Fbe.repeatedly(area, p_every_hours, fb: Fbe.fb, judge: $judge, loog: $loog,
     f.what = judge
   end
   Fbe.overwrite(f, 'when', Time.now)
-  yield fb.query("(and (eq what '#{judge}'))").each.first
+  yield(fb.query("(and (eq what '#{judge}'))").each.first)
   nil
 end

data/lib/fbe/sec.rb

@@ -24,7 +24,7 @@ require_relative '../fbe'
 #   puts Fbe.sec(build_fact, :duration)  # => "2 hours ago"
 def Fbe.sec(fact, prop = :seconds)
   s = fact[prop.to_s]
-  raise "There is no #{prop.inspect} property" if s.nil?
-  s = s.first.to_i
+  raise(Fbe::Error, "There is no #{prop.inspect} property") if s.nil?
+  s = Integer(s.first.to_s, 10)
   (Time.now + s).ago
 end

data/lib/fbe/tombstone.rb

@@ -27,35 +27,41 @@ class Fbe::Tombstone
   # @param [Integer] repo ID of repository
   # @return [Array<Integer>] IDs of issue
   def issues(where, repo)
-    raise 'The type of "where" is not String' unless where.is_a?(String)
-    raise 'The type of "repo" is not Integer' unless repo.is_a?(Integer)
+    raise(Fbe::Error, 'The type of "where" is not String') unless where.is_a?(String)
+    raise(Fbe::Error, 'The type of "repo" is not Integer') unless repo.is_a?(Integer)
     f = @fb.query(
       "(and (eq where '#{where}') (eq what 'tombstone') (eq repository #{repo}) (exists issues))"
     ).each.first
     return [] if f.nil?
-    f['issues'].map do |ii|
-      a, b = ii.split('-').map(&:to_i)
+    f['issues'].flat_map do |ii|
+      a, b = ii.split('-').map { |i| Integer(i, 10) }
       b = a if b.nil?
-      (a..b).map { |i| i }
-    end.flatten
+      (a..b).to_a
+    end
   end
 
   # Put it there.
   # @param [String] where The place, e.g. "github"
   # @param [Integer] repo ID of repository
   # @param [Integer, Array<Integer>] issue ID of issue (or array of them)
-  def bury!(where, repo, issue)
-    raise 'The type of "where" is not String' unless where.is_a?(String)
-    raise 'The type of "repo" is not Integer' unless repo.is_a?(Integer)
-    raise 'The type of "issue" is neither Integer nor Array' unless issue.is_a?(Integer) || issue.is_a?(Array)
+  def bury!(where, repo, issue) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+    raise(Fbe::Error, 'The type of "where" is not String') unless where.is_a?(String)
+    raise(Fbe::Error, 'The type of "repo" is not Integer') unless repo.is_a?(Integer)
+    unless issue.is_a?(Integer) || issue.is_a?(Array)
+      raise(Fbe::Error, 'The type of "issue" is neither Integer nor Array')
+    end
     f =
       Fbe.if_absent(fb: @fb, always: true) do |n|
         n.what = 'tombstone'
         n.where = where
         n.repository = repo
       end
-    f.send(:"#{@fid}=", SecureRandom.random_number(99_999)) if f[@fid].nil?
-    nn = f['issues']&.map { |ii| ii.split('-').map(&:to_i).then { |ii| ii.size == 1 ? ii << ii[0] : ii } } || []
+    f.public_send(:"#{@fid}=", SecureRandom.random_number(99_999)) if f[@fid].nil?
+    nn = f['issues']&.map do |ii|
+      ii.split('-').map do |i|
+        Integer(i, 10)
+      end.then { |ii| ii.size == 1 ? ii << ii[0] : ii }
+    end || []
     issue = [issue] unless issue.is_a?(Array)
     issue.each do |i|
       nn << [i, i]
@@ -80,9 +86,11 @@ class Fbe::Tombstone
   # @param [Integer, Array<Integer>] issue ID of issue (or array of them)
   # @return [Boolean] True if it's there
   def has?(where, repo, issue)
-    raise 'The type of "where" is not String' unless where.is_a?(String)
-    raise 'The type of "repo" is not Integer' unless repo.is_a?(Integer)
-    raise 'The type of "issue" is neither Integer nor Array' unless issue.is_a?(Integer) || issue.is_a?(Array)
+    raise(Fbe::Error, 'The type of "where" is not String') unless where.is_a?(String)
+    raise(Fbe::Error, 'The type of "repo" is not Integer') unless repo.is_a?(Integer)
+    unless issue.is_a?(Integer) || issue.is_a?(Array)
+      raise(Fbe::Error, 'The type of "issue" is neither Integer nor Array')
+    end
     f = @fb.query(
       "(and (eq where '#{where}') (eq what 'tombstone') (eq repository #{repo}) (exists issues))"
     ).each.first
@@ -90,7 +98,7 @@ class Fbe::Tombstone
     issue = [issue] unless issue.is_a?(Array)
     issue.all? do |i|
       f['issues'].any? do |ii|
-        a, b = ii.split('-').map(&:to_i)
+        a, b = ii.split('-').map { |i| Integer(i, 10) }
         b.nil? ? a == i : (a..b).cover?(i)
       end
     end

data/lib/fbe/unmask_repos.rb

@@ -23,7 +23,7 @@ require_relative 'over'
 # @raise [RuntimeError] If organization part contains asterisk
 def Fbe.mask_to_regex(mask)
   org, repo = mask.split('/')
-  raise "Org '#{org}' can't have an asterisk" if org.include?('*')
+  raise(Fbe::Error, "Org '#{org}' can't have an asterisk") if org.include?('*')
   Regexp.compile("#{org}/#{repo.gsub('*', '.*')}", Regexp::IGNORECASE)
 end
 
@@ -59,12 +59,12 @@ end
 # @raise [RuntimeError] If no repositories match the provided masks
 # @note Exclusion patterns must start with '-' (e.g., '-org/pattern*')
 # @note Results are shuffled to distribute load when processing
-def Fbe.unmask_repos(
+def Fbe.unmask_repos( # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
   options: $options, global: $global, loog: $loog, epoch: $epoch || Time.now, kickoff: $kickoff || Time.now,
   quota_aware: true, lifetime_aware: true, timeout_aware: true
 )
-  raise 'Repositories mask is not specified' unless options.repositories
-  raise 'Repositories mask is empty' if options.repositories.empty?
+  raise(Fbe::Error, 'Repositories mask is not specified') unless options.repositories
+  raise(Fbe::Error, 'Repositories mask is empty') if options.repositories.empty?
   return if block_given? && Fbe.over?(
     global:, options:, loog:, epoch:, kickoff:, quota_aware:, lifetime_aware:, timeout_aware:
   )
@@ -77,7 +77,14 @@ def Fbe.unmask_repos(
       next
     end
     re = Fbe.mask_to_regex(mask)
-    octo.repositories(mask.split('/')[0]).each do |r|
+    org = mask.split('/')[0]
+    list =
+      begin
+        octo.organization_repositories(org, type: 'all')
+      rescue Octokit::NotFound
+        octo.repositories(org)
+      end
+    list.each do |r|
       repos << r[:full_name] if re.match?(r[:full_name])
     end
   end
@@ -86,13 +93,13 @@ def Fbe.unmask_repos(
     repos.reject! { |r| re.match?(r) }
   end
   repos.reject! { |repo| octo.repository(repo)[:archived] }
-  raise "No repos found matching: #{options.repositories.inspect}" if repos.empty?
+  raise(Fbe::Error, "No repos found matching: #{options.repositories.inspect}") if repos.empty?
   repos.shuffle!
   loog.debug("Scanning #{repos.size} repositories: #{repos.joined}...")
   repos.each { |repo| octo.repository(repo) }
   return repos unless block_given?
   repos.each do |repo|
     break if Fbe.over?(global:, options:, loog:, epoch:, kickoff:, quota_aware:, lifetime_aware:, timeout_aware:)
-    yield repo
+    yield(repo)
   end
 end

data/lib/fbe/who.rb

@@ -28,7 +28,7 @@ require_relative 'octo'
 #   puts Fbe.who(contributor, :author_id)  # => "@yegor256"
 def Fbe.who(fact, prop = :who, options: $options, global: $global, loog: $loog)
   id = fact[prop.to_s]
-  raise "There is no #{prop.inspect} property" if id.nil?
-  id = id.first.to_i
+  raise(Fbe::Error, "There is no #{prop.inspect} property") if id.nil?
+  id = Integer(id.first.to_s, 10)
   "@#{Fbe.octo(options:, global:, loog:).user_name_by_id(id)}"
 end

data/lib/fbe.rb

@@ -9,6 +9,6 @@
 # Copyright:: Copyright (c) 2024-2026 Zerocracy
 # License:: MIT
 module Fbe
-  # Current version of the gem (changed by +.rultor.yml+ on every release)
-  VERSION = '0.48.3' unless const_defined?(:VERSION)
+  VERSION = '0.48.4' unless const_defined?(:VERSION)
+  class Error < StandardError; end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fbe
 version: !ruby/object:Gem::Version
-  version: 0.48.3
+  version: 0.48.4
 platform: ruby
 authors:
 - Yegor Bugayenko
@@ -422,7 +422,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '3.0'
+      version: '3.2'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="