process_executer 3.2.0 → 3.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3af694308b0e9c5119b2ff63d11b6b351a25cc9f7d66ac2dd482c8525736595a
-  data.tar.gz: a669d2c3cceadeb7544be9765793eff22addc8c7bde10e13e4d36de447b26cb5
+  metadata.gz: 2000a0333ebf2ac629edac374ca29355a557c05e512783570b650738c439719c
+  data.tar.gz: 0347f4f0a06370cfbfd4e4240947cf940879acecc6d072fad77711bb04598125
 SHA512:
-  metadata.gz: 670dcd425f0879def69e0875a90edae0f79280b44d4ea73d117b3875ad3eeb1cd6329f4b2d8a6a8c0d601f7b79eb72989288255d57067714f2e830f4dde3ddb2
-  data.tar.gz: 6a18db9777ecb26ebaa7b30c3fbadb81ae8adba1dcd60c92bf6d30510f30275336ef34c4b1c05c5df6356bcd62cd5b783736adb25462dd55d9d87deec6015945
+  metadata.gz: 4553524b57e82b0a601f94d84e2f4dcda5e0ffc761774d95a083673fcd0eb56fd8608a4124d1fdf84a7d815220f57f0c8753f60f85d3754ae2bfb486b0666d36
+  data.tar.gz: bd5516506787274bf2a9baf3b20b2738bde3b71c4874f024e457e08dfad49070204bec7c8ee4aec01113db113397a9c17a15f86184dc8e7f03f80ae8da68e8e1

data/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "3.2.3"
+}

data/CHANGELOG.md

@@ -5,6 +5,33 @@ All notable changes to the process_executer gem will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [3.2.3](https://github.com/main-branch/process_executer/compare/v3.2.2...v3.2.3) (2025-04-17)
+
+
+### Bug Fixes
+
+* Do not trigger build workflows after merging to main or for release PRs ([0b2701c](https://github.com/main-branch/process_executer/commit/0b2701c4eef8ecc929f9f987433599145b224256))
+
+## [3.2.2](https://github.com/main-branch/process_executer/compare/v3.2.1...v3.2.2) (2025-04-16)
+
+
+### Bug Fixes
+
+* Automate commit-to-publish workflow ([c51e8d2](https://github.com/main-branch/process_executer/commit/c51e8d2dfcff21ccd634fe58d5eb3b733869877d))
+
+## v3.2.1 (2025-04-08)
+
+[Full Changelog](https://github.com/main-branch/process_executer/compare/v3.2.0..v3.2.1)
+
+Changes since v3.2.0:
+
+* d1e19a5 test: assert that MonitoredPipe has no open instances after each test
+* aa71f8e test: ensure MonitoredPipe cleans up open instances in specs
+* 987b0c9 fix: ensure that all pipes are closed even when there is an IOError
+* 65e8db0 fix: ensure that MonitoredPipe cleans up after itself even when there is IOError
+* ed2454e chore: integrate track_open_instances gem to report on leaked MonitoredPipe instances
+* f25c87d chore: release v3.2.0
+
 ## v3.2.0 (2025-04-04)
 
 [Full Changelog](https://github.com/main-branch/process_executer/compare/v3.1.0..v3.2.0)

data/README.md

@@ -4,8 +4,6 @@
 [![Documentation](https://img.shields.io/badge/Documentation-Latest-green)](https://rubydoc.info/gems/process_executer/)
 [![Change Log](https://img.shields.io/badge/CHANGELOG-Latest-green)](https://rubydoc.info/gems/process_executer/file/CHANGELOG.md)
 [![Build Status](https://github.com/main-branch/process_executer/actions/workflows/continuous-integration.yml/badge.svg)](https://github.com/main-branch/process_executer/actions/workflows/continuous-integration.yml)
-[![Maintainability](https://api.codeclimate.com/v1/badges/0b5c67e5c2a773009cd0/maintainability)](https://codeclimate.com/github/main-branch/process_executer/maintainability)
-[![Test Coverage](https://api.codeclimate.com/v1/badges/0b5c67e5c2a773009cd0/test_coverage)](https://codeclimate.com/github/main-branch/process_executer/test_coverage)
 [![Conventional
 Commits](https://img.shields.io/badge/Conventional%20Commits-1.0.0-%23FE5196?logo=conventionalcommits&logoColor=white)](https://conventionalcommits.org)
 [![Slack](https://img.shields.io/badge/slack-main--branch/process__executer-yellow.svg?logo=slack)](https://main-branch.slack.com/archives/C07NG2BPG8Y)

data/Rakefile

@@ -28,6 +28,13 @@ rescue Bundler::BundlerError => e
   exit e.status_code
 end
 
+# Make it so that calling `rake release` just calls `rake release:rubygems_push` to
+# avoid creating and pushing a new tag.
+
+Rake::Task['release'].clear
+desc 'Customized release task to avoid creating a new tag'
+task release: 'release:rubygem_push'
+
 CLEAN << 'pkg'
 CLOBBER << 'Gemfile.lock'
 

data/lib/process_executer/monitored_pipe.rb

@@ -2,6 +2,7 @@
 
 require 'stringio'
 require 'io/wait'
+require 'track_open_instances'
 
 module ProcessExecuter
   # Write data sent through a pipe to a destination
@@ -41,6 +42,8 @@ module ProcessExecuter
   # @api public
   #
   class MonitoredPipe
+    include TrackOpenInstances
+
     # Create a new monitored pipe
     #
     # Creates a IO.pipe and starts a monitoring thread to read data written to the pipe.
@@ -64,6 +67,8 @@ module ProcessExecuter
       @pipe_reader, @pipe_writer = IO.pipe
       @state = :open
       @thread = start_monitoring_thread
+
+      self.class.add_open_instance(self)
     end
 
     # Set the state to `:closing` and wait for the state to be set to `:closed`
@@ -83,18 +88,15 @@ module ProcessExecuter
     #
     def close
       mutex.synchronize do
-        return unless state == :open
-
-        @state = :closing
-      end
-
-      mutex.synchronize do
-        condition_variable.wait(mutex) while @state != :closed
+        if state == :open
+          @state = :closing
+          condition_variable.wait(mutex) while @state != :closed
+        end
       end
 
       thread.join
-
       destination.close
+      self.class.remove_open_instance(self)
     end
 
     # Return the write end of the pipe so that data can be written to it

data/lib/process_executer/runner.rb

@@ -52,7 +52,8 @@ module ProcessExecuter
       opened_pipes = wrap_stdout_stderr(options)
       ProcessExecuter.spawn_and_wait_with_options(command, options)
     ensure
-      opened_pipes.each { |key, value| close_pipe(command, key, value) }
+      opened_pipes.each_value(&:close)
+      opened_pipes.each { |option_key, pipe| raise_pipe_error(command, option_key, pipe) }
     end
 
     # Wrap the stdout and stderr redirection options with a MonitoredPipe
@@ -81,16 +82,6 @@ module ProcessExecuter
         ProcessExecuter::Destinations.compatible_with_monitored_pipe?(value)
     end
 
-    # Close the pipe and raise an error if the pipe raised an exception
-    # @return [void]
-    # @raise [ProcessExecuter::ProcessIOError] If an exception was raised while
-    #   collecting subprocess output
-    # @api private
-    def close_pipe(command, option_key, pipe)
-      pipe.close
-      raise_pipe_error(command, option_key, pipe) if pipe.exception
-    end
-
     # Process the result of the command and return a ProcessExecuter::Result
     #
     # Log the command and result, and raise an error if the command failed.
@@ -144,6 +135,8 @@ module ProcessExecuter
     # @api private
     #
     def raise_pipe_error(command, option_key, pipe)
+      return unless pipe.exception
+
       error = ProcessExecuter::ProcessIOError.new("Pipe Exception for #{command}: #{option_key.inspect}")
       raise(error, cause: pipe.exception)
     end

data/lib/process_executer/version.rb

@@ -2,5 +2,5 @@
 
 module ProcessExecuter
   # The current Gem version
-  VERSION = '3.2.0'
+  VERSION = '3.2.3'
 end

data/process_executer.gemspec

@@ -37,6 +37,8 @@ Gem::Specification.new do |spec|
     'Ruby: MRI 3.1 or later, TruffleRuby 24 or later, or JRuby 9.4 or later'
   ]
 
+  spec.add_dependency 'track_open_instances', '~> 0.1'
+
   spec.add_development_dependency 'bundler-audit', '~> 0.9'
   spec.add_development_dependency 'create_github_release', '~> 2.1'
   spec.add_development_dependency 'main_branch_shared_rubocop_config', '~> 0.1'

data/release-please-config.json

@@ -0,0 +1,22 @@
+{
+  "bootstrap-sha": "d12757fe8f44f5cf2bb9092c686d642fe41f436f",
+  "packages": {
+    ".": {
+      "release-type": "ruby",
+      "package-name": "process_executer",
+      "changelog-path": "CHANGELOG.md",
+      "version-file": "lib/process_executer/version.rb",
+      "bump-minor-pre-major": true,
+      "bump-patch-for-minor-pre-major": true,
+      "draft": false,
+      "prerelease": false,
+      "include-component-in-tag": false
+    }
+  },
+  "plugins": [
+    {
+      "type": "sentence-case"
+    }
+  ],
+  "$schema": "https://raw.githubusercontent.com/googleapis/release-please/main/schemas/config.json"
+}

metadata

@@ -1,14 +1,28 @@
 --- !ruby/object:Gem::Specification
 name: process_executer
 version: !ruby/object:Gem::Version
-  version: 3.2.0
+  version: 3.2.3
 platform: ruby
 authors:
 - James Couball
 bindir: exe
 cert_chain: []
-date: 2025-04-05 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: track_open_instances
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
 - !ruby/object:Gem::Dependency
   name: bundler-audit
   requirement: !ruby/object:Gem::Requirement
@@ -207,6 +221,7 @@ files:
 - ".commitlintrc.yml"
 - ".husky/commit-msg"
 - ".markdownlint.yml"
+- ".release-please-manifest.json"
 - ".rspec"
 - ".rubocop.yml"
 - ".yardopts"
@@ -243,6 +258,7 @@ files:
 - lib/process_executer/version.rb
 - package.json
 - process_executer.gemspec
+- release-please-config.json
 homepage: https://github.com/main-branch/process_executer
 licenses:
 - MIT
@@ -250,8 +266,8 @@ metadata:
   allowed_push_host: https://rubygems.org
   homepage_uri: https://github.com/main-branch/process_executer
   source_code_uri: https://github.com/main-branch/process_executer
-  documentation_uri: https://rubydoc.info/gems/process_executer/3.2.0
-  changelog_uri: https://rubydoc.info/gems/process_executer/3.2.0/file/CHANGELOG.md
+  documentation_uri: https://rubydoc.info/gems/process_executer/3.2.3
+  changelog_uri: https://rubydoc.info/gems/process_executer/3.2.3/file/CHANGELOG.md
   rubygems_mfa_required: 'true'
 rdoc_options: []
 require_paths:
@@ -269,7 +285,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements:
 - 'Platform: Mac, Linux, or Windows'
 - 'Ruby: MRI 3.1 or later, TruffleRuby 24 or later, or JRuby 9.4 or later'
-rubygems_version: 3.6.2
+rubygems_version: 3.6.7
 specification_version: 4
 summary: An API for executing commands in a subprocess
 test_files: []