optimize builder attribute

Author: maxbeckers

docs/features/configuration.md

@@ -105,10 +105,8 @@ The `#[Builder]` attribute provides fine-grained control over individual builder
     className: 'CustomUserBuilder',
     namespace: 'App\\Builders',
     fluent: true,
-    generateFactory: false,
     exclude: ['password'],
     include: ['name', 'email'],
-    immutable: false,
     builderMethod: 'create'
 )]
 class User { ... }
@@ -121,10 +119,8 @@ class User { ... }
 | `className` | `?string` | `null` | Custom name for the generated builder class |
 | `namespace` | `?string` | `null` | Custom namespace for the generated builder |
 | `fluent` | `bool` | `true` | Whether setter methods should return `self` for chaining |
-| `generateFactory` | `bool` | `false` | Generate an additional factory class |
 | `exclude` | `array` | `[]` | Property names to exclude from the builder |
 | `include` | `array` | `[]` | Only include these properties (if set, overrides exclude) |
-| `immutable` | `bool` | `false` | Whether to treat the target class as immutable |
 | `builderMethod` | `string` | `'builder'` | Name of the static factory method |
 
 ## Advanced Configuration Examples
@@ -203,19 +199,6 @@ $user = UserBuilder::create()  // Instead of ::builder()
     ->build();
 ```
 
-### Immutable Objects
-
-```php
-#[Builder(immutable: true)]
-class Money
-{
-    public function __construct(
-        public readonly int $amount,
-        public readonly string $currency
-    ) {}
-}
-```
-
 ## Generator-Specific Configuration
 
 The `generator-config` section allows for future extensibility:

docs/getting-started/quick-start.md

@@ -126,16 +126,4 @@ class Email
 }
 ```
 
-### Immutable Object
-```php
-#[Builder(immutable: true)]
-class Money
-{
-    public function __construct(
-        public readonly int $amount,
-        public readonly string $currency
-    ) {}
-}
-```
-
 Ready to dive deeper? Check out our [configuration guide](../features/configuration.md)!

src/Attribute/Builder.php

@@ -13,10 +13,8 @@ public function __construct(
         public ?string $className = null,
         public ?string $namespace = null,
         public bool $fluent = true,
-        public bool $generateFactory = false,
         public array $exclude = [],
         public array $include = [],
-        public bool $immutable = false,
         public string $builderMethod = 'builder'
     ) {
     }

src/Generator/BuilderGenerator.php

@@ -62,7 +62,6 @@ public function generate(GenerationContext $context): array
 
         $results = [];
 
-        // Generate main builder
         $builderCode = $this->templateEngine->render('builder.php.twig', $templateContext);
         $results[] = [
             'type' => 'builder',
@@ -71,20 +70,6 @@ public function generate(GenerationContext $context): array
             'path' => $this->getOutputPath($context, $builderClassName, $builderNamespace)
         ];
 
-        // Generate factory if requested
-        if ($builderAttribute->generateFactory) {
-            $factoryClassName = $builderClassName . 'Factory';
-            $templateContext['factory_class_name'] = $factoryClassName;
-
-            $factoryCode = $this->templateEngine->render('factory.php.twig', $templateContext);
-            $results[] = [
-                'type' => 'factory',
-                'class_name' => $factoryClassName,
-                'content' => $factoryCode,
-                'path' => $this->getOutputPath($context, $factoryClassName, $builderNamespace)
-            ];
-        }
-
         return $results;
     }
 

tests/Unit/Attribute/BuilderTest.php

@@ -16,10 +16,8 @@ public function testBuilderAttributeWithDefaults(): void
         $this->assertNull($builder->className);
         $this->assertNull($builder->namespace);
         $this->assertTrue($builder->fluent);
-        $this->assertFalse($builder->generateFactory);
         $this->assertEquals([], $builder->exclude);
         $this->assertEquals([], $builder->include);
-        $this->assertFalse($builder->immutable);
         $this->assertEquals('builder', $builder->builderMethod);
     }
 
@@ -29,20 +27,16 @@ public function testBuilderAttributeWithCustomValues(): void
             className: 'CustomBuilder',
             namespace: 'Custom\\Namespace',
             fluent: false,
-            generateFactory: true,
             exclude: ['password'],
             include: ['name', 'email'],
-            immutable: true,
             builderMethod: 'builder'
         );
 
         $this->assertEquals('CustomBuilder', $builder->className);
         $this->assertEquals('Custom\\Namespace', $builder->namespace);
         $this->assertFalse($builder->fluent);
-        $this->assertTrue($builder->generateFactory);
         $this->assertEquals(['password'], $builder->exclude);
         $this->assertEquals(['name', 'email'], $builder->include);
-        $this->assertTrue($builder->immutable);
         $this->assertEquals('builder', $builder->builderMethod);
     }
 }