Scott Trinh

Results 319 comments of Scott Trinh

Another cause of nondeterminism here is that we run the query generation in parallel (see https://github.com/edgedb/edgedb-js/blob/316a398f9dbb413a880440086df7b5bebc7e68b5/packages/generate/src/queries.ts#L57) so to get truly deterministic ordering we'd need to do it in series.

@awerlogus Some story for improving the composability of query builder expressions that take parameters is definitely high up on the priority list of things to figure out. Nothing to share...

@MaDsEm88 No ETA on that, but we're open to getting help from the community on adding integrations here, if you have familiarity with Deno Fresh and it's idioms!

> Would also make reporting minimized JS/TS related bugs easier. Great idea, I hadn't considered having a "minimal" preset/template here, but that makes sense. Could even have it provide a...

> we can move the dbschema and add the using extension auth bit to the base template Maybe we should have a separate recipe for "auth" rather than moving the...

We recently did a round of improvements to the error messaging (https://github.com/edgedb/edgedb-js/pull/731). Which version of `edgedb` are you on?

We should be inferring the same cardinality as the EdgeQL compiler, which is the real bug here. We detect the cardinality of the query here as `ONE` (based on the...

This is somewhat related to the work I've started in #606 . It's a little tricky because TypeScript and EdgeQL have slightly different type systems, so I'm trying to tread...

@jaclarke Can you give some detail about which parts of the code gen are non-deterministic? All three generators? Do you have an suspicions about what the reason for the non-determinism...

@pk-nb No timeline on this yet, but it's on my radar to try to bring the linked PR current against `master`. There are some potential interactions between this and some...