doing 2.1.88 → 2.1.90

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

data/bin/commands/last.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@last
 desc 'Show the last entry, optionally edit'
 long_desc 'Shows the last entry. Using --search and --tag filters, you can view/edit the last entry matching a filter,
@@ -6,10 +8,12 @@ command :last do |c|
   c.example 'doing last', desc: 'Show the most recent entry in all sections'
   c.example 'doing last -s Later', desc: 'Show the most recent entry in the Later section'
   c.example 'doing last --tag project1,work --bool AND', desc: 'Show most recent entry tagged @project1 and @work'
-  c.example 'doing last --search "side hustle"', desc: 'Show most recent entry containing "side hustle" (fuzzy matching)'
+  c.example 'doing last --search "side hustle"',
+            desc: 'Show most recent entry containing "side hustle" (fuzzy matching)'
   c.example 'doing last --search "\'side hustle"', desc: 'Show most recent entry containing "side hustle" (exact match)'
   c.example 'doing last --edit', desc: 'Open the most recent entry in an editor for modifications'
-  c.example 'doing last --search "\'side hustle" --edit', desc: 'Open most recent entry containing "side hustle" (exact match) in editor'
+  c.example 'doing last --search "\'side hustle" --edit',
+            desc: 'Open most recent entry containing "side hustle" (exact match) in editor'
 
   c.desc 'Specify a section'
   c.arg_name 'NAME'
@@ -18,10 +22,10 @@ command :last do |c|
   c.desc "Edit entry with #{Doing::Util.default_editor}"
   c.switch %i[e editor], negatable: false, default_value: false
 
-  c.desc "Delete the last entry"
+  c.desc 'Delete the last entry'
   c.switch %i[d delete], negatable: false, default_value: false
 
-  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 'Show elapsed time if entry is not tagged @done'
@@ -32,7 +36,7 @@ command :last do |c|
   add_options(:tag_filter, c)
   add_options(:save, c)
 
-  c.action do |global_options, options, _args|
+  c.action do |_global_options, options, _args|
     options[:fuzzy] = false
     raise InvalidArgument, '--tag and --search can not be used together' if options[:tag] && options[:search]
 
@@ -42,34 +46,34 @@ command :last do |c|
 
     if options[:editor]
       @wwid.edit_last(section: options[:section],
-                     options: {
-                       search: options[:search],
-                       fuzzy: options[:fuzzy],
-                       case: options[:case],
-                       tag: options[:tag],
-                       tag_bool: options[:bool],
-                       not: options[:not],
-                       val: options[:val],
-                       bool: options[:bool]
-                     })
-    else
-      last = @wwid.last(times: true, section: options[:section],
-                     options: {
-                        config_template: options[:config_template],
-                        template: options[:template],
-                        duration: options[:duration],
+                      options: {
                         search: options[:search],
                         fuzzy: options[:fuzzy],
                         case: options[:case],
-                        hilite: options[:hilite],
-                        negate: options[:not],
-                        output: options[:output],
                         tag: options[:tag],
                         tag_bool: options[:bool],
-                        delete: options[:delete],
-                        bool: options[:bool],
-                        val: options[:val]
+                        not: options[:not],
+                        val: options[:val],
+                        bool: options[:bool]
                       })
+    else
+      last = @wwid.last(times: true, section: options[:section],
+                        options: {
+                          config_template: options[:config_template],
+                          template: options[:template],
+                          duration: options[:duration],
+                          search: options[:search],
+                          fuzzy: options[:fuzzy],
+                          case: options[:case],
+                          hilite: options[:hilite],
+                          negate: options[:not],
+                          output: options[:output],
+                          tag: options[:tag],
+                          tag_bool: options[:bool],
+                          delete: options[:delete],
+                          bool: options[:bool],
+                          val: options[:val]
+                        })
       Doing::Pager.page last.strip if last
       Doing.config.save_view(options.to_view, options[:save].downcase) if options[:save]
     end

data/bin/commands/meanwhile.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@meanwhile
 desc 'Finish any running @meanwhile tasks and optionally create a new one'
 long_desc 'The @meanwhile tag allows you to have long-running entries that encompass smaller entries.
@@ -6,10 +8,12 @@ big project` to start a @meanwhile entry, add other entries as you work on the p
 itself to mark the entry as @done.'
 arg_name 'ENTRY', optional: true
 command :meanwhile do |c|
-  c.example 'doing meanwhile "Long task that will have others after it before it\'s done"', desc: 'Add a new long-running entry, completing any current @meanwhile entry'
+  c.example 'doing meanwhile "Long task that will have others after it before it\'s done"',
+            desc: 'Add a new long-running entry, completing any current @meanwhile entry'
   c.example 'doing meanwhile', desc: 'Finish any open @meanwhile entry'
   c.example 'doing meanwhile --archive', desc: 'Finish any open @meanwhile entry and archive it'
-  c.example 'doing meanwhile --back 2h "Something I\'ve been working on for a while', desc: 'Add a @meanwhile entry with a start date 2 hours ago'
+  c.example 'doing meanwhile --back 2h "Something I\'ve been working on for a while',
+            desc: 'Add a @meanwhile entry with a start date 2 hours ago'
 
   c.desc 'Section'
   c.arg_name 'NAME'

data/bin/commands/note.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@note
 desc 'Add a note to the last entry'
 long_desc %(
@@ -12,7 +14,8 @@ command :note do |c|
   c.example 'doing note', desc: 'Open the last entry in $EDITOR to append a note'
   c.example 'doing note "Just a quick annotation"', desc: 'Add a quick note to the last entry'
   c.example 'doing note --tag done "Keeping it real or something"', desc: 'Add a note to the last item tagged @done'
-  c.example 'doing note --search "late night" -e', desc: 'Open $EDITOR to add a note to the last item containing "late night" (fuzzy matched)'
+  c.example 'doing note --search "late night" -e',
+            desc: 'Open $EDITOR to add a note to the last item containing "late night" (fuzzy matched)'
 
   c.desc 'Section'
   c.arg_name 'NAME'
@@ -71,8 +74,8 @@ command :note do |c|
     end
 
     if (new_note.empty? && !options[:remove]) || options[:ask]
-      $stderr.puts last_note if last_note.good?
-      $stderr.puts new_note if new_note.good?
+      warn last_note if last_note.good?
+      warn new_note if new_note.good?
       new_note.add(Doing::Prompt.read_lines(prompt: 'Add a note'))
     end
 

data/bin/commands/now.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@now @@next
 
 module Doing

data/bin/commands/open.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@open
 desc 'Open the "doing" file in an editor'
 long_desc "`doing open` defaults to using the editors.doing_file setting
@@ -25,7 +27,10 @@ command :open do |c|
     end
 
     if options[:editor]
-      raise MissingEditor, "Editor #{options[:editor]} not found" unless Doing::Util.exec_available(options[:editor].split(/ /).first)
+      unless Doing::Util.exec_available(options[:editor].split(/ /).first)
+        raise MissingEditor,
+              "Editor #{options[:editor]} not found"
+      end
 
       editor = TTY::Which.which(options[:editor])
       system %(#{editor} "#{File.expand_path(@wwid.doing_file)}")

data/bin/commands/plugins.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@plugins
 desc 'List installed plugins'
 long_desc %(Lists available plugins, including user-installed plugins.

data/bin/commands/recent.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@recent
 desc 'List recent entries'
 default_value 10
@@ -21,39 +23,51 @@ command :recent do |c|
   add_options(:save, c)
 
   c.action do |global_options, options, args|
-    section = @wwid.guess_section(options[:section]) || options[:section].cap_first
-
-    unless global_options[:version]
-      config_count = if Doing.setting('templates.recent.count')
-                       Doing.setting('templates.recent.count').to_i
-                     else
-                       10
-                     end
-
-      count = args.empty? ? config_count : args[0].to_i
-
-      options[:times] = true if options[:totals]
-      options[:sort_tags] = options[:tag_sort]
-
-      template = Doing.setting('templates.recent').deep_merge(Doing.setting('templates.default'))
-      tags_color = template.key?('tags_color') ? template['tags_color'] : nil
-
-      opts = {
-        config_template: options[:config_template],
-        duration: options[:duration],
-        interactive: options[:interactive],
-        output: options[:output],
-        sort_tags: options[:sort_tags],
-        tags_color: tags_color,
-        template: options[:template],
-        times: options[:times],
-        totals: options[:totals]
-      }
-
-      Doing::Pager.page @wwid.recent(count, section.cap_first, opts)
-      opts[:count] = count
-      opts[:title] = options[:title]
-      Doing.config.save_view(opts.to_view, options[:save].downcase) if options[:save]
+    Doing.logger.measure(:recent_command) do
+      section = @wwid.guess_section(options[:section]) || options[:section].cap_first
+
+      unless global_options[:version]
+        count = nil
+        opts = nil
+
+        Doing.logger.measure(:recent_config) do
+          config_count = if Doing.setting('templates.recent.count')
+                           Doing.setting('templates.recent.count').to_i
+                         else
+                           10
+                         end
+
+          count = args.empty? ? config_count : args[0].to_i
+
+          options[:times] = true if options[:totals]
+          options[:sort_tags] = options[:tag_sort]
+
+          template = Doing.setting('templates.recent').deep_merge(Doing.setting('templates.default'))
+          tags_color = template.key?('tags_color') ? template['tags_color'] : nil
+
+          opts = {
+            config_template: options[:config_template],
+            duration: options[:duration],
+            interactive: options[:interactive],
+            output: options[:output],
+            sort_tags: options[:sort_tags],
+            tags_color: tags_color,
+            template: options[:template],
+            times: options[:times],
+            totals: options[:totals]
+          }
+        end
+
+        Doing.logger.measure(:recent_pager) do
+          Doing::Pager.page @wwid.recent(count, section.cap_first, opts)
+        end
+
+        Doing.logger.measure(:recent_save) do
+          opts[:count] = count
+          opts[:title] = options[:title]
+          Doing.config.save_view(opts.to_view, options[:save].downcase) if options[:save]
+        end
+      end
     end
   end
 end

data/bin/commands/reset.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@reset @@begin
 desc 'Reset the start time of an entry'
 long_desc 'Update the start time of the last entry or the last entry matching a tag/search filter.
@@ -6,9 +8,11 @@ If a date string is provided as an argument, the start time will be set to the p
 arg_name 'DATE_STRING', optional: true
 command %i[reset begin] do |c|
   c.example 'doing reset', desc: 'Reset the start time of the last entry to the current time'
-  c.example 'doing reset --tag project1', desc: 'Reset the start time of the most recent entry tagged @project1 to the current time'
+  c.example 'doing reset --tag project1',
+            desc: 'Reset the start time of the most recent entry tagged @project1 to the current time'
   c.example 'doing reset 3pm', desc: 'Reset the start time of the last entry to 3pm of the current day'
-  c.example 'doing begin --tag todo --resume', desc: 'alias for reset. Updates the last @todo entry to the current time, removing @done tag.'
+  c.example 'doing begin --tag todo --resume',
+            desc: 'alias for reset. Updates the last @todo entry to the current time, removing @done tag.'
 
   c.desc 'Limit search to section'
   c.arg_name 'NAME'
@@ -37,7 +41,7 @@ command %i[reset begin] do |c|
   add_options(:search, c)
   add_options(:tag_filter, c)
 
-  c.action do |global_options, options, args|
+  c.action do |_global_options, options, args|
     if args.count.positive?
       reset_date = args.join(' ').chronify(guess: :begin)
       raise InvalidArgument, 'Invalid date string' unless reset_date

data/bin/commands/rotate.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@rotate
 desc 'Move entries to archive file'
 long_desc 'As your doing file grows, commands can get slow. Given that your historical data (and your archive section)
@@ -5,8 +7,10 @@ probably aren\'t providing any useful insights a year later, use this command to
 file. You\'ll still have access to all historical data, but it won\'t be slowing down daily operation.'
 command :rotate do |c|
   c.example 'doing rotate', desc: 'Move all entries in doing file to a dated secondary file'
-  c.example 'doing rotate --section Archive --keep 10', desc: 'Move entries in the Archive section to a secondary file, keeping the most recent 10 entries'
-  c.example 'doing rotate --tag project1,done --bool AND', desc: 'Move entries tagged @project1 and @done to a secondary file'
+  c.example 'doing rotate --section Archive --keep 10',
+            desc: 'Move entries in the Archive section to a secondary file, keeping the most recent 10 entries'
+  c.example 'doing rotate --tag project1,done --bool AND',
+            desc: 'Move entries tagged @project1 and @done to a secondary file'
 
   c.desc 'How many items to keep in each section (most recent)'
   c.arg_name 'X'
@@ -56,4 +60,3 @@ end
 #     @wwid.write(@wwid.doing_file)
 #   end
 # end
-

data/bin/commands/sections.rb

@@ -51,10 +51,11 @@ command :sections do |c|
       items = @wwid.content.in_section(section)
 
       if items.count.positive?
-        res = Doing::Prompt.yn("#{options[:archive] ? 'Archive' : 'Delete'} #{items.count} entries from #{section}", default_response: 'n')
+        res = Doing::Prompt.yn("#{options[:archive] ? 'Archive' : 'Delete'} #{items.count} entries from #{section}",
+                               default_response: 'n')
 
         if options[:archive] && res
-          @wwid.archive(section, {keep: 0})
+          @wwid.archive(section, { keep: 0 })
         elsif res
           items.each { |item| @wwid.content.delete_item(item) }
         end
@@ -83,4 +84,3 @@ command :add_section do |c|
     action.call(g, o, a)
   end
 end
-

data/bin/commands/select.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@select
 desc 'Display an interactive menu to perform operations'
 long_desc 'List all entries and select with typeahead fuzzy matching.

data/bin/commands/show.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@show
 desc 'List all entries'
 long_desc %(
@@ -15,14 +17,16 @@ command :show do |c|
   c.example 'doing show Later @doing', desc: 'Show entries from the Later section tagged @doing'
   c.example 'doing show @oracle @writing --bool and', desc: 'Show entries tagged both @oracle and @writing'
   c.example 'doing show Currently @devo --bool not', desc: 'Show entries in Currently NOT tagged @devo'
-  c.example 'doing show Ideas @doing --from "mon to fri"', desc: 'Show entries tagged @doing from the Ideas section added between monday and friday of the current week.'
-  c.example 'doing show --interactive Later @doing', desc: 'Create a menu from entries from the Later section tagged @doing to perform batch actions'
+  c.example 'doing show Ideas @doing --from "mon to fri"',
+            desc: 'Show entries tagged @doing from the Ideas section added between monday and friday of the current week.'
+  c.example 'doing show --interactive Later @doing',
+            desc: 'Create a menu from entries from the Later section tagged @doing to perform batch actions'
 
   c.desc 'Max count to show'
   c.arg_name 'MAX'
   c.flag %i[c count], default_value: 0, must_match: /^\d+$/, type: Integer
 
-  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}"
@@ -111,11 +115,11 @@ command :show do |c|
     options[:times] = true if options[:totals]
 
     template = Doing.setting(['templates', options[:config_template]]).deep_merge({
-                              'wrap_width' => Doing.setting('wrap_width') || 0,
-                              'date_format' => Doing.setting('default_date_format'),
-                              'order' => Doing.setting('order')&.normalize_order || :asc,
-                              'tags_color' => Doing.setting('tags_color')
-                            })
+                                                                                    'wrap_width' => Doing.setting('wrap_width') || 0,
+                                                                                    'date_format' => Doing.setting('default_date_format'),
+                                                                                    'order' => Doing.setting('order')&.normalize_order || :asc,
+                                                                                    'tags_color' => Doing.setting('tags_color')
+                                                                                  })
 
     if options[:search]
       search = options[:search]
@@ -138,19 +142,19 @@ command :show do |c|
     items = @wwid.filter_items([], opt: options)
 
     if options[:menu]
-      Doing.logger.benchmark(:menu, :start)
-      tag = @wwid.choose_tag(section, items: items, include_all: true)
-      raise UserCancelled unless tag
-
-      tags = tag.split(/ +/).map { |t| t.strip.sub(/^@?/, '') } if tag =~ /^@/
-      if tags.good?
-        tag_filter = {
-          'tags' => tags,
-          'bool' => options[:bool]
-        }
-        options[:tag_filter] = tag_filter
+      Doing.logger.measure(:menu) do
+        tag = @wwid.choose_tag(section, items: items, include_all: true)
+        raise UserCancelled unless tag
+
+        tags = tag.split(/ +/).map { |t| t.strip.sub(/^@?/, '') } if tag =~ /^@/
+        if tags.good?
+          tag_filter = {
+            'tags' => tags,
+            'bool' => options[:bool]
+          }
+          options[:tag_filter] = tag_filter
+        end
       end
-      Doing.logger.benchmark(:menu, :finish)
     end
 
     options[:age] ||= :newest

data/bin/commands/since.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @since
 desc 'List entries since a date'
 long_desc %(Date argument can be natural language and are always interpreted as being in the past. "thursday" would be interpreted as "last thursday,"
@@ -5,7 +7,8 @@ and "2d" would be interpreted as "two days ago.")
 arg_name 'DATE_STRING'
 command :since do |c|
   c.example 'doing since 7/30', desc: 'List all entries created since 12am on 7/30 of the current year'
-  c.example 'doing since "monday 3pm" --output json', desc: 'Show entries since 3pm on Monday of the current week, output in JSON format'
+  c.example 'doing since "monday 3pm" --output json',
+            desc: 'Show entries since 3pm on Monday of the current week, output in JSON format'
 
   c.desc 'Section'
   c.arg_name 'NAME'
@@ -17,7 +20,10 @@ command :since do |c|
   add_options(:search, c)
   add_options(:save, c)
   c.action do |_global_options, options, args|
-    raise DoingRuntimeError, %(Invalid output type "#{options[:output]}") if options[:output] && options[:output] !~ Doing::Plugins.plugin_regex(type: :export)
+    if options[:output] && options[:output] !~ Doing::Plugins.plugin_regex(type: :export)
+      raise DoingRuntimeError,
+            %(Invalid output type "#{options[:output]}")
+    end
 
     raise MissingArgument, 'Missing date argument' if args.empty?
 
@@ -36,7 +42,8 @@ command :since do |c|
     options[:times] = true if options[:totals]
     options[:sort_tags] = options[:tag_sort]
 
-    Doing::Pager.page @wwid.list_date([start, finish], options[:section], options[:times], options[:output], options).chomp
+    Doing::Pager.page @wwid.list_date([start, finish], options[:section], options[:times], options[:output],
+                                      options).chomp
     if options[:save]
       options[:after] = Doing.original_options[:date_begin] if Doing.original_options[:date_begin].good?
       Doing.config.save_view(options.to_view, options[:save].downcase)

data/bin/commands/tag.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@tag
 desc 'Add tag(s) to last entry'
 long_desc 'Add (or remove) tags from the last entry, or from multiple entries
@@ -17,9 +19,12 @@ arg_name 'TAG', :multiple
 command :tag do |c|
   c.example 'doing tag mytag', desc: 'Add @mytag to the last entry created'
   c.example 'doing tag --remove mytag', desc: 'Remove @mytag from the last entry created'
-  c.example 'doing tag --rename "other*" --count 10 newtag', desc: 'Rename tags beginning with "other" (wildcard) to @newtag on the last 10 entries'
-  c.example 'doing tag --search "developing" coding', desc: 'Add @coding to the last entry containing string "developing" (fuzzy matching)'
-  c.example 'doing tag --interactive --tag project1 coding', desc: 'Create an interactive menu from entries tagged @project1, selection(s) will be tagged with @coding'
+  c.example 'doing tag --rename "other*" --count 10 newtag',
+            desc: 'Rename tags beginning with "other" (wildcard) to @newtag on the last 10 entries'
+  c.example 'doing tag --search "developing" coding',
+            desc: 'Add @coding to the last entry containing string "developing" (fuzzy matching)'
+  c.example 'doing tag --interactive --tag project1 coding',
+            desc: 'Create an interactive menu from entries tagged @project1, selection(s) will be tagged with @coding'
 
   c.desc 'Section'
   c.arg_name 'SECTION_NAME'
@@ -69,9 +74,7 @@ command :tag do |c|
 
     section = 'All'
 
-    if options[:section]
-      section = @wwid.guess_section(options[:section]) || options[:section].cap_first
-    end
+    section = @wwid.guess_section(options[:section]) || options[:section].cap_first if options[:section]
 
     search_tags = options[:tag].nil? ? [] : options[:tag]
 
@@ -112,16 +115,15 @@ command :tag do |c|
       matches = @wwid.filter_items([], opt: options).count
 
       if matches > 5
-        if options[:search]
-          section_q = ' matching your search terms'
-        elsif options[:tag]
-          section_q = ' matching your tag search'
-        elsif section == 'All'
-          section_q = ''
-        else
-          section_q = " in section #{section}"
-        end
-
+        section_q = if options[:search]
+                      ' matching your search terms'
+                    elsif options[:tag]
+                      ' matching your tag search'
+                    elsif section == 'All'
+                      ''
+                    else
+                      " in section #{section}"
+                    end
 
         question = if options[:autotag]
                      "Are you sure you want to autotag #{matches} records#{section_q}"

data/bin/commands/tag_dir.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@tag_dir
 desc 'Set the default tags for the current directory'
 long_desc 'Adds default_tags to a .doingrc file in the current directory. Any entry created in this directory or its
@@ -5,7 +7,8 @@ subdirectories will be tagged with the default tags. You can modify these any ti
 manually editing the .doingrc file.'
 arg_name 'TAG [TAG..]'
 command :tag_dir do |c|
-  c.example 'doing tag_dir project1 project2', desc: 'Add @project1 and @project2 to to any entries created from the current directory'
+  c.example 'doing tag_dir project1 project2',
+            desc: 'Add @project1 and @project2 to to any entries created from the current directory'
   c.example 'doing tag_dir --clear', desc: 'Clear the default tags for the directory'
 
   c.desc 'Remove all default_tags from the local .doingrc'
@@ -20,7 +23,7 @@ command :tag_dir do |c|
   c.action do |global, options, args|
     if args.empty? && !options[:clear] && !options[:editor]
       all_tags = @wwid.content.all_tags
-      $stderr.puts Doing::Color.boldwhite('Enter tags separated by spaces, tab to complete')
+      warn Doing::Color.boldwhite('Enter tags separated by spaces, tab to complete')
       input = Doing::Prompt.read_line(prompt: "Tags to #{options[:remove] ? 'remove' : 'add'}", completions: all_tags)
       tags = input.split_tags
     else

data/bin/commands/tags.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # @@tags
 desc 'List all tags in the current Doing file'
 arg_name 'MAX_COUNT', optional: true, type: Integer
@@ -27,40 +29,38 @@ command :tags do |c|
   add_options(:tag_filter, c)
 
   c.action do |_global, options, args|
-    section = @wwid.guess_section(options[:section]) || options[:section].cap_first
+    @wwid.guess_section(options[:section]) || options[:section].cap_first
     options[:count] = args.count.positive? ? args[0].to_i : 0
 
     items = @wwid.filter_items([], opt: options)
 
     if options[:interactive]
       items = Doing::Prompt.choose_from_items(items, include_section: options[:section].nil?,
-        menu: true,
-        header: '',
-        prompt: 'Select entries to scan > ',
-        multiple: true,
-        sort: true,
-        show_if_single: true)
+                                                     menu: true,
+                                                     header: '',
+                                                     prompt: 'Select entries to scan > ',
+                                                     multiple: true,
+                                                     sort: true,
+                                                     show_if_single: true)
     end
 
     # items = @wwid.content.in_section(section)
     tags = @wwid.all_tags(items, counts: true)
 
-    if options[:sort] =~ /^n/i
-      tags = tags.sort_by { |tag, count| tag }
-    else
-      tags = tags.sort_by { |tag, count| count }
-    end
+    tags = if options[:sort] =~ /^n/i
+             tags.sort_by { |tag, _count| tag }
+           else
+             tags.sort_by { |_tag, count| count }
+           end
 
     tags.reverse! if options[:order] == :desc
 
     if options[:counts]
       tags.each { |t, c| puts "#{t} (#{c})" }
+    elsif options[:line]
+      puts tags.map { |t, _c| t }.to_tags.join(' ')
     else
-      if options[:line]
-        puts tags.map { |t, c| t }.to_tags.join(' ')
-      else
-        tags.each { |t, c| puts "#{t}" }
-      end
+      tags.each_key { |t| puts t }
     end
   end
 end