doing 2.1.89 → 2.1.90

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e585a15c1f864107c306b3d5ccc3f1bdd17fd2ef2bed91ff8af739298bdbc084
-  data.tar.gz: c0f4fe1eb92de431853688d51b6ecb648974493b00d754cbe6d006d3831f8447
+  metadata.gz: 2b48179e41ac5f44a285eed64eb39f8a32b59ec4604a97c1aa50cc1f5c8ffa59
+  data.tar.gz: dd306d8216ee4510a4333572fd7cfdcd8f6d6f77a576fc452da0240f278c3259
 SHA512:
-  metadata.gz: ec2970300b08bab22fe623afef0b9a16fc7a76d7b2ef6a006ca6cdb2005a12d85957f2b0cd05f9e794899f9d5efc35f1886bb5708c782129af7320ea92552dfc
-  data.tar.gz: 38c648f42239d7cc7b8e158fc9b0a5791c4a65eb05e61ff278dd54a6e69b1ae28cacdbe28aa77c0bb60f6a61f63e63c3378cd0202d5e91319860404b303af2ef
+  metadata.gz: 87bcc1db3cbd88ed8c8375a13f0b39f79c58b51d37add223088fcbe71a44b04574b7e88a17b56bd627ede0dbd0ee4a36b6b6c6f56f1430eb6e26effc744dd998
+  data.tar.gz: f24b726c9058732c25692214fc6aecdc53c8b08fd61fa0122d5970483c6572218b5ca60321ee8df3fd384765841a4e4d26ea9664a40bb08331b9072a5cc98750

data/.irbrc

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'lib/doing'
 include Doing
 

data/.rubocop.yml

@@ -0,0 +1 @@
+inherit_from: .rubocop_todo.yml

data/.rubocop_todo.yml

@@ -0,0 +1,251 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2025-10-23 11:08:49 UTC using RuboCop version 1.75.7.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 2
+# Configuration parameters: Severity, Include.
+# Include: **/*.gemspec
+Gemspec/RequiredRubyVersion:
+  Exclude:
+    - 'doing.gemspec'
+    - 'pkg/doing-2.1.89/doing.gemspec'
+
+# Offense count: 2
+# This cop supports safe autocorrection (--autocorrect).
+Layout/MultilineBlockLayout:
+  Exclude:
+    - 'lib/doing/wwid/interactive.rb'
+    - 'pkg/doing-2.1.89/lib/doing/wwid/interactive.rb'
+
+# Offense count: 2
+Lint/DuplicateMethods:
+  Exclude:
+    - 'lib/doing/wwid/wwid.rb'
+    - 'pkg/doing-2.1.89/lib/doing/wwid/wwid.rb'
+
+# Offense count: 2
+# Configuration parameters: AllowComments.
+Lint/EmptyWhen:
+  Exclude:
+    - 'lib/doing/completion.rb'
+    - 'pkg/doing-2.1.89/lib/doing/completion.rb'
+
+# Offense count: 4
+# Configuration parameters: AllowedParentClasses.
+Lint/MissingSuper:
+  Exclude:
+    - 'lib/doing/errors.rb'
+    - 'pkg/doing-2.1.89/lib/doing/errors.rb'
+
+# Offense count: 4
+Lint/ShadowingOuterLocalVariable:
+  Exclude:
+    - 'bin/commands/tags.rb'
+    - 'lib/helpers/threaded_tests.rb'
+    - 'pkg/doing-2.1.89/bin/commands/tags.rb'
+    - 'pkg/doing-2.1.89/lib/helpers/threaded_tests.rb'
+
+# Offense count: 8
+# Configuration parameters: AllowKeywordBlockArguments.
+Lint/UnderscorePrefixedVariableName:
+  Exclude:
+    - 'lib/doing/markdown_document_listener.rb'
+    - 'pkg/doing-2.1.89/lib/doing/markdown_document_listener.rb'
+
+# Offense count: 19
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AutoCorrect, AllowUnusedKeywordArguments, IgnoreEmptyMethods, IgnoreNotImplementedMethods, NotImplementedExceptions.
+# NotImplementedExceptions: NotImplementedError
+Lint/UnusedMethodArgument:
+  Enabled: false
+
+# Offense count: 2
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AutoCorrect, CheckForMethodsWithNoSideEffects.
+Lint/Void:
+  Exclude:
+    - 'lib/doing/string/transform.rb'
+    - 'pkg/doing-2.1.89/lib/doing/string/transform.rb'
+
+# Offense count: 316
+# Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes.
+Metrics/AbcSize:
+  Max: 198
+
+# Offense count: 154
+# Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
+# AllowedMethods: refine
+Metrics/BlockLength:
+  Max: 197
+
+# Offense count: 36
+# Configuration parameters: CountBlocks, CountModifierForms.
+Metrics/BlockNesting:
+  Max: 5
+
+# Offense count: 54
+# Configuration parameters: CountComments, CountAsOne.
+Metrics/ClassLength:
+  Max: 413
+
+# Offense count: 199
+# Configuration parameters: AllowedMethods, AllowedPatterns.
+Metrics/CyclomaticComplexity:
+  Max: 75
+
+# Offense count: 363
+# Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
+Metrics/MethodLength:
+  Max: 187
+
+# Offense count: 20
+# Configuration parameters: CountComments, CountAsOne.
+Metrics/ModuleLength:
+  Max: 372
+
+# Offense count: 16
+# Configuration parameters: CountKeywordArgs, MaxOptionalParameters.
+Metrics/ParameterLists:
+  Max: 9
+
+# Offense count: 163
+# Configuration parameters: AllowedMethods, AllowedPatterns.
+Metrics/PerceivedComplexity:
+  Max: 82
+
+# Offense count: 4
+Naming/AccessorMethodName:
+  Exclude:
+    - 'lib/doing/plugins/export/dayone_export.rb'
+    - 'lib/doing/plugins/export/markdown_export.rb'
+    - 'pkg/doing-2.1.89/lib/doing/plugins/export/dayone_export.rb'
+    - 'pkg/doing-2.1.89/lib/doing/plugins/export/markdown_export.rb'
+
+# Offense count: 8
+# Configuration parameters: ExpectMatchingDefinition, CheckDefinitionPathHierarchy, CheckDefinitionPathHierarchyRoots, Regex, IgnoreExecutableScripts, AllowedAcronyms.
+# CheckDefinitionPathHierarchyRoots: lib, spec, test, src
+# AllowedAcronyms: CLI, DSL, ACL, API, ASCII, CPU, CSS, DNS, EOF, GUID, HTML, HTTP, HTTPS, ID, IP, JSON, LHS, QPS, RAM, RHS, RPC, SLA, SMTP, SQL, SSH, TCP, TLS, TTL, UDP, UI, UID, UUID, URI, URL, UTF8, VM, XML, XMPP, XSRF, XSS
+Naming/FileName:
+  Exclude:
+    - 'Rakefile.rb'
+    - 'test/doing_unit-good_test.rb'
+    - 'test/doing_unit-item_test.rb'
+    - 'test/doing_unit-items_test.rb'
+    - 'test/doing_unit-normalize_test.rb'
+    - 'test/doing_unit-note_test.rb'
+    - 'test/doing_unit-time_test.rb'
+    - 'test/doing_unit-util_test.rb'
+    - 'test/helpers/doing-helpers.rb'
+
+# Offense count: 12
+# Configuration parameters: MinNameLength, AllowNamesEndingInNumbers, AllowedNames, ForbiddenNames.
+# AllowedNames: as, at, by, cc, db, id, if, in, io, ip, of, on, os, pp, to
+Naming/MethodParameterName:
+  Exclude:
+    - 'lib/helpers/threaded_tests.rb'
+    - 'pkg/doing-2.1.89/lib/helpers/threaded_tests.rb'
+    - 'test/doing_chronify_test.rb'
+    - 'test/doing_done_test.rb'
+    - 'test/doing_finish_test.rb'
+    - 'test/doing_finishtimed_test.rb'
+    - 'test/helpers/doing-helpers.rb'
+    - 'test/helpers/fake_std_out.rb'
+
+# Offense count: 2
+# Configuration parameters: NamePrefix, ForbiddenPrefixes, AllowedMethods, MethodDefinitionMacros, UseSorbetSigs.
+# NamePrefix: is_, has_, have_, does_
+# ForbiddenPrefixes: is_, has_, have_, does_
+# AllowedMethods: is_a?
+# MethodDefinitionMacros: define_method, define_singleton_method
+Naming/PredicateName:
+  Exclude:
+    - 'spec/**/*'
+    - 'lib/doing/chronify/string.rb'
+    - 'pkg/doing-2.1.89/lib/doing/chronify/string.rb'
+
+# Offense count: 8
+# Configuration parameters: EnforcedStyle, CheckMethodNames, CheckSymbols, AllowedIdentifiers, AllowedPatterns.
+# SupportedStyles: snake_case, normalcase, non_integer
+# AllowedIdentifiers: TLS1_1, TLS1_2, capture3, iso8601, rfc1123_date, rfc822, rfc2822, rfc3339, x86_64
+Naming/VariableNumber:
+  Exclude:
+    - 'lib/helpers/fzf/test/test_go.rb'
+    - 'pkg/doing-2.1.89/lib/helpers/fzf/test/test_go.rb'
+
+# Offense count: 2
+Security/Eval:
+  Exclude:
+    - 'Rakefile'
+    - 'pkg/doing-2.1.89/Rakefile'
+
+# Offense count: 2
+# This cop supports unsafe autocorrection (--autocorrect-all).
+Style/CombinableLoops:
+  Exclude:
+    - 'lib/helpers/threaded_tests.rb'
+    - 'pkg/doing-2.1.89/lib/helpers/threaded_tests.rb'
+
+# Offense count: 132
+# Configuration parameters: AllowedConstants.
+Style/Documentation:
+  Enabled: false
+
+# Offense count: 10
+# Configuration parameters: AllowedVariables.
+Style/GlobalVars:
+  Exclude:
+    - 'pkg/doing-2.1.89/scripts/deploy.rb'
+    - 'scripts/deploy.rb'
+
+# Offense count: 4
+# This cop supports unsafe autocorrection (--autocorrect-all).
+Style/IdenticalConditionalBranches:
+  Exclude:
+    - 'lib/doing/plugins/export/dayone_export.rb'
+    - 'pkg/doing-2.1.89/lib/doing/plugins/export/dayone_export.rb'
+
+# Offense count: 84
+# This cop supports safe autocorrection (--autocorrect).
+Style/IfUnlessModifier:
+  Enabled: false
+
+# Offense count: 8
+Style/MixinUsage:
+  Exclude:
+    - '.irbrc'
+    - 'bin/doing'
+    - 'pkg/doing-2.1.89/.irbrc'
+    - 'pkg/doing-2.1.89/bin/doing'
+
+# Offense count: 2
+Style/MultilineBlockChain:
+  Exclude:
+    - 'lib/doing/logger.rb'
+    - 'pkg/doing-2.1.89/lib/doing/logger.rb'
+
+# Offense count: 20
+Style/OptionalArguments:
+  Exclude:
+    - 'lib/doing/wwid/display.rb'
+    - 'lib/doing/wwid/modify.rb'
+    - 'pkg/doing-2.1.89/lib/doing/wwid/display.rb'
+    - 'pkg/doing-2.1.89/lib/doing/wwid/modify.rb'
+
+# Offense count: 2
+# Configuration parameters: AllowedMethods.
+# AllowedMethods: respond_to_missing?
+Style/OptionalBooleanParameter:
+  Exclude:
+    - 'lib/doing/wwid/display.rb'
+    - 'pkg/doing-2.1.89/lib/doing/wwid/display.rb'
+
+# Offense count: 190
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, AllowedPatterns, SplitStrings.
+# URISchemes: http, https
+Layout/LineLength:
+  Max: 334

data/CHANGELOG.md

@@ -1,3 +1,25 @@
+### 2.1.90
+
+2025-10-23 06:25
+
+#### NEW
+
+- Add benchmark_stats() and all_benchmark_stats() for detailed analysis
+- Add benchmark_summary() for compact performance overview
+
+#### IMPROVED
+
+- Add 4-decimal-place rounding to all benchmark timing outputs
+- Implement new measure() method for automatic start/finish timing
+- Optimize log_benchmarks() with better sorting and I/O efficiency
+
+#### FIXED
+
+- Fix coloring? method to return boolean instead of nil
+- Move color method definitions to class methods (Doing::Color.reset)
+- Add string extension methods for all color attributes ("text".reset)
+- Move attributes() method to class methods for proper access
+
 ### 2.1.89
 
 2025-05-19 04:35

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'http://rubygems.org'
 
 gem 'rubocop'

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    doing (2.1.89)
+    doing (2.1.90)
       base64 (~> 0.2)
       chronic (~> 0.10, >= 0.10.2)
       csv (~> 3.3)

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rake/clean'
 require 'rubygems'
 require 'rubygems/package_task'
@@ -11,9 +13,9 @@ require 'parallel_tests/tasks'
 require 'tty-spinner'
 
 YARD::Rake::YardocTask.new do |t|
- t.files = ['lib/doing/*.rb']
- t.options = ['--markup-provider=redcarpet', '--markup=markdown', '--no-private', '-p', 'yard_templates']
- # t.stats_options = ['--list-undoc']
+  t.files = ['lib/doing/*.rb']
+  t.options = ['--markup-provider=redcarpet', '--markup=markdown', '--no-private', '-p', 'yard_templates']
+  # t.stats_options = ['--list-undoc']
 end
 
 task :doc, [*Rake.application[:yard].arg_names] => [:yard]

data/bin/commands/changes.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@changelog @@changes
 require 'tty-markdown'
 
@@ -18,7 +20,7 @@ MARKDOWN_THEME = {
 module Doing
   # changes command methods
   class ChangesCommand
-    CHANGE_RX = /^(?:(?:(?:[<>=]+|p(?:rior)|b(?:efore)|o(?:lder)|s(?:ince)|a(?:fter)|n(?:ewer))? *[0-9.*?]{1,10} *)+|(?:[\d.]+ *(?:-|to)+ *[0-9.]{1,10}))$/
+    CHANGE_RX = /^(?:(?:(?:[<>=]+|p(?:rior)|b(?:efore)|o(?:lder)|s(?:ince)|a(?:fter)|n(?:ewer))? *[0-9.*?]{1,10} *)+|(?:[\d.]+ *(?:-|to)+ *[0-9.]{1,10}))$/.freeze
 
     def add_options(cmd)
       cmd.desc 'Display all versions'

data/bin/commands/choose.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@choose
 desc 'Select a section to display from a menu'
 command :choose do |c|

data/bin/commands/colors.rb

@@ -1,16 +1,18 @@
+# frozen_string_literal: true
+
 # @@colors
 desc 'List available color variables for configuration templates and views'
 command :colors do |c|
   c.action do |_global_options, _options, _args|
     bgs = []
     fgs = []
-    @colors::attributes.each do |color|
+    @colors.attributes.each do |color|
       colname = color.to_s
       colname << " (#{color.to_s.sub(/bold/, 'bright')})" if colname =~ /bold/
       if color.to_s =~ /bg/
-        bgs.push("#{@colors.send(color, "    ")}#{@colors.default} <-- #{colname}")
+        bgs.push("#{@colors.send(color, '    ')}#{@colors.default} <-- #{colname}")
       else
-        fgs.push("#{@colors.send(color, "XXXX")}#{@colors.default} <-- #{colname}")
+        fgs.push("#{@colors.send(color, 'XXXX')}#{@colors.default} <-- #{colname}")
       end
     end
     out = []

data/bin/commands/commands_accepting.rb

@@ -17,7 +17,7 @@ module Doing
     def all_flags?(options, args)
       args.each do |arg|
         has_flag = false
-        options.flags.merge(options.switches).each do |_, flag|
+        options.flags.merge(options.switches).each_value do |flag|
           if flag.name == arg.to_sym || flag.aliases&.include?(arg.to_sym)
             has_flag = true
             break
@@ -31,7 +31,7 @@ module Doing
 
     def any_flags?(options, args)
       args.each do |option|
-        options.flags.merge(options.switches).each do |_, flag|
+        options.flags.merge(options.switches).each_value do |flag|
           return true if flag.name == option.to_sym || flag.aliases&.include?(option.to_sym)
         end
       end
@@ -41,7 +41,7 @@ module Doing
 
     def no_flags?(options, args)
       args.each do |option|
-        options.flags.merge(options.switches).each do |_, flag|
+        options.flags.merge(options.switches).each_value do |flag|
           return false if flag.name == option.to_sym || flag.aliases&.include?(option.to_sym)
         end
       end

data/bin/commands/completion.rb

@@ -22,7 +22,8 @@ command :completion do |c|
   c.arg_name "[#{SHELLS.join('|')}]"
   c.command :generate do |gen|
     gen.example 'doing completion generate fish', desc: 'Generate fish completion script and link to autoload directory'
-    gen.example 'doing completion generate zsh --file ~/.zsh-completions/doing.zsh', desc: 'Generate zsh completion script and save to alternative file'
+    gen.example 'doing completion generate zsh --file ~/.zsh-completions/doing.zsh',
+                desc: 'Generate zsh completion script and save to alternative file'
 
     gen.desc 'Alternative file to write output to'
     gen.arg_name 'PATH'

data/bin/commands/config.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@config
 desc 'Edit the configuration file or output a value from it'
 long_desc %(Run without arguments, `doing config` opens your `config.yml` in an editor.
@@ -232,18 +234,18 @@ command :config do |c|
 
       cfg = Doing::Util.safe_load_file(config_file) || {}
 
-      $stderr.puts ">      Config: Updating #{config_file}".yellow
+      warn ">      Config: Updating #{config_file}".yellow
 
       if options[:remove]
-        $stderr.puts "#{'Deleting key:'.yellow} #{real_path.join('.').boldwhite}"
+        warn "#{'Deleting key:'.yellow} #{real_path.join('.').boldwhite}"
         cfg.deep_set(real_path, nil)
       else
         current_value = cfg.dig(*real_path)
         cfg.deep_set(real_path, value.set_type(old_type))
-        $stderr.puts "#{'     Key path:'.yellow} #{real_path.join('.').boldwhite}"
-        $stderr.puts "#{'    Inherited:'.yellow} #{(old_value ? old_value.to_s : 'empty').boldwhite}"
-        $stderr.puts "#{'      Current:'.yellow} #{ (current_value ? current_value.to_s : 'empty').boldwhite }"
-        $stderr.puts "#{'          New:'.yellow} #{value.set_type(old_type).to_s.boldwhite}"
+        warn "#{'     Key path:'.yellow} #{real_path.join('.').boldwhite}"
+        warn "#{'    Inherited:'.yellow} #{(old_value ? old_value.to_s : 'empty').boldwhite}"
+        warn "#{'      Current:'.yellow} #{(current_value ? current_value.to_s : 'empty').boldwhite}"
+        warn "#{'          New:'.yellow} #{value.set_type(old_type).to_s.boldwhite}"
       end
 
       res = Doing::Prompt.yn('Update selected config', default_response: true)

data/bin/commands/done.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@done @@did
 desc 'Add a completed item with @done(date). No argument finishes last entry'
 long_desc 'Use this command to add an entry after you\'ve already finished it. It will be immediately marked as @done.
@@ -7,8 +9,10 @@ arg_name 'ENTRY', optional: true
 command %i[done did] do |c|
   c.example 'doing done', desc: 'Tag the last entry @done'
   c.example 'doing done I already finished this', desc: 'Add a new entry and immediately mark it @done'
-  c.example 'doing done --back 30m This took me half an hour', desc: 'Add an entry with a start date 30 minutes ago and a @done date of right now'
-  c.example 'doing done --at 3pm --took 1h Started and finished this afternoon', desc: 'Add an entry with a @done date of 3pm and a start date of 2pm (3pm - 1h)'
+  c.example 'doing done --back 30m This took me half an hour',
+            desc: 'Add an entry with a start date 30 minutes ago and a @done date of right now'
+  c.example 'doing done --at 3pm --took 1h Started and finished this afternoon',
+            desc: 'Add an entry with a @done date of 3pm and a start date of 2pm (3pm - 1h)'
 
   c.desc 'Remove @done tag'
   c.switch %i[r remove], negatable: false, default_value: false
@@ -78,11 +82,11 @@ command %i[done did] do |c|
       donedate = finish_date.strftime('%F %R')
     end
 
-    if options[:section]
-      section = @wwid.guess_section(options[:section]) || options[:section].cap_first
-    else
-      section = Doing.setting('current_section')
-    end
+    section = if options[:section]
+                @wwid.guess_section(options[:section]) || options[:section].cap_first
+              else
+                Doing.setting('current_section')
+              end
 
     note = Doing::Note.new
     note.add(options[:note]) if options[:note]
@@ -95,7 +99,9 @@ command %i[done did] do |c|
       is_new = false
 
       if args.empty?
-        last_entry = @wwid.filter_items([], opt: { unfinished: options[:unfinished], section: section, count: 1, age: :newest }).max_by { |item| item.date }
+        last_entry = @wwid.filter_items([],
+                                        opt: { unfinished: options[:unfinished], section: section, count: 1,
+                                               age: :newest }).max_by(&:date)
 
         unless last_entry
           Doing.logger.debug('Skipped:', options[:unfinished] ? 'No unfinished entry' : 'Last entry already @done')
@@ -104,7 +110,8 @@ command %i[done did] do |c|
 
         old_entry = last_entry.clone
         last_entry.note.add(note)
-        input = ["#{last_entry.date.strftime('%F %R | ')}#{last_entry.title}", last_entry.note.strip_lines.join("\n")].join("\n")
+        input = ["#{last_entry.date.strftime('%F %R | ')}#{last_entry.title}",
+                 last_entry.note.strip_lines.join("\n")].join("\n")
       else
         is_new = true
         input = ["#{date.strftime('%F %R | ')}#{args.join(' ')}", note.strip_lines.join("\n")].join("\n")
@@ -135,7 +142,7 @@ command %i[done did] do |c|
         end
       end
 
-      if (is_new)
+      if is_new
         Doing::Hooks.trigger :pre_entry_add, @wwid, new_entry
         @wwid.content.push(new_entry)
         Doing::Hooks.trigger :post_entry_added, @wwid, new_entry
@@ -163,7 +170,7 @@ command %i[done did] do |c|
           note: note,
           section: section,
           tags: ['done'],
-          took: took == 0 ? nil : took,
+          took: took.zero? ? nil : took,
           unfinished: options[:unfinished]
         }
         @wwid.tag_last(opt)
@@ -196,7 +203,7 @@ command %i[done did] do |c|
       @wwid.write(@wwid.doing_file)
       Doing.logger.info('New entry:', %(added "#{new_entry.date.relative_date}: #{new_entry.title}" to #{section}))
     elsif !global_options[:stdin].nil?
-      note = Doing::Note.new(options[:note])
+      Doing::Note.new(options[:note])
       d, title, note = @wwid.format_input(global_options[:stdin])
       unless d.nil?
         Doing.logger.debug('Parser:', 'Date detected in input, overriding command line values')

data/bin/commands/finish.rb

@@ -10,8 +10,10 @@ module Doing
 
     def add_examples(cmd)
       cmd.example 'doing finish', desc: 'Mark the last entry @done'
-      cmd.example 'doing finish --auto --section Later 10', desc: 'Add @done to any unfinished entries in the last 10 in Later, setting the finish time based on the start time of the task after it'
-      cmd.example 'doing finish --search "a specific entry" --at "yesterday 3pm"', desc: 'Search for an entry containing string and set its @done time to yesterday at 3pm'
+      cmd.example 'doing finish --auto --section Later 10',
+                  desc: 'Add @done to any unfinished entries in the last 10 in Later, setting the finish time based on the start time of the task after it'
+      cmd.example 'doing finish --search "a specific entry" --at "yesterday 3pm"',
+                  desc: 'Search for an entry containing string and set its @done time to yesterday at 3pm'
     end
 
     def add_options(cmd)

data/bin/commands/grep.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@grep @@search
 desc 'Search for entries'
 long_desc %(
@@ -8,9 +10,11 @@ To search with regular expressions, single quote the string and surround with sl
 arg_name 'SEARCH_PATTERN'
 command %i[grep search] do |c|
   c.example 'doing grep "doing wiki"', desc: 'Find entries containing "doing wiki" using fuzzy matching'
-  c.example 'doing search "\'search command"', desc: 'Find entries containing "search command" using exact matching (search is an alias for grep)'
+  c.example 'doing search "\'search command"',
+            desc: 'Find entries containing "search command" using exact matching (search is an alias for grep)'
   c.example 'doing grep "/do.*?wiki.*?@done/"', desc: 'Find entries matching regular expression'
-  c.example 'doing search --before 12/21 "doing wiki"', desc: 'Find entries containing "doing wiki" with entry dates before 12/21 of the current year'
+  c.example 'doing search --before 12/21 "doing wiki"',
+            desc: 'Find entries containing "doing wiki" with entry dates before 12/21 of the current year'
   c.example 'doing grep @project1 -s currently -s projects', desc: 'Search only in Currently and Projects'
 
   c.desc 'Section'
@@ -29,13 +33,13 @@ command %i[grep search] do |c|
                   default_value: Doing.setting('search.case', :smart).normalize_case,
                   type: CaseSymbol
 
-  c.desc "Highlight search matches in output. Only affects command line output"
+  c.desc 'Highlight search matches in output. Only affects command line output'
   c.switch %i[h hilite], default_value: Doing.settings.dig('search', 'highlight')
 
   c.desc "Edit matching entries with #{Doing::Util.default_editor}"
   c.switch %i[e editor], negatable: false, default_value: false
 
-  c.desc "Delete matching entries"
+  c.desc 'Delete matching entries'
   c.switch %i[d delete], negatable: false, default_value: false
 
   c.desc 'Display an interactive menu of results to perform further operations'
@@ -76,6 +80,6 @@ command %i[grep search] do |c|
       options[:after] = Doing.original_options[:date_end] if Doing.original_options[:date_end].good?
       options[:from] = Doing.original_options[:date_range] if Doing.original_options[:date_range].good?
       Doing.config.save_view(options.to_view, options[:save].downcase)
-   end
+    end
   end
 end

data/bin/commands/import.rb

@@ -1,11 +1,16 @@
+# frozen_string_literal: true
+
 # @@import
 desc 'Import entries from an external source'
-long_desc "Imports entries from other sources. Available plugins: #{Doing::Plugins.plugin_names(type: :import, separator: ', ')}"
+long_desc "Imports entries from other sources. Available plugins: #{Doing::Plugins.plugin_names(type: :import,
+                                                                                                separator: ', ')}"
 arg_name 'PATH'
 command :import do |c|
   c.example 'doing import --type timing "~/Desktop/All Activities.json"', desc: 'Import a Timing.app JSON report'
-  c.example 'doing import --type doing --tag imported --no-autotag ~/doing_backup.md', desc: 'Import an Doing archive, tag all entries with @imported, skip autotagging'
-  c.example 'doing import --type doing --from "10/1 to 10/15" ~/doing_backup.md', desc: 'Import a Doing archive, only importing entries between two dates'
+  c.example 'doing import --type doing --tag imported --no-autotag ~/doing_backup.md',
+            desc: 'Import an Doing archive, tag all entries with @imported, skip autotagging'
+  c.example 'doing import --type doing --from "10/1 to 10/15" ~/doing_backup.md',
+            desc: 'Import a Doing archive, only importing entries between two dates'
 
   c.desc "Import type (#{Doing::Plugins.plugin_names(type: :import)})"
   c.arg_name 'TYPE'
@@ -40,9 +45,7 @@ command :import do |c|
 
   c.action do |_global_options, options, args|
     options[:fuzzy] = false
-    if options[:section]
-      options[:section] = @wwid.guess_section(options[:section]) || options[:section].cap_first
-    end
+    options[:section] = @wwid.guess_section(options[:section]) || options[:section].cap_first if options[:section]
 
     if options[:search]
       search = options[:search]
@@ -60,12 +63,12 @@ command :import do |c|
       options[:date_filter][0] = options[:after] || Time.now - (1 << 64)
     end
 
-    if options[:type] =~ Doing::Plugins.plugin_regex(type: :import)
-      options[:no_overlap] = !options[:overlap]
-      @wwid.import(args, options)
-      @wwid.write(@wwid.doing_file)
-    else
+    unless options[:type] =~ Doing::Plugins.plugin_regex(type: :import)
       raise InvalidPluginType, "Invalid import type: #{options[:type]}"
     end
+
+    options[:no_overlap] = !options[:overlap]
+    @wwid.import(args, options)
+    @wwid.write(@wwid.doing_file)
   end
 end

data/bin/commands/install_fzf.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@install_fzf
 command :install_fzf do |c|
   c.desc 'Force reinstall'
@@ -6,12 +8,12 @@ command :install_fzf do |c|
   c.desc 'Uninstall'
   c.switch %i[u uninstall], default_value: false, negatable: false
 
-  c.action do |g, o, a|
+  c.action do |_g, o, _a|
     if o[:uninstall]
       Doing::Prompt.uninstall_fzf
     else
       Doing.logger.warn('fzf:', 'force reinstall') if o[:reinstall]
-      res = Doing::Prompt.install_fzf(force: o[:reinstall])
+      Doing::Prompt.install_fzf(force: o[:reinstall])
     end
   end
 end