fix: Ensure object properties can have a different title

Author: tymondesigns

src/Contracts/JsonSchema.php

@@ -98,4 +98,9 @@ public function version(SchemaVersion $schemaVersion): static;
      * Get the JSON Schema version for this schema.
      */
     public function getVersion(): SchemaVersion;
+
+    /**
+     * Get the initial title.
+     */
+    public function getInitialTitle(): ?string;
 }

src/Types/AbstractSchema.php

@@ -19,6 +19,7 @@
 use Cortex\JsonSchema\Types\Concerns\HasDefinitions;
 use Cortex\JsonSchema\Types\Concerns\HasDescription;
 use Cortex\JsonSchema\Types\Concerns\HasConditionals;
+use Cortex\JsonSchema\Types\Concerns\HasInitialTitle;
 use Cortex\JsonSchema\Types\Concerns\ValidatesVersionFeatures;
 
 abstract class AbstractSchema implements JsonSchema
@@ -33,8 +34,9 @@ abstract class AbstractSchema implements JsonSchema
     use HasReadWrite;
     use HasValidation;
     use HasDescription;
-    use HasConditionals;
     use HasDefinitions;
+    use HasConditionals;
+    use HasInitialTitle;
     use ValidatesVersionFeatures;
 
     protected SchemaVersion $schemaVersion = SchemaVersion::Draft_2020_12;
@@ -47,6 +49,7 @@ public function __construct(
         ?string $title = null,
         ?SchemaVersion $schemaVersion = null,
     ) {
+        $this->initialTitle = $title;
         $this->title = $title;
         $this->schemaVersion = $schemaVersion ?? SchemaVersion::default();
     }

src/Types/Concerns/HasInitialTitle.php

@@ -0,0 +1,21 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Cortex\JsonSchema\Types\Concerns;
+
+/** @mixin \Cortex\JsonSchema\Contracts\JsonSchema */
+trait HasInitialTitle
+{
+    protected ?string $initialTitle = null;
+
+    /**
+     * Get the initial title.
+     *
+     * @internal
+     */
+    public function getInitialTitle(): ?string
+    {
+        return $this->initialTitle;
+    }
+}

src/Types/Concerns/HasProperties.php

@@ -55,7 +55,7 @@ trait HasProperties
     public function properties(JsonSchema ...$properties): static
     {
         foreach ($properties as $property) {
-            $title = $property->getTitle();
+            $title = $this->resolvePropertyTitle($property);
 
             if ($title === null) {
                 throw new SchemaException('Property must have a title');
@@ -253,12 +253,8 @@ public function hasRequiredProperties(): bool
      */
     public function requireAll(): static
     {
-        foreach ($this->properties as $property) {
-            $title = $property->getTitle();
-
-            if ($title !== null) {
-                $this->requiredProperties[] = $title;
-            }
+        foreach ($this->properties as $name => $property) {
+            $this->requiredProperties[] = $name;
         }
 
         return $this;
@@ -277,7 +273,15 @@ protected function addPropertiesToSchema(array $schema): array
             $schema['properties'] = [];
 
             foreach ($this->properties as $name => $prop) {
-                $schema['properties'][$name] = $prop->toArray(includeSchemaRef: false, includeTitle: false);
+                $propertySchema = $prop->toArray(includeSchemaRef: false, includeTitle: true);
+
+                // If the property schema has a title and it matches the name,
+                // then we don't need to include it in the schema
+                if (array_key_exists('title', $propertySchema) && $propertySchema['title'] === $name) {
+                    unset($propertySchema['title']);
+                }
+
+                $schema['properties'][$name] = $propertySchema;
             }
         }
 
@@ -331,6 +335,14 @@ protected function addPropertiesToSchema(array $schema): array
         return $schema;
     }
 
+    /**
+     * Resolve the property title from a schema instance.
+     */
+    protected function resolvePropertyTitle(JsonSchema $property): ?string
+    {
+        return $property->getInitialTitle() ?? $property->getTitle();
+    }
+
     /**
      * Get unevaluated properties features used by this schema.
      *

tests/Unit/Types/ObjectSchemaTest.php

@@ -113,6 +113,21 @@
     }
 })->throws(SchemaException::class);
 
+it('can set set the property title separately from the property key', function (): void {
+    $objectSchema = Schema::object()
+        ->properties(
+            Schema::string('foo')->title('Foo'),
+            Schema::string('bar'),
+        );
+
+    $schemaArray = $objectSchema->toArray();
+
+    expect($schemaArray)->toHaveKey('properties.foo.type', 'string');
+    expect($schemaArray)->toHaveKey('properties.foo.title', 'Foo');
+    expect($schemaArray)->toHaveKey('properties.bar.type', 'string');
+    expect($schemaArray['properties']['bar'])->not->toHaveKey('title');
+});
+
 it('can create an object schema with additional properties control', function (): void {
     $objectSchema = Schema::object('config')
         ->description('Configuration object')