Skip to content

Commit b48904e

Browse files
committed
fix: make launcher_type optional in launcher_create for backwards compatibility
1 parent 7040c5a commit b48904e

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

bases/renku_data_services/mcp_api/server.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -532,8 +532,8 @@ async def launcher_create(
532532
resource_class_id: Annotated[int, Field(description="Resource class ID (from resource_classes())")],
533533
environment: Annotated[dict[str, Any], Field(description="Environment definition dict")],
534534
launcher_type: Annotated[
535-
str, Field(description="'interactive' for sessions, 'non_interactive' for jobs")
536-
] = "interactive",
535+
str | None, Field(description="'interactive' for sessions, 'non_interactive' for jobs — omit for older API versions")
536+
] = None,
537537
description: Annotated[str, Field(description="Optional description")] = "",
538538
) -> dict[str, Any]:
539539
"""Create a session launcher. Always call resource_classes(cpu=..., memory=...) first.
@@ -560,8 +560,9 @@ async def launcher_create(
560560
"name": name,
561561
"resource_class_id": resource_class_id,
562562
"environment": environment,
563-
"launcher_type": launcher_type,
564563
}
564+
if launcher_type is not None:
565+
body["launcher_type"] = launcher_type
565566
if description:
566567
body["description"] = description
567568
return _launcher_summary(await _api(ctx, "POST", "/session_launchers", body))

0 commit comments

Comments
 (0)