Skip to content
This repository was archived by the owner on Jul 16, 2025. It is now read-only.
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions src/Platform/Message/AssistantMessage.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,16 @@
namespace PhpLlm\LlmChain\Platform\Message;

use PhpLlm\LlmChain\Platform\Response\ToolCall;
use Symfony\Component\Uid\AbstractUid;
use Symfony\Component\Uid\TimeBasedUidInterface;
use Symfony\Component\Uid\Uuid;

/**
* @author Denis Zunke <denis.zunke@gmail.com>
*/
final readonly class AssistantMessage implements MessageInterface
{
public Uuid $id;
public AbstractUid&TimeBasedUidInterface $id;

/**
* @param ?ToolCall[] $toolCalls
Expand All @@ -29,7 +31,7 @@ public function getRole(): Role
return Role::Assistant;
}

public function getId(): Uuid
public function getId(): AbstractUid&TimeBasedUidInterface
{
return $this->id;
}
Expand Down
5 changes: 3 additions & 2 deletions src/Platform/Message/MessageInterface.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@

namespace PhpLlm\LlmChain\Platform\Message;

use Symfony\Component\Uid\Uuid;
use Symfony\Component\Uid\AbstractUid;
use Symfony\Component\Uid\TimeBasedUidInterface;

/**
* @author Denis Zunke <denis.zunke@gmail.com>
Expand All @@ -13,5 +14,5 @@ interface MessageInterface
{
public function getRole(): Role;

public function getId(): Uuid;
public function getId(): AbstractUid&TimeBasedUidInterface;
}
6 changes: 4 additions & 2 deletions src/Platform/Message/SystemMessage.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,16 @@

namespace PhpLlm\LlmChain\Platform\Message;

use Symfony\Component\Uid\AbstractUid;
use Symfony\Component\Uid\TimeBasedUidInterface;
use Symfony\Component\Uid\Uuid;

/**
* @author Denis Zunke <denis.zunke@gmail.com>
*/
final readonly class SystemMessage implements MessageInterface
{
public Uuid $id;
public AbstractUid&TimeBasedUidInterface $id;

public function __construct(public string $content)
{
Expand All @@ -23,7 +25,7 @@ public function getRole(): Role
return Role::System;
}

public function getId(): Uuid
public function getId(): AbstractUid&TimeBasedUidInterface
{
return $this->id;
}
Expand Down
6 changes: 4 additions & 2 deletions src/Platform/Message/ToolCallMessage.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,16 @@
namespace PhpLlm\LlmChain\Platform\Message;

use PhpLlm\LlmChain\Platform\Response\ToolCall;
use Symfony\Component\Uid\AbstractUid;
use Symfony\Component\Uid\TimeBasedUidInterface;
use Symfony\Component\Uid\Uuid;

/**
* @author Denis Zunke <denis.zunke@gmail.com>
*/
final readonly class ToolCallMessage implements MessageInterface
{
public Uuid $id;
public AbstractUid&TimeBasedUidInterface $id;

public function __construct(
public ToolCall $toolCall,
Expand All @@ -26,7 +28,7 @@ public function getRole(): Role
return Role::ToolCall;
}

public function getId(): Uuid
public function getId(): AbstractUid&TimeBasedUidInterface
{
return $this->id;
}
Expand Down
6 changes: 4 additions & 2 deletions src/Platform/Message/UserMessage.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
use PhpLlm\LlmChain\Platform\Message\Content\ContentInterface;
use PhpLlm\LlmChain\Platform\Message\Content\Image;
use PhpLlm\LlmChain\Platform\Message\Content\ImageUrl;
use Symfony\Component\Uid\AbstractUid;
use Symfony\Component\Uid\TimeBasedUidInterface;
use Symfony\Component\Uid\Uuid;

/**
Expand All @@ -20,7 +22,7 @@
*/
public array $content;

public Uuid $id;
public AbstractUid&TimeBasedUidInterface $id;

public function __construct(
ContentInterface ...$content,
Expand All @@ -34,7 +36,7 @@ public function getRole(): Role
return Role::User;
}

public function getId(): Uuid
public function getId(): AbstractUid&TimeBasedUidInterface
{
return $this->id;
}
Expand Down
4 changes: 3 additions & 1 deletion tests/Platform/ContractTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@
use PHPUnit\Framework\Attributes\Test;
use PHPUnit\Framework\Attributes\UsesClass;
use PHPUnit\Framework\TestCase;
use Symfony\Component\Uid\AbstractUid;
use Symfony\Component\Uid\TimeBasedUidInterface;
use Symfony\Component\Uid\Uuid;

#[Large]
Expand Down Expand Up @@ -201,7 +203,7 @@ public function getRole(): Role
return Role::User;
}

public function getId(): Uuid
public function getId(): AbstractUid&TimeBasedUidInterface
{
return Uuid::v7();
}
Expand Down
12 changes: 12 additions & 0 deletions tests/Platform/Message/AssistantMessageTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@
use PHPUnit\Framework\Attributes\Test;
use PHPUnit\Framework\Attributes\UsesClass;
use PHPUnit\Framework\TestCase;
use Symfony\Component\Uid\AbstractUid;
use Symfony\Component\Uid\TimeBasedUidInterface;
use Symfony\Component\Uid\UuidV7;

#[CoversClass(AssistantMessage::class)]
Expand Down Expand Up @@ -79,4 +81,14 @@ public function sameMessagesHaveDifferentUids(): void
self::assertIsUuidV7($message1->getId()->toRfc4122());
self::assertIsUuidV7($message2->getId()->toRfc4122());
}

#[Test]
public function messageIdImplementsRequiredInterfaces(): void
{
$message = new AssistantMessage('test');

self::assertInstanceOf(AbstractUid::class, $message->getId());
self::assertInstanceOf(TimeBasedUidInterface::class, $message->getId());
self::assertInstanceOf(UuidV7::class, $message->getId());
}
}
12 changes: 12 additions & 0 deletions tests/Platform/Message/SystemMessageTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@
use PHPUnit\Framework\Attributes\Small;
use PHPUnit\Framework\Attributes\Test;
use PHPUnit\Framework\TestCase;
use Symfony\Component\Uid\AbstractUid;
use Symfony\Component\Uid\TimeBasedUidInterface;
use Symfony\Component\Uid\UuidV7;

#[CoversClass(SystemMessage::class)]
Expand Down Expand Up @@ -59,4 +61,14 @@ public function sameMessagesHaveDifferentUids(): void
self::assertIsUuidV7($message1->getId()->toRfc4122());
self::assertIsUuidV7($message2->getId()->toRfc4122());
}

#[Test]
public function messageIdImplementsRequiredInterfaces(): void
{
$message = new SystemMessage('test');

self::assertInstanceOf(AbstractUid::class, $message->getId());
self::assertInstanceOf(TimeBasedUidInterface::class, $message->getId());
self::assertInstanceOf(UuidV7::class, $message->getId());
}
}
13 changes: 13 additions & 0 deletions tests/Platform/Message/ToolCallMessageTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@
use PHPUnit\Framework\Attributes\Test;
use PHPUnit\Framework\Attributes\UsesClass;
use PHPUnit\Framework\TestCase;
use Symfony\Component\Uid\AbstractUid;
use Symfony\Component\Uid\TimeBasedUidInterface;
use Symfony\Component\Uid\UuidV7;

#[CoversClass(ToolCallMessage::class)]
Expand Down Expand Up @@ -65,4 +67,15 @@ public function sameMessagesHaveDifferentUids(): void
self::assertIsUuidV7($message1->getId()->toRfc4122());
self::assertIsUuidV7($message2->getId()->toRfc4122());
}

#[Test]
public function messageIdImplementsRequiredInterfaces(): void
{
$toolCall = new ToolCall('foo', 'bar');
$message = new ToolCallMessage($toolCall, 'test');

self::assertInstanceOf(AbstractUid::class, $message->getId());
self::assertInstanceOf(TimeBasedUidInterface::class, $message->getId());
self::assertInstanceOf(UuidV7::class, $message->getId());
}
}
12 changes: 12 additions & 0 deletions tests/Platform/Message/UserMessageTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@
use PHPUnit\Framework\Attributes\Test;
use PHPUnit\Framework\Attributes\UsesClass;
use PHPUnit\Framework\TestCase;
use Symfony\Component\Uid\AbstractUid;
use Symfony\Component\Uid\TimeBasedUidInterface;
use Symfony\Component\Uid\UuidV7;

#[CoversClass(UserMessage::class)]
Expand Down Expand Up @@ -109,4 +111,14 @@ public function sameMessagesHaveDifferentUids(): void
self::assertIsUuidV7($message1->getId()->toRfc4122());
self::assertIsUuidV7($message2->getId()->toRfc4122());
}

#[Test]
public function messageIdImplementsRequiredInterfaces(): void
{
$message = new UserMessage(new Text('test'));

self::assertInstanceOf(AbstractUid::class, $message->getId());
self::assertInstanceOf(TimeBasedUidInterface::class, $message->getId());
self::assertInstanceOf(UuidV7::class, $message->getId());
}
}