stefon 0.0.2

checksums.yaml

@@ -0,0 +1,15 @@
+---
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    OWNhNzAwM2UyYWVmNDA5NGE4MTEwZjAyMTYwODFmOGM1MWEwZjFlOQ==
+  data.tar.gz: !binary |-
+    NTJmYzhhZmE5NGQxYjllMmY0ZmQzM2U3OTMzOGM0ZTBjMGQ3MjlkNA==
+SHA512:
+  metadata.gz: !binary |-
+    NjZmNTgyNzY0ZGZjMjRlYmJiZTA5ODk1ODAxMTk4MTI3ZmVlMWY0N2I1OTAz
+    M2Y2NDg1OTNjNDM3MTM2YjlkZDYzYTgzZWU5NzdmNmRiZjFhOTAxOWJjMjBl
+    MDc0MGY3OTNjN2FmOWNhZWE3ZGM4MzgwZmZhMzM2OThjNTFhNWE=
+  data.tar.gz: !binary |-
+    NGE4MGQ4NWU1ODA2ZTQxOTExNzZhNjIxYTM0MTY3YTI2ODAyNGNiZjQwZDk2
+    ZDY4OWNhYWNkNDJiZDY4NGQ2ZTNkYTBjMWQyNDY0MTc0MTcxZTYxYmExYmUy
+    YTJmZjJlNGZkMzA3YTQ4NmMyMWUxOTE0ZTY3MjRlNTE1NzQxOTA=

data/.gitignore

@@ -0,0 +1,18 @@
+*.gem
+*.rbc
+.bundle
+.config
+.yardoc
+Gemfile.lock
+InstalledFiles
+_yardoc
+coverage
+doc/
+lib/bundler/man
+pkg
+rdoc
+spec/reports
+test/tmp
+test/version_tmp
+tmp
+.DS_Store

data/Gemfile

@@ -0,0 +1,3 @@
+source 'https://rubygems.org'
+
+gemspec

data/LICENSE.txt

@@ -0,0 +1,22 @@
+Copyright (c) 2014 Ilya Kavalerov
+
+MIT License
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/README.md

@@ -0,0 +1,75 @@
+# Stefon
+
+Get tips on who to ask for a code review.
+
+A name inspired by the SNL character Bill Hader played:
+![stefon on wikipedia](http://upload.wikimedia.org/wikipedia/en/c/c1/Stefon%2C_SNL_Character.jpg)
+
+SNL Stefon always gave advice on the best places to go to in the city. Stefon the gem will do the same: he tells you who to go to for a code review (or send a PR to) based on whose code you changed most.
+
+## Installation
+
+Add this line to your application's Gemfile:
+
+    gem 'stefon'
+
+And then execute:
+
+    $ bundle
+
+Or install it yourself as:
+
+    $ gem install stefon
+
+## Usage
+Execute:
+
+    $ stefon
+
+To get the top 4 people to send a code review to.
+
+You can use the -l option to change the number of people Stefon's recommendations are limited to:
+Execute:
+
+    $ stefon --limit 42
+
+To get the top 42 people to ask for a code review (identical to `stefon -l 42` ).
+
+To get a detailed report of code that you changed, execute:
+
+    $ stefon --full-report
+which is identical to `stefon -f`, to see something like:
+
+    $ stefon -f
+    Finished in 0.592418 seconds
+    Added 1 line to files written by: Cindy Sherman
+    Deleted 2 lines written by: Cindy Sherman
+    Added 1 line to files written by: Ed Ruscha
+    Deleted 1 line written by: Ed Ruscha
+    Deleted 1 file written by: Ed Ruscha
+    Deleted 1 line written by: Gerhard Richter
+    The top commiter in this repo is: Van Gogh
+
+
+## Todos / Comming improvements
+* give identity of predominant author of a file you have deleted
+* use Github API to get a list of contributors for a project (and filter suggestions)
+* loading user preferences (list of users to exclude) - maybe with dotfiles instead of yml files
+* handle errors from surveryors
+* remove dependency of Grit class on Git module
+* exclude authors from
+    * the grit module - file top author
+    * the git module - repo top commiter
+
+
+## Contributing
+
+1. Fork it
+2. Create your feature branch (`git checkout -b my-new-feature`)
+3. Commit your changes (`git commit -am 'Add some feature'`)
+4. Push to the branch (`git push origin my-new-feature`)
+5. Create new Pull Request
+
+### The narrative of stefon's code
+
+Stefon lives in the shell. We ask for suggestions by interacting with the `CLI` (using the trollop gem for getting cmd line options), the `CLI` gets an `Editor` to write a script for Stefon. The `Editor` has a team of `Surveyors` that look for certain changes in the codebase (`AddedLines`, `DeletedFiles`, etc.). To detect these changes, the `Surveyors` use `GitUtils` (a wrapper for cmd line git interface) and `GritUtils` (a wrapper for another gem called grit), and store scores in a `SurveyorStor`, which acts like a `Hash`. The `Editor` is responsible for assembling the team, telling them how detailed of a report he needs (`--full-report` option) and for how many users (--limit option), and combinning their reports, so that the `CLI` can print it out on the screen.

data/Rakefile

@@ -0,0 +1,11 @@
+# encoding: utf-8
+
+require 'rubygems'
+require "bundler/gem_tasks"
+require 'stefon/rake_task'
+
+
+desc 'Run stefon over himself'
+task :default do
+  Stefon::RakeTask.new
+end

data/bin/stefon

@@ -0,0 +1,34 @@
+#!/usr/bin/env ruby
+# encoding: utf-8
+
+if RUBY_VERSION >= '1.9.2' && RUBY_VERSION < '2.0.0'
+  $LOAD_PATH.unshift(File.dirname(File.realpath(__FILE__)) + '/../lib')
+
+  # test for presence of sed dependency
+  sed_location = `which sed`
+  if sed_location == ''
+    puts 'Stefon requires the cmd line utility sed.\
+      Install it with: brew install sed'
+    exit(-1)
+  end
+
+  require 'stefon'
+  require 'benchmark'
+  require 'trollop'
+
+  cli, result, opts = Stefon::CLI.new
+
+  time = Benchmark.realtime do
+    opts = Trollop.options(&Stefon::Options.get)
+    result = cli.run(opts)
+  end
+
+  puts "Finished in #{time} seconds"
+  header = 'Stefon recommends that you ask for a code review from:'
+  puts header unless opts[:full_report]
+  puts result
+  exit(0)
+else
+  puts 'Stefon supports only Ruby versions 1.9.X where X >= 2'
+  exit(-1)
+end

data/lib/stefon.rb

@@ -0,0 +1,14 @@
+# encoding: utf-8
+
+require 'stefon/version'
+require 'stefon/cli'
+require 'stefon/config'
+
+require 'stefon/surveyor/surveyor'
+require 'stefon/surveyor/grit_util'
+require 'stefon/surveyor/git_util'
+require 'stefon/surveyor/added_files'
+require 'stefon/surveyor/deleted_files'
+require 'stefon/surveyor/added_lines'
+require 'stefon/surveyor/deleted_lines'
+require 'stefon/surveyor/editor'

data/lib/stefon/cli.rb

@@ -0,0 +1,37 @@
+# encoding: utf-8
+
+module Stefon
+  # This class is responsible for handling the command line interface
+  class CLI
+    # The entry point for the application logic
+    def run(opts)
+      options = opts.merge(Config::Weights.get)
+      editor = Stefon::Editor.new(options)
+      editor.summarize_results
+    end
+  end
+
+  # This module holds custom behavior for dealing with the gem trollop
+  module Options
+    def self.get
+      proc do
+        version "stefon #{Stefon::VERSION} (c) 2014 Ilya Kavalerov"
+        banner <<-EOS
+          Stefon is a utilty that recommends who to ask for a code review.
+          He lets you know whose code you are affecting the most.
+
+          Usage:
+                 stefon [options]
+          where [options] are:
+        EOS
+
+        opt :limit, 'Limit the number of people that stephon suggests sending ' +
+          'a code review to',
+          default: 4, short: '-l'
+        opt :full_report, "Boolean for whether or not to include information " +
+          "about how you affected someone's code",
+          default: false, short: '-f'
+      end
+    end
+  end
+end

data/lib/stefon/config.rb

@@ -0,0 +1,36 @@
+# encoding: utf-8
+
+module Stefon
+  # this class represents extra information that the editor needs
+  # to know before writing a script for Stefon
+  module Config
+    # This module is responsible for ruling out/in authors for the repository
+    # based on user specification, or repo members on github
+    module ExcludedAuthors
+      def valid?(author)
+        # exclude_filter(author) && include_filter(author)
+        true
+      end
+    end
+
+    # This module is in charge of providing weights to rank certain kind of
+    # edits over others, Eg. Deleting Stephanie's line of code is more
+    # important than adding a line of code to Stephanie's file
+    module Weights
+      attr_reader :default
+      attr_accessor :custom
+
+      def self.get(custom = nil)
+        # in the future this will be provided arguments by the config loader
+        # class after it loads them from yaml files
+        @default = {
+          deleted_line: 2,
+          deleted_file: 4,
+          added_line: 1,
+          added_file: 1
+        }
+        @default unless custom
+      end
+    end
+  end
+end

data/lib/stefon/rake_task.rb

@@ -0,0 +1,32 @@
+# encoding: utf-8
+
+require 'rake'
+require 'rake/tasklib'
+
+module Stefon
+  # Provides a single rake task.
+  class RakeTask < Rake::TaskLib
+    attr_accessor :name
+    attr_accessor :verbose
+    attr_accessor :fail_on_error
+    attr_accessor :patterns
+    attr_accessor :formatters
+
+    def initialize
+      desc 'Run Stefon'
+
+      RakeFileUtils.send(:verbose, verbose) do
+        run_task(verbose)
+      end
+    end
+
+    def run_task(verbose)
+      require 'stefon'
+      cli = CLI.new
+      puts 'Running Stefon...' if verbose
+      result = cli.run(limit: 4)
+      puts result
+      abort('Stefon failed!') if fail_on_error
+    end
+  end
+end

data/lib/stefon/surveyor/added_files.rb

@@ -0,0 +1,28 @@
+# encoding: utf-8
+
+module Stefon
+  module Surveyor
+    # This class gives points to the top commiter of the repo
+    # proportionally to each file that a user adds
+    class AddedFiles < Surveyor::Base
+      def call
+        score_added_files.weight_scores(@weight)
+      end
+
+      def call_verbose
+        # There is only 1 top commiter
+        author = GitUtil.top_commiter.first
+        Surveyor::SurveyorStore[[
+          [author, ["The top commiter in this repo is: #{author}"]]
+        ]]
+      end
+
+      def score_added_files
+        if (num_added_files = @@grit.repo.status.added.count) > 0
+          @scores[GitUtil.top_commiter] += num_added_files
+        end
+        @scores
+      end
+    end
+  end
+end

data/lib/stefon/surveyor/added_lines.rb

@@ -0,0 +1,41 @@
+# encoding: utf-8
+
+module Stefon
+  module Surveyor
+    # This class gives points to the top author of a file in which
+    # a user deleted lines
+    class AddedLines < Surveyor::Base
+      def call
+        score_added_lines.weight_scores(@weight)
+      end
+
+      def call_verbose
+        array_version = score_added_lines.to_a.map do |pair|
+          desc = "Added #{pair.last} #{pair.last == 1 ? 'line' : 'lines' } " +
+            "to files written by: #{pair.first}"
+          [pair.first, [desc]]
+        end
+        Surveyor::SurveyorStore[array_version]
+      end
+
+      def score_added_lines
+        # give credit to the most frequent commiter in the file
+        added_lines_by_file.each_pair do |filename, numlines|
+          blame = @@grit.blame_for(filename)
+          top_author = @@grit.file_valid_top_author(blame, filename)
+          # multiplied by the number of lines that are added in the staged commit
+          @scores[top_author] += numlines
+        end
+        @scores
+      end
+
+      def added_lines_by_file
+        lines_per_file_store = Hash.new(0)
+        GitUtil.added_lines_by_file do |filename, line_in_file|
+          lines_per_file_store[filename] += 1
+        end
+        lines_per_file_store
+      end
+    end
+  end
+end

data/lib/stefon/surveyor/deleted_files.rb

@@ -0,0 +1,31 @@
+# encoding: utf-8
+
+module Stefon
+  module Surveyor
+    # This class gives points to the predominant author of a file
+    # that a user deletes
+    class DeletedFiles < Surveyor::Base
+      def call
+        score_deleted_files.weight_scores(@weight)
+      end
+
+      def call_verbose
+        array_version = score_deleted_files.to_a.map do |pair|
+          desc = "Deleted #{pair.last} #{pair.last == 1 ? 'file' : 'files' } " +
+            "written by: #{pair.first}"
+          [pair.first, [desc]]
+        end
+        Surveyor::SurveyorStore[array_version]
+      end
+
+      def score_deleted_files
+        @@grit.repo.status.deleted.keys.each do |filename|
+          blame = @@grit.blame_for(filename)
+          top_author = @@grit.file_valid_top_author(blame, filename)
+          @scores[top_author] += 1
+        end
+        @scores
+      end
+    end
+  end
+end

data/lib/stefon/surveyor/deleted_lines.rb

@@ -0,0 +1,39 @@
+# encoding: utf-8
+
+module Stefon
+  module Surveyor
+    class DeletedLines < Surveyor::Base
+      def call
+        score_deleted_lines.weight_scores(@weight)
+      end
+
+      def call_verbose
+        array_version = score_deleted_lines.to_a.map do |pair|
+          desc = "Deleted #{pair.last} #{pair.last == 1 ? 'line' : 'lines' } " +
+            "written by: #{pair.first}"
+          [pair.first, [desc]]
+        end
+        Surveyor::SurveyorStore[array_version]
+      end
+
+      def score_deleted_lines
+        deleted_lines_by_file.each_pair do |filename, lines|
+          blame = @@grit.blame_for(filename)
+          lines.each do |deleted_line|
+            valid_author = @@grit.valid_line_author(blame, deleted_line)
+            @scores[valid_author] += 1 if valid_author
+          end
+        end
+        @scores
+      end
+
+      def deleted_lines_by_file
+        lines_by_file_store = Hash.new([])
+        GitUtil.deleted_lines_by_file do |filename, line_in_file|
+          lines_by_file_store[filename] += [line_in_file]
+        end
+        lines_by_file_store
+      end
+    end
+  end
+end

data/lib/stefon/surveyor/editor.rb

@@ -0,0 +1,54 @@
+# encoding: utf-8
+
+module Stefon
+  # The editor is responsible for forming a team of surveyors and
+  # asking for and combining their results. The editor decides what story
+  # to run, i.e. to print recommendations or why recommendations are impossible
+  class Editor
+    attr_reader :options, :team
+    attr_accessor :errors
+
+
+    def initialize(options)
+      @options = options
+      # currently unused
+      @errors = []
+      # The editor has a team of surveyors, and tells them their importance
+      @team = [
+        Surveyor::AddedFiles.new(options[:added_file]),
+        Surveyor::AddedLines.new(options[:added_line]),
+        Surveyor::DeletedFiles.new(options[:deleted_file]),
+        Surveyor::DeletedLines.new(options[:deleted_line])
+      ]
+    end
+
+    def combine_short_reports
+      @team.reduce(Surveyor::SurveyorStore.new) do |a, e|
+        a.merge_scores(e.call)
+      end
+    end
+
+    def short_report
+      combine_short_reports.sort_by { |k, v| -v }
+    end
+
+    def combine_full_reports
+      @team.reduce(Surveyor::SurveyorStore.new([])) do |a, e|
+        a.merge_scores(e.call_verbose)
+      end
+    end
+
+    def full_report
+      # sort by the scores
+      combine_full_reports.sort_by { |k, v| -combine_short_reports[k] }
+    end
+
+    def summarize_results
+      if @options[:full_report]
+        full_report.first(@options[:limit]).map(&:last).flatten
+      else
+        short_report.first(@options[:limit]).map(&:first).flatten
+      end
+    end
+  end
+end

data/lib/stefon/surveyor/git_util.rb

@@ -0,0 +1,53 @@
+# encoding: utf-8
+
+module Stefon
+  module Surveyor
+    # A module of usefull utils for dealing with the git cli directly
+    module GitUtil
+      CURRENT_BRANCH ||= %x(git rev-parse --abbrev-ref HEAD).sub("\n", '')
+
+      def self.top_commiter
+        git_commiters = %x(git shortlog -s -n)
+        # make a hash of authors pointing to num commits
+        top_commiters = Hash[
+          git_commiters.split("\n").map do |numcommits_author|
+            numcommits_author.strip.split("\t").reverse
+          end
+        ]
+        top_commiters.sort_by { |a, numlines| numlines.to_i }.first
+      end
+
+      def self.diff_as_array(mode = nil)
+        # mode should be '+' for added lines, '-' for deleted lines, and
+        # none for all looks at a diff, optionally matches lines starting
+        # with mode char, cut off first char of each line
+        %x(git diff HEAD~#{GritUtil.new.num_sui_commits} -U0 |
+          #{mode ? "grep ^#{mode} | " : ""} sed 's/^.//'
+        ).split("\n").map(&:strip)
+      end
+
+      def self.added_lines_by_file(&block)
+        lines_by_file(diff_as_array('+'), '++', block)
+      end
+
+      def self.deleted_lines_by_file(&block)
+        lines_by_file(diff_as_array('-'), '--', block)
+      end
+
+      def self.lines_by_file(git_diff_as_array, filename_marker, block)
+        # delivers a line and its parent filename to a block
+        git_diff_as_array.each_with_index do |e, i|
+          line, lines_ahead = e, 1
+          # if the line is a filename, we want it to point to its lines
+          while (line[0..1] == filename_marker) && (i + lines_ahead < git_diff_as_array.length) do
+            next_line = git_diff_as_array[i + lines_ahead]
+            # next_lines should not be filenames
+            break if next_line[0..1] == filename_marker
+            block.call(line[5..-1], next_line)
+            lines_ahead += 1
+          end
+        end
+      end
+    end
+  end
+end

data/lib/stefon/surveyor/grit_util.rb

@@ -0,0 +1,52 @@
+# encoding: utf-8
+
+require 'grit'
+
+module Stefon
+  module Surveyor
+    # A class that abstracts dealing with the grit gem while respecting/knowing some
+    # restrictions, namely excluding authors
+    class GritUtil
+      attr_reader :repo, :num_sui_commits, :last_xenocommit
+
+      include Config::ExcludedAuthors
+
+      # About the attribute names
+      # num_sui_commits is important when a user makes multiple commits when
+      # working on someone else's project, to ensure that diffs do not
+      # include recent changes made by the user, the number of commits by the
+      # user (sui ~ self / origin) should be taken into account when calling
+      # a diff, so as to compare changes against the last commit not made by
+      # the user (self), but by another person (xeno ~ not self / not origin)
+      def initialize
+        @repo = Grit::Repo.new('.')
+        commits = @repo.commits(GitUtil::CURRENT_BRANCH)
+        # If you are the only commiter _ever_,  then num_sui_commits would be
+        # irrelevant perhaps an error should be raised? Since in this case
+        # only yourself would be recommended
+        @num_sui_commits = commits.find_index { |c| c.author.name != @repo.config['user.name'] } || 0
+        @last_xeno_commit = commits[@num_sui_commits]
+      end
+
+      def blame_for(filename)
+        @repo.blame(filename, @last_xeno_commit)
+      end
+
+      def valid_line_author(blame, line)
+        matched_line = blame.lines.detect { |l| l.line.strip == line }
+        author = matched_line.commit.author.name if matched_line
+        author if valid?(author)
+      end
+
+      def file_valid_top_author(blame, filename)
+        authored_lines = Hash.new(0)
+        blame.lines.each do |l|
+          author = l.commit.author.name
+          authored_lines[author] += 1 if valid?(author)
+        end
+        top_author_line_pair = authored_lines.max_by { |a, numlines| numlines }
+        top_author_line_pair.first if top_author_line_pair
+      end
+    end
+  end
+end

data/lib/stefon/surveyor/surveyor.rb

@@ -0,0 +1,44 @@
+# encoding: utf-8
+
+module Stefon
+  module Surveyor
+    # A store for the scores that each surveyor calculates, it takes
+    # the form of a hash where author names are keys, and points are
+    # values. Points are counts of lines or commits that belong to a person
+    class SurveyorStore < ::Hash
+      def initialize(default_val = 0)
+        super(default_val)
+      end
+
+      def merge_scores(scores_hash)
+        dup = self.dup
+        scores_hash.each_pair do |name, score|
+          dup[name] += score
+        end
+        dup
+      end
+
+      def weight_scores(weight)
+        dup = self.dup
+        dup.each_pair do |name, score|
+          dup[name] *= weight
+        end
+        dup
+      end
+    end
+
+    # A scaffold for concrete surveyors, meant to be extended
+    # This class calculates whose code the gem user is affecting
+    # the most for a particular kind of behavior (eg. line / file deletion)
+    class Base
+      attr_reader :weight
+      attr_accessor :scores
+
+      def initialize(weight)
+        @@grit ||= GritUtil.new
+        @scores = SurveyorStore.new
+        @weight = weight
+      end
+    end
+  end
+end

data/lib/stefon/version.rb

@@ -0,0 +1,5 @@
+# encoding: utf-8
+
+module Stefon
+  VERSION = '0.0.2'
+end

data/spec/cli_spec.rb

@@ -0,0 +1,15 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe 'CLI' do
+  describe 'integration spec' do
+    let(:cli) { Stefon::CLI.new }
+
+    context 'in general' do
+      it 'runs without errors' do
+        cli.run(limit: 4)
+      end
+    end
+  end
+end

data/spec/config_spec.rb

@@ -0,0 +1,27 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Stefon::Config do
+
+  describe 'reading configuration from files' do
+    it 'reads the default configuration'
+    it 'read the user provided configuration'
+  end
+
+  describe 'loading configuration from github' do
+    it 'can connect and form a list of users to filter by'
+  end
+
+  context 'without github options enabled' do
+    describe 'merging configurations' do
+      it 'ensures user configuration takes prescedence'
+    end
+  end
+
+  context 'with github enabled' do
+    describe 'merging configurations' do
+      it 'ensures github user configuration takes prescedence over user yml config'
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,11 @@
+# encoding: utf-8
+
+$:.unshift(File.dirname(__FILE__) + '/../lib')
+require 'stefon'
+require 'pry'
+
+RSpec.configure do |config|
+  config.treat_symbols_as_metadata_keys_with_true_values = true
+  config.run_all_when_everything_filtered = true
+  config.filter_run :focus
+end

data/spec/surveyor/editor_spec.rb

@@ -0,0 +1,57 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe 'Editor' do
+  let(:options) {{
+    limit: 4,
+    deleted_line: 2,
+    deleted_file: 4,
+    added_line: 1,
+    added_file: 1
+  }}
+
+  describe 'summarize results (integration)' do
+    before(:each) do
+      [
+        Stefon::Surveyor::AddedFiles,
+        Stefon::Surveyor::AddedLines,
+        Stefon::Surveyor::DeletedFiles,
+        Stefon::Surveyor::DeletedLines
+      ].each do |klass|
+        klass.any_instance.stub(:call).and_return(
+          'Cy Twombly' => 4,
+          'Christian Boltanski' => 2
+        )
+        klass.any_instance.stub(:call_verbose).and_return(
+          'Cy Twombly' => ['Deleted 4 lines by Cy Twombly'],
+          'Christian Boltanski' => ['Deleted 2 lines by Christian Boltanski']
+        )
+      end
+    end
+
+    it 'does not include numbers in the short report' do
+      options[:full_report] = false
+      editor = Stefon::Editor.new(options)
+      results = editor.summarize_results
+      results.should include 'Cy Twombly'
+      results.should include 'Christian Boltanski'
+      results.each do |reported_result|
+        reported_result.should_not =~ /\d/
+      end
+    end
+
+    it 'reports numbers in the verbose report' do
+      options[:full_report] = true
+      editor = Stefon::Editor.new(options)
+      results = editor.summarize_results
+      results.any? { |line| line == 'Deleted 4 lines by Cy Twombly' }.should be_true
+      results.any? { |line| line == 'Deleted 2 lines by Christian Boltanski' }.should be_true
+      results.each do |reported_result|
+        unless reported_result =~ /The top commiter in this repo is/
+          reported_result.should =~ /\d/
+        end
+      end
+    end
+  end
+end

data/spec/surveyor/git_util_spec.rb

@@ -0,0 +1,50 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Stefon::Surveyor::GitUtil do
+  describe '#lines_by_file' do
+    let(:git_diff_as_array) {[
+      "-- a/bin/stefon",
+      "-- a/lib/stefon/surveyor/deleted_lines.rb",
+      "-- a/lib/stefon/surveyor/git_util.rb",
+      "block.call(line, next_line)",
+      "-- a/lib/stefon/surveyor/grit_util.rb",
+      "repo.blame(filename, @last_xenocommit)",
+      "-- a/spec/surveyor/git_util_spec.rb",
+      "describe Stefon::Surveyor::GitUtil do",
+      "describe '#lines_by_file' do",
+      "it 'correctly yields lines belonging to files'",
+      "end"
+    ]}
+    let(:right_structure) {{
+      "lib/stefon/surveyor/git_util.rb" => ["block.call(line, next_line)"],
+      "lib/stefon/surveyor/grit_util.rb" => ["repo.blame(filename, @last_xenocommit)"],
+      "spec/surveyor/git_util_spec.rb" => [
+        "describe Stefon::Surveyor::GitUtil do",
+        "describe '#lines_by_file' do",
+        "it 'correctly yields lines belonging to files'",
+        "end"
+      ]
+    }}
+    let(:save_structure) { Hash.new([]) }
+    let(:block) { ->(filename, line) { save_structure[filename] += [line] } }
+
+    it 'correctly yields lines belonging to files' do
+      Stefon::Surveyor::GitUtil.lines_by_file(git_diff_as_array, '--', block)
+      save_structure.should == right_structure
+    end
+  end
+
+  describe '#diff_as_array' do
+    it 'correctly formats git diff output for - mode' do
+      pending('Would test this, but relies on console methods like sed')
+    end
+  end
+
+  describe '#top_commiter' do
+    it 'returns the top commiter for a repo' do
+      pending('Would test this, but relies on console methods like sed')
+    end
+  end
+end

data/spec/surveyor/surveyor_spec.rb

@@ -0,0 +1,32 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Stefon::Surveyor::SurveyorStore do
+  let(:first) { Stefon::Surveyor::SurveyorStore.new }
+  let(:second) { Stefon::Surveyor::SurveyorStore.new }
+
+  before(:each) do
+    first['Van Gogh'] += 1
+    second['Gerhard Richter'] += 1
+  end
+
+  it 'merges scores correctly' do
+    combo = first.merge_scores(second)
+    combo_same = second.merge_scores(first)
+    combo.should be == combo_same
+    combo.should == { 'Van Gogh' => 1, 'Gerhard Richter' => 1 }
+  end
+
+  it 'merges without mutating the receiver score' do
+    expect do
+      first.merge_scores(second)
+    end.to_not change { first }
+  end
+
+  it 'merges without mutating the message score' do
+    expect do
+      first.merge_scores(second)
+    end.to_not change { second }
+  end
+end

data/stefon.gemspec

@@ -0,0 +1,30 @@
+# encoding: utf-8
+
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require 'stefon/version'
+
+Gem::Specification.new do |spec|
+  spec.name          = "stefon"
+  spec.version       = Stefon::VERSION
+  spec.platform      = Gem::Platform::RUBY
+  spec.authors       = ["Ilya Kavalerov"]
+  spec.email         = ["ilya@artsymail.com"]
+  spec.description   = <<-EOF
+    A utility that recommends who to ask for a code review.
+    Stefon tells you whose code you are affecting the most.
+  EOF
+  spec.summary       = %q{A utility that recommends who to ask for a code review}
+  spec.homepage      = "https://github.com/ilyakava/stefon"
+  spec.license       = "MIT"
+
+  spec.files         = `git ls-files`.split($/)
+  spec.executables   = spec.files.grep(/^bin\//) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = ["lib"]
+
+  spec.add_runtime_dependency "grit", "2.5.0"
+  spec.add_runtime_dependency "trollop", "~> 2.0"
+  spec.add_development_dependency "bundler", "~> 1.3"
+  spec.add_development_dependency "rake"
+end

metadata

@@ -0,0 +1,134 @@
+--- !ruby/object:Gem::Specification
+name: stefon
+version: !ruby/object:Gem::Version
+  version: 0.0.2
+platform: ruby
+authors:
+- Ilya Kavalerov
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2014-01-27 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: grit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 2.5.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 2.5.0
+- !ruby/object:Gem::Dependency
+  name: trollop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+description: ! "    A utility that recommends who to ask for a code review.\n    Stefon
+  tells you whose code you are affecting the most.\n"
+email:
+- ilya@artsymail.com
+executables:
+- stefon
+extensions: []
+extra_rdoc_files: []
+files:
+- .gitignore
+- Gemfile
+- LICENSE.txt
+- README.md
+- Rakefile
+- bin/stefon
+- lib/stefon.rb
+- lib/stefon/cli.rb
+- lib/stefon/config.rb
+- lib/stefon/rake_task.rb
+- lib/stefon/surveyor/added_files.rb
+- lib/stefon/surveyor/added_lines.rb
+- lib/stefon/surveyor/deleted_files.rb
+- lib/stefon/surveyor/deleted_lines.rb
+- lib/stefon/surveyor/editor.rb
+- lib/stefon/surveyor/git_util.rb
+- lib/stefon/surveyor/grit_util.rb
+- lib/stefon/surveyor/surveyor.rb
+- lib/stefon/version.rb
+- spec/cli_spec.rb
+- spec/config_spec.rb
+- spec/spec_helper.rb
+- spec/surveyor/editor_spec.rb
+- spec/surveyor/git_util_spec.rb
+- spec/surveyor/surveyor_spec.rb
+- stefon.gemspec
+homepage: https://github.com/ilyakava/stefon
+licenses:
+- MIT
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.1.5
+signing_key: 
+specification_version: 4
+summary: A utility that recommends who to ask for a code review
+test_files:
+- spec/cli_spec.rb
+- spec/config_spec.rb
+- spec/spec_helper.rb
+- spec/surveyor/editor_spec.rb
+- spec/surveyor/git_util_spec.rb
+- spec/surveyor/surveyor_spec.rb