minor #657 [AI Bundle][MCP Bundle][Platform][Store] Remove useless inline comments from codebase (OskarStark)

This PR was squashed before being merged into the main branch.

Discussion
----------

[AI Bundle][MCP Bundle][Platform][Store] Remove useless inline comments from codebase

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| Docs?         | no
| Issues        | --
| License       | MIT

Remove redundant inline comments that simply restate what the code is doing, including obvious state transitions in tests, section separators, and workflow comments that don't add meaningful context.

Commits
-------

b6df2ff3 [AI Bundle][MCP Bundle][Platform][Store] Remove useless inline comments from codebase

Author: OskarStark

tests/Bridge/Gemini/Contract/MessageBagNormalizerTest.php

@@ -66,11 +66,9 @@ public function testNormalize(MessageBag $bag, array $expected)
     {
         $normalizer = new MessageBagNormalizer();
 
-        // Set up the inner normalizers
         $userMessageNormalizer = new UserMessageNormalizer();
         $assistantMessageNormalizer = new AssistantMessageNormalizer();
 
-        // Mock a normalizer that delegates to the appropriate concrete normalizer
         $mockNormalizer = $this->createMock(NormalizerInterface::class);
         $mockNormalizer->method('normalize')
             ->willReturnCallback(function ($message) use ($userMessageNormalizer, $assistantMessageNormalizer): ?array {

tests/Bridge/HuggingFace/Contract/MessageBagNormalizerTest.php

@@ -55,10 +55,8 @@ public function testNormalize(MessageBag $bag, array $expected)
     {
         $normalizer = new MessageBagNormalizer();
 
-        // Set up the concrete user message normalizer
         $userMessageNormalizer = new UserMessageNormalizer();
 
-        // Mock a normalizer that delegates to the concrete normalizer
         $mockNormalizer = $this->createMock(NormalizerInterface::class);
         $mockNormalizer->method('normalize')
             ->willReturnCallback(function ($messages) use ($userMessageNormalizer): array {

tests/Bridge/HuggingFace/Output/FillMaskResultTest.php

@@ -168,7 +168,6 @@ public function testFromArrayWithVariousFormats()
 
         $this->assertCount(3, $result->fills);
 
-        // Test edge cases are properly handled
         $this->assertSame(0, $result->fills[0]->token);
         $this->assertSame('', $result->fills[0]->tokenStr);
         $this->assertSame('', $result->fills[0]->sequence);

tests/Bridge/VertexAi/Contract/MessageBagNormalizerTest.php

@@ -66,11 +66,9 @@ public function testNormalize(MessageBag $bag, array $expected)
     {
         $normalizer = new MessageBagNormalizer();
 
-        // Set up the inner normalizers
         $userMessageNormalizer = new UserMessageNormalizer();
         $assistantMessageNormalizer = new AssistantMessageNormalizer();
 
-        // Mock a normalizer that delegates to the appropriate concrete normalizer
         $mockNormalizer = $this->createMock(NormalizerInterface::class);
         $mockNormalizer->method('normalize')
             ->willReturnCallback(function ($message) use ($userMessageNormalizer, $assistantMessageNormalizer): ?array {

tests/Bridge/VertexAi/Embeddings/ModelClientTest.php

@@ -32,7 +32,6 @@ final class ModelClientTest extends TestCase
 {
     public function testItGeneratesTheEmbeddingSuccessfully()
     {
-        // Assert
         $expectedResponse = [
             'predictions' => [
                 ['embeddings' => ['values' => [0.3, 0.4, 0.4]]],
@@ -44,10 +43,8 @@ public function testItGeneratesTheEmbeddingSuccessfully()
 
         $model = new Model(Model::GEMINI_EMBEDDING_001, ['outputDimensionality' => 1536, 'task_type' => TaskType::CLASSIFICATION]);
 
-        // Act
         $result = $client->request($model, 'test payload');
 
-        // Assert
         $this->assertSame($expectedResponse, $result->getData());
     }
 }

tests/Bridge/VertexAi/Embeddings/ResultConverterTest.php

@@ -31,7 +31,6 @@ final class ResultConverterTest extends TestCase
 {
     public function testItConvertsAResponseToAVectorResult()
     {
-        // Assert
         $expectedResponse = [
             'predictions' => [
                 ['embeddings' => ['values' => [0.3, 0.4, 0.4]]],

tests/Bridge/VertexAi/Gemini/ModelClientTest.php

@@ -27,7 +27,6 @@ final class ModelClientTest extends TestCase
 {
     public function testItInvokesTheTextModelsSuccessfully()
     {
-        // Arrange
         $payload = [
             'content' => [
                 ['parts' => ['text' => 'Hello, world!']],
@@ -42,12 +41,10 @@ public function testItInvokesTheTextModelsSuccessfully()
 
         $client = new ModelClient($httpClient, 'global', 'test');
 
-        // Act
         $result = $client->request(new Model(Model::GEMINI_2_0_FLASH), $payload);
         $data = $result->getData();
         $info = $result->getObject()->getInfo();
 
-        // Assert
         $this->assertNotEmpty($data);
         $this->assertNotEmpty($info);
         $this->assertSame('POST', $info['http_method']);

tests/Bridge/VertexAi/Gemini/ResultConverterTest.php

@@ -28,7 +28,6 @@ final class ResultConverterTest extends TestCase
 {
     public function testItConvertsAResponseToAVectorResult()
     {
-        // Arrange
         $payload = [
             'content' => ['parts' => [['text' => 'Hello, world!']]],
         ];
@@ -42,10 +41,8 @@ public function testItConvertsAResponseToAVectorResult()
 
         $resultConverter = new ResultConverter();
 
-        // Act
         $result = $resultConverter->convert(new RawHttpResult($response));
 
-        // Assert
         $this->assertInstanceOf(TextResult::class, $result);
         $this->assertSame('Hello, world!', $result->getContent());
     }

tests/Bridge/VertexAi/TokenOutputProcessorTest.php

@@ -48,7 +48,6 @@ public function testItDoesNothingWithoutRawResponse()
 
     public function testItAddsUsageTokensToMetadata()
     {
-        // Arrange
         $textResult = new TextResult('test');
 
         $rawResponse = $this->createRawResponse([
@@ -64,10 +63,8 @@ public function testItAddsUsageTokensToMetadata()
         $processor = new TokenOutputProcessor();
         $output = $this->createOutput($textResult);
 
-        // Act
         $processor->processOutput($output);
 
-        // Assert
         $metadata = $output->result->getMetadata();
         $tokenUsage = $metadata->get('token_usage');
 
@@ -81,7 +78,6 @@ public function testItAddsUsageTokensToMetadata()
 
     public function testItHandlesMissingUsageFields()
     {
-        // Arrange
         $textResult = new TextResult('test');
 
         $rawResponse = $this->createRawResponse([
@@ -94,10 +90,8 @@ public function testItHandlesMissingUsageFields()
         $processor = new TokenOutputProcessor();
         $output = $this->createOutput($textResult);
 
-        // Act
         $processor->processOutput($output);
 
-        // Assert
         $metadata = $output->result->getMetadata();
         $tokenUsage = $metadata->get('token_usage');
 
@@ -111,17 +105,14 @@ public function testItHandlesMissingUsageFields()
 
     public function testItAddsEmptyTokenUsageWhenUsageMetadataNotPresent()
     {
-        // Arrange
         $textResult = new TextResult('test');
         $rawResponse = $this->createRawResponse(['other' => 'data']);
         $textResult->setRawResult($rawResponse);
         $processor = new TokenOutputProcessor();
         $output = $this->createOutput($textResult);
 
-        // Act
         $processor->processOutput($output);
 
-        // Assert
         $metadata = $output->result->getMetadata();
         $tokenUsage = $metadata->get('token_usage');