Document in AGENTS.md reasoning for not using dead code analysis (#843)

Author: mateusz834

.github/workflows/appinspect.yml

@@ -24,3 +24,4 @@ jobs:
           deps-group: lint
       - name: Run AppInspect
         uses: ./.github/actions/run-appinspect
+        continue-on-error: ${{ github.event_name == 'pull_request' }}

AGENTS.md

@@ -62,3 +62,10 @@ New code must not introduce regressions in tests.
 
 Be concise and direct in your responses.
 Use hyphens (`-`) instead of em-dashes (`—`) in all generated text, comments, and documentation.
+
+## Dead code
+
+- SDK does not rely on any dead code detection, since this SDK is held to a backwards-compatibility contract:
+  public API surface (classes, functions, methods) can appear unused within this repo while still being consumed
+  by external callers. Do not remove code solely because it looks unreferenced internally - check whether it's
+  part of the public API first.

tests/integration/test_kvstore_data.py

@@ -79,7 +79,7 @@ def test_params_data_type_conversion(self):
         )
         for x in range(50):
             self.col.insert(json.dumps({"_key": str(x), "data": str(x), "ignore": x}))
-        data = self.col.query(sort="data:-1", limit=20, fields="data,_id:0", skip=10)
+        data = self.col.query(sort="data:-1", limit=20, fields="data", skip=10)
         self.assertEqual(len(data), 20)
         for x in range(20):
             self.assertEqual(data[x]["data"], 39 - x)