Fimeg/letta-server
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

feat: add path parameter validation for group_id (#5521)

Browse Source 
🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Claude <noreply@anthropic.com>
This commit is contained in:
Kian Jones
2025-10-17 16:39:08 -07:00
committed by Caren Thomas
parent 8a35b886ea
commit 3b56c53b6c
2 changed files with 64 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

64
fern/openapi.json
View File

@@ -7314,8 +7314,14 @@
"required": true,
"schema": {
"type": "string",
"minLength": 42,
"maxLength": 42,
"pattern": "^group-[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$",
"description": "The ID of the group in the format 'group-<uuid4>'",
"examples": ["group-123e4567-e89b-42d3-8456-426614174000"],
"title": "Group Id"
}
},
"description": "The ID of the group in the format 'group-<uuid4>'"
}
],
"responses": {
@@ -7353,8 +7359,14 @@
"required": true,
"schema": {
"type": "string",
"minLength": 42,
"maxLength": 42,
"pattern": "^group-[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$",
"description": "The ID of the group in the format 'group-<uuid4>'",
"examples": ["group-123e4567-e89b-42d3-8456-426614174000"],
"title": "Group Id"
}
},
"description": "The ID of the group in the format 'group-<uuid4>'"
},
{
"name": "X-Project",
@@ -7420,8 +7432,14 @@
"required": true,
"schema": {
"type": "string",
"minLength": 42,
"maxLength": 42,
"pattern": "^group-[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$",
"description": "The ID of the group in the format 'group-<uuid4>'",
"examples": ["group-123e4567-e89b-42d3-8456-426614174000"],
"title": "Group Id"
}
},
"description": "The ID of the group in the format 'group-<uuid4>'"
}
],
"responses": {
@@ -7459,8 +7477,14 @@
"required": true,
"schema": {
"type": "string",
"minLength": 42,
"maxLength": 42,
"pattern": "^group-[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$",
"description": "The ID of the group in the format 'group-<uuid4>'",
"examples": ["group-123e4567-e89b-42d3-8456-426614174000"],
"title": "Group Id"
}
},
"description": "The ID of the group in the format 'group-<uuid4>'"
}
],
"requestBody": {
@@ -7508,8 +7532,14 @@
"required": true,
"schema": {
"type": "string",
"minLength": 42,
"maxLength": 42,
"pattern": "^group-[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$",
"description": "The ID of the group in the format 'group-<uuid4>'",
"examples": ["group-123e4567-e89b-42d3-8456-426614174000"],
"title": "Group Id"
}
},
"description": "The ID of the group in the format 'group-<uuid4>'"
},
{
"name": "before",
@@ -7670,8 +7700,14 @@
"required": true,
"schema": {
"type": "string",
"minLength": 42,
"maxLength": 42,
"pattern": "^group-[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$",
"description": "The ID of the group in the format 'group-<uuid4>'",
"examples": ["group-123e4567-e89b-42d3-8456-426614174000"],
"title": "Group Id"
}
},
"description": "The ID of the group in the format 'group-<uuid4>'"
}
],
"requestBody": {
@@ -7722,8 +7758,14 @@
"required": true,
"schema": {
"type": "string",
"minLength": 42,
"maxLength": 42,
"pattern": "^group-[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$",
"description": "The ID of the group in the format 'group-<uuid4>'",
"examples": ["group-123e4567-e89b-42d3-8456-426614174000"],
"title": "Group Id"
}
},
"description": "The ID of the group in the format 'group-<uuid4>'"
},
{
"name": "message_id",
@@ -7845,8 +7887,14 @@
"required": true,
"schema": {
"type": "string",
"minLength": 42,
"maxLength": 42,
"pattern": "^group-[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$",
"description": "The ID of the group in the format 'group-<uuid4>'",
"examples": ["group-123e4567-e89b-42d3-8456-426614174000"],
"title": "Group Id"
}
},
"description": "The ID of the group in the format 'group-<uuid4>'"
}
],
"responses": {

16
letta/server/rest_api/routers/v1/groups.py
View File

@@ -69,7 +69,7 @@ async def count_groups(
@router.get("/{group_id}", response_model=Group, operation_id="retrieve_group")
async def retrieve_group(
group_id: str,
group_id: str = PATH_VALIDATORS["group"],
server: "SyncServer" = Depends(get_letta_server),
headers: HeaderParams = Depends(get_headers),
):
@@ -98,7 +98,7 @@ async def create_group(
@router.patch("/{group_id}", response_model=Group, operation_id="modify_group")
async def modify_group(
group_id: str,
group_id: str = PATH_VALIDATORS["group"],
group: GroupUpdate = Body(...),
server: "SyncServer" = Depends(get_letta_server),
headers: HeaderParams = Depends(get_headers),
@@ -115,7 +115,7 @@ async def modify_group(
@router.delete("/{group_id}", response_model=None, operation_id="delete_group")
async def delete_group(
group_id: str,
group_id: str = PATH_VALIDATORS["group"],
server: "SyncServer" = Depends(get_letta_server),
headers: HeaderParams = Depends(get_headers),
):
@@ -133,7 +133,7 @@ async def delete_group(
operation_id="send_group_message",
)
async def send_group_message(
group_id: str,
group_id: str = PATH_VALIDATORS["group"],
server: SyncServer = Depends(get_letta_server),
request: LettaRequest = Body(...),
headers: HeaderParams = Depends(get_headers),
@@ -171,7 +171,7 @@ async def send_group_message(
},
)
async def send_group_message_streaming(
group_id: str,
group_id: str = PATH_VALIDATORS["group"],
server: SyncServer = Depends(get_letta_server),
request: LettaStreamingRequest = Body(...),
headers: HeaderParams = Depends(get_headers),
@@ -203,7 +203,7 @@ GroupMessagesResponse = Annotated[
@router.patch("/{group_id}/messages/{message_id}", response_model=LettaMessageUnion, operation_id="modify_group_message")
async def modify_group_message(
group_id: str,
group_id: str = PATH_VALIDATORS["group"],
message_id: str = PATH_VALIDATORS["message"],
request: LettaMessageUpdateUnion = Body(...),
server: "SyncServer" = Depends(get_letta_server),
@@ -219,7 +219,7 @@ async def modify_group_message(
@router.get("/{group_id}/messages", response_model=GroupMessagesResponse, operation_id="list_group_messages")
async def list_group_messages(
group_id: str,
group_id: str = PATH_VALIDATORS["group"],
before: Optional[str] = Query(
None,
description="Message ID cursor for pagination. Returns messages that come before this message ID in the specified sort order",
@@ -274,7 +274,7 @@ async def list_group_messages(
@router.patch("/{group_id}/reset-messages", response_model=None, operation_id="reset_group_messages")
async def reset_group_messages(
group_id: str,
group_id: str = PATH_VALIDATORS["group"],
server: "SyncServer" = Depends(get_letta_server),
headers: HeaderParams = Depends(get_headers),
):