refactor(php-shim): replace literal sprintf calls with format!

Convert every sprintf() call with a compile-time literal format string to
format!, implementing Display for PhpMixed (delegating to php_to_string) so
PhpMixed values render with PHP string semantics through {}. Also merge the
format!-wrapped and conditional-literal dynamic sites into single format!
calls. Genuinely runtime format strings (table styles, configurable error
messages, command synopsis, progress-bar modifiers, regex-built messages)
still go through sprintf.

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

1 files changed, 8 insertions, 6 deletions

diff --git a/crates/shirabe/src/util/filesystem.rs b/crates/shirabe/src/util/filesystem.rs
index f54dfca..f693598 100644
--- a/crates/shirabe/src/util/filesystem.rs
+++ b/crates/shirabe/src/util/filesystem.rs
@@ -519,9 +519,10 @@ impl Filesystem {
             // Returning early-formatted Result is not possible without changing signature; panic to surface in tests.
             panic!(
                 "{}",
-                sprintf(
-                    "$from (%s) and $to (%s) must be absolute paths.",
-                    &[from.to_string().into(), to.to_string().into()]
+                format!(
+                    "$from ({}) and $to ({}) must be absolute paths.",
+                    from.to_string(),
+                    to.to_string()
                 )
             );
         }
@@ -583,9 +584,10 @@ impl Filesystem {
         if !self.is_absolute_path(from) || !self.is_absolute_path(to) {
             panic!(
                 "{}",
-                sprintf(
-                    "$from (%s) and $to (%s) must be absolute paths.",
-                    &[from.to_string().into(), to.to_string().into()]
+                format!(
+                    "$from ({}) and $to ({}) must be absolute paths.",
+                    from.to_string(),
+                    to.to_string()
                 )
             );
         }