[PR #432] [MERGED] fix: dont include new message in chat history #502

Closed
opened 2026-02-15 20:15:04 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/run-llama/create-llama/pull/432
Author: @thucpn
Created: 11/18/2024
Status: Merged
Merged: 11/18/2024
Merged by: @marcusschiesser

Base: mainHead: fix/dont-include-new-message-in-chat-history


📝 Commits (2)

  • 8ef0091 fix: dont include new message in chat history
  • c9c9f18 Create brave-pears-reflect.md

📊 Changes

3 files changed (+7 additions, -2 deletions)

View changed files

.changeset/brave-pears-reflect.md (+5 -0)
📝 templates/types/streaming/express/src/controllers/chat.controller.ts (+1 -1)
📝 templates/types/streaming/nextjs/app/api/chat/route.ts (+1 -1)

📄 Description

Summary by CodeRabbit

  • New Features

    • Enhanced chat functionality by refining the context of chat history, now excluding the last user message for improved processing.
  • Bug Fixes

    • Adjusted how chat history is populated to ensure more relevant context is passed to the chat engine.
    • Implemented a fix to prevent the inclusion of user messages in the chat history, enhancing privacy.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/run-llama/create-llama/pull/432 **Author:** [@thucpn](https://github.com/thucpn) **Created:** 11/18/2024 **Status:** ✅ Merged **Merged:** 11/18/2024 **Merged by:** [@marcusschiesser](https://github.com/marcusschiesser) **Base:** `main` ← **Head:** `fix/dont-include-new-message-in-chat-history` --- ### 📝 Commits (2) - [`8ef0091`](https://github.com/run-llama/create-llama/commit/8ef009168a9f9b46c20ecda286dc332d3a87017c) fix: dont include new message in chat history - [`c9c9f18`](https://github.com/run-llama/create-llama/commit/c9c9f18620f13d44625fd573899834277500174b) Create brave-pears-reflect.md ### 📊 Changes **3 files changed** (+7 additions, -2 deletions) <details> <summary>View changed files</summary> ➕ `.changeset/brave-pears-reflect.md` (+5 -0) 📝 `templates/types/streaming/express/src/controllers/chat.controller.ts` (+1 -1) 📝 `templates/types/streaming/nextjs/app/api/chat/route.ts` (+1 -1) </details> ### 📄 Description <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Enhanced chat functionality by refining the context of chat history, now excluding the last user message for improved processing. - **Bug Fixes** - Adjusted how chat history is populated to ensure more relevant context is passed to the chat engine. - Implemented a fix to prevent the inclusion of user messages in the chat history, enhancing privacy. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 20:15:04 -05:00
yindo closed this issue 2026-02-15 20:15:04 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/create-llama#502