Skip to content

Commit c911db9

Browse files
committed
fix append
1 parent d95f26a commit c911db9

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/openai_messages_token_helper/message_builder.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -44,15 +44,15 @@ def __init__(self, system_content: str):
4444
self.messages: list[ChatCompletionMessageParam] = []
4545

4646
@property
47-
def all_messages(self):
47+
def all_messages(self) -> list[ChatCompletionMessageParam]:
4848
return [self.system_message] + self.messages
4949

5050
def insert_message(
51-
self, role: ChatCompletionRole, content: Union[str, Iterable[ChatCompletionContentPartParam]], index: int = 1
51+
self, role: ChatCompletionRole, content: Union[str, Iterable[ChatCompletionContentPartParam]], index: int = 0
5252
):
5353
"""
5454
Inserts a message into the conversation at the specified index,
55-
or at index 1 (after system message) if no index is specified.
55+
or at index 0 if no index is specified.
5656
Args:
5757
role (str): The role of the message sender (either "user", "system", or "assistant").
5858
content (str | List[ChatCompletionContentPartParam]): The content of the message.
@@ -106,7 +106,7 @@ def build_messages(
106106
raise ValueError("Few-shot messages must have both role and content")
107107
message_builder.insert_message(shot["role"], shot["content"])
108108

109-
append_index = len(few_shots) + 1
109+
append_index = len(few_shots)
110110

111111
if new_user_content:
112112
message_builder.insert_message("user", new_user_content, index=append_index)

0 commit comments

Comments
 (0)