[PR #693] [MERGED] Recover from fatal Ollama crash from LangChain library #3459

Closed
opened 2026-02-22 18:33:49 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Mintplex-Labs/anything-llm/pull/693
Author: @timothycarambat
Created: 2/7/2024
Status: Merged
Merged: 2/7/2024
Merged by: @timothycarambat

Base: masterHead: 670-prevent-ollama-fatal-crash


📝 Commits (1)

  • a2a073e Resolve fatal crash from Ollama failure

📊 Changes

1 file changed (+51 additions, -26 deletions)

View changed files

📝 server/utils/AiProviders/ollama/index.js (+51 -26)

📄 Description

Pull Request Type

  • feat
  • 🐛 fix
  • ♻️ refactor
  • 💄 style
  • 🔨 chore
  • 📝 docs

Relevant Issues

resolves #670

What is in this change?

Appropriately handle the LangChain streaming error from the LMChain failing for any reason. Report this error back to the user frontend for chat. Do not crash fatally - disrupting all other client chat streams that may be pending.

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

🔄 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/Mintplex-Labs/anything-llm/pull/693 **Author:** [@timothycarambat](https://github.com/timothycarambat) **Created:** 2/7/2024 **Status:** ✅ Merged **Merged:** 2/7/2024 **Merged by:** [@timothycarambat](https://github.com/timothycarambat) **Base:** `master` ← **Head:** `670-prevent-ollama-fatal-crash` --- ### 📝 Commits (1) - [`a2a073e`](https://github.com/Mintplex-Labs/anything-llm/commit/a2a073efe1f60193174ac9bc62fb228e6e50bdd8) Resolve fatal crash from Ollama failure ### 📊 Changes **1 file changed** (+51 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `server/utils/AiProviders/ollama/index.js` (+51 -26) </details> ### 📄 Description ### Pull Request Type <!-- For change type, change [ ] to [x]. --> - [ ] ✨ feat - [x] 🐛 fix - [ ] ♻️ refactor - [ ] 💄 style - [ ] 🔨 chore - [ ] 📝 docs ### Relevant Issues <!-- Use "resolves #xxx" to auto resolve on merge. Otherwise, please use "connect #xxx" --> resolves #670 ### What is in this change? Appropriately handle the LangChain streaming error from the LMChain failing for any reason. Report this error back to the user frontend for chat. Do not crash fatally - disrupting all other client chat streams that may be pending. ### Developer Validations <!-- All of the applicable items should be checked. --> - [x] I ran `yarn lint` from the root of the repo & committed changes - [x] Relevant documentation has been updated - [x] I have tested my code functionality - [x] Docker build succeeds locally --- <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-22 18:33:49 -05:00
yindo closed this issue 2026-02-22 18:33:50 -05:00
yindo changed title from [PR #693] Recover from fatal Ollama crash from LangChain library to [PR #693] [MERGED] Recover from fatal Ollama crash from LangChain library 2026-06-05 15:13:31 -04:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Mintplex-Labs/anything-llm#3459