Merge pull request #3216 from huginn/ruby-3.2

Upgrade Ruby to 3.2

.github/workflows/ci.yml

@@ -21,7 +21,7 @@ jobs:
           - mysql2
           - postgresql
         ruby:
-          - "2.7"
+          - "3.2"
     env:
       DATABASE_ADAPTER: ${{ matrix.database_adapter }}
       DATABASE_HOST: "127.0.0.1"
@@ -68,6 +68,9 @@ jobs:
       - name: Run tests
         run: bundle exec rake
 
+      - name: Coveralls
+        uses: coverallsapp/github-action@v1
+
   ghcr-build-docker-images:
     name: ghcr-docker-build-${{ matrix.docker_image }}
     needs: run-tests

Gemfile

@@ -1,6 +1,6 @@
 source 'https://rubygems.org'
 
-ruby '>=2.7.0'
+ruby '>=3.2.2'
 
 # Ensure github repositories are fetched using HTTPS
 git_source(:github) do |repo_name|
@@ -29,18 +29,18 @@ end
 
 # Optional libraries.  To conserve RAM, comment out any that you don't need,
 # then run `bundle` and commit the updated Gemfile and Gemfile.lock.
-gem 'twilio-ruby', '~> 5.62.0'    # TwilioAgent
-gem 'ruby-growl', '~> 4.1.0'      # GrowlAgent
-gem 'net-ftp-list', '~> 3.2.8'    # FtpsiteAgent
-gem 'forecast_io', '~> 2.0.0'     # WeatherAgent
-gem 'rturk', '~> 2.12.1'          # HumanTaskAgent
 gem 'erector', github: 'dsander/erector', branch: 'rails6'
+gem 'forecast_io', '~> 2.0.0'     # WeatherAgent
 gem 'hipchat', '~> 1.2.0'         # HipchatAgent
+gem 'hypdf', '~> 1.0.10'          # PDFInfoAgent
 gem 'mini_racer'                  # JavaScriptAgent
-gem 'xmpp4r',  '~> 0.5.6'         # JabberAgent
 gem 'mqtt'                        # MQTTAgent
+gem 'net-ftp'
+gem 'net-ftp-list'                # FtpsiteAgent
+gem 'rturk', '~> 2.12.1'          # HumanTaskAgent
 gem 'slack-notifier', '~> 1.0.0'  # SlackAgent
-gem 'hypdf', '~> 1.0.10'          # PDFInfoAgent
+gem 'twilio-ruby', '~> 5.62.0'    # TwilioAgent
+gem 'xmpp4r', '~> 0.5.6'          # JabberAgent
 
 # Weibo Agents
 # FIXME needs to loosen omniauth dependency, add rest-client
@@ -51,14 +51,15 @@ gem 'google-api-client', '~> 0.13'
 gem 'google-cloud-translate', '~> 2.0', require: 'google/cloud/translate'
 
 # Twitter Agents
+gem 'omniauth-twitter'
 gem 'twitter', github: 'sferik/twitter' # Must to be loaded before cantino-twitter-stream.
 gem 'twitter-stream', github: 'cantino/twitter-stream', branch: 'huginn'
-gem 'omniauth-twitter'
 
 # Tumblr Agents
 # until merge of https://github.com/tumblr/tumblr_client/pull/61
-gem 'tumblr_client', github: 'albertsun/tumblr_client', branch: 'master', ref: 'e046fe6e39291c173add0a49081630c7b60a36c7'
 gem 'omniauth-tumblr'
+gem 'tumblr_client', github: 'albertsun/tumblr_client', branch: 'master',
+                     ref: 'e046fe6e39291c173add0a49081630c7b60a36c7'
 
 # Dropbox Agents
 gem 'dropbox-api', github: 'dsander/dropbox-api', ref: '86cb7b5a1254dc5b054de7263835713c4c1018c7'
@@ -68,8 +69,8 @@ gem 'omniauth-dropbox-oauth2', github: 'huginn/omniauth-dropbox-oauth2'
 gem 'haversine'
 
 # EvernoteAgent
-gem 'omniauth-evernote'
 gem 'evernote_oauth'
+gem 'omniauth-evernote'
 
 # LocalFileAgent (watch functionality)
 gem 'listen', '~> 3.0.5', require: false
@@ -78,17 +79,18 @@ gem 'listen', '~> 3.0.5', require: false
 gem 'aws-sdk-s3', '~> 1'
 
 # ImapFolderAgent
-gem 'omniauth-google-oauth2', '>= 0.8.0'
 gem 'gmail_xoauth' # support for Gmail using OAuth
+gem 'omniauth-google-oauth2', '>= 0.8.0'
 
 # Bundler <1.5 does not recognize :x64_mingw as a valid platform name.
 # Unfortunately, it can't self-update because it errors when encountering :x64_mingw.
 unless Gem::Version.new(Bundler::VERSION) >= Gem::Version.new('1.5.0')
-  STDERR.puts "Bundler >=1.5.0 is required.  Please upgrade bundler with 'gem install bundler'"
+  warn "Bundler >=1.5.0 is required.  Please upgrade bundler with 'gem install bundler'"
   exit 1
 end
 
-gem 'ace-rails-ap', '~> 2.0.1'
+gem 'ace-rails-ap'
+gem 'bootsnap', require: false
 gem 'bootstrap-kaminari-views', '~> 0.0.3'
 gem 'bundler', '>= 1.5.0'
 gem 'coffee-rails', '~> 5'
@@ -97,17 +99,18 @@ gem 'delayed_job'
 gem 'delayed_job_active_record'
 gem 'devise', '~> 4.8'
 gem 'em-http-request', '~> 1.1.2'
+gem 'execjs'
 gem 'faraday', '~> 0.9'
 gem 'faraday_middleware', '~> 0.12.2'
 gem 'feedjira', '~> 3.1'
 gem 'font-awesome-sass', '~> 4.7.0'
-gem 'foreman', '~> 0.63.0'
+gem 'foreman', '~> 0.87.2'
 gem 'geokit', '~> 1.13'
 gem 'geokit-rails', '~> 2.3'
-gem 'httparty', '~> 0.13'
 gem 'httmultiparty', '~> 0.3.16'
-gem 'jquery-rails', '~> 4.2.1'
+gem 'httparty', '~> 0.13'
 gem 'huginn_agent'
+gem 'jquery-rails', '~> 4.2.1'
 gem 'json', '~> 2.3'
 gem 'jsonpath', '~> 1.1'
 gem 'kaminari', '~> 1.2'
@@ -120,16 +123,15 @@ gem 'multi_xml'
 gem "nokogiri", ">= 1.10.8"
 gem 'omniauth'
 gem 'rails', '~> 6.1.7'
-gem 'sprockets', '~> 3.7.2'
 gem 'rails-html-sanitizer', '~> 1.2'
 gem 'rufus-scheduler', '~> 3.4', require: false
 gem 'sass-rails', '>= 6.0'
-gem 'select2-rails', '~> 3.5.4'
+gem 'select2-rails'
 gem 'spectrum-rails'
-gem 'execjs'
+gem 'sprockets'
+gem 'terser'
 gem 'typhoeus', '~> 1.3.1'
 gem 'uglifier', '~> 2.7.2'
-gem 'bootsnap', require: false
 
 group :development do
   gem 'better_errors'
@@ -137,38 +139,41 @@ group :development do
   gem 'guard'
   gem 'guard-livereload'
   gem 'guard-rspec'
-  gem 'rack-livereload'
   gem 'letter_opener_web', '~> 1.4' # 2.0+ requires Ruby 2.7
+  gem 'rack-livereload'
   gem 'web-console', '>= 3.3.0'
 
   gem 'capistrano'
-  gem 'capistrano-rails'
   gem 'capistrano-bundler'
+  gem 'capistrano-rails'
+
+  gem 'rubocop', require: false
+  gem 'rubocop-performance', require: false
+  gem 'rubocop-rspec', require: false
 
   if_true(ENV['SPRING']) do
-    gem 'spring-commands-rspec'
     gem 'spring'
+    gem 'spring-commands-rspec'
     gem 'spring-watcher-listen'
   end
 
   group :test do
-    gem 'coveralls', require: false
-    gem 'capybara', '~> 2.18'
-    gem 'capybara-screenshot'
-    gem 'capybara-select-2', github: 'Hirurg103/capybara_select2', ref: 'fbf22fb74dec10fa0edcd26da7c5184ba8fa2c76', require: false
-    gem 'poltergeist'
-    gem 'pry-rails'
-    gem 'pry-byebug'
+    gem 'capybara'
+    gem 'capybara-select-2', github: 'Hirurg103/capybara_select2', require: false
+    gem 'puma'
+    gem 'rails-controller-testing'
     gem 'rr', require: false
     gem 'rspec'
-    gem 'rspec-mocks'
-    gem 'rspec-rails'
     gem 'rspec-collection_matchers'
     gem 'rspec-html-matchers'
-    gem 'rails-controller-testing'
+    gem 'rspec-mocks'
+    gem 'rspec-rails'
+    gem 'selenium-webdriver'
     gem 'shoulda-matchers'
+    gem 'simplecov', require: false
+    gem 'simplecov-lcov', '~> 0.8.0', require: false
     gem 'vcr'
-    gem 'webmock', '~> 3.5.1'
+    gem 'webmock'
   end
 end
 
@@ -178,18 +183,17 @@ end
 
 # Platform requirements.
 require 'rbconfig'
-gem 'ffi', '>= 1.9.4'		# required by typhoeus; 1.9.4 has fixes for *BSD.
+gem 'ffi', '>= 1.9.4'	# required by typhoeus; 1.9.4 has fixes for *BSD.
 gem 'tzinfo', '>= 1.2.0'	# required by rails; 1.2.0 has support for *BSD and Solaris.
 # Windows does not have zoneinfo files, so bundle the tzinfo-data gem.
 gem 'tzinfo-data', platforms: [:mingw, :mswin, :x64_mingw]
 # BSD systems require rb-kqueue for "listen" to avoid polling for changes.
 gem 'rb-kqueue', '>= 0.2', require: /bsd|dragonfly/i === RbConfig::CONFIG['target_os']
 
-
 on_heroku = ENV['ON_HEROKU'] ||
-            ENV['HEROKU_POSTGRESQL_ROSE_URL'] ||
-            ENV['HEROKU_POSTGRESQL_GOLD_URL'] ||
-            File.read(File.join(File.dirname(__FILE__), 'Procfile')) =~ /intended for Heroku/
+  ENV['HEROKU_POSTGRESQL_ROSE_URL'] ||
+  ENV['HEROKU_POSTGRESQL_GOLD_URL'] ||
+  File.read(File.join(File.dirname(__FILE__), 'Procfile')) =~ /intended for Heroku/
 
 ENV['DATABASE_ADAPTER'] ||=
   if on_heroku

Gemfile.lock

@@ -1,9 +1,8 @@
 GIT
   remote: https://github.com/Hirurg103/capybara_select2.git
-  revision: fbf22fb74dec10fa0edcd26da7c5184ba8fa2c76
-  ref: fbf22fb74dec10fa0edcd26da7c5184ba8fa2c76
+  revision: 48e2c458c2827cf2af2b0a287b30468fa8d761bf
   specs:
-    capybara-select-2 (0.3.2)
+    capybara-select-2 (0.5.1)
 
 GIT
   remote: https://github.com/albertsun/tumblr_client.git
@@ -67,15 +66,15 @@ GIT
 
 GIT
   remote: https://github.com/sferik/twitter.git
-  revision: d11707edf4abd13f7ada0eef57fc1eaa1062d75b
+  revision: 5a49cb6b6c84ccc8f2f980c42d3e9f852033735a
   specs:
-    twitter (5.15.0)
+    twitter (8.0.0)
       addressable (~> 2.3)
-      buftok (~> 0.2.0)
+      buftok (~> 0.3.0)
       equalizer (~> 0.0.11)
-      http (~> 2.0)
-      http-form_data (~> 1.0)
-      http_parser.rb (~> 0.6.0)
+      http (~> 5.1)
+      http-form_data (~> 2.3)
+      llhttp-ffi (~> 0.4.0)
       memoizable (~> 0.4.0)
       multipart-post (~> 2.0)
       naught (~> 1.0)
@@ -91,7 +90,7 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    ace-rails-ap (2.0.1)
+    ace-rails-ap (4.5)
     actioncable (6.1.7.2)
       actionpack (= 6.1.7.2)
       activesupport (= 6.1.7.2)
@@ -151,10 +150,11 @@ GEM
       minitest (>= 5.1)
       tzinfo (~> 2.0)
       zeitwerk (~> 2.3)
-    addressable (2.8.1)
+    addressable (2.8.4)
       public_suffix (>= 2.0.2, < 6.0)
     airbrussh (1.4.0)
       sshkit (>= 1.6.1, != 1.7.0)
+    ast (2.4.2)
     aws-eventstream (1.2.0)
     aws-partitions (1.547.0)
     aws-sdk-core (3.125.2)
@@ -184,9 +184,8 @@ GEM
     bootstrap-kaminari-views (0.0.5)
       kaminari (>= 0.13)
       rails (>= 3.1)
-    buftok (0.2.0)
+    buftok (0.3.0)
     builder (3.2.4)
-    byebug (11.1.3)
     capistrano (3.16.0)
       airbrussh (>= 1.0.0)
       i18n
@@ -197,17 +196,15 @@ GEM
     capistrano-rails (1.6.1)
       capistrano (~> 3.1)
       capistrano-bundler (>= 1.1, < 3)
-    capybara (2.18.0)
+    capybara (3.39.0)
       addressable
+      matrix
       mini_mime (>= 0.1.3)
-      nokogiri (>= 1.3.3)
-      rack (>= 1.0.0)
-      rack-test (>= 0.5.4)
-      xpath (>= 2.0, < 4.0)
-    capybara-screenshot (1.0.17)
-      capybara (>= 1.0, < 3)
-      launchy
-    cliver (0.3.2)
+      nokogiri (~> 1.8)
+      rack (>= 1.6.0)
+      rack-test (>= 0.6.3)
+      regexp_parser (>= 1.5, < 3.0)
+      xpath (~> 3.2)
     coderay (1.1.3)
     coffee-rails (5.0.0)
       coffee-script (>= 2.2.0)
@@ -216,14 +213,8 @@ GEM
       coffee-script-source
       execjs
     coffee-script-source (1.12.2)
-    concurrent-ruby (1.2.0)
+    concurrent-ruby (1.2.2)
     cookiejar (0.3.3)
-    coveralls (0.8.23)
-      json (>= 1.8, < 3)
-      simplecov (~> 0.16.1)
-      term-ansicolor (~> 1.3)
-      thor (>= 0.19.4, < 2.0)
-      tins (~> 1.6)
     crack (0.4.5)
       rexml
     crass (1.0.6)
@@ -244,7 +235,7 @@ GEM
       warden (~> 1.2.3)
     diff-lcs (1.5.0)
     docile (1.4.0)
-    domain_name (0.5.20170404)
+    domain_name (0.5.20190701)
       unf (>= 0.0.5, < 1.0.0)
     em-http-request (1.1.7)
       addressable (>= 2.3.4)
@@ -269,7 +260,7 @@ GEM
       evernote-thrift
       oauth (>= 0.4.1)
     execjs (2.8.1)
-    faraday (0.17.4)
+    faraday (0.17.6)
       multipart-post (>= 1.2, < 3)
     faraday_middleware (0.12.2)
       faraday (>= 0.7.4, < 1.0)
@@ -277,15 +268,16 @@ GEM
       loofah (>= 2.3.1)
       sax-machine (>= 1.0)
     ffi (1.15.5)
+    ffi-compiler (1.0.1)
+      ffi (>= 1.0.0)
+      rake
     font-awesome-sass (4.7.0)
       sass (>= 3.2)
     forecast_io (2.0.1)
       faraday
       hashie
       multi_json
-    foreman (0.63.0)
-      dotenv (>= 0.7)
-      thor (>= 0.13.6)
+    foreman (0.87.2)
     formatador (1.1.0)
     fugit (1.5.2)
       et-orbi (~> 1.1, >= 1.1.8)
@@ -393,14 +385,14 @@ GEM
       httparty (>= 0.7.3)
       mimemagic
       multipart-post
-    http (2.1.0)
-      addressable (~> 2.3)
+    http (5.1.1)
+      addressable (~> 2.8)
       http-cookie (~> 1.0)
-      http-form_data (~> 1.0.1)
-      http_parser.rb (~> 0.6.0)
-    http-cookie (1.0.3)
+      http-form_data (~> 2.2)
+      llhttp-ffi (~> 0.4.0)
+    http-cookie (1.0.5)
       domain_name (~> 0.5)
-    http-form_data (1.0.1)
+    http-form_data (2.3.0)
     http_parser.rb (0.6.0)
     httparty (0.14.0)
       multi_xml (>= 0.5.2)
@@ -417,7 +409,7 @@ GEM
       rails-dom-testing (>= 1, < 3)
       railties (>= 4.2.0)
       thor (>= 0.14, < 2.0)
-    json (2.6.1)
+    json (2.6.3)
     jsonpath (1.1.0)
       multi_json
     jwt (2.3.0)
@@ -449,22 +441,24 @@ GEM
     libv8-node (16.10.0.0-arm64-darwin)
     libv8-node (16.10.0.0-x86_64-darwin)
     libv8-node (16.10.0.0-x86_64-linux)
-    liquid (5.3.0)
+    liquid (5.4.0)
     listen (3.0.8)
       rb-fsevent (~> 0.9, >= 0.9.4)
       rb-inotify (~> 0.9, >= 0.9.7)
+    llhttp-ffi (0.4.0)
+      ffi-compiler (~> 1.0)
+      rake (~> 13.0)
     loofah (2.20.0)
       crass (~> 1.0.2)
       nokogiri (>= 1.5.9)
     lumberjack (1.2.8)
-    macaddr (1.7.1)
-      systemu (~> 2.6.2)
     mail (2.8.1)
       mini_mime (>= 0.1.1)
       net-imap
       net-pop
       net-smtp
     marcel (1.0.2)
+    matrix (0.4.2)
     memoist (0.16.2)
     memoizable (0.4.2)
       thread_safe (~> 0.3, >= 0.3.1)
@@ -480,15 +474,18 @@ GEM
     mini_portile2 (2.8.1)
     mini_racer (0.6.3)
       libv8-node (~> 16.10.0.0)
-    minitest (5.17.0)
+    minitest (5.18.0)
     mqtt (0.3.1)
     msgpack (1.4.2)
     multi_json (1.15.0)
     multi_xml (0.6.0)
-    multipart-post (2.1.1)
+    multipart-post (2.3.0)
     mysql2 (0.5.4)
     naught (1.1.0)
     nenv (0.3.0)
+    net-ftp (0.2.0)
+      net-protocol
+      time
     net-ftp-list (3.2.8)
     net-imap (0.3.4)
       date
@@ -551,28 +548,23 @@ GEM
       rack
     orm_adapter (0.5.0)
     os (1.1.4)
+    parallel (1.22.1)
+    parser (3.2.2.0)
+      ast (~> 2.4.1)
     pg (1.4.4)
-    poltergeist (1.8.1)
-      capybara (~> 2.1)
-      cliver (~> 0.3.1)
-      multi_json (~> 1.0)
-      websocket-driver (>= 0.2.0)
     polyglot (0.3.5)
     pry (0.13.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
-    pry-byebug (3.9.0)
-      byebug (~> 11.0)
-      pry (~> 0.13.0)
-    pry-rails (0.3.9)
-      pry (>= 0.10.4)
-    public_suffix (5.0.0)
+    public_suffix (5.0.1)
+    puma (6.2.1)
+      nio4r (~> 2.0)
     raabro (1.4.0)
     racc (1.6.2)
     rack (2.2.6.4)
     rack-livereload (0.3.17)
       rack
-    rack-test (2.0.2)
+    rack-test (2.1.0)
       rack (>= 1.3)
     rails (6.1.7.2)
       actioncable (= 6.1.7.2)
@@ -589,10 +581,10 @@ GEM
       bundler (>= 1.15.0)
       railties (= 6.1.7.2)
       sprockets-rails (>= 2.0.0)
-    rails-controller-testing (1.0.4)
-      actionpack (>= 5.0.1.x)
-      actionview (>= 5.0.1.x)
-      activesupport (>= 5.0.1.x)
+    rails-controller-testing (1.0.5)
+      actionpack (>= 5.0.1.rc1)
+      actionview (>= 5.0.1.rc1)
+      activesupport (>= 5.0.1.rc1)
     rails-dom-testing (2.0.3)
       activesupport (>= 4.2.0)
       nokogiri (>= 1.6)
@@ -604,6 +596,7 @@ GEM
       method_source
       rake (>= 12.2)
       thor (~> 1.0)
+    rainbow (3.1.1)
     raindrops (0.20.0)
     rake (13.0.6)
     rb-fsevent (0.11.2)
@@ -611,6 +604,7 @@ GEM
       ffi (~> 1.0)
     rb-kqueue (0.2.4)
       ffi (>= 0.5.0)
+    regexp_parser (2.7.0)
     representable (3.1.1)
       declarative (< 0.1.0)
       trailblazer-option (>= 0.1.1, < 0.2.0)
@@ -625,7 +619,7 @@ GEM
     retriable (3.1.2)
     rexml (3.2.5)
     rly (0.2.3)
-    rr (3.0.9)
+    rr (3.1.0)
     rspec (3.12.0)
       rspec-core (~> 3.12.0)
       rspec-expectations (~> 3.12.0)
@@ -640,7 +634,7 @@ GEM
     rspec-html-matchers (0.10.0)
       nokogiri (~> 1)
       rspec (>= 3.0.0.a)
-    rspec-mocks (3.12.3)
+    rspec-mocks (3.12.5)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.12.0)
     rspec-rails (6.0.1)
@@ -656,8 +650,28 @@ GEM
       erector
       nokogiri
       rest-client
-    ruby-growl (4.1)
-      uuid (~> 2.3, >= 2.3.5)
+    rubocop (1.50.1)
+      json (~> 2.3)
+      parallel (~> 1.10)
+      parser (>= 3.2.0.0)
+      rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8, < 3.0)
+      rexml (>= 3.2.5, < 4.0)
+      rubocop-ast (>= 1.28.0, < 2.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 2.4.0, < 3.0)
+    rubocop-ast (1.28.0)
+      parser (>= 3.2.1.0)
+    rubocop-capybara (2.17.1)
+      rubocop (~> 1.41)
+    rubocop-performance (1.17.1)
+      rubocop (>= 1.7.0, < 2.0)
+      rubocop-ast (>= 0.4.0)
+    rubocop-rspec (2.18.1)
+      rubocop (~> 1.33)
+      rubocop-capybara (~> 2.17)
+    ruby-progressbar (1.13.0)
+    rubyzip (2.3.2)
     rufus-scheduler (3.8.1)
       fugit (~> 1.1, >= 1.1.6)
     sass (3.7.4)
@@ -676,7 +690,11 @@ GEM
       sprockets-rails
       tilt
     sax-machine (1.3.2)
-    select2-rails (3.5.11)
+    select2-rails (4.0.13)
+    selenium-webdriver (4.8.6)
+      rexml (~> 3.2, >= 3.2.5)
+      rubyzip (>= 1.2.2, < 3.0)
+      websocket (~> 1.0)
     shellany (0.0.1)
     shoulda-matchers (4.0.1)
       activesupport (>= 4.2.0)
@@ -686,11 +704,13 @@ GEM
       jwt (>= 1.5, < 3.0)
       multi_json (~> 1.10)
     simple_oauth (0.3.1)
-    simplecov (0.16.1)
+    simplecov (0.22.0)
       docile (~> 1.1)
-      json (>= 1.8, < 3)
-      simplecov-html (~> 0.10.0)
-    simplecov-html (0.10.2)
+      simplecov-html (~> 0.11)
+      simplecov_json_formatter (~> 0.1)
+    simplecov-html (0.12.3)
+    simplecov-lcov (0.8.0)
+    simplecov_json_formatter (0.1.4)
     slack-notifier (1.0.0)
     spectrum-rails (1.3.4)
       railties (>= 3.1)
@@ -700,9 +720,9 @@ GEM
     spring-watcher-listen (2.1.0)
       listen (>= 2.7, < 4.0)
       spring (>= 4)
-    sprockets (3.7.2)
+    sprockets (4.2.0)
       concurrent-ruby (~> 1.0)
-      rack (> 1, < 3)
+      rack (>= 2.2.4, < 4)
     sprockets-rails (3.4.2)
       actionpack (>= 5.2)
       activesupport (>= 5.2)
@@ -710,16 +730,14 @@ GEM
     sshkit (1.21.2)
       net-scp (>= 1.1.2)
       net-ssh (>= 2.8.0)
-    sync (0.5.0)
-    systemu (2.6.4)
-    term-ansicolor (1.7.1)
-      tins (~> 1.0)
+    terser (1.1.14)
+      execjs (>= 0.3.0, < 3)
     thor (1.2.1)
     thread_safe (0.3.6)
     tilt (2.0.10)
+    time (0.2.2)
+      date
     timeout (0.3.1)
-    tins (1.31.0)
-      sync
     trailblazer-option (0.1.2)
     treetop (1.6.10)
       polyglot (~> 0.3)
@@ -737,13 +755,12 @@ GEM
       json (>= 1.8.0)
     unf (0.1.4)
       unf_ext
-    unf_ext (0.0.7.4)
+    unf_ext (0.0.8.2)
+    unicode-display_width (2.4.2)
     unicorn (6.1.0)
       kgio (~> 2.6)
       raindrops (~> 0.7)
-    uuid (2.3.7)
-      macaddr (~> 1.0)
-    vcr (3.0.3)
+    vcr (6.1.0)
     warden (1.2.9)
       rack (>= 2.0.9)
     web-console (4.2.0)
@@ -751,16 +768,17 @@ GEM
       activemodel (>= 6.0.0)
       bindex (>= 0.4.0)
       railties (>= 6.0.0)
-    webmock (3.5.1)
-      addressable (>= 2.3.6)
+    webmock (3.18.1)
+      addressable (>= 2.8.0)
       crack (>= 0.3.2)
-      hashdiff
+      hashdiff (>= 0.4.0, < 2.0.0)
     webrick (1.7.0)
+    websocket (1.2.9)
     websocket-driver (0.7.5)
       websocket-extensions (>= 0.1.0)
     websocket-extensions (0.1.5)
     xmpp4r (0.5.6)
-    xpath (3.0.0)
+    xpath (3.2.0)
       nokogiri (~> 1.8)
     zeitwerk (2.6.7)
 
@@ -772,7 +790,7 @@ PLATFORMS
   x86_64-linux
 
 DEPENDENCIES
-  ace-rails-ap (~> 2.0.1)
+  ace-rails-ap
   aws-sdk-s3 (~> 1)
   better_errors
   binding_of_caller
@@ -782,11 +800,9 @@ DEPENDENCIES
   capistrano
   capistrano-bundler
   capistrano-rails
-  capybara (~> 2.18)
-  capybara-screenshot
+  capybara
   capybara-select-2!
   coffee-rails (~> 5)
-  coveralls
   daemons (~> 1.1.9)
   delayed_job
   delayed_job_active_record
@@ -804,7 +820,7 @@ DEPENDENCIES
   ffi (>= 1.9.4)
   font-awesome-sass (~> 4.7.0)
   forecast_io (~> 2.0.0)
-  foreman (~> 0.63.0)
+  foreman (~> 0.87.2)
   geokit (~> 1.13)
   geokit-rails (~> 2.3)
   gmail_xoauth
@@ -834,7 +850,8 @@ DEPENDENCIES
   mqtt
   multi_xml
   mysql2 (~> 0.5)
-  net-ftp-list (~> 3.2.8)
+  net-ftp
+  net-ftp-list
   nokogiri (>= 1.10.8)
   omniauth
   omniauth-dropbox-oauth2!
@@ -843,9 +860,7 @@ DEPENDENCIES
   omniauth-tumblr
   omniauth-twitter
   pg (~> 1.1)
-  poltergeist
-  pry-byebug
-  pry-rails
+  puma
   rack-livereload
   rails (~> 6.1.7)
   rails-controller-testing
@@ -858,17 +873,23 @@ DEPENDENCIES
   rspec-mocks
   rspec-rails
   rturk (~> 2.12.1)
-  ruby-growl (~> 4.1.0)
+  rubocop
+  rubocop-performance
+  rubocop-rspec
   rufus-scheduler (~> 3.4)
   sass-rails (>= 6.0)
-  select2-rails (~> 3.5.4)
+  select2-rails
+  selenium-webdriver
   shoulda-matchers
+  simplecov
+  simplecov-lcov (~> 0.8.0)
   slack-notifier (~> 1.0.0)
   spectrum-rails
   spring
   spring-commands-rspec
   spring-watcher-listen
-  sprockets (~> 3.7.2)
+  sprockets
+  terser
   tumblr_client!
   twilio-ruby (~> 5.62.0)
   twitter!
@@ -880,12 +901,12 @@ DEPENDENCIES
   unicorn
   vcr
   web-console (>= 3.3.0)
-  webmock (~> 3.5.1)
+  webmock
   weibo_2!
   xmpp4r (~> 0.5.6)
 
 RUBY VERSION
-   ruby 2.7.6p219
+   ruby 3.2.2p53
 
 BUNDLED WITH
    2.4.12

app/assets/config/manifest.js

@@ -0,0 +1,3 @@
+//= link_tree ../images
+//= link_directory ../javascripts .js
+//= link_directory ../stylesheets .css

app/assets/javascripts/ace.js

@@ -0,0 +1 @@
+//= require ace-rails-ap

app/assets/javascripts/ace.js.coffee

@@ -1,8 +0,0 @@
-#= require ace/ace
-#= require ace/mode-javascript.js
-#= require ace/mode-markdown.js
-#= require ace/mode-coffee.js
-#= require ace/mode-sql.js
-#= require ace/mode-json.js
-#= require ace/mode-yaml.js
-#= require ace/mode-text.js

app/assets/javascripts/application.js

@@ -0,0 +1,13 @@
+//= require jquery
+//= require rails-ujs
+//= require typeahead.bundle
+//= require bootstrap
+//= require select2
+//= require json2
+//= require jquery.json-editor
+//= require jquery.serializeObject
+//= require latlon_and_geo
+//= require spectrum
+//= require_tree ./components
+//= require_tree ./pages
+//= require_self

app/assets/javascripts/application.js.coffee

@@ -1,13 +0,0 @@
-#= require jquery
-#= require rails-ujs
-#= require typeahead.bundle
-#= require bootstrap
-#= require select2
-#= require json2
-#= require jquery.json-editor
-#= require jquery.serializeObject
-#= require latlon_and_geo
-#= require spectrum
-#= require_tree ./components
-#= require_tree ./pages
-#= require_self

app/assets/javascripts/components/core.js

@@ -0,0 +1,56 @@
+$(function () {
+  // Flash
+  if ($(".flash").length) {
+    setTimeout(() => $(".flash").slideUp(() => $(".flash").remove()), 5000);
+  }
+
+  // Help popovers
+  $(".hover-help").popover({ trigger: "hover", html: true });
+
+  // Pressing '/' selects the search box.
+  $("body").on("keypress", function (e) {
+    if (e.keyCode === 47) {
+      // The '/' key
+      if (e.target.nodeName === "BODY") {
+        e.preventDefault();
+        return $agentNavigate.focus();
+      }
+    }
+  });
+
+  // Select2 Selects
+  $(".select2").select2({ width: "resolve" });
+
+  $(".select2-linked-tags").select2({
+    width: "resolve",
+    templateSelection: ({ id, text, element }) => {
+      const a = document.createElement("a");
+      a.href = `${element.closest("select").dataset.urlPrefix}/${id}/edit`;
+      a.onClick = "Utils.select2TagClickHandler(event, this)";
+      a.appendChild(document.createTextNode(text));
+      return a;
+    },
+  });
+
+  // Helper for selecting text when clicked
+  $(".selectable-text").each(function () {
+    return $(this).click(function () {
+      const range = document.createRange();
+      range.setStartBefore(this.firstChild);
+      range.setEndAfter(this.lastChild);
+      const sel = window.getSelection();
+      sel.removeAllRanges();
+      return sel.addRange(range);
+    });
+  });
+
+  // Agent navbar dropdown
+  return $(".navbar .dropdown.dropdown-hover").hover(
+    function () {
+      return $(this).addClass("open");
+    },
+    function () {
+      return $(this).removeClass("open");
+    }
+  );
+});

app/assets/javascripts/components/core.js.coffee

@@ -1,36 +0,0 @@
-$ ->
-  # Flash
-  if $(".flash").length
-    setTimeout((-> $(".flash").slideUp(-> $(".flash").remove())), 5000)
-
-  # Help popovers
-  $('.hover-help').popover(trigger: 'hover', html: true)
-
-  # Pressing '/' selects the search box.
-  $("body").on "keypress", (e) ->
-    if e.keyCode == 47 # The '/' key
-      if e.target.nodeName == "BODY"
-        e.preventDefault()
-        $agentNavigate.focus()
-
-  # Select2 Selects
-  $(".select2").select2(width: 'resolve')
-
-  $(".select2-linked-tags").select2(
-    width: 'resolve',
-    formatSelection: (obj) ->
-      "<a href=\"#{this.element.data('urlPrefix')}/#{obj.id}/edit\" onClick=\"Utils.select2TagClickHandler(event, this)\">#{Utils.escape(obj.text)}</a>"
-  )
-
-  # Helper for selecting text when clicked
-  $('.selectable-text').each ->
-    $(this).click ->
-      range = document.createRange()
-      range.setStartBefore(this.firstChild)
-      range.setEndAfter(this.lastChild)
-      sel = window.getSelection()
-      sel.removeAllRanges();
-      sel.addRange(range)
-
-  # Agent navbar dropdown
-  $('.navbar .dropdown.dropdown-hover').hover (-> $(this).addClass('open')), (-> $(this).removeClass('open'))

app/assets/javascripts/components/form_configurable.js

@@ -0,0 +1,95 @@
+$(function () {
+  const getFormData = function (elem) {
+    const form_data = $("#edit_agent, #new_agent").serializeObject();
+    const attribute = $(elem).data("attribute");
+    form_data["attribute"] = attribute;
+    delete form_data["_method"];
+    return form_data;
+  };
+
+  return (window.initializeFormCompletable = function () {
+    $("input[role~=validatable], select[role~=validatable]").on(
+      "change",
+      (e) => {
+        const form_data = getFormData(e.currentTarget);
+        const form_group = $(e.currentTarget).closest(".form-group");
+        return $.ajax("/agents/validate", {
+          type: "POST",
+          data: form_data,
+          success: (data) => {
+            form_group.addClass("has-feedback").removeClass("has-error");
+            form_group.find("span").addClass("hidden");
+            form_group.find(".glyphicon-ok").removeClass("hidden");
+          },
+          error: (data) => {
+            form_group.addClass("has-feedback").addClass("has-error");
+            form_group.find("span").addClass("hidden");
+            form_group.find(".glyphicon-remove").removeClass("hidden");
+          },
+        });
+      }
+    );
+
+    $("input[role~=validatable], select[role~=validatable]").trigger("change");
+
+    $.each($("select[role~=completable]"), (i, select) => {
+      const $select = $(select);
+      const value = $select.data("value");
+
+      const setValue = (value) => {
+        if (
+          $select
+            .find("option")
+            .toArray()
+            .some((option) => option.value == value)
+        ) {
+          $select.val(value).trigger("change");
+        } else {
+          $select
+            .append(new Option(value, value, true, true))
+            .trigger("change");
+        }
+      };
+
+      if ($select.data("cacheResponse")) {
+        const loadData = (data) => {
+          $select.select2({ data: data, tags: true });
+          setValue(value);
+        };
+
+        $.ajax("/agents/complete", {
+          type: "POST",
+          data: getFormData(select),
+          success: (data) => loadData(data),
+          error: (data) =>
+            loadData([{ id: undefined, text: "Error loading data." }]),
+        });
+      } else {
+        $select.select2({
+          ajax: {
+            url: "/agents/complete",
+            type: "POST",
+            data: (params) => getFormData(select),
+            processResults: (data) => ({ results: data }),
+          },
+          tags: true,
+        });
+        setValue(value);
+      }
+    });
+
+    $("input[type=radio][role~=form-configurable]").change(function (e) {
+      const input = $(e.currentTarget)
+        .parents()
+        .siblings(
+          `input[data-attribute=${$(e.currentTarget).data("attribute")}]`
+        );
+      if ($(e.currentTarget).val() === "manual") {
+        input.removeClass("hidden");
+      } else {
+        input.val($(e.currentTarget).val());
+        input.addClass("hidden");
+      }
+    });
+  });
+});

app/assets/javascripts/components/form_configurable.js.coffee

@@ -1,81 +0,0 @@
-$ ->
-  getFormData = (elem) ->
-    form_data = $("#edit_agent, #new_agent").serializeObject()
-    attribute = $(elem).data('attribute')
-    form_data['attribute'] = attribute
-    delete form_data['_method']
-    form_data
-
-  window.initializeFormCompletable = ->
-    returnedResults = {}
-    completableDefaultOptions = (input) ->
-      results: [
-        (returnedResults[$(input).data('attribute')] || {text: 'Options', children: [{id: undefined, text: 'loading ...'}]}),
-        {
-          text: 'Current',
-          children: [id: $(input).val(), text: $(input).val()]
-        },
-        {
-          text: 'Custom',
-          children: [id: 'manualInput', text: 'manual input']
-        },
-      ]
-
-    $("input[role~=validatable], select[role~=validatable]").on 'change', (e) =>
-      form_data = getFormData(e.currentTarget)
-      form_group = $(e.currentTarget).closest('.form-group')
-      $.ajax '/agents/validate',
-        type: 'POST',
-        data: form_data
-        success: (data) ->
-          form_group.addClass('has-feedback').removeClass('has-error')
-          form_group.find('span').addClass('hidden')
-          form_group.find('.glyphicon-ok').removeClass('hidden')
-          returnedResults = {}
-        error: (data) ->
-          form_group.addClass('has-feedback').addClass('has-error')
-          form_group.find('span').addClass('hidden')
-          form_group.find('.glyphicon-remove').removeClass('hidden')
-          returnedResults = {}
-
-    $("input[role~=validatable], select[role~=validatable]").trigger('change')
-
-    $.each $("input[role~=completable]"), (i, input) ->
-      $(input).select2(
-        data: ->
-          completableDefaultOptions(input)
-      ).on("change", (e) ->
-        if e.added && e.added.id == 'manualInput'
-          $(e.currentTarget).select2("destroy")
-          $(e.currentTarget).val(e.removed.id)
-      )
-
-    updateDropdownData = (form_data, element, data) ->
-      returnedResults[form_data.attribute] = {text: 'Options', children: data}
-      $(element).trigger('change')
-      $("input[role~=completable]").off 'select2-opening', select2OpeningCallback
-      $(element).select2('open')
-      $("input[role~=completable]").on 'select2-opening', select2OpeningCallback
-
-    select2OpeningCallback = (e) ->
-      form_data = getFormData(e.currentTarget)
-      delete returnedResults[form_data.attribute] if returnedResults[form_data.attribute] && !$(e.currentTarget).data('cacheResponse')
-      return if returnedResults[form_data.attribute]
-
-      $.ajax '/agents/complete',
-        type: 'POST',
-        data: form_data
-        success: (data) ->
-          updateDropdownData(form_data, e.currentTarget, data)
-        error: (data) ->
-          updateDropdownData(form_data, e.currentTarget, [{id: undefined, text: 'Error loading data.'}])
-
-    $("input[role~=completable]").on 'select2-opening', select2OpeningCallback
-
-    $("input[type=radio][role~=form-configurable]").change (e) ->
-      input = $(e.currentTarget).parents().siblings("input[data-attribute=#{$(e.currentTarget).data('attribute')}]")
-      if $(e.currentTarget).val() == 'manual'
-        input.removeClass('hidden')
-      else
-        input.val($(e.currentTarget).val())
-        input.addClass('hidden')

app/assets/javascripts/components/json-editor.js.coffee.erb

@@ -1,14 +0,0 @@
-window.setupJsonEditor = ($editors = $(".live-json-editor")) ->
-  JSONEditor.prototype.ADD_IMG = '<%= image_path 'json-editor/add.png' %>'
-  JSONEditor.prototype.DELETE_IMG = '<%= image_path 'json-editor/delete.png' %>'
-  editors = []
-  $editors.each ->
-    $editor = $(this)
-    jsonEditor = new JSONEditor($editor, $editor.data('width') || 400, $editor.data('height') || 500)
-    jsonEditor.doTruncation true
-    jsonEditor.showFunctionButtons()
-    editors.push jsonEditor
-  return editors
-
-$ ->
-  window.jsonEditor = setupJsonEditor()[0]

app/assets/javascripts/components/json-editor.js.erb

@@ -0,0 +1,22 @@
+window.setupJsonEditor = function ($editors) {
+  if ($editors == null) {
+    $editors = $(".live-json-editor");
+  }
+  JSONEditor.prototype.ADD_IMG = "<%= image_path 'json-editor/add.png' %>";
+  JSONEditor.prototype.DELETE_IMG = "<%= image_path 'json-editor/delete.png' %>";
+  const editors = [];
+  $editors.each(function () {
+    const $editor = $(this);
+    const jsonEditor = new JSONEditor(
+      $editor,
+      $editor.data("width") || 400,
+      $editor.data("height") || 500
+    );
+    jsonEditor.doTruncation(true);
+    jsonEditor.showFunctionButtons();
+    return editors.push(jsonEditor);
+  });
+  return editors;
+};
+
+$(() => (window.jsonEditor = setupJsonEditor()[0]));

app/assets/javascripts/components/search.js

@@ -0,0 +1,51 @@
+$(function () {
+  const $agentNavigate = $("#agent-navigate");
+
+  // initialize typeahead listener
+  $agentNavigate.bind("typeahead:selected", function (event, object, name) {
+    const item = object["value"];
+    $agentNavigate.typeahead("val", "");
+    if (window.agentPaths[item]) {
+      $(".spinner").show();
+      const navigationData = window.agentPaths[item];
+      if (
+        !(navigationData instanceof Object) ||
+        !navigationData.method ||
+        navigationData.method === "GET"
+      ) {
+        return (window.location = navigationData.url || navigationData);
+      } else {
+        return $.rails.handleMethod.apply(
+          $(
+            `<a href='${navigationData.url}' data-method='${navigationData.method}'></a>`
+          )
+            .appendTo($("body"))
+            .get(0)
+        );
+      }
+    }
+  });
+
+  // substring matcher for typeahead
+  const substringMatcher = function (strings) {
+    let findMatches;
+    return (findMatches = function (query, callback) {
+      const matches = [];
+      const substrRegex = new RegExp(query, "i");
+      $.each(strings, function (i, str) {
+        if (substrRegex.test(str)) {
+          return matches.push({ value: str });
+        }
+      });
+      return callback(matches.slice(0, 6));
+    });
+  };
+
+  return $agentNavigate.typeahead(
+    {
+      minLength: 1,
+      highlight: true,
+    },
+    { source: substringMatcher(window.agentNames) }
+  );
+});

app/assets/javascripts/components/search.js.coffee

@@ -1,29 +0,0 @@
-$ ->
-  $agentNavigate = $('#agent-navigate')
-
-  # initialize typeahead listener
-  $agentNavigate.bind "typeahead:selected", (event, object, name) ->
-    item = object['value']
-    $agentNavigate.typeahead('val', '')
-    if window.agentPaths[item]
-      $(".spinner").show()
-      navigationData = window.agentPaths[item]
-      if !(navigationData instanceof Object) || !navigationData.method || navigationData.method == 'GET'
-        window.location = navigationData.url || navigationData
-      else
-        $.rails.handleMethod.apply $("<a href='#{navigationData.url}' data-method='#{navigationData.method}'></a>").appendTo($("body")).get(0)
-
-  # substring matcher for typeahead
-  substringMatcher = (strings) ->
-    findMatches = (query, callback) ->
-      matches = []
-      substrRegex = new RegExp(query, "i")
-      $.each strings, (i, str) ->
-        matches.push value: str  if substrRegex.test(str)
-      callback(matches.slice(0,6))
-
-  $agentNavigate.typeahead
-    minLength: 1,
-    highlight: true,
-  ,
-    source: substringMatcher(window.agentNames)

app/assets/javascripts/components/utils.js

@@ -0,0 +1,221 @@
+(function () {
+  let escapeMap = undefined;
+  let createEscaper = undefined;
+  const Cls = (this.Utils = class Utils {
+    static initClass() {
+      // _.escape from underscore: https://github.com/jashkenas/underscore/blob/1e68f06610fa4ecb7f2c45d1eb2ad0173d6a2cc1/underscore.js#L1411-L1436
+      escapeMap = {
+        "&": "&",
+        "<": "&lt;",
+        ">": "&gt;",
+        '"': "&quot;",
+        "'": "&#x27;",
+        "`": "&#x60;",
+      };
+
+      createEscaper = function (map) {
+        const escaper = (match) => map[match];
+
+        // Regexes for identifying a key that needs to be escaped.
+        const source = "(?:" + Object.keys(map).join("|") + ")";
+        const testRegexp = RegExp(source);
+        const replaceRegexp = RegExp(source, "g");
+        return function (string) {
+          string = string === null ? "" : "" + string;
+          if (testRegexp.test(string)) {
+            return string.replace(replaceRegexp, escaper);
+          } else {
+            return string;
+          }
+        };
+      };
+
+      this.escape = createEscaper(escapeMap);
+    }
+    static navigatePath(path) {
+      if (!path.match(/^\//)) {
+        path = "/" + path;
+      }
+      return (window.location.href = path);
+    }
+
+    static currentPath() {
+      return window.location.href.replace(/https?:\/\/.*?\//g, "");
+    }
+
+    static registerPage(klass, options) {
+      if (options == null) {
+        options = {};
+      }
+      if (options.forPathsMatching != null) {
+        if (Utils.currentPath().match(options.forPathsMatching)) {
+          return (window.currentPage = new klass());
+        }
+      } else {
+        return new klass();
+      }
+    }
+
+    static showDynamicModal(content, param) {
+      if (content == null) {
+        content = "";
+      }
+      if (param == null) {
+        param = {};
+      }
+      const { title, body, onHide } = param;
+      $("body").append(`\
+<div class="modal fade" tabindex="-1" id='dynamic-modal' role="dialog" aria-labelledby="dynamic-modal-label" aria-hidden="true">
+  <div class="modal-dialog modal-lg">
+    <div class="modal-content">
+      <div class="modal-header">
+        <button type="button" class="close" data-dismiss="modal"><span aria-hidden="true">&times;</span><span class="sr-only">Close</span></button>
+        <h4 class="modal-title" id="dynamic-modal-label"></h4>
+      </div>
+      <div class="modal-body">${content}</div>
+    </div>
+  </div>
+</div>\
+`);
+      const modal = document.querySelector("#dynamic-modal");
+      $(modal)
+        .find(".modal-title")
+        .text(title || "")
+        .end()
+        .on("hidden.bs.modal", function () {
+          $("#dynamic-modal").remove();
+          return typeof onHide === "function" ? onHide() : undefined;
+        });
+      if (typeof body === "function") {
+        body(modal.querySelector(".modal-body"));
+      }
+      return $(modal).modal("show");
+    }
+
+    static handleDryRunButton(button, data) {
+      if (data == null) {
+        data = button.form
+          ? $(':input[name!="_method"]', button.form).serialize()
+          : "";
+      }
+      $(button).prop("disabled", true);
+      const cleanup = () => $(button).prop("disabled", false);
+
+      const url = $(button).data("action-url");
+      const with_event_mode = $(button).data("with-event-mode");
+
+      if (with_event_mode === "no") {
+        return this.invokeDryRun(url, data, cleanup);
+      }
+      return $.ajax(url, {
+        method: "GET",
+        data: {
+          with_event_mode,
+          source_ids: $.map($(".link-region select option:selected"), (el) =>
+            $(el).val()
+          ),
+        },
+        success: (modal_data) => {
+          return Utils.showDynamicModal(modal_data, {
+            body: (body) => {
+              let previous;
+              const form = $(body).find(".dry-run-form");
+              const payload_editor = form.find(".payload-editor");
+
+              if ((previous = $(button).data("payload"))) {
+                payload_editor.text(previous);
+              }
+
+              const editor = window.setupJsonEditor(payload_editor)[0];
+
+              $(body)
+                .find(".dry-run-event-sample")
+                .click((e) => {
+                  e.preventDefault();
+                  editor.json = $(e.currentTarget).data("payload");
+                  return editor.rebuild();
+                });
+
+              form.submit((e) => {
+                let dry_run_data;
+                e.preventDefault();
+                let json = $(e.target).find(".payload-editor").val();
+                if (json === "") {
+                  json = "{}";
+                }
+                try {
+                  const payload = JSON.parse(
+                    json.replace(/\\\\([n|r|t])/g, "\\$1")
+                  );
+                  if (payload.constructor !== Object) {
+                    throw true;
+                  }
+                  if (Object.keys(payload).length === 0) {
+                    json = "";
+                  } else {
+                    json = JSON.stringify(payload);
+                  }
+                } catch (error) {
+                  alert("Invalid JSON object.");
+                  return;
+                }
+                if (json === "") {
+                  if (with_event_mode === "yes") {
+                    alert("Event is required for this agent to run.");
+                    return;
+                  }
+                  dry_run_data = data;
+                  $(button).data("payload", null);
+                } else {
+                  dry_run_data = `event=${encodeURIComponent(json)}&${data}`;
+                  $(button).data("payload", json);
+                }
+                return $(body)
+                  .closest("[role=dialog]")
+                  .on("hidden.bs.modal", () => {
+                    return this.invokeDryRun(url, dry_run_data, cleanup);
+                  })
+                  .modal("hide");
+              });
+              return $(body)
+                .closest("[role=dialog]")
+                .on("shown.bs.modal", function () {
+                  return $(this).find(".btn-primary").focus();
+                });
+            },
+            title: "Dry Run",
+            onHide: cleanup,
+          });
+        },
+      });
+    }
+
+    static invokeDryRun(url, data, callback) {
+      $("body").css({ cursor: "progress" });
+      return $.ajax({ type: "POST", url, dataType: "html", data })
+        .always(() => {
+          return $("body").css({ cursor: "auto" });
+        })
+        .done((modal_data) => {
+          return Utils.showDynamicModal(modal_data, {
+            title: "Dry Run Results",
+            onHide: callback,
+          });
+        })
+        .fail(function (xhr, status, error) {
+          alert("Error: " + error);
+          return callback();
+        });
+    }
+
+    static select2TagClickHandler(e, elem) {
+      if (e.which === 1) {
+        return (window.location = $(elem).attr("href"));
+      } else {
+        return window.open($(elem).attr("href"));
+      }
+    }
+  });
+  Cls.initClass();
+  return Cls;
+})();

app/assets/javascripts/components/utils.js.coffee

@@ -1,138 +0,0 @@
-class @Utils
-  @navigatePath: (path) ->
-    path = "/" + path unless path.match(/^\//)
-    window.location.href = path
-
-  @currentPath: ->
-    window.location.href.replace(/https?:\/\/.*?\//g, '')
-
-  @registerPage: (klass, options = {}) ->
-    if options.forPathsMatching?
-      if Utils.currentPath().match(options.forPathsMatching)
-        window.currentPage = new klass()
-    else
-      new klass()
-
-  @showDynamicModal: (content = '', { title, body, onHide } = {}) ->
-    $("body").append """
-      <div class="modal fade" tabindex="-1" id='dynamic-modal' role="dialog" aria-labelledby="dynamic-modal-label" aria-hidden="true">
-        <div class="modal-dialog modal-lg">
-          <div class="modal-content">
-            <div class="modal-header">
-              <button type="button" class="close" data-dismiss="modal"><span aria-hidden="true">&times;</span><span class="sr-only">Close</span></button>
-              <h4 class="modal-title" id="dynamic-modal-label"></h4>
-            </div>
-            <div class="modal-body">#{content}</div>
-          </div>
-        </div>
-      </div>
-      """
-    modal = document.querySelector('#dynamic-modal')
-    $(modal).find('.modal-title').text(title || '').end().on 'hidden.bs.modal', ->
-      $('#dynamic-modal').remove()
-      onHide?()
-    body?(modal.querySelector('.modal-body'))
-    $(modal).modal('show')
-
-  @handleDryRunButton: (button, data = if button.form then $(':input[name!="_method"]', button.form).serialize() else '') ->
-    $(button).prop('disabled', true)
-    cleanup = -> $(button).prop('disabled', false)
-
-    url = $(button).data('action-url')
-    with_event_mode = $(button).data('with-event-mode')
-
-    if with_event_mode is 'no'
-      return @invokeDryRun(url, data, cleanup)
-    $.ajax url,
-      method: 'GET',
-      data:
-        with_event_mode: with_event_mode
-        source_ids: $.map($(".link-region select option:selected"), (el) -> $(el).val() )
-      success: (modal_data) =>
-        Utils.showDynamicModal modal_data,
-          body: (body) =>
-            form = $(body).find('.dry-run-form')
-            payload_editor = form.find('.payload-editor')
-
-            if previous = $(button).data('payload')
-              payload_editor.text(previous)
-
-            editor = window.setupJsonEditor(payload_editor)[0]
-
-            $(body).find('.dry-run-event-sample').click (e) =>
-              e.preventDefault()
-              editor.json = $(e.currentTarget).data('payload')
-              editor.rebuild()
-
-            form.submit (e) =>
-              e.preventDefault()
-              json = $(e.target).find('.payload-editor').val()
-              json = '{}' if json == ''
-              try
-                payload = JSON.parse(json.replace(/\\\\([n|r|t])/g, "\\$1"))
-                throw true unless payload.constructor is Object
-                if Object.keys(payload).length == 0
-                  json = ''
-                else
-                  json = JSON.stringify(payload)
-              catch
-                alert 'Invalid JSON object.'
-                return
-              if json == ''
-                if with_event_mode is 'yes'
-                  alert 'Event is required for this agent to run.'
-                  return
-                dry_run_data = data
-                $(button).data('payload', null)
-              else
-                dry_run_data = "event=#{encodeURIComponent(json)}&#{data}"
-                $(button).data('payload', json)
-              $(body).closest('[role=dialog]').on 'hidden.bs.modal', =>
-                @invokeDryRun(url, dry_run_data, cleanup)
-              .modal('hide')
-            $(body).closest('[role=dialog]').on 'shown.bs.modal', ->
-              $(this).find('.btn-primary').focus()
-          title: 'Dry Run'
-          onHide: cleanup
-
-  @invokeDryRun: (url, data, callback) ->
-    $('body').css(cursor: 'progress')
-    $.ajax type: 'POST', url: url, dataType: 'html', data: data
-      .always =>
-        $('body').css(cursor: 'auto')
-      .done (modal_data) =>
-        Utils.showDynamicModal modal_data,
-          title: 'Dry Run Results',
-          onHide: callback
-      .fail (xhr, status, error) ->
-        alert('Error: ' + error)
-        callback()
-
-  @select2TagClickHandler: (e, elem) ->
-    if e.which == 1
-      window.location = $(elem).attr('href')
-    else
-      window.open($(elem).attr('href'))
-
-  # _.escape from underscore: https://github.com/jashkenas/underscore/blob/1e68f06610fa4ecb7f2c45d1eb2ad0173d6a2cc1/underscore.js#L1411-L1436
-  escapeMap =
-    '&': '&amp;'
-    '<': '&lt;'
-    '>': '&gt;'
-    '"': '&quot;'
-    '\'': '&#x27;'
-    '`': '&#x60;'
-
-  createEscaper = (map) ->
-    escaper = (match) ->
-      map[match]
-
-    # Regexes for identifying a key that needs to be escaped.
-    source = '(?:' + Object.keys(map).join('|') + ')'
-    testRegexp = RegExp(source)
-    replaceRegexp = RegExp(source, 'g')
-    (string) ->
-      string = if string == null then '' else '' + string
-      if testRegexp.test(string) then string.replace(replaceRegexp, escaper) else string
-
-  @escape = createEscaper(escapeMap)

app/assets/javascripts/components/worker-checker.js

@@ -0,0 +1,90 @@
+$(function () {
+  let sinceId = null;
+  let previousJobs = null;
+
+  if ($(".job-indicator").length) {
+    var check = function () {
+      const query = sinceId != null ? "?since_id=" + sinceId : "";
+      return $.getJSON("/worker_status" + query, function (json) {
+        for (var method of ["pending", "awaiting_retry", "recent_failures"]) {
+          var count = json[method];
+          var elem = $(`.job-indicator[role=${method}]`);
+          if (count > 0) {
+            var tooltipOptions = {
+              title: `${count} jobs ${method.split("_").join(" ")}`,
+              delay: 0,
+              placement: "bottom",
+              trigger: "hover",
+            };
+            if (elem.is(":visible")) {
+              elem
+                .tooltip("destroy")
+                .tooltip(tooltipOptions)
+                .find(".number")
+                .text(count);
+            } else {
+              elem
+                .tooltip("destroy")
+                .tooltip(tooltipOptions)
+                .fadeIn()
+                .find(".number")
+                .text(count);
+            }
+          } else {
+            if (elem.is(":visible")) {
+              elem.tooltip("destroy").fadeOut();
+            }
+          }
+        }
+
+        if (sinceId != null && json.event_count > 0) {
+          $("#event-indicator")
+            .tooltip("destroy")
+            .tooltip({
+              title: "Click to see the events",
+              delay: 0,
+              placement: "bottom",
+              trigger: "hover",
+            })
+            .find("a")
+            .attr({ href: json.events_url })
+            .end()
+            .fadeIn()
+            .find(".number")
+            .text(json.event_count);
+        } else {
+          $("#event-indicator").tooltip("destroy").fadeOut();
+        }
+
+        if (sinceId == null) {
+          sinceId = json.max_id;
+        }
+        const currentJobs = [
+          json.pending,
+          json.awaiting_retry,
+          json.recent_failures,
+        ];
+        if (
+          document.location.pathname === "/jobs" &&
+          $(".modal[aria-hidden=false]").length === 0 &&
+          previousJobs != null &&
+          previousJobs.join(",") !== currentJobs.join(",")
+        ) {
+          if (
+            !document.location.search ||
+            document.location.search === "?page=1"
+          ) {
+            $.get("/jobs", (data) => {
+              return $("#main-content").html(data);
+            });
+          }
+        }
+        previousJobs = currentJobs;
+
+        return (window.workerCheckTimeout = setTimeout(check, 2000));
+      });
+    };
+
+    return check();
+  }
+});

app/assets/javascripts/components/worker-checker.js.coffee

@@ -1,51 +0,0 @@
-$ ->
-  sinceId = null
-  previousJobs = null
-
-  if $(".job-indicator").length
-    check = ->
-      query =
-        if sinceId?
-          '?since_id=' + sinceId
-        else
-          ''
-      $.getJSON "/worker_status" + query, (json) ->
-        for method in ['pending', 'awaiting_retry', 'recent_failures']
-          count = json[method]
-          elem = $(".job-indicator[role=#{method}]")
-          if count > 0
-            tooltipOptions = {
-              title: "#{count} jobs #{method.split('_').join(' ')}"
-              delay: 0
-              placement: "bottom"
-              trigger: "hover"
-            }
-            if elem.is(":visible")
-              elem.tooltip('destroy').tooltip(tooltipOptions).find(".number").text(count)
-            else
-              elem.tooltip('destroy').tooltip(tooltipOptions).fadeIn().find(".number").text(count)
-          else
-            if elem.is(":visible")
-              elem.tooltip('destroy').fadeOut()
-
-        if sinceId? && json.event_count > 0
-          $("#event-indicator").tooltip('destroy').
-                                tooltip(title: "Click to see the events", delay: 0, placement: "bottom", trigger: "hover").
-                                find('a').attr(href: json.events_url).end().
-                                fadeIn().
-                                find(".number").
-                                text(json.event_count)
-        else
-          $("#event-indicator").tooltip('destroy').fadeOut()
-
-        sinceId ?= json.max_id
-        currentJobs = [json.pending, json.awaiting_retry, json.recent_failures]
-        if document.location.pathname == '/jobs' && $(".modal[aria-hidden=false]").length == 0 && previousJobs? && previousJobs.join(',') != currentJobs.join(',')
-          if !document.location.search || document.location.search == '?page=1'
-            $.get '/jobs', (data) =>
-              $("#main-content").html(data)
-        previousJobs = currentJobs
-
-        window.workerCheckTimeout = setTimeout check, 2000
-
-    check()

app/assets/javascripts/diagram.js

@@ -0,0 +1,29 @@
+// This is not included in the core application.js bundle.
+
+$(function () {
+  const svg = document.querySelector(".agent-diagram svg.diagram");
+  const overlay = document.querySelector(".agent-diagram .overlay");
+  $(overlay).width($(svg).width()).height($(svg).height());
+  const getTopLeft = function (node) {
+    const bbox = node.getBBox();
+    const point = svg.createSVGPoint();
+    point.x = bbox.x + bbox.width;
+    point.y = bbox.y;
+    return point.matrixTransform(node.getCTM());
+  };
+  return $(svg)
+    .find("g.node[data-badge-id]")
+    .each(function () {
+      const tl = getTopLeft(this);
+      $("#" + this.getAttribute("data-badge-id"), overlay).each(function () {
+        const badge = $(this);
+        badge
+          .css({
+            left: tl.x - badge.outerWidth() * (2 / 3),
+            top: tl.y - badge.outerHeight() * (1 / 3),
+            "background-color": badge.find(".label").css("background-color"),
+          })
+          .show();
+      });
+    });
+});

app/assets/javascripts/diagram.js.coffee

@@ -1,23 +0,0 @@
-# This is not included in the core application.js bundle.
-
-$ ->
-  svg = document.querySelector('.agent-diagram svg.diagram')
-  overlay = document.querySelector('.agent-diagram .overlay')
-  $(overlay).width($(svg).width()).height($(svg).height())
-  getTopLeft = (node) ->
-    bbox = node.getBBox()
-    point = svg.createSVGPoint()
-    point.x = bbox.x + bbox.width
-    point.y = bbox.y
-    point.matrixTransform(node.getCTM())
-  $(svg).find('g.node[data-badge-id]').each ->
-    tl = getTopLeft(this)
-    $('#' + this.getAttribute('data-badge-id'), overlay).each ->
-      badge = $(this)
-      badge.css
-        left: tl.x - badge.outerWidth()  * (2/3)
-        top:  tl.y - badge.outerHeight() * (1/3)
-        'background-color': badge.find('.label').css('background-color')
-      .show()
-      return
-    return

app/assets/javascripts/graphing.js

@@ -0,0 +1,76 @@
+//= require d3
+//= require rickshaw
+//= require_self
+
+// This is not included in the core application.js bundle.
+
+window.renderGraph = function ($chart, data, peaks, name) {
+  const graph = new Rickshaw.Graph({
+    element: $chart.find(".chart").get(0),
+    width: 700,
+    height: 240,
+    series: [
+      {
+        data,
+        name,
+        color: "steelblue",
+      },
+    ],
+  });
+
+  const x_axis = new Rickshaw.Graph.Axis.Time({ graph });
+
+  const annotator = new Rickshaw.Graph.Annotate({
+    graph,
+    element: $chart.find(".timeline").get(0),
+  });
+  $.each(peaks, function () {
+    return annotator.add(this, "Peak");
+  });
+
+  const y_axis = new Rickshaw.Graph.Axis.Y({
+    graph,
+    orientation: "left",
+    tickFormat: Rickshaw.Fixtures.Number.formatKMBT,
+    element: $chart.find(".y-axis").get(0),
+  });
+
+  graph.onUpdate(function () {
+    const mean = d3.mean(data, (i) => i.y);
+    const standard_deviation = Math.sqrt(
+      d3.mean(data.map((i) => Math.pow(i.y - mean, 2)))
+    );
+    const minX = d3.min(data, (i) => i.x);
+    const maxX = d3.max(data, (i) => i.x);
+    graph.vis
+      .append("svg:line")
+      .attr("x1", graph.x(minX))
+      .attr("x2", graph.x(maxX))
+      .attr("y1", graph.y(mean))
+      .attr("y2", graph.y(mean))
+      .attr("class", "summary-statistic mean");
+    graph.vis
+      .append("svg:line")
+      .attr("x1", graph.x(minX))
+      .attr("x2", graph.x(maxX))
+      .attr("y1", graph.y(mean + standard_deviation))
+      .attr("y2", graph.y(mean + standard_deviation))
+      .attr("class", "summary-statistic one-std");
+    graph.vis
+      .append("svg:line")
+      .attr("x1", graph.x(minX))
+      .attr("x2", graph.x(maxX))
+      .attr("y1", graph.y(mean + 2 * standard_deviation))
+      .attr("y2", graph.y(mean + 2 * standard_deviation))
+      .attr("class", "summary-statistic two-std");
+    return graph.vis
+      .append("svg:line")
+      .attr("x1", graph.x(minX))
+      .attr("x2", graph.x(maxX))
+      .attr("y1", graph.y(mean + 3 * standard_deviation))
+      .attr("y2", graph.y(mean + 3 * standard_deviation))
+      .attr("class", "summary-statistic three-std");
+  });
+
+  return graph.render();
+};

app/assets/javascripts/graphing.js.coffee

@@ -1,62 +0,0 @@
-#= require d3
-#= require rickshaw
-#= require_self
-
-# This is not included in the core application.js bundle.
-
-window.renderGraph = ($chart, data, peaks, name) ->
-  graph = new Rickshaw.Graph
-    element: $chart.find(".chart").get(0)
-    width: 700
-    height: 240
-    series: [
-      data: data
-      name: name
-      color: 'steelblue'
-    ]
-
-  x_axis = new Rickshaw.Graph.Axis.Time(graph: graph)
-
-  annotator = new Rickshaw.Graph.Annotate
-     graph: graph
-     element: $chart.find(".timeline").get(0)
-  $.each peaks, ->
-    annotator.add this, "Peak"
-
-  y_axis = new Rickshaw.Graph.Axis.Y
-    graph: graph
-    orientation: 'left'
-    tickFormat: Rickshaw.Fixtures.Number.formatKMBT
-    element: $chart.find(".y-axis").get(0)
-
-  graph.onUpdate ->
-    mean = d3.mean data, (i) -> i.y
-    standard_deviation = Math.sqrt(d3.mean(data.map((i) -> Math.pow(i.y - mean, 2))))
-    minX = d3.min data, (i) -> i.x
-    maxX = d3.max data, (i) -> i.x
-    graph.vis.append("svg:line")
-      .attr('x1', graph.x(minX))
-      .attr('x2', graph.x(maxX))
-      .attr('y1', graph.y(mean))
-      .attr('y2', graph.y(mean))
-      .attr 'class', 'summary-statistic mean'
-    graph.vis.append("svg:line")
-      .attr('x1', graph.x(minX))
-      .attr('x2', graph.x(maxX))
-      .attr('y1', graph.y(mean + standard_deviation))
-      .attr('y2', graph.y(mean + standard_deviation))
-      .attr 'class', 'summary-statistic one-std'
-    graph.vis.append("svg:line")
-      .attr('x1', graph.x(minX))
-      .attr('x2', graph.x(maxX))
-      .attr('y1', graph.y(mean + 2 * standard_deviation))
-      .attr('y2', graph.y(mean + 2 * standard_deviation))
-      .attr 'class', 'summary-statistic two-std'
-    graph.vis.append("svg:line")
-      .attr('x1', graph.x(minX))
-      .attr('x2', graph.x(maxX))
-      .attr('y1', graph.y(mean + 3 * standard_deviation))
-      .attr('y2', graph.y(mean + 3 * standard_deviation))
-      .attr 'class', 'summary-statistic three-std'
-
-  graph.render()

app/assets/javascripts/map_marker.js

@@ -0,0 +1,48 @@
+window.map_marker = function (map, options) {
+  let marker;
+  if (options == null) {
+    options = {};
+  }
+  const pos = new google.maps.LatLng(options.lat, options.lng);
+
+  if (options.radius > 0) {
+    marker = new google.maps.Circle({
+      map,
+      strokeColor: "#FF0000",
+      strokeOpacity: 0.8,
+      strokeWeight: 2,
+      fillColor: "#FF0000",
+      fillOpacity: 0.35,
+      center: pos,
+      radius: options.radius,
+    });
+    return marker;
+  } else if (options.course) {
+    const p1 = new LatLon(pos.lat(), pos.lng());
+    const speed = options.speed != null ? options.speed : 1;
+    const p2 = p1.destinationPoint(options.course, Math.max(0.2, speed) * 0.1);
+
+    const lineCoordinates = [pos, new google.maps.LatLng(p2.lat(), p2.lon())];
+
+    const lineSymbol = { path: google.maps.SymbolPath.FORWARD_CLOSED_ARROW };
+
+    const arrow = new google.maps.Polyline({
+      map,
+      path: lineCoordinates,
+      icons: [
+        {
+          icon: lineSymbol,
+          offset: "100%",
+        },
+      ],
+    });
+    return arrow;
+  } else {
+    marker = new google.maps.Marker({
+      map,
+      position: pos,
+      title: "Recorded Location",
+    });
+    return marker;
+  }
+};

app/assets/javascripts/map_marker.js.coffee

@@ -1,43 +0,0 @@
-window.map_marker = (map, options = {}) ->
-  pos = new google.maps.LatLng(options.lat, options.lng)
-
-  if options.radius > 0
-    marker = new google.maps.Circle
-      map: map
-      strokeColor: '#FF0000'
-      strokeOpacity: 0.8
-      strokeWeight: 2
-      fillColor: '#FF0000'
-      fillOpacity: 0.35
-      center: pos
-      radius: options.radius
-    return marker
-  else if options.course
-    p1 = new LatLon(pos.lat(), pos.lng())
-    speed = options.speed ? 1
-    p2 = p1.destinationPoint(options.course, Math.max(0.2, speed) * 0.1)
-
-    lineCoordinates = [
-      pos
-      new google.maps.LatLng(p2.lat(), p2.lon())
-    ]
-
-    lineSymbol =
-      path: google.maps.SymbolPath.FORWARD_CLOSED_ARROW
-
-    arrow = new google.maps.Polyline
-      map: map
-      path: lineCoordinates
-      icons: [
-        {
-          icon: lineSymbol
-          offset: '100%'
-        }
-      ]
-    return arrow
-  else
-    marker = new google.maps.Marker
-      map: map
-      position: pos
-      title: 'Recorded Location'
-    return marker

app/assets/javascripts/pages/agent-edit-page.js

@@ -0,0 +1,340 @@
+(function () {
+  let formatAgentForSelect = undefined;
+  const Cls = (this.AgentEditPage = class AgentEditPage {
+    static initClass() {
+      formatAgentForSelect = function (agent) {
+        const originalOption = agent.element;
+        const description = agent.element[0].title;
+        return "<strong>" + agent.text + "</strong><br/>" + description;
+      };
+    }
+    constructor() {
+      this.invokeDryRun = this.invokeDryRun.bind(this);
+      $("#agent_source_ids").on("change", this.showEventDescriptions);
+      this.showCorrectRegionsOnStartup();
+      $("form.agent-form").on("submit", () => this.updateFromEditors());
+
+      // Validate agents_options Json on form submit
+      $("form.agent-form").submit(function (e) {
+        if ($("textarea#agent_options").length) {
+          try {
+            JSON.parse($("#agent_options").val());
+          } catch (err) {
+            e.preventDefault();
+            alert(
+              "Sorry, there appears to be an error in your JSON input. Please fix it before continuing."
+            );
+            return false;
+          }
+        }
+
+        if (
+          $(".link-region").length &&
+          $(".link-region").data("can-receive-events") === false
+        ) {
+          $(".link-region .select2-linked-tags option:selected").removeAttr(
+            "selected"
+          );
+        }
+
+        if (
+          $(".control-link-region").length &&
+          $(".control-link-region").data("can-control-other-agents") === false
+        ) {
+          $(
+            ".control-link-region .select2-linked-tags option:selected"
+          ).removeAttr("selected");
+        }
+
+        if (
+          $(".event-related-region").length &&
+          $(".event-related-region").data("can-create-events") === false
+        ) {
+          return $(
+            ".event-related-region .select2-linked-tags option:selected"
+          ).removeAttr("selected");
+        }
+      });
+
+      $("#agent_name").each(function () {
+        // Select the number suffix if this is a cloned agent.
+        let matches;
+        if ((matches = this.value.match(/ \(\d+\)$/))) {
+          this.focus();
+          if (this.selectionStart != null) {
+            this.selectionStart = matches.index;
+            return (this.selectionEnd = this.value.length);
+          }
+        }
+      });
+
+      // The type selector is only available on the new agent form.
+      if ($("#agent_type").length) {
+        $("#agent_type").on("change", () => this.handleTypeChange(false));
+        this.handleTypeChange(true);
+
+        // Update the dropdown to match agent description as well as agent name
+        $("select#agent_type").select2({
+          width: "resolve",
+          formatResult: formatAgentForSelect,
+          escapeMarkup: (m) => m,
+          matcher: (params, data) => {
+            const term = params.term;
+            if (term == null) return data;
+            const upperTerm = term.toUpperCase();
+            return data.text.toUpperCase().indexOf(upperTerm) >= 0 ||
+              data.title.toUpperCase().indexOf(upperTerm) >= 0
+              ? data
+              : null;
+          },
+        });
+      } else {
+        this.enableDryRunButton();
+        this.buildAce();
+      }
+    }
+
+    handleTypeChange(firstTime) {
+      $(".event-descriptions").html("").hide();
+      const type = $("#agent_type").val();
+
+      if (type === "Agent") {
+        $(".agent-settings").hide();
+        return $(".description").hide();
+      } else {
+        $(".agent-settings").show();
+        $("#agent-spinner").fadeIn();
+        if (!firstTime) {
+          $(".model-errors").hide();
+        }
+        return $.getJSON("/agents/type_details", { type }, (json) => {
+          if (json.can_be_scheduled) {
+            if (firstTime) {
+              this.showSchedule();
+            } else {
+              this.showSchedule(json.default_schedule);
+            }
+          } else {
+            this.hideSchedule();
+          }
+
+          if (json.can_receive_events) {
+            this.showLinks();
+          } else {
+            this.hideLinks();
+          }
+
+          if (json.can_control_other_agents) {
+            this.showControlLinks();
+          } else {
+            this.hideControlLinks();
+          }
+
+          if (json.can_create_events) {
+            this.showEventCreation();
+          } else {
+            this.hideEventCreation();
+          }
+
+          if (json.description_html != null) {
+            $(".description").show().html(json.description_html);
+          }
+
+          if (!firstTime) {
+            if (json.oauthable != null) {
+              $(".oauthable-form").html(json.oauthable);
+            }
+            if (json.form_options != null) {
+              $(".agent-options").html(json.form_options);
+            }
+            window.jsonEditor = setupJsonEditor()[0];
+          }
+
+          this.enableDryRunButton();
+          this.buildAce();
+
+          window.initializeFormCompletable();
+
+          return $("#agent-spinner").stop(true, true).fadeOut();
+        });
+      }
+    }
+
+    hideSchedule() {
+      $(".schedule-region .can-be-scheduled").hide();
+      return $(".schedule-region .cannot-be-scheduled").show();
+    }
+
+    showSchedule(defaultSchedule = null) {
+      if (defaultSchedule != null) {
+        $(".schedule-region select").val(defaultSchedule).change();
+      }
+      $(".schedule-region .can-be-scheduled").show();
+      return $(".schedule-region .cannot-be-scheduled").hide();
+    }
+
+    hideLinks() {
+      $(".link-region .select2-container").hide();
+      $(".link-region .propagate-immediately").hide();
+      $(".link-region .cannot-receive-events").show();
+      return $(".link-region").data("can-receive-events", false);
+    }
+
+    showLinks() {
+      $(".link-region .select2-container").show();
+      $(".link-region .propagate-immediately").show();
+      $(".link-region .cannot-receive-events").hide();
+      $(".link-region").data("can-receive-events", true);
+      return this.showEventDescriptions();
+    }
+
+    hideControlLinks() {
+      $(".control-link-region").hide();
+      return $(".control-link-region").data("can-control-other-agents", false);
+    }
+
+    showControlLinks() {
+      $(".control-link-region").show();
+      return $(".control-link-region").data("can-control-other-agents", true);
+    }
+
+    hideEventCreation() {
+      $(".event-related-region .select2-container").hide();
+      $(".event-related-region .cannot-create-events").show();
+      return $(".event-related-region").data("can-create-events", false);
+    }
+
+    showEventCreation() {
+      $(".event-related-region .select2-container").show();
+      $(".event-related-region .cannot-create-events").hide();
+      return $(".event-related-region").data("can-create-events", true);
+    }
+
+    showEventDescriptions() {
+      if ($("#agent_source_ids").val()) {
+        return $.getJSON(
+          "/agents/event_descriptions",
+          { ids: $("#agent_source_ids").val().join(",") },
+          (json) => {
+            if (json.description_html != null) {
+              return $(".event-descriptions")
+                .show()
+                .html(json.description_html);
+            } else {
+              return $(".event-descriptions").hide();
+            }
+          }
+        );
+      } else {
+        return $(".event-descriptions").html("").hide();
+      }
+    }
+
+    showCorrectRegionsOnStartup() {
+      if ($(".schedule-region")) {
+        if ($(".schedule-region").data("can-be-scheduled") === true) {
+          this.showSchedule();
+        } else {
+          this.hideSchedule();
+        }
+      }
+
+      if ($(".link-region")) {
+        if ($(".link-region").data("can-receive-events") === true) {
+          this.showLinks();
+        } else {
+          this.hideLinks();
+        }
+      }
+
+      if ($(".control-link-region")) {
+        if (
+          $(".control-link-region").data("can-control-other-agents") === true
+        ) {
+          this.showControlLinks();
+        } else {
+          this.hideControlLinks();
+        }
+      }
+
+      if ($(".event-related-region")) {
+        if ($(".event-related-region").data("can-create-events") === true) {
+          return this.showEventCreation();
+        } else {
+          return this.hideEventCreation();
+        }
+      }
+    }
+
+    buildAce() {
+      return $(".ace-editor").each(function () {
+        if (!$(this).data("initialized")) {
+          const $this = $(this);
+          $this.data("initialized", true);
+          const $source = $($this.data("source")).hide();
+          const editor = ace.edit(this);
+          $this.data("ace-editor", editor);
+          const session = editor.getSession();
+          session.setTabSize(2);
+          session.setUseSoftTabs(true);
+          session.setUseWrapMode(false);
+
+          const setSyntax = function () {
+            let mode, theme;
+            if ((mode = $this.data("mode"))) {
+              session.setMode("ace/mode/" + mode);
+            }
+
+            if ((theme = $this.data("theme"))) {
+              editor.setTheme("ace/theme/" + theme);
+            }
+
+            if ((mode = $("[name='agent[options][language]']").val())) {
+              switch (mode) {
+                case "JavaScript":
+                  return session.setMode("ace/mode/javascript");
+                case "CoffeeScript":
+                  return session.setMode("ace/mode/coffee");
+                default:
+                  return session.setMode("ace/mode/" + mode);
+              }
+            }
+          };
+
+          $("[name='agent[options][language]']").on("change", setSyntax);
+          setSyntax();
+
+          return session.setValue($source.val());
+        }
+      });
+    }
+
+    updateFromEditors() {
+      return $(".ace-editor").each(function () {
+        const $source = $($(this).data("source"));
+        return $source.val($(this).data("ace-editor").getSession().getValue());
+      });
+    }
+
+    enableDryRunButton() {
+      return $(".agent-dry-run-button")
+        .prop("disabled", false)
+        .off()
+        .on("click", this.invokeDryRun);
+    }
+
+    disableDryRunButton() {
+      return $(".agent-dry-run-button").prop("disabled", true);
+    }
+
+    invokeDryRun(e) {
+      e.preventDefault();
+      this.updateFromEditors();
+      return Utils.handleDryRunButton(e.currentTarget);
+    }
+  });
+  Cls.initClass();
+  return Cls;
+})();
+
+$(() => Utils.registerPage(AgentEditPage, { forPathsMatching: /^agents/ }));

app/assets/javascripts/pages/agent-edit-page.js.coffee

@@ -1,231 +0,0 @@
-class @AgentEditPage
-  constructor: ->
-    $("#agent_source_ids").on "change", @showEventDescriptions
-    @showCorrectRegionsOnStartup()
-    $("form.agent-form").on "submit", => @updateFromEditors()
-
-    # Validate agents_options Json on form submit
-    $('form.agent-form').submit (e) ->
-      if $('textarea#agent_options').length
-        try
-          JSON.parse $('#agent_options').val()
-        catch err
-          e.preventDefault()
-          alert 'Sorry, there appears to be an error in your JSON input. Please fix it before continuing.'
-          return false
-
-      if $(".link-region").length && $(".link-region").data("can-receive-events") == false
-        $(".link-region .select2-linked-tags option:selected").removeAttr('selected')
-
-      if $(".control-link-region").length && $(".control-link-region").data("can-control-other-agents") == false
-        $(".control-link-region .select2-linked-tags option:selected").removeAttr('selected')
-
-      if $(".event-related-region").length && $(".event-related-region").data("can-create-events") == false
-        $(".event-related-region .select2-linked-tags option:selected").removeAttr('selected')
-
-    $("#agent_name").each ->
-      # Select the number suffix if this is a cloned agent.
-      if matches = this.value.match(/ \(\d+\)$/)
-        this.focus()
-        if this.selectionStart?
-          this.selectionStart = matches.index
-          this.selectionEnd = this.value.length
-
-    # The type selector is only available on the new agent form.
-    if $("#agent_type").length
-      $("#agent_type").on "change", => @handleTypeChange(false)
-      @handleTypeChange(true)
-
-      # Update the dropdown to match agent description as well as agent name
-      $('select#agent_type').select2
-        width: 'resolve'
-        formatResult: formatAgentForSelect
-        escapeMarkup: (m) ->
-          m
-        matcher: (term, text, opt) ->
-          description = opt.attr('title')
-          text.toUpperCase().indexOf(term.toUpperCase()) >= 0 or description.toUpperCase().indexOf(term.toUpperCase()) >= 0
-
-    else
-      @enableDryRunButton()
-      @buildAce()
-
-  handleTypeChange: (firstTime) ->
-    $(".event-descriptions").html("").hide()
-    type = $('#agent_type').val()
-
-    if type == 'Agent'
-      $(".agent-settings").hide()
-      $(".description").hide()
-    else
-      $(".agent-settings").show()
-      $("#agent-spinner").fadeIn()
-      $(".model-errors").hide() unless firstTime
-      $.getJSON "/agents/type_details", { type: type }, (json) =>
-        if json.can_be_scheduled
-          if firstTime
-            @showSchedule()
-          else
-            @showSchedule(json.default_schedule)
-        else
-          @hideSchedule()
-
-        if json.can_receive_events
-          @showLinks()
-        else
-          @hideLinks()
-
-        if json.can_control_other_agents
-          @showControlLinks()
-        else
-          @hideControlLinks()
-
-        if json.can_create_events
-          @showEventCreation()
-        else
-          @hideEventCreation()
-
-        $(".description").show().html(json.description_html) if json.description_html?
-
-        unless firstTime
-          $('.oauthable-form').html(json.oauthable) if json.oauthable?
-          $('.agent-options').html(json.form_options) if json.form_options?
-          window.jsonEditor = setupJsonEditor()[0]
-
-        @enableDryRunButton()
-        @buildAce()
-
-        window.initializeFormCompletable()
-
-        $("#agent-spinner").stop(true, true).fadeOut();
-
-  hideSchedule: ->
-    $(".schedule-region .can-be-scheduled").hide()
-    $(".schedule-region .cannot-be-scheduled").show()
-
-  showSchedule: (defaultSchedule = null) ->
-    if defaultSchedule?
-      $(".schedule-region select").val(defaultSchedule).change()
-    $(".schedule-region .can-be-scheduled").show()
-    $(".schedule-region .cannot-be-scheduled").hide()
-
-  hideLinks: ->
-    $(".link-region .select2-container").hide()
-    $(".link-region .propagate-immediately").hide()
-    $(".link-region .cannot-receive-events").show()
-    $(".link-region").data("can-receive-events", false)
-
-  showLinks: ->
-    $(".link-region .select2-container").show()
-    $(".link-region .propagate-immediately").show()
-    $(".link-region .cannot-receive-events").hide()
-    $(".link-region").data("can-receive-events", true)
-    @showEventDescriptions()
-
-  hideControlLinks: ->
-    $(".control-link-region").hide()
-    $(".control-link-region").data("can-control-other-agents", false)
-
-  showControlLinks: ->
-    $(".control-link-region").show()
-    $(".control-link-region").data("can-control-other-agents", true)
-
-  hideEventCreation: ->
-    $(".event-related-region .select2-container").hide()
-    $(".event-related-region .cannot-create-events").show()
-    $(".event-related-region").data("can-create-events", false)
-
-  showEventCreation: ->
-    $(".event-related-region .select2-container").show()
-    $(".event-related-region .cannot-create-events").hide()
-    $(".event-related-region").data("can-create-events", true)
-
-  showEventDescriptions: ->
-    if $("#agent_source_ids").val()
-      $.getJSON "/agents/event_descriptions", { ids: $("#agent_source_ids").val().join(",") }, (json) =>
-        if json.description_html?
-          $(".event-descriptions").show().html(json.description_html)
-        else
-          $(".event-descriptions").hide()
-    else
-      $(".event-descriptions").html("").hide()
-
-  showCorrectRegionsOnStartup: ->
-    if $(".schedule-region")
-      if $(".schedule-region").data("can-be-scheduled") == true
-        @showSchedule()
-      else
-        @hideSchedule()
-
-    if $(".link-region")
-      if $(".link-region").data("can-receive-events") == true
-        @showLinks()
-      else
-        @hideLinks()
-
-    if $(".control-link-region")
-      if $(".control-link-region").data("can-control-other-agents") == true
-        @showControlLinks()
-      else
-        @hideControlLinks()
-
-    if $(".event-related-region")
-      if $(".event-related-region").data("can-create-events") == true
-        @showEventCreation()
-      else
-        @hideEventCreation()
-
-  buildAce: ->
-    $(".ace-editor").each ->
-      unless $(this).data('initialized')
-        $this = $(this)
-        $this.data('initialized', true)
-        $source = $($this.data('source')).hide()
-        editor = ace.edit(this)
-        $this.data('ace-editor', editor)
-        session = editor.getSession()
-        session.setTabSize(2)
-        session.setUseSoftTabs(true)
-        session.setUseWrapMode(false)
-
-        setSyntax = ->
-          if mode = $this.data('mode')
-            session.setMode("ace/mode/" + mode)
-
-          if theme = $this.data('theme')
-            editor.setTheme("ace/theme/" + theme);
-
-          if mode = $("[name='agent[options][language]']").val()
-            switch mode
-              when 'JavaScript' then session.setMode("ace/mode/javascript")
-              when 'CoffeeScript' then session.setMode("ace/mode/coffee")
-              else session.setMode("ace/mode/" + mode)
-
-        $("[name='agent[options][language]']").on 'change', setSyntax
-        setSyntax()
-
-        session.setValue($source.val())
-
-  updateFromEditors: ->
-    $(".ace-editor").each ->
-      $source = $($(this).data('source'))
-      $source.val($(this).data('ace-editor').getSession().getValue())
-
-  enableDryRunButton: ->
-    $(".agent-dry-run-button").prop('disabled', false).off().on "click", @invokeDryRun
-
-  disableDryRunButton: ->
-    $(".agent-dry-run-button").prop('disabled', true)
-
-  invokeDryRun: (e) =>
-    e.preventDefault()
-    @updateFromEditors()
-    Utils.handleDryRunButton(e.currentTarget)
-
-  formatAgentForSelect = (agent) ->
-    originalOption = agent.element
-    description = agent.element[0].title
-    '<strong>' + agent.text + '</strong><br/>' + description
-
-$ ->
-  Utils.registerPage(AgentEditPage, forPathsMatching: /^agents/)

app/assets/javascripts/pages/agent-show-page.js

@@ -0,0 +1,112 @@
+this.AgentShowPage = class AgentShowPage {
+  constructor() {
+    let tab;
+    $(".agent-show #show-tabs a[href='#logs'], #logs .refresh").on(
+      "click",
+      this.fetchLogs
+    );
+    $(".agent-show #logs .clear").on("click", this.clearLogs);
+    $(".agent-show #memory .clear").on("click", this.clearMemory);
+    $("#toggle-memory").on("click", this.toggleMemory);
+
+    // Trigger tabs when navigated to.
+    if (
+      (tab = __guard__(window.location.href.match(/tab=(\w+)\b/i), (x) => x[1]))
+    ) {
+      if (["details", "logs"].includes(tab)) {
+        $(`.agent-show .nav-pills li a[href='#${tab}']`).click();
+      }
+    }
+  }
+
+  fetchLogs(e) {
+    const agentId = $(e.target).closest("[data-agent-id]").data("agent-id");
+    e.preventDefault();
+    $("#logs .spinner").show();
+    $("#logs .refresh, #logs .clear").hide();
+    return $.get(`/agents/${agentId}/logs`, (html) => {
+      $("#logs .logs").html(html);
+      $("#logs .logs .show-log-details").each(function () {
+        const $button = $(this);
+        return $button.on("click", function (e) {
+          e.preventDefault();
+          return Utils.showDynamicModal("<pre></pre>", {
+            title: $button.data("modal-title"),
+            body(body) {
+              return $(body).find("pre").text($button.data("modal-content"));
+            },
+          });
+        });
+      });
+
+      return $("#logs .spinner")
+        .stop(true, true)
+        .fadeOut(() => $("#logs .refresh, #logs .clear").show());
+    });
+  }
+
+  clearLogs(e) {
+    if (confirm("Are you sure you want to clear all logs for this Agent?")) {
+      const agentId = $(e.target).closest("[data-agent-id]").data("agent-id");
+      e.preventDefault();
+      $("#logs .spinner").show();
+      $("#logs .refresh, #logs .clear").hide();
+      return $.post(
+        `/agents/${agentId}/logs/clear`,
+        { _method: "DELETE" },
+        (html) => {
+          $("#logs .logs").html(html);
+          $("#show-tabs li a.recent-errors").removeClass("recent-errors");
+          return $("#logs .spinner")
+            .stop(true, true)
+            .fadeOut(() => $("#logs .refresh, #logs .clear").show());
+        }
+      );
+    }
+  }
+
+  toggleMemory(e) {
+    e.preventDefault();
+    if ($("pre.memory").hasClass("hidden")) {
+      $("pre.memory").removeClass("hidden");
+      return $("#toggle-memory").text("Hide");
+    } else {
+      $("pre.memory").addClass("hidden");
+      return $("#toggle-memory").text("Show");
+    }
+  }
+
+  clearMemory(e) {
+    if (
+      confirm(
+        "Are you sure you want to completely clear the memory of this Agent?"
+      )
+    ) {
+      const agentId = $(e.target).closest("[data-agent-id]").data("agent-id");
+      e.preventDefault();
+      $("#memory .spinner").css({ display: "inline-block" });
+      $("#memory .clear").hide();
+      return $.post(`/agents/${agentId}/memory`, { _method: "DELETE" })
+        .done(() =>
+          $("#memory .spinner").fadeOut(() =>
+            $("#memory + .memory").text("{\n}\n")
+          )
+        )
+        .fail(() =>
+          $("#memory .spinner").fadeOut(() =>
+            $("#memory .clear").css({ display: "inline-block" })
+          )
+        );
+    }
+  }
+};
+
+$(() =>
+  Utils.registerPage(AgentShowPage, { forPathsMatching: /^agents\/\d+/ })
+);
+
+function __guard__(value, transform) {
+  return typeof value !== "undefined" && value !== null
+    ? transform(value)
+    : undefined;
+}

app/assets/javascripts/pages/agent-show-page.js.coffee

@@ -1,68 +0,0 @@
-class @AgentShowPage
-  constructor: ->
-    $(".agent-show #show-tabs a[href='#logs'], #logs .refresh").on "click", @fetchLogs
-    $(".agent-show #logs .clear").on "click", @clearLogs
-    $(".agent-show #memory .clear").on "click", @clearMemory
-    $('#toggle-memory').on "click", @toggleMemory
-
-    # Trigger tabs when navigated to.
-    if tab = window.location.href.match(/tab=(\w+)\b/i)?[1]
-      if tab in ["details", "logs"]
-        $(".agent-show .nav-pills li a[href='##{tab}']").click()
-
-  fetchLogs: (e) ->
-    agentId = $(e.target).closest("[data-agent-id]").data("agent-id")
-    e.preventDefault()
-    $("#logs .spinner").show()
-    $("#logs .refresh, #logs .clear").hide()
-    $.get "/agents/#{agentId}/logs", (html) =>
-      $("#logs .logs").html html
-      $("#logs .logs .show-log-details").each ->
-        $button = $(this)
-        $button.on 'click', (e) ->
-          e.preventDefault()
-          Utils.showDynamicModal '<pre></pre>',
-            title: $button.data('modal-title'),
-            body: (body) ->
-              $(body).find('pre').text $button.data('modal-content')
-
-      $("#logs .spinner").stop(true, true).fadeOut ->
-        $("#logs .refresh, #logs .clear").show()
-
-  clearLogs: (e) ->
-    if confirm("Are you sure you want to clear all logs for this Agent?")
-      agentId = $(e.target).closest("[data-agent-id]").data("agent-id")
-      e.preventDefault()
-      $("#logs .spinner").show()
-      $("#logs .refresh, #logs .clear").hide()
-      $.post "/agents/#{agentId}/logs/clear", { "_method": "DELETE" }, (html) =>
-        $("#logs .logs").html html
-        $("#show-tabs li a.recent-errors").removeClass 'recent-errors'
-        $("#logs .spinner").stop(true, true).fadeOut ->
-          $("#logs .refresh, #logs .clear").show()
-
-  toggleMemory: (e) ->
-    e.preventDefault()
-    if $('pre.memory').hasClass('hidden')
-      $('pre.memory').removeClass 'hidden'
-      $('#toggle-memory').text('Hide')
-    else
-      $('pre.memory').addClass 'hidden'
-      $('#toggle-memory').text('Show')
-
-  clearMemory: (e) ->
-    if confirm("Are you sure you want to completely clear the memory of this Agent?")
-      agentId = $(e.target).closest("[data-agent-id]").data("agent-id")
-      e.preventDefault()
-      $("#memory .spinner").css(display: 'inline-block')
-      $("#memory .clear").hide()
-      $.post "/agents/#{agentId}/memory", { "_method": "DELETE" }
-        .done ->
-          $("#memory .spinner").fadeOut ->
-            $("#memory + .memory").text "{\n}\n"
-        .fail ->
-          $("#memory .spinner").fadeOut ->
-            $("#memory .clear").css(display: 'inline-block')
-
-$ ->
-  Utils.registerPage(AgentShowPage, forPathsMatching: /^agents\/\d+/)

app/assets/javascripts/pages/scenario-form-page.js

@@ -0,0 +1,23 @@
+this.ScenarioFormPage = class ScenarioFormPage {
+  constructor() {
+    this.enabledSelect2();
+  }
+
+  format(icon) {
+    const originalOption = icon.element;
+    return (
+      '<i class="fa ' + $(originalOption).data("icon") + '"></i> ' + icon.text
+    );
+  }
+
+  enabledSelect2() {
+    return $(".select2-fountawesome-icon").select2({
+      width: "100%",
+      formatResult: this.format,
+    });
+  }
+};
+
+$(() =>
+  Utils.registerPage(ScenarioFormPage, { forPathsMatching: /^scenarios/ })
+);

app/assets/javascripts/pages/scenario-form-page.js.coffee

@@ -1,15 +0,0 @@
-class @ScenarioFormPage
-  constructor:() ->
-    @enabledSelect2()
-
-  format: (icon) ->
-    originalOption = icon.element
-    '<i class="fa ' + $(originalOption).data('icon') + '"></i> ' + icon.text
-
-  enabledSelect2: () ->
-    $('.select2-fountawesome-icon').select2
-      width: '100%'
-      formatResult: @format
-      
-$ ->
-  Utils.registerPage(ScenarioFormPage, forPathsMatching: /^scenarios/)

app/assets/javascripts/pages/scenario-show-page.js

@@ -0,0 +1,24 @@
+this.ScenarioShowPage = class ScenarioShowPage {
+  constructor() {
+    this.changeModalText();
+  }
+
+  changeModalText() {
+    $("#disable-all").click(function () {
+      $("#enable-disable-agents .modal-body").text(
+        "Would you like to disable all agents?"
+      );
+      return $("#scenario-disabled-value").val("true");
+    });
+    return $("#enable-all").click(function () {
+      $("#enable-disable-agents .modal-body").text(
+        "Would you like to enable all agents?"
+      );
+      return $("#scenario-disabled-value").val("false");
+    });
+  }
+};
+
+$(() =>
+  Utils.registerPage(ScenarioShowPage, { forPathsMatching: /^scenarios/ })
+);

app/assets/javascripts/pages/scenario-show-page.js.coffee

@@ -1,15 +0,0 @@
-class @ScenarioShowPage
-  constructor:() ->
-    @changeModalText()
-
-  changeModalText: () ->
-    $('#disable-all').click ->
-      $('#enable-disable-agents .modal-body').text 'Would you like to disable all agents?'
-      $('#scenario-disabled-value').val 'true'
-    $('#enable-all').click ->
-      $('#enable-disable-agents .modal-body').text 'Would you like to enable all agents?'
-      $('#scenario-disabled-value').val 'false'
-
-$ ->
-  Utils.registerPage(ScenarioShowPage, forPathsMatching: /^scenarios/)
-

app/assets/javascripts/pages/user-credential-page.js

@@ -0,0 +1,33 @@
+this.UserCredentialPage = class UserCredentialPage {
+  constructor() {
+    const editor = ace.edit("ace-credential-value");
+    editor.getSession().setTabSize(2);
+    editor.getSession().setUseSoftTabs(true);
+    editor.getSession().setUseWrapMode(false);
+
+    const setMode = function () {
+      const mode = $("#user_credential_mode").val();
+      if (mode === "java_script") {
+        return editor.getSession().setMode("ace/mode/javascript");
+      } else {
+        return editor.getSession().setMode("ace/mode/text");
+      }
+    };
+
+    setMode();
+    $("#user_credential_mode").on("change", setMode);
+
+    const $textarea = $("#user_credential_credential_value").hide();
+    editor.getSession().setValue($textarea.val());
+
+    $textarea
+      .closest("form")
+      .on("submit", () => $textarea.val(editor.getSession().getValue()));
+  }
+};
+
+$(() =>
+  Utils.registerPage(UserCredentialPage, {
+    forPathsMatching: /^user_credentials\/(\d+|new)/,
+  })
+);

app/assets/javascripts/pages/user-credential-page.js.coffee

@@ -1,25 +0,0 @@
-class @UserCredentialPage
-  constructor: ->
-    editor = ace.edit("ace-credential-value")
-    editor.getSession().setTabSize(2)
-    editor.getSession().setUseSoftTabs(true)
-    editor.getSession().setUseWrapMode(false)
-
-    setMode = ->
-      mode = $("#user_credential_mode").val()
-      if mode == 'java_script'
-        editor.getSession().setMode("ace/mode/javascript")
-      else
-        editor.getSession().setMode("ace/mode/text")
-
-    setMode()
-    $("#user_credential_mode").on 'change', setMode
-
-    $textarea = $('#user_credential_credential_value').hide()
-    editor.getSession().setValue($textarea.val())
-
-    $textarea.closest('form').on 'submit', ->
-      $textarea.val(editor.getSession().getValue())
-
-$ ->
-  Utils.registerPage(UserCredentialPage, forPathsMatching: /^user_credentials\/(\d+|new)/)

app/assets/javascripts/tweets.js

@@ -0,0 +1,15 @@
+$(() =>
+  $(".tweet-body").each(function () {
+    return $(this).click(function () {
+      $(this).off("click");
+      return twttr.widgets
+        .createTweet(
+          this.dataset.tweetId,
+          this
+          // conversation: 'none'
+          // cards: 'hidden'
+        )
+        .then((el) => (el.previousSibling.style.display = "none"));
+    });
+  })
+);

app/assets/javascripts/tweets.js.coffee

@@ -1,11 +0,0 @@
-$ ->
-  $('.tweet-body').each ->
-    $(this).click ->
-      $(this).off('click')
-      twttr.widgets.createTweet(
-        this.dataset.tweetId
-        this
-        # conversation: 'none'
-        # cards: 'hidden'
-      ).then (el) ->
-        el.previousSibling.style.display = 'none'

app/concerns/dry_runnable.rb

@@ -21,23 +21,25 @@ module DryRunnable
 
     begin
       raise "#{short_type} does not support dry-run" unless can_dry_run?
+
       readonly!
       @dry_run_started_at = Time.zone.now
       @dry_run_logger.info('Dry Run started')
       if event
         raise "This agent cannot receive an event!" unless can_receive_events?
+
         receive([event])
       else
         check
       end
       @dry_run_logger.info('Dry Run finished')
-    rescue => e
+    rescue StandardError => e
       @dry_run_logger.info('Dry Run failed')
       error "Exception during dry-run. #{e.message}: #{e.backtrace.join("\n")}"
     end
 
     @dry_run_results.update(
-      memory: memory,
+      memory:,
       log: log.string,
     )
   ensure
@@ -57,35 +59,41 @@ module DryRunnable
 
     def logger
       return super unless dry_run?
+
       @dry_run_logger
     end
 
-    def save(options = {})
+    def save(**options)
       return super unless dry_run?
+
       perform_validations(options)
     end
 
-    def save!(options = {})
+    def save!(**options)
       return super unless dry_run?
-      save(options) or raise_record_invalid
+
+      save(**options) or raise_record_invalid
     end
 
     def log(message, options = {})
       return super unless dry_run?
-      case options[:level] || 3
-      when 0..2
-        sev = Logger::DEBUG
-      when 3
-        sev = Logger::INFO
-      else
-        sev = Logger::ERROR
-      end
+
+      sev =
+        case options[:level] || 3
+        when 0..2
+          Logger::DEBUG
+        when 3
+          Logger::INFO
+        else
+          Logger::ERROR
+        end
 
       logger.log(sev, message)
     end
 
     def create_event(event)
       return super unless dry_run?
+
       if can_create_events?
         event = build_event(event)
         @dry_run_results[:events] << event.payload

app/concerns/email_concern.rb

@@ -8,12 +8,15 @@ module EmailConcern
   end
 
   def validate_email_options
-    errors.add(:base, "subject and expected_receive_period_in_days are required") unless options['subject'].present? && options['expected_receive_period_in_days'].present?
+    errors.add(
+      :base,
+      "subject and expected_receive_period_in_days are required"
+    ) unless options['subject'].present? && options['expected_receive_period_in_days'].present?
 
     if options['recipients'].present?
       emails = options['recipients']
       emails = [emails] if emails.is_a?(String)
-      unless emails.all? { |email| email =~ Devise.email_regexp || email =~ /\{/ }
+      unless emails.all? { |email| Devise.email_regexp === email || /\{/ === email }
         errors.add(:base, "'when provided, 'recipients' should be an email address or an array of email addresses")
       end
     end
@@ -40,16 +43,16 @@ module EmailConcern
     if payload.is_a?(Hash)
       payload = ActiveSupport::HashWithIndifferentAccess.new(payload)
       MAIN_KEYS.each do |key|
-        return { :title => payload[key].to_s, :entries => present_hash(payload, key) } if payload.has_key?(key)
+        return { title: payload[key].to_s, entries: present_hash(payload, key) } if payload.has_key?(key)
       end
 
-      { :title => "Event", :entries => present_hash(payload) }
+      { title: "Event", entries: present_hash(payload) }
     else
-      { :title => payload.to_s, :entries => [] }
+      { title: payload.to_s, entries: [] }
     end
   end
 
   def present_hash(hash, skip_key = nil)
-    hash.to_a.sort_by {|a| a.first.to_s }.map { |k, v| "#{k}: #{v}" unless k.to_s == skip_key.to_s }.compact
+    hash.to_a.sort_by { |a| a.first.to_s }.map { |k, v| "#{k}: #{v}" unless k.to_s == skip_key.to_s }.compact
   end
 end

app/concerns/event_headers_concern.rb

@@ -14,11 +14,11 @@ module EventHeadersConcern
   def event_headers_normalizer
     case interpolated['event_headers_style']
     when nil, '', 'capitalized'
-      ->name { name.gsub(/[^-]+/, &:capitalize) }
+      ->(name) { name.gsub(/[^-]+/, &:capitalize) }
     when 'downcased'
       :downcase.to_proc
-    when 'snakecased', nil
-      ->name { name.tr('A-Z-', 'a-z_') }
+    when 'snakecased'
+      ->(name) { name.tr('A-Z-', 'a-z_') }
     when 'raw'
       :itself.to_proc
     else

app/concerns/liquid_droppable.rb

@@ -1,11 +1,6 @@
 # frozen_string_literal: true
 
-# Include this mix-in to make a class droppable to Liquid, and adjust
-# its behavior in Liquid by implementing its dedicated Drop class
-# named with a "Drop" suffix.
 module LiquidDroppable
-  extend ActiveSupport::Concern
-
   class Drop < Liquid::Drop
     def initialize(object)
       @object = object
@@ -23,21 +18,9 @@ module LiquidDroppable
 
     def as_json
       return {} unless defined?(self.class::METHODS)
-      Hash[self.class::METHODS.map { |m| [m, send(m).as_json]}]
-    end
-  end
 
-  included do
-    const_set :Drop,
-              if Kernel.const_defined?(drop_name = "#{name}Drop")
-                Kernel.const_get(drop_name)
-              else
-                Kernel.const_set(drop_name, Class.new(Drop))
-              end
-  end
-
-  def to_liquid
-    self.class::Drop.new(self)
+      self.class::METHODS.to_h { |m| [m, send(m).as_json] }
+    end
   end
 
   class MatchDataDrop < Drop

app/concerns/twitter_concern.rb

@@ -85,19 +85,10 @@ module TwitterConcern
     text.gsub!(RE_HTML_ENTITIES, HTML_ENTITIES)
     expanded_text.gsub!(RE_HTML_ENTITIES, HTML_ENTITIES)
 
-    if attrs[:text]
-      {
-        **attrs,
-        text: text,
-        expanded_text: expanded_text,
-      }
-    else
-      {
-        **attrs,
-        full_text: text,
-        expanded_text: expanded_text,
-      }
-    end
+    attrs[:text] &&= text
+    attrs[:full_text] &&= text
+
+    attrs.update(expanded_text:)
   end
 
   module_function :format_tweet

app/concerns/web_request_concern.rb

@@ -15,11 +15,11 @@ module WebRequestConcern
   end
 
   class CharacterEncoding < Faraday::Middleware
-    def initialize(app, force_encoding: nil, default_encoding: nil, unzip: nil)
+    def initialize(app, options = {})
       super(app)
-      @force_encoding   = force_encoding
-      @default_encoding = default_encoding
-      @unzip            = unzip
+      @force_encoding   = options[:force_encoding]
+      @default_encoding = options[:default_encoding]
+      @unzip            = options[:unzip]
     end
 
     def call(env)
@@ -38,8 +38,12 @@ module WebRequestConcern
           # Not all Faraday adapters support automatic charset
           # detection, so we do that.
           case env[:response_headers][:content_type]
-          when /;\s*charset\s*=\s*([^()<>@,;:\\\"\/\[\]?={}\s]+)/i
-            encoding = Encoding.find($1) rescue @default_encoding
+          when /;\s*charset\s*=\s*([^()<>@,;:\\"\/\[\]?={}\s]+)/i
+            encoding = begin
+              Encoding.find($1)
+            rescue StandardError
+              @default_encoding
+            end
           when /\A\s*(?:text\/[^\s;]+|application\/(?:[^\s;]+\+)?(?:xml|json))\s*(?:;|\z)/i
             encoding = @default_encoding
           else
@@ -62,11 +66,11 @@ module WebRequestConcern
     if options['user_agent'].present?
       errors.add(:base, "user_agent must be a string") unless options['user_agent'].is_a?(String)
     end
-    
+
     if options['proxy'].present?
       errors.add(:base, "proxy must be a string") unless options['proxy'].is_a?(String)
     end
-    
+
     if options['disable_ssl_verification'].present? && boolify(options['disable_ssl_verification']).nil?
       errors.add(:base, "if provided, disable_ssl_verification must be true or false")
     end
@@ -112,13 +116,13 @@ module WebRequestConcern
     @faraday ||= Faraday.new(faraday_options) { |builder|
       builder.response :character_encoding,
                        force_encoding: interpolated['force_encoding'].presence,
-                       default_encoding: default_encoding,
+                       default_encoding:,
                        unzip: interpolated['unzip'].presence
 
       builder.headers = headers if headers.length > 0
 
       builder.headers[:user_agent] = user_agent
-      
+
       builder.proxy interpolated['proxy'].presence
 
       unless boolify(interpolated['disable_redirect_follow'])
@@ -140,8 +144,8 @@ module WebRequestConcern
       builder.use FaradayMiddleware::Gzip
 
       case backend = faraday_backend
-        when :typhoeus
-          require 'typhoeus/adapters/faraday'
+      when :typhoeus
+        require 'typhoeus/adapters/faraday'
       end
       builder.adapter backend
     }
@@ -153,12 +157,12 @@ module WebRequestConcern
 
   def basic_auth_credentials(value = interpolated['basic_auth'])
     case value
-      when nil, ''
-        return nil
-      when Array
-        return value if value.size == 2
-      when /:/
-        return value.split(/:/, 2)
+    when nil, ''
+      return nil
+    when Array
+      return value if value.size == 2
+    when /:/
+      return value.split(/:/, 2)
     end
     raise ArgumentError.new("bad value for basic_auth: #{value.inspect}")
   end

app/controllers/agents/dry_runs_controller.rb

@@ -7,7 +7,7 @@ module Agents
                   current_user.agents.find_by(id: params[:agent_id]).received_events.limit(5)
                 elsif params[:source_ids]
                   Event.where(agent_id: current_user.agents.where(id: params[:source_ids]).pluck(:id))
-                       .order("id DESC").limit(5)
+                    .order("id DESC").limit(5)
                 else
                   []
                 end
@@ -40,11 +40,14 @@ module Agents
 
         @results = agent.dry_run!(event)
       else
-        @results = { events: [], memory: [],
-                     log:  [
-                       "#{pluralize(agent.errors.count, "error")} prohibited this Agent from being saved:",
-                       *agent.errors.full_messages
-                     ].join("\n- ") }
+        @results = {
+          events: [],
+          memory: [],
+          log: [
+            "#{pluralize(agent.errors.count, "error")} prohibited this Agent from being saved:",
+            *agent.errors.full_messages
+          ].join("\n- ")
+        }
       end
 
       render layout: false

app/helpers/dot_helper.rb

@@ -1,6 +1,6 @@
 module DotHelper
   def render_agents_diagram(agents, layout: nil)
-    if svg = dot_to_svg(agents_dot(agents, rich: true, layout: layout))
+    if svg = dot_to_svg(agents_dot(agents, rich: true, layout:))
       decorate_svg(svg, agents).html_safe
     else
       # Google chart request url
@@ -141,7 +141,7 @@ module DotHelper
   end
 
   def agents_dot(agents, rich: false, layout: nil)
-    draw(agents: agents,
+    draw(agents:,
          agent_id: ->(agent) { 'a%d' % agent.id },
          agent_label: ->(agent) {
            agent.name.gsub(/(.{20}\S*)\s+/) {
@@ -150,7 +150,7 @@ module DotHelper
            }
          },
          agent_url: ->(agent) { agent_path(agent.id) },
-         rich: rich) {
+         rich:) {
       @disabled = '#999999'
 
       def agent_node(agent)
@@ -175,7 +175,7 @@ module DotHelper
       block('digraph', 'Agent Event Flow') {
         layout ||= ENV['DIAGRAM_DEFAULT_LAYOUT'].presence
         if rich && /\A[a-z]+\z/ === layout
-          statement 'graph', layout: layout, overlap: 'false'
+          statement 'graph', layout:, overlap: 'false'
         end
         statement 'node',
                   shape: 'box',
@@ -253,7 +253,7 @@ module DotHelper
           }
         }
       }
-      # See also: app/assets/diagram.js.coffee
+      # See also: app/assets/diagram.js
     }.at('div.agent-diagram').to_s
   end
 end

app/helpers/jobs_helper.rb

@@ -1,5 +1,4 @@
 module JobsHelper
-
   def status(job)
     case
     when job.failed_at
@@ -24,7 +23,7 @@ module JobsHelper
   #
   # Can return nil, or an instance of Agent.
   def agent_from_job(job)
-    data = YAML.load(job.handler.to_s).try(:job_data)
+    data = YAML.unsafe_load(job.handler.to_s).try(:job_data)
     case data['job_class']
     when 'AgentCheckJob', 'AgentReceiveJob'
       Agent.find_by_id(data['arguments'][0])

app/helpers/scenario_helper.rb

@@ -1,7 +1,6 @@
 module ScenarioHelper
-
   def style_colors(scenario)
-    colors = {
+    {
       color: scenario.tag_fg_color || default_scenario_fg_color,
       background_color: scenario.tag_bg_color || default_scenario_bg_color
     }.map { |key, value| "#{key.to_s.dasherize}:#{value}" }.join(';')
@@ -19,5 +18,4 @@ module ScenarioHelper
   def default_scenario_fg_color
     '#FFFFFF'
   end
-
 end

app/models/agent.rb

@@ -11,7 +11,6 @@ class Agent < ActiveRecord::Base
   include WorkingHelpers
   include LiquidInterpolatable
   include HasGuid
-  include LiquidDroppable
   include DryRunnable
   include SortableEvents
 
@@ -325,7 +324,7 @@ class Agent < ActiveRecord::Base
         # Give it a unique name
         2.step do |i|
           name = '%s (%d)' % [original.name, i]
-          unless exists?(name: name)
+          unless exists?(name:)
             clone.name = name
             break
           end
@@ -454,7 +453,7 @@ class Agent < ActiveRecord::Base
     def run_schedule(schedule)
       return if schedule == 'never'
 
-      types = where(schedule: schedule).group(:type).pluck(:type)
+      types = where(schedule:).group(:type).pluck(:type)
       types.each do |type|
         next unless valid_type?(type)
 
@@ -478,40 +477,47 @@ class Agent < ActiveRecord::Base
       AgentCheckJob.perform_later(agent_id)
     end
   end
-end
 
-class AgentDrop
-  def type
-    @object.short_type
-  end
-
-  METHODS = %i[
-    id
-    name
-    type
-    options
-    memory
-    sources
-    receivers
-    schedule
-    controllers
-    control_targets
-    disabled
-    keep_events_for
-    propagate_immediately
-  ]
+  public def to_liquid
+    Drop.new(self)
+  end
 
-  METHODS.each { |attr|
-    define_method(attr) {
-      @object.__send__(attr)
-    } unless method_defined?(attr)
-  }
+  class Drop < LiquidDroppable::Drop
+    def type
+      @object.short_type
+    end
 
-  def working
-    @object.working?
-  end
+    METHODS = %i[
+      id
+      name
+      type
+      options
+      memory
+      sources
+      receivers
+      schedule
+      controllers
+      control_targets
+      disabled
+      keep_events_for
+      propagate_immediately
+    ]
+
+    METHODS.each { |attr|
+      define_method(attr) {
+        @object.__send__(attr)
+      } unless method_defined?(attr)
+    }
+
+    def working
+      @object.working?
+    end
 
-  def url
-    Rails.application.routes.url_helpers.agent_url(@object, Rails.application.config.action_mailer.default_url_options)
+    def url
+      Rails.application.routes.url_helpers.agent_url(
+        @object,
+        Rails.application.config.action_mailer.default_url_options
+      )
+    end
   end
 end

app/models/agent_log.rb

@@ -3,11 +3,11 @@
 # Agents' `last_error_log_at` column.  These are often used to determine if an Agent is `working?`.
 class AgentLog < ActiveRecord::Base
   belongs_to :agent
-  belongs_to :inbound_event, :class_name => "Event", optional: true
-  belongs_to :outbound_event, :class_name => "Event", optional: true
+  belongs_to :inbound_event, class_name: "Event", optional: true
+  belongs_to :outbound_event, class_name: "Event", optional: true
 
   validates_presence_of :message
-  validates_numericality_of :level, :only_integer => true, :greater_than_or_equal_to => 0, :less_than => 5
+  validates_numericality_of :level, only_integer: true, greater_than_or_equal_to: 0, less_than: 5
 
   before_validation :scrub_message
   before_save :truncate_message
@@ -15,7 +15,7 @@ class AgentLog < ActiveRecord::Base
   def self.log_for_agent(agent, message, options = {})
     puts "Agent##{agent.id}: #{message}" unless Rails.env.test?
 
-    log = agent.logs.create! options.merge(:message => message)
+    log = agent.logs.create! options.merge(message:)
     if agent.logs.count > log_length
       oldest_id_to_keep = agent.logs.limit(1).offset(log_length - 1).pluck("agent_logs.id")
       agent.logs.where("agent_logs.id < ?", oldest_id_to_keep).delete_all
@@ -35,7 +35,7 @@ class AgentLog < ActiveRecord::Base
   def scrub_message
     if message_changed? && !message.nil?
       self.message = message.inspect unless message.is_a?(String)
-      self.message.scrub!{ |bytes| "<#{bytes.unpack('H*')[0]}>" }
+      self.message.scrub! { |bytes| "<#{bytes.unpack1('H*')}>" }
     end
     true
   end

app/models/agents/adioso_agent.rb

@@ -2,26 +2,25 @@ module Agents
   class AdiosoAgent < Agent
     cannot_receive_events!
 
-  	default_schedule "every_1d"
+    default_schedule "every_1d"
 
-    description <<-MD
-  		The Adioso Agent will tell you the minimum airline prices between a pair of cities, and within a certain period of time.
+    description <<~MD
+      The Adioso Agent will tell you the minimum airline prices between a pair of cities, and within a certain period of time.
 
-      The currency is USD. Please make sure that the difference between `start_date` and `end_date` is less than 150 days. You will need to contact [Adioso](http://adioso.com/)
-  		for a `username` and `password`.
+      The currency is USD. Please make sure that the difference between `start_date` and `end_date` is less than 150 days. You will need to contact [Adioso](http://adioso.com/) for a `username` and `password`.
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       If flights are present then events look like:
 
           {
             "cost": 75.23,
             "date": "June 25, 2013",
-  			    "route": "New York to Chicago"
+            "route": "New York to Chicago"
           }
 
       otherwise
-    
+
           {
             "nonetodest": "No flights found to the specified destination"
           }
@@ -30,12 +29,12 @@ module Agents
     def default_options
       {
         'start_date' => Date.today.httpdate[0..15],
-        'end_date'   => Date.today.plus_with_duration(100).httpdate[0..15],
-        'from'       => "New York",
-        'to'         => "Chicago",
-        'username'   => "xx",
-        'password'   => "xx",
-				'expected_update_period_in_days' => "1"
+        'end_date' => Date.today.plus_with_duration(100).httpdate[0..15],
+        'from' => "New York",
+        'to' => "Chicago",
+        'username' => "xx",
+        'password' => "xx",
+        'expected_update_period_in_days' => "1"
       }
     end
 
@@ -44,10 +43,12 @@ module Agents
     end
 
     def validate_options
-			unless %w[start_date end_date from to username password expected_update_period_in_days].all? { |field| options[field].present? }
-				errors.add(:base, "All fields are required")
-			end
-		end
+      unless %w[
+        start_date end_date from to username password expected_update_period_in_days
+      ].all? { |field| options[field].present? }
+        errors.add(:base, "All fields are required")
+      end
+    end
 
     def date_to_unix_epoch(date)
       date.to_time.to_i
@@ -60,19 +61,24 @@ module Agents
           password: interpolated[:password]
         }
       }
-      parse_response = HTTParty.get "http://api.adioso.com/v2/search/parse?#{{ q: "#{interpolated[:from]} to #{interpolated[:to]}" }.to_query}", auth_options
-      fare_request = parse_response["search_url"].gsub /(end=)(\d*)([^\d]*)(\d*)/, "\\1#{date_to_unix_epoch(interpolated['end_date'])}\\3#{date_to_unix_epoch(interpolated['start_date'])}"
+      parse_response = HTTParty.get(
+        "http://api.adioso.com/v2/search/parse?#{{ q: "#{interpolated[:from]} to #{interpolated[:to]}" }.to_query}",
+        auth_options
+      )
+      fare_request = parse_response["search_url"].gsub(
+        /(end=)(\d*)([^\d]*)(\d*)/,
+        "\\1#{date_to_unix_epoch(interpolated['end_date'])}\\3#{date_to_unix_epoch(interpolated['start_date'])}"
+      )
       fare = HTTParty.get fare_request, auth_options
 
-			if fare["warnings"]
-				create_event :payload => fare["warnings"]
-			else
-				event = fare["results"].min {|a,b| a["cost"] <=> b["cost"]}
-				event["date"]  = Time.at(event["date"]).to_date.httpdate[0..15]
-				event["route"] = "#{interpolated['from']} to #{interpolated['to']}"
-				create_event :payload => event
-			end
+      if fare["warnings"]
+        create_event payload: fare["warnings"]
+      else
+        event = fare["results"].min_by { |x| x["cost"] }
+        event["date"]  = Time.at(event["date"]).to_date.httpdate[0..15]
+        event["route"] = "#{interpolated['from']} to #{interpolated['to']}"
+        create_event payload: event
+      end
     end
   end
 end
-

app/models/agents/aftership_agent.rb

@@ -2,21 +2,20 @@ require 'uri'
 
 module Agents
   class AftershipAgent < Agent
-
     cannot_receive_events!
 
     default_schedule "every_10m"
 
-    description <<-MD
+    description <<~MD
       The Aftership agent allows you to track your shipment from aftership and emit them into events.
 
       To be able to use the Aftership API, you need to generate an `API Key`. You need a paying plan to use their tracking feature.
 
       You can use this agent to retrieve tracking data.
- 
-      Provide the `path` for the API endpoint that you'd like to hit. For example, for all active packages, enter `trackings` 
-      (see https://www.aftership.com/docs/api/4/trackings), for a specific package, use `trackings/SLUG/TRACKING_NUMBER` 
-      and replace `SLUG` with a courier code and `TRACKING_NUMBER` with the tracking number. You can request last checkpoint of a package 
+
+      Provide the `path` for the API endpoint that you'd like to hit. For example, for all active packages, enter `trackings`
+      (see https://www.aftership.com/docs/api/4/trackings), for a specific package, use `trackings/SLUG/TRACKING_NUMBER`
+      and replace `SLUG` with a courier code and `TRACKING_NUMBER` with the tracking number. You can request last checkpoint of a package
       by providing `last_checkpoint/SLUG/TRACKING_NUMBER` instead.
 
       You can get a list of courier information here `https://www.aftership.com/courier`
@@ -27,7 +26,7 @@ module Agents
       * `path request and its full path`
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       A typical tracking event have 2 important objects (tracking, and checkpoint) and the tracking/checkpoint looks like this.
 
           "trackings": [
@@ -87,8 +86,9 @@ module Agents
     MD
 
     def default_options
-      { 'api_key' => 'YOUR_API_KEY',
-        'path' => 'trackings'
+      {
+        'api_key' => 'YOUR_API_KEY',
+        'path' => 'trackings',
       }
     end
 
@@ -104,10 +104,11 @@ module Agents
     def check
       response = HTTParty.get(event_url, request_options)
       events = JSON.parse response.body
-      create_event :payload => events
+      create_event payload: events
     end
 
-  private
+    private
+
     def base_url
       "https://api.aftership.com/v4/"
     end
@@ -117,7 +118,12 @@ module Agents
     end
 
     def request_options
-      {:headers => {"aftership-api-key" => interpolated['api_key'], "Content-Type"=>"application/json"} }
+      {
+        headers: {
+          "aftership-api-key" => interpolated['api_key'],
+          "Content-Type" => "application/json",
+        }
+      }
     end
   end
 end

app/models/agents/attribute_difference_agent.rb

@@ -2,7 +2,7 @@ module Agents
   class AttributeDifferenceAgent < Agent
     cannot_be_scheduled!
 
-    description <<-MD
+    description <<~MD
       The Attribute Difference Agent receives events and emits a new event with
       the difference or change of a specific attribute in comparison to the previous
       event received.
@@ -30,7 +30,7 @@ module Agents
       All configuration options will be liquid interpolated based on the incoming event.
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       This will change based on the source event.
     MD
 
@@ -80,23 +80,26 @@ module Agents
         payload[opts['output']] = difference
       end
 
-      created_event = create_event(payload: payload)
+      created_event = create_event(payload:)
       log('Propagating new event', outbound_event: created_event, inbound_event: event)
       update_memory(attribute_value)
     end
 
     def calculate_integer_difference(new_value)
       return 0 if last_value.nil?
+
       (new_value.to_i - last_value.to_i)
     end
 
     def calculate_decimal_difference(new_value, dec_pre)
       return 0.0 if last_value.nil?
+
       (new_value.to_f - last_value.to_f).round(dec_pre.to_i)
     end
 
     def calculate_percentage_change(new_value, dec_pre)
       return 0.0 if last_value.nil?
+
       (((new_value.to_f / last_value.to_f) * 100) - 100).round(dec_pre.to_i)
     end
 

app/models/agents/change_detector_agent.rb

@@ -2,7 +2,7 @@ module Agents
   class ChangeDetectorAgent < Agent
     cannot_be_scheduled!
 
-    description <<-MD
+    description <<~MD
       The Change Detector Agent receives a stream of events and emits a new event when a property of the received event changes.
 
       `property` specifies a [Liquid](https://github.com/huginn/huginn/wiki/Formatting-Events-using-Liquid) template that expands to the property to be watched, where you can use a variable `last_property` for the last property value.  If you want to detect a new lowest price, try this: `{% assign drop = last_property | minus: price %}{% if last_property == blank or drop > 0 %}{{ price | default: last_property }}{% else %}{{ last_property }}{% endif %}`
@@ -12,22 +12,22 @@ module Agents
       The resulting event will be a copy of the received event.
     MD
 
-    event_description <<-MD
-    This will change based on the source event. If you were event from the ShellCommandAgent, your outbound event might look like:
+    event_description <<~MD
+      This will change based on the source event. If you were event from the ShellCommandAgent, your outbound event might look like:
 
-      {
-        'command' => 'pwd',
-        'path' => '/home/Huginn',
-        'exit_status' => '0',
-        'errors' => '',
-        'output' => '/home/Huginn'
-      }
+          {
+            'command' => 'pwd',
+            'path' => '/home/Huginn',
+            'exit_status' => '0',
+            'errors' => '',
+            'output' => '/home/Huginn'
+          }
     MD
 
     def default_options
       {
-          'property' => '{{output}}',
-          'expected_update_period_in_days' => 1
+        'property' => '{{output}}',
+        'expected_update_period_in_days' => 1
       }
     end
 
@@ -55,12 +55,13 @@ module Agents
     def handle(opts, event = nil)
       property = opts['property']
       if has_changed?(property)
-        created_event = create_event :payload => event.payload
+        created_event = create_event payload: event.payload
 
-        log("Propagating new event as property has changed to #{property} from #{last_property}", :outbound_event => created_event, :inbound_event => event )
+        log("Propagating new event as property has changed to #{property} from #{last_property}",
+            outbound_event: created_event, inbound_event: event)
         update_memory(property)
       else
-        log("Not propagating as incoming event has not changed from #{last_property}.", :inbound_event => event )
+        log("Not propagating as incoming event has not changed from #{last_property}.", inbound_event: event)
       end
     end
 

app/models/agents/commander_agent.rb

@@ -4,7 +4,7 @@ module Agents
 
     cannot_create_events!
 
-    description <<-MD
+    description <<~MD
       The Commander Agent is triggered by schedule or an incoming event, and commands other agents ("targets") to run, disable, configure, or enable themselves.
 
       # Action types
@@ -28,7 +28,7 @@ module Agents
 
       - If you want to update a WeatherAgent based on a UserLocationAgent, you could use `'action': 'configure'` and set 'configure_options' to `{ 'location': '{{_location_.latlng}}' }`.
 
-      - In templating, you can use the variable `target` to refer to each target agent, which has the following attributes: #{AgentDrop.instance_methods(false).map { |m| "`#{m}`" }.to_sentence}.
+      - In templating, you can use the variable `target` to refer to each target agent, which has the following attributes: #{Agent::Drop.instance_methods(false).map { |m| "`#{m}`" }.to_sentence}.
 
       # Targets
 

app/models/agents/csv_agent.rb

@@ -19,7 +19,7 @@ module Agents
     end
 
     description do
-      <<-MD
+      <<~MD
         The `CsvAgent` parses or serializes CSV data. When parsing, events can either be emitted for the entire CSV, or one per row.
 
         Set `mode` to `parse` to parse CSV from incoming event, when set to `serialize` the agent serilizes the data of events to CSV.
@@ -53,28 +53,44 @@ module Agents
     end
 
     event_description do
-      "Events will looks like this:\n\n    %s" % if interpolated['mode'] == 'parse'
-        rows = if boolify(interpolated['with_header'])
-          [{'column' => 'row1 value1', 'column2' => 'row1 value2'}, {'column' => 'row2 value3', 'column2' => 'row2 value4'}]
-        else
-          [['row1 value1', 'row1 value2'], ['row2 value1', 'row2 value2']]
-        end
-        if interpolated['output'] == 'event_per_row'
-          Utils.pretty_print(interpolated['data_key'] => rows[0])
+      data =
+        if interpolated['mode'] == 'parse'
+          rows =
+            if boolify(interpolated['with_header'])
+              [
+                { 'column' => 'row1 value1', 'column2' => 'row1 value2' },
+                { 'column' => 'row2 value3', 'column2' => 'row2 value4' },
+              ]
+            else
+              [
+                ['row1 value1', 'row1 value2'],
+                ['row2 value1', 'row2 value2'],
+              ]
+            end
+          if interpolated['output'] == 'event_per_row'
+            rows[0]
+          else
+            rows
+          end
         else
-          Utils.pretty_print(interpolated['data_key'] => rows)
+          <<~EOS
+            "generated","csv","data"
+            "column1","column2","column3"
+          EOS
         end
-      else
-        Utils.pretty_print(interpolated['data_key'] => '"generated","csv","data"' + "\n" + '"column1","column2","column3"')
-      end
+
+      "Events will looks like this:\n\n    " +
+        Utils.pretty_print({
+          interpolated['data_key'] => data
+        })
     end
 
-    form_configurable :mode, type: :array, values: %w(parse serialize)
+    form_configurable :mode, type: :array, values: %w[parse serialize]
     form_configurable :separator, type: :string
     form_configurable :data_key, type: :string
     form_configurable :with_header, type: :boolean
     form_configurable :use_fields, type: :string
-    form_configurable :output, type: :array, values: %w(event_per_row event_per_file)
+    form_configurable :output, type: :array, values: %w[event_per_row event_per_file]
     form_configurable :data_path, type: :string
 
     def validate_options
@@ -100,27 +116,28 @@ module Agents
     end
 
     private
+
     def serialize(incoming_events)
       mo = interpolated(incoming_events.first)
       rows = rows_from_events(incoming_events, mo)
-      csv = CSV.generate(col_sep: separator(mo), force_quotes: true ) do |csv|
+      csv = CSV.generate(col_sep: separator(mo), force_quotes: true) do |csv|
         if boolify(mo['with_header']) && rows.first.is_a?(Hash)
-          if mo['use_fields'].present?
-            csv << extract_options(mo)
-          else
-            csv << rows.first.keys
-          end
+          csv << if mo['use_fields'].present?
+                   extract_options(mo)
+                 else
+                   rows.first.keys
+                 end
         end
         rows.each do |data|
-          if data.is_a?(Hash)
-            if mo['use_fields'].present?
-              csv << data.extract!(*extract_options(mo)).values
-            else
-              csv << data.values
-            end
-          else
-            csv << data
-          end
+          csv << if data.is_a?(Hash)
+                   if mo['use_fields'].present?
+                     data.extract!(*extract_options(mo)).values
+                   else
+                     data.values
+                   end
+                 else
+                   data
+                 end
         end
       end
       create_event payload: { mo['data_key'] => csv }
@@ -143,6 +160,7 @@ module Agents
       incoming_events.each do |event|
         mo = interpolated(event)
         next unless io = local_get_io(event)
+
         if mo['output'] == 'event_per_row'
           parse_csv(io, mo) do |payload|
             create_event payload: { mo['data_key'] => payload }

app/models/agents/data_output_agent.rb

@@ -5,13 +5,13 @@ module Agents
     cannot_be_scheduled!
     cannot_create_events!
 
-    description  do
-      <<-MD
+    description do
+      <<~MD
         The Data Output Agent outputs received events as either RSS or JSON.  Use it to output a public or private stream of Huginn data.
 
         This Agent will output data at:
 
-        `https://#{ENV['DOMAIN']}#{Rails.application.routes.url_helpers.web_requests_path(agent_id: ':id', user_id: user_id, secret: ':secret', format: :xml)}`
+        `https://#{ENV['DOMAIN']}#{Rails.application.routes.url_helpers.web_requests_path(agent_id: ':id', user_id:, secret: ':secret', format: :xml)}`
 
         where `:secret` is one of the allowed secrets specified in your options and the extension can be `xml` or `json`.
 
@@ -104,7 +104,8 @@ module Agents
       end
 
       unless options['expected_receive_period_in_days'].present? && options['expected_receive_period_in_days'].to_i > 0
-        errors.add(:base, "Please provide 'expected_receive_period_in_days' to indicate how many days can pass before this Agent is considered to be not working")
+        errors.add(:base,
+                   "Please provide 'expected_receive_period_in_days' to indicate how many days can pass before this Agent is considered to be not working")
       end
 
       unless options['template'].present? && options['template']['item'].present? && options['template']['item'].is_a?(Hash)
@@ -153,11 +154,12 @@ module Agents
 
     def feed_url(options = {})
       interpolated['template']['self'].presence ||
-        feed_link + Rails.application.routes.url_helpers.
-                    web_requests_path(agent_id: id || ':id',
-                                      user_id: user_id,
-                                      secret: options[:secret],
-                                      format: options[:format])
+        feed_link + Rails.application.routes.url_helpers.web_requests_path(
+          agent_id: id || ':id',
+          user_id:,
+          secret: options[:secret],
+          format: options[:format]
+        )
     end
 
     def feed_icon
@@ -165,9 +167,9 @@ module Agents
     end
 
     def itunes_icon
-      if(boolify(interpolated['ns_itunes']))
+      if boolify(interpolated['ns_itunes'])
         "<itunes:image href=#{feed_icon.encode(xml: :attr)} />"
-      end  
+      end
     end
 
     def feed_description
@@ -181,13 +183,13 @@ module Agents
     def xml_namespace
       namespaces = ['xmlns:atom="http://www.w3.org/2005/Atom"']
 
-      if (boolify(interpolated['ns_dc']))
+      if boolify(interpolated['ns_dc'])
         namespaces << 'xmlns:dc="http://purl.org/dc/elements/1.1/"'
       end
-      if (boolify(interpolated['ns_media']))
+      if boolify(interpolated['ns_media'])
         namespaces << 'xmlns:media="http://search.yahoo.com/mrss/"'
       end
-      if (boolify(interpolated['ns_itunes']))
+      if boolify(interpolated['ns_itunes'])
         namespaces << 'xmlns:itunes="http://www.itunes.com/dtds/podcast-1.0.dtd"'
       end
       namespaces.join(' ')
@@ -211,8 +213,8 @@ module Agents
 
       events =
         if (event_ids = memory[:event_ids]) &&
-           memory[:events_order] == events_order &&
-           memory[:events_to_show] >= events_to_show
+            memory[:events_order] == events_order &&
+            memory[:events_to_show] >= events_to_show
           received_events.where(id: event_ids).to_a
         else
           memory[:last_event_id] = nil
@@ -231,8 +233,8 @@ module Agents
             source_ids.flat_map { |source_id|
               # dig twice as many events as the number of
               # `events_to_show`
-              received_events.where(agent_id: source_id).
-                last(2 * events_to_show)
+              received_events.where(agent_id: source_id)
+                .last(2 * events_to_show)
             }.sort_by(&:id)
           end
 
@@ -260,18 +262,20 @@ module Agents
         end
       end
 
-      source_events = sort_events(latest_events(), 'events_list_order')
+      source_events = sort_events(latest_events, 'events_list_order')
 
       interpolate_with('events' => source_events) do
         items = source_events.map do |event|
           interpolated = interpolate_options(options['template']['item'], event)
-          interpolated['guid'] = {'_attributes' => {'isPermaLink' => 'false'},
-                                  '_contents' => interpolated['guid'].presence || event.id}
+          interpolated['guid'] = {
+            '_attributes' => { 'isPermaLink' => 'false' },
+            '_contents' => interpolated['guid'].presence || event.id
+          }
           date_string = interpolated['pubDate'].to_s
           date =
             begin
-              Time.zone.parse(date_string)  # may return nil
-            rescue => e
+              Time.zone.parse(date_string) # may return nil
+            rescue StandardError => e
               error "Error parsing a \"pubDate\" value \"#{date_string}\": #{e.message}"
               nil
             end || event.created_at
@@ -299,23 +303,23 @@ module Agents
 
           items = items_to_xml(items)
 
-          return [<<-XML, 200, rss_content_type, interpolated['response_headers'].presence]
-<?xml version="1.0" encoding="UTF-8" ?>
-<rss version="2.0" #{xml_namespace}>
-<channel>
- <atom:link href=#{feed_url(secret: params['secret'], format: :xml).encode(xml: :attr)} rel="self" type="application/rss+xml" />
- <atom:icon>#{feed_icon.encode(xml: :text)}</atom:icon>
- #{itunes_icon}
-#{hub_links}
- <title>#{feed_title.encode(xml: :text)}</title>
- <description>#{feed_description.encode(xml: :text)}</description>
- <link>#{feed_link.encode(xml: :text)}</link>
- <lastBuildDate>#{now.rfc2822.to_s.encode(xml: :text)}</lastBuildDate>
- <pubDate>#{now.rfc2822.to_s.encode(xml: :text)}</pubDate>
- <ttl>#{feed_ttl}</ttl>
-#{items}
-</channel>
-</rss>
+          return [<<~XML, 200, rss_content_type, interpolated['response_headers'].presence]
+            <?xml version="1.0" encoding="UTF-8" ?>
+            <rss version="2.0" #{xml_namespace}>
+            <channel>
+             <atom:link href=#{feed_url(secret: params['secret'], format: :xml).encode(xml: :attr)} rel="self" type="application/rss+xml" />
+             <atom:icon>#{feed_icon.encode(xml: :text)}</atom:icon>
+             #{itunes_icon}
+            #{hub_links}
+             <title>#{feed_title.encode(xml: :text)}</title>
+             <description>#{feed_description.encode(xml: :text)}</description>
+             <link>#{feed_link.encode(xml: :text)}</link>
+             <lastBuildDate>#{now.rfc2822.to_s.encode(xml: :text)}</lastBuildDate>
+             <pubDate>#{now.rfc2822.to_s.encode(xml: :text)}</pubDate>
+             <ttl>#{feed_ttl}</ttl>
+            #{items}
+            </channel>
+            </rss>
           XML
         end
       end
@@ -336,13 +340,17 @@ module Agents
 
     class XMLNode
       def initialize(tag_name, attributes, contents)
-        @tag_name, @attributes, @contents = tag_name, attributes, contents
+        @tag_name = tag_name
+        @attributes = attributes
+        @contents = contents
       end
 
       def to_xml(options)
         if @contents.is_a?(Hash)
           options[:builder].tag! @tag_name, @attributes do
-            @contents.each { |key, value| ActiveSupport::XmlMini.to_tag(key, value, options.merge(skip_instruct: true)) }
+            @contents.each { |key, value|
+              ActiveSupport::XmlMini.to_tag(key, value, options.merge(skip_instruct: true))
+            }
           end
         else
           options[:builder].tag! @tag_name, @attributes, @contents
@@ -353,15 +361,16 @@ module Agents
     def simplify_item_for_xml(item)
       if item.is_a?(Hash)
         item.each.with_object({}) do |(key, value), memo|
-          if value.is_a?(Hash)
-            if value.key?('_attributes') || value.key?('_contents')
-              memo[key] = XMLNode.new(key, value['_attributes'], simplify_item_for_xml(value['_contents']))
+          memo[key] =
+            if value.is_a?(Hash)
+              if value.key?('_attributes') || value.key?('_contents')
+                XMLNode.new(key, value['_attributes'], simplify_item_for_xml(value['_contents']))
+              else
+                simplify_item_for_xml(value)
+              end
             else
-              memo[key] = simplify_item_for_xml(value)
+              value
             end
-          else
-            memo[key] = value
-          end
         end
       elsif item.is_a?(Array)
         item.map { |value| simplify_item_for_xml(value) }
@@ -375,13 +384,14 @@ module Agents
         item.each.with_object({}) do |(key, value), memo|
           if value.is_a?(Hash)
             if value.key?('_attributes') || value.key?('_contents')
-              contents = if value['_contents'] && value['_contents'].is_a?(Hash)
-                           simplify_item_for_json(value['_contents'])
-                         elsif value['_contents']
-                           { "contents" => value['_contents'] }
-                         else
-                           {}
-                         end
+              contents =
+                if value['_contents'] && value['_contents'].is_a?(Hash)
+                  simplify_item_for_json(value['_contents'])
+                elsif value['_contents']
+                  { "contents" => value['_contents'] }
+                else
+                  {}
+                end
 
               memo[key] = contents.merge(value['_attributes'] || {})
             else
@@ -436,8 +446,8 @@ module Agents
           'hub.mode' => 'publish',
           'hub.url' => url
         }
-     rescue => e
-       error "Push failed: #{e.message}"
+      rescue StandardError => e
+        error "Push failed: #{e.message}"
       end
     end
   end

app/models/agents/de_duplication_agent.rb

@@ -3,7 +3,7 @@ module Agents
     include FormConfigurable
     cannot_be_scheduled!
 
-    description <<-MD
+    description <<~MD
       The De-duplication Agent receives a stream of events and remits the event if it is not a duplicate.
 
       `property` the value that should be used to determine the uniqueness of the event (empty to use the whole payload)
@@ -13,7 +13,7 @@ module Agents
       `expected_update_period_in_days` is used to determine if the Agent is working.
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       The DeDuplicationAgent just reemits events it received.
     MD
 
@@ -56,18 +56,22 @@ module Agents
     def handle(opts, event = nil)
       property = get_hash(options['property'].blank? ? JSON.dump(event.payload) : opts['property'])
       if is_unique?(property)
-        created_event = create_event :payload => event.payload
+        outbound_event = create_event payload: event.payload
 
-        log("Propagating new event as '#{property}' is a new unique property.", :inbound_event => event )
+        log(
+          "Propagating new event as '#{property}' is a new unique property.",
+          inbound_event: event,
+          outbound_event:
+        )
         update_memory(property, opts['lookback'].to_i)
       else
-        log("Not propagating as incoming event is a duplicate.", :inbound_event => event )
+        log("Not propagating as incoming event is a duplicate.", inbound_event: event)
       end
     end
 
     def get_hash(property)
       if property.to_s.length > 10
-        Zlib::crc32(property).to_s
+        Zlib.crc32(property).to_s
       else
         property
       end

app/models/agents/delay_agent.rb

@@ -4,7 +4,7 @@ module Agents
 
     default_schedule 'every_12h'
 
-    description <<-MD
+    description <<~MD
       The DelayAgent stores received Events and emits copies of them on a schedule. Use this as a buffer or queue of Events.
 
       `max_events` should be set to the maximum number of events that you'd like to hold in the buffer. When this number is
@@ -33,7 +33,8 @@ module Agents
 
     def validate_options
       unless options['expected_receive_period_in_days'].present? && options['expected_receive_period_in_days'].to_i > 0
-        errors.add(:base, "Please provide 'expected_receive_period_in_days' to indicate how many days can pass before this Agent is considered to be not working")
+        errors.add(:base,
+                   "Please provide 'expected_receive_period_in_days' to indicate how many days can pass before this Agent is considered to be not working")
       end
 
       unless options['keep'].present? && options['keep'].in?(%w[newest oldest])

app/models/agents/digest_agent.rb

@@ -4,7 +4,7 @@ module Agents
 
     default_schedule "6am"
 
-    description <<-MD
+    description <<~MD
       The Digest Agent collects any Events sent to it and emits them as a single event.
 
       The resulting Event will have a payload message of `message`. You can use liquid templating in the `message`, have a look at the [Wiki](https://github.com/huginn/huginn/wiki/Formatting-Events-using-Liquid) for details.
@@ -16,7 +16,7 @@ module Agents
       For instance, say `retained_events` is set to 3 and the Agent has received Events `5`, `4`, and `3`. When a digest is sent, Events `5`, `4`, and `3` are retained for a future digest. After Event `6` is received, the next digest will contain Events `6`, `5`, and `4`.
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       Events look like this:
 
           {
@@ -27,9 +27,9 @@ module Agents
 
     def default_options
       {
-          "expected_receive_period_in_days" => "2",
-          "message" => "{{ events | map: 'message' | join: ',' }}",
-          "retained_events" => "0"
+        "expected_receive_period_in_days" => "2",
+        "message" => "{{ events | map: 'message' | join: ',' }}",
+        "retained_events" => "0"
       }
     end
 
@@ -38,7 +38,8 @@ module Agents
     form_configurable :retained_events
 
     def validate_options
-      errors.add(:base, 'retained_events must be 0 to 999') unless options['retained_events'].to_i >= 0 && options['retained_events'].to_i < 1000
+      errors.add(:base,
+                 'retained_events must be 0 to 999') unless options['retained_events'].to_i >= 0 && options['retained_events'].to_i < 1000
     end
 
     def working?
@@ -60,7 +61,7 @@ module Agents
         events = received_events.where(id: self.memory["queue"]).order(id: :asc).to_a
         payload = { "events" => events.map { |event| event.payload } }
         payload["message"] = interpolated(payload)["message"]
-        create_event :payload => payload
+        create_event(payload:)
         if interpolated["retained_events"].to_i == 0
           self.memory["queue"] = []
         end

app/models/agents/dropbox_file_url_agent.rb

@@ -6,7 +6,7 @@ module Agents
     no_bulk_receive!
     can_dry_run!
 
-    description <<-MD
+    description <<~MD
       The _DropboxFileUrlAgent_ is used to work with Dropbox. It takes a file path (or multiple files paths) and emits events with either [temporary links](https://www.dropbox.com/developers/core/docs#media) or [permanent links](https://www.dropbox.com/developers/core/docs#shares).
 
       #{'## Include the `dropbox-api` and `omniauth-dropbox` gems in your `Gemfile` and set `DROPBOX_OAUTH_KEY` and `DROPBOX_OAUTH_SECRET` in your environment to use Dropbox Agents.' if dependencies_missing?}
@@ -34,39 +34,42 @@ module Agents
     MD
 
     event_description do
-      "Events will looks like this:\n\n    %s" % if options['link_type'] == 'permanent'
-        Utils.pretty_print({
-          url: "https://www.dropbox.com/s/abcde3/example?dl=1",
-          :".tag" => "file",
-          id: "id:abcde3",
-          name: "hi",
-          path_lower: "/huginn/hi",
-          link_permissions:          {
-            resolved_visibility: {:".tag"=>"public"},
-            requested_visibility: {:".tag"=>"public"},
-            can_revoke: true
-          },
-          client_modified: "2017-10-14T18:38:39Z",
-          server_modified: "2017-10-14T18:38:45Z",
-          rev: "31db0615354b",
-          size: 0
-        })
-      else
-        Utils.pretty_print({
-          url: "https://dl.dropboxusercontent.com/apitl/1/somelongurl",
-          metadata: {
-            name: "hi",
-            path_lower: "/huginn/hi",
-            path_display: "/huginn/hi",
-            id: "id:abcde3",
-            client_modified: "2017-10-14T18:38:39Z",
-            server_modified: "2017-10-14T18:38:45Z",
-            rev: "31db0615354b",
-            size: 0,
-            content_hash: "e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855"
-          }
-        })
-      end
+      "Events will looks like this:\n\n    " +
+        Utils.pretty_print(
+          if options['link_type'] == 'permanent'
+            {
+              url: "https://www.dropbox.com/s/abcde3/example?dl=1",
+              ".tag": "file",
+              id: "id:abcde3",
+              name: "hi",
+              path_lower: "/huginn/hi",
+              link_permissions: {
+                resolved_visibility: { ".tag": "public" },
+                requested_visibility: { ".tag": "public" },
+                can_revoke: true
+              },
+              client_modified: "2017-10-14T18:38:39Z",
+              server_modified: "2017-10-14T18:38:45Z",
+              rev: "31db0615354b",
+              size: 0
+            }
+          else
+            {
+              url: "https://dl.dropboxusercontent.com/apitl/1/somelongurl",
+              metadata: {
+                name: "hi",
+                path_lower: "/huginn/hi",
+                path_display: "/huginn/hi",
+                id: "id:abcde3",
+                client_modified: "2017-10-14T18:38:39Z",
+                server_modified: "2017-10-14T18:38:45Z",
+                rev: "31db0615354b",
+                size: 0,
+                content_hash: "e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855"
+              }
+            }
+          end
+        )
     end
 
     def default_options
@@ -96,10 +99,8 @@ module Agents
 
     def permanent_url_for(path)
       dropbox.find(path).share_url.response.tap do |response|
-        response['url'].gsub!('?dl=0','?dl=1')
+        response['url'].gsub!('?dl=0', '?dl=1')
       end
     end
-
   end
-
 end

app/models/agents/dropbox_watch_agent.rb

@@ -5,13 +5,13 @@ module Agents
     cannot_receive_events!
     default_schedule "every_1m"
 
-    description <<-MD
+    description <<~MD
       The Dropbox Watch Agent watches the given `dir_to_watch` and emits events with the detected changes.
-      
+
       #{'## Include the `dropbox-api` and `omniauth-dropbox` gems in your `Gemfile` and set `DROPBOX_OAUTH_KEY` and `DROPBOX_OAUTH_SECRET` in your environment to use Dropbox Agents.' if dependencies_missing?}
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       The event payload will contain the following fields:
 
           {
@@ -34,7 +34,8 @@ module Agents
 
     def validate_options
       errors.add(:base, 'The `dir_to_watch` property is required.') unless options['dir_to_watch'].present?
-      errors.add(:base, 'Invalid `expected_update_period_in_days` format.') unless options['expected_update_period_in_days'].present? && is_positive_integer?(options['expected_update_period_in_days'])
+      errors.add(:base,
+                 'Invalid `expected_update_period_in_days` format.') unless options['expected_update_period_in_days'].present? && is_positive_integer?(options['expected_update_period_in_days'])
     end
 
     def working?
@@ -53,8 +54,8 @@ module Agents
 
     def ls(dir_to_watch)
       dropbox.ls(dir_to_watch)
-             .select { |entry| entry.respond_to?(:rev) }
-             .map { |file| { 'path' => file.path, 'rev' => file.rev, 'modified' => file.server_modified } }
+        .select { |entry| entry.respond_to?(:rev) }
+        .map { |file| { 'path' => file.path, 'rev' => file.rev, 'modified' => file.server_modified } }
     end
 
     def previous_contents
@@ -69,7 +70,8 @@ module Agents
 
     class DropboxDirDiff
       def initialize(previous, current)
-        @previous, @current = [previous || [], current || []]
+        @previous = previous || []
+        @current = current || []
       end
 
       def empty?
@@ -101,7 +103,5 @@ module Agents
         array.find { |entry| entry['path'] == path }
       end
     end
-
   end
-
 end

app/models/agents/email_agent.rb

@@ -9,7 +9,7 @@ module Agents
     cannot_create_events!
     no_bulk_receive!
 
-    description <<-MD
+    description <<~MD
       The Email Agent sends any events it receives via email immediately.
 
       You can specify the email's subject line by providing a `subject` option, which can contain [Liquid](https://github.com/huginn/huginn/wiki/Formatting-Events-using-Liquid) formatting.  E.g.,
@@ -35,9 +35,9 @@ module Agents
 
     def default_options
       {
-          'subject' => "You have a notification!",
-          'headline' => "Your notification:",
-          'expected_receive_period_in_days' => "2"
+        'subject' => "You have a notification!",
+        'headline' => "Your notification:",
+        'expected_receive_period_in_days' => "2"
       }
     end
 
@@ -48,21 +48,19 @@ module Agents
     def receive(incoming_events)
       incoming_events.each do |event|
         recipients(event.payload).each do |recipient|
-          begin
-            SystemMailer.send_message(
-              to: recipient,
-              from: interpolated(event)['from'],
-              subject: interpolated(event)['subject'],
-              headline: interpolated(event)['headline'],
-              body: interpolated(event)['body'],
-              content_type: interpolated(event)['content_type'],
-              groups: [present(event.payload)]
-            ).deliver_now
-            log "Sent mail to #{recipient} with event #{event.id}"
-          rescue => e
-            error("Error sending mail to #{recipient} with event #{event.id}: #{e.message}")
-            raise
-          end
+          SystemMailer.send_message(
+            to: recipient,
+            from: interpolated(event)['from'],
+            subject: interpolated(event)['subject'],
+            headline: interpolated(event)['headline'],
+            body: interpolated(event)['body'],
+            content_type: interpolated(event)['content_type'],
+            groups: [present(event.payload)]
+          ).deliver_now
+          log "Sent mail to #{recipient} with event #{event.id}"
+        rescue StandardError => e
+          error("Error sending mail to #{recipient} with event #{event.id}: #{e.message}")
+          raise
         end
       end
     end

app/models/agents/email_digest_agent.rb

@@ -8,7 +8,7 @@ module Agents
 
     cannot_create_events!
 
-    description <<-MD
+    description <<~MD
       The Email Digest Agent collects any Events sent to it and sends them all via email when scheduled. The number of
       used events also relies on the `Keep events` option of the emitting Agent, meaning that if events expire before
       this agent is scheduled to run, they will not appear in the email.
@@ -30,9 +30,9 @@ module Agents
 
     def default_options
       {
-          'subject' => "You have some notifications!",
-          'headline' => "Your notifications:",
-          'expected_receive_period_in_days' => "2"
+        'subject' => "You have some notifications!",
+        'headline' => "Your notifications:",
+        'expected_receive_period_in_days' => "2"
       }
     end
 
@@ -52,21 +52,19 @@ module Agents
         payloads = received_events.reorder("events.id ASC").where(id: self.memory['events']).pluck(:payload).to_a
         groups = payloads.map { |payload| present(payload) }
         recipients.each do |recipient|
-          begin
-            SystemMailer.send_message(
-              to: recipient,
-              from: interpolated['from'],
-              subject: interpolated['subject'],
-              headline: interpolated['headline'],
-              content_type: interpolated['content_type'],
-              groups: groups
-            ).deliver_now
+          SystemMailer.send_message(
+            to: recipient,
+            from: interpolated['from'],
+            subject: interpolated['subject'],
+            headline: interpolated['headline'],
+            content_type: interpolated['content_type'],
+            groups:
+          ).deliver_now
 
-            log "Sent digest mail to #{recipient}"
-          rescue => e
-            error("Error sending digest mail to #{recipient}: #{e.message}")
-            raise
-          end
+          log "Sent digest mail to #{recipient}"
+        rescue StandardError => e
+          error("Error sending digest mail to #{recipient}: #{e.message}")
+          raise
         end
         self.memory['events'] = []
       end

app/models/agents/event_formatting_agent.rb

@@ -3,7 +3,7 @@ module Agents
     cannot_be_scheduled!
     can_dry_run!
 
-    description <<-MD
+    description <<~MD
       The Event Formatting Agent allows you to format incoming Events, adding new fields as needed.
 
       For example, here is a possible Event:
@@ -34,7 +34,7 @@ module Agents
 
       The special key `created_at` refers to the timestamp of the Event, which can be reformatted by the `date` filter, like `{{created_at | date:"at %I:%M %p" }}`.
 
-      The upstream agent of each received event is accessible via the key `agent`, which has the following attributes: #{''.tap { |s| s << AgentDrop.instance_methods(false).map { |m| "`#{m}`" }.join(', ') }}.
+      The upstream agent of each received event is accessible via the key `agent`, which has the following attributes: #{''.tap { |s| s << Agent::Drop.instance_methods(false).map { |m| "`#{m}`" }.join(', ') }}.
 
       Have a look at the [Wiki](https://github.com/huginn/huginn/wiki/Formatting-Events-using-Liquid) to learn more about liquid templating.
 
@@ -96,9 +96,10 @@ module Agents
     end
 
     def validate_options
-      errors.add(:base, "instructions and mode need to be present.") unless options['instructions'].present? && options['mode'].present?
+      errors.add(:base,
+                 "instructions and mode need to be present.") unless options['instructions'].present? && options['mode'].present?
 
-      if options['mode'].present? && !options['mode'].to_s.include?('{{') && !%[clean merge].include?(options['mode'].to_s)
+      if options['mode'].present? && !options['mode'].to_s.include?('{{') && !%(clean merge).include?(options['mode'].to_s)
         errors.add(:base, "mode must be 'clean' or 'merge'")
       end
 
@@ -108,7 +109,7 @@ module Agents
     def default_options
       {
         'instructions' => {
-          'message' =>  "You received a text {{text}} from {{fields.from}}",
+          'message' => "You received a text {{text}} from {{fields.from}}",
           'agent' => "{{agent.type}}",
           'some_other_field' => "Looks like the weather is going to be {{fields.weather}}"
         },
@@ -156,7 +157,7 @@ module Agents
         if regexp.present?
           begin
             Regexp.new(regexp)
-          rescue
+          rescue StandardError
             errors.add(:base, "bad regexp found in matchers: #{regexp}")
           end
         else

app/models/agents/evernote_agent.rb

@@ -2,7 +2,7 @@ module Agents
   class EvernoteAgent < Agent
     include EvernoteConcern
 
-    description <<-MD
+    description <<~MD
       The Evernote Agent connects with a user's Evernote note store.
 
       Visit [Evernote](https://dev.evernote.com/doc/) to set up an Evernote app and receive an api key and secret.
@@ -53,7 +53,7 @@ module Agents
                 }
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       When `mode` is `update`, events look like:
 
           {
@@ -106,7 +106,7 @@ module Agents
     end
 
     def validate_options
-      errors.add(:base, "mode must be 'update' or 'read'") unless %w(read update).include?(options[:mode])
+      errors.add(:base, "mode must be 'update' or 'read'") unless %w[read update].include?(options[:mode])
 
       if options[:mode] == "update" && schedule != "never"
         errors.add(:base, "when mode is set to 'update', schedule must be 'never'")
@@ -116,7 +116,8 @@ module Agents
         errors.add(:base, "when mode is set to 'read', agent must have a schedule")
       end
 
-      errors.add(:base, "expected_update_period_in_days is required") unless options['expected_update_period_in_days'].present?
+      errors.add(:base,
+                 "expected_update_period_in_days is required") unless options['expected_update_period_in_days'].present?
 
       if options[:mode] == "update" && options[:note].values.all?(&:empty?)
         errors.add(:base, "you must specify at least one note parameter to create or update a note")
@@ -131,7 +132,7 @@ module Agents
       if options[:mode] == "update"
         incoming_events.each do |event|
           note = note_store.create_or_update_note(note_params(event))
-          create_event :payload => note.attr(include_content: include_xhtml_content?)
+          create_event payload: note.attr(include_content: include_xhtml_content?)
         end
       end
     end
@@ -146,15 +147,17 @@ module Agents
         opts.merge!(last_checked_at: (memory[:last_checked_at] ||= created_at.to_i * 1000))
 
         if opts[:tagNames]
-          opts.merge!(notes_with_tags: (memory[:notes_with_tags] ||=
-            NoteStore::Search.new(note_store, {tagNames: opts[:tagNames]}).note_guids))
+          notes_with_tags =
+            memory[:notes_with_tags] ||=
+              NoteStore::Search.new(note_store, { tagNames: opts[:tagNames] }).note_guids
+          opts.merge!(notes_with_tags:)
         end
 
         notes = NoteStore::Search.new(note_store, opts).notes
         notes.each do |note|
           memory[:notes_with_tags] << note.guid unless memory[:notes_with_tags].include?(note.guid)
 
-          create_event :payload => note.attr(include_resources: true, include_content: include_xhtml_content?)
+          create_event payload: note.attr(include_resources: true, include_content: include_xhtml_content?)
         end
 
         memory[:last_checked_at] = Time.now.to_i * 1000
@@ -185,19 +188,20 @@ module Agents
     # https://dev.evernote.com/doc/reference/
     class NoteStore
       attr_reader :en_note_store
+
       delegate :createNote, :updateNote, :getNote, :listNotebooks, :listTags, :getNotebook,
-               :createNotebook, :findNotesMetadata, :getNoteTagNames, :to => :en_note_store
+               :createNotebook, :findNotesMetadata, :getNoteTagNames, to: :en_note_store
 
       def initialize(en_note_store)
         @en_note_store = en_note_store
       end
 
       def create_or_update_note(params)
-        search = Search.new(self, {title: params[:title], notebook: params[:notebook]})
+        search = Search.new(self, { title: params[:title], notebook: params[:notebook] })
 
         # evernote search can only filter notes with titles containing a substring;
         # this finds a note with the exact title
-        note = search.notes.detect {|note| note.title == params[:title]}
+        note = search.notes.detect { |note| note.title == params[:title] }
 
         if note
           # a note with specified title and notebook exists, so update it
@@ -227,7 +231,8 @@ module Agents
         # evernote will create any new tags
         tags = getNoteTagNames(params[:guid])
         tags.each { |tag|
-          params[:tagNames] << tag unless params[:tagNames].include?(tag) }
+          params[:tagNames] << tag unless params[:tagNames].include?(tag)
+        }
 
         note = Evernote::EDAM::Type::Note.new(params)
         updateNote(note)
@@ -247,19 +252,19 @@ module Agents
       end
 
       def find_tags(guids)
-        listTags.select {|tag| guids.include?(tag.guid)}
+        listTags.select { |tag| guids.include?(tag.guid) }
       end
 
       def find_notebook(params)
         if params[:guid]
-          listNotebooks.detect {|notebook| notebook.guid == params[:guid]}
+          listNotebooks.detect { |notebook| notebook.guid == params[:guid] }
         elsif params[:name]
-          listNotebooks.detect {|notebook| notebook.name == params[:name]}
+          listNotebooks.detect { |notebook| notebook.name == params[:name] }
         end
       end
 
       def create_notebook(name)
-        notebook = Evernote::EDAM::Type::Notebook.new(name: name)
+        notebook = Evernote::EDAM::Type::Notebook.new(name:)
         createNotebook(notebook)
       end
 
@@ -270,7 +275,7 @@ module Agents
           params[:content] =
             "<?xml version=\"1.0\" encoding=\"UTF-8\"?>" \
             "<!DOCTYPE en-note SYSTEM \"http://xml.evernote.com/pub/enml2.dtd\">" \
-            "<en-note>#{params[:content].encode(:xml => :text)}</en-note>"
+            "<en-note>#{params[:content].encode(xml: :text)}</en-note>"
         end
 
         params
@@ -278,6 +283,7 @@ module Agents
 
       class Search
         attr_reader :note_store, :opts
+
         def initialize(note_store, opts)
           @note_store = note_store
           @opts = opts
@@ -297,7 +303,7 @@ module Agents
             # and notes that recently had the specified tags added
             metadata.select! do |note_data|
               note_data.updated > opts[:last_checked_at] ||
-              !opts[:notes_with_tags].include?(note_data.guid)
+                !opts[:notes_with_tags].include?(note_data.guid)
             end
 
           elsif opts[:last_checked_at]
@@ -326,7 +332,8 @@ module Agents
         private
 
         def filtered_metadata
-          filter, spec = create_filter, create_spec
+          filter = create_filter
+          spec = create_spec
           metadata = note_store.findNotesMetadata(filter, 0, 100, spec).notes
         end
 
@@ -346,8 +353,9 @@ module Agents
     class Note
       attr_accessor :en_note
       attr_reader :notebook, :tags
+
       delegate :guid, :notebookGuid, :title, :tagGuids, :content, :resources,
-               :attributes, :to => :en_note
+               :attributes, to: :en_note
 
       def initialize(en_note, notebook, tags)
         @en_note = en_note
@@ -357,11 +365,11 @@ module Agents
 
       def attr(opts = {})
         return_attr = {
-          title:        title,
-          notebook:     notebook,
-          tags:         tags,
-          source:       attributes.source,
-          source_url:   attributes.sourceURL
+          title:,
+          notebook:,
+          tags:,
+          source: attributes.source,
+          source_url: attributes.sourceURL
         }
 
         return_attr[:content] = content if opts[:include_content]

app/models/agents/ftpsite_agent.rb

@@ -11,7 +11,7 @@ module Agents
     emits_file_pointer!
 
     description do
-      <<-MD
+      <<~MD
         The Ftp Site Agent checks an FTP site and creates Events based on newly uploaded files in a directory. When receiving events it creates files on the configured FTP server.
 
         #{'## Include `net-ftp-list` in your Gemfile to use this Agent!' if dependencies_missing?}
@@ -40,7 +40,7 @@ module Agents
       MD
     end
 
-    event_description <<-MD
+    event_description <<~MD
       Events look like this:
 
           {
@@ -83,7 +83,7 @@ module Agents
           URI::FTP === uri or raise
           errors.add(:base, "url must end with a slash") if uri.path.present? && !uri.path.end_with?('/')
         end
-      rescue
+      rescue StandardError
         errors.add(:base, "url must be a valid FTP URL")
       end
 
@@ -118,18 +118,20 @@ module Agents
       if (timestamp = options['timestamp']).present?
         begin
           Time.parse(timestamp)
-        rescue
+        rescue StandardError
           errors.add(:base, "timestamp cannot be parsed as time")
         end
       end
 
       if options['expected_update_period_in_days'].present?
-        errors.add(:base, "Invalid expected_update_period_in_days format") unless is_positive_integer?(options['expected_update_period_in_days'])
+        errors.add(:base,
+                   "Invalid expected_update_period_in_days format") unless is_positive_integer?(options['expected_update_period_in_days'])
       end
     end
 
     def check
       return if interpolated['mode'] != 'read'
+
       saving_entries do |found|
         each_entry { |filename, mtime|
           found[filename, mtime]
@@ -139,9 +141,14 @@ module Agents
 
     def receive(incoming_events)
       return if interpolated['mode'] != 'write'
+
       incoming_events.each do |event|
         mo = interpolated(event)
-        mo['data'].encode!(interpolated['force_encoding'], invalid: :replace, undef: :replace) if interpolated['force_encoding'].present?
+        mo['data'].encode!(
+          interpolated['force_encoding'],
+          invalid: :replace,
+          undef: :replace
+        ) if interpolated['force_encoding'].present?
         open_ftp(base_uri) do |ftp|
           ftp.storbinary("STOR #{mo['filename']}", StringIO.new(mo['data']), Net::FTP::DEFAULT_BLOCKSIZE)
         end
@@ -168,7 +175,7 @@ module Agents
           entry = Net::FTP::List.parse line
           filename = entry.basename
           mtime = Time.parse(entry.mtime.to_s).utc
-          
+
           patterns.any? { |pattern|
             File.fnmatch?(pattern, filename)
           } or next

app/models/agents/gap_detector_agent.rb

@@ -2,7 +2,7 @@ module Agents
   class GapDetectorAgent < Agent
     default_schedule "every_10m"
 
-    description <<-MD
+    description <<~MD
       The Gap Detector Agent will watch for holes or gaps in a stream of incoming Events and generate "no data alerts".
 
       The `value_path` value is a [JSONPath](http://goessner.net/articles/JsonPath/) to a value of interest. If either
@@ -10,7 +10,7 @@ module Agents
       a payload of `message`.
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       Events look like:
 
           {
@@ -58,8 +58,10 @@ module Agents
       if memory['newest_event_created_at'].present? && Time.at(memory['newest_event_created_at']) < window
         unless memory['alerted_at']
           memory['alerted_at'] = Time.now.to_i
-          create_event payload: { message: interpolated['message'],
-                                  gap_started_at: memory['newest_event_created_at'] }
+          create_event payload: {
+            message: interpolated['message'],
+            gap_started_at: memory['newest_event_created_at']
+          }
         end
       end
     end

app/models/agents/google_calendar_publish_agent.rb

@@ -8,7 +8,7 @@ module Agents
 
     gem_dependency_check { defined?(Google) && defined?(Google::Apis::CalendarV3) }
 
-    description <<-MD
+    description <<~MD
       The Google Calendar Publish Agent creates events on your Google Calendar.
 
       #{'## Include `google-api-client` in your Gemfile to use this Agent!' if dependencies_missing?}
@@ -73,19 +73,22 @@ module Agents
       }</code></pre>
     MD
 
-    event_description <<-MD
-      {
-        'success' => true,
-        'published_calendar_event' => {
-           ....
-        },
-        'agent_id' => 1234,
-        'event_id' => 3432
-      }
+    event_description <<~MD
+      Events look like:
+
+          {
+            'success' => true,
+            'published_calendar_event' => {
+               ....
+            },
+            'agent_id' => 1234,
+            'event_id' => 3432
+          }
     MD
 
     def validate_options
-      errors.add(:base, "expected_update_period_in_days is required") unless options['expected_update_period_in_days'].present?
+      errors.add(:base,
+                 "expected_update_period_in_days is required") unless options['expected_update_period_in_days'].present?
     end
 
     def working?
@@ -108,7 +111,6 @@ module Agents
       require 'google_calendar'
       incoming_events.each do |event|
         GoogleCalendar.open(interpolate_options(options, event), Rails.logger) do |calendar|
-
           cal_message = event.payload["message"]
           if cal_message["start"].present? && cal_message["start"]["dateTime"].present? && !cal_message["start"]["date_time"].present?
             cal_message["start"]["date_time"] = cal_message["start"].delete "dateTime"
@@ -118,11 +120,11 @@ module Agents
           end
 
           calendar_event = calendar.publish_as(
-                interpolated(event)['calendar_id'],
-                cal_message
-              )
+            interpolated(event)['calendar_id'],
+            cal_message
+          )
 
-          create_event :payload => {
+          create_event payload: {
             'success' => true,
             'published_calendar_event' => calendar_event,
             'agent_id' => event.agent_id,
@@ -133,4 +135,3 @@ module Agents
     end
   end
 end
-

app/models/agents/google_translation_agent.rb

@@ -4,7 +4,7 @@ module Agents
 
     gem_dependency_check { defined?(Google) && defined?(Google::Cloud::Translate) }
 
-    description <<-MD
+    description <<~MD
       The Translation Agent will attempt to translate text between natural languages.
 
       #{'## Include `google-api-client` in your Gemfile to use this Agent!' if dependencies_missing?}
@@ -76,7 +76,7 @@ module Agents
     end
 
     def translate_service
-      @translate_service ||= google_client.discovered_api('translate','v2')
+      @translate_service ||= google_client.discovered_api('translate', 'v2')
     end
 
     def cloud_translate_service

app/models/agents/growl_agent.rb

@@ -1,93 +0,0 @@
-module Agents
-  class GrowlAgent < Agent
-    include FormConfigurable
-    attr_reader :growler
-
-    cannot_be_scheduled!
-    cannot_create_events!
-    can_dry_run!
-
-    gem_dependency_check { defined?(Growl) }
-
-    description <<-MD
-      The Growl Agent sends any events it receives to a Growl GNTP server immediately.
-
-      #{'## Include `ruby-growl` in your Gemfile to use this Agent!' if dependencies_missing?}
-
-      The option `message`, which will hold the body of the growl notification, and the `subject` option,
-      which will have the headline of the Growl notification are required. All other options are optional.
-      When `callback_url` is set to a URL clicking on the notification will open the link in your default browser.
-
-      Set `expected_receive_period_in_days` to the maximum amount of time that you'd expect to pass between
-      Events being received by this Agent.
-
-      Have a look at the [Wiki](https://github.com/huginn/huginn/wiki/Formatting-Events-using-Liquid) to learn
-      more about liquid templating.
-    MD
-
-    def default_options
-      {
-          'growl_server' => 'localhost',
-          'growl_password' => '',
-          'growl_app_name' => 'HuginnGrowl',
-          'growl_notification_name' => 'Notification',
-          'expected_receive_period_in_days' => "2",
-          'subject' => '{{subject}}',
-          'message' => '{{message}}',
-          'sticky' => 'false',
-          'priority' => '0'
-      }
-    end
-
-    form_configurable :growl_server
-    form_configurable :growl_password
-    form_configurable :growl_app_name
-    form_configurable :growl_notification_name
-    form_configurable :expected_receive_period_in_days
-    form_configurable :subject
-    form_configurable :message, type: :text
-    form_configurable :sticky, type: :boolean
-    form_configurable :priority
-    form_configurable :callback_url
-
-    def working?
-      last_receive_at && last_receive_at > options['expected_receive_period_in_days'].to_i.days.ago && !recent_error_logs?
-    end
-
-    def validate_options
-      unless options['growl_server'].present? && options['expected_receive_period_in_days'].present?
-        errors.add(:base, "growl_server and expected_receive_period_in_days are required fields")
-      end
-    end
-
-    def register_growl
-      @growler = Growl::GNTP.new(interpolated['growl_server'], interpolated['growl_app_name'])
-      @growler.password = interpolated['growl_password']
-      @growler.add_notification(interpolated['growl_notification_name'])
-    end
-
-    def notify_growl(subject:, message:, priority:, sticky:, callback_url:)
-      @growler.notify(interpolated['growl_notification_name'], subject, message, priority, sticky, nil, callback_url)
-    end
-
-    def receive(incoming_events)
-      incoming_events.each do |event|
-        interpolate_with(event) do
-          register_growl
-          message = interpolated[:message]
-          subject = interpolated[:subject]
-          if message.present? && subject.present?
-            log "Sending Growl notification '#{subject}': '#{message}' to #{interpolated(event)['growl_server']} with event #{event.id}"
-            notify_growl(subject: subject,
-                         message: message,
-                         priority: interpolated[:priority].to_i,
-                         sticky: boolify(interpolated[:sticky]) || false,
-                         callback_url: interpolated[:callback_url].presence)
-          else
-            log "Event #{event.id} not sent, message and subject expected"
-          end
-        end
-      end
-    end
-  end
-end

app/models/agents/hipchat_agent.rb

@@ -8,7 +8,7 @@ module Agents
 
     gem_dependency_check { defined?(HipChat) }
 
-    description <<-MD
+    description <<~MD
       The Hipchat Agent sends messages to a Hipchat Room
 
       #{'## Include `hipchat` in your Gemfile to use this Agent!' if dependencies_missing?}
@@ -52,16 +52,18 @@ module Agents
       client.rooms
       true
     rescue HipChat::UnknownResponseCode
-      return false
+      false
     end
 
     def complete_room_name
-      client.rooms.collect { |room| {text: room.name, id: room.name} }
+      client.rooms.collect { |room| { text: room.name, id: room.name } }
     end
 
     def validate_options
-      errors.add(:base, "you need to specify a hipchat auth_token or provide a credential named hipchat_auth_token") unless options['auth_token'].present? || credential('hipchat_auth_token').present?
-      errors.add(:base, "you need to specify a room_name or a room_name_path") if options['room_name'].blank? && options['room_name_path'].blank?
+      errors.add(:base,
+                 "you need to specify a hipchat auth_token or provide a credential named hipchat_auth_token") unless options['auth_token'].present? || credential('hipchat_auth_token').present?
+      errors.add(:base,
+                 "you need to specify a room_name or a room_name_path") if options['room_name'].blank? && options['room_name_path'].blank?
     end
 
     def working?
@@ -71,15 +73,18 @@ module Agents
     def receive(incoming_events)
       incoming_events.each do |event|
         mo = interpolated(event)
-        client[mo[:room_name]].send(mo[:username][0..14], mo[:message],
-                                      notify: boolify(mo[:notify]),
-                                      color: mo[:color],
-                                      message_format: mo[:format].presence || 'html'
-                                    )
+        client[mo[:room_name]].send(
+          mo[:username][0..14],
+          mo[:message],
+          notify: boolify(mo[:notify]),
+          color: mo[:color],
+          message_format: mo[:format].presence || 'html'
+        )
       end
     end
 
     private
+
     def client
       @client ||= HipChat::Client.new(interpolated[:auth_token].presence || credential('hipchat_auth_token'))
     end

app/models/agents/http_status_agent.rb

@@ -1,9 +1,7 @@
 require 'time_tracker'
 
 module Agents
-
   class HttpStatusAgent < Agent
-
     include WebRequestConcern
     include FormConfigurable
 
@@ -17,7 +15,7 @@ module Agents
     form_configurable :changes_only, type: :boolean
     form_configurable :headers_to_save
 
-    description <<-MD
+    description <<~MD
       The HttpStatusAgent will check a url and emit the resulting HTTP status code with the time that it waited for a reply. Additionally, it will optionally emit the value of one or more specified headers.
 
       Specify a `Url` and the Http Status Agent will produce an event with the HTTP status code. If you specify one or more `Headers to save` (comma-delimited) as well, that header or headers' value(s) will be included in the event.
@@ -27,7 +25,7 @@ module Agents
       The `changes only` option causes the Agent to report an event only when the status changes. If set to false, an event will be created for every check.  If set to true, an event will only be created when the status changes (like if your site goes from 200 to 500).
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       Events will have the following fields:
 
           {
@@ -86,27 +84,28 @@ module Agents
       # Deal with failures
       if measured_result.result
         final_url = boolify(interpolated['disable_redirect_follow']) ? url : measured_result.result.env.url.to_s
-        payload.merge!({ 'final_url' => final_url, 'redirected' => (url != final_url), 'response_received' => true, 'status' => current_status })
+        payload.merge!({ 'final_url' => final_url, 'redirected' => (url != final_url), 'response_received' => true,
+                         'status' => current_status })
         # Deal with headers
         if local_headers.present?
-          header_results = local_headers.each_with_object({}) { |header, hash| hash[header] = measured_result.result.headers[header] }
+          header_results = local_headers.each_with_object({}) { |header, hash|
+            hash[header] = measured_result.result.headers[header]
+          }
           payload.merge!({ 'headers' => header_results })
         end
-        create_event payload: payload
+        create_event(payload:)
         memory['last_status'] = measured_result.status.to_s
       else
-        create_event payload: payload
+        create_event(payload:)
         memory['last_status'] = nil
       end
-
     end
 
     def ping(url)
       result = faraday.get url
       result.status > 0 ? result : nil
-    rescue
+    rescue StandardError
       nil
     end
   end
-
 end

app/models/agents/human_task_agent.rb

@@ -4,7 +4,7 @@ module Agents
 
     gem_dependency_check { defined?(RTurk) }
 
-    description <<-MD
+    description <<~MD
       The Human Task Agent is used to create Human Intelligence Tasks (HITs) on Mechanical Turk.
 
       #{'## Include `rturk` in your Gemfile to use this Agent!' if dependencies_missing?}
@@ -118,7 +118,7 @@ module Agents
       As with most Agents, `expected_receive_period_in_days` is required if `trigger_on` is set to `event`.
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       Events look like:
 
           {
@@ -135,24 +135,45 @@ module Agents
       options['hit'] ||= {}
       options['hit']['questions'] ||= []
 
-      errors.add(:base, "'trigger_on' must be one of 'schedule' or 'event'") unless %w[schedule event].include?(options['trigger_on'])
-      errors.add(:base, "'hit.assignments' should specify the number of HIT assignments to create") unless options['hit']['assignments'].present? && options['hit']['assignments'].to_i > 0
+      errors.add(
+        :base, "'trigger_on' must be one of 'schedule' or 'event'"
+      ) unless %w[schedule event].include?(options['trigger_on'])
+      errors.add(
+        :base,
+        "'hit.assignments' should specify the number of HIT assignments to create"
+      ) unless options['hit']['assignments'].present? &&
+        options['hit']['assignments'].to_i > 0
       errors.add(:base, "'hit.title' must be provided") unless options['hit']['title'].present?
       errors.add(:base, "'hit.description' must be provided") unless options['hit']['description'].present?
-      errors.add(:base, "'hit.questions' must be provided") unless options['hit']['questions'].present? && options['hit']['questions'].length > 0
+      errors.add(:base, "'hit.questions' must be provided") unless options['hit']['questions'].present?
 
       if options['trigger_on'] == "event"
-        errors.add(:base, "'expected_receive_period_in_days' is required when 'trigger_on' is set to 'event'") unless options['expected_receive_period_in_days'].present?
+        errors.add(
+          :base,
+          "'expected_receive_period_in_days' is required when 'trigger_on' is set to 'event'"
+        ) unless options['expected_receive_period_in_days'].present?
       elsif options['trigger_on'] == "schedule"
-        errors.add(:base, "'submission_period' must be set to a positive number of hours when 'trigger_on' is set to 'schedule'") unless options['submission_period'].present? && options['submission_period'].to_i > 0
+        errors.add(
+          :base,
+          "'submission_period' must be set to a positive number of hours when 'trigger_on' is set to 'schedule'"
+        ) unless options['submission_period'].present? &&
+          options['submission_period'].to_i > 0
       end
 
-      if options['hit']['questions'].any? { |question| %w[key name required type question].any? {|k| !question[k].present? } }
+      if options['hit']['questions'].any? { |question|
+           %w[key name required type question].any? { |k| question[k].blank? }
+         }
         errors.add(:base, "all questions must set 'key', 'name', 'required', 'type', and 'question'")
       end
 
-      if options['hit']['questions'].any? { |question| question['type'] == "selection" && (!question['selections'].present? || question['selections'].length == 0 || !question['selections'].all? {|s| s['key'].present? } || !question['selections'].all? { |s| s['text'].present? })}
-        errors.add(:base, "all questions of type 'selection' must have a selections array with selections that set 'key' and 'name'")
+      if options['hit']['questions'].any? { |question|
+           question['type'] == "selection" && (
+             question['selections'].blank? ||
+               question['selections'].any? { |s| s['key'].blank? || s['text'].blank? }
+           )
+         }
+        errors.add(:base,
+                   "all questions of type 'selection' must have a selections array with selections that set 'key' and 'name'")
       end
 
       if take_majority? && options['hit']['questions'].any? { |question| question['type'] != "selection" }
@@ -160,7 +181,14 @@ module Agents
       end
 
       if create_poll?
-        errors.add(:base, "poll_options is required when combination_mode is set to 'poll' and must have the keys 'title', 'instructions', 'row_template', and 'assignments'") unless options['poll_options'].is_a?(Hash) && options['poll_options']['title'].present? && options['poll_options']['instructions'].present? && options['poll_options']['row_template'].present? && options['poll_options']['assignments'].to_i > 0
+        errors.add(
+          :base,
+          "poll_options is required when combination_mode is set to 'poll' and must have the keys 'title', 'instructions', 'row_template', and 'assignments'"
+        ) unless options['poll_options'].is_a?(Hash) &&
+          options['poll_options']['title'].present? &&
+          options['poll_options']['instructions'].present? &&
+          options['poll_options']['row_template'].present? &&
+          options['poll_options']['assignments'].to_i > 0
       end
     end
 
@@ -229,7 +257,6 @@ module Agents
     protected
 
     if defined?(RTurk)
-
       def take_majority?
         interpolated['combination_mode'] == "take_majority" || interpolated['take_majority'] == "true"
       end
@@ -266,139 +293,151 @@ module Agents
           assignments = hit.assignments
 
           log "Looking at HIT #{hit_id}.  I found #{assignments.length} assignments#{" with the statuses: #{assignments.map(&:status).to_sentence}" if assignments.length > 0}"
-          if assignments.length == hit.max_assignments && assignments.all? { |assignment| assignment.status == "Submitted" }
-            inbound_event = event_for_hit(hit_id)
+          next unless assignments.length == hit.max_assignments &&
+            assignments.all? { |assignment|
+              assignment.status == "Submitted"
+            }
 
-            if hit_type(hit_id) == 'poll'
-              # handle completed polls
+          inbound_event = event_for_hit(hit_id)
 
-              log "Handling a poll: #{hit_id}"
+          if hit_type(hit_id) == 'poll'
+            # handle completed polls
 
-              scores = {}
-              assignments.each do |assignment|
-                assignment.answers.each do |index, rating|
-                  scores[index] ||= 0
-                  scores[index] += rating.to_i
-                end
+            log "Handling a poll: #{hit_id}"
+
+            scores = {}
+            assignments.each do |assignment|
+              assignment.answers.each do |index, rating|
+                scores[index] ||= 0
+                scores[index] += rating.to_i
               end
+            end
 
-              top_answer = scores.to_a.sort {|b, a| a.last <=> b.last }.first.first
+            top_answer = scores.to_a.sort { |b, a| a.last <=> b.last }.first.first
 
-              payload = {
-                'answers' => memory['hits'][hit_id]['answers'],
-                'poll' => assignments.map(&:answers),
-                'best_answer' => memory['hits'][hit_id]['answers'][top_answer.to_i - 1]
-              }
+            payload = {
+              'answers' => memory['hits'][hit_id]['answers'],
+              'poll' => assignments.map(&:answers),
+              'best_answer' => memory['hits'][hit_id]['answers'][top_answer.to_i - 1]
+            }
 
-              event = create_event :payload => payload
-              log "Event emitted with answer(s) for poll", :outbound_event => event, :inbound_event => inbound_event
-            else
-              # handle normal completed HITs
-              payload = { 'answers' => assignments.map(&:answers) }
-
-              if take_majority?
-                counts = {}
-                options['hit']['questions'].each do |question|
-                  question_counts = question['selections'].inject({}) { |memo, selection| memo[selection['key']] = 0; memo }
-                  assignments.each do |assignment|
-                    answers = ActiveSupport::HashWithIndifferentAccess.new(assignment.answers)
-                    answer = answers[question['key']]
-                    question_counts[answer] += 1
-                  end
-                  counts[question['key']] = question_counts
+            event = create_event(payload:)
+            log("Event emitted with answer(s) for poll", outbound_event: event, inbound_event:)
+          else
+            # handle normal completed HITs
+            payload = { 'answers' => assignments.map(&:answers) }
+
+            if take_majority?
+              counts = {}
+              options['hit']['questions'].each do |question|
+                question_counts = question['selections'].each_with_object({}) { |selection, memo|
+                  memo[selection['key']] = 0
+                }
+                assignments.each do |assignment|
+                  answers = ActiveSupport::HashWithIndifferentAccess.new(assignment.answers)
+                  answer = answers[question['key']]
+                  question_counts[answer] += 1
                 end
-                payload['counts'] = counts
+                counts[question['key']] = question_counts
+              end
+              payload['counts'] = counts
 
-                majority_answer = counts.inject({}) do |memo, (key, question_counts)|
-                  memo[key] = question_counts.to_a.sort {|a, b| a.last <=> b.last }.last.first
-                  memo
-                end
-                payload['majority_answer'] = majority_answer
-
-                if all_questions_are_numeric?
-                  average_answer = counts.inject({}) do |memo, (key, question_counts)|
-                    sum = divisor = 0
-                    question_counts.to_a.each do |num, count|
-                      sum += num.to_s.to_f * count
-                      divisor += count
-                    end
-                    memo[key] = sum / divisor.to_f
-                    memo
+              majority_answer = counts.each_with_object({}) do |(key, question_counts), memo|
+                memo[key] = question_counts.to_a.sort_by(&:last).last.first
+              end
+              payload['majority_answer'] = majority_answer
+
+              if all_questions_are_numeric?
+                average_answer = counts.each_with_object({}) do |(key, question_counts), memo|
+                  sum = divisor = 0
+                  question_counts.to_a.each do |num, count|
+                    sum += num.to_s.to_f * count
+                    divisor += count
                   end
-                  payload['average_answer'] = average_answer
+                  memo[key] = sum / divisor.to_f
                 end
+                payload['average_answer'] = average_answer
               end
+            end
 
-              if create_poll?
-                questions = []
-                selections = 5.times.map { |i| { 'key' => i+1, 'text' => i+1 } }.reverse
-                assignments.length.times do |index|
-                  questions << {
-                    'type' => "selection",
-                    'name' => "Item #{index + 1}",
-                    'key' => index,
-                    'required' => "true",
-                    'question' => interpolate_string(options['poll_options']['row_template'], assignments[index].answers),
-                    'selections' => selections
-                  }
-                end
+            if create_poll?
+              questions = []
+              selections = 5.times.map { |i| { 'key' => i + 1, 'text' => i + 1 } }.reverse
+              assignments.length.times do |index|
+                questions << {
+                  'type' => "selection",
+                  'name' => "Item #{index + 1}",
+                  'key' => index,
+                  'required' => "true",
+                  'question' => interpolate_string(options['poll_options']['row_template'],
+                                                   assignments[index].answers),
+                  'selections' => selections
+                }
+              end
 
-                poll_hit = create_hit 'title' => options['poll_options']['title'],
-                                      'description' => options['poll_options']['instructions'],
-                                      'questions' => questions,
-                                      'assignments' => options['poll_options']['assignments'],
-                                      'lifetime_in_seconds' => options['poll_options']['lifetime_in_seconds'],
-                                      'reward' => options['poll_options']['reward'],
-                                      'payload' => inbound_event && inbound_event.payload,
-                                      'metadata' => { 'type' => 'poll',
-                                                      'original_hit' => hit_id,
-                                                      'answers' => assignments.map(&:answers),
-                                                      'event_id' => inbound_event && inbound_event.id }
-
-                log "Poll HIT created with ID #{poll_hit.id} and URL #{poll_hit.url}.  Original HIT: #{hit_id}", :inbound_event => inbound_event
-              else
-                if options[:separate_answers]
-                  payload['answers'].each.with_index do |answer, index|
-                    sub_payload = payload.dup
-                    sub_payload.delete('answers')
-                    sub_payload['answer'] = answer
-                    event = create_event :payload => sub_payload
-                    log "Event emitted with answer ##{index}", :outbound_event => event, :inbound_event => inbound_event
-                  end
-                else
-                  event = create_event :payload => payload
-                  log "Event emitted with answer(s)", :outbound_event => event, :inbound_event => inbound_event
-                end
+              poll_hit = create_hit(
+                'title' => options['poll_options']['title'],
+                'description' => options['poll_options']['instructions'],
+                'questions' => questions,
+                'assignments' => options['poll_options']['assignments'],
+                'lifetime_in_seconds' => options['poll_options']['lifetime_in_seconds'],
+                'reward' => options['poll_options']['reward'],
+                'payload' => inbound_event && inbound_event.payload,
+                'metadata' => {
+                  'type' => 'poll',
+                  'original_hit' => hit_id,
+                  'answers' => assignments.map(&:answers),
+                  'event_id' => inbound_event && inbound_event.id
+                }
+              )
+
+              log(
+                "Poll HIT created with ID #{poll_hit.id} and URL #{poll_hit.url}.  Original HIT: #{hit_id}",
+                inbound_event:
+              )
+            elsif options[:separate_answers]
+              payload['answers'].each.with_index do |answer, index|
+                sub_payload = payload.dup
+                sub_payload.delete('answers')
+                sub_payload['answer'] = answer
+                event = create_event payload: sub_payload
+                log("Event emitted with answer ##{index}", outbound_event: event, inbound_event:)
               end
+            else
+              event = create_event(payload:)
+              log("Event emitted with answer(s)", outbound_event: event, inbound_event:)
             end
+          end
 
-            assignments.each(&:approve!)
-            hit.dispose!
+          assignments.each(&:approve!)
+          hit.dispose!
 
-            memory['hits'].delete(hit_id)
-          end
+          memory['hits'].delete(hit_id)
         end
       end
 
       def all_questions_are_numeric?
         interpolated['hit']['questions'].all? do |question|
           question['selections'].all? do |selection|
-            selection['key'] == selection['key'].to_f.to_s || selection['key'] == selection['key'].to_i.to_s
+            value = selection['key']
+            value == value.to_f.to_s || value == value.to_i.to_s
           end
         end
       end
 
       def create_basic_hit(event = nil)
-        hit = create_hit 'title' => options['hit']['title'],
-                         'description' => options['hit']['description'],
-                         'questions' => options['hit']['questions'],
-                         'assignments' => options['hit']['assignments'],
-                         'lifetime_in_seconds' => options['hit']['lifetime_in_seconds'],
-                         'reward' => options['hit']['reward'],
-                         'payload' => event && event.payload,
-                         'metadata' => { 'event_id' => event && event.id }
-
-        log "HIT created with ID #{hit.id} and URL #{hit.url}", :inbound_event => event
+        hit = create_hit(
+          'title' => options['hit']['title'],
+          'description' => options['hit']['description'],
+          'questions' => options['hit']['questions'],
+          'assignments' => options['hit']['assignments'],
+          'lifetime_in_seconds' => options['hit']['lifetime_in_seconds'],
+          'reward' => options['hit']['reward'],
+          'payload' => event && event.payload,
+          'metadata' => { 'event_id' => event && event.id }
+        )
+
+        log("HIT created with ID #{hit.id} and URL #{hit.url}", inbound_event: event)
       end
 
       def create_hit(opts = {})
@@ -406,13 +445,13 @@ module Agents
         title = interpolate_string(opts['title'], payload).strip
         description = interpolate_string(opts['description'], payload).strip
         questions = interpolate_options(opts['questions'], payload)
-        hit = RTurk::Hit.create(title: title) do |hit|
+        hit = RTurk::Hit.create(title:) do |hit|
           hit.max_assignments = (opts['assignments'] || 1).to_i
           hit.description = description
           hit.lifetime = (opts['lifetime_in_seconds'] || 24 * 60 * 60).to_i
-          hit.question_form AgentQuestionForm.new(:title => title, :description => description, :questions => questions)
+          hit.question_form AgentQuestionForm.new(title:, description:, questions:)
           hit.reward = (opts['reward'] || 0.05).to_f
-          #hit.qualifications.add :approval_rate, { :gt => 80 }
+          # hit.qualifications.add :approval_rate, { gt: 80 }
         end
         memory['hits'] ||= {}
         memory['hits'][hit.id] = opts['metadata'] || {}

app/models/agents/imap_folder_agent.rb

@@ -14,7 +14,7 @@ module Agents
 
     default_schedule "every_30m"
 
-    description <<-MD
+    description <<~MD
       The Imap Folder Agent checks an IMAP server in specified folders and creates Events based on new mails found since the last run. In the first visit to a folder, this agent only checks for the initial status and does not create events.
 
       Specify an IMAP server to connect with `host`, and set `ssl` to true if the server supports IMAP over SSL.  Specify `port` if you need to connect to a port other than standard (143 or 993 depending on the `ssl` value), and specify login credentials in `username` and `password`.
@@ -50,7 +50,7 @@ module Agents
           If this key is unspecified or set to null, it is ignored.
 
       - `has_attachment`
-      
+
           Setting this to true or false means only mails that does or does not have an attachment are selected.
 
           If this key is unspecified or set to null, it is ignored.
@@ -73,7 +73,7 @@ module Agents
       Also, in order to avoid duplicated notification it keeps a list of Message-Id's of 100 most recent mails, so if multiple mails of the same Message-Id are found, you will only see one event out of them.
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       Events look like this:
 
           {
@@ -132,10 +132,8 @@ module Agents
       end
 
       %w[ssl mark_as_read delete include_raw_mail].each { |key|
-        if options[key].present?
-          if boolify(options[key]).nil?
-            errors.add(:base, '%s must be a boolean value' % key)
-          end
+        if options[key].present? && boolify(options[key]).nil?
+          errors.add(:base, '%s must be a boolean value' % key)
         end
       }
 
@@ -175,7 +173,7 @@ module Agents
             when String
               begin
                 Regexp.new(value)
-              rescue
+              rescue StandardError
                 errors.add(:base, 'conditions.%s contains an invalid regexp' % key)
               end
             else
@@ -187,7 +185,7 @@ module Agents
               when String
                 begin
                   glob_match?(pattern, '')
-                rescue
+                rescue StandardError
                   errors.add(:base, 'conditions.%s contains an invalid glob pattern' % key)
                 end
               else
@@ -207,7 +205,10 @@ module Agents
       end
 
       if options['expected_update_period_in_days'].present?
-        errors.add(:base, "Invalid expected_update_period_in_days format") unless is_positive_integer?(options['expected_update_period_in_days'])
+        errors.add(
+          :base,
+          "Invalid expected_update_period_in_days format"
+        ) unless is_positive_integer?(options['expected_update_period_in_days'])
       end
     end
 
@@ -240,14 +241,14 @@ module Agents
             value.present? or next true
             re = Regexp.new(value)
             matched_part = body_parts.find { |part|
-               if m = re.match(part.scrubbed(:decoded))
-                 m.names.each { |name|
-                   matches[name] = m[name]
-                 }
-                 true
-               else
-                 false
-               end
+              if m = re.match(part.scrubbed(:decoded))
+                m.names.each { |name|
+                  matches[name] = m[name]
+                }
+                true
+              else
+                false
+              end
             }
           when 'from', 'to', 'cc'
             value.present? or next true
@@ -266,7 +267,7 @@ module Agents
           when 'has_attachment'
             boolify(value) == mail.has_attachment?
           when 'is_unread'
-            true  # already filtered out by each_unread_mail
+            true # already filtered out by each_unread_mail
           else
             log 'Unknown condition key ignored: %s' % key
             true
@@ -295,7 +296,12 @@ module Agents
             'from' => mail.from_addrs.first,
             'to' => mail.to_addrs,
             'cc' => mail.cc_addrs,
-            'date' => (mail.date.iso8601 rescue nil),
+            'date' =>
+              begin
+                mail.date.iso8601
+              rescue StandardError
+                nil
+              end,
             'mime_type' => mime_type,
             'body' => body,
             'matches' => matches,
@@ -309,12 +315,12 @@ module Agents
           if interpolated['event_headers'].present?
             headers = mail.header.each_with_object({}) { |field, hash|
               name = field.name
-              hash[name] = (v = hash[name]) ? "#{v}\n#{field.value.to_s}" : field.value.to_s
+              hash[name] = (v = hash[name]) ? "#{v}\n#{field.value}" : field.value.to_s
             }
             payload.update(event_headers_payload(headers))
           end
 
-          create_event payload: payload
+          create_event(payload:)
 
           notified << mail.message_id if mail.message_id
         end
@@ -346,7 +352,7 @@ module Agents
       port = (Integer(port) if port.present?)
 
       log "Connecting to #{host}#{':%d' % port if port}#{' via SSL' if ssl}"
-      Client.open(host, port: port, ssl: ssl) { |imap|
+      Client.open(host, port:, ssl:) { |imap|
         log "Logging in as #{username}"
         if service
           imap.authenticate('XOAUTH2', username, password)
@@ -355,7 +361,8 @@ module Agents
         end
 
         # 'lastseen' keeps a hash of { uidvalidity => lastseenuid, ... }
-        lastseen, seen = self.lastseen, self.make_seen
+        lastseen = self.lastseen
+        seen = self.make_seen
 
         # 'notified' keeps an array of message-ids of {IDCACHE_SIZE}
         # most recent notified mails.
@@ -384,8 +391,8 @@ module Agents
           seen[uidvalidity] = lastseenuid
           is_unread = boolify(interpolated['conditions']['is_unread'])
 
-          uids = imap.uid_fetch((lastseenuid + 1)..-1, 'FLAGS').
-                 each_with_object([]) { |data, ret|
+          uids = imap.uid_fetch((lastseenuid + 1)..-1, 'FLAGS')
+            .each_with_object([]) { |data, ret|
             uid, flags = data.attr.values_at('UID', 'FLAGS')
             seen[uidvalidity] = uid
             next if uid <= lastseenuid
@@ -430,8 +437,8 @@ module Agents
       Seen.new(memory['lastseen'])
     end
 
-    def lastseen= value
-      memory.delete('seen')  # obsolete key
+    def lastseen=(value)
+      memory.delete('seen') # obsolete key
       memory['lastseen'] = value
     end
 
@@ -443,7 +450,7 @@ module Agents
       Notified.new(memory['notified'])
     end
 
-    def notified= value
+    def notified=(value)
       memory['notified'] = value
     end
 
@@ -540,7 +547,7 @@ module Agents
       module Scrubbed
         def scrubbed(method)
           (@scrubbed ||= {})[method.to_sym] ||=
-            __send__(method).try(:scrub) { |bytes| "<#{bytes.unpack('H*')[0]}>" }
+            __send__(method).try(:scrub) { |bytes| "<#{bytes.unpack1('H*')}>" }
         end
       end
 
@@ -588,7 +595,7 @@ module Agents
           [mail]
         end.select { |part|
           if part.multipart? || part.attachment? || !part.text? ||
-             !mime_types.include?(part.mime_type)
+              !mime_types.include?(part.mime_type)
             false
           else
             part.extend(Scrubbed)

app/models/agents/jabber_agent.rb

@@ -7,7 +7,7 @@ module Agents
 
     gem_dependency_check { defined?(Jabber) }
 
-    description <<-MD
+    description <<~MD
       The Jabber Agent will send any events it receives to your Jabber/XMPP IM account.
 
       #{'## Include `xmpp4r` in your Gemfile to use this Agent!' if dependencies_missing?}
@@ -23,7 +23,7 @@ module Agents
       Have a look at the [Wiki](https://github.com/huginn/huginn/wiki/Formatting-Events-using-Liquid) to learn more about liquid templating.
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       `event` will be set to either `on_join`, `on_leave`, `on_message`, `on_room_message` or `on_subject`
 
           {
@@ -36,12 +36,12 @@ module Agents
 
     def default_options
       {
-        'jabber_server'   => '127.0.0.1',
-        'jabber_port'     => '5222',
-        'jabber_sender'   => 'huginn@localhost',
+        'jabber_server' => '127.0.0.1',
+        'jabber_port' => '5222',
+        'jabber_sender' => 'huginn@localhost',
         'jabber_receiver' => 'muninn@localhost',
         'jabber_password' => '',
-        'message'         => 'It will be {{temp}} out tomorrow',
+        'message' => 'It will be {{temp}} out tomorrow',
         'expected_receive_period_in_days' => "2"
       }
     end
@@ -71,7 +71,7 @@ module Agents
     end
 
     def deliver(text)
-      client.send Jabber::Message::new(interpolated['jabber_receiver'], text).set_type(:chat)
+      client.send Jabber::Message.new(interpolated['jabber_receiver'], text).set_type(:chat)
     end
 
     def start_worker?
@@ -81,7 +81,7 @@ module Agents
     private
 
     def client
-      Jabber::Client.new(Jabber::JID::new(interpolated['jabber_sender'])).tap do |sender|
+      Jabber::Client.new(Jabber::JID.new(interpolated['jabber_sender'])).tap do |sender|
         sender.connect(interpolated['jabber_server'], interpolated['jabber_port'] || '5222')
         sender.auth interpolated['jabber_password']
       end
@@ -96,7 +96,7 @@ module Agents
     end
 
     class Worker < LongRunnable::Worker
-      IGNORE_MESSAGES_FOR=5
+      IGNORE_MESSAGES_FOR = 5
 
       def setup
         require 'xmpp4r/muc/helper/simplemucclient'
@@ -124,7 +124,7 @@ module Agents
         time, nick, message = normalize_args(event, args)
 
         AgentRunner.with_connection do
-          agent.create_event(payload: {event: event, time: time, nick: nick, message: message})
+          agent.create_event(payload: { event:, time:, nick:, message: })
         end
       end
 
@@ -139,6 +139,7 @@ module Agents
       end
 
       private
+
       def normalize_args(event, args)
         case event
         when :on_join, :on_leave

app/models/agents/java_script_agent.rb

@@ -11,7 +11,7 @@ module Agents
 
     gem_dependency_check { defined?(MiniRacer) }
 
-    description <<-MD
+    description <<~MD
       The JavaScript Agent allows you to write code in JavaScript that can create and receive events.  If other Agents aren't meeting your needs, try this one!
 
       #{'## Include `mini_racer` in your Gemfile to use this Agent!' if dependencies_missing?}
@@ -45,7 +45,8 @@ module Agents
     def validate_options
       cred_name = credential_referenced_by_code
       if cred_name
-        errors.add(:base, "The credential '#{cred_name}' referenced by code cannot be found") unless credential(cred_name).present?
+        errors.add(:base,
+                   "The credential '#{cred_name}' referenced by code cannot be found") unless credential(cred_name).present?
       else
         errors.add(:base, "The 'code' option is required") unless options['code'].present?
       end
@@ -114,22 +115,22 @@ module Agents
       context = MiniRacer::Context.new
       context.eval(setup_javascript)
 
-      context.attach("doCreateEvent", -> (y) { create_event(payload: clean_nans(JSON.parse(y))).payload.to_json })
+      context.attach("doCreateEvent", ->(y) { create_event(payload: clean_nans(JSON.parse(y))).payload.to_json })
       context.attach("getIncomingEvents", -> { incoming_events.to_json })
       context.attach("getOptions", -> { interpolated.to_json })
-      context.attach("doLog", -> (x) { log x })
-      context.attach("doError", -> (x) { error x })
+      context.attach("doLog", ->(x) { log x })
+      context.attach("doError", ->(x) { error x })
       context.attach("getMemory", -> { memory.to_json })
-      context.attach("setMemoryKey", -> (x, y) { memory[x] = clean_nans(y) })
-      context.attach("setMemory", -> (x) { memory.replace(clean_nans(x)) })
-      context.attach("deleteKey", -> (x) { memory.delete(x).to_json })
-      context.attach("escapeHtml", -> (x) { CGI.escapeHTML(x) })
-      context.attach("unescapeHtml", -> (x) { CGI.unescapeHTML(x) })
-      context.attach('getCredential', -> (k) { credential(k); })
-      context.attach('setCredential', -> (k, v) { set_credential(k, v) })
+      context.attach("setMemoryKey", ->(x, y) { memory[x] = clean_nans(y) })
+      context.attach("setMemory", ->(x) { memory.replace(clean_nans(x)) })
+      context.attach("deleteKey", ->(x) { memory.delete(x).to_json })
+      context.attach("escapeHtml", ->(x) { CGI.escapeHTML(x) })
+      context.attach("unescapeHtml", ->(x) { CGI.unescapeHTML(x) })
+      context.attach('getCredential', ->(k) { credential(k); })
+      context.attach('setCredential', ->(k, v) { set_credential(k, v) })
 
       if (options['language'] || '').downcase == 'coffeescript'
-        context.eval(CoffeeScript.compile code)
+        context.eval(CoffeeScript.compile(code))
       else
         context.eval(code)
       end
@@ -223,20 +224,19 @@ module Agents
     end
 
     def log_errors
-      begin
-        yield
-      rescue MiniRacer::Error => e
-        error "JavaScript error: #{e.message}"
-      end
+      yield
+    rescue MiniRacer::Error => e
+      error "JavaScript error: #{e.message}"
     end
 
     def clean_nans(input)
-      if input.is_a?(Array)
-        input.map {|v| clean_nans(v) }
-      elsif input.is_a?(Hash)
-        input.inject({}) { |m, (k, v)| m[k] = clean_nans(v); m }
-      elsif input.is_a?(Float) && input.nan?
-        'NaN'
+      case input
+      when Array
+        input.map { |v| clean_nans(v) }
+      when Hash
+        input.transform_values { |v| clean_nans(v) }
+      when Float
+        input.nan? ? 'NaN' : input
       else
         input
       end

app/models/agents/jira_agent.rb

@@ -10,11 +10,11 @@ module Agents
 
     cannot_receive_events!
 
-    description <<-MD
+    description <<~MD
       The Jira Agent subscribes to Jira issue updates.
 
       - `jira_url` specifies the full URL of the jira installation, including https://
-      - `jql` is an optional Jira Query Language-based filter to limit the flow of events. See [JQL Docs](https://confluence.atlassian.com/display/JIRA/Advanced+Searching) for details. 
+      - `jql` is an optional Jira Query Language-based filter to limit the flow of events. See [JQL Docs](https://confluence.atlassian.com/display/JIRA/Advanced+Searching) for details.#{' '}
       - `username` and `password` are optional, and may need to be specified if your Jira instance is read-protected
       - `timeout` is an optional parameter that specifies how long the request processing may take in minutes.
 
@@ -23,18 +23,18 @@ module Agents
       NOTE: upon the first execution, the agent will fetch everything available by the JQL query. So if it's not desirable, limit the `jql` query by date.
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       Events are the raw JSON generated by Jira REST API
 
-      {
-        "expand": "editmeta,renderedFields,transitions,changelog,operations",
-        "id": "80127",
-        "self": "https://jira.atlassian.com/rest/api/2/issue/80127",
-        "key": "BAM-3512",
-        "fields": {
-          ...
-        }
-      }
+          {
+            "expand": "editmeta,renderedFields,transitions,changelog,operations",
+            "id": "80127",
+            "self": "https://jira.atlassian.com/rest/api/2/issue/80127",
+            "key": "BAM-3512",
+            "fields": {
+              ...
+            }
+          }
     MD
 
     default_schedule "every_10m"
@@ -42,7 +42,7 @@ module Agents
 
     def default_options
       {
-        'username'  => '',
+        'username' => '',
         'password' => '',
         'jira_url' => 'https://jira.atlassian.com',
         'jql' => '',
@@ -52,9 +52,11 @@ module Agents
     end
 
     def validate_options
-      errors.add(:base, "you need to specify password if user name is set") if options['username'].present? and not options['password'].present?
+      errors.add(:base,
+                 "you need to specify password if user name is set") if options['username'].present? and !options['password'].present?
       errors.add(:base, "you need to specify your jira URL") unless options['jira_url'].present?
-      errors.add(:base, "you need to specify the expected update period") unless options['expected_update_period_in_days'].present?
+      errors.add(:base,
+                 "you need to specify the expected update period") unless options['expected_update_period_in_days'].present?
       errors.add(:base, "you need to specify request timeout") unless options['timeout'].present?
     end
 
@@ -74,41 +76,50 @@ module Agents
 
         # this check is more precise than in get_issues()
         # see get_issues() for explanation
-        if not last_run or updated > last_run
-          create_event :payload => issue
+        if !last_run or updated > last_run
+          create_event payload: issue
         end
       end
 
       memory[:last_run] = current_run
     end
 
-  private
+    private
+
     def request_url(jql, start_at)
-      "#{interpolated[:jira_url]}/rest/api/2/search?jql=#{CGI::escape(jql)}&fields=*all&startAt=#{start_at}"
+      "#{interpolated[:jira_url]}/rest/api/2/search?jql=#{CGI.escape(jql)}&fields=*all&startAt=#{start_at}"
     end
 
     def request_options
-      ropts = { headers: {"User-Agent" => user_agent} }
+      ropts = { headers: { "User-Agent" => user_agent } }
 
       if !interpolated[:username].empty?
-        ropts = ropts.merge({:basic_auth => {:username =>interpolated[:username], :password=>interpolated[:password]}})
+        ropts = ropts.merge({
+          basic_auth: {
+            username: interpolated[:username],
+            password: interpolated[:password]
+          }
+        })
       end
 
       ropts
     end
 
     def get(url, options)
-        response = HTTParty.get(url, options)
-
-        if response.code == 400
-          raise RuntimeError.new("Jira error: #{response['errorMessages']}") 
-        elsif response.code == 403
-          raise RuntimeError.new("Authentication failed: Forbidden (403)")
-        elsif response.code != 200
-          raise RuntimeError.new("Request failed: #{response}")
-        end
+      response = HTTParty.get(url, options)
+
+      case response.code
+      when 200
+        # OK
+      when 400
+        raise "Jira error: #{response['errorMessages']}"
+      when 403
+        raise "Authentication failed: Forbidden (403)"
+      else
+        raise "Request failed: #{response}"
+      end
 
-        response
+      response
     end
 
     def get_issues(since)
@@ -120,7 +131,7 @@ module Agents
       # earlier and filter out unnecessary ones at a later
       # stage. Fortunately, the 'updated' field has GMT
       # offset
-      since -= 24*60*60 if since
+      since -= 24 * 60 * 60 if since
 
       jql = ""
 
@@ -138,26 +149,24 @@ module Agents
         response = get(request_url(jql, startAt), request_options)
 
         if response['issues'].length == 0
-          request_limit+=1
+          request_limit += 1
         end
 
         if request_limit > MAX_EMPTY_REQUESTS
-          raise RuntimeError.new("There is no progress while fetching issues")
+          raise "There is no progress while fetching issues"
         end
 
         if Time.now > start_time + interpolated['timeout'].to_i * 60
-          raise RuntimeError.new("Timeout exceeded while fetching issues")
+          raise "Timeout exceeded while fetching issues"
         end
 
         issues += response['issues']
         startAt += response['issues'].length
- 
+
         break if startAt >= response['total']
       end
 
       issues
     end
-
   end
 end
-

app/models/agents/jq_agent.rb

@@ -29,7 +29,7 @@ module Agents
 
     gem_dependency_check { jq_version }
 
-    description <<-MD
+    description <<~MD
       The Jq Agent allows you to process incoming Events with [jq](https://stedolan.github.io/jq/) the JSON processor. (#{jq_info})
 
       It allows you to filter, transform and restructure Events in the way you want using jq's powerful features.
@@ -97,7 +97,8 @@ module Agents
 
     def validate_options
       errors.add(:base, "filter needs to be present.") if !options['filter'].is_a?(String)
-      errors.add(:base, "variables must be a hash if present.") if options.key?('variables') && !options['variables'].is_a?(Hash)
+      errors.add(:base,
+                 "variables must be a hash if present.") if options.key?('variables') && !options['variables'].is_a?(Hash)
     end
 
     def default_options
@@ -196,7 +197,7 @@ module Agents
           log "Creating #{results.size} events"
 
           results.each do |payload|
-            create_event payload: payload
+            create_event(payload:)
           end
         end
       end

app/models/agents/json_parse_agent.rb

@@ -5,7 +5,7 @@ module Agents
     cannot_be_scheduled!
     can_dry_run!
 
-    description <<-MD
+    description <<~MD
       The JSON Parse Agent parses a JSON string and emits the data in a new event or merge with with the original event.
 
       `data` is the JSON to parse. Use [Liquid](https://github.com/huginn/huginn/wiki/Formatting-Events-using-Liquid) templating to specify the JSON string.
@@ -24,7 +24,7 @@ module Agents
     end
 
     event_description do
-      "Events will looks like this:\n\n    %s" % Utils.pretty_print(interpolated['data_key'] => {parsed: 'object'})
+      "Events will looks like this:\n\n    %s" % Utils.pretty_print(interpolated['data_key'] => { parsed: 'object' })
     end
 
     form_configurable :data
@@ -34,7 +34,7 @@ module Agents
     def validate_options
       errors.add(:base, "data needs to be present") if options['data'].blank?
       errors.add(:base, "data_key needs to be present") if options['data_key'].blank?
-      if options['mode'].present? && !options['mode'].to_s.include?('{{') && !%[clean merge].include?(options['mode'].to_s)
+      if options['mode'].present? && !options['mode'].to_s.include?('{{') && !%(clean merge).include?(options['mode'].to_s)
         errors.add(:base, "mode must be 'clean' or 'merge'")
       end
     end
@@ -45,13 +45,11 @@ module Agents
 
     def receive(incoming_events)
       incoming_events.each do |event|
-        begin
-          mo = interpolated(event)
-          existing_payload = mo['mode'].to_s == 'merge' ? event.payload : {}
-          create_event payload: existing_payload.merge({ mo['data_key'] => JSON.parse(mo['data']) })
-        rescue JSON::JSONError => e
-          error("Could not parse JSON: #{e.class} '#{e.message}'")
-        end
+        mo = interpolated(event)
+        existing_payload = mo['mode'].to_s == 'merge' ? event.payload : {}
+        create_event payload: existing_payload.merge({ mo['data_key'] => JSON.parse(mo['data']) })
+      rescue JSON::JSONError => e
+        error("Could not parse JSON: #{e.class} '#{e.message}'")
       end
     end
   end

app/models/agents/key_value_store_agent.rb

@@ -6,7 +6,7 @@ module Agents
     cannot_be_scheduled!
     cannot_create_events!
 
-    description <<-MD
+    description <<~MD
       The Key-Value Store Agent is a data storage that keeps an associative array in its memory.  It receives events to store values and provides the data to other agents as an object via Liquid Templating.
 
       Liquid templates specified in the `key` and `value` options are evaluated for each received event to be stored in the memory.

app/models/agents/liquid_output_agent.rb

@@ -8,24 +8,24 @@ module Agents
     DATE_UNITS = %w[second seconds minute minutes hour hours day days week weeks month months year years]
 
     description  do
-      <<-MD
-        The Liquid Output Agent outputs events through a Liquid template you provide.  Use it to create a HTML page, or a json feed, or anything else that can be rendered as a string from your stream of Huginn data. 
+      <<~MD
+        The Liquid Output Agent outputs events through a Liquid template you provide.  Use it to create a HTML page, or a json feed, or anything else that can be rendered as a string from your stream of Huginn data.
 
         This Agent will output data at:
 
-        `https://#{ENV['DOMAIN']}#{Rails.application.routes.url_helpers.web_requests_path(agent_id: ':id', user_id: user_id, secret: ':secret', format: :any_extension)}`
+        `https://#{ENV['DOMAIN']}#{Rails.application.routes.url_helpers.web_requests_path(agent_id: ':id', user_id:, secret: ':secret', format: :any_extension)}`
 
         where `:secret` is the secret specified in your options.  You can use any extension you wish.
 
         Options:
 
-          * `secret` - A token that the requestor must provide for light-weight authentication.
-          * `expected_receive_period_in_days` - How often you expect data to be received by this Agent from other Agents.
-          * `content` - The content to display when someone requests this page.
-          * `mime_type` - The mime type to use when someone requests this page.
-          * `response_headers` - An object with any custom response headers. (example: `{"Access-Control-Allow-Origin": "*"}`)
-          * `mode` - The behavior that determines what data is passed to the Liquid template.
-          * `event_limit` - A limit applied to the events passed to a template when in "Last X events" mode. Can be a count like "1", or an amount of time like "1 day" or "5 minutes".
+        * `secret` - A token that the requestor must provide for light-weight authentication.
+        * `expected_receive_period_in_days` - How often you expect data to be received by this Agent from other Agents.
+        * `content` - The content to display when someone requests this page.
+        * `mime_type` - The mime type to use when someone requests this page.
+        * `response_headers` - An object with any custom response headers. (example: `{"Access-Control-Allow-Origin": "*"}`)
+        * `mode` - The behavior that determines what data is passed to the Liquid template.
+        * `event_limit` - A limit applied to the events passed to a template when in "Last X events" mode. Can be a count like "1", or an amount of time like "1 day" or "5 minutes".
 
         # Liquid Templating
 
@@ -35,61 +35,56 @@ module Agents
 
         ### Merge events
 
-          The data for incoming events will be merged. So if two events come in like this:
+        The data for incoming events will be merged. So if two events come in like this:
 
-```
-{ 'a' => 'b',  'c' => 'd'}
-{ 'a' => 'bb', 'e' => 'f'}
-```
+        ```
+        { 'a' => 'b',  'c' => 'd'}
+        { 'a' => 'bb', 'e' => 'f'}
+        ```
 
-          The final result will be:
+        The final result will be:
 
-```
-{ 'a' => 'bb', 'c' => 'd', 'e' => 'f'}
-```
+        ```
+        { 'a' => 'bb', 'c' => 'd', 'e' => 'f'}
+        ```
 
         This merged version will be passed to the Liquid template.
 
         ### Last event in
 
-          The data from the last event will be passed to the template.
+        The data from the last event will be passed to the template.
 
         ### Last X events
 
-          All of the events received by this agent will be passed to the template
-          as the ```events``` array.
+        All of the events received by this agent will be passed to the template as the `events` array.
 
-          The number of events can be controlled via the ```event_limit``` option.
-          If ```event_limit``` is an integer X, the last X events will be passed
-          to the template.  If ```event_limit``` is an integer with a unit of
-          measure like "1 day" or "5 minutes" or "9 years", a date filter will
-          be applied to the events passed to the template.  If no ```event_limit```
-          is provided, then all of the events for the agent will be passed to
-          the template. 
-          
-          For performance, the maximum ```event_limit``` allowed is 1000.
+        The number of events can be controlled via the `event_limit` option.
+        If `event_limit` is an integer X, the last X events will be passed to the template.
+        If `event_limit` is an integer with a unit of measure like "1 day" or "5 minutes" or "9 years", a date filter will be applied to the events passed to the template.
+        If no `event_limit` is provided, then all of the events for the agent will be passed to the template.
 
+        For performance, the maximum `event_limit` allowed is 1000.
       MD
     end
 
     def default_options
-      content = <<EOF
-When you use the "Last event in" or "Merge events" option, you can use variables from the last event received, like this:
-
-Name: {{name}}
-Url:  {{url}}
-
-If you use the "Last X Events" mode, a set of events will be passed to your Liquid template.  You can use them like this:
-
-<table class="table">
-  {% for event in events %}
-    <tr>
-      <td>{{ event.title }}</td>
-      <td><a href="{{ event.url }}">Click here to see</a></td>
-    </tr>
-  {% endfor %}
-</table>
-EOF
+      content = <<~EOF
+        When you use the "Last event in" or "Merge events" option, you can use variables from the last event received, like this:
+
+        Name: {{name}}
+        Url:  {{url}}
+
+        If you use the "Last X Events" mode, a set of events will be passed to your Liquid template.  You can use them like this:
+
+        <table class="table">
+          {% for event in events %}
+            <tr>
+              <td>{{ event.title }}</td>
+              <td><a href="{{ event.url }}">Click here to see</a></td>
+            </tr>
+          {% endfor %}
+        </table>
+      EOF
       {
         "secret" => "a-secret-key",
         "expected_receive_period_in_days" => 2,
@@ -104,7 +99,7 @@ EOF
     form_configurable :expected_receive_period_in_days
     form_configurable :content, type: :text
     form_configurable :mime_type
-    form_configurable :mode, type: :array, values: [ 'Last event in', 'Merge events', 'Last X events']
+    form_configurable :mode, type: :array, values: ['Last event in', 'Merge events', 'Last X events']
     form_configurable :event_limit
 
     def working?
@@ -125,35 +120,49 @@ EOF
       end
 
       unless options['expected_receive_period_in_days'].present? && options['expected_receive_period_in_days'].to_i > 0
-        errors.add(:base, "Please provide 'expected_receive_period_in_days' to indicate how many days can pass before this Agent is considered to be not working")
+        errors.add(
+          :base,
+          "Please provide 'expected_receive_period_in_days' to indicate how many days can pass before this Agent is considered to be not working"
+        )
       end
 
-      if options['event_limit'].present?
-        if (Integer(options['event_limit']) rescue false) == false && date_limit.blank?
-          errors.add(:base, "Event limit must be an integer that is less than 1001 or an integer plus a valid unit.")
-        elsif (options['event_limit'].to_i > 1000)
-          errors.add(:base, "For performance reasons, you cannot have an event limit greater than 1000.")
+      event_limit =
+        if value = options['event_limit'].presence
+          begin
+            Integer(value)
+          rescue StandardError
+            false
+          end
         end
-      else
+
+      if event_limit == false && date_limit.blank?
+        errors.add(:base, "Event limit must be an integer that is less than 1001 or an integer plus a valid unit.")
+      elsif event_limit && event_limit > 1000
+        errors.add(:base, "For performance reasons, you cannot have an event limit greater than 1000.")
       end
     end
 
     def receive(incoming_events)
       return unless ['merge events', 'last event in'].include?(mode)
+
       memory['last_event'] ||= {}
       incoming_events.each do |event|
-        case mode
-        when 'merge events'
-          memory['last_event'] = memory['last_event'].merge(event.payload)
-        else
-          memory['last_event'] = event.payload
-        end
+        memory['last_event'] =
+          case mode
+          when 'merge events'
+            memory['last_event'].merge(event.payload)
+          else
+            event.payload
+          end
       end
     end
 
     def receive_web_request(params, method, format)
-      valid_authentication?(params) ? [liquified_content, 200, mime_type, interpolated['response_headers'].presence]
-                                    : [unauthorized_content(format), 401]
+      if valid_authentication?(params)
+        [liquified_content, 200, mime_type, interpolated['response_headers'].presence]
+      else
+        [unauthorized_content(format), 401]
+      end
     end
 
     private
@@ -163,8 +172,11 @@ EOF
     end
 
     def unauthorized_content(format)
-      format =~ /json/ ? { error: "Not Authorized" }
-                       : "Not Authorized"
+      if format =~ /json/
+        { error: "Not Authorized" }
+      else
+        "Not Authorized"
+      end
     end
 
     def valid_authentication?(params)
@@ -193,19 +205,29 @@ EOF
     end
 
     def count_limit
-      limit = Integer(options['event_limit']) rescue 1000
+      limit = begin
+        Integer(options['event_limit'])
+      rescue StandardError
+        1000
+      end
       limit <= 1000 ? limit : 1000
     end
 
     def date_limit
       return nil unless options['event_limit'].to_s.include?(' ')
+
       value, unit = options['event_limit'].split(' ')
-      value = Integer(value) rescue nil
+      value = begin
+        Integer(value)
+      rescue StandardError
+        nil
+      end
       return nil unless value
+
       unit = unit.to_s.downcase
       return nil unless DATE_UNITS.include?(unit)
+
       value.send(unit.to_sym).ago
     end
-
   end
 end

app/models/agents/local_file_agent.rb

@@ -13,7 +13,7 @@ module Agents
     end
 
     description do
-      <<-MD
+      <<~MD
         The LocalFileAgent can watch a file/directory for changes or emit an event for every file in that directory. When receiving an event it writes the received data into a file.
 
         `mode` determines if the agent is emitting events for (changed) files or writing received event data to disk.
@@ -41,22 +41,23 @@ module Agents
     end
 
     event_description do
-      "Events will looks like this:\n\n    %s" % if boolify(interpolated['watch'])
-        Utils.pretty_print(
-          "file_pointer" => {
-            "file" => "/tmp/test/filename",
-            "agent_id" => id
-          },
-          "event_type" => "modified/added/removed"
-        )
-      else
-        Utils.pretty_print(
-          "file_pointer" => {
-            "file" => "/tmp/test/filename",
-            "agent_id" => id
-          }
-        )
-      end
+      "Events will looks like this:\n\n    " +
+        if boolify(interpolated['watch'])
+          Utils.pretty_print(
+            "file_pointer" => {
+              "file" => "/tmp/test/filename",
+              "agent_id" => id
+            },
+            "event_type" => "modified/added/removed"
+          )
+        else
+          Utils.pretty_print(
+            "file_pointer" => {
+              "file" => "/tmp/test/filename",
+              "agent_id" => id
+            }
+          )
+        end
     end
 
     def default_options
@@ -69,8 +70,8 @@ module Agents
       }
     end
 
-    form_configurable :mode, type: :array, values: %w(read write)
-    form_configurable :watch, type: :array, values: %w(true false)
+    form_configurable :mode, type: :array, values: %w[read write]
+    form_configurable :watch, type: :array, values: %w[true false]
     form_configurable :path, type: :string
     form_configurable :append, type: :boolean
     form_configurable :data, type: :string
@@ -98,6 +99,7 @@ module Agents
     def check
       return if interpolated['mode'] != 'read' || boolify(interpolated['watch']) || !should_run?
       return unless check_path_existance(true)
+
       if File.directory?(expanded_path)
         Dir.glob(File.join(expanded_path, '*')).select { |f| File.file?(f) }
       else
@@ -109,6 +111,7 @@ module Agents
 
     def receive(incoming_events)
       return if interpolated['mode'] != 'write' || !should_run?
+
       incoming_events.each do |event|
         mo = interpolated(event)
         expanded_path = File.expand_path(mo['path'])
@@ -153,7 +156,8 @@ module Agents
     class Worker < LongRunnable::Worker
       def setup
         require 'listen'
-        @listener = Listen.to(*listen_options, &method(:callback))
+        path, options = listen_options
+        @listener = Listen.to(path, **options, &method(:callback))
       end
 
       def run
@@ -173,7 +177,7 @@ module Agents
         AgentRunner.with_connection do
           changes.zip([:modified, :added, :removed]).each do |files, event_type|
             files.each do |file|
-              agent.create_event payload: agent.get_file_pointer(file).merge(event_type: event_type)
+              agent.create_event payload: agent.get_file_pointer(file).merge(event_type:)
             end
           end
           agent.touch(:last_check_at)
@@ -182,9 +186,15 @@ module Agents
 
       def listen_options
         if File.directory?(agent.expanded_path)
-          [agent.expanded_path, ignore!: [] ]
+          [
+            agent.expanded_path,
+            ignore!: []
+          ]
         else
-          [File.dirname(agent.expanded_path), { ignore!: [], only: /\A#{Regexp.escape(File.basename(agent.expanded_path))}\z/ } ]
+          [
+            File.dirname(agent.expanded_path),
+            ignore!: [], only: /\A#{Regexp.escape(File.basename(agent.expanded_path))}\z/
+          ]
         end
       end
     end

app/models/agents/manual_event_agent.rb

@@ -3,7 +3,7 @@ module Agents
     cannot_be_scheduled!
     cannot_receive_events!
 
-    description <<-MD
+    description <<~MD
       The Manual Event Agent is used to manually create Events for testing or other purposes.
 
       Connect this Agent to other Agents and create Events using the UI provided on this Agent's Summary page.
@@ -24,15 +24,16 @@ module Agents
       if params['payload']
         json = interpolate_options(JSON.parse(params['payload']))
         if json['payloads'] && (json.keys - ['payloads']).length > 0
-          { :success => false, :error => "If you provide the 'payloads' key, please do not provide any other keys at the top level." }
+          { success: false,
+            error: "If you provide the 'payloads' key, please do not provide any other keys at the top level." }
         else
           [json['payloads'] || json].flatten.each do |payload|
-            create_event(:payload => payload)
+            create_event(payload:)
           end
-          { :success => true }
+          { success: true }
         end
       else
-        { :success => false, :error => "You must provide a JSON payload" }
+        { success: false, error: "You must provide a JSON payload" }
       end
     end
 

app/models/agents/mqtt_agent.rb

@@ -1,11 +1,10 @@
-# encoding: utf-8 
 require "json"
 
 module Agents
   class MqttAgent < Agent
     gem_dependency_check { defined?(MQTT) }
 
-    description <<-MD
+    description <<~MD
       The MQTT Agent allows both publication and subscription to an MQTT topic.
 
       #{'## Include `mqtt` in your Gemfile to use this Agent!' if dependencies_missing?}
@@ -59,19 +58,19 @@ module Agents
       Find out more detail on [subscription wildcards](http://www.eclipse.org/paho/files/mqttdoc/Cclient/wildcard.html)
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       Events are simply nested MQTT payloads. For example, an MQTT payload for Owntracks
 
-      <pre><code>{
-        "topic": "owntracks/kcqlmkgx/Dan",
-        "message": {"_type": "location", "lat": "-34.8493644", "lon": "138.5218119", "tst": "1401771049", "acc": "50.0", "batt": "31", "desc": "Home", "event": "enter"},
-        "time": 1401771051
-      }</code></pre>
+          {
+            "topic": "owntracks/kcqlmkgx/Dan",
+            "message": {"_type": "location", "lat": "-34.8493644", "lon": "138.5218119", "tst": "1401771049", "acc": "50.0", "batt": "31", "desc": "Home", "event": "enter"},
+            "time": 1401771051
+          }
     MD
 
     def validate_options
       unless options['uri'].present? &&
-             options['topic'].present?
+          options['topic'].present?
         errors.add(:base, "topic and uri are required")
       end
     end
@@ -84,7 +83,7 @@ module Agents
       {
         'uri' => 'mqtts://user:pass@localhost:8883',
         'ssl' => :TLSv1,
-        'ca_file'  => './ca.pem',
+        'ca_file' => './ca.pem',
         'cert_file' => './client.crt',
         'key_file' => './client.key',
         'topic' => 'huginn',
@@ -94,16 +93,14 @@ module Agents
     end
 
     def mqtt_client
-      @client ||= begin
-        MQTT::Client.new(interpolated['uri']).tap do |c|
-          if interpolated['ssl']
-            c.ssl = interpolated['ssl'].to_sym
-            c.ca_file = interpolated['ca_file']
-            c.cert_file = interpolated['cert_file']
-            c.key_file = interpolated['key_file']
-          end
+      @client ||= MQTT::Client.new(interpolated['uri']).tap { |c|
+        if interpolated['ssl']
+          c.ssl = interpolated['ssl'].to_sym
+          c.ca_file = interpolated['ca_file']
+          c.cert_file = interpolated['cert_file']
+          c.key_file = interpolated['key_file']
         end
-      end
+      }
     end
 
     def receive(incoming_events)
@@ -114,7 +111,6 @@ module Agents
       end
     end
 
-
     def check
       last_message = memory['last_message']
       mqtt_client.connect
@@ -131,7 +127,7 @@ module Agents
           # A lot of services generate JSON, so try that.
           begin
             payload = JSON.parse(payload)
-          rescue
+          rescue StandardError
           end
 
           create_event payload: {
@@ -151,6 +147,5 @@ module Agents
       self.memory['last_message'] = last_message
       save!
     end
-
   end
 end

app/models/agents/pdf_info_agent.rb

@@ -3,13 +3,12 @@ require 'hypdf'
 
 module Agents
   class PdfInfoAgent < Agent
-
     gem_dependency_check { defined?(HyPDF) }
 
     cannot_be_scheduled!
     no_bulk_receive!
 
-    description <<-MD
+    description <<~MD
       The PDF Info Agent returns the metadata contained within a given PDF file, using HyPDF.
 
       #{'## Include the `hypdf` gem in your `Gemfile` to use PDFInfo Agents.' if dependencies_missing?}
@@ -19,24 +18,24 @@ module Agents
       It works by acting on events that contain a key `url` in their payload, and runs the [pdfinfo](https://devcenter.heroku.com/articles/hypdf#pdfinfo) command on them.
     MD
 
-    event_description <<-MD
-    This will change based on the metadata in the pdf.
-
-      { "Title"=>"Everyday Rails Testing with RSpec", 
-        "Author"=>"Aaron Sumner",
-        "Creator"=>"LaTeX with hyperref package",
-        "Producer"=>"xdvipdfmx (0.7.8)",
-        "CreationDate"=>"Fri Aug  2 05",
-        "32"=>"50 2013",
-        "Tagged"=>"no",
-        "Pages"=>"150",
-        "Encrypted"=>"no",
-        "Page size"=>"612 x 792 pts (letter)",
-        "Optimized"=>"no",
-        "PDF version"=>"1.5",
-        "url": "your url"
-      }
-    MD
+    event_description do
+      "This will change based on the metadata in the pdf.\n\n    " +
+        Utils.pretty_print({
+          "Title" => "Everyday Rails Testing with RSpec",
+          "Author" => "Aaron Sumner",
+          "Creator" => "LaTeX with hyperref package",
+          "Producer" => "xdvipdfmx (0.7.8)",
+          "CreationDate" => "Fri Aug  2 05",
+          "32" => "50 2013",
+          "Tagged" => "no",
+          "Pages" => "150",
+          "Encrypted" => "no",
+          "Page size" => "612 x 792 pts (letter)",
+          "Optimized" => "no",
+          "PDF version" => "1.5",
+          "url": "your url"
+        })
+    end
 
     def working?
       !recent_error_logs?
@@ -57,12 +56,12 @@ module Agents
 
     def check_url(in_url, payload)
       return unless in_url.present?
+
       Array(in_url).each do |url|
         log "Fetching #{url}"
         info = HyPDF.pdfinfo(open(url))
-        create_event :payload => info.merge(payload)
+        create_event payload: info.merge(payload)
       end
     end
-
   end
 end

app/models/agents/peak_detector_agent.rb

@@ -1,12 +1,10 @@
-require 'pp'
-
 module Agents
   class PeakDetectorAgent < Agent
     cannot_be_scheduled!
 
     DEFAULT_SEARCH_URL = 'https://twitter.com/search?q={q}'
 
-    description <<-MD
+    description <<~MD
       The Peak Detector Agent will watch for peaks in an event stream.  When a peak is detected, the resulting Event will have a payload message of `message`.  You can include extractions in the message, for example: `I saw a bar of: {{foo.bar}}`, have a look at the [Wiki](https://github.com/huginn/huginn/wiki/Formatting-Events-using-Liquid) for details.
 
       The `value_path` value is a [JSONPath](http://goessner.net/articles/JsonPath/) to the value of interest.  `group_by_path` is a JSONPath that will be used to group values, if present.
@@ -20,7 +18,7 @@ module Agents
       You may set `search_url` to point to something else than Twitter search, using the URI Template syntax defined in [RFC 6570](https://tools.ietf.org/html/rfc6570). Default value is `#{DEFAULT_SEARCH_URL}` where `{q}` will be replaced with group name.
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       Events look like:
 
           {
@@ -37,7 +35,7 @@ module Agents
       end
       begin
         tmpl = search_url
-      rescue => e
+      rescue StandardError => e
         errors.add(:base, "search_url must be a valid URI template: #{e.message}")
       else
         unless tmpl.keys.include?('q')
@@ -81,13 +79,18 @@ module Agents
       return if memory['data'][group].length <= options['min_events'].to_i
 
       if memory['peaks'][group].empty? || memory['peaks'][group].last < event.created_at.to_i - peak_spacing
-        average_value, standard_deviation = stats_for(group, :skip_last => 1)
+        average_value, standard_deviation = stats_for(group, skip_last: 1)
         newest_value, newest_time = memory['data'][group][-1].map(&:to_f)
 
         if newest_value > average_value + std_multiple * standard_deviation
           memory['peaks'][group] << newest_time
           memory['peaks'][group].reject! { |p| p <= newest_time - window_duration }
-          create_event :payload => { 'message' => interpolated(event)['message'], 'peak' => newest_value, 'peak_time' => newest_time, 'grouped_by' => group.to_s }
+          create_event payload: {
+            'message' => interpolated(event)['message'],
+            'peak' => newest_value,
+            'peak_time' => newest_time,
+            'grouped_by' => group.to_s
+          }
         end
       end
     end
@@ -132,19 +135,20 @@ module Agents
     end
 
     def group_for(event)
-      ((interpolated['group_by_path'].present? && Utils.value_at(event.payload, interpolated['group_by_path'])) || 'no_group')
+      group_by_path = interpolated['group_by_path'].presence
+      (group_by_path && Utils.value_at(event.payload, group_by_path)) || 'no_group'
     end
 
     def remember(group, event)
       memory['data'] ||= {}
       memory['data'][group] ||= []
-      memory['data'][group] << [ Utils.value_at(event.payload, interpolated['value_path']).to_f, event.created_at.to_i ]
+      memory['data'][group] << [Utils.value_at(event.payload, interpolated['value_path']).to_f, event.created_at.to_i]
       cleanup group
     end
 
     def cleanup(group)
       newest_time = memory['data'][group].last.last
-      memory['data'][group].reject! { |value, time| time <= newest_time - window_duration }
+      memory['data'][group].reject! { |_value, time| time <= newest_time - window_duration }
     end
   end
 end

app/models/agents/phantom_js_cloud_agent.rb

@@ -11,7 +11,7 @@ module Agents
 
     default_schedule 'every_12h'
 
-    description <<-MD
+    description <<~MD
       This Agent generates [PhantomJs Cloud](https://phantomjscloud.com/) URLs that can be used to render JavaScript-heavy webpages for content extraction.
 
       URLs generated by this Agent are formulated in accordance with the [PhantomJs Cloud API](https://phantomjscloud.com/docs/index.html).
@@ -37,8 +37,9 @@ module Agents
       As this agent only provides a limited subset of the most commonly used options, you can follow [this guide](https://github.com/huginn/huginn/wiki/Browser-Emulation-Using-PhantomJS-Cloud) to make full use of additional options PhantomJsCloud provides.
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       Events look like this:
+
           {
             "url": "..."
           }

app/models/agents/post_agent.rb

@@ -13,7 +13,7 @@ module Agents
     default_schedule "never"
 
     description do
-      <<-MD
+      <<~MD
         A Post Agent receives events from other agents (or runs periodically), merges those events with the [Liquid-interpolated](https://github.com/huginn/huginn/wiki/Formatting-Events-using-Liquid) contents of `payload`, and sends the results as POST (or GET) requests to a specified url.  To skip merging in the incoming event, but still send the interpolated payload, set `no_merge` to `true`.
 
         The `post_url` field must specify where you would like to send requests. Please include the URI scheme (`http` or `https`).
@@ -56,16 +56,17 @@ module Agents
       MD
     end
 
-    event_description <<-MD
+    event_description <<~MD
       Events look like this:
-        {
-          "status": 200,
-          "headers": {
-            "Content-Type": "text/html",
-            ...
-          },
-          "body": "<html>Some data...</html>"
-        }
+
+          {
+            "status": 200,
+            "headers": {
+              "Content-Type": "text/html",
+              ...
+            },
+            "body": "<html>Some data...</html>"
+          }
 
       Original event contents will be merged when `output_mode` is set to `merge`.
     MD
@@ -89,7 +90,7 @@ module Agents
 
     def working?
       return false if recent_error_logs?
-      
+
       if interpolated['expected_receive_period_in_days'].present?
         return false unless last_receive_at && last_receive_at > interpolated['expected_receive_period_in_days'].to_i.days.ago
       end
@@ -106,7 +107,8 @@ module Agents
         errors.add(:base, "post_url is a required field")
       end
 
-      if options['payload'].present? && %w[get delete].include?(method) && !(options['payload'].is_a?(Hash) || options['payload'].is_a?(Array))
+      if options['payload'].present? && %w[get
+                                           delete].include?(method) && !(options['payload'].is_a?(Hash) || options['payload'].is_a?(Array))
         errors.add(:base, "if provided, payload must be a hash or an array")
       end
 
@@ -134,11 +136,11 @@ module Agents
         errors.add(:base, "method must be 'post', 'get', 'put', 'delete', or 'patch'")
       end
 
-      if options['no_merge'].present? && !%[true false].include?(options['no_merge'].to_s)
+      if options['no_merge'].present? && !%(true false).include?(options['no_merge'].to_s)
         errors.add(:base, "if provided, no_merge must be 'true' or 'false'")
       end
 
-      if options['output_mode'].present? && !options['output_mode'].to_s.include?('{') && !%[clean merge].include?(options['output_mode'].to_s)
+      if options['output_mode'].present? && !options['output_mode'].to_s.include?('{') && !%(clean merge).include?(options['output_mode'].to_s)
         errors.add(:base, "if provided, output_mode must be 'clean' or 'merge'")
       end
 
@@ -173,7 +175,8 @@ module Agents
 
       case method
       when 'get', 'delete'
-        params, body = data, nil
+        params = data
+        body = nil
       when 'post', 'put', 'patch'
         params = nil
 

app/models/agents/public_transport_agent.rb

@@ -6,7 +6,7 @@ module Agents
 
     default_schedule "every_2m"
 
-    description <<-MD
+    description <<~MD
       The Public Transport Request Agent generates Events based on NextBus GPS transit predictions.
 
       Specify the following user settings:
@@ -17,7 +17,7 @@ module Agents
 
       First, select an agency by visiting [http://www.nextbus.com/predictor/adaAgency.jsp](http://www.nextbus.com/predictor/adaAgency.jsp) and finding your transit system.  Once you find it, copy the part of the URL after `?a=`.  For example, for the San Francisco MUNI system, you would end up on [http://www.nextbus.com/predictor/adaDirection.jsp?a=**sf-muni**](http://www.nextbus.com/predictor/adaDirection.jsp?a=sf-muni) and copy "sf-muni".  Put that into this Agent's agency setting.
 
-      Next, find the stop tags that you care about. 
+      Next, find the stop tags that you care about.
 
       Select your destination and lets use the n-judah route. The link should be [http://www.nextbus.com/predictor/adaStop.jsp?a=sf-muni&r=N](http://www.nextbus.com/predictor/adaStop.jsp?a=sf-muni&r=N) Once you find it, copy the part of the URL after `r=`.
 
@@ -42,18 +42,22 @@ module Agents
           alert_window_in_minutes: 5
     MD
 
-    event_description <<-MD
-    Events look like this:
-      { "routeTitle":"N-Judah",
-        "stopTag":"5215",
-        "prediction":
-           {"epochTime":"1389622846689",
-            "seconds":"3454","minutes":"57","isDeparture":"false",
-            "affectedByLayover":"true","dirTag":"N__OB4KJU","vehicle":"1489",
-            "block":"9709","tripTag":"5840086"
-            }
-      }
-    MD
+    event_description "Events look like this:\n\n    " +
+      Utils.pretty_print({
+        "routeTitle": "N-Judah",
+        "stopTag": "5215",
+        "prediction": {
+          "epochTime": "1389622846689",
+          "seconds": "3454",
+          "minutes": "57",
+          "isDeparture": "false",
+          "affectedByLayover": "true",
+          "dirTag": "N__OB4KJU",
+          "vehicle": "1489",
+          "block": "9709",
+          "tripTag": "5840086"
+        }
+      })
 
     def check_url
       query = URI.encode_www_form([
@@ -65,27 +69,27 @@ module Agents
     end
 
     def stops
-      interpolated["stops"].collect{|a| a.split("|").last}
+      interpolated["stops"].collect { |a| a.split("|").last }
     end
 
     def check
       hydra = Typhoeus::Hydra.new
-      request = Typhoeus::Request.new(check_url, :followlocation => true)
+      request = Typhoeus::Request.new(check_url, followlocation: true)
       request.on_success do |response|
         page = Nokogiri::XML response.body
         predictions = page.css("//prediction")
         predictions.each do |pr|
           parent = pr.parent.parent
-          vals = {"routeTitle" => parent["routeTitle"], "stopTag" => parent["stopTag"]}
-          if pr["minutes"] && pr["minutes"].to_i < interpolated["alert_window_in_minutes"].to_i
-            vals = vals.merge Hash.from_xml(pr.to_xml)
-            if not_already_in_memory?(vals)
-              create_event(:payload => vals)
-              log "creating event..."
-              update_memory(vals)
-            else
-              log "not creating event since already in memory"
-            end
+          vals = { "routeTitle" => parent["routeTitle"], "stopTag" => parent["stopTag"] }
+          next unless pr["minutes"] && pr["minutes"].to_i < interpolated["alert_window_in_minutes"].to_i
+
+          vals = vals.merge Hash.from_xml(pr.to_xml)
+          if not_already_in_memory?(vals)
+            create_event(payload: vals)
+            log "creating event..."
+            update_memory(vals)
+          else
+            log "not creating event since already in memory"
           end
         end
       end
@@ -100,20 +104,26 @@ module Agents
 
     def cleanup_old_memory
       self.memory["existing_routes"] ||= []
-      self.memory["existing_routes"].reject!{|h| h["currentTime"].to_time <= (Time.now - 2.hours)}
+      time = 2.hours.ago
+      self.memory["existing_routes"].reject! { |h| h["currentTime"].to_time <= time }
     end
 
     def add_to_memory(vals)
-      self.memory["existing_routes"] ||= []
-      self.memory["existing_routes"] << {"stopTag" => vals["stopTag"], "tripTag" => vals["prediction"]["tripTag"], "epochTime" => vals["prediction"]["epochTime"], "currentTime" => Time.now}
+      (self.memory["existing_routes"] ||= []) << {
+        "stopTag" => vals["stopTag"],
+        "tripTag" => vals["prediction"]["tripTag"],
+        "epochTime" => vals["prediction"]["epochTime"],
+        "currentTime" => Time.now
+      }
     end
 
     def not_already_in_memory?(vals)
       m = self.memory["existing_routes"] || []
-      m.select{|h| h['stopTag'] == vals["stopTag"] &&
-                h['tripTag'] == vals["prediction"]["tripTag"] &&
-                h['epochTime'] == vals["prediction"]["epochTime"]
-              }.count == 0
+      m.select { |h|
+        h['stopTag'] == vals["stopTag"] &&
+          h['tripTag'] == vals["prediction"]["tripTag"] &&
+          h['epochTime'] == vals["prediction"]["epochTime"]
+      }.count == 0
     end
 
     def default_options

app/models/agents/pushbullet_agent.rb

@@ -10,13 +10,13 @@ module Agents
 
     API_BASE = 'https://api.pushbullet.com/v2/'
     TYPE_TO_ATTRIBUTES = {
-            'note'    => [:title, :body],
-            'link'    => [:title, :body, :url],
-            'address' => [:name, :address]
+      'note' => [:title, :body],
+      'link' => [:title, :body, :url],
+      'address' => [:name, :address]
     }
     class Unauthorized < StandardError; end
 
-    description <<-MD
+    description <<~MD
       The Pushbullet agent sends pushes to a pushbullet device
 
       To authenticate you need to either the `api_key` or create a `pushbullet_api_key` credential, you can find yours at your account page:
@@ -60,9 +60,11 @@ module Agents
     def validate_options
       errors.add(:base, "you need to specify a pushbullet api_key") if options['api_key'].blank?
       errors.add(:base, "you need to specify a device_id") if options['device_id'].blank?
-      errors.add(:base, "you need to specify a valid message type") if options['type'].blank? or not ['note', 'link', 'address'].include?(options['type'])
+      errors.add(:base, "you need to specify a valid message type") if options['type'].blank? ||
+        !['note', 'link', 'address'].include?(options['type'])
       TYPE_TO_ATTRIBUTES[options['type']].each do |attr|
-        errors.add(:base, "you need to specify '#{attr.to_s}' for the type '#{options['type']}'") if options[attr].blank?
+        errors.add(:base,
+                   "you need to specify '#{attr}' for the type '#{options['type']}'") if options[attr].blank?
       end
     end
 
@@ -75,7 +77,7 @@ module Agents
 
     def complete_device_id
       devices
-        .map { |d| {text: d['nickname'], id: d['iden']} }
+        .map { |d| { text: d['nickname'], id: d['iden'] } }
         .unshift(text: 'All Devices', id: '__ALL__')
     end
 
@@ -92,6 +94,7 @@ module Agents
     end
 
     private
+
     def safely
       yield
     rescue Unauthorized => e
@@ -101,6 +104,7 @@ module Agents
     def request(http_method, method, options)
       response = JSON.parse(HTTParty.send(http_method, API_BASE + method, options).body)
       raise Unauthorized, response['error']['message'] if response['error'].present?
+
       response
     end
 
@@ -113,20 +117,21 @@ module Agents
 
     def create_device
       return if options['device_id'].present?
+
       safely do
-        response = request(:post, 'devices', basic_auth.merge(body: {nickname: 'Huginn', type: 'stream'}))
+        response = request(:post, 'devices', basic_auth.merge(body: { nickname: 'Huginn', type: 'stream' }))
         self.options[:device_id] = response['iden']
       end
     end
 
     def basic_auth
-      {basic_auth: {username: interpolated[:api_key].presence || credential('pushbullet_api_key'), password: ''}}
+      { basic_auth: { username: interpolated[:api_key].presence || credential('pushbullet_api_key'), password: '' } }
     end
 
     def query_options(event)
       mo = interpolated(event)
       dev_ident = mo[:device_id] == "__ALL__" ? '' : mo[:device_id]
-      basic_auth.merge(body: {device_iden: dev_ident, type: mo[:type]}.merge(payload(mo)))
+      basic_auth.merge(body: { device_iden: dev_ident, type: mo[:type] }.merge(payload(mo)))
     end
 
     def payload(mo)

app/models/agents/pushover_agent.rb

@@ -5,10 +5,9 @@ module Agents
     cannot_create_events!
     no_bulk_receive!
 
-
     API_URL = 'https://api.pushover.net/1/messages.json'
 
-    description <<-MD
+    description <<~MD
       The Pushover Agent receives and collects events and sends them via push notification to a user/group.
 
       **You need a Pushover API Token:** [https://pushover.net/apps/build](https://pushover.net/apps/build)
@@ -88,15 +87,15 @@ module Agents
             retry
             expire
           ].each do |key|
-            if value = String.try_convert(interpolated[key].presence)
-              case key
-              when 'url'
-                value.slice!(512..-1)
-              when 'url_title'
-                value.slice!(100..-1)
-              end
-              post_params[key] = value
+            value = String.try_convert(interpolated[key].presence) or next
+
+            case key
+            when 'url'
+              value.slice!(512..-1)
+            when 'url_title'
+              value.slice!(100..-1)
             end
+            post_params[key] = value
           end
           # html is special because String.try_convert(true) gives nil (not even "nil", just nil)
           if value = interpolated['html'].presence

app/models/agents/read_file_agent.rb

@@ -13,7 +13,7 @@ module Agents
     end
 
     description do
-      <<-MD
+      <<~MD
         The ReadFileAgent takes events from `FileHandling` agents, reads the file, and emits the contents as a string.
 
         `data_key` specifies the key of the emitted event which contains the file contents.
@@ -22,10 +22,12 @@ module Agents
       MD
     end
 
-    event_description <<-MD
-      {
-        "data" => '...'
-      }
+    event_description <<~MD
+      Events look like:
+
+          {
+            "data" => '...'
+          }
     MD
 
     form_configurable :data_key, type: :string
@@ -43,6 +45,7 @@ module Agents
     def receive(incoming_events)
       incoming_events.each do |event|
         next unless io = get_io(event)
+
         create_event payload: { interpolated['data_key'] => io.read }
       end
     end

app/models/agents/rss_agent.rb

@@ -11,7 +11,7 @@ module Agents
     DEFAULT_EVENTS_ORDER = [['{{date_published}}', 'time'], ['{{last_updated}}', 'time']]
 
     description do
-      <<-MD
+      <<~MD
         The RSS Agent consumes RSS feeds and emits events when they change.
 
         This agent, using [Feedjira](https://github.com/feedjira/feedjira) as a base, can parse various types of RSS and Atom feeds and has some special handlers for FeedBurner, iTunes RSS, and so on.  However, supported fields are limited by its general and abstract nature.  For complex feeds with additional field types, we recommend using a WebsiteAgent.  See [this example](https://github.com/huginn/huginn/wiki/Agent-configuration-examples#itunes-trailers).
@@ -49,7 +49,7 @@ module Agents
       }
     end
 
-    event_description <<-MD
+    event_description <<~MD
       Events look like:
 
           {
@@ -131,11 +131,13 @@ module Agents
       errors.add(:base, "url is required") unless options['url'].present?
 
       unless options['expected_update_period_in_days'].present? && options['expected_update_period_in_days'].to_i > 0
-        errors.add(:base, "Please provide 'expected_update_period_in_days' to indicate how many days can pass without an update before this Agent is considered to not be working")
+        errors.add(:base,
+                   "Please provide 'expected_update_period_in_days' to indicate how many days can pass without an update before this Agent is considered to not be working")
       end
 
       if options['remembered_id_count'].present? && options['remembered_id_count'].to_i < 1
-        errors.add(:base, "Please provide 'remembered_id_count' as a number bigger than 0 indicating how many IDs should be saved to distinguish between new and old IDs in RSS feeds. Delete option to use default (500).")
+        errors.add(:base,
+                   "Please provide 'remembered_id_count' as a number bigger than 0 indicating how many IDs should be saved to distinguish between new and old IDs in RSS feeds. Delete option to use default (500).")
       end
 
       validate_web_request_options!
@@ -161,17 +163,15 @@ module Agents
       max_events = (interpolated['max_events_per_run'].presence || 0).to_i
 
       urls.each do |url|
-        begin
-          response = faraday.get(url)
-          if response.success?
-            feed = Feedjira.parse(preprocessed_body(response))
-            new_events.concat feed_to_events(feed)
-          else
-            error "Failed to fetch #{url}: #{response.inspect}"
-          end
-        rescue => e
-          error "Failed to fetch #{url} with message '#{e.message}': #{e.backtrace}"
+        response = faraday.get(url)
+        if response.success?
+          feed = Feedjira.parse(preprocessed_body(response))
+          new_events.concat feed_to_events(feed)
+        else
+          error "Failed to fetch #{url}: #{response.inspect}"
         end
+      rescue StandardError => e
+        error "Failed to fetch #{url} with message '#{e.message}': #{e.backtrace}"
       end
 
       events = sort_events(new_events).select.with_index { |event, index|
@@ -210,7 +210,8 @@ module Agents
       else
         # Encoding is already known, so do not let the parser detect
         # it from the XML declaration in the content.
-        body.sub!(/(?<noenc>\A\u{FEFF}?\s*<\?xml(?:\s+\w+(?<av>\s*=\s*(?:'[^']*'|"[^"]*")))*?)\s+encoding\g<av>/, '\\k<noenc>')
+        body.sub!(/(?<noenc>\A\u{FEFF}?\s*<\?xml(?:\s+\w+(?<av>\s*=\s*(?:'[^']*'|"[^"]*")))*?)\s+encoding\g<av>/,
+                  '\\k<noenc>')
       end
       body
     end
@@ -226,7 +227,7 @@ module Agents
 
       {
         id: feed.feed_id,
-        type: type,
+        type:,
         url: feed.url,
         links: feed.links,
         title: feed.title,
@@ -257,15 +258,15 @@ module Agents
           itunes_summary
           language
         ].each { |attr|
-          if value = feed.try(attr).presence
-            data[attr] =
-              case attr
-              when :itunes_summary
-                clean_fragment(value)
-              else
-                value
-              end
-          end
+          next unless value = feed.try(attr).presence
+
+          data[attr] =
+            case attr
+            when :itunes_summary
+              clean_fragment(value)
+            else
+              value
+            end
         }
       end
       data

app/models/agents/s3_agent.rb

@@ -11,7 +11,7 @@ module Agents
     gem_dependency_check { defined?(Aws::S3) }
 
     description do
-      <<-MD
+      <<~MD
         The S3Agent can watch a bucket for changes or emit an event for every file in that bucket. When receiving events, it writes the data into a file on S3.
 
         #{'## Include `aws-sdk-core` in your Gemfile to use this Agent!' if dependencies_missing?}
@@ -41,22 +41,23 @@ module Agents
     end
 
     event_description do
-      "Events will looks like this:\n\n    %s" % if boolify(interpolated['watch'])
-        Utils.pretty_print({
-          "file_pointer" => {
-            "file" => "filename",
-            "agent_id" => id
-          },
-          "event_type" => "modified/added/removed"
-        })
-      else
-        Utils.pretty_print({
-          "file_pointer" => {
-            "file" => "filename",
-            "agent_id" => id
-          }
-        })
-      end
+      "Events will looks like this:\n\n    " +
+        if boolify(interpolated['watch'])
+          Utils.pretty_print({
+            "file_pointer" => {
+              "file" => "filename",
+              "agent_id" => id
+            },
+            "event_type" => "modified/added/removed"
+          })
+        else
+          Utils.pretty_print({
+            "file_pointer" => {
+              "file" => "filename",
+              "agent_id" => id
+            }
+          })
+        end
     end
 
     def default_options
@@ -70,11 +71,12 @@ module Agents
       }
     end
 
-    form_configurable :mode, type: :array, values: %w(read write)
+    form_configurable :mode, type: :array, values: %w[read write]
     form_configurable :access_key_id, roles: :validatable
     form_configurable :access_key_secret, roles: :validatable
-    form_configurable :region, type: :array, values: %w(us-east-1 us-west-1 us-west-2 eu-west-1 eu-central-1 ap-southeast-1 ap-southeast-2 ap-northeast-1 ap-northeast-2 sa-east-1)
-    form_configurable :watch, type: :array, values: %w(true false)
+    form_configurable :region, type: :array,
+                               values: %w[us-east-1 us-west-1 us-west-2 eu-west-1 eu-central-1 ap-southeast-1 ap-southeast-2 ap-northeast-1 ap-northeast-2 sa-east-1]
+    form_configurable :watch, type: :array, values: %w[true false]
     form_configurable :bucket, roles: :completable
     form_configurable :filename
     form_configurable :data
@@ -114,7 +116,7 @@ module Agents
     end
 
     def complete_bucket
-      (buckets || []).collect { |room| {text: room.name, id: room.name} }
+      (buckets || []).collect { |room| { text: room.name, id: room.name } }
     end
 
     def working?
@@ -123,9 +125,10 @@ module Agents
 
     def check
       return if interpolated['mode'] != 'read'
+
       contents = safely do
-                   get_bucket_contents
-                 end
+        get_bucket_contents
+      end
       if boolify(interpolated['watch'])
         watch(contents)
       else
@@ -141,6 +144,7 @@ module Agents
 
     def receive(incoming_events)
       return if interpolated['mode'] != 'write'
+
       incoming_events.each do |event|
         safely do
           mo = interpolated(event)
@@ -155,7 +159,7 @@ module Agents
       yield
     rescue Aws::S3::Errors::AccessDenied => e
       error("Could not access '#{interpolated['bucket']}' #{e.class} #{e.message}")
-    rescue Aws::S3::Errors::ServiceError =>e
+    rescue Aws::S3::Errors::ServiceError => e
       error("#{e.class}: #{e.message}")
     end
 
@@ -175,7 +179,7 @@ module Agents
         end
         contents.delete(key)
       end
-      contents.each do |key, etag|
+      contents.each do |key, _etag|
         create_event payload: get_file_pointer(key).merge(event_type: :added)
       end
 

app/models/agents/scheduler_agent.rb

@@ -12,7 +12,7 @@ module Agents
 
     cattr_reader :second_precision_enabled
 
-    description <<-MD
+    description <<~MD
       The Scheduler Agent periodically takes an action on target Agents according to a user-defined schedule.
 
       # Action types

app/models/agents/sentiment_agent.rb

@@ -6,7 +6,7 @@ module Agents
 
     cannot_be_scheduled!
 
-    description <<-MD
+    description <<~MD
       The Sentiment Agent generates `good-bad` (psychological valence or happiness index), `active-passive` (arousal), and  `strong-weak` (dominance) score. It will output a value between 1 and 9. It will only work on English content.
 
       Make sure the content this agent is analyzing is of sufficient length to get respectable results.
@@ -14,7 +14,7 @@ module Agents
       Provide a JSONPath in `content` field where content is residing and set `expected_receive_period_in_days` to the maximum number of days you would allow to be passed between events being received by this agent.
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       Events look like:
 
           {
@@ -41,17 +41,22 @@ module Agents
       incoming_events.each do |event|
         Utils.values_at(event.payload, interpolated['content']).each do |content|
           sent_values = sentiment_values anew, content
-          create_event :payload => { 'content' => content,
-                                     'valence' => sent_values[0],
-                                     'arousal' => sent_values[1],
-                                     'dominance' => sent_values[2],
-                                     'original_event' => event.payload }
+          create_event payload: {
+            'content' => content,
+            'valence' => sent_values[0],
+            'arousal' => sent_values[1],
+            'dominance' => sent_values[2],
+            'original_event' => event.payload
+          }
         end
       end
     end
 
     def validate_options
-      errors.add(:base, "content and expected_receive_period_in_days must be present") unless options['content'].present? && options['expected_receive_period_in_days'].present?
+      errors.add(
+        :base,
+        "content and expected_receive_period_in_days must be present"
+      ) unless options['content'].present? && options['expected_receive_period_in_days'].present?
     end
 
     def self.sentiment_hash
@@ -67,18 +72,18 @@ module Agents
     def sentiment_values(anew, text)
       valence, arousal, dominance, freq = [0] * 4
       text.downcase.strip.gsub(/[^a-z ]/, "").split.each do |word|
-        if anew.has_key? word
-          valence += anew[word][0]
-          arousal += anew[word][1]
-          dominance += anew[word][2]
-          freq += 1
-        end
+        next unless anew.has_key? word
+
+        valence += anew[word][0]
+        arousal += anew[word][1]
+        dominance += anew[word][2]
+        freq += 1
       end
       if valence != 0
-        [valence/freq, arousal/freq, dominance/freq]
+        [valence / freq, arousal / freq, dominance / freq]
       else
         ["Insufficient data for meaningful answer"] * 3
       end
     end
   end
-end
+end

app/models/agents/shell_command_agent.rb

@@ -5,12 +5,11 @@ module Agents
     can_dry_run!
     no_bulk_receive!
 
-
     def self.should_run?
       ENV['ENABLE_INSECURE_AGENTS'] == "true"
     end
 
-    description <<-MD
+    description <<~MD
       The Shell Command Agent will execute commands on your local system, returning the output.
 
       `command` specifies the command (either a shell command line string or an array of command line arguments) to be executed, and `path` will tell ShellCommandAgent in what directory to run this command.  The content of `stdin` will be fed to the command via the standard input.
@@ -33,26 +32,26 @@ module Agents
       You can enable this Agent in your .env file by setting `ENABLE_INSECURE_AGENTS` to `true`.
     MD
 
-    event_description <<-MD
-    Events look like this:
+    event_description <<~MD
+      Events look like this:
 
-        {
-          "command": "pwd",
-          "path": "/home/Huginn",
-          "exit_status": 0,
-          "errors": "",
-          "output": "/home/Huginn"
-        }
+          {
+            "command": "pwd",
+            "path": "/home/Huginn",
+            "exit_status": 0,
+            "errors": "",
+            "output": "/home/Huginn"
+          }
     MD
 
     def default_options
       {
-          'path' => "/",
-          'command' => "pwd",
-          'unbundle' => false,
-          'suppress_on_failure' => false,
-          'suppress_on_empty_output' => false,
-          'expected_update_period_in_days' => 1
+        'path' => "/",
+        'command' => "pwd",
+        'unbundle' => false,
+        'suppress_on_failure' => false,
+        'suppress_on_empty_output' => false,
+        'expected_update_period_in_days' => 1
       }
     end
 
@@ -98,7 +97,7 @@ module Agents
         path = opts['path']
         stdin = opts['stdin']
 
-        result, errors, exit_status = run_command(path, command, stdin, interpolated.slice(:unbundle).symbolize_keys)
+        result, errors, exit_status = run_command(path, command, stdin, **interpolated.slice(:unbundle).symbolize_keys)
 
         payload = {
           'command' => command,
@@ -109,7 +108,7 @@ module Agents
         }
 
         unless suppress_event?(payload)
-          created_event = create_event payload: payload
+          created_event = create_event(payload:)
         end
 
         log("Ran '#{command}' under '#{path}'", outbound_event: created_event, inbound_event: event)
@@ -146,7 +145,7 @@ module Agents
 
         _, status = Process.wait2(pid)
         exit_status = status.exitstatus
-      rescue => e
+      rescue StandardError => e
         errors = e.to_s
         result = ''.freeze
         exit_status = nil

app/models/agents/slack_agent.rb

@@ -10,7 +10,7 @@ module Agents
 
     gem_dependency_check { defined?(Slack) }
 
-    description <<-MD
+    description <<~MD
       The Slack Agent lets you receive events and send notifications to [Slack](https://slack.com/).
 
       #{'## Include `slack-notifier` in your Gemfile to use this Agent!' if dependencies_missing?}
@@ -38,7 +38,7 @@ module Agents
 
     def validate_options
       unless options['webhook_url'].present? ||
-             (options['auth_token'].present? && options['team_name'].present?)  # compatibility
+          (options['auth_token'].present? && options['team_name'].present?)  # compatibility
         errors.add(:base, "webhook_url is required")
       end
 
@@ -65,11 +65,11 @@ module Agents
     end
 
     def slack_notifier
-      @slack_notifier ||= Slack::Notifier.new(webhook_url, username: username)
+      @slack_notifier ||= Slack::Notifier.new(webhook_url, username:)
     end
 
     def filter_options(opts)
-      opts.select { |key, value| ALLOWED_PARAMS.include? key }.symbolize_keys
+      opts.select { |key, _value| ALLOWED_PARAMS.include? key }.symbolize_keys
     end
 
     def receive(incoming_events)

app/models/agents/stubhub_agent.rb

@@ -2,24 +2,25 @@ module Agents
   class StubhubAgent < Agent
     cannot_receive_events!
 
-    description <<-MD
+    description <<~MD
       The StubHub Agent creates an event for a given StubHub Event.
 
       It can be used to track how many tickets are available for the event and the minimum and maximum price. All that is required is that you paste in the url from the actual event, e.g. https://www.stubhub.com/outside-lands-music-festival-tickets/outside-lands-music-festival-3-day-pass-san-francisco-golden-gate-park-polo-fields-8-8-2014-9020701/
     MD
 
-    event_description <<-MD
+    event_description <<~MD
       Events looks like this:
-        {
-          "url": "https://stubhub.com/valid-event-url"
-          "name": "Event Name"
-          "date": "2014-08-01"
-          "max_price": "999.99"
-          "min_price": "100.99"
-          "total_postings": "50"
-          "total_tickets": "150"
-          "venue_name": "Venue Name"
-        }
+
+          {
+            "url": "https://stubhub.com/valid-event-url"
+            "name": "Event Name"
+            "date": "2014-08-01"
+            "max_price": "999.99"
+            "min_price": "100.99"
+            "total_postings": "50"
+            "total_tickets": "150"
+            "venue_name": "Venue Name"
+          }
     MD
 
     default_schedule "every_1d"
@@ -29,7 +30,7 @@ module Agents
     end
 
     def default_options
-      { 'url' =>  'https://stubhub.com/enter-your-event-here' }
+      { 'url' => 'https://stubhub.com/enter-your-event-here' }
     end
 
     def validate_options
@@ -41,7 +42,7 @@ module Agents
     end
 
     def check
-      create_event :payload => fetch_stubhub_data(url)
+      create_event payload: fetch_stubhub_data(url)
     end
 
     def fetch_stubhub_data(url)
@@ -49,7 +50,6 @@ module Agents
     end
 
     class StubhubFetcher
-
       def self.call(url)
         new(url).fields
       end
@@ -63,7 +63,7 @@ module Agents
       end
 
       def base_url
-       'https://www.stubhub.com/listingCatalog/select/?q='
+        'https://www.stubhub.com/listingCatalog/select/?q='
       end
 
       def build_url
@@ -96,7 +96,6 @@ module Agents
       private
 
       attr_reader :url
-
     end
   end
 end