Closed cjellick closed 1 day ago
This is one of the workflows were we see the Share Context
not working when knowledge tool is added using Add Knowledge
option.
Add Knowledge
to add a knowledge file.How to raise money to start a Business . Use knowledge."
This behavior of needing to ask it to explicitly use knowledge
tool is not needed if I have workspace
tool added to the assistant.
This should not needed since we already have Share Context: github.com/gptscript-ai/context/workspace
included in knowledge
tool.
I'm assigning to myself because I guess the best way forward with this is, as @thedadams mentioned, to support dynamic tool adding in core gptscript. So, I'll talk to @ibuildthecloud about this on monday and see where he wants to go with it.
fixed in core gptscript here: https://github.com/gptscript-ai/gptscript/pull/827
Testing desktop app with gptscript version - v0.0.0-dev+f2bfa9f3
I am still able to reproduce the issue with having to ask Knowledge tool to used explicitly when testing the scenario outlined in https://github.com/gptscript-ai/desktop/issues/259#issuecomment-2305824276 . This might be relating to issue - https://github.com/gptscript-ai/desktop/issues/272.
When I tried again with another knowledge file, I was able to get the Knowledge tool to get invoked without having to ask it to use knowledge explicitly. Also I see the workspace context being used in the stack trace view.
I will also test cases mentioned in https://github.com/g-linville/context-testing
When testing scenarios by adding the tools dynamically to a thread using "Add Tool" from https://github.com/g-linville/context-testing , following are the observations made:
When adding tool - github.com/g-linville/context-testing/direct
When adding tool - github.com/g-linville/context-testing/indirect
In the case of github.com/g-linville/context-testing/direct
, the assistant doesn't have "direct" access to the context. Only the tool blue
does because the context is not shared. Therefore, the context is only run when the blue
tool is run.
For example:
Similarly for the https://github.com/g-linville/context-testing/indirect
tool:
Let me know what you think about these results.
Closing this issue as per testing based on https://github.com/gptscript-ai/desktop/issues/259#issuecomment-2320021127.
Best example of this is the knowledge tool, which has this line:
Grant put together tests here: 'https://github.com/g-linville/context-testing
Follow up question: does Share Tools work? seems like it does, but we should verify