squared 0.4.14 → 0.5.0

data/lib/squared/workspace/project/base.rb

@@ -99,10 +99,15 @@ module Squared
             last: last,
             error: error
           }
-          @envname = env_key(@name).freeze
+          @envname = @name.gsub(/[^\w]+/, '_').upcase.freeze
           @desc = (@name.include?(':') ? @name.split(':').join(ARG[:SPACE]) : @name).freeze
           @parent = nil
           @global = false
+          @log = nil
+          @dev = nil
+          @prod = nil
+          @withargs = nil
+          @session = nil
           @index = -1
           run_set(kwargs[:run], kwargs[:env], opts: kwargs.fetch(:opts, true))
           exception_set kwargs[:exception]
@@ -200,7 +205,7 @@ module Squared
             file &&= @workspace.home.join(env('LOG_DIR', ''), file).realdirpath
           rescue StandardError => e
             file = nil
-            warn log_message(Logger::WARN, e) if warning?
+            warn log_message(Logger::WARN, e, pass: true) if warning?
           end
           log[:progname] ||= @name
           if (val = env('LOG_LEVEL', ignore: false))
@@ -245,14 +250,14 @@ module Squared
           return 1 if a.include?(other)
 
           c, d = graph_deps other
-          e = b - d
-          f = d - b
+          e = b.reject { |val| d.include?(val) }
+          f = d.reject { |val| b.include?(val) }
           if parent == other.parent
             g = []
             h = []
           else
-            g = a - c
-            h = c - a
+            g = a.reject { |val| c.include?(val) }
+            h = c.reject { |val| a.include?(val) }
           end
           g << self
           h << other
@@ -266,8 +271,8 @@ module Squared
             -1
           elsif f.any? { |val| e.include?(val) }
             1
-          elsif @index != -1 && (i = other.instance_variable_get(:@index)) != -1
-            @index < i ? -1 : 1
+          elsif @index >= 0 && (i = other.instance_variable_get(:@index)) >= 0
+            @index <= i ? -1 : 1
           else
             0
           end
@@ -296,31 +301,31 @@ module Squared
 
                     format_desc action, flag, '(-)project*'
                     task flag do |_, args|
-                      args = args.to_a.reject { |val| name == val.to_s }
+                      args = args.to_a.reject { |val| name == val }
                       if flag == :run
                         graph args
                       else
                         out, done = graph(args, out: [])
                         out.map! do |val|
                           done.each_with_index do |proj, i|
-                            next unless val.match?(/ #{Regexp.escape(proj.name)}(?:@\d|\z)/)
-
-                            val += " (#{i.succ})"
-                            break
+                            if val.match?(/ #{Regexp.escape(proj.name)}(?:@\d|\z)/)
+                              val += " (#{i.succ})"
+                              break
+                            end
                           end
                           val
                         end
                         emphasize(out, title: path, right: true, border: borderstyle, sub: [
                           { pat: /\A(#{Regexp.escape(path.to_s)})(.*)\z/, styles: theme[:header] },
                           { pat: /\A(#{Regexp.escape(name)})(.*)\z/, styles: theme[:active] },
-                          { pat: /\A(.+ )(\()(\d+)(\))(.*)\z/, styles: theme[:inline], index: 3 }
+                          { pat: /\A((?~ \() \()(\d+)(\).*)\z/, styles: theme[:inline], index: 2 }
                         ])
                       end
                     end
                   when 'unpack'
                     format_desc(action, flag, 'tag/url,dir,digest?,f|force?', before: flag == :ext ? 'ext' : nil)
                     params = %i[tag dir digest force]
-                    params.unshift(:ext) if flag == :ext
+                    params.prepend(:ext) if flag == :ext
                     task flag, params do |_, args|
                       ext = flag == :ext ? param_guard(action, flag, args: args, key: :ext) : flag.to_s
                       tag = param_guard(action, flag, args: args, key: :tag)
@@ -375,17 +380,8 @@ module Squared
             return self
           elsif !projectpath?(path = basepath(path)) || !checkdir?(path)
             return self
-          else
-            name = case name
-                   when String, Symbol
-                     name.to_s
-                   end
-          end
-          if @withargs
-            data = @withargs.dup
-            data.merge!(kwargs)
-            kwargs = data
           end
+          kwargs = @withargs.yield_self { |data| data.dup.update(kwargs) } if @withargs
           kwargs[:group] = group unless kwargs.key?(:group)
           kwargs[:ref] = ref unless kwargs.key?(:ref)
           parent = self
@@ -415,7 +411,7 @@ module Squared
 
           out = obj.link(self, *args, **kwargs, &blk) if obj.respond_to?(:link)
           if !out
-            warn log_message(Logger::WARN, 'link not compatible', subject: obj.to_s, hint: name)
+            warn log_message(Logger::WARN, 'link not compatible', subject: obj.to_s, hint: name, pass: true)
           elsif out.respond_to?(:build)
             out.build
           end
@@ -457,7 +453,7 @@ module Squared
 
                 cmd << a if (a = compose(as_get(b), d, script: true, args: e, from: from))
               end
-              var.merge!(c) if c.is_a?(Hash)
+              var.update(c) if c.is_a?(Hash)
             end
             cmd = cmd.join(' && ')
           else
@@ -470,9 +466,9 @@ module Squared
             case opts
             when Hash
               opts = append_hash(opts, build: true)
-              cmd = Array(cmd).push(flags).concat(opts).compact.join(' ')
+              cmd = as_a(cmd).append(flags).concat(opts).compact.join(' ')
             when Enumerable
-              cmd = Array(cmd).concat(opts.to_a)
+              cmd = as_a(cmd).concat(opts.to_a)
               cmd.map! { |val| "#{val} #{flags}" } if flags
               cmd = cmd.join(' && ')
             else
@@ -508,7 +504,9 @@ module Squared
                     next
                   end
                 end
-                warn log_message(Logger::WARN, name, 'method not found', subject: 'prereqs', hint: meth)
+                if warning?
+                  warn log_message(Logger::WARN, name, 'method not found', subject: 'prereqs', hint: meth, pass: true)
+                end
               end
             elsif proj.build?
               proj.build(sync: sync)
@@ -620,6 +618,7 @@ module Squared
         end
 
         def unpack(target, uri:, sync: true, digest: nil, ext: nil, force: false, depth: 1, headers: {}, from: :unpack)
+          require 'open-uri'
           if !target.exist?
             target.mkpath
           elsif !target.directory?
@@ -652,8 +651,8 @@ module Squared
             headers = headers.is_a?(Hash) ? headers.merge(val) : val
           end
           data = nil
-          (uri = Array(uri)).each_with_index do |url, index|
-            fetch_uri(url, headers) do |f|
+          (uri = as_a(uri)).each_with_index do |url, index|
+            URI.open(url, headers) do |f|
               data = f.read
               if algo && algo.hexdigest(data) != digest
                 data = nil
@@ -690,7 +689,7 @@ module Squared
             file.write(data)
             file.close
             if create
-              warn log_message(Logger::WARN, 'force remove', subject: name, hint: target)
+              warn log_message(Logger::WARN, 'force remove', subject: name, hint: target, pass: true)
               target.rmtree
               target.mkpath
             end
@@ -725,7 +724,7 @@ module Squared
                 i += 1
               end
               FileUtils.mv(entry, dest)
-              dest.children.each { |child| FileUtils.mv(child, target) }
+              dest.each_child { |child| FileUtils.mv(child, target) }
               dest.rmdir
               target = entry
               depth -= 1
@@ -783,6 +782,13 @@ module Squared
         end
 
         def variable_set(key, *args, **kwargs, &blk)
+          if block_given?
+            if blocks.include?(key)
+              series key, &blk
+              return self
+            end
+            args = block_args args, &blk
+          end
           if variables.include?(key) || blocks.include?(key)
             val = case args.size
                   when 0
@@ -803,8 +809,6 @@ module Squared
               exclude_set val
             when :parent
               parent_set val
-            when :archive
-              archive_set val
             when :run
               run_set(*args, **kwargs)
             when :script
@@ -814,14 +818,7 @@ module Squared
             when :dependfile
               @dependfile = basepath(*args)
             else
-              if block_given?
-                if blocks.include?(key)
-                  series key, &blk
-                  return self
-                end
-                val = block_args val, &blk
-              end
-              instance_variable_set(:"@#{key}", val.first)
+              instance_variable_set(:"@#{key}", val)
             end
           else
             log&.warn "variable_set: @#{key} (private)"
@@ -966,32 +963,20 @@ module Squared
           puts_oe(*args, pipe: pipe)
         end
 
-        def run(cmd = @session, var = nil, exception: @exception, sync: true, from: nil, banner: true, chdir: path,
-                interactive: nil, **)
+        def run(cmd = @session, var = nil, exception: @exception, sync: true, from: nil, banner: true, chdir: path, **)
           unless cmd
-            from = from&.to_s || 'unknown'
-            return warn log_message(Logger::WARN, 'no command given', subject: project, hint: from, pass: true)
+            if warning?
+              from &&= from.to_s
+              warn log_message(Logger::WARN, from || 'unknown', subject: project, hint: 'no command given', pass: true)
+            end
+            return
           end
-          i = interactive && !(@session && option('y'))
           cmd = cmd.target if cmd.is_a?(OptionPartition)
           cmd = session_done cmd
-          if i
-            title, y = case interactive
-                       when Array
-                         interactive
-                       when String
-                         [interactive, 'N']
-                       else
-                         ['Run', 'Y']
-                       end
-            unless confirm("#{title}? [#{sub_style(cmd, styles: theme[:inline])}] #{y == 'Y' ? '[Y/n]' : '[y/N]'} ", y)
-              raise_error('user cancelled', hint: from)
-            end
-          end
           log&.info cmd
           on :first, from
           begin
-            if cmd.match?(/\A[^:]+:[^:]/) && workspace.task_defined?(cmd)
+            if cmd.start_with?(/[^:]+:[^:]/) && workspace.task_defined?(cmd)
               log&.warn "ENV discarded: #{var}" if var
               task_invoke(cmd, exception: exception, warning: warning?)
             else
@@ -1003,7 +988,7 @@ module Squared
                 when Enumerable
                   cmd = command(*pre.to_a, cmd)
                 else
-                  cmd = command(pre, cmd)
+                  cmd = command pre, cmd
                 end
               end
               args = var.is_a?(Hash) ? [var, cmd] : [cmd]
@@ -1048,7 +1033,7 @@ module Squared
             elsif obj.is_a?(Array) && obj.any? { |val| !val.is_a?(String) }
               build(*obj, **kwargs)
             elsif obj
-              run_s(*Array(obj), **kwargs)
+              run_s(obj.is_a?(Enumerable) ? obj.to_a : obj, **kwargs)
             end
           end
         end
@@ -1095,7 +1080,7 @@ module Squared
 
             t = dedupe.call(proj.name)
             j = if out
-                  if i == items.size - 1 || check.call(post = items[(i + 1)..-1]).empty?
+                  if i == items.size - 1 || check.call(post = items[i + 1..-1]).empty?
                     true
                   elsif !t.empty? && depth > 0
                     post.reject { |pr| t.include?(pr) }.empty?
@@ -1107,9 +1092,9 @@ module Squared
             end
             if !out
               if !tasks && (script = workspace.script_get(:graph, group: proj.group, ref: proj.allref))
-                tasks = script[:graph]
+                group = script[:graph]
               end
-              (tasks || (dev? ? ['build', 'copy'] : ['depend', 'build'])).each do |meth|
+              (tasks || group || (dev? ? ['build', 'copy'] : ['depend', 'build'])).each do |meth|
                 next if pass.include?(meth)
 
                 if workspace.task_defined?(cmd = task_join(proj.name, meth))
@@ -1120,7 +1105,7 @@ module Squared
                   end
                   run(cmd, sync: false, banner: false)
                   ENV.delete(key) if key
-                elsif proj.has?(meth, tasks ? nil : workspace.baseref)
+                elsif proj.has?(meth, tasks || group ? nil : workspace.baseref)
                   proj.__send__(meth.to_sym, sync: sync)
                 end
               end
@@ -1157,6 +1142,7 @@ module Squared
             else
               items = workspace.find(group: val, ref: val.to_sym)
             end
+
             items.each do |proj|
               next if pass.include?(proj.name)
 
@@ -1201,7 +1187,7 @@ module Squared
                 end
           return ret == equals.to_s unless equals.nil?
 
-          ret.empty? || (ignore && Array(ignore).any? { |val| ret == val.to_s }) ? default : ret
+          ret.empty? || (ignore && as_a(ignore).any? { |val| ret == val.to_s }) ? default : ret
         end
 
         def session(*cmd, prefix: cmd.first, main: true, path: true, options: true)
@@ -1249,18 +1235,15 @@ module Squared
         def option(*args, target: @session, prefix: target&.first, **kwargs)
           if prefix
             args.each do |val|
-              ret = env(env_key(stripext(prefix), val), **kwargs)
-              return ret if ret
+              next unless (ret = env("#{stripext(prefix)}_#{val.gsub(/\W/, '_')}".upcase, **kwargs))
+              return ret unless block_given?
+
+              return yield ret
             end
           end
           nil
         end
 
-        def option_sanitize(opts, list, target: @session, **kwargs)
-          op = OptionPartition.new(opts, list, target, project: self, **kwargs)
-          [op.extras, op.values]
-        end
-
         def option_clear(opts, target: @session, **kwargs)
           return unless target
 
@@ -1286,7 +1269,7 @@ module Squared
               styles = [:bold] + styles
             end
           end
-          n = Project.max_width(lines)
+          n = line_width lines
           ch = ' ' * pad
           index = -1
           lines.map! do |val|
@@ -1302,7 +1285,7 @@ module Squared
         end
 
         def print_footer(*lines, sub: nil, reverse: false, right: false, **kwargs)
-          n = Project.max_width(lines)
+          n = line_width lines
           lines.map! do |val|
             s = right ? val.rjust(n) : val.ljust(n)
             sub&.each { |h| s = sub_style(s, **h) }
@@ -1313,46 +1296,33 @@ module Squared
           ret.join("\n")
         end
 
-        def print_status(*args, from: nil)
-          return if stdin?
-
-          case from
-          when :outdated
-            out = print_footer("major #{args[0]} / minor #{args[1]} / patch #{args[2]}", right: true).split("\n")
-            out[1] = sub_style(out[1], pat: /^( +major )(\d+)(.+)$/, styles: theme[:major], index: 2)
-            out[1] = sub_style(out[1], pat: /^(.+)(minor )(\d+)(.+)$/, styles: theme[:active], index: 3)
-            puts out
-          end
-        end
-
         def format_desc(action, flag, opts = nil, **kwargs)
           return unless TASK_METADATA
 
-          ret = [@desc, action]
-          ret << flag if flag
-          workspace.format_desc(ret, opts, **kwargs)
+          workspace.format_desc([@desc, action, flag].compact, opts, **kwargs)
         end
 
         def format_banner(cmd, banner: true)
           return unless banner && banner?
 
           if (data = workspace.banner_get(*@ref, group: group))
-            return if data.empty?
+            return if !data.command && data.order.empty?
 
             client = true
           else
-            data = { command: true, order: [:path], styles: theme[:banner], border: theme[:border] }
+            data = Workspace::Support::BannerData.new(true, [:path], theme[:banner], theme[:border])
           end
           if verbose
             out = []
-            if data[:command]
+            if data.command
               if cmd =~ /\A(?:"((?:[^"]|(?<=\\)")+)"|'((?:[^']|(?<=\\)')+)'|(\S+))( |\z)/
                 path = $3 || $2 || $1
-                cmd = cmd.sub(path, stripext(path).upcase)
+                name = stripext path
+                cmd = cmd.sub(path, data.command == 0 ? name : name.upcase)
               end
               out << cmd
             end
-            data[:order].each do |val|
+            data.order.each do |val|
               if val.is_a?(Array)
                 s = ' '
                 found = false
@@ -1362,7 +1332,7 @@ module Squared
                     meth
                   elsif respond_to?(meth)
                     found = true
-                    __send__(meth)
+                    __send__ meth
                   end
                 end
                 val = val.compact.join(s)
@@ -1372,9 +1342,9 @@ module Squared
               end
               out << val.to_s
             end
-            print_banner(*out, styles: data[:styles], border: data[:border], client: client)
+            print_banner(*out, styles: data.styles, border: data.border, client: client)
           elsif workspace.series.multiple?
-            "## #{__send__(data[:order].first || :path)} ##"
+            "## #{__send__(data.order.first || :path)} ##"
           end
         end
 
@@ -1398,7 +1368,7 @@ module Squared
               out << ''
             end
             if from
-              out << (from = from.to_s)
+              out << from
               pat = /\A(#{Regexp.escape(from)})(.*)\z/
             end
           else
@@ -1451,14 +1421,15 @@ module Squared
         end
 
         def append_any(val, target: @session, build: false, delim: false)
+          return unless val
+
           if delim && !target.include?('--')
             target << '--'
           else
             delim = false
           end
+          val = shell_split(val) if val.is_a?(String)
           case val
-          when String
-            target << val
           when Hash
             append_hash(val, target: target, build: build)
           when Enumerable
@@ -1489,7 +1460,7 @@ module Squared
             return target << (if flag
                                 shell_option(opt, equals ? val : nil, quote: quote, escape: escape, force: force)
                               else
-                                shell_quote(val)
+                                shell_quote val
                               end)
           end
           nil
@@ -1499,17 +1470,18 @@ module Squared
                           **kwargs)
           return if list.empty?
 
-          ret = []
-          list.flatten.each do |flag|
-            next unless (val = option(flag, target: target, **kwargs))
+          [].tap do |ret|
+            list.flatten.each do |flag|
+              next unless (val = option(flag, target: target, **kwargs))
 
-            if val == '0' && no
-              flag = "no-#{flag}"
-              val = nil
+              if val == '0' && no
+                flag = "no-#{flag}"
+                val = nil
+              end
+              ret << shell_option(flag, equals ? val : nil, escape: escape, quote: quote, force: force)
             end
-            ret << shell_option(flag, equals ? val : nil, escape: escape, quote: quote, force: force)
+            ret.each { |val| target << val } unless ret.empty?
           end
-          ret.each { |val| target << val } unless ret.empty?
         end
 
         def append_nocolor(target: @session)
@@ -1557,9 +1529,9 @@ module Squared
         end
 
         def collect_hash(data, pass: [])
-          ret = []
-          data.each { |key, val| ret.concat(val) unless pass.include?(key) }
-          ret
+          [].tap do |ret|
+            data.each { |key, val| ret.concat(val) unless pass.include?(key) }
+          end
         end
 
         def parse_json(val, kind: Hash, hint: nil)
@@ -1567,20 +1539,11 @@ module Squared
           raise_error("invalid JSON #{kind.name}", val, hint: hint) if kind && !ret.is_a?(kind)
         rescue StandardError => e
           log&.warn e
-          warn log_message(Logger::WARN, e, subject: name) if warning?
+          warn log_message(Logger::WARN, e, subject: name, pass: true) if warning?
         else
           ret
         end
 
-        def fetch_uri(*args, **kwargs, &blk)
-          require 'open-uri'
-          if RUBY_VERSION < '2.5'
-            open(*args, **kwargs, &blk)
-          else
-            URI.open(*args, **kwargs, &blk)
-          end
-        end
-
         def param_guard(action, flag, args:, key: nil, pat: nil, values: nil)
           if args && key
             val = args.fetch(key, nil)
@@ -1595,7 +1558,7 @@ module Squared
           args
         end
 
-        def confirm_outdated(pkg, ver, rev, lock: false)
+        def confirm_outdated(pkg, ver, rev, cur = nil, lock: false, col1: 0)
           a = sub_style(case rev
                         when 1
                           'MAJOR'
@@ -1603,11 +1566,11 @@ module Squared
                           'MINOR'
                         else
                           'PATCH'
-                        end, styles: theme[:header])
-          b = sub_style("#{pkg} #{ver}", styles: theme[:inline])
+                        end, styles: (rev == 1 && theme[:major]) || theme[:header])
+          b = sub_style(pkg.ljust(col1), styles: theme[:inline])
           c, d = rev == 1 || lock ? ['y/N', 'N'] : ['Y/n', 'Y']
-          e = lock ? " #{sub_style('(locked)', styles: color(:red))}" : ''
-          confirm "Upgrade to #{a}? #{b + e} [#{c}] ", d
+          e = lock ? sub_style((cur || 'locked').rjust(7), styles: color(:red)) : cur&.rjust(7)
+          confirm "#{a}: #{b}#{e} #{sub_style(ver.rjust(col1 > 0 ? 10 : 0), styles: theme[:inline])} [#{c}] ", d
         end
 
         def choice_index(msg, list, values: nil, accept: nil, series: false, trim: nil, column: nil,
@@ -1624,28 +1587,25 @@ module Squared
           end
           ret = multiple ? ret.map! { |val| val.sub(trim, '') } : ret.sub(trim, '') if trim
           if column
-            a, b = Array(column)
-            ret = Array(ret).map! { |val| val[a, b || 1] }
+            a, b = as_a column
+            ret = as_a(ret).map! { |val| val[a, b || 1] }
             ret = ret.first unless multiple
           end
           if accept
-            hint = Array(ret).map { |val| sub_style(val, styles: theme[:inline]) }.join(', ')
-            accept = Array(accept).map { |val| Array(val) }
+            a = as_a(ret).map { |val| sub_style(val, styles: theme[:inline]) }.join(', ')
+            accept = as_a(accept).map { |val| as_a(val) }
             if accept.any? { |val| val[1] == true }
               ret = [ret]
               multiple = -1
             end
             loop do
-              item = accept.first
-              d, e = item[2] ? ['Y', '[Y/n]'] : ['N', '[y/N]']
-              c = confirm("#{item[0]}#{a ? " [#{a}]" : ''} #{e} ", d, timeout: 60)
-              if item[1] == true
+              c = confirm("#{accept.first[0]}#{a ? " [#{a}]" : ''} [y/N] ", 'N', timeout: 60)
+              if accept.shift[1] == true
                 ret << c
               elsif !c
                 break
               end
-              hint = nil
-              accept.shift
+              a = nil
               break if accept.empty?
             end
             exit 1 unless accept.empty?
@@ -1676,7 +1636,7 @@ module Squared
           if (ret = instance_eval(&blk)).nil?
             val
           else
-            Array(ret)
+            ret.is_a?(Array) ? ret : [ret]
           end
         end
 
@@ -1725,29 +1685,23 @@ module Squared
         end
 
         def semscan(val, fill: true)
-          ret = val.scan(SEM_VER).first
-          fill ? semver(ret) : ret
+          val.scan(SEM_VER).first.yield_self { |data| fill ? semver(data) : data }
         end
 
         def indexitem(val)
-          [$1.to_i, $2 && $2[1..-1]] if val =~ /\A[=^#{indexchar}](\d+)(:.+)?\z/
+          [$1.to_i, $2 && $2[1..-1]] if val =~ /\A\^(\d+)(:.+)?\z/
         end
 
         def indexerror(val, list = nil)
           raise_error("requested index #{val}", hint: list && "of #{list.size}")
         end
 
-        def indexchar
-          workspace.windows? ? '=' : '^'
-        end
-
         def printsucc
           @@print_order += 1
         end
 
         def color(val)
-          ret = theme[val]
-          ret && !ret.empty? ? ret : [val]
+          theme[val].yield_self { |styles| styles && !styles.empty? ? styles : [val] }
         end
 
         def colormap(val)
@@ -1789,40 +1743,22 @@ module Squared
           end
         end
 
-        def pwd_set(done = nil, pass: false, from: nil, dryrun: false)
-          pwd = Pathname.pwd
-          if block_given?
-            begin
-              pass = semscan(pass).join <= RUBY_VERSION if pass.is_a?(String)
-              if (path == pwd || pass == true) && !workspace.jruby_win?
-                ret = yield
-              else
-                Dir.chdir(path)
-                ret = yield
-                Dir.chdir(pwd)
-              end
-            rescue StandardError => e
-              log&.error e
-              unless dryrun
-                ret = on(:error, from, e)
-                raise if exception && ret != true
-              end
-            else
-              ret
-            end
-          elsif @pwd == pwd
-            @pwd = nil
-            pwd unless done
-          elsif @pwd
-            return unless path == pwd
-
-            Dir.chdir(@pwd)
-            @pwd = nil
-          elsif !done && path != pwd
-            @pwd = pwd
+        def pwd_set(pass: false, from: nil)
+          pwd = Dir.pwd
+          if path.to_s == pwd || pass == true || (pass.is_a?(String) && semscan(pass).join <= RUBY_VERSION)
+            ret = yield
+          else
             Dir.chdir(path)
-            @pwd
+            ret = yield
+            Dir.chdir(pwd)
           end
+        rescue StandardError => e
+          puts e
+          log&.error e
+          ret = on(:error, from, e)
+          raise if exception && ret != true
+        else
+          ret
         end
 
         def run_set(cmd, val = nil, opts: nil, **)
@@ -1916,7 +1852,7 @@ module Squared
 
         def graph_set(val)
           @graph = if val
-                     Array(val).map { |s| workspace.prefix ? workspace.task_name(s).to_sym : s.to_sym }.freeze
+                     as_a(val).map { |s| workspace.prefix ? workspace.task_name(s).to_sym : s.to_sym }.freeze
                    end
         end
 
@@ -1969,7 +1905,7 @@ module Squared
               unless @pass.include?(key.to_s) || ws.task_defined?(name, action) || ws.task_exclude?(action, self)
                 ws.task_desc(@desc, action)
                 task action do
-                  __send__(key)
+                  __send__ key
                 end
               end
               next if (items = @children.select { |item| item.task_include?(key) }).empty?
@@ -1981,8 +1917,9 @@ module Squared
         end
 
         def projectpath?(val)
-          val = Pathname.new(val).cleanpath
-          val.absolute? ? val.to_s.start_with?(File.join(path, '')) : !val.to_s.start_with?(File.join('..', ''))
+          Pathname.new(val).cleanpath.yield_self do |file|
+            file.absolute? ? file.to_s.start_with?(File.join(path, '')) : !file.to_s.start_with?(File.join('..', ''))
+          end
         end
 
         def checkdir?(val)
@@ -2034,8 +1971,7 @@ module Squared
           return false if workspace.series.chain?(val = task_join(name, action))
           return true if task_invoked?(val) && (!task_invoked?(ac) || !workspace.task_defined?(ac, 'sync'))
 
-          val = workspace.series.name_get(action)
-          val != action && invoked_sync?(val)
+          workspace.series.name_get(action).yield_self { |name| name != action && invoked_sync?(name) }
         end
 
         def success?(ret)
@@ -2046,10 +1982,6 @@ module Squared
           ARG[:BANNER] && !env('BANNER', equals: '0')
         end
 
-        def pwd?
-          path == Pathname.pwd
-        end
-
         def stdin?
           pipe == 0
         end
@@ -2082,7 +2014,7 @@ module Squared
         end
 
         def borderstyle
-          ((data = workspace.banner_get(*@ref, group: group)) && data[:border]) || theme[:border]
+          workspace.banner_get(*@ref, group: group)&.border || theme[:border]
         end
 
         def headerstyle
@@ -2095,7 +2027,7 @@ module Squared
       end
 
       Application.implement(Base, base: true)
-      Application.attr_banner = Common::SymSet.new(%i[name project path ref group parent])
+      Application.attr_banner = Set.new(%i[name project path ref group parent])
     end
   end
 end