**3 issues found** across 5 files

Prompt for AI agents (unresolved issues)

```text Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately. P2: The embedding transaction is committed without a subsequent `Discard`, which the Dgraph client docs recommend to clean up resources (and it’s safe after commit). Add a discard after commit and on commit error to avoid leaking txn resources. P2: The client cache key does not include the new EmbeddingProvider option. Creating a client with a different embedding provider will reuse an existing cached client and ignore the new provider, which can generate embeddings with the wrong model or unexpectedly skip embedding generation. P1: Bug: `defer cleanup()` releases the pooled `*dgo.Dgraph` connection back to the pool when `SimilarToText` returns, but the returned `*dg.QueryBlock` still holds a reference to a transaction on that connection. When the caller later calls `.Scan()`, the underlying connection may already be in use by another operation, causing data races or query failures. The cleanup function and the `*dgo.Dgraph` client should not be deferred here — they need to remain alive until after the caller finishes with the QueryBlock. Consider either: (1) returning the cleanup function alongside the QueryBlock so the caller can manage the lifecycle, or (2) executing the query inside this function and returning the results directly. ```

_{Reply with feedback, questions, or to request a fix. Tag `@cubic-dev-ai` to re-run a review.}