Skip to content

Commit 9b13b6e

Browse files
committed
update library and tests to use snake_case
1 parent 34b708a commit 9b13b6e

File tree

12 files changed

+296
-238
lines changed

12 files changed

+296
-238
lines changed

src/a2a/client/helpers.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,5 +18,7 @@ def create_text_message_object(
1818
A `Message` object with a new UUID messageId.
1919
"""
2020
return Message(
21-
role=role, parts=[Part(TextPart(text=content))], messageId=str(uuid4())
21+
role=role,
22+
parts=[Part(root=TextPart(text=content))],
23+
message_id=str(uuid4()),
2224
)

src/a2a/server/tasks/task_manager.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ async def save_task_event(
9696
when the TaskManager's ID is already set.
9797
"""
9898
task_id_from_event = (
99-
event.id if isinstance(event, Task) else event.taskId
99+
event.id if isinstance(event, Task) else event.task_id
100100
)
101101
# If task id is known, make sure it is matched
102102
if self.task_id and self.task_id != task_id_from_event:
@@ -107,8 +107,8 @@ async def save_task_event(
107107
)
108108
if not self.task_id:
109109
self.task_id = task_id_from_event
110-
if not self.context_id and self.context_id != event.contextId:
111-
self.context_id = event.contextId
110+
if not self.context_id and self.context_id != event.context_id:
111+
self.context_id = event.context_id
112112

113113
logger.debug(
114114
'Processing save of task event of type %s for task_id: %s',
@@ -160,12 +160,12 @@ async def ensure_task(
160160
if not task:
161161
logger.info(
162162
'Task not found or task_id not set. Creating new task for event (task_id: %s, context_id: %s).',
163-
event.taskId,
164-
event.contextId,
163+
event.task_id,
164+
event.context_id,
165165
)
166166
# streaming agent did not previously stream task object.
167167
# Create a task object with the available information and persist the event
168-
task = self._init_task_obj(event.taskId, event.contextId)
168+
task = self._init_task_obj(event.task_id, event.context_id)
169169
await self._save_task(task)
170170

171171
return task
@@ -207,7 +207,7 @@ def _init_task_obj(self, task_id: str, context_id: str) -> Task:
207207
history = [self._initial_message] if self._initial_message else []
208208
return Task(
209209
id=task_id,
210-
contextId=context_id,
210+
context_id=context_id,
211211
status=TaskStatus(state=TaskState.submitted),
212212
history=history,
213213
)
@@ -224,7 +224,7 @@ async def _save_task(self, task: Task) -> None:
224224
if not self.task_id:
225225
logger.info('New task created with id: %s', task.id)
226226
self.task_id = task.id
227-
self.context_id = task.contextId
227+
self.context_id = task.context_id
228228

229229
def update_with_message(self, message: Message, task: Task) -> Task:
230230
"""Updates a task object in memory by adding a new message to its history.

src/a2a/utils/artifact.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ def new_artifact(
2121
A new `Artifact` object with a generated artifactId.
2222
"""
2323
return Artifact(
24-
artifactId=str(uuid.uuid4()),
24+
artifact_id=str(uuid.uuid4()),
2525
parts=parts,
2626
name=name,
2727
description=description,

src/a2a/utils/helpers.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -36,12 +36,12 @@ def create_task_obj(message_send_params: MessageSendParams) -> Task:
3636
Returns:
3737
A new `Task` object initialized with 'submitted' status and the input message in history.
3838
"""
39-
if not message_send_params.message.contextId:
40-
message_send_params.message.contextId = str(uuid4())
39+
if not message_send_params.message.context_id:
40+
message_send_params.message.context_id = str(uuid4())
4141

4242
return Task(
4343
id=str(uuid4()),
44-
contextId=message_send_params.message.contextId,
44+
context_id=message_send_params.message.context_id,
4545
status=TaskStatus(state=TaskState.submitted),
4646
history=[message_send_params.message],
4747
)
@@ -62,15 +62,15 @@ def append_artifact_to_task(task: Task, event: TaskArtifactUpdateEvent) -> None:
6262
task.artifacts = []
6363

6464
new_artifact_data: Artifact = event.artifact
65-
artifact_id: str = new_artifact_data.artifactId
65+
artifact_id: str = new_artifact_data.artifact_id
6666
append_parts: bool = event.append or False
6767

6868
existing_artifact: Artifact | None = None
6969
existing_artifact_list_index: int | None = None
7070

7171
# Find existing artifact by its id
7272
for i, art in enumerate(task.artifacts):
73-
if art.artifactId == artifact_id:
73+
if art.artifact_id == artifact_id:
7474
existing_artifact = art
7575
existing_artifact_list_index = i
7676
break
@@ -115,7 +115,7 @@ def build_text_artifact(text: str, artifact_id: str) -> Artifact:
115115
"""
116116
text_part = TextPart(text=text)
117117
part = Part(root=text_part)
118-
return Artifact(parts=[part], artifactId=artifact_id)
118+
return Artifact(parts=[part], artifact_id=artifact_id)
119119

120120

121121
def validate(

src/a2a/utils/message.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,9 @@ def new_agent_text_message(
2828
return Message(
2929
role=Role.agent,
3030
parts=[Part(root=TextPart(text=text))],
31-
messageId=str(uuid.uuid4()),
32-
taskId=task_id,
33-
contextId=context_id,
31+
message_id=str(uuid.uuid4()),
32+
task_id=task_id,
33+
context_id=context_id,
3434
)
3535

3636

@@ -52,9 +52,9 @@ def new_agent_parts_message(
5252
return Message(
5353
role=Role.agent,
5454
parts=parts,
55-
messageId=str(uuid.uuid4()),
56-
taskId=task_id,
57-
contextId=context_id,
55+
message_id=str(uuid.uuid4()),
56+
task_id=task_id,
57+
context_id=context_id,
5858
)
5959

6060

src/a2a/utils/task.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,9 @@ def new_task(request: Message) -> Task:
1818
"""
1919
return Task(
2020
status=TaskStatus(state=TaskState.submitted),
21-
id=(request.taskId if request.taskId else str(uuid.uuid4())),
22-
contextId=(
23-
request.contextId if request.contextId else str(uuid.uuid4())
21+
id=(request.task_id if request.task_id else str(uuid.uuid4())),
22+
context_id=(
23+
request.context_id if request.context_id else str(uuid.uuid4())
2424
),
2525
history=[request],
2626
)
@@ -51,7 +51,7 @@ def completed_task(
5151
return Task(
5252
status=TaskStatus(state=TaskState.completed),
5353
id=task_id,
54-
contextId=context_id,
54+
context_id=context_id,
5555
artifacts=artifacts,
5656
history=history,
5757
)

0 commit comments

Comments
 (0)