Skip to content

fix: retry causes repeated messages, update add citation prompt#13

Merged
aydin-utting merged 3 commits into
mainfrom
fix/too-many-citations
Oct 15, 2025
Merged

fix: retry causes repeated messages, update add citation prompt#13
aydin-utting merged 3 commits into
mainfrom
fix/too-many-citations

Conversation

@aydin-utting

@aydin-utting aydin-utting commented Oct 14, 2025

Copy link
Copy Markdown
Collaborator

LLM adapters stored list of messages in the conversation. If the chat or structured_chat function failed and retried then you got repeated messaged added to the chatbot which were all sent on each retry.

@aydin-utting aydin-utting requested a review from a team as a code owner October 14, 2025 08:39
@aydin-utting aydin-utting force-pushed the fix/too-many-citations branch 2 times, most recently from 5b128c3 to 38ec96e Compare October 14, 2025 09:49
@aydin-utting aydin-utting changed the title fix: retry causes repeated messages fix: retry causes repeated messages, update add citation prompt Oct 14, 2025
Comment thread common/templates/add_citations.py Outdated
matches = cluster_pattern.finditer(minute)
for match in matches:
citation_cluster = match.group()
citation_pattern = re.compile(r"\d+")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can move to module?

@aydin-utting aydin-utting force-pushed the fix/too-many-citations branch from ff8e7bc to d24cb8e Compare October 15, 2025 12:14
@aydin-utting aydin-utting merged commit d785cfe into main Oct 15, 2025
17 of 18 checks passed
@aydin-utting aydin-utting deleted the fix/too-many-citations branch October 15, 2025 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants