Skip to content

Commit fecd95f

Browse files
refactor: rename the transports for consistency
1 parent 1171aca commit fecd95f

File tree

4 files changed

+10
-8
lines changed

4 files changed

+10
-8
lines changed

src/Http/Controllers/SseTransportController.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,14 @@
55
namespace PhpMcp\Laravel\Http\Controllers;
66

77
use Illuminate\Http\Request;
8-
use PhpMcp\Laravel\Transports\LaravelHttpTransport;
8+
use PhpMcp\Laravel\Transports\HttpServerTransport;
99
use PhpMcp\Server\Server;
1010
use Symfony\Component\HttpFoundation\Response;
1111
use Symfony\Component\HttpFoundation\StreamedResponse;
1212

1313
class SseTransportController
1414
{
15-
protected LaravelHttpTransport $transport;
15+
protected HttpServerTransport $transport;
1616

1717
/**
1818
* MCP Controller Constructor
@@ -21,7 +21,7 @@ class SseTransportController
2121
*/
2222
public function __construct(Server $server)
2323
{
24-
$this->transport = new LaravelHttpTransport($server->getSessionManager());
24+
$this->transport = new HttpServerTransport($server->getSessionManager());
2525
$server->listen($this->transport, false);
2626
}
2727

src/Http/Controllers/StreamableTransportController.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,22 +5,22 @@
55
namespace PhpMcp\Laravel\Http\Controllers;
66

77
use Illuminate\Http\Request;
8-
use PhpMcp\Laravel\Transports\LaravelStreamableHttpTransport;
8+
use PhpMcp\Laravel\Transports\StreamableHttpServerTransport;
99
use PhpMcp\Server\Contracts\EventStoreInterface;
1010
use PhpMcp\Server\Server;
1111
use Symfony\Component\HttpFoundation\Response;
1212
use Symfony\Component\HttpFoundation\StreamedResponse;
1313

1414
class StreamableTransportController
1515
{
16-
private LaravelStreamableHttpTransport $transport;
16+
private StreamableHttpServerTransport $transport;
1717

1818
public function __construct(Server $server)
1919
{
2020
$eventStore = $this->createEventStore();
2121
$sessionManager = $server->getSessionManager();
2222

23-
$this->transport = new LaravelStreamableHttpTransport($sessionManager, $eventStore);
23+
$this->transport = new StreamableHttpServerTransport($sessionManager, $eventStore);
2424
$server->listen($this->transport, false);
2525
}
2626

src/Transports/LaravelHttpTransport.php renamed to src/Transports/HttpServerTransport.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919

2020
use function React\Promise\resolve;
2121

22-
class LaravelHttpTransport implements ServerTransportInterface
22+
class HttpServerTransport implements ServerTransportInterface
2323
{
2424
use EventEmitterTrait;
2525

src/Transports/LaravelStreamableHttpTransport.php renamed to src/Transports/StreamableHttpServerTransport.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323

2424
use function React\Promise\resolve;
2525

26-
class LaravelStreamableHttpTransport implements ServerTransportInterface
26+
class StreamableHttpServerTransport implements ServerTransportInterface
2727
{
2828
use EventEmitterTrait;
2929

@@ -73,6 +73,8 @@ public function sendMessage(Message $message, string $sessionId, array $context
7373
*/
7474
public function handlePostRequest(Request $request): Response
7575
{
76+
$this->collectSessionGarbage();
77+
7678
$acceptHeader = $request->header('Accept', '');
7779
if (!str_contains($acceptHeader, 'application/json') && !str_contains($acceptHeader, 'text/event-stream')) {
7880
$error = Error::forInvalidRequest('Not Acceptable: Client must accept application/json or text/event-stream');

0 commit comments

Comments
 (0)