Skip to content

🔧 fix: Import Path for Custom Configuration Loading - #8319

Merged
danny-avila merged 1 commit into
LibreChat-AI:devfrom
nhtruong:fix-uncached-config
Jul 9, 2025
Merged

danny-avila merged 1 commit into
LibreChat-AI:devfrom
nhtruong:fix-uncached-config

Conversation

@nhtruong

@nhtruong nhtruong commented Jul 8, 2025 •

Copy link
Copy Markdown
Contributor

Update import path for custom configuration loading to use the cache instead of reading and parsing the config file every time a message is saved.

Summary

Both Conversation.js and Message.js used loadCustomConfig function to retrieve configuration of the app. This resulted in the config file to be read and parsed every time a message is saved, which is not performant and it also caused MCP tool lists to change unpredictably.

This fix will make those files get the info from the cache instead.

Change Type

  • Bug fix (non-breaking change which fixes an issue)

Testing

Run npm run backend and interact with any LLM model, you will see the config being printed out every a message is sent. This print out is called from the loadCustomConfig function.

Checklist

Please delete any irrelevant options.

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code

… cache instead of reading and parsing the config file everytime a message is saved.
@danny-avila danny-avila changed the title 🔧 fix: Update import path for custom configuration loading … 🔧 fix: Import Path for Custom Configuration Loading Jul 9, 2025
@danny-avila
danny-avila changed the base branch from main to dev July 9, 2025 01:06
@danny-avila
danny-avila merged commit e57fc83 into LibreChat-AI:dev Jul 9, 2025
@nhtruong
nhtruong deleted the fix-uncached-config branch July 9, 2025 14:41
kenshinsamue pushed a commit to intelequia/LibreChat that referenced this pull request Aug 4, 2025
xycjscs pushed a commit to xycjscs/LibreChat that referenced this pull request Aug 9, 2025
patricksn3ll pushed a commit to patricksn3ll/LibreChat that referenced this pull request Dec 11, 2025
jcbartle pushed a commit to jcbartle/LibreChat that referenced this pull request May 11, 2026
ThomasVuNguyen pushed a commit to ThomasVuNguyen/LibreChat that referenced this pull request Jul 15, 2026
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