diff --git a/app/Mcp/Servers/EinundzwanzigServer.php b/app/Mcp/Servers/EinundzwanzigServer.php index f1cf08c..2e114c1 100644 --- a/app/Mcp/Servers/EinundzwanzigServer.php +++ b/app/Mcp/Servers/EinundzwanzigServer.php @@ -27,6 +27,7 @@ use App\Mcp\Tools\Search\ListCountriesTool; use App\Mcp\Tools\Search\SearchCitiesTool; use App\Mcp\Tools\Search\SearchCoursesTool; use App\Mcp\Tools\Search\SearchLecturersTool; +use App\Mcp\Tools\Search\SearchMeetupsTool; use App\Mcp\Tools\Search\SearchVenuesTool; use App\Mcp\Tools\Venue\CreateVenueTool; use App\Mcp\Tools\Venue\ListMyVenuesTool; @@ -57,6 +58,12 @@ Entitäten immer über ihren NAMEN: übergeben (Parameter z. B. "city", "country", "lecturer", "course", "venue"); bei Unsicherheit vorher mit search-cities / search-venues / search-lecturers / search-courses / list-countries den genauen Namen ermitteln. +Bevor ein NEUES Meetup angelegt wird (create-meetup): IMMER zuerst mit search-meetups nach +einem bestehenden Meetup suchen – sowohl nach dem Namen als auch nach dem Stadtnamen. Existiert +bereits ein passendes Meetup, KEIN Duplikat anlegen, sondern dem Nutzer das gefundene Meetup +nennen. Nur wenn die Suche kein passendes Meetup liefert, den Nutzer fragen, ob ein neues +Meetup erstellt werden soll – und erst nach Bestätigung create-meetup aufrufen. + Termine/Events (Meetup-Termine, Kurs-Events) haben keinen Namen. Hier zuerst list-my-meetup- events bzw. list-my-course-events aufrufen, dem Nutzer die Einträge zur Auswahl anbieten und die ID des gewählten Eintrags übergeben – ebenfalls ohne den Nutzer nach der ID zu fragen. @@ -123,6 +130,7 @@ class EinundzwanzigServer extends Server UpdateCourseEventTool::class, // Suche / Stammdaten-Lookups + SearchMeetupsTool::class, SearchCitiesTool::class, SearchVenuesTool::class, SearchLecturersTool::class, diff --git a/app/Mcp/Tools/Meetup/CreateMeetupTool.php b/app/Mcp/Tools/Meetup/CreateMeetupTool.php index 4994eeb..92af741 100644 --- a/app/Mcp/Tools/Meetup/CreateMeetupTool.php +++ b/app/Mcp/Tools/Meetup/CreateMeetupTool.php @@ -15,7 +15,7 @@ use Laravel\Mcp\Response; use Laravel\Mcp\Server\Attributes\Description; use Laravel\Mcp\Server\Tool; -#[Description('Legt ein neues Meetup für den authentifizierten Nutzer an. Die Stadt wird über ihren Namen angegeben; der Ersteller (created_by) wird automatisch gesetzt.')] +#[Description('Legt ein neues Meetup für den authentifizierten Nutzer an. Die Stadt wird über ihren Namen angegeben; der Ersteller (created_by) wird automatisch gesetzt. WICHTIG: Vorher mit search-meetups (Name UND Stadtname) prüfen, ob bereits ein Meetup existiert, und erst nach Rückfrage beim Nutzer anlegen, wenn keines gefunden wurde.')] class CreateMeetupTool extends Tool { use ResolvesEntities; diff --git a/app/Mcp/Tools/Search/SearchMeetupsTool.php b/app/Mcp/Tools/Search/SearchMeetupsTool.php new file mode 100644 index 0000000..20a5267 --- /dev/null +++ b/app/Mcp/Tools/Search/SearchMeetupsTool.php @@ -0,0 +1,59 @@ +get('search'); + + $meetups = Meetup::query() + ->select('id', 'name', 'city_id', 'slug') + ->with(['city:id,name,country_id', 'city.country:id,name']) + ->orderBy('name') + ->when( + is_string($search) && $search !== '', + function (Builder $query) use ($search): void { + $needle = '%'.mb_strtolower(trim((string) $search)).'%'; + + $query->where(function (Builder $inner) use ($needle): void { + $inner->whereRaw('LOWER(name) LIKE ?', [$needle]) + ->orWhereHas('city', fn (Builder $city) => $city->whereRaw('LOWER(cities.name) LIKE ?', [$needle])); + }); + } + ) + ->limit(10) + ->get() + ->map(fn (Meetup $meetup): array => [ + 'id' => $meetup->id, + 'name' => $meetup->name, + 'city' => $meetup->city?->name, + 'country' => $meetup->city?->country?->name, + ]); + + return Response::json($meetups->values()); + } + + /** + * @return array + */ + public function schema(JsonSchema $schema): array + { + return [ + 'search' => $schema->string()->description('Suchbegriff: Meetup-Name oder Stadtname (z. B. "München").'), + ]; + } +} diff --git a/tests/Feature/Mcp/EinundzwanzigServerTest.php b/tests/Feature/Mcp/EinundzwanzigServerTest.php index 4dde26d..7137107 100644 --- a/tests/Feature/Mcp/EinundzwanzigServerTest.php +++ b/tests/Feature/Mcp/EinundzwanzigServerTest.php @@ -17,7 +17,7 @@ it('registers every domain tool on the server', function () { $property = (new ReflectionClass(EinundzwanzigServer::class))->getProperty('tools'); $tools = $property->getDefaultValue(); - expect($tools)->toHaveCount(30) + expect($tools)->toHaveCount(31) ->and($tools)->toContain(CreateMeetupTool::class) ->and($tools)->toContain(UpdateCourseEventTool::class) ->and($tools)->toContain(SearchCitiesTool::class); diff --git a/tests/Feature/Mcp/SearchMeetupsMcpTest.php b/tests/Feature/Mcp/SearchMeetupsMcpTest.php new file mode 100644 index 0000000..cc1cb14 --- /dev/null +++ b/tests/Feature/Mcp/SearchMeetupsMcpTest.php @@ -0,0 +1,35 @@ +create(['name' => 'Einundzwanzig München']); + + EinundzwanzigServer::actingAs(User::factory()->create()) + ->tool(SearchMeetupsTool::class, ['search' => 'münchen']) + ->assertOk() + ->assertSee('Einundzwanzig München'); +}); + +it('finds an existing meetup by its city name', function () { + $city = City::factory()->create(['name' => 'Nürnberg']); + Meetup::factory()->create(['name' => 'Bitcoin Treff', 'city_id' => $city->id]); + + EinundzwanzigServer::actingAs(User::factory()->create()) + ->tool(SearchMeetupsTool::class, ['search' => 'Nürnberg']) + ->assertOk() + ->assertSee('Bitcoin Treff'); +}); + +it('returns no match for an unknown city so a new meetup can be proposed', function () { + Meetup::factory()->create(['name' => 'Einundzwanzig München']); + + $response = EinundzwanzigServer::actingAs(User::factory()->create()) + ->tool(SearchMeetupsTool::class, ['search' => 'Gibtsnichtstadt']); + + $response->assertOk()->assertDontSee('Einundzwanzig München'); +});