refactor(host_runtime): keep the install-input error path log-free

The moved executor returns `SkillManagementCapabilityError`, and routing it
through `skill_management_error` would have added a `debug!` line to a path
that had none before the move. A move-only change must not add one, so the
install-input arm maps the kind directly and the `dispatch` arm keeps the
record it already had.

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

crates/ironclaw_host_runtime/src/first_party_tools/skill_management.rs

@@ -138,8 +138,13 @@ impl FirstPartyCapabilityHandler for SkillManagementToolHandler {
                 &mut usage,
             )
             .await
+            // Deliberately NOT `skill_management_error`: the install-input path
+            // never logged before this executor moved out of the crate, and a
+            // move-only change must not add a log line. The `dispatch` arm below
+            // keeps the debug record it already had.
             .map_err(|error| {
-                skill_management_error(error).with_usage(usage_with_elapsed(&usage, started))
+                FirstPartyCapabilityError::new(error.kind())
+                    .with_usage(usage_with_elapsed(&usage, started))
             })?
         } else {
             request.input.clone()