From 84655bf3a4a21dbbe7aec8e3aee1e661505bbb6d Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sat, 20 Jun 2026 23:37:42 +0900 Subject: 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 --- .../src/symfony/console/helper/process_helper.rs | 30 +++++++++++++++++++--- 1 file changed, 26 insertions(+), 4 deletions(-) (limited to 'crates/shirabe-external-packages/src/symfony/console/helper') diff --git a/crates/shirabe-external-packages/src/symfony/console/helper/process_helper.rs b/crates/shirabe-external-packages/src/symfony/console/helper/process_helper.rs index a32c95d..e62b9ac 100644 --- a/crates/shirabe-external-packages/src/symfony/console/helper/process_helper.rs +++ b/crates/shirabe-external-packages/src/symfony/console/helper/process_helper.rs @@ -85,7 +85,13 @@ impl ProcessHelper { ProcessHelperCmdElement::Process(_) => unreachable!(), }) .collect(); - process = Process::new(command, None, None, None, Some(60.0)); + process = Process::new( + command, + None, + None, + shirabe_php_shim::PhpMixed::Null, + Some(60.0), + )?; cmd = vec![]; } Some(ProcessHelperCmdElement::Process(_)) => { @@ -124,7 +130,23 @@ impl ProcessHelper { }; // PHP passes the remaining `$cmd` array as the `$env` argument to Process::run. - process.run(callback); + let env: indexmap::IndexMap = cmd + .iter() + .enumerate() + .filter_map(|(i, element)| match element { + ProcessHelperCmdElement::String(s) => { + Some((i.to_string(), shirabe_php_shim::PhpMixed::String(s.clone()))) + } + ProcessHelperCmdElement::Process(_) => None, + }) + .collect(); + let callback: Option bool>> = callback.map(|mut cb| { + Box::new(move |r#type: &str, buffer: &str| -> bool { + cb(r#type, buffer); + false + }) as Box bool> + }); + process.run(callback, env)?; if verbosity <= output.borrow().get_verbosity() { let message = if process.is_successful() { @@ -183,7 +205,7 @@ impl ProcessHelper { error: Option<&str>, callback: Option>, ) -> anyhow::Result { - let process = self.run( + let mut process = self.run( output, cmd, error, @@ -192,7 +214,7 @@ impl ProcessHelper { )?; if !process.is_successful() { - anyhow::bail!(ProcessFailedException::new(&process)); + anyhow::bail!(ProcessFailedException::new(&mut process)?); } Ok(process) -- cgit v1.3.1