Skip to content

Conversation

@wipash
Copy link
Contributor

@wipash wipash commented Jan 23, 2026

Summary

The execute_code tool description in LibreChat doesn't specify that files must be written to /mnt/data. This PR makes that the default so the LC agent doesn't have to endure a write failure and try to work it out by itself.

Code executed via the API now runs with /mnt/data as the working directory, allowing relative file paths to work correctly.

Also, includes a change to loosen up the is_fork ci check, so that PRs within a forked repo can still trigger builds.

Type of change

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

How Has This Been Tested?

  • just test-k8s
  • deployed and tested in test cluster

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@wipash wipash requested a review from aron-muon as a code owner January 23, 2026 01:41
@aron-muon aron-muon merged commit e8ceb10 into aron-muon:main Jan 23, 2026
28 checks passed
@github-actions
Copy link

🎉 This PR is included in version 2.1.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants