Fix CwaFixtureBuilder: timestamp components added directly via GroupBuilder::add()

Author: silverbackdan

CLAUDE.md

@@ -614,6 +614,12 @@ Services instrumented with optional `?CwaCollectorData` arg: `JWTEventListener`,
 
 ---
 
+## ~~Bug: `CwaFixtureBuilder::createPositions()` does not timestamp components~~ — FIXED
+
+**Fix (committed PLACEHOLDER):** In `createPositions()`, before `$this->persistWithAssociations($component)`, guard with `isConfigured()` and call `persistTimestampedFields($component, true)` when the component is not already in `$persistedEntities`. PHPUnit test added to `tests/Fixture/CwaFixtureBuilderTest.php`.
+
+---
+
 ## Known Configuration Quirks
 
 *(none)*

src/Fixture/CwaFixtureBuilder.php

@@ -534,6 +534,10 @@ private function createPositions(GroupBuilder $groupBuilder): bool
             $position->sortValue = $item['sort'];
             $position->component = $component;
             $position->componentGroup = $componentGroup;
+            if (!isset($this->persistedEntities[spl_object_id($component)])
+                && $this->timestampedPersister->isConfigured($component)) {
+                $this->timestampedPersister->persistTimestampedFields($component, true);
+            }
             $this->timestampedPersister->persistTimestampedFields($position, true);
             $this->persistWithAssociations($component);
             $this->manager->persist($position);

tests/Fixture/CwaFixtureBuilderTest.php

@@ -1999,6 +1999,35 @@ public function test_on_routes_created_fires_exactly_once_across_multiple_flushe
         $this->assertSame(1, $callCount, 'onRoutesCreated must fire exactly once even when flush() is called multiple times');
     }
 
+    public function test_timestamped_persister_called_for_component_added_directly_to_group(): void
+    {
+        $calls = [];
+        $component = new class extends AbstractComponent {};
+
+        $persister = $this->createStub(TimestampedDataPersister::class);
+        $persister->method('isConfigured')->willReturnCallback(
+            static fn (object $entity): bool => $entity instanceof AbstractComponent
+        );
+        $persister->method('persistTimestampedFields')->willReturnCallback(
+            static function (object $entity, bool $isNew) use (&$calls): void {
+                $calls[] = ['entity' => $entity, 'isNew' => $isNew];
+            }
+        );
+
+        $builder = $this->makeBuilder(timestampedPersister: $persister);
+        $builder->layout('main', 'CwaLayoutPrimary');
+        $builder->page('home', 'Template', layout: 'main', isTemplate: true)
+            ->group('primary')
+            ->add($component);
+        $builder->flush();
+
+        $componentCalls = array_values(array_filter($calls, static fn ($c) => $c['entity'] === $component));
+        $this->assertNotEmpty($componentCalls, 'persistTimestampedFields must be called for component added via GroupBuilder::add() when isConfigured() returns true');
+        foreach ($componentCalls as $call) {
+            $this->assertTrue($call['isNew']);
+        }
+    }
+
     // --- readProperty: DoWhile traverses parent class hierarchy (kills not-covered DoWhile mutant line 578) ---
 
     public function test_read_property_finds_property_declared_only_in_parent_class(): void