feature_map 1.2.2 → 1.2.3

data/lib/feature_map/private/code_cov.rb

@@ -1,4 +1,3 @@
-# typed: strict
 # frozen_string_literal: true
 
 require 'faraday'
@@ -12,26 +11,6 @@ module FeatureMap
       class ApiError < StandardError; end
       class ConfigurationError < StandardError; end
 
-      extend T::Sig
-
-      FilePath = T.type_alias { String }
-      CoverageStat = T.type_alias { String }
-
-      Coverage = T.type_alias do
-        T::Hash[
-          CoverageStat,
-          Integer
-        ]
-      end
-
-      TestCoverageStats = T.type_alias do
-        T::Hash[
-          FilePath,
-          Coverage
-        ]
-      end
-
-      sig { params(commit_sha: String, api_token: String).returns(TestCoverageStats) }
       def self.fetch_coverage_stats(commit_sha, api_token)
         commit_details_response = fetch_commit_details(commit_sha, api_token)
         raise ApiError, "Failed to retrieve CodeCov stats for commit #{commit_sha}. Response: #{commit_details_response.status} - #{commit_details_response.body}" unless commit_details_response.success?
@@ -39,14 +18,12 @@ module FeatureMap
         build_coverage_status(commit_details_response.body)
       end
 
-      sig { params(commit_sha: String, api_token: String).returns(T.untyped) }
       def self.fetch_commit_details(commit_sha, api_token)
         conn.get("#{service}/#{owner}/repos/#{repo}/commits/#{commit_sha}",
                  {},
                  { 'Authorization' => "Bearer #{api_token}" })
       end
 
-      sig { params(commit_details: T::Hash[T.untyped, T.untyped]).returns(TestCoverageStats) }
       def self.build_coverage_status(commit_details)
         file_coverage_details = commit_details.dig('report', 'files')
         raise ApiError, 'No file coverage information retruned from CodeCov.' unless file_coverage_details
@@ -66,30 +43,26 @@ module FeatureMap
       end
 
       # TODO: Move these values to config.
-      sig { returns(String) }
       def self.service
         Private.configuration.code_cov['service'] ||
           (raise ConfigurationError, 'Missing CodeCov configuration: service')
       end
 
-      sig { returns(String) }
       def self.owner
         Private.configuration.code_cov['owner'] ||
           (raise ConfigurationError, 'Missing CodeCov configuration: owner')
       end
 
-      sig { returns(String) }
       def self.repo
         Private.configuration.code_cov['repo'] ||
           (raise ConfigurationError, 'Missing CodeCov configuration: repo')
       end
 
-      sig { returns(Faraday::Connection) }
       def self.conn
-        @conn ||= T.let(Faraday.new(url: CODE_COV_API_BASE_URL) do |f|
+        @conn ||= Faraday.new(url: CODE_COV_API_BASE_URL) do |f|
           f.request :json
           f.response :json
-        end, T.nilable(Faraday::Connection))
+        end
       end
     end
   end

data/lib/feature_map/private/cyclomatic_complexity_calculator.rb

@@ -1,4 +1,3 @@
-# typed: strict
 # frozen_string_literal: true
 
 require 'parser/current'
@@ -6,20 +5,16 @@ require 'parser/current'
 module FeatureMap
   module Private
     class CyclomaticComplexityCalculator
-      extend T::Sig
-
       COMPLEXITY_NODES = %i[
         if case while until for
         rescue when and or
       ].freeze
 
-      sig { params(ast: T.nilable(Parser::AST::Node)).void }
       def initialize(ast)
         @ast = ast
-        @complexity = T.let(1, Integer) # Start at 1 for the base path
+        @complexity = 1 # Start at 1 for the base path
       end
 
-      sig { returns(Integer) }
       def calculate
         process(@ast)
         @complexity
@@ -27,7 +22,6 @@ module FeatureMap
 
       private
 
-      sig { params(node: T.nilable(T.any(Parser::AST::Node, Symbol, Integer, String, NilClass))).void }
       def process(node)
         return unless node.is_a?(Parser::AST::Node)