Merge pull request #16 from presubmit/dev

Updated review prompt with code analysis guidelines
presubmit · Nov 21, 2024 · 2b2d6de · 2b2d6de
2 parents ba50bfd + ca52cf6
commit 2b2d6de
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/dist/index.js b/dist/index.js
diff --git a/src/prompts.ts b/src/prompts.ts
@@ -34,6 +34,8 @@ export async function runSummaryPrompt(
 - When quoting variables or names from the code, use backticks (\`).
 - Return a summary for each single affected file or if there is nothing to summarize simply use the status of the change (ie. "New file").
 - Start the overview with a verb at past tense like "Started", "Commented", "Generated" etc...
+
+IMPORTANT: Do not make assumptions about the code outside the diff. Do not assume variable could be optional if you don't see the type declaration. Do not suggest null checks unless you are sure this could lead to a runtime error.
 \n`;
 
   let userPrompt = `