wslc-wip 0.17.4 → 0.17.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0c7d06fb49766e980be92c2d70d3a0c86dc68b16c0c2b5d84e05d5baa1dd40f1
-  data.tar.gz: 61ad143b173b17849427d03efdaa451849b0e58d16a3eb97c55855d641ced273
+  metadata.gz: ae440c584eee01b5fca703f10fe051f457cd24534a54b8d2c76fe485d33fc256
+  data.tar.gz: '0078b9598b5084d7b62ff7cf3aa85589c61739801bdae6d8bfa76e3218459fde'
 SHA512:
-  metadata.gz: c1accaef0df664f9d35659d08cb8782c1e54ea9a56cae60e1bd53ddd347c2736501e1fc785be9667541c225f88340e448fd8b2fab32a4f41171f613863533ea4
-  data.tar.gz: 235f4c9ad2997138d262eaac4098edfe0582db8bf56dc8743250a17d10fac14df96fdea3ab575faef91f096e4f091ad2a1ffb06931da59aa5e683eaf40327c03
+  metadata.gz: 8d92d054446d0b517db26312afa196e0f4486181ea3c6254d7eecb11a4bd239c5b0a48ff28804d3c2806d1acfc0f9beeaebcce5dcacdd73855b3ed629386774d
+  data.tar.gz: 0b19afc749ee393752613e27306b56559e8ff5c31d29f0248ddc2bee05e0ba4b327725cef1a371b9f24f4fc9fef85c5972e1a005e041003100bd2e44a2c9f664

data/lib/wip/build_context.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'find'
 require 'fileutils'
 require 'pathname'
 require 'tmpdir'
@@ -41,13 +42,31 @@ module Wip
       end
     end
 
+    # Walks the tree by hand (rather than Dir.glob-ing it all up front) so an
+    # ignored directory — node_modules, vendor/bundle, a multi-gigabyte
+    # storage/ — is never descended into just to be thrown away afterward.
     def included_files
-      Dir.glob('**/*', File::FNM_DOTMATCH, base: @root.to_s).select do |entry|
-        next false if %w[. ..].include?(entry)
+      result = []
+      Find.find(@root.to_s) do |path|
+        next if path == @root.to_s
 
-        path = @root.join(entry)
-        (path.file? || path.symlink?) && !@ignore.ignored?(entry)
+        entry = Pathname(path).relative_path_from(@root).to_s
+        result << entry if visit?(path, entry)
       end
+      result
+    end
+
+    # Returns whether `path` belongs in the staged context, pruning the walk
+    # when it's an ignored directory so nothing under it is even visited —
+    # unless a later negated rule could still re-include something beneath it.
+    def visit?(path, entry)
+      ignored = @ignore.ignored?(entry)
+      if !File.symlink?(path) && File.directory?(path)
+        Find.prune if ignored && @ignore.prunable?(entry)
+        return false
+      end
+
+      !ignored && (File.file?(path) || File.symlink?(path))
     end
   end
 end

data/lib/wip/cli.rb

@@ -69,12 +69,13 @@ module Wip
     def build(*extra)
       extra.shift if extra.first == '--'
       settings = load_config.command('build') || {}
-      context = settings['context'] || '.'
+      context = Pathname(load_config.path).dirname.join(settings['context'] || '.').to_s
       warn "wip: staging build context (#{context})"
       progress = StagingProgress.new
       BuildContext.new(context).stage(on_progress: progress.method(:tick)) do |staged_context|
         progress.finish
-        execute(builder.build(settings: settings.merge('context' => staged_context), extra: extra))
+        built = builder.build(settings: settings.merge('context' => staged_context), extra: extra)
+        execute(built, interactive: tty?(true))
       end
     ensure
       progress&.finish
@@ -301,11 +302,23 @@ module Wip
     # `wip up` invocation, mirroring ensure_sync_image's "build once, not every tick"
     # approach. A no-op for mode: container/compose and for build:-less services.
     def ensure_compose_images
-      load_config.compose_build_specs.each_value do |spec|
-        extra = spec['dockerfile'] ? ['-f', spec['dockerfile']] : []
-        spec['args']&.each { |key, value| extra.push('--build-arg', "#{key}=#{value}") }
-        execute(builder.build(settings: { 'context' => spec['context'], 'tag' => spec['tag'] }, extra: extra))
+      load_config.compose_build_specs.each_value { |spec| build_compose_image(spec) }
+    end
+
+    # Stages spec['context'] the same way `wip build` does, so a .dockerignore next to
+    # a compose-native service's build: is honored and progress is reported here too.
+    def build_compose_image(spec)
+      extra = spec['dockerfile'] ? ['-f', spec['dockerfile']] : []
+      spec['args']&.each { |key, value| extra.push('--build-arg', "#{key}=#{value}") }
+      warn "wip: staging build context (#{spec['context']})"
+      progress = StagingProgress.new
+      BuildContext.new(spec['context']).stage(on_progress: progress.method(:tick)) do |staged_context|
+        progress.finish
+        settings = { 'context' => staged_context, 'tag' => spec['tag'] }
+        execute(builder.build(settings: settings, extra: extra), interactive: tty?(true))
       end
+    ensure
+      progress&.finish
     end
 
     # Builds sync.build's image once per invocation (not per --watch tick, which

data/lib/wip/command_runner.rb

@@ -27,7 +27,7 @@ module Wip
         status = wait.value
       end
       report_hint(captured) unless status.success?
-      status.exitstatus
+      exitstatus(status)
     rescue Errno::ENOENT => e
       @stderr.puts e.message
       127
@@ -38,6 +38,13 @@ module Wip
 
     private
 
+    # exitstatus is nil when the process was killed by a signal instead of
+    # exiting normally; fall back to the conventional 128+signal shell code
+    # so callers always get a comparable integer.
+    def exitstatus(status)
+      status.exitstatus || (128 + status.termsig)
+    end
+
     def report_hint(captured)
       hint = @interpreter.interpret(captured.force_encoding(Encoding::UTF_8).scrub)
       @stderr.puts("\n#{hint}") if hint
@@ -50,7 +57,7 @@ module Wip
     def run_attached(command, env)
       pid = Process.spawn(env, *command)
       _, status = Process.wait2(pid)
-      status.exitstatus
+      exitstatus(status)
     rescue Errno::ENOENT => e
       @stderr.puts e.message
       127

data/lib/wip/docker_ignore.rb

@@ -30,8 +30,26 @@ module Wip
       end
     end
 
+    # Whether a walker can stop descending into an ignored directory without
+    # risking a miss: false if some later negated rule (e.g. `!node_modules/pkg`
+    # after `node_modules`) could still match something beneath it.
+    def prunable?(relative_path)
+      dir_components = relative_path.split('/')
+      @rules.none? { |rule| rule.negate && targets_descendant?(rule.pattern, dir_components) }
+    end
+
     private
 
+    def targets_descendant?(pattern, dir_components)
+      pattern_components = pattern.split('/')
+      return true if pattern_components.first == '**'
+      return false if pattern_components.size <= dir_components.size
+
+      pattern_components.first(dir_components.size).each_with_index.all? do |component, i|
+        File.fnmatch(component, dir_components[i], FNMATCH_FLAGS)
+      end
+    end
+
     def parse(line)
       line = line.strip
       return nil if line.empty? || line.start_with?('#')

data/lib/wip/doctor.rb

@@ -151,6 +151,8 @@ module Wip
                  condition == :ok ? ok_message : fail_message)
     end
 
-    def command_available?(name) = ENV.fetch('PATH', '').split(File::PATH_SEPARATOR).any? { |dir| File.executable?(File.join(dir, name)) }
+    def command_available?(name)
+      ENV.fetch('PATH', '').split(File::PATH_SEPARATOR).any? { |dir| File.executable?(File.join(dir, name)) }
+    end
   end
 end

data/lib/wip/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Wip
-  VERSION = '0.17.4'
+  VERSION = '0.17.5'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: wslc-wip
 version: !ruby/object:Gem::Version
-  version: 0.17.4
+  version: 0.17.5
 platform: ruby
 authors:
 - Wip contributors