From 2c288ee479fabd466f1d04b370db3f0e0aaedd2c Mon Sep 17 00:00:00 2001
From: romanskas <30618745+romanskas@users.noreply.github.com>
Date: Wed, 13 May 2026 16:53:03 +0200
Subject: [PATCH] SERVER-125920 Introduce jstests-local CLAUDE.md (#53103)

GitOrigin-RevId: 14131f41956ea00f3f1657c651a0462834f7167f
---
 jstests/AGENTS.md  | 61 ++++++++++++++++++++++++++++++++++++++++++++++
 jstests/CLAUDE.md  |  1 +
 jstests/OWNERS.yml |  5 +---
 3 files changed, 63 insertions(+), 4 deletions(-)
 create mode 100644 jstests/AGENTS.md
 create mode 100644 jstests/CLAUDE.md

diff --git a/jstests/AGENTS.md b/jstests/AGENTS.md
new file mode 100644
index 00000000000..905db8abf99
--- /dev/null
+++ b/jstests/AGENTS.md
@@ -0,0 +1,61 @@
+## JavaScript Test Code Conventions
+
+### Assertion Library
+
+ALWAYS use our own assertion library that is defined at `src/mongo/shell/assert.js` and
+automatically loaded into the global scope before running each test.
+
+ALWAYS wrap commands with `assert.commandWorked()`, unless failures are expected and checked. Use
+`assert.writeOK()` when working with legacy bulk write APIs.
+
+Use `assert.soon()` only for asynchronous or eventually-consistent conditions.
+
+### Mocha Style for New Test Files
+
+Use the mocha style to structure test cases in newly added test files. The
+`jstests/libs/mochalite.js` module implements `describe`, `it`, `before`, `beforeEach`, `afterEach`,
+and `after`. Import only what you need. Feel free to add other mocha helper functions to
+`jstests/libs/mochalite.js` if they would make the current test cleaner.
+
+```js
+import {before, describe, it} from "jstests/libs/mochalite.js";
+
+describe("feature under test", function () {
+  before(function () {
+    /* one-time setup */
+  });
+  it("does X", function () {
+    /* ... */
+  });
+});
+```
+
+### Use JSON-Conformant Object Serialization in Logs
+
+Pass objects to the `attr` parameter of assertion and logging functions instead of concatenating
+`tojson()` into the message string. Most `assert.*` functions accept an `attr` object as their last
+argument.
+
+```js
+// Bad — tojson() in the message string
+assert(cursor.hasOwnProperty("metrics"), "metrics missing: " + tojson(cursor));
+
+// Good — object passed via attr
+assert(cursor.hasOwnProperty("metrics"), "metrics missing", {cursor});
+```
+
+For logging:
+
+```js
+// Bad
+jsTest.log.info("got cursor: " + tojson(cursor));
+
+// Good
+jsTest.log.info("got cursor", {cursor});
+```
+
+### Resource and Settings Cleanup
+
+ALWAYS clean up resources and settings immediately after they are no longer needed. Examples of
+server resources are cursors and collections. Examples of settings are server parameters and test
+framework settings stored in the global `TestData` object.
diff --git a/jstests/CLAUDE.md b/jstests/CLAUDE.md
new file mode 100644
index 00000000000..43c994c2d36
--- /dev/null
+++ b/jstests/CLAUDE.md
@@ -0,0 +1 @@
+@AGENTS.md
diff --git a/jstests/OWNERS.yml b/jstests/OWNERS.yml
index d4914d74764..0a54c0ec06b 100644
--- a/jstests/OWNERS.yml
+++ b/jstests/OWNERS.yml
@@ -1,9 +1,6 @@
 version: 2.0.0
 filters:
-  - "README.md":
-    approvers:
-      - 10gen/devprod-test-infrastructure
-  - "tags.md":
+  - "*.md":
     approvers:
       - 10gen/devprod-test-infrastructure
   - "**/*analyze_shard_key*":