feat: add credential management #91
Conversation
Signed-off-by: Grant Linville <[email protected]>
Signed-off-by: Grant Linville <[email protected]>
Signed-off-by: Grant Linville <[email protected]>
Signed-off-by: Grant Linville <[email protected]>
Signed-off-by: Grant Linville <[email protected]>
g-linville
requested review from
drpebcak,
thedadams,
StrongMonkey and
tylerslaton
src/gptscript.ts
Outdated
| const resp = await fetch(`${GPTScript.serverURL}/credentials`, { | ||
| method: "POST", | ||
| body: JSON.stringify({context, allContexts}) | ||
| }) |
There was a problem hiding this comment.
Does runSubcommand not work for these?
There was a problem hiding this comment.
I remember trying it and it not working for some reason, but I don't remember why...let me try that again and see.
There was a problem hiding this comment.
Taking a look at the code again, RunSubcommand doesn't let me specify an arbitrary request body to use. I can only pass it a tool and some run options. But I need to be able to pass things like a credential (in the case of create), or name/context information for the other operations.
There was a problem hiding this comment.
Let me know if you want me to change this or leave it as is @thedadams
There was a problem hiding this comment.
Can you try something like this?
const r: Run = new RunSubcommand("credentials", undefined, {}, GPTScript.serverURL)
r.request(payload)
await r.text()
There was a problem hiding this comment.
That worked, thanks. This is updated now
Signed-off-by: Grant Linville <[email protected]>
Signed-off-by: Grant Linville <[email protected]>
Done |
No description provided.