gitlab-dangerfiles 2.11.0 → 3.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 708f86af0c7d76e5df58d4ea85223461f39f1f7fb4248b3236d7baa92b626ccf
-  data.tar.gz: 22f2ee0d777dbc7ab95956f5bf2191c03449bc170baf9c9bacbd1e1e4a32ed69
+  metadata.gz: b25f2e8617ff29a3ea964b59774b02c395127c4800bfa1552affd2e71bf62a89
+  data.tar.gz: 813ca1c6dbd433cd43f1f5ef0efc44a514af3edf7cf7ed07d75eea50640817da
 SHA512:
-  metadata.gz: ad25c4c671fcde850a9c1740fb4e8dc847e4593cff3a55b4ef502bcb902cf84d5ddb4c7a0e64063371d252624f506b47047d32d292efa6ebee92f6db948162c5
-  data.tar.gz: '066975079aae0a2307c5b2a5734f4ff43d6556bd149c4dcdcac3586646e416c2abc1234ff3833359f24b4e5f19a1eece8af66c7c32b5b813df79cc5fb93bfbe6'
+  metadata.gz: d5208cc7b7c588c5fe7403f3b79631313a71c64007ed572fe8a5848a1c229f59f9e653f0b263c5c73efc85a1339fda852d2f44f376cca19f2d2ecac8e406b111
+  data.tar.gz: 4b31682875c0c7f5037463d035a57cb9ccd4215d68879105ce4a369539d1e155b6da7b620f942611d9d0fc92251bf58338614230bd16008dbd715974ffee57d9

data/README.md

@@ -67,6 +67,8 @@ Gitlab::Dangerfiles.for_project(self, 'my-project') do |dangerfiles|
 end
 ```
 
+Note that your custom plugins and rules (unless you exclude them with `except`) are automatically imported by the gem.
+
 ### Plugins
 
 Danger plugins are located under `lib/danger/plugins`.
@@ -85,8 +87,10 @@ Alternatively, you can also get/set configuration on the engine directly via `Gi
 
 #### Available general configurations
 
+- `project_root`: The project root path. You shouldn't have to override it.
 - `project_name`: The project name. Currently used by the Roulette plugin to fetch relevant
   reviewers/maintainers based on the project name. Default to `ENV["CI_PROJECT_NAME"]`.
+- `ci_only_rules`: A list of rules that cannot run locally.
 - `files_to_category`: A hash of the form `{ filename_regex => categories, [filename_regex, changes_regex] => categories }`.
   `filename_regex` is the regex pattern to match file names. `changes_regex` is the regex pattern to
   match changed lines in files that match `filename_regex`. Used in `helper.changes_by_category`, `helper.changes`, and `helper.categories_for_file`.
@@ -131,7 +135,9 @@ project. To use it in your project, perform the following steps:
 
 #### `type_label`
 
-This rule ensures the merge request has a proper [type label](https://about.gitlab.com/handbook/engineering/metrics/#work-type-classification) set.
+This rule warns when the merge request is missing a [type label](https://about.gitlab.com/handbook/engineering/metrics/#work-type-classification).
+
+If the `DANGER_ERROR_WHEN_TYPE_LABEL_IS_MISSING` environment variable evaluates to `true`, Danger will error when a type label is missing.
 
 If the `changelog` plugin is available, it also tries to infer a type label from the `Changelog` trailer of the MR.
 
@@ -166,6 +172,43 @@ include:
 
 See a [real world example](https://gitlab.com/gitlab-org/ruby/gems/gitlab-styles/-/merge_requests/105).
 
+#### Without `Gemfile`
+
+Danger is a Ruby project and uses [`bundler`](https://bundler.io/) to manage
+its dependencies. This requires a project to have a `Gemfile` and
+`Gemfile.lock` commited. This is helpful especially if Danger is also used
+locally - with `lefthook`, for example.
+
+In order to skip Ruby and `bundler` dependency in a project, use `bundle` commands directly
+in the CI configuration:
+
+```yaml
+include:
+  - project: 'gitlab-org/quality/pipeline-common'
+    file: '/ci/danger-review.yml'
+
+danger-review:
+  before_script:
+    - bundle init
+    # For latest version
+    - bundle add gitlab-dangerfiles
+    # OR
+    # For a pinned version
+    - bundle add gitlab-dangerfiles --version 3.1.0
+```
+
+## Rake tasks
+
+You can import this gem's Rake tasks by adding the following to your project's `Rakefile`:
+
+```ruby
+require 'gitlab-dangerfiles'
+
+Gitlab::Dangerfiles.load_tasks
+```
+
+That will add the `danger_local` Rake task that allows to run Danger locally.
+
 ## Documentation
 
 Latest documentation can be found at <https://www.rubydoc.info/gems/gitlab-dangerfiles>.
@@ -174,7 +217,9 @@ Latest documentation can be found at <https://www.rubydoc.info/gems/gitlab-dange
 
 After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.
 
-To install this gem onto your local machine, run `bundle exec rake install`. To release a new version, update the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, push git commits and tags, and push the `.gem` file to [rubygems.org](https://rubygems.org).
+To install this gem onto your local machine, run `bundle exec rake install`. 
+
+To release a new version, update the version number in `version.rb`, and get the MR merged by a maintainer. This will be then be packaged into a gem and pushed to [rubygems.org](https://rubygems.org) by the CI/CD.
 
 ## Contributing
 

data/gitlab-dangerfiles.gemspec

@@ -27,6 +27,7 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
+  spec.add_dependency "rake", ">= 0"
   spec.add_dependency "danger-gitlab", ">= 8.0.0"
   spec.add_dependency "danger", ">= 8.4.5"
 

data/lib/danger/plugins/changelog.rb

@@ -4,6 +4,7 @@ module Danger
   # Contains method to check the presense and validity of changelogs.
   class Changelog < Danger::Plugin
     NO_CHANGELOG_LABELS = [
+      "maintenance::refactor",
       "maintenance::pipelines",
       "maintenance::workflow",
       "ci-build",
@@ -39,7 +40,6 @@ module Danger
         This merge request requires a changelog entry because it [%<reason>s](https://docs.gitlab.com/ee/development/changelog.html#what-warrants-a-changelog-entry).
       MSG
     }.freeze
-    CHANGELOG_CONFIG_FILE = "#{ENV["CI_PROJECT_DIR"]}/.gitlab/changelog_config.yml"
     DEFAULT_CHANGELOG_CATEGORIES = %w[
       added
       fixed
@@ -235,21 +235,25 @@ module Danger
 
     private
 
+    def changelog_config_file
+      @changelog_config_file ||= File.join(helper.config.project_root, ".gitlab/changelog_config.yml")
+    end
+
     def valid_categories
       return @categories if defined?(@categories)
 
-      @categories = if File.exist?(CHANGELOG_CONFIG_FILE)
+      @categories = if File.exist?(changelog_config_file)
           begin
             YAML
-              .load_file(CHANGELOG_CONFIG_FILE)
+              .load_file(changelog_config_file)
               .fetch("categories")
               .keys
               .freeze
           rescue Psych::SyntaxError, Psych::DisallowedClass => ex
-            puts "#{CHANGELOG_CONFIG_FILE} doesn't seem to be a valid YAML file:\n#{ex.message}\nFallbacking to the default categories: #{DEFAULT_CHANGELOG_CATEGORIES}"
+            puts "#{changelog_config_file} doesn't seem to be a valid YAML file:\n#{ex.message}\nFallbacking to the default categories: #{DEFAULT_CHANGELOG_CATEGORIES}"
             DEFAULT_CHANGELOG_CATEGORIES
           rescue => ex
-            puts "Received an unexpected failure while trying to fetch categories at #{CHANGELOG_CONFIG_FILE}:\n#{ex.message}\nFallbacking to the default categories: #{DEFAULT_CHANGELOG_CATEGORIES}"
+            puts "Received an unexpected failure while trying to fetch categories at #{changelog_config_file}:\n#{ex.message}\nFallbacking to the default categories: #{DEFAULT_CHANGELOG_CATEGORIES}"
             DEFAULT_CHANGELOG_CATEGORIES
           end
         else

data/lib/danger/plugins/internal/helper.rb

@@ -17,7 +17,7 @@ module Danger
       none: "",
       qa: "~QA",
       ux: "~UX",
-      test: "~test ~Quality for `spec/features/*`",
+      test: "~test for `spec/features/*`",
       # Deprecated as of 2.3.0 in favor of tooling
       engineering_productivity: '~"Engineering Productivity" for CI, Danger',
       tooling: '~"maintenance::workflow" / ~"maintenance::pipelines" for CI, Danger',
@@ -142,7 +142,7 @@ module Danger
     end
 
     def release_automation?
-      gitlab_helper&.mr_author == RELEASE_TOOLS_BOT
+      mr_author == RELEASE_TOOLS_BOT
     end
 
     # @param items [Array<String>] An array of items to transform into a bullet list.

data/lib/danger/rules/commit_messages/Dangerfile

@@ -97,7 +97,7 @@ def lint_commits(commits)
       warn_or_fail_commits(multi_line_commit_linter)
       commit_linters.delete(multi_line_commit_linter) # Don't show an error (here) and a warning (below)
     elsif helper.ci? # We don't have access to the MR title locally
-      title_linter = lint_mr_title(gitlab.mr_json['title'])
+      title_linter = lint_mr_title(helper.mr_title)
       if title_linter.failed?
         warn_or_fail_commits(title_linter)
       end

data/lib/danger/rules/commits_counter/Dangerfile

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+if git.commits.size > 20
+  failure 'This MR has more than 20 commits. You need to rebase this branch to have fewer commits.'
+end

data/lib/danger/rules/type_label/Dangerfile

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+TYPE_LABEL_MISSING_MESSAGE = "Please add a [merge request type](https://about.gitlab.com/handbook/engineering/metrics/#work-type-classification) to this merge request."
+
 require_relative "../../../gitlab/dangerfiles/type_label_guesser"
 
 if respond_to?(:changelog) && !helper.has_scoped_label_with_scope?("type")
@@ -7,6 +9,10 @@ if respond_to?(:changelog) && !helper.has_scoped_label_with_scope?("type")
   helper.labels_to_add.concat(type_label_guesser.labels_from_changelog_categories(changelog.categories))
 end
 
-unless helper.has_scoped_label_with_scope?("type")
-  warn "Please add a [merge request type](https://about.gitlab.com/handbook/engineering/metrics/#work-type-classification) to this merge request."
+return if helper.has_scoped_label_with_scope?("type")
+
+if ENV["DANGER_ERROR_WHEN_TYPE_LABEL_IS_MISSING"]
+  fail TYPE_LABEL_MISSING_MESSAGE
+else
+  warn TYPE_LABEL_MISSING_MESSAGE
 end

data/lib/gitlab/dangerfiles/config.rb

@@ -3,10 +3,18 @@
 module Gitlab
   module Dangerfiles
     class Config
+      # @!attribute project_root
+      #   @return [String] the project root folder path.
+      attr_accessor :project_root
+
       # @!attribute project_name
       #   @return [String] the project name. Currently used by the Roulette plugin to fetch relevant reviewers/maintainers based on the project name. Default to +ENV["CI_PROJECT_NAME"]+.
       attr_accessor :project_name
 
+      # @!attribute ci_only_rules
+      #   @return [Array<String>] rules that cannot be run locally.
+      attr_accessor :ci_only_rules
+
       # @!attribute files_to_category
       #   @return [{Regexp => Array<Symbol>}, {Array<Regexp> => Array<Symbol>}] A hash of the form +{ filename_regex => categories, [filename_regex, changes_regex] => categories }+.
       #           +filename_regex+ is the regex pattern to match file names. +changes_regex+ is the regex pattern to
@@ -26,7 +34,9 @@ module Gitlab
 
       def initialize
         @files_to_category = {}
+        @project_root = nil
         @project_name = ENV["CI_PROJECT_NAME"]
+        @ci_only_rules = []
         @code_size_thresholds = DEFAULT_CHANGES_SIZE_THRESHOLDS
         @max_commits_count = DEFAULT_COMMIT_MESSAGES_MAX_COMMITS_COUNT
       end

data/lib/gitlab/dangerfiles/spec_helper.rb

@@ -24,7 +24,9 @@ module DangerSpecHelper
   # A stubbed out Dangerfile for use in tests
   def self.testing_dangerfile
     env = Danger::EnvironmentManager.new(testing_env)
-    Danger::Dangerfile.new(env, testing_ui)
+    Danger::Dangerfile.new(env, testing_ui).tap do |dangerfile|
+      dangerfile.defined_in_file = Dir.pwd
+    end
   end
 
   def self.fake_danger

data/lib/gitlab/dangerfiles/task_loader.rb

@@ -0,0 +1,15 @@
+require "rake"
+
+module Gitlab
+  module Dangerfiles
+    module TaskLoader
+      module_function
+
+      TASKS_DIR = File.expand_path("tasks", __dir__)
+
+      def load_tasks
+        Rake.application.add_import(*Dir.glob(File.join(TASKS_DIR, "*.rake")))
+      end
+    end
+  end
+end

data/lib/gitlab/dangerfiles/tasks/main.rake

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+desc "Run local Danger rules"
+task :danger_local do
+  require "open3"
+
+  stdout, stderr, status = Open3.capture3({}, *%w{bundle exec danger dry_run})
+
+  puts("#{stdout}#{stderr}")
+
+  exit(status.exitstatus.to_i)
+end

data/lib/gitlab/dangerfiles/version.rb

@@ -1,5 +1,5 @@
 module Gitlab
   module Dangerfiles
-    VERSION = "2.11.0"
+    VERSION = "3.2.0"
   end
 end

data/lib/gitlab/dangerfiles/weightage/reviewers.rb

@@ -13,15 +13,15 @@ module Gitlab
       # | reduced capacity reviewer    |                              1 |
       # | reviewer                     |                              2 |
       # | hungry reviewer              |                              4 |
-      # | reduced capacity traintainer |                              3 |
-      # | traintainer                  |                              6 |
-      # | hungry traintainer           |                              8 |
+      # | reduced capacity traintainer |                              1 |
+      # | traintainer                  |                              2 |
+      # | hungry traintainer           |                              6 |
       # +------------------------------+--------------------------------+
       #
       # @api private
       class Reviewers
         DEFAULT_REVIEWER_WEIGHT = Gitlab::Dangerfiles::Weightage::CAPACITY_MULTIPLIER * Gitlab::Dangerfiles::Weightage::BASE_REVIEWER_WEIGHT
-        TRAINTAINER_WEIGHT = 3
+        TRAINTAINER_WEIGHT = 2
 
         def initialize(reviewers, traintainers)
           @reviewers = reviewers
@@ -32,6 +32,8 @@ module Gitlab
           # TODO: take CODEOWNERS into account?
           # https://gitlab.com/gitlab-org/gitlab/issues/26723
 
+          remove_traintainers_from_reviewers!
+
           weighted_reviewers + weighted_traintainers
         end
 
@@ -39,25 +41,33 @@ module Gitlab
 
         attr_reader :reviewers, :traintainers
 
+        def remove_traintainers_from_reviewers!
+          # Sometimes folks will add themselves as traintainers and not remove themselves as reviewers.
+          # There seems no way currently to ensure only one of these entries exists for a person.
+          # We need to protect ourselves from that scenario here as the code assumes a reviewer will only
+          # appear in reviewers or traintainers, not both.
+          reviewers.reject! { |reviewer| traintainers.include?(reviewer) }
+        end
+
         def weighted_reviewers
           reviewers.each_with_object([]) do |reviewer, total_reviewers|
-            add_weighted_reviewer(total_reviewers, reviewer, Gitlab::Dangerfiles::Weightage::BASE_REVIEWER_WEIGHT)
+            add_weighted_reviewer(total_reviewers, reviewer, DEFAULT_REVIEWER_WEIGHT)
           end
         end
 
         def weighted_traintainers
           traintainers.each_with_object([]) do |reviewer, total_traintainers|
-            add_weighted_reviewer(total_traintainers, reviewer, TRAINTAINER_WEIGHT)
+            add_weighted_reviewer(total_traintainers, reviewer, DEFAULT_REVIEWER_WEIGHT + TRAINTAINER_WEIGHT)
           end
         end
 
-        def add_weighted_reviewer(reviewers, reviewer, weight)
+        def add_weighted_reviewer(reviewers, reviewer, added_weight_for_hungry)
           if reviewer.reduced_capacity
-            reviewers.fill(reviewer, reviewers.size, weight)
+            reviewers.fill(reviewer, reviewers.size, Gitlab::Dangerfiles::Weightage::BASE_REVIEWER_WEIGHT)
           elsif reviewer.hungry
-            reviewers.fill(reviewer, reviewers.size, weight * Gitlab::Dangerfiles::Weightage::CAPACITY_MULTIPLIER + DEFAULT_REVIEWER_WEIGHT)
+            reviewers.fill(reviewer, reviewers.size, DEFAULT_REVIEWER_WEIGHT + added_weight_for_hungry)
           else
-            reviewers.fill(reviewer, reviewers.size, weight * Gitlab::Dangerfiles::Weightage::CAPACITY_MULTIPLIER)
+            reviewers.fill(reviewer, reviewers.size, DEFAULT_REVIEWER_WEIGHT)
           end
         end
       end

data/lib/gitlab/dangerfiles.rb

@@ -1,18 +1,9 @@
 require "gitlab/dangerfiles/version"
+require "gitlab/dangerfiles/task_loader"
 
 module Gitlab
   module Dangerfiles
     RULES_DIR = File.expand_path("../danger/rules", __dir__)
-    EXISTING_RULES = Dir.glob(File.join(RULES_DIR, "*")).each_with_object([]) do |path, memo|
-      if File.directory?(path)
-        memo << File.basename(path)
-      end
-    end
-    LOCAL_RULES = %w[
-      changelog
-      changes_size
-      commit_messages
-    ].freeze
     CI_ONLY_RULES = %w[
       simple_roulette
       type_label
@@ -20,6 +11,10 @@ module Gitlab
       z_retry_link
     ].freeze
 
+    def self.load_tasks
+      TaskLoader.load_tasks
+    end
+
     # Utility method to construct a [Gitlab::Dangerfiles::Engine] instance,
     # which is yielded to the given block.
     #
@@ -29,6 +24,7 @@ module Gitlab
     # @return [Gitlab::Dangerfiles::Engine]
     def self.for_project(dangerfile, project_name = nil)
       Engine.new(dangerfile).tap do |engine|
+        engine.config.project_root = Pathname.new(File.dirname(dangerfile.defined_in_file))
         engine.config.project_name = project_name if project_name
 
         yield engine
@@ -60,15 +56,15 @@ module Gitlab
       #   end
       def import_plugins
         danger_plugin.import_plugin(File.expand_path("../danger/plugins/*.rb", __dir__))
+
+        Dir.glob(File.expand_path("danger/plugins/*.rb", config.project_root)).sort.each do |path|
+          puts "Importing plugin at #{path}" if dangerfile.verbose
+          danger_plugin.import_plugin(path)
+        end
       end
 
       # Import available Dangerfiles.
       #
-      # @deprecated
-      # @param rules [Symbol, Array<String>] Can be either +:all+ (default) to import all rules,
-      #   or an array of rules.
-      #   Available rules are: +changes_size+.
-      #
       # @param only [Symbol, Array<String>] An array of rules to import (defaults to all rules).
       #   Available rules are: +changes_size+.
       #
@@ -87,14 +83,15 @@ module Gitlab
       #     # Or import only a subset of rules, except a subset of rules
       #     dangerfiles.import_dangerfiles(only: %w[changes_size], except: %w[commit_messages])
       #   end
-      def import_dangerfiles(rules: nil, only: nil, except: [])
-        puts "The `:rules` parameter is deprecated in favor of `:only`." unless rules.nil?
+      def import_dangerfiles(only: nil, except: [])
+        return if helper_plugin.release_automation?
 
-        only ||= EXISTING_RULES if rules == :all
-        only ||= rules || EXISTING_RULES
+        rules = filtered_rules(only, except)
+        puts "Running rules: #{rules}\n" if dangerfile.verbose
 
-        filtered_rules(only, except).each do |rule|
-          danger_plugin.import_dangerfile(path: File.join(RULES_DIR, rule))
+        rules.each do |rule, path|
+          puts "Importing rule #{rule} at #{path}" if dangerfile.verbose
+          danger_plugin.import_dangerfile(path: path)
         end
       end
 
@@ -119,14 +116,43 @@ module Gitlab
 
       attr_reader :dangerfile
 
-      def allowed_rules
-        return LOCAL_RULES unless helper_plugin.respond_to?(:ci?)
+      def all_gem_rules
+        @all_gem_rules ||= Dir.glob(File.join(RULES_DIR, "*")).sort.each_with_object({}) do |path, memo|
+          rule_name = File.basename(path)
+          memo[rule_name] = path if File.directory?(path) && File.exist?(File.join(path, "Dangerfile"))
+        end
+      end
+
+      def custom_rules
+        @custom_rules ||= Dir.glob(File.expand_path("danger/*", config.project_root)).sort.each_with_object({}) do |path, memo|
+          rule_name = File.basename(path)
+          memo[rule_name] = path if File.directory?(path) && File.exist?(File.join(path, "Dangerfile"))
+        end
+      end
+
+      def all_rules
+        all_gem_rules.merge(custom_rules)
+      end
 
-        helper_plugin.ci? ? LOCAL_RULES | CI_ONLY_RULES : LOCAL_RULES
+      def local_rules
+        ci_only_rules = CI_ONLY_RULES | config.ci_only_rules
+        all_rules.reject { |rule, _v| ci_only_rules.include?(rule) }
       end
 
-      def filtered_rules(only, except)
-        (Array(only).map(&:to_s) & EXISTING_RULES & allowed_rules) - except
+      def allowed_rules_based_on_context
+        helper_plugin.ci? ? all_rules : local_rules
+      end
+
+      def filtered_rules(only_rules, except_rules)
+        only_rules = Array(only_rules).compact.map(&:to_s)
+
+        rules = allowed_rules_based_on_context.reject { |rule, _v| except_rules.include?(rule) }
+
+        if only_rules.any?
+          rules.select! { |rule, _v| only_rules.include?(rule) }
+        end
+
+        rules
       end
 
       def danger_plugin

metadata

@@ -1,15 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-dangerfiles
 version: !ruby/object:Gem::Version
-  version: 2.11.0
+  version: 3.2.0
 platform: ruby
 authors:
 - GitLab
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-03-15 00:00:00.000000000 Z
+date: 2022-06-14 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: danger-gitlab
   requirement: !ruby/object:Gem::Requirement
@@ -155,6 +169,7 @@ files:
 - lib/danger/rules/changelog/Dangerfile
 - lib/danger/rules/changes_size/Dangerfile
 - lib/danger/rules/commit_messages/Dangerfile
+- lib/danger/rules/commits_counter/Dangerfile
 - lib/danger/rules/simple_roulette/Dangerfile
 - lib/danger/rules/type_label/Dangerfile
 - lib/danger/rules/z_add_labels/Dangerfile
@@ -169,6 +184,8 @@ files:
 - lib/gitlab/dangerfiles/emoji_checker.rb
 - lib/gitlab/dangerfiles/merge_request_linter.rb
 - lib/gitlab/dangerfiles/spec_helper.rb
+- lib/gitlab/dangerfiles/task_loader.rb
+- lib/gitlab/dangerfiles/tasks/main.rake
 - lib/gitlab/dangerfiles/teammate.rb
 - lib/gitlab/dangerfiles/title_linting.rb
 - lib/gitlab/dangerfiles/type_label_guesser.rb