fix(input): return a bool|string|string[]|null enum from get_option

`InputInterface::get_option` returned `PhpMixed`, so the negation branch
of `Input::get_option` reproduced PHP's `return !$value;` as
`!value.as_bool().unwrap_or(false)`, which inverts the result for a
string value instead of leaving it `false`. It now returns
`InputOptionValue`, whose `to_bool` is PHP's truthiness cast.

The narrowing also removes the `Vec<PhpMixed>` element handling the
commands carried for array options: `as_array` hands back `&[String]`,
so the `filter_map(|v| v.as_string())` chains at eight call sites
collapse. Its other accessors keep `PhpMixed`'s names and meanings
(`is_null`, `as_bool`, `as_string`, `to_bool`), and
`From<InputOptionValue> for PhpMixed` covers the callers that feed the
value back into an `IndexMap<String, PhpMixed>` or a `PhpMixed`
parameter.

`Input` keeps its parsed options and `InputOption` its defaults as
`PhpMixed`, so `Input::get_option` is where the narrowing happens and
where a value outside the domain panics.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

6 files changed, 103 insertions, 10 deletions

diff --git a/crates/shirabe-symfony-console/src/input/argv_input.rs b/crates/shirabe-symfony-console/src/input/argv_input.rs
index 94e5299d..6138c195 100644
--- a/crates/shirabe-symfony-console/src/input/argv_input.rs
+++ b/crates/shirabe-symfony-console/src/input/argv_input.rs
@@ -4,6 +4,7 @@ use crate::exception::RuntimeException;
 use crate::input::Input;
 use crate::input::InputDefinition;
 use crate::input::InputInterface;
+use crate::input::InputOptionValue;
 use crate::input::StreamableInputInterface;
 use indexmap::IndexMap;
 use shirabe_php_shim::{PhpMixed, php_regex, preg_match};
@@ -593,7 +594,7 @@ impl InputInterface for ArgvInput {
         self.inner.get_options()
     }
 
-    fn get_option(&self, name: &str) -> anyhow::Result<PhpMixed> {
+    fn get_option(&self, name: &str) -> anyhow::Result<InputOptionValue> {
         self.inner.get_option(name)
     }
 
diff --git a/crates/shirabe-symfony-console/src/input/array_input.rs b/crates/shirabe-symfony-console/src/input/array_input.rs
index a032432b..c9bc9dce 100644
--- a/crates/shirabe-symfony-console/src/input/array_input.rs
+++ b/crates/shirabe-symfony-console/src/input/array_input.rs
@@ -5,6 +5,7 @@ use crate::exception::InvalidOptionException;
 use crate::input::Input;
 use crate::input::InputDefinition;
 use crate::input::InputInterface;
+use crate::input::InputOptionValue;
 use crate::input::StreamableInputInterface;
 use indexmap::IndexMap;
 use shirabe_php_shim::PhpMixed;
@@ -336,7 +337,7 @@ impl InputInterface for ArrayInput {
         self.inner.get_options()
     }
 
-    fn get_option(&self, name: &str) -> anyhow::Result<PhpMixed> {
+    fn get_option(&self, name: &str) -> anyhow::Result<InputOptionValue> {
         self.inner.get_option(name)
     }
 
diff --git a/crates/shirabe-symfony-console/src/input/input.rs b/crates/shirabe-symfony-console/src/input/input.rs
index 88bb9f94..795e9c81 100644
--- a/crates/shirabe-symfony-console/src/input/input.rs
+++ b/crates/shirabe-symfony-console/src/input/input.rs
@@ -3,6 +3,7 @@
 use crate::exception::InvalidArgumentException;
 use crate::exception::RuntimeException;
 use crate::input::InputDefinition;
+use crate::input::InputOptionValue;
 use indexmap::IndexMap;
 use shirabe_php_shim::{PhpMixed, PhpResource, php_regex, preg_is_match};
 
@@ -156,14 +157,14 @@ impl Input {
         )
     }
 
-    pub fn get_option(&self, name: &str) -> anyhow::Result<PhpMixed> {
+    pub fn get_option(&self, name: &str) -> anyhow::Result<InputOptionValue> {
         if self.definition.has_negation(name) {
             let value = self.get_option(&self.definition.negation_to_name(name)?)?;
-            if matches!(value, PhpMixed::Null) {
+            if value.is_null() {
                 return Ok(value);
             }
 
-            return Ok(PhpMixed::Bool(!value.as_bool().unwrap_or(false)));
+            return Ok(InputOptionValue::Bool(!value.to_bool()));
         }
 
         if !self.definition.has_option(name) {
@@ -174,10 +175,11 @@ impl Input {
             .into());
         }
 
-        Ok(if self.options.contains_key(name) {
-            self.options[name].clone()
+        Ok(if let Some(value) = self.options.get(name) {
+            InputOptionValue::from_php_mixed(value)
         } else {
-            self.definition.get_option(name)?.get_default().clone()
+            let option = self.definition.get_option(name)?;
+            InputOptionValue::from_php_mixed(option.get_default())
         })
     }
 
diff --git a/crates/shirabe-symfony-console/src/input/input_interface.rs b/crates/shirabe-symfony-console/src/input/input_interface.rs
index 4491ca8c..3c3ec2d5 100644
--- a/crates/shirabe-symfony-console/src/input/input_interface.rs
+++ b/crates/shirabe-symfony-console/src/input/input_interface.rs
@@ -1,6 +1,7 @@
 //! ref: composer/vendor/symfony/console/Input/InputInterface.php
 
 use crate::input::InputDefinition;
+use crate::input::InputOptionValue;
 use crate::input::StreamableInputInterface;
 use shirabe_php_shim::PhpMixed;
 
@@ -33,7 +34,7 @@ pub trait InputInterface: std::fmt::Debug + shirabe_php_shim::AsAny {
 
     fn get_options(&self) -> indexmap::IndexMap<String, PhpMixed>;
 
-    fn get_option(&self, name: &str) -> anyhow::Result<PhpMixed>;
+    fn get_option(&self, name: &str) -> anyhow::Result<InputOptionValue>;
 
     fn set_option(&mut self, name: &str, value: PhpMixed) -> anyhow::Result<()>;
 
diff --git a/crates/shirabe-symfony-console/src/input/input_option.rs b/crates/shirabe-symfony-console/src/input/input_option.rs
index 3b24d914..d37e18ae 100644
--- a/crates/shirabe-symfony-console/src/input/input_option.rs
+++ b/crates/shirabe-symfony-console/src/input/input_option.rs
@@ -191,3 +191,90 @@ impl InputOption {
             && option.is_value_optional() == self.is_value_optional()
     }
 }
+
+/// The `bool|string|string[]|null` domain of a parsed option value, as returned by
+/// [`InputInterface::get_option`](crate::input::InputInterface::get_option).
+#[derive(Debug, Clone, PartialEq)]
+pub enum InputOptionValue {
+    Null,
+    Bool(bool),
+    String(String),
+    Array(Vec<String>),
+}
+
+impl InputOptionValue {
+    /// Narrows a raw option value to this domain.
+    ///
+    /// TODO(type-model): `Input` keeps parsed options and `InputOption` defaults as `PhpMixed`, so
+    /// a value outside this domain — an int, a float, or an array holding one — can only be
+    /// rejected here.
+    pub(crate) fn from_php_mixed(value: &PhpMixed) -> Self {
+        match value {
+            PhpMixed::Null => Self::Null,
+            PhpMixed::Bool(b) => Self::Bool(*b),
+            PhpMixed::String(s) => Self::String(s.clone()),
+            PhpMixed::List(_) | PhpMixed::Array(_) => Self::Array(
+                value
+                    .values()
+                    .into_iter()
+                    .map(|item| match item {
+                        PhpMixed::String(s) => s.clone(),
+                        other => panic!("an option array holds {:?}, not a string", other),
+                    })
+                    .collect(),
+            ),
+            other => panic!(
+                "an option holds {:?}, not a bool, string, array or null",
+                other
+            ),
+        }
+    }
+
+    pub fn is_null(&self) -> bool {
+        matches!(self, Self::Null)
+    }
+
+    pub fn as_bool(&self) -> Option<bool> {
+        match self {
+            Self::Bool(b) => Some(*b),
+            _ => None,
+        }
+    }
+
+    pub fn as_string(&self) -> Option<&str> {
+        match self {
+            Self::String(s) => Some(s.as_str()),
+            _ => None,
+        }
+    }
+
+    pub fn as_array(&self) -> Option<&[String]> {
+        match self {
+            Self::Array(items) => Some(items),
+            _ => None,
+        }
+    }
+
+    /// PHP loose boolean cast `(bool) $value`.
+    pub fn to_bool(&self) -> bool {
+        match self {
+            Self::Null => false,
+            Self::Bool(b) => *b,
+            Self::String(s) => !s.is_empty() && s != "0",
+            Self::Array(items) => !items.is_empty(),
+        }
+    }
+}
+
+impl From<InputOptionValue> for PhpMixed {
+    fn from(value: InputOptionValue) -> Self {
+        match value {
+            InputOptionValue::Null => PhpMixed::Null,
+            InputOptionValue::Bool(b) => PhpMixed::Bool(b),
+            InputOptionValue::String(s) => PhpMixed::String(s),
+            InputOptionValue::Array(items) => {
+                PhpMixed::List(items.into_iter().map(PhpMixed::String).collect())
+            }
+        }
+    }
+}
diff --git a/crates/shirabe-symfony-console/src/input/string_input.rs b/crates/shirabe-symfony-console/src/input/string_input.rs
index bdd21577..9fcb3fc8 100644
--- a/crates/shirabe-symfony-console/src/input/string_input.rs
+++ b/crates/shirabe-symfony-console/src/input/string_input.rs
@@ -4,6 +4,7 @@ use crate::exception::InvalidArgumentException;
 use crate::input::ArgvInput;
 use crate::input::InputDefinition;
 use crate::input::InputInterface;
+use crate::input::InputOptionValue;
 use crate::input::StreamableInputInterface;
 use indexmap::IndexMap;
 use shirabe_php_shim::{PhpMixed, php_regex, preg_match};
@@ -177,7 +178,7 @@ impl InputInterface for StringInput {
         InputInterface::get_options(&self.inner)
     }
 
-    fn get_option(&self, name: &str) -> anyhow::Result<PhpMixed> {
+    fn get_option(&self, name: &str) -> anyhow::Result<InputOptionValue> {
         self.inner.get_option(name)
     }