doing 2.1.26 → 2.1.27

data/bin/commands/commands_accepting.rb

@@ -1,25 +1,76 @@
+# frozen_string_literal: true
+
 # @@commands_accepting
 arg_name 'OPTION'
 command :commands_accepting do |c|
   c.desc 'Output in single column for completion'
   c.switch %i[c column]
 
+  c.desc 'Join multiple arguments using boolean (AND|OR|NOT)'
+  c.flag [:bool], must_match: REGEX_BOOL,
+                  default_value: :and,
+                  type: BooleanSymbol
+
   c.action do |g, o, a|
-    a.each do |option|
-      cmds = []
-      commands.each do |cmd, v|
-        v.flags.merge(v.switches).each do |n, flag|
-          if flag.name == option.to_sym || flag.aliases&.include?(option.to_sym)
-            cmds.push(cmd)
-          end
+    cmds = []
+    commands.each { |cmd, v| cmds.push(cmd) if has_flags?(v, a, o[:bool]) }
+
+    if o[:column]
+      puts cmds.sort
+    else
+      description = "Commands "
+      description += "not " if o[:bool] == :not
+      description += "accepting "
+      description += a.map { |arg| "--#{arg}" }.join(o[:bool] == :and ? ' and ' : ' or ')
+      puts "#{description}: #{cmds.sort.join(', ')}"
+    end
+  end
+
+  def has_flags?(options, args, bool)
+    case bool
+    when :and
+      all_flags?(options, args)
+    when :not
+      no_flags?(options, args)
+    else
+      any_flags?(options, args)
+    end
+  end
+
+  def all_flags?(options, args)
+    args.each do |arg|
+      has_flag = false
+      options.flags.merge(options.switches).each do |_, flag|
+        if flag.name == arg.to_sym || flag.aliases&.include?(arg.to_sym)
+          has_flag = true
+          break
         end
       end
+      return false unless has_flag
+    end
+
+    true
+  end
+
+  def any_flags?(options, args)
+    args.each do |option|
+      options.flags.merge(options.switches).each do |_, flag|
+        return true if flag.name == option.to_sym || flag.aliases&.include?(option.to_sym)
+      end
+    end
 
-      if o[:column]
-        puts cmds.sort
-      else
-        puts "Commands accepting --#{option}: #{cmds.sort.join(', ')}"
+    false
+  end
+
+  def no_flags?(options, args)
+    args.each do |option|
+      options.flags.merge(options.switches).each do |_, flag|
+        return false if flag.name == option.to_sym || flag.aliases&.include?(option.to_sym)
       end
     end
+
+    true
   end
 end
+
+

data/bin/commands/completion.rb

@@ -1,12 +1,17 @@
+# frozen_string_literal: true
+
 # @@completion
 desc 'Generate shell completion scripts'
-long_desc 'Generates the necessary scripts to add command line completion to various shells, so typing \'doing\' and hitting
-tab will offer completions of subcommands and their options.'
+long_desc 'Generates the necessary scripts to add command line completion to various shells,
+so typing \'doing\' and hitting tab will offer completions of subcommands and their options.'
 command :completion do |c|
   c.example 'doing completion', desc: 'Output zsh (default) to STDOUT'
-  c.example 'doing completion --type zsh --file ~/.zsh-completions/_doing.zsh', desc: 'Output zsh completions to file'
-  c.example 'doing completion --type fish --file ~/.config/fish/completions/doing.fish', desc: 'Output fish completions to file'
-  c.example 'doing completion --type bash --file ~/.bash_it/completion/enabled/doing.bash', desc: 'Output bash completions to file'
+  c.example 'doing completion --type zsh --file ~/.zsh-completions/_doing.zsh',
+            desc: 'Output zsh completions to file'
+  c.example 'doing completion --type fish --file ~/.config/fish/completions/doing.fish',
+            desc: 'Output fish completions to file'
+  c.example 'doing completion --type bash --file ~/.bash_it/completion/enabled/doing.bash',
+            desc: 'Output bash completions to file'
 
   c.desc 'Shell to generate for (bash, zsh, fish)'
   c.arg_name 'SHELL'
@@ -17,8 +22,6 @@ command :completion do |c|
   c.flag %i[f file], default_value: 'STDOUT'
 
   c.action do |_global_options, options, _args|
-    script_dir = File.join(File.dirname(__FILE__), '..', 'scripts')
-
     Doing::Completion.generate_completion(type: options[:type], file: options[:file])
   end
 end

data/bin/commands/config.rb

@@ -218,7 +218,7 @@ command :config do |c|
 
       config_file = @config.choose_config(create: true)
 
-      cfg = YAML.safe_load_file(config_file) || {}
+      cfg = Doing::Util.safe_load_file(config_file) || {}
 
       $stderr.puts "Updating #{config_file}".yellow
 

data/bin/commands/done.rb

@@ -24,10 +24,6 @@ command %i[done did] do |c|
   c.arg_name 'DATE_STRING'
   c.flag %i[at finished], type: DateEndString
 
-  c.desc 'Backdate start date by interval or set to time [4pm|20m|2h|"yesterday noon"]'
-  c.arg_name 'DATE_STRING'
-  c.flag %i[b back started], type: DateBeginString
-
   c.desc %(
         Start and end times as a date/time range `doing done --from "1am to 8am"`.
         Overrides other date flags.
@@ -45,24 +41,14 @@ command %i[done did] do |c|
   c.arg_name 'NAME'
   c.flag %i[s section]
 
-  c.desc "Edit entry with #{Doing::Util.default_editor} (with no arguments, edits the last entry)"
-  c.switch %i[e editor], negatable: false, default_value: false
-
-  c.desc 'Include a note'
-  c.arg_name 'TEXT'
-  c.flag %i[n note]
-
-  c.desc 'Prompt for note via multi-line input'
-  c.switch %i[ask], negatable: false, default_value: false
-
   c.desc 'Finish last entry not already marked @done'
   c.switch %i[u unfinished], negatable: false, default_value: false
 
-  # c.desc "Edit entry with specified app"
-  # c.arg_name 'editor_app'
-  # # c.flag [:a, :app]
+  add_options(:add_entry, c)
 
   c.action do |_global_options, options, args|
+    @wwid.auto_tag = !options[:noauto]
+
     took = 0
     donedate = nil
 

data/bin/commands/finish.rb

@@ -22,37 +22,10 @@ command :finish do |c|
   c.arg_name 'DATE_STRING'
   c.flag %i[at finished], type: DateEndString
 
-  c.desc 'Finish the last X entries containing TAG.
-  Separate multiple tags with comma (--tag=tag1,tag2), combine with --bool. Wildcards allowed (*, ?).'
-  c.arg_name 'TAG'
-  c.flag [:tag], type: TagArray
+  c.desc 'Overwrite existing @done tag with new date'
+  c.switch %i[update], negatable: false, default_value: false
 
-  c.desc 'Finish the last X entries matching search filter, surround with slashes for regex (e.g. "/query.*/"), start with single quote for exact match ("\'query")'
-  c.arg_name 'QUERY'
-  c.flag [:search]
-
-  c.desc 'Perform a tag value query ("@done > two hours ago" or "@progress < 50"). May be used multiple times, combined with --bool'
-  c.arg_name 'QUERY'
-  c.flag [:val], multiple: true, must_match: REGEX_VALUE_QUERY
-
-  # c.desc '[DEPRECATED] Use alternative fuzzy matching for search string'
-  # c.switch [:fuzzy], default_value: false, negatable: false
-
-  c.desc 'Force exact search string matching (case sensitive)'
-  c.switch %i[x exact], default_value: @config.exact_match?, negatable: @config.exact_match?
-
-  c.desc 'Finish items that *don\'t* match search/tag filters'
-  c.switch [:not], default_value: false, negatable: false
-
-  c.desc 'Case sensitivity for search string matching [(c)ase-sensitive, (i)gnore, (s)mart]'
-  c.arg_name 'TYPE'
-  c.flag [:case], must_match: /^[csi]/, default_value: @settings.dig('search', 'case')
-
-  c.desc 'Boolean (AND|OR|NOT) with which to combine multiple tag filters. Use PATTERN to parse + and - as booleans'
-  c.arg_name 'BOOLEAN'
-  c.flag [:bool], must_match: REGEX_BOOL, default_value: 'PATTERN'
-
-  c.desc 'Remove done tag'
+  c.desc 'Remove @done tag'
   c.switch %i[r remove], negatable: false, default_value: false
 
   c.desc 'Finish last entry (or entries) not already marked @done'
@@ -73,6 +46,9 @@ command :finish do |c|
   c.desc 'Select item(s) to finish from a menu of matching entries'
   c.switch %i[i interactive], negatable: false, default_value: false
 
+  add_options(:search, c)
+  add_options(:tag_filter, c)
+
   c.action do |_global_options, options, args|
     options[:fuzzy] = false
     unless options[:auto]
@@ -141,6 +117,7 @@ command :finish do |c|
       tags: ['done'],
       took: options[:took],
       unfinished: options[:unfinished],
+      update: options[:update],
       val: options[:val]
     }
 

data/bin/commands/flag.rb

@@ -26,39 +26,12 @@ command %i[mark flag] do |c|
   c.desc 'Flag last entry (or entries) not marked @done'
   c.switch %i[u unfinished], negatable: false, default_value: false
 
-  c.desc 'Flag the last entry containing TAG.
-  Separate multiple tags with comma (--tag=tag1,tag2), combine with --bool. Wildcards allowed (*, ?).'
-  c.arg_name 'TAG'
-  c.flag [:tag], type: TagArray
-
-  c.desc 'Flag the last entry matching search filter, surround with slashes for regex (e.g. "/query.*/"), start with single quote for exact match ("\'query")'
-  c.arg_name 'QUERY'
-  c.flag [:search]
-
-  c.desc 'Perform a tag value query ("@done > two hours ago" or "@progress < 50"). May be used multiple times, combined with --bool'
-  c.arg_name 'QUERY'
-  c.flag [:val], multiple: true, must_match: REGEX_VALUE_QUERY
-
-  # c.desc '[DEPRECATED] Use alternative fuzzy matching for search string'
-  # c.switch [:fuzzy], default_value: false, negatable: false
-
-  c.desc 'Force exact search string matching (case sensitive)'
-  c.switch %i[x exact], default_value: @config.exact_match?, negatable: @config.exact_match?
-
-  c.desc 'Flag items that *don\'t* match search/tag/date filters'
-  c.switch [:not], default_value: false, negatable: false
-
-  c.desc 'Case sensitivity for search string matching [(c)ase-sensitive, (i)gnore, (s)mart]'
-  c.arg_name 'TYPE'
-  c.flag [:case], must_match: /^[csi]/, default_value: @settings.dig('search', 'case')
-
-  c.desc 'Boolean (AND|OR|NOT) with which to combine multiple tag filters. Use PATTERN to parse + and - as booleans'
-  c.arg_name 'BOOLEAN'
-  c.flag [:bool], must_match: REGEX_BOOL, default_value: 'PATTERN'
-
   c.desc 'Select item(s) to flag from a menu of matching entries'
   c.switch %i[i interactive], negatable: false, default_value: false
 
+  add_options(:search, c)
+  add_options(:tag_filter, c)
+
   c.action do |_global_options, options, _args|
     options[:fuzzy] = false
     mark = @settings['marker_tag'] || 'flagged'
@@ -67,15 +40,9 @@ command %i[mark flag] 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]
 
-    if options[:tag].nil?
-      search_tags = []
-    else
-      search_tags = options[:tag]
-    end
+    search_tags = options[:tag].nil? ? [] : options[:tag]
 
     if options[:interactive]
       count = 0
@@ -84,8 +51,6 @@ command %i[mark flag] do |c|
       count = options[:count].to_i
     end
 
-    options[:case] = options[:case].normalize_case
-
     if options[:search]
       search = options[:search]
       search.sub!(/^'?/, "'") if options[:exact]
@@ -93,16 +58,15 @@ command %i[mark flag] do |c|
     end
 
     if count.zero? && !options[:force]
-      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[:remove]
                    "Are you sure you want to unflag all entries#{section_q}"
@@ -119,7 +83,7 @@ command %i[mark flag] do |c|
     options[:section] = section
     options[:tag] = search_tags
     options[:tags] = [mark]
-    options[:tag_bool] = options[:bool].normalize_bool
+    options[:tag_bool] = options[:bool]
 
     @wwid.tag_last(options)
   end

data/bin/commands/grep.rb

@@ -16,25 +16,6 @@ command %i[grep search] do |c|
   c.arg_name 'NAME'
   c.flag %i[s section], default_value: 'All'
 
-  c.desc 'Search entries older than date. If this is only a time (8am, 1:30pm, 15:00), all dates will be included, but entries will be filtered by time of day'
-  c.arg_name 'DATE_STRING'
-  c.flag [:before], type: DateBeginString
-
-  c.desc 'Search entries newer than date. If this is only a time (8am, 1:30pm, 15:00), all dates will be included, but entries will be filtered by time of day'
-  c.arg_name 'DATE_STRING'
-  c.flag [:after], type: DateEndString
-
-  c.desc %(
-      Date range to show, or a single day to filter date on.
-      Date range argument should be quoted. Date specifications can be natural language.
-      To specify a range, use "to" or "through": `doing search --from "monday 8am to friday 5pm"`.
-
-      If values are only time(s) (6am to noon) all dates will be included, but entries will be filtered
-      by time of day.
-    )
-  c.arg_name 'DATE_OR_RANGE'
-  c.flag [:from], type: DateRangeString
-
   c.desc "Output to export format (#{Doing::Plugins.plugin_names(type: :export)})"
   c.arg_name 'FORMAT'
   c.flag %i[o output]
@@ -57,10 +38,9 @@ command %i[grep search] do |c|
   c.switch [:totals], default_value: false, negatable: false
 
   c.desc 'Sort tags by (name|time)'
-  default = 'time'
-  default = @settings['tag_sort'] || 'name'
+  default = @settings['tag_sort'].normalize_tag_sort || :name
   c.arg_name 'KEY'
-  c.flag [:tag_sort], must_match: /^(?:name|time)$/i, default_value: default
+  c.flag [:tag_sort], must_match: REGEX_TAG_SORT, default_value: default, type: TagSortSymbol
 
   c.desc 'Only show items with recorded time intervals'
   c.switch [:only_timed], default_value: false, negatable: false
@@ -76,7 +56,9 @@ command %i[grep search] do |c|
 
   c.desc 'Case sensitivity for search string matching [(c)ase-sensitive, (i)gnore, (s)mart]'
   c.arg_name 'TYPE'
-  c.flag [:case], must_match: /^[csi]/, default_value: @settings.dig('search', 'case')
+  c.flag [:case], must_match: REGEX_CASE,
+                  default_value: @settings.dig('search', 'case').normalize_case,
+                  type: CaseSymbol
 
   c.desc "Highlight search matches in output. Only affects command line output"
   c.switch %i[h hilite], default_value: @settings.dig('search', 'highlight')
@@ -95,7 +77,12 @@ command %i[grep search] do |c|
   c.flag [:val], multiple: true, must_match: REGEX_VALUE_QUERY
 
   c.desc 'Combine multiple tags or value queries using AND, OR, or NOT'
-  c.flag [:bool], must_match: REGEX_BOOL, default_value: 'AND'
+  c.arg_name 'BOOLEAN'
+  c.flag [:bool], must_match: REGEX_BOOL,
+                  default_value: :pattern,
+                  type: BooleanSymbol
+
+  add_options(:date_filter, c)
 
   c.action do |_global_options, options, args|
     options[:fuzzy] = false
@@ -106,14 +93,11 @@ command %i[grep search] do |c|
 
     section = @wwid.guess_section(options[:section]) if options[:section]
 
-    options[:case] = options[:case].normalize_case
-    options[:bool] = options[:bool].normalize_bool
-
     search = args.join(' ')
     search.sub!(/^'?/, "'") if options[:exact]
 
     options[:times] = true if options[:totals]
-    options[:sort_tags] = options[:tag_sort] =~ /^n/i
+    options[:sort_tags] = options[:tag_sort]
     options[:highlight] = true
     options[:search] = search
     options[:section] = section

data/bin/commands/import.rb

@@ -11,23 +11,9 @@ command :import do |c|
   c.arg_name 'TYPE'
   c.flag :type, default_value: 'doing'
 
-  c.desc 'Only import items matching search. Surround with slashes for regex (/query/), start with single quote for exact match ("\'query")'
-  c.arg_name 'QUERY'
-  c.flag [:search]
-
-  # c.desc '[DEPRECATED] Use alternative fuzzy matching for search string'
-  # c.switch [:fuzzy], default_value: false, negatable: false
-
-  c.desc 'Force exact search string matching (case sensitive)'
-  c.switch %i[x exact], default_value: @config.exact_match?, negatable: @config.exact_match?
-
   c.desc 'Import items that *don\'t* match search/tag/date filters'
   c.switch [:not], default_value: false, negatable: false
 
-  c.desc 'Case sensitivity for search string matching [(c)ase-sensitive, (i)gnore, (s)mart]'
-  c.arg_name 'TYPE'
-  c.flag [:case], must_match: /^[csi]/, default_value: @settings.dig('search', 'case')
-
   c.desc 'Only import items with recorded time intervals'
   c.switch [:only_timed], default_value: false, negatable: false
 
@@ -46,26 +32,12 @@ command :import do |c|
   c.arg_name 'PREFIX'
   c.flag :prefix
 
-  # TODO: Allow time range filtering
-  c.desc 'Import entries older than date'
-  c.arg_name 'DATE_STRING'
-  c.flag [:before], type: DateBeginString
-
-  c.desc 'Import entries newer than date'
-  c.arg_name 'DATE_STRING'
-  c.flag [:after], type: DateEndString
-
-  c.desc %(
-    Date range to import. Date range argument should be quoted. Date specifications can be natural language.
-    To specify a range, use "to" or "through": `--from "monday to friday"` or `--from 10/1 to 10/31`.
-    Has no effect unless the import plugin has implemented date range filtering.
-  )
-  c.arg_name 'DATE_OR_RANGE'
-  c.flag %i[f from], type: DateRangeString
-
   c.desc 'Allow entries that overlap existing times'
   c.switch [:overlap], negatable: true
 
+  add_options(:search, c)
+  add_options(:date_filter, c)
+
   c.action do |_global_options, options, args|
     options[:fuzzy] = false
     if options[:section]
@@ -88,8 +60,6 @@ command :import do |c|
       options[:date_filter][0] = options[:after] || Time.now - (1 << 64)
     end
 
-    options[:case] = options[:case].normalize_case
-
     if options[:type] =~ Doing::Plugins.plugin_regex(type: :import)
       options[:no_overlap] = !options[:overlap]
       @wwid.import(args, options)

data/bin/commands/last.rb

@@ -21,18 +21,6 @@ command :last do |c|
   c.desc "Delete the last entry"
   c.switch %i[d delete], negatable: false, default_value: false
 
-  c.desc 'Tag filter, combine multiple tags with a comma. Wildcards allowed (*, ?)'
-  c.arg_name 'TAG'
-  c.flag [:tag], type: TagArray
-
-  c.desc 'Tag boolean (AND|OR|NOT). Use PATTERN to parse + and - as booleans'
-  c.arg_name 'BOOLEAN'
-  c.flag [:bool], must_match: REGEX_BOOL, default_value: 'PATTERN'
-
-  c.desc 'Search filter, surround with slashes for regex (/query/), start with single quote for exact match ("\'query")'
-  c.arg_name 'QUERY'
-  c.flag [:search]
-
   c.desc "Output using a template from configuration"
   c.arg_name 'TEMPLATE_KEY'
   c.flag [:config_template], type: TemplateName, default_value: 'last'
@@ -44,38 +32,17 @@ command :last do |c|
   c.desc "Highlight search matches in output. Only affects command line output"
   c.switch %i[h hilite], default_value: @settings.dig('search', 'highlight')
 
-  c.desc 'Perform a tag value query ("@done > two hours ago" or "@progress < 50"). May be used multiple times, combined with --bool'
-  c.arg_name 'QUERY'
-  c.flag [:val], multiple: true, must_match: REGEX_VALUE_QUERY
-
   c.desc 'Show elapsed time if entry is not tagged @done'
   c.switch [:duration]
 
-  # c.desc '[DEPRECATED] Use alternative fuzzy matching for search string'
-  # c.switch [:fuzzy], default_value: false, negatable: false
-
-  c.desc 'Force exact search string matching (case sensitive)'
-  c.switch %i[x exact], default_value: @config.exact_match?, negatable: @config.exact_match?
-
-  c.desc 'Show items that *don\'t* match search string or tag filter'
-  c.switch [:not], default_value: false, negatable: false
-
-  c.desc 'Case sensitivity for search string matching [(c)ase-sensitive, (i)gnore, (s)mart]'
-  c.arg_name 'TYPE'
-  c.flag [:case], must_match: /^[csi]/, default_value: @settings.dig('search', 'case')
+  add_options(:search, c)
+  add_options(:tag_filter, c)
 
   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]
 
-    if options[:tag].nil?
-      options[:tag] = []
-    else
-      options[:tag] = options[:tag]
-      options[:bool] = options[:bool].normalize_bool
-    end
-
-    options[:case] = options[:case].normalize_case
+    options[:tag] ||= []
 
     options[:search] = options[:search].sub(/^'?/, "'") if options[:search] && options[:exact]
 

data/bin/commands/meanwhile.rb

@@ -15,24 +15,14 @@ command :meanwhile do |c|
   c.arg_name 'NAME'
   c.flag %i[s section]
 
-  c.desc "Edit entry with #{Doing::Util.default_editor}"
-  c.switch %i[e editor], negatable: false, default_value: false
-
   c.desc 'Archive previous @meanwhile entry'
   c.switch %i[a archive], negatable: false, default_value: false
 
-  c.desc 'Backdate start date for new entry to date string [4pm|20m|2h|yesterday noon]'
-  c.arg_name 'DATE_STRING'
-  c.flag %i[b back started], type: DateBeginString
-
-  c.desc 'Note'
-  c.arg_name 'TEXT'
-  c.flag %i[n note]
-
-  c.desc 'Prompt for note via multi-line input'
-  c.switch %i[ask], negatable: false, default_value: false
+  add_options(:add_entry, c)
 
   c.action do |_global_options, options, args|
+    @wwid.auto_tag = !options[:noauto]
+
     if options[:back]
       date = options[:back]
 

data/bin/commands/note.rb

@@ -24,50 +24,19 @@ command :note do |c|
   c.desc "Replace/Remove last entry's note (default append)"
   c.switch %i[r remove], negatable: false, default_value: false
 
-  c.desc 'Add/remove note from last entry matching tag. Wildcards allowed (*, ?)'
-  c.arg_name 'TAG'
-  c.flag [:tag], type: TagArray
-
-  c.desc 'Add/remove note from last entry matching search filter, surround with slashes for regex (e.g. "/query.*/"), start with single quote for exact match ("\'query")'
-  c.arg_name 'QUERY'
-  c.flag [:search]
-
-  c.desc 'Perform a tag value query ("@done > two hours ago" or "@progress < 50"). May be used multiple times, combined with --bool'
-  c.arg_name 'QUERY'
-  c.flag [:val], multiple: true, must_match: REGEX_VALUE_QUERY
-
-  # c.desc '[DEPRECATED] Use alternative fuzzy matching for search string'
-  # c.switch [:fuzzy], default_value: false, negatable: false
-
-  c.desc 'Force exact search string matching (case sensitive)'
-  c.switch %i[x exact], default_value: @config.exact_match?, negatable: @config.exact_match?
-
-  c.desc 'Add note to item that *doesn\'t* match search/tag filters'
-  c.switch [:not], default_value: false, negatable: false
-
-  c.desc 'Case sensitivity for search string matching [(c)ase-sensitive, (i)gnore, (s)mart]'
-  c.arg_name 'TYPE'
-  c.flag [:case], must_match: /^[csi]/, default_value: @settings.dig('search', 'case')
-
-  c.desc 'Boolean (AND|OR|NOT) with which to combine multiple tag filters. Use PATTERN to parse + and - as booleans'
-  c.arg_name 'BOOLEAN'
-  c.flag [:bool], must_match: REGEX_BOOL, default_value: 'PATTERN'
-
   c.desc 'Select item for new note from a menu of matching entries'
   c.switch %i[i interactive], negatable: false, default_value: false
 
   c.desc 'Prompt for note via multi-line input'
   c.switch %i[ask], negatable: false, default_value: false
 
+  add_options(:search, c)
+  add_options(:tag_filter, 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[:tag_bool] = options[:bool].normalize_bool
-
-    options[:case] = options[:case].normalize_case
+    options[:section] = @wwid.guess_section(options[:section]) || options[:section].cap_first if options[:section]
+    options[:tag_bool] = options[:bool]
 
     if options[:search]
       search = options[:search]
@@ -78,30 +47,22 @@ command :note do |c|
     last_entry = @wwid.last_entry(options)
     old_entry = last_entry.clone
 
-    unless last_entry
-      Doing.logger.warn('Not found:', 'No entry matching parameters was found.')
-      return
-    end
+    raise NoResults, 'No entry matching parameters was found.' unless last_entry
 
     last_note = last_entry.note || Doing::Note.new
     new_note = Doing::Note.new
 
-    if $stdin.stat.size.positive?
-      new_note.add($stdin.read.strip)
-    end
-
-    unless args.empty?
-      new_note.add(args.join(' '))
-    end
+    new_note.add($stdin.read.strip) if $stdin.stat.size.positive?
+    new_note.add(args.join(' ')) unless args.empty?
 
     if options[:editor]
       raise MissingEditor, 'No EDITOR variable defined in environment' if Doing::Util.default_editor.nil?
 
-      if options[:remove]
-        input = Doing::Note.new
-      else
-        input = last_entry.note || Doing::Note.new
-      end
+      input = if options[:remove]
+                Doing::Note.new
+              else
+                last_entry.note || Doing::Note.new
+              end
 
       input.add(new_note)