Fixed: enforce absolute path validation for workdir in execute_command

The BashArgs.workdir documentation states it must be an absolute path,
but this constraint was not enforced. Added is_absolute() check before
is_dir() check to provide a clearer error message when a relative path
is provided.

Author: Sewer56

src/llm-coding-tools-core/src/operations/bash/async_impl.rs

@@ -16,6 +16,12 @@ pub async fn execute_command(
     timeout: Duration,
 ) -> ToolResult<BashOutput> {
     if let Some(dir) = workdir {
+        if !dir.is_absolute() {
+            return Err(ToolError::InvalidPath(format!(
+                "working directory must be an absolute path: {}",
+                dir.display()
+            )));
+        }
         if !dir.is_dir() {
             return Err(ToolError::InvalidPath(format!(
                 "working directory does not exist: {}",

src/llm-coding-tools-core/src/operations/bash/blocking_impl.rs

@@ -16,6 +16,12 @@ pub fn execute_command(
     timeout: Duration,
 ) -> ToolResult<BashOutput> {
     if let Some(dir) = workdir {
+        if !dir.is_absolute() {
+            return Err(ToolError::InvalidPath(format!(
+                "working directory must be an absolute path: {}",
+                dir.display()
+            )));
+        }
         if !dir.is_dir() {
             return Err(ToolError::InvalidPath(format!(
                 "working directory does not exist: {}",