这是indexloc提供的服务,不要输入任何密码
Skip to content

Allow static defined prompt variables to be accessed by any authorized user #4086

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

shatfield4
Copy link
Collaborator

Pull Request Type

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

Relevant Issues

resolves #4081

What is in this change?

  • Allow all static defined variables to be accessed in system prompts with any authorized user
  • Allow admins to see all defined variables, not just variables defined by their user account
  • Makes admin-created system prompt variables globally accessible
  • Tested in both multiuser and single user modes

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

@shatfield4 shatfield4 linked an issue Jul 2, 2025 that may be closed by this pull request
@shatfield4 shatfield4 added the PR:needs review Needs review by core team label Jul 2, 2025
@scottybo
Copy link
Contributor

scottybo commented Jul 2, 2025

Thank you for resolving this so quickly!

…the-llm' of github.com:Mintplex-Labs/anything-llm into 4081-bug-system-prompts-not-visible-to-other-admins-or-the-llm
@timothycarambat timothycarambat merged commit f3839b3 into master Jul 2, 2025
@timothycarambat timothycarambat deleted the 4081-bug-system-prompts-not-visible-to-other-admins-or-the-llm branch July 2, 2025 15:09
cabwds pushed a commit to cabwds/anything-llm that referenced this pull request Jul 3, 2025
…d user (Mintplex-Labs#4086)

* allow static defined prompt variables to be accessed by any authd user

* improve filtering logic and comments

---------

Co-authored-by: Timothy Carambat <rambat1010@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR:needs review Needs review by core team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG]: System prompts not visible to other admins or the LLM
3 participants