ukiryu 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 576af6468ec00f6a995cf66ae57339f5edde1acd80cad8bc0ad9ba031b40c886
-  data.tar.gz: '039581b42d07ef648270d326fbd03e086b1b8cce3e91eacc1e5da024c6fc4776'
+  metadata.gz: b172d6156b86340038461644b956c478709d55f34ee393d6e8768aeb424266f7
+  data.tar.gz: 85b530c9f1b1aadc4b43bddc0d6ada89f6ccb3ba225d92dbb703f070cb3b7275
 SHA512:
-  metadata.gz: aa59d7622011cc4bd44d46a083b41a8454e3d804e83898a6edc73dad7b06ab0c0541e81422637e62761fb245e0eb584cc5bc0baea99992ea566ef634b996dfa8
-  data.tar.gz: 9c7667660e2c143abcd848aa26eaf2592ce650185d50761754bec29d092aee91342b8715ea48921ff32c26c32e1b74448a5f332797a62a3844b42381f411da04
+  metadata.gz: 7ac908e5fe60994361b0a7d2a8fc975d52f3969d0a516696749fe8de7b870147a7f5410344e4bcb92c2b5008cfdcbffca9894bfa1199c486f405ff05e8d1e6d1
+  data.tar.gz: 75c41074359f2ef89d505c77370f2e97826d0fa57afac1f59f5e0867b96788dde9dfb58db19d050b5d4677615ea084b18137a6e7ba72cd11fbdbd385a6564b5c

data/lib/ukiryu/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Ukiryu
-  VERSION = '0.3.0'
+  VERSION = '0.3.1'
 end

data/spec/support/tool_helper.rb

@@ -27,7 +27,56 @@ module ToolHelper
     tool.version if tool.available?
   end
 
+  # True when the local ImageMagick can ENCODE PNG. Some runner builds
+  # (tool-setup/static installs) ship decode-only delegate sets: identify
+  # works but every PNG write fails with "no encode delegate for this
+  # image format `''". The fixture-copy specs decode fine; the
+  # resize/mogrify specs need the encoder.
+  def png_encode_available?
+    magick = %w[magick convert].find do |m|
+      system(m, "-version", out: File::NULL, err: File::NULL)
+    end
+    return false unless magick
+
+    require "tmpdir"
+    probe = File.join(Dir.tmpdir, "ukiryu_png_probe_#{Process.pid}.png")
+    File.delete(probe) if File.exist?(probe)
+    begin
+      ok = system(magick, "-size", "4x4", "xc:white", "png:#{probe}",
+                  out: File::NULL, err: File::NULL)
+      ok && File.exist?(probe) && !File.zero?(probe)
+    ensure
+      File.delete(probe) if File.exist?(probe)
+    end
+  end
+
+  # True when mogrify can rewrite an image IN PLACE. ImageMagick
+  # 7.1.2-33 on macOS 26 regressed here: the same build encodes PNG
+  # (the probe above passes) yet in-place mogrify dies with
+  # "no encode delegate for this image format `'". Version-dependent,
+  # so the probe — not a version list — is the gate.
+  def mogrify_in_place_available?
+    magick = %w[magick convert].find do |m|
+      system(m, "-version", out: File::NULL, err: File::NULL)
+    end
+    return false unless magick
+
+    require "tmpdir"
+    require "fileutils"
+    dir = Dir.mktmpdir("ukiryu_mogrify_probe")
+    src = File.join(dir, "src.png")
+    ok = system(magick, "-size", "4x4", "xc:white", "png:#{src}",
+                out: File::NULL, err: File::NULL)
+    return false unless ok && File.exist?(src) && !File.zero?(src)
+
+    system(magick, "mogrify", "-resize", "2x2!", src,
+           out: File::NULL, err: File::NULL)
+  ensure
+    FileUtils.rm_rf(dir) if dir
+  end
+
   # Create a temporary test image by copying a fixture image
+
   # This avoids using ImageMagick's built-in formats which aren't available on all builds
   # @param path [String] the path to create the image at
   # @param size [String] the image size (e.g., "100x100", "200x150")

data/spec/ukiryu/cli_stdin_spec.rb

@@ -39,20 +39,23 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
                                  Open3.capture3(env, 'bash', '-c', command)
                                end
     end
-    [stdout + stderr, status]
+    # Streams stay separate: engine-detection warnings legitimately land
+    # on stderr (e.g. leptris's "native acceleration unavailable" at
+    # require), and JSON assertions must parse the stream they mean.
+    [stdout, stderr, status]
   end
 
   def run_ukiryu_exec(args, stdin_data = nil)
     cmd = "bundle exec ukiryu #{args}"
-    output, status = run_cli_command(cmd, stdin_data)
-    [output, status]
+    stdout, stderr, status = run_cli_command(cmd, stdin_data)
+    [stdout, stderr, status]
   end
 
   describe '--stdin flag' do
     it 'reads from stdin pipe' do
       skip 'Unix pipe tests require Unix shell' if platform == :windows
 
-      output, status = run_ukiryu_exec('exec jq process --stdin filter=".test" --format=json', '{"test": "value"}')
+      output, stderr, status = run_ukiryu_exec('exec jq process --stdin filter=".test" --format=json', '{"test": "value"}')
 
       # Parse JSON and extract stdout
       parsed = JSON.parse(output)
@@ -64,7 +67,7 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
       skip 'Unix pipe tests require Unix shell' if platform == :windows
 
       json_data = '{"lines": ["line1", "line2", "line3"]}'
-      output, status = run_ukiryu_exec('exec jq process --stdin filter=".lines" --format=json', json_data)
+      output, stderr, status = run_ukiryu_exec('exec jq process --stdin filter=".lines" --format=json', json_data)
 
       parsed = JSON.parse(output)
       stdout = parsed['output']['stdout']
@@ -80,7 +83,7 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
     it 'reads from stdin pipe when stdin=- is specified' do
       skip 'Unix pipe tests require Unix shell' if platform == :windows
 
-      output, status = run_ukiryu_exec('exec jq process stdin=- filter=".foo" --format=json', '{"foo": "bar"}')
+      output, stderr, status = run_ukiryu_exec('exec jq process stdin=- filter=".foo" --format=json', '{"foo": "bar"}')
 
       parsed = JSON.parse(output)
       expect(parsed['output']['stdout'].strip).to eq('"bar"')
@@ -92,8 +95,8 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
 
       stdin_data = '{"test": "data"}'
 
-      output1, status1 = run_ukiryu_exec('exec jq process --stdin filter=".test" --format=json', stdin_data)
-      output2, status2 = run_ukiryu_exec('exec jq process stdin=- filter=".test" --format=json', stdin_data)
+      output1, stderr1, status1 = run_ukiryu_exec('exec jq process --stdin filter=".test" --format=json', stdin_data)
+      output2, stderr2, status2 = run_ukiryu_exec('exec jq process stdin=- filter=".test" --format=json', stdin_data)
 
       parsed1 = JSON.parse(output1)
       parsed2 = JSON.parse(output2)
@@ -111,11 +114,11 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
         file.close
 
         # On Windows, we need to redirect stderr separately to avoid mixing with JSON output
-        output, status = if platform == :windows
-                           run_cli_command("bundle exec ukiryu exec jq process stdin=@#{file.path} filter=\".file\" --format=json 2>$null")
-                         else
-                           run_cli_command("bundle exec ukiryu exec jq process stdin=@#{file.path} filter=\".file\" --format=json")
-                         end
+        output, stderr, status = if platform == :windows
+                                   run_cli_command("bundle exec ukiryu exec jq process stdin=@#{file.path} filter=\".file\" --format=json 2>$null")
+                                 else
+                                   run_cli_command("bundle exec ukiryu exec jq process stdin=@#{file.path} filter=\".file\" --format=json")
+                                 end
 
         parsed = JSON.parse(output)
         expect(parsed['output']['stdout'].strip).to eq('"content"')
@@ -124,9 +127,9 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
     end
 
     it 'shows error when file does not exist' do
-      output, status = run_cli_command('bundle exec ukiryu exec jq process stdin=@/nonexistent/file.json filter="." 2>&1')
+      output, stderr, status = run_cli_command('bundle exec ukiryu exec jq process stdin=@/nonexistent/file.json filter="." 2>&1')
 
-      expect(output).to include('File not found')
+      expect(output + stderr).to include('File not found')
       expect(status.success?).to be false
     end
   end
@@ -140,7 +143,7 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
         file.write('{"test": "value"}')
         file.flush # Ensure data is written to disk
 
-        output, status = run_ukiryu_exec("exec jq process stdin=@#{file.path} filter=.test --format=json")
+        output, stderr, status = run_ukiryu_exec("exec jq process stdin=@#{file.path} filter=.test --format=json")
 
         parsed = JSON.parse(output)
         expect(parsed).not_to be_nil
@@ -156,10 +159,10 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
 
       # Use NO_COLOR to avoid ANSI escape codes in output
       # Use -D for dry-run (uppercase D, not lowercase d which is now --definition)
-      output, status = run_ukiryu_exec('exec jq process --stdin -D filter="."', '{"test": "value"}')
+      output, stderr, status = run_ukiryu_exec('exec jq process --stdin -D filter="."', '{"test": "value"}')
 
-      expect(output).to include('DRY RUN')
-      expect(output).to include('stdin:')
+      expect(output + stderr).to include('DRY RUN')
+      expect(output + stderr).to include('stdin:')
       # Output has escaped backslashes in the inspection format
       expect(output).to include('test')
       expect(output).to include('value')
@@ -171,7 +174,7 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
 
       long_data = 'x' * 200
       # Use -D for dry-run (uppercase D, not lowercase d which is now --definition)
-      output, status = run_ukiryu_exec('exec jq process --stdin -D filter="."', long_data)
+      output, stderr, status = run_ukiryu_exec('exec jq process --stdin -D filter="."', long_data)
 
       expect(output).to include('...')
       # Preview should be truncated (check in the output, not just the matched string)
@@ -185,7 +188,7 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
     it 'outputs JSON format with stdin' do
       skip 'Unix pipe tests require Unix shell' if platform == :windows
 
-      output, status = run_ukiryu_exec('exec jq process --stdin filter=".test" --format=json', '{"test": "value"}')
+      output, stderr, status = run_ukiryu_exec('exec jq process --stdin filter=".test" --format=json', '{"test": "value"}')
 
       parsed = JSON.parse(output)
       expect(parsed['output']['stdout']).to include('"value"')
@@ -195,7 +198,7 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
     it 'outputs YAML format with stdin' do
       skip 'Unix pipe tests require Unix shell' if platform == :windows
 
-      output, status = run_ukiryu_exec('exec jq process --stdin filter=".test" --format=yaml', '{"test": "value"}')
+      output, stderr, status = run_ukiryu_exec('exec jq process --stdin filter=".test" --format=yaml', '{"test": "value"}')
 
       expect(output).to include('stdout:')
       expect(output).to include('value')
@@ -213,7 +216,7 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
         file.close
 
         # Use cat instead of echo to avoid escaping issues
-        result, status = run_cli_command("cat #{file.path} | bundle exec ukiryu exec jq process --stdin filter='.items' --format=json | head -10")
+        result, stderr, status = run_cli_command("cat #{file.path} | bundle exec ukiryu exec jq process --stdin filter='.items' --format=json | head -10")
 
         # The output should be valid JSON, and head should truncate it
         expect(result).not_to be_empty
@@ -227,7 +230,7 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
 
       # Simulate curl output using echo instead of actual network call
       json_data = '{"slideshow": {"title": "Test Presentation"}}'
-      result, status = run_cli_command("echo '#{json_data}' | bundle exec ukiryu exec jq process --stdin filter='.slideshow.title' --format=json")
+      result, stderr, status = run_cli_command("echo '#{json_data}' | bundle exec ukiryu exec jq process --stdin filter='.slideshow.title' --format=json")
 
       expect(result).not_to be_empty
       expect(result).to include('Test Presentation')
@@ -239,7 +242,7 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
     it 'outputs only the command stdout without wrapping' do
       skip 'Unix pipe tests require Unix shell' if platform == :windows
 
-      output, status = run_ukiryu_exec('exec jq process --stdin --raw filter=".test"', '{"test": "value"}')
+      output, stderr, status = run_ukiryu_exec('exec jq process --stdin --raw filter=".test"', '{"test": "value"}')
 
       # Should output just "value" without YAML/JSON wrapping
       expect(output.strip).to eq('"value"')
@@ -252,12 +255,12 @@ RSpec.describe 'Ukiryu::CliCommands::RunCommand stdin handling' do
       skip 'Unix pipe tests require Unix shell' if platform == :windows
 
       # Test with a command that produces stderr - use syntax error in jq
-      output, = run_ukiryu_exec('exec jq process --stdin --raw filter=".foo[bar"', '{}')
+      output, stderr, status = run_ukiryu_exec('exec jq process --stdin --raw filter=".foo[bar"', '{}')
 
-      # jq should produce an error in the output
-      expect(output).not_to be_empty
-      expect(output).to include('error')
-      expect(output).to include('syntax error')
+      # jq's error passes through to the CLI's stderr
+      expect(stderr).not_to be_empty
+      expect(output + stderr).to include('error')
+      expect(output + stderr).to include('syntax error')
     end
   end
 end

data/spec/ukiryu/tools/imagemagick_spec.rb

@@ -43,7 +43,10 @@ RSpec.describe 'ImageMagick Tool Profile' do
   end
 
   describe 'convert command' do
-    before(:each) { skip_unless_tool_available(:imagemagick) }
+    before(:each) do
+      skip_unless_tool_available(:imagemagick)
+      skip 'ImageMagick lacks a PNG encode delegate on this runner' unless png_encode_available?
+    end
 
     it 'converts between formats' do
       input = File.join(@temp_dir, 'input.png')
@@ -140,7 +143,10 @@ RSpec.describe 'ImageMagick Tool Profile' do
   end
 
   describe 'identify command' do
-    before(:each) { skip_unless_tool_available(:imagemagick) }
+    before(:each) do
+      skip_unless_tool_available(:imagemagick)
+      skip 'ImageMagick lacks a PNG encode delegate on this runner' unless png_encode_available?
+    end
 
     it 'identifies image format and characteristics' do
       input = File.join(@temp_dir, 'test.png')
@@ -168,7 +174,11 @@ RSpec.describe 'ImageMagick Tool Profile' do
   end
 
   describe 'mogrify command' do
-    before(:each) { skip_unless_tool_available(:imagemagick) }
+    before(:each) do
+      skip_unless_tool_available(:imagemagick)
+      skip 'ImageMagick lacks a PNG encode delegate on this runner' unless png_encode_available?
+      skip 'ImageMagick regressed in-place mogrify on this runner' unless mogrify_in_place_available?
+    end
 
     it 'modifies images in place' do
       input = File.join(@temp_dir, 'test.png')

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ukiryu
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Ribose Inc.
@@ -29,28 +29,28 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 6.2.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 6.2.1
 - !ruby/object:Gem::Dependency
   name: lutaml-model
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.0
+        version: 0.8.114
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.0
+        version: 0.8.114
 - !ruby/object:Gem::Dependency
   name: lutaml-xsd
   requirement: !ruby/object:Gem::Requirement
@@ -426,7 +426,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.11
+rubygems_version: 4.0.23
 specification_version: 4
 summary: The "OpenAPI" for Command Line Interfaces
 test_files: