dug 0.2.1 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9aafb19505a577e0a92f20d17a00c462967940d5
-  data.tar.gz: aff86cd29722fee471ad48976485d68d67a616cd
+  metadata.gz: 165051f76455d5a9922ab89df1c866abe8f655aa
+  data.tar.gz: 4549c773dfb062ca0bbf472d0b096fe897ad4b28
 SHA512:
-  metadata.gz: d9ed3803b8e8ab13f029b66a81ca19e0010b630d5f96d9462be464cc2e32fefebafd6741512a1866d5ea6f791511ac2ba2163662c5490fdbe881ded47f72f467
-  data.tar.gz: 889aa7a2ae8ee3020188897be7dd79196770a41cfb1a5c4d5e691315e50fb435f4a33bcb6ebc4b03c282002121863024901f9f879c75758e294b341dc627372f
+  metadata.gz: 4d9357bda24f78bbe23a47fcfe74dca8c59604eea493e98a0b7493d9d3495f4e816b9e4909ea05bb1b605b9e16eec0ca5bdd7b28243d9aead462e6fe466f695a
+  data.tar.gz: 830abd72bb43db5ddbd6e5c45a6439522c2941e92ef6c386761463bdf869bc445220e672c25acc47ec926f8c34d13acff07bbfff4ea1c141e0b08f66f0b8ebb3

data/.travis.yml

@@ -1,4 +1,7 @@
 language: ruby
+branches:
+  only:
+    - master
 rvm:
   - 2.1
   - 2.2

data/CHANGELOG.md

@@ -0,0 +1,9 @@
+### Unreleased
+[Full Changelog](http://github.com/chrisarcand/dug/compare/v1.0.0...master)
+
+Nothing yet!
+
+### 1.0.0 / 2016-02-11
+[Full Changelog](http://github.com/chrisarcand/dug/compare/65eec3580879c3d4dc27bb6d6ed7716f965ec6ff...v1.0.0)
+
+1.0!

data/Gemfile

@@ -1,8 +1,3 @@
 source 'https://rubygems.org'
 
-# Specify your gem's dependencies in dug.gemspec
 gemspec
-
-group :test do
-  gem "codeclimate-test-reporter", require: nil
-end

data/README.md

@@ -1,10 +1,11 @@
 # Dug
 [![Gem Version](https://badge.fury.io/rb/dug.svg)](https://badge.fury.io/rb/dug)
 [![Build Status](https://travis-ci.org/chrisarcand/dug.svg?branch=master)](https://travis-ci.org/chrisarcand/dug)
-[![Code Climate](https://codeclimate.com/github/chrisarcand/dug/badges/gpa.svg)](https://codeclimate.com/github/chrisarcand/dug)
 [![Test Coverage](https://codeclimate.com/github/chrisarcand/dug/badges/coverage.svg)](https://codeclimate.com/github/chrisarcand/dug/coverage)
+[![Code Climate](https://codeclimate.com/github/chrisarcand/dug/badges/gpa.svg)](https://codeclimate.com/github/chrisarcand/dug)
+[![License](http://img.shields.io/:license-mit-blue.svg?style=flat)](http://chrisarcand.mit-license.org)
 
-Created [out of frustration.](http://chrisarcand.com/introducing-dug/) _"[D]amn yo[u], [G]mail!"_
+Created out of frustration. _"[D]amn yo[u], [G]mail!"_
 
 **Dug is a simple, configurable gem to organize your GitHub notification emails
 in ways Gmail can't and in an easier-to-maintain way than large, slow Google
@@ -13,16 +14,12 @@ people usually do with Gmail filters (label by organization name, repository
 name) as well as parse GitHub's custom X headers to label your messages with
 things like "Mentioned by name", "Assigned to me", "Commented", etc.
 
-![](http://screenshots.chrisarcand.com/permxqkc5.jpg)
-
-## Quick Installation
+![](http://screenshots.chrisarcand.com/permd0u3k.jpg)
 
-Dug is meant to be stupid simple. It's practically a gemified script with a
-configurable API. As such, you can programmatically configure and execute Dug's
-runner class in any way you see fit (within a web app hook, a Rake task, a script,
-whatever).
+You can read more about the reasoning behind dug and why it's superior compared to other labeling methods in [my post
+introducing this project.](http://chrisarcand.com/introducing-dug/)
 
-Basic installation steps:
+## Quick Installation
 
 1. **Install Dug**
 
@@ -35,45 +32,67 @@ Basic installation steps:
   ```
   ---
   organizations:
-    - rails
-    - name: rspec
-      label: RSpec
-      repositories:
-        - name: rspec-expectations
-          label: RSpec/rspec-expectations
-    - ManageIQ
+    rails: Rails
+    rspec: RSpec
+    ManageIQ: ManageIQ
+
+  repositories:
+    rspec-expectations: RSpec/rspec-expectations
+    dug: dug
+    dotfiles:
+      - remote: chrisarcand
+        label: My dotfiles
+      - remote: juliancheal
+        label: Julian's dotfiles
 
   reasons:
-    author:
-      label: Participating
-    comment:
-      label: Participating
-    mention:
-      label: Mentioned by name
-    team_mention:
-      label: Team mention
-    assign:
-      label: Assigned to me
+    author: Participating
+    comment: Participating
+    mention: Mentioned
+    team_mention: Team mention
+    assign: Assigned to me
+
+  states:
+    merged: Merged
+    closed: Closed
   ```
 
   The above rule file will:
-  * Label all notifications from the organization `rails` with the label `rails`, `rspec` with `RSpec`, `ManageIQ` with `ManageIQ`.
-  * Label all notifications from the repository `rspec-expectations` with the label `RSpec/rspec-expectations`
+
+  **For organizations...**
+
+  * Label notifications from the organization `rails` with the label `rails`, `rspec` with `RSpec`, `ManageIQ` with `ManageIQ`.
+
+  **For repositories...**
+
+  * Label notifications from the repository `rspec-expectations` with the label `RSpec/rspec-expectations`, `dug` with `dug`.
+  * Label notifications from chrisarcand's `dotfiles` repository with the label `My dotfiles`, juliancheal's with `Julian's dotfiles`.
+
+  **For the reason you're being notified...**
+
   * Label notifications with `Participating` if I am the author of the Issue/PR or if I commented on it.
   * Label notifications with `Mentioned by name` if I'm directly mentioned in it.
   * Label notifications with `Team mention` if a team I am a part of is mentioned in it.
   * Label notifications with `Assigned to me` if the Issue/PR is assigned to me.
 
+  **For state changes to the Issue/PR...**
+
+  * Label notifications that signal the issue as closed with `Closed`. This label will be automatically removed if the issue is reopened!
+  * Label notifications that signal the issue as merged with `Merged`
+
 3. **Configure Gmail.** In Gmail...
 
    * Create the label "GitHub" and then "Unprocessed" nested underneath it (will show up as "GitHub/Unprocessed").
    * Create all of the labels in the preceding step if you don't have them already.
-   * Set up the following filters:
+   * Set up the following filters. The 'GitHub/Unprocessed' label is the only required part, but I recommend you
+     skip your inbox or you will be pinged with GitHub notifications _a lot_.
      ```
      Matches: from:(notifications@github.com)
-     Do this: Apply label "GitHub/Unprocessed"
+     Do this: Apply label "GitHub/Unprocessed", Skip Inbox
 
-     # This filter is optional but highly recommended.
+     # Optionally, you may want to *remove the Skip Inbox from the previous filter* and
+     # add this one. This means messages directed at you go to your Inbox (setting
+     # off a notification) and the rest going straight to your archive (no notification).
      Matches: from:(notifications@github.com) -{cc: youremail@example.non}
      Do this: Skip Inbox
      ```
@@ -98,14 +117,16 @@ Basic installation steps:
    Dug::Runner.run
    ```
 
-6. **Run the script** and watch your notifications get organized! The first time your run this you will be given a link to
-   visit in your browser to sign in to Gmail verify via a one time token.
+6. **Run the script** and watch your notifications get organized! The first
+   time you run this you will be given a link to visit in your browser to sign
+   in to Gmail verify via a one time token. Also note each call to `#run`
+   processes 100 unprocessed notifications at a time.
 
   ```
   $ ruby script.rb
   ```
 
-7. Set a cron and forget about it (this is what I do, 60 second polling). Or deploy Dug in a web application. Or even
+7. Set a cron and forget about it. I do, on a private VPS, with 60 second polling. Or deploy Dug in a web application. Or even
    just write a loop in your script `loop do; Dug::Runner.run; sleep 60; end`. How you run it is completely up to you,
    and really doesn't matter.
 
@@ -159,40 +180,12 @@ Dug.configure do |config|
 end
 ```
 
-### More configuration options
-
-You can use public methods in Dug's configuration class to programmatically
-configure Dug without a Rules YAML file.  As a documenting example, here is the
-same scenario in the YAML file used previously in this README but within a config
-block:
-
-```ruby
-Dug.configure do |config|
-  config.set_organization_rule('rails')
-  config.set_organization_rule('rspec', label: 'RSpec')
-  config.set_organization_rule('ManageIQ')
-
-  config.set_repository_rule('rspec-expectations', label: 'RSpec/rspec-expectations')
-
-  config.set_reason_rule('author',       label: 'Participating')
-  config.set_reason_rule('comment',      label: 'Participating')
-  config.set_reason_rule('mention',      label: 'Mention by name')
-  config.set_reason_rule('team_mention', label: 'Team mention')
-end
-```
-
 ## Development
 
 After checking out the repo, run `bin/setup` to install dependencies. Then, run
 `rake test` to run the tests. You can also run `bin/console` for an interactive
 prompt that will allow you to experiment.
 
-## TODO
-* Add option to parse message body for labelling
-* Add more built-in reasons such as "Closed" statuses
-* The Google Auth Library has an API for Redis that could be supported easily.
-* Moar tests, plz
-
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/chrisarcand/dug.

data/bin/console

@@ -3,12 +3,5 @@
 require "bundler/setup"
 require "dug"
 
-# You can add fixtures and/or initialization code here to make experimenting
-# with your gem easier. You can also use a different console, if you like.
-
-# (If you use this, don't forget to add pry to your Gemfile!)
-# require "pry"
-# Pry.start
-
-require "irb"
-IRB.start
+require "pry"
+Pry.start

data/dug.gemspec

@@ -10,7 +10,7 @@ Gem::Specification.new do |spec|
   spec.email         = ["chris@chrisarcand.com"]
 
   spec.summary       = %q{[D]amn yo[u], [G]mail. A gem to organize your GitHub notification emails in ways Gmail filters can't.}
-  spec.description   = %q{[D]amn yo[u], [G]mail. A gemified script to organize your GitHub notification emails using a simple configuration file in ways Gmail filters can't, such as X-GitHub-Reason headers.}
+  spec.description   = %q{A simple, configurable gem to organize your GitHub notification emails in ways Gmail can't and in an easier-to-maintain way than large, slow Google Apps Scripts.}
   spec.homepage      = "https://github.com/chrisarcand/dug"
   spec.license       = "MIT"
 
@@ -20,8 +20,14 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_dependency "google-api-client", "~> 0.9"
+  spec.add_dependency "activesupport", ">= 2.2.1"
 
   spec.add_development_dependency "bundler", "~> 1.11"
   spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "pry", "~> 0"
   spec.add_development_dependency "minitest", "~> 5.0"
+  spec.add_development_dependency "mocha", "~> 1.0"
+  spec.add_development_dependency "minitest-reporters", "~> 1.0"
+  spec.add_development_dependency "codeclimate-test-reporter", "~> 0"
+  spec.add_development_dependency "simplecov", "~> 0"
 end

data/lib/dug.rb

@@ -1,11 +1,17 @@
 require "dug/version"
+require "dug/validations"
 require "dug/configurator"
 require "dug/gmail_servicer"
 require "dug/logger"
+require "dug/message_processor"
 require "dug/notification_decorator"
 require "dug/runner"
 
 module Dug
+  LABEL_RULE_TYPES = %w(organization repository reason state)
+  GITHUB_REASONS = %w(author comment mention team_mention state_change assign manual subscribed)
+  ISSUE_STATES = %(merged closed reopened)
+
   def self.authorize!
     Dug::GmailServicer.new.authorize!
   end

data/lib/dug/configurator.rb

@@ -1,49 +1,20 @@
 require 'yaml'
+require 'active_support/inflector'
 
 module Dug
   class Configurator
-    LABEL_RULE_TYPES = %i(organization repository reason)
-    GITHUB_REASONS = %w(author comment mention team_mention state_change assign manual subscribed)
+    include Dug::Validations
 
     attr_accessor :client_id
     attr_accessor :client_secret
     attr_accessor :token_store
     attr_accessor :application_credentials_file
+    attr_accessor :unprocessed_label_name
 
     def initialize
-      self.label_rules = { "organizations" => {}, "reasons" => {} }
-    end
-
-    def set_organization_rule(name, label: nil)
-      organizations[name] ||= { "repositories" => {} }
-      organizations[name]["label"] = label || name
-    end
-
-    def set_repository_rule(name, organization:, label: nil)
-      organizations[organization] ||= { "repositories" => {} }
-      organizations[organization]["repositories"][name] ||= {}
-      organizations[organization]["repositories"][name]["label"] = label || name
-    end
-
-    def set_reason_rule(name, label: nil)
-      validate_reason(name)
-      reasons[name] ||= {}
-      reasons[name]["label"] = label || name
-    end
-
-    def label_for(type, name, organization: nil)
-      validate_label_type(type)
-      case type
-      when :organization
-        organizations.fetch(name, {})["label"]
-      when :repository
-        raise ArgumentError, "Repository label rules require an organization to be specified" unless organization
-        organizations.fetch(organization, {})
-                     .fetch("repositories", {})
-                     .fetch(name, {})["label"]
-      when :reason
-        validate_reason(name)
-        reasons.fetch(name, {})["label"]
+      @label_rules = {}
+      LABEL_RULE_TYPES.each do |type|
+        label_rules[type.pluralize] = {}
       end
     end
 
@@ -63,6 +34,10 @@ module Dug
       ENV['TOKEN_STORE_PATH'] || @token_store || File.join(Dir.home, ".dug", "authorization.yaml")
     end
 
+    def unprocessed_label_name
+      @unprocessed_label_name || "GitHub/Unprocessed"
+    end
+
     def rule_file
       @rule_file
     end
@@ -73,53 +48,60 @@ module Dug
       @rule_file
     end
 
-    private
-
-    attr_accessor :label_rules
+    def label_for(type, name, opts={})
+      type = type.to_s
+      run_validations(type, name)
 
-    def load_rules
-      file = YAML.load_file(rule_file)
-
-      file["organizations"].each do |org|
-        case org
-        when String
-          set_organization_rule(org)
-        when Hash
-          set_organization_rule(org["name"], label: org["label"])
-          if repos = org["repositories"]
-            repos.each do |repo|
-              set_repository_rule(repo["name"], organization: org["name"], label: repo["label"])
-            end
+      rule = label_rules[type.pluralize][name]
+      case rule
+      when String, nil
+        rule
+      when Array
+        if type == 'repository'
+          raise ArgumentError, "Multiple remotes possible and no remote specified" unless opts.keys.include?(:remote)
+          rule = rule.detect do |r|
+            r['remote'] == opts[:remote]
           end
+          rule['label'] if rule
         end
       end
-
-      file["reasons"].keys.each do |reason|
-        validate_reason(reason)
-        set_reason_rule(reason, label: file["reasons"][reason]["label"])
-      end
     end
 
-    def validate_label_type(type)
-      unless LABEL_RULE_TYPES.include?(type)
-        raise ConfigurationError, "'#{type}' is not a valid label rule type. Valid types: #{}"
+    def _clear!
+      %w(
+        application_credentials_file
+        client_id
+        client_secret
+        token_store
+        unprocessed_label_name
+        rule_file
+      ).each do |var|
+        instance_variable_set("@#{var}", nil)
       end
+
+      initialize
     end
 
-    def validate_reason(reason)
-      unless GITHUB_REASONS.include?(reason)
-        raise ConfigurationError, "'#{reason}' is not a valid GitHub notification reason. Valid reasons include: #{GITHUB_REASONS.map { |x| "'#{x}'" }.join(', ')}"
+    private
+
+    attr_accessor :label_rules
+
+    def load_rules
+      # TODO should validate incoming YAML
+      loaded_rules = YAML.load_file(rule_file)
+
+      LABEL_RULE_TYPES.each do |type|
+        type = type.pluralize
+        @label_rules[type] = loaded_rules[type] if loaded_rules[type]
       end
-    end
 
-    def organizations
-      label_rules["organizations"]
+      @label_rules
     end
 
-    def reasons
-      label_rules["reasons"]
+    def run_validations(type, name)
+      validate_rule_type!(type)
+      validate_reason!(name) if type == 'reason'
+      validate_state!(name) if type == 'state'
     end
   end
-
-  ConfigurationError = Class.new(StandardError)
 end

data/lib/dug/gmail_servicer.rb

@@ -5,6 +5,7 @@ require 'fileutils'
 require 'forwardable'
 
 module Dug
+  # @private
   class GmailServicer
     extend Forwardable
 
@@ -14,7 +15,8 @@ module Dug
 
     def_delegators :@gmail, :get_user_message,
                             :list_user_messages,
-                            :modify_message
+                            :modify_message,
+                            :modify_thread
 
     def initialize
       @gmail = Google::Apis::GmailV1::GmailService.new
@@ -29,13 +31,13 @@ module Dug
     end
 
     def add_labels_by_name(*args)
-      modify_message_request(*args) do |request, ids|
+      modification_request(*args) do |request, ids|
         request.add_label_ids = ids
       end
     end
 
     def remove_labels_by_name(*args)
-      modify_message_request(*args) do |request, ids|
+      modification_request(*args) do |request, ids|
         request.remove_label_ids = ids
       end
     end
@@ -73,7 +75,7 @@ module Dug
 
     private
 
-    def modify_message_request(message, label_names)
+    def modification_request(message, label_names, entire_thread: false)
       ids = []
       label_names.each do |name|
         unless labels[name] && id = labels[name].id
@@ -81,9 +83,20 @@ module Dug
         end
         ids << id
       end
-      request = Google::Apis::GmailV1::ModifyMessageRequest.new
+      request =
+        if entire_thread
+          Google::Apis::GmailV1::ModifyThreadRequest.new
+        else
+          Google::Apis::GmailV1::ModifyMessageRequest.new
+        end
+
       yield request, ids
-      modify_message('me', message.id, request)
+
+      if entire_thread
+        modify_thread('me', message.thread_id, request)
+      else
+        modify_message('me', message.id, request)
+      end
     end
   end
 

data/lib/dug/logger.rb

@@ -1,7 +1,12 @@
 module Dug
+  # @private
   module Logger
+    # TODO This obviously needs a lot of love
+
     def log(message, level: :info)
-      puts "[#{level.to_s.upcase}] #{Time.now} - #{message}"
+      unless $testing
+        puts "[#{level.to_s.upcase}] #{Time.now} - #{message}"
+      end
     end
   end
 end

data/lib/dug/message_processor.rb

@@ -0,0 +1,70 @@
+module Dug
+  # @private
+  class MessageProcessor
+    include Dug::Logger
+
+    def initialize(message_id, servicer)
+      @servicer = servicer
+      @message  = NotificationDecorator.new(servicer.get_user_message('me', message_id))
+    end
+
+    def execute
+      %w(organization repository reason).each do |type|
+        if message_data = message.public_send(type)
+          opts = type == 'repository' ? { remote: message.public_send(:organization) } : {}
+          label = Dug.configuration.label_for(type, message_data, opts)
+          labels_to_add << label if label
+        end
+      end
+
+      %w(merged closed).each do |state|
+        if message.public_send("indicates_#{state}?")
+          label = Dug.configuration.label_for(:state, state)
+          labels_to_add << label if label
+        end
+      end
+
+      if message.indicates_reopened?
+        label = Dug.configuration.label_for(:state, 'closed')
+        reopened_label = Dug.configuration.label_for(:state, 'reopened')
+        labels_to_remove << label if label
+        labels_to_add << reopened_label if reopened_label
+      end
+
+      info = "Processing message:"
+      info << "\n    ID: #{message.id}"
+      %w(Date From Subject).each do |header|
+        info << "\n    #{header}: #{message.headers[header]}"
+      end
+      info << "\n    * Applying labels: #{labels_to_add.join(' | ')} *"
+      info << "\n    * Removing labels: #{labels_to_remove.join(' | ')} *"
+      log(info)
+
+      servicer.add_labels_by_name(message, labels_to_add)
+      servicer.remove_labels_by_name(message,
+                                     labels_to_remove,
+                                     entire_thread: modify_entire_thread?)
+    end
+
+    private
+
+    attr_reader :message
+    attr_reader :servicer
+
+    def labels_to_add
+      @labels_to_add ||= ["GitHub"]
+    end
+
+    def labels_to_remove
+      @labels_to_remove ||= [Dug.configuration.unprocessed_label_name]
+      if @labels_to_remove.size > 1
+        @modify_entire_thread = true
+      end
+      @labels_to_remove
+    end
+
+    def modify_entire_thread?
+      !!@modify_entire_thread
+    end
+  end
+end

data/lib/dug/notification_decorator.rb

@@ -7,11 +7,11 @@ module Dug
     end
 
     def organization
-      list_match[1]
+      list_match(1)
     end
 
     def repository
-      list_match[2]
+      list_match(2)
     end
 
     def reason
@@ -24,10 +24,30 @@ module Dug
       end
     end
 
+    def indicates_merged?
+      !!(plaintext_body =~ /^Merged #(?:\d+)\./)
+    end
+
+    def indicates_closed?
+      # Note: Purposely more lax than Merged
+      # Issues can be closed via PR/commit ie "Closed #123 via #456."
+      !!(plaintext_body =~ /^Closed #(?:\d+)/)
+    end
+
+    def indicates_reopened?
+      !!(plaintext_body =~ /^Reopened #(?:\d+)\./)
+    end
+
     private
 
-    def list_match
-      headers["List-ID"].match(/^([\w\-_]+)\/([\w\-_]+)/)
+    def list_match(index)
+      headers["List-ID"] && headers["List-ID"].match(/^([\w\-_]+)\/([\w\-_]+)/)[index]
+    end
+
+    def plaintext_body
+      payload.parts.detect { |part|
+        part.mime_type == 'text/plain'
+      }.body.data
     end
   end
 end

data/lib/dug/runner.rb

@@ -17,7 +17,7 @@ module Dug
       if unprocessed_notifications?
         log("Processing #{unprocessed_notifications.size} GitHub notifications...")
         unprocessed_notifications.each do |message|
-          process_message(message.id)
+          Dug::MessageProcessor.new(message.id, servicer).execute
         end
         log("Finished processing #{unprocessed_notifications.size} GitHub notifications.")
       else
@@ -27,42 +27,23 @@ module Dug
 
     private
 
-    def process_message(id)
-      message = NotificationDecorator.new(servicer.get_user_message('me', id))
-
-      labels_to_add    = ["GitHub"]
-      labels_to_remove = ["GitHub/Unprocessed"]
-      if message.reason
-        labels_to_add << Dug.configuration.label_for(:reason, message.reason)
-      end
-      labels_to_add << Dug.configuration.label_for(:organization, message.organization)
-      labels_to_add << Dug.configuration.label_for(:repository,
-                                                   message.repository,
-                                                   organization: message.organization)
-      labels_to_add.flatten! and labels_to_remove.flatten!
-      labels_to_add.compact! and labels_to_remove.compact!
-
-      info = "Processing message:"
-      info << "\n    ID: #{message.id}"
-      %w(Date From Subject).each do |header|
-        info << "\n    #{header}: #{message.headers[header]}"
-      end
-      info << "\n    * Applying labels: #{labels_to_add.join(' | ')} *"
-      log(info)
-
-      servicer.add_labels_by_name(message, labels_to_add)
-      servicer.remove_labels_by_name(message, labels_to_remove)
-    end
-
     def unprocessed_notifications(use_cache: true)
       unless use_cache
         log("Requesting latest emails from Gmail...")
         @unprocessed_notifications = nil
       end
-      unprocessed_label = servicer.labels(use_cache: use_cache)["GitHub/Unprocessed"]
-      @unprocessed_notifications ||= servicer
-        .list_user_messages('me', label_ids: [unprocessed_label.id])
-        .messages
+      unprocessed_label = servicer.labels(use_cache: use_cache)[Dug.configuration.unprocessed_label_name]
+
+      # The reverse! is required because we want to process messages in order
+      # and Google doesn't allow you to sort by anything because labels. Order
+      # is required here to account for state changes like reopened issues
+      @unprocessed_notifications ||=
+        begin
+          messages = servicer
+            .list_user_messages('me', label_ids: [unprocessed_label.id])
+            .messages
+          messages.reverse! if messages
+        end
     end
 
     def unprocessed_notifications?

data/lib/dug/validations.rb

@@ -0,0 +1,38 @@
+module Dug
+  module Validations
+    def valid_rule_type?(type)
+      LABEL_RULE_TYPES.include?(type.to_s)
+    end
+
+    def validate_rule_type!(type)
+      unless valid_rule_type?(type)
+        raise InvalidRuleType, "'#{type}' is not a valid label rule type. Valid types: #{LABEL_RULE_TYPES}"
+      end
+    end
+
+    def valid_reason?(reason)
+      GITHUB_REASONS.include?(reason.to_s)
+    end
+
+    def validate_reason!(reason)
+      unless valid_reason?(reason)
+        raise InvalidGitHubReason, "'#{reason}' is not a valid GitHub notification reason. Valid reasons include: #{GITHUB_REASONS.map { |x| "'#{x}'" }.join(', ')}"
+      end
+    end
+
+    def valid_state?(state)
+      ISSUE_STATES.include?(state.to_s)
+    end
+
+    def validate_state!(state)
+      unless valid_state?(state)
+        raise InvalidIssueState, "'#{state}' is not a valid issue state. Valid reasons include: #{ISSUE_STATES.map { |x| "'#{x}'" }.join(', ')}"
+      end
+    end
+  end
+
+  InvalidRuleType     = Class.new(StandardError)
+  InvalidGitHubReason = Class.new(StandardError)
+  InvalidIssueState   = Class.new(StandardError)
+end
+

data/lib/dug/version.rb

@@ -1,3 +1,3 @@
 module Dug
-  VERSION = "0.2.1"
+  VERSION = "1.0.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dug
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 1.0.0
 platform: ruby
 authors:
 - Chris Arcand
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-02-02 00:00:00.000000000 Z
+date: 2016-02-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: google-api-client
@@ -24,6 +24,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.9'
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.2.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.2.1
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -52,6 +66,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '10.0'
+- !ruby/object:Gem::Dependency
+  name: pry
+  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'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -66,9 +94,65 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '5.0'
-description: "[D]amn yo[u], [G]mail. A gemified script to organize your GitHub notification
-  emails using a simple configuration file in ways Gmail filters can't, such as X-GitHub-Reason
-  headers."
+- !ruby/object:Gem::Dependency
+  name: mocha
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: minitest-reporters
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: codeclimate-test-reporter
+  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'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  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 simple, configurable gem to organize your GitHub notification emails
+  in ways Gmail can't and in an easier-to-maintain way than large, slow Google Apps
+  Scripts.
 email:
 - chris@chrisarcand.com
 executables: []
@@ -77,6 +161,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".travis.yml"
+- CHANGELOG.md
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -88,8 +173,10 @@ files:
 - lib/dug/configurator.rb
 - lib/dug/gmail_servicer.rb
 - lib/dug/logger.rb
+- lib/dug/message_processor.rb
 - lib/dug/notification_decorator.rb
 - lib/dug/runner.rb
+- lib/dug/validations.rb
 - lib/dug/version.rb
 homepage: https://github.com/chrisarcand/dug
 licenses: