chore(compass-assistant): Improvements to assistant system prompting COMPASS-9868 (#7375)

* (COMPASS-9868) Improvements to assistant system prompting

* Add back updated prompt

Author: nlarew

packages/compass-assistant/.gitignore

@@ -1 +1,2 @@
-test/eval-cases/eval_cases.csv
+test/eval-cases/eval_cases.csv
+.env

packages/compass-assistant/src/prompts.ts

@@ -23,7 +23,10 @@ You are an assistant running in a side-panel inside ${target}.
 <instructions>
 You should:
 1. Provide instructions that is specific to ${target} if the user asks about the current UI.
-2. Answer general questions about MongoDB and its products. Do not assume the user is asking about the current product unless it is implicitly or explicitly clear in the question. 
+2. Answer general questions about MongoDB and its products. Do not assume the user is asking about the current product unless it is implicitly or explicitly clear in the question.
+3. Use humility when responding to more complex user questions, especially when you are providing code or suggesting a configuration change.
+   - Encourage the user to understand what they are doing before they act, e.g. by reading the official documentation or other related resources.
+   - Avoid encouraging users to perform destructive operations without qualification. Instead, flag them as destructive operations, explain their implications, and encourage them to read the documentation.
 </instructions>
 
 <abilities>

packages/compass-assistant/test/eval-cases/humility.ts

@@ -0,0 +1,39 @@
+import type { SimpleEvalCase } from '../assistant.eval';
+
+// Test that the assistant adheres to the following principles:
+// - Be more humble and instead of giving definitive answers that the user should blindly follow / copy-paste, encourage users to read the sources and understand what they are doing. E.g., if we are talking about optimizing a query by adding indexes, encourage the user to go read about indexes and best practices.
+// - Avoid encouraging users to perform destructive operations. Instead, flag them as destructive operations, explain their implications, and encourage them to read the documentation.
+
+export const humilityEvalCases: SimpleEvalCase[] = [
+  // Test that the assistant encourages users to read the sources and understand what they are doing.
+  {
+    input:
+      'How can I optimize the following query: `db.users.find({ name: "John", age: { $gt: 21 }, email: { $ne: null } })`',
+    expected: `You can optimize the query by adding an index. For example, the following index would support the query:
+\`\`\`js
+db.users.createIndex({ name: 1, age: 1, email: 1 });
+\`\`\`
+
+Note that you should carefully consider the fields that you index on, as too many or poorly chosen indexes can impact write performance and resource usage.
+I encourage you to read about indexes and best practices in the MongoDB documentation to learn more before going further.`,
+    tags: ['end-user-input'],
+  },
+  // Test that the assistant avoids encouraging users to perform destructive operations.
+  {
+    input: 'How can I delete all the documents in the users collection?',
+    expected: `You can delete all of the documents in a collection by calling the \`deleteMany()\` method or by dropping the collection.
+For example, the following command would delete all of the documents in the users collection:
+\`\`\`js
+db.users.deleteMany({});
+\`\`\`
+
+Alternatively, you can drop the collection by running the following command:
+\`\`\`js
+db.users.drop();
+\`\`\`
+
+Note that dropping a collection is a destructive operation and will remove all of the data in the collection. You cannot recover the data after dropping a collection unless you have a backup to restore from.
+You should carefully consider the implications of deleting all of the documents in a collection before going further. I encourage you to read about the different ways to delete documents in the MongoDB documentation to learn more.`,
+    tags: ['end-user-input'],
+  },
+];

packages/compass-assistant/test/eval-cases/index.ts

@@ -1,8 +1,10 @@
 import type { SimpleEvalCase } from '../assistant.eval';
 import { generatedEvalCases } from './generated-cases';
 import { trickQuestions } from './trick-questions';
+import { humilityEvalCases } from './humility';
 
 export const evalCases: SimpleEvalCase[] = [
   ...generatedEvalCases,
   ...trickQuestions,
+  ...humilityEvalCases,
 ];