Apply review feedback

Author: svenmuennich

src/Capability/Discovery/Discoverer.php

@@ -114,7 +114,6 @@ public function discover(string $basePath, array $directories, array $excludeDir
         } catch (\Throwable $e) {
             $this->logger->error('Error during file finding process for MCP discovery'.json_encode($e->getTrace(), \JSON_PRETTY_PRINT), [
                 'exception' => $e,
-                'trace' => $e->getTraceAsString(),
             ]);
         }
 
@@ -196,7 +195,6 @@ private function processFile(SplFileInfo $file, array &$discoveredCount, array &
                 'file' => $file->getPathname(),
                 'class' => $className,
                 'exception' => $e,
-                'trace' => $e->getTraceAsString(),
             ]);
         }
     }
@@ -297,9 +295,15 @@ private function processMethod(\ReflectionMethod $method, array &$discoveredCoun
                     break;
             }
         } catch (ExceptionInterface $e) {
-            $this->logger->error("Failed to process MCP attribute on {$className}::{$methodName}", ['attribute' => $attributeClassName, 'exception' => $e, 'trace' => $e->getPrevious() ? $e->getPrevious()->getTraceAsString() : $e->getTraceAsString()]);
+            $this->logger->error("Failed to process MCP attribute on {$className}::{$methodName}", [
+                'attribute' => $attributeClassName,
+                'exception' => $e,
+            ]);
         } catch (\Throwable $e) {
-            $this->logger->error("Unexpected error processing attribute on {$className}::{$methodName}", ['attribute' => $attributeClassName, 'exception' => $e, 'trace' => $e->getTraceAsString()]);
+            $this->logger->error("Unexpected error processing attribute on {$className}::{$methodName}", [
+                'attribute' => $attributeClassName,
+                'exception' => $e,
+            ]);
         }
     }
 

src/Capability/Discovery/DocBlockParser.php

@@ -47,9 +47,7 @@ public function parseDocBlock(string|false|null $docComment): ?DocBlock
         } catch (\Throwable $e) {
             // Log error or handle gracefully if invalid DocBlock syntax is encountered
             $this->logger->warning('Failed to parse DocBlock', [
-                'error' => $e->getMessage(),
                 'exception' => $e,
-                'exception_trace' => $e->getTraceAsString(),
             ]);
 
             return null;

src/Capability/Discovery/SchemaValidator.php

@@ -88,8 +88,6 @@ public function validateAgainstJsonSchema(mixed $data, array|object $schema): ar
         } catch (\Throwable $e) {
             $this->logger->error('MCP SDK: JSON Schema validation failed internally.', [
                 'exception' => $e,
-                'exception_message' => $e->getMessage(),
-                'exception_trace' => $e->getTraceAsString(),
                 'data' => json_encode($dataToValidate),
                 'schema' => json_encode($schemaObject),
             ]);

src/Client/Transport/HttpTransport.php

@@ -170,7 +170,7 @@ public function close(): void
                 $this->httpClient->sendRequest($request);
                 $this->logger->info('Session closed', ['session_id' => $this->sessionId]);
             } catch (\Throwable $e) {
-                $this->logger->warning('Failed to close session', ['error' => $e->getMessage(), 'exception' => $e]);
+                $this->logger->warning('Failed to close session', ['exception' => $e]);
             }
         }