feat(symfony-process): port full Process class from PHP

Faithfully port every method, field and constant of Symfony's
Process.php into process.rs, replacing the reduced stub. Add the
supporting pipes module (PipesInterface/AbstractPipes/UnixPipes/
WindowsPipes), ProcessUtils and the missing process exceptions
(LogicException/InvalidArgumentException) with constructors.

Methods now return anyhow::Result where PHP throws, take the env
argument and a bool-returning callback, and borrow &mut for
status-updating accessors; all callers are updated accordingly.
Extend the php-shim with proc_open/proc_close (PHP-compatible
signatures), proc_get_status, proc_terminate, posix_kill, uniqid,
ftruncate, ftell_stream, fseek3, stream_get_contents3 and env
helpers.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/crates/shirabe/src/package/version/version_guesser.rs b/crates/shirabe/src/package/version/version_guesser.rs
index 42646cb..7f0625f 100644
--- a/crates/shirabe/src/package/version/version_guesser.rs
+++ b/crates/shirabe/src/package/version/version_guesser.rs
@@ -533,14 +533,14 @@ impl VersionGuesser {
                         },
                         &scm_cmdline,
                     );
-                    let process = tokio::runtime::Runtime::new()
+                    let mut process = tokio::runtime::Runtime::new()
                         .unwrap()
                         .block_on(self.process.borrow_mut().execute_async(&cmd_line, path))?;
                     if !process.is_successful() {
                         continue;
                     }
 
-                    let output = process.get_output();
+                    let output = process.get_output()?;
                     // overwrite existing if we have a shorter diff, or we have an equal diff and an index that comes later in the array (i.e. older version)
                     // as newer versions typically have more commits, if the feature branch is based on a newer branch it should have a longer diff to the old version
                     // but if it doesn't and they have equal diffs, then it probably is based on the old version