address typing

Author: blink1073

CONTRIBUTING.md

@@ -520,4 +520,5 @@ To profile a test script and generate a flame graph, follow these steps:
 Dependabot will raise PRs at most once per week, grouped by GitHub Actions updates and Python requirement
 file updates.  We have a pre-commit hook that will update the `uv.lock` file when requirements change.
 To update the lock file on a failing PR, you can use a method like `gh pr checkout <pr number>`, then run
-`just lint uv-lock` to update the lock file, and then push the changes.
+`just lint uv-lock` to update the lock file, and then push the changes.  If a typing dependency has changed,
+also run `just typing` and handle any new findings.

bson/__init__.py

@@ -1327,7 +1327,7 @@ def decode_iter(
         elements = data[position : position + obj_size]
         position += obj_size
 
-        yield _bson_to_dict(elements, opts)  # type:ignore[misc]
+        yield _bson_to_dict(elements, opts)
 
 
 @overload
@@ -1373,7 +1373,7 @@ def decode_file_iter(
             raise InvalidBSON("cut off in middle of objsize")
         obj_size = _UNPACK_INT_FROM(size_data, 0)[0] - 4
         elements = size_data + file_obj.read(max(0, obj_size))
-        yield _bson_to_dict(elements, opts)  # type:ignore[arg-type, misc]
+        yield _bson_to_dict(elements, opts)  # type:ignore[misc]
 
 
 def is_valid(bson: bytes) -> bool:

test/test_typing.py

@@ -170,7 +170,7 @@ def test_bulk_write_heterogeneous(self):
             InsertOne(Movie(name="American Graffiti", year=1973)),
             ReplaceOne(
                 {},
-                {"name": "American Graffiti", "year": "WRONG_TYPE"},  # type:ignore[typeddict-item]
+                {"name": "American Graffiti", "year": "WRONG_TYPE"},  # type:ignore[arg-type]
             ),
             DeleteOne({}),
         ]
@@ -243,7 +243,7 @@ def test_with_options(self) -> None:
         assert retrieved is not None
         assert retrieved["name"] == "foo"
         # We expect a type error here.
-        assert retrieved["other"] == 1  # type:ignore[typeddict-item]
+        assert retrieved["other"] == 1  # type:ignore[misc]
 
 
 class TestDecode(unittest.TestCase):
@@ -416,11 +416,11 @@ def test_typeddict_document_type_insertion(self) -> None:
         bad_mov = {"name": "THX-1138", "year": "WRONG TYPE"}
         bad_movie = Movie(name="THX-1138", year="WRONG TYPE")  # type: ignore[typeddict-item]
         coll.insert_one(bad_mov)  # type:ignore[arg-type]
-        coll.insert_one({"name": "THX-1138", "year": "WRONG TYPE"})  # type: ignore[typeddict-item]
+        coll.insert_one({"name": "THX-1138", "year": "WRONG TYPE"})  # type: ignore[arg-type]
         coll.insert_one(bad_movie)
         coll.insert_many([bad_mov])  # type: ignore[list-item]
         coll.insert_many(
-            [{"name": "THX-1138", "year": "WRONG TYPE"}]  # type: ignore[typeddict-item]
+            [{"name": "THX-1138", "year": "WRONG TYPE"}]  # type: ignore[list-item]
         )
         coll.insert_many([bad_movie])