Carson Full

Alright @iRyusa on to you

Mmm see your problem is you are mutating 😛 Jokes aside, that's _not_ relevant to this. TS thinks `entries()` returns ```ts IterableIterator ``` Which it's obviously now wrong about. The...

I'm all for this! --- It's unfortunate the pervasive use of this type. It would be much better to refactor it out. ```ts export type ModuleRef = Type | DynamicModule...

@scotttrinh what do you think about an intermediate object to decouple input/output? ```ts await queryRag(message, context).text() await queryRag(message, context).response() for await (part of queryRag(message, context)) { ... } ```

> @CarsonF > > > what do you think about an intermediate object to decouple input/output? > > I'm not sure I understand the suggestion here 🤔 Instead of 3...

As far as # 1 goes, fetch has the same API ```ts await fetch(...).json() await fetch(...).arrayBuffer() ``` I don't see any problems here. I can't speak to 2 & 6....

This is what I do https://github.com/SeedCompany/cord-api-v3/blob/24dce245ab4e2ab6a116b45bd68dff5f0d8eeb88/src/core/edgedb/edgedb.service.ts#L36-L87

Maybe the python library should change too 😛

> IMO, on earlier versions we continue to silently fail, which is exactly as bad as the current state. Well it was only a silent failure for me before because...

Awesome thanks man 👍