fix: rename create_collection indexes->index + doc notes (mongo-3/4/5)

- mongo-3: create_collection param renamed indexes -> index (it seeds ONE index; a key-spec
  builds one compound index, not two). update keyword call sites indexes= -> index=.
- mongo-4: docstring notes an instance is single-use after close() (pymongo 4.x close is
  irreversible; connect() only pings).
- mongo-5: docstring notes construction may raise InvalidURI for a malformed URI.
- README: create_collection index note. bump v0.1.5 -> v0.1.6

Signed-off-by: disqualifier <dev@disqualifier.me>

README.md

@@ -8,18 +8,18 @@ helpers for the common paths, with a raw escape hatch for everything else.
 `requirements.txt`:
 
 ```
-mongo @ git+ssh://git@git.rethinkstudios.io/rethink-public/mongo.git@v0.1.4
+mongo @ git+ssh://git@git.rethinkstudios.io/rethink-public/mongo.git@v0.1.6
 ```
 
 Direct:
 
 ```bash
-pip install "mongo @ git+ssh://git@git.rethinkstudios.io/rethink-public/mongo.git@v0.1.4"
+pip install "mongo @ git+ssh://git@git.rethinkstudios.io/rethink-public/mongo.git@v0.1.6"
 ```
 
 Requires `motor` and `pymongo` (pulled transitively).
 
-Drop the `@v0.1.4` suffix from the line above to install the latest unpinned.
+Drop the `@v0.1.6` suffix from the line above to install the latest unpinned.
 
 ## Usage
 
@@ -87,6 +87,9 @@ are included.
 
 - `from mongo import func` won't see the proxy (resolved at import, before `init`).
   Use `import mongo` then `mongo.func(...)`.
+- `create_collection(collection, index=...)` seeds **one** index (the param is `index`,
+  renamed from `indexes` in v0.1.6). A key-spec like `[("a",1),("b",1)]` builds one
+  compound index, not two — call `create_index` per index for multiples.
 - `find_one_and_update` returns the **after** image by default (`return_after=True`).
 - `bulk_write` takes pymongo ops the caller builds:
   ```python

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "mongo"
-version = "0.1.4"
+version = "0.1.6"
 description = "async mongodb wrapper over motor with a raw escape hatch"
 requires-python = ">=3.10"
 dependencies = [

src/mongo/mongo.py

@@ -53,6 +53,10 @@ class MongoDB:
     """async mongodb wrapper; one client per process, attach to bot as bot.db"""
 
     def __init__(self, connection_string: str, database: str):
+        # motor builds the client eagerly here (no network I/O yet). a syntactically
+        # malformed URI (e.g. "mongodb://") makes the driver raise InvalidURI at
+        # construction, before connect() — a deploy-time value, so it surfaces at startup
+        # either way and there is nothing to clean up.
         self._client = AsyncIOMotorClient(connection_string)
         self._db = self._client[database]
 
@@ -62,12 +66,24 @@ class MongoDB:
         motor connects lazily, so this is optional — call it to fail early on a bad
         URI/credentials rather than on the first real op (parallel to the trio's
         connect()). raises on a bad connection, unlike the swallowing wrapped methods.
+
+        an instance is single-use: pymongo 4.x `close()` is irreversible, and connect()
+        only pings (it does not rebuild the client). do not reuse an instance after
+        close()/`async with` exit — construct a fresh MongoDB.
         """
         await self._client.admin.command("ping")
         return self
 
     async def __aenter__(self) -> "MongoDB":
+        try:
             return await self.connect()
+        except BaseException:
+            # connect()/ping failing here would otherwise leak the motor client (built
+            # eagerly in __init__ with a background topology monitor + pool) — __aexit__
+            # is not called when __aenter__ raises. close it before propagating so a
+            # retry loop against a flapping server doesn't accumulate live clients.
+            self.close()
+            raise
 
     async def __aexit__(self, exc_type, exc, tb) -> None:
         self.close()
@@ -101,13 +117,19 @@ class MongoDB:
     # -------------------------------------------------------------------------
     #   collection / index management
 
-    async def create_collection(self, collection: str, indexes=None) -> bool:
-        """create a collection, optionally seeding an index"""
+    async def create_collection(self, collection: str, index=None) -> bool:
+        """create a collection, optionally seeding ONE index
+
+        `index` is a single index key-spec (e.g. `[("a", 1), ("b", 1)]` builds one
+        compound index, NOT two). for multiple indexes call `create_index` per index.
+        renamed from `indexes` in v0.1.6 to reflect that it seeds a single index; pass it
+        positionally, or update a keyword call site from `indexes=` to `index=`.
+        """
         try:
             await self._db.create_collection(collection)
-            if indexes:
-                await self._db[collection].create_index(indexes)
-                log.info(f"created indexes for {collection}")
+            if index:
+                await self._db[collection].create_index(index)
+                log.info(f"created index for {collection}")
             return True
         except Exception:
             log.exception(f"db.create_collection() for {collection}")