Standardise Role case names (#116)

* Standardize `Role` case names across enums, annotations, and tests.

* Update minimum code coverage threshold to 92%

Author: pushpak1300

composer.json

@@ -80,7 +80,7 @@
             "pint --test",
             "rector --dry-run"
         ],
-        "test:unit": "pest --ci --coverage --min=91.7",
+        "test:unit": "pest --ci --coverage --min=92",
         "test:types": "phpstan",
         "test": [
             "@test:lint",

src/Enums/Role.php

@@ -6,6 +6,6 @@
 
 enum Role: string
 {
-    case ASSISTANT = 'assistant';
-    case USER = 'user';
+    case Assistant = 'assistant';
+    case User = 'user';
 }

src/Response.php

@@ -22,7 +22,7 @@ class Response
 
     protected function __construct(
         protected Content $content,
-        protected Role $role = Role::USER,
+        protected Role $role = Role::User,
         protected bool $isError = false,
     ) {
         //
@@ -127,7 +127,7 @@ public static function image(): Content
 
     public function asAssistant(): static
     {
-        return new static($this->content, Role::ASSISTANT, $this->isError);
+        return new static($this->content, Role::Assistant, $this->isError);
     }
 
     public function isNotification(): bool

tests/Unit/Methods/ListResourcesTest.php

@@ -143,7 +143,7 @@ public function shouldRegister(Request $request): bool
 it('includes annotations when a resource has annotations', function (): void {
     $listResources = new ListResources;
 
-    $resource = new #[Audience([Role::USER, Role::ASSISTANT])]
+    $resource = new #[Audience([Role::User, Role::Assistant])]
     #[Priority(0.8)]
     #[LastModified('2025-01-12T15:00:58Z')]
     class extends Resource
@@ -205,7 +205,7 @@ public function handle(): string
 it('handles mixed resources with and without annotations', function (): void {
     $listResources = new ListResources;
 
-    $annotatedResource = new #[Audience(Role::USER)]
+    $annotatedResource = new #[Audience(Role::User)]
     #[Priority(0.5)]
     class extends Resource
     {

tests/Unit/Resources/ResourceAnnotationsTest.php

@@ -27,7 +27,7 @@ public function handle(): Response
 });
 
 it('includes audience annotation in resource', function (): void {
-    $resource = new #[Audience([Role::USER, Role::ASSISTANT])] class extends Resource
+    $resource = new #[Audience([Role::User, Role::Assistant])] class extends Resource
     {
         public function description(): string
         {
@@ -84,7 +84,7 @@ public function handle(): Response
 });
 
 it('includes multiple annotations in resource', function (): void {
-    $resource = new #[Audience(Role::USER)] #[Priority(1.0)] #[LastModified('2025-01-12T15:00:58Z')] class extends Resource
+    $resource = new #[Audience(Role::User)] #[Priority(1.0)] #[LastModified('2025-01-12T15:00:58Z')] class extends Resource
     {
         public function description(): string
         {
@@ -107,7 +107,7 @@ public function handle(): Response
 });
 
 it('includes annotations in resource toArray', function (): void {
-    $resource = new #[Audience(Role::USER)] #[Priority(0.5)] class extends Resource
+    $resource = new #[Audience(Role::User)] #[Priority(0.5)] class extends Resource
     {
         public function description(): string
         {
@@ -273,7 +273,7 @@ public function handle(): Response
 });
 
 it('accepts only user role', function (): void {
-    $resource = new #[Audience(Role::USER)] class extends Resource
+    $resource = new #[Audience(Role::User)] class extends Resource
     {
         public function description(): string
         {
@@ -290,7 +290,7 @@ public function handle(): Response
 });
 
 it('accepts only an assistant role', function (): void {
-    $resource = new #[Audience(Role::ASSISTANT)] class extends Resource
+    $resource = new #[Audience(Role::Assistant)] class extends Resource
     {
         public function description(): string
         {

tests/Unit/ResponseTest.php

@@ -13,37 +13,37 @@
 it('creates a notification response', function (): void {
     $response = Response::notification('test.method', ['key' => 'value']);
 
-    expect($response->content())->toBeInstanceOf(Notification::class);
-    expect($response->isNotification())->toBeTrue();
-    expect($response->isError())->toBeFalse();
-    expect($response->role())->toBe(Role::USER);
+    expect($response->content())->toBeInstanceOf(Notification::class)
+        ->and($response->isNotification())->toBeTrue()
+        ->and($response->isError())->toBeFalse()
+        ->and($response->role())->toBe(Role::User);
 });
 
 it('creates a text response', function (): void {
     $response = Response::text('Hello world');
 
-    expect($response->content())->toBeInstanceOf(Text::class);
-    expect($response->isNotification())->toBeFalse();
-    expect($response->isError())->toBeFalse();
-    expect($response->role())->toBe(Role::USER);
+    expect($response->content())->toBeInstanceOf(Text::class)
+        ->and($response->isNotification())->toBeFalse()
+        ->and($response->isError())->toBeFalse()
+        ->and($response->role())->toBe(Role::User);
 });
 
 it('creates a blob response', function (): void {
     $response = Response::blob('binary content');
 
-    expect($response->content())->toBeInstanceOf(Blob::class);
-    expect($response->isNotification())->toBeFalse();
-    expect($response->isError())->toBeFalse();
-    expect($response->role())->toBe(Role::USER);
+    expect($response->content())->toBeInstanceOf(Blob::class)
+        ->and($response->isNotification())->toBeFalse()
+        ->and($response->isError())->toBeFalse()
+        ->and($response->role())->toBe(Role::User);
 });
 
 it('creates an error response', function (): void {
     $response = Response::error('Something went wrong');
 
-    expect($response->content())->toBeInstanceOf(Text::class);
-    expect($response->isNotification())->toBeFalse();
-    expect($response->isError())->toBeTrue();
-    expect($response->role())->toBe(Role::USER);
+    expect($response->content())->toBeInstanceOf(Text::class)
+        ->and($response->isNotification())->toBeFalse()
+        ->and($response->isError())->toBeTrue()
+        ->and($response->role())->toBe(Role::User);
 });
 
 it('throws exception for audio method', function (): void {
@@ -62,28 +62,28 @@
     $response = Response::text('Original message');
     $assistantResponse = $response->asAssistant();
 
-    expect($assistantResponse->content())->toBeInstanceOf(Text::class);
-    expect($assistantResponse->role())->toBe(Role::ASSISTANT);
-    expect($assistantResponse->isError())->toBeFalse();
+    expect($assistantResponse->content())->toBeInstanceOf(Text::class)
+        ->and($assistantResponse->role())->toBe(Role::Assistant)
+        ->and($assistantResponse->isError())->toBeFalse();
 });
 
 it('preserves error state when converting to assistant role', function (): void {
     $response = Response::error('Error message');
     $assistantResponse = $response->asAssistant();
 
-    expect($assistantResponse->content())->toBeInstanceOf(Text::class);
-    expect($assistantResponse->role())->toBe(Role::ASSISTANT);
-    expect($assistantResponse->isError())->toBeTrue();
+    expect($assistantResponse->content())->toBeInstanceOf(Text::class)
+        ->and($assistantResponse->role())->toBe(Role::Assistant)
+        ->and($assistantResponse->isError())->toBeTrue();
 });
 
 it('creates a json response', function (): void {
     $data = ['key' => 'value', 'number' => 123];
     $response = Response::json($data);
 
-    expect($response->content())->toBeInstanceOf(Text::class);
-    expect($response->isNotification())->toBeFalse();
-    expect($response->isError())->toBeFalse();
-    expect($response->role())->toBe(Role::USER);
+    expect($response->content())->toBeInstanceOf(Text::class)
+        ->and($response->isNotification())->toBeFalse()
+        ->and($response->isError())->toBeFalse()
+        ->and($response->role())->toBe(Role::User);
 
     $content = $response->content();
     expect((string) $content)->toBe(json_encode($data, JSON_THROW_ON_ERROR | JSON_PRETTY_PRINT));

tests/Unit/Tools/ToolAnnotationsTest.php

@@ -28,7 +28,7 @@ public function handle(Request $request): Response
 
 it('rejects resource annotations on tools', function (): void {
     expect(function (): void {
-        $tool = new #[Audience(Role::ASSISTANT)] class extends Tool
+        $tool = new #[Audience(Role::Assistant)] class extends Tool
         {
             public function handle(Request $request): Response
             {