[PR #4203] [CLOSED] Variable integrity in agent flow variable replacement #4551

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

📋 Pull Request Information

Original PR: https://github.com/Mintplex-Labs/anything-llm/pull/4203
Author: @shatfield4
Created: 7/25/2025
Status: Closed

Base: masterHead: 4187-bug-api-request-body-is-sent-blank-if-there-are-variables-within-request-body-json-also-some-suggestions


📝 Commits (5)

  • 955e24d add protections in flow executor for variable replacement + add tests
  • c66844c lint
  • 9663ee3 wip force aibitat to replace vars in between function calls
  • 4ae68fa Merge branch 'master' into 4187-bug-api-request-body-is-sent-blank-if-there-are-variables-within-request-body-json-also-some-suggestions
  • a5535fe Merge branch 'master' into 4187-bug-api-request-body-is-sent-blank-if-there-are-variables-within-request-body-json-also-some-suggestions

📊 Changes

4 files changed (+368 additions, -16 deletions)

View changed files

server/__tests__/utils/agentFlows/aibitat.spec.js (+61 -0)
server/__tests__/utils/agentFlows/executor.spec.js (+228 -0)
📝 server/utils/agentFlows/executor.js (+59 -16)
📝 server/utils/agents/aibitat/index.js (+20 -0)

📄 Description

Pull Request Type

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

Relevant Issues

resolves #xxx

What is in this change?

Additional Information

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/4203 **Author:** [@shatfield4](https://github.com/shatfield4) **Created:** 7/25/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `4187-bug-api-request-body-is-sent-blank-if-there-are-variables-within-request-body-json-also-some-suggestions` --- ### 📝 Commits (5) - [`955e24d`](https://github.com/Mintplex-Labs/anything-llm/commit/955e24d8cb3e1afa3f3f066e40ebe69c869410ab) add protections in flow executor for variable replacement + add tests - [`c66844c`](https://github.com/Mintplex-Labs/anything-llm/commit/c66844c38778d1b81f5233a2aab42eef6e6bd8fb) lint - [`9663ee3`](https://github.com/Mintplex-Labs/anything-llm/commit/9663ee3ded3747bb285e684633a2790d32fa87f0) wip force aibitat to replace vars in between function calls - [`4ae68fa`](https://github.com/Mintplex-Labs/anything-llm/commit/4ae68fa9c20c36c331bb4a532b54f967f9d01d25) Merge branch 'master' into 4187-bug-api-request-body-is-sent-blank-if-there-are-variables-within-request-body-json-also-some-suggestions - [`a5535fe`](https://github.com/Mintplex-Labs/anything-llm/commit/a5535fe47a63d4f112a252beb330daedb6baaed1) Merge branch 'master' into 4187-bug-api-request-body-is-sent-blank-if-there-are-variables-within-request-body-json-also-some-suggestions ### 📊 Changes **4 files changed** (+368 additions, -16 deletions) <details> <summary>View changed files</summary> ➕ `server/__tests__/utils/agentFlows/aibitat.spec.js` (+61 -0) ➕ `server/__tests__/utils/agentFlows/executor.spec.js` (+228 -0) 📝 `server/utils/agentFlows/executor.js` (+59 -16) 📝 `server/utils/agents/aibitat/index.js` (+20 -0) </details> ### 📄 Description ### Pull Request Type <!-- For change type, change [ ] to [x]. --> - [ ] ✨ feat - [ ] 🐛 fix - [ ] ♻️ refactor - [ ] 💄 style - [ ] 🔨 chore - [ ] 📝 docs ### Relevant Issues <!-- Use "resolves #xxx" to auto resolve on merge. Otherwise, please use "connect #xxx" --> resolves #xxx ### What is in this change? <!-- Describe the changes in this PR that are impactful to the repo. --> ### Additional Information <!-- Add any other context about the Pull Request here that was not captured above. --> ### Developer Validations <!-- All of the applicable items should be checked. --> - [ ] 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 --- <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:36:03 -05:00
yindo closed this issue 2026-02-22 18:36:03 -05:00
yindo changed title from [PR #4203] Variable integrity in agent flow variable replacement to [PR #4203] [CLOSED] Variable integrity in agent flow variable replacement 2026-06-05 15:19:08 -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#4551