curl escape (#534)

microsoft · Jun 11, 2024 · 2f19b60 · 2f19b60
1 parent 5483560
commit 2f19b60
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/docs/src/content/docs/guides/pull-request-reviewer.mdx b/docs/src/content/docs/guides/pull-request-reviewer.mdx
@@ -14,7 +14,9 @@ The reviewer can also suggest changes to the code, documentation, or other files
 
 The output of the LLM is inserted as a comment in the pull request conversation 
 (and updated as needed to avoid duplicates). 
-Here is an [example](https://github.com/microsoft/genaiscript/pull/504#issuecomment-2145273728) in the GenAIScript repository.
+
+Here is an [pull request](https://github.com/microsoft/genaiscript/pull/534) in the GenAIScript repository
+with genai-generated description, comments and reviews.
 
 ## Step 1: The script
 

diff --git a/packages/core/src/openai.ts b/packages/core/src/openai.ts
@@ -130,12 +130,12 @@ export const OpenAIChatCompletion: ChatCompletionHandler = async (
 
     trace.detailsFenced(
         `✉️ fetch`,
-        `curl ${url} \\
+        `curl -X POST ${url} \\
 -H "Content-Type: application/json" \\
 ${Object.entries(cfg.curlHeaders || {})
     .map(([k, v]) => `-H "${k}: ${v}" \\`)
     .join("\n")}
--d '${JSON.stringify(postReq, null, 2)}' 
+-d '${JSON.stringify(postReq).replace(/'/g, "'\\''")}' 
 `,
         "bash"
     )

diff --git a/packages/vscode/src/vshost.ts b/packages/vscode/src/vshost.ts
@@ -254,6 +254,9 @@ export class VSCodeHost extends EventTarget implements Host {
             })
             if (!azureToken) throw new Error("Azure token not available")
             tok.token = "Bearer " + azureToken
+            tok.curlHeaders = {
+                Authorization: "Bearer ***",
+            }
         }
         return tok
     }