K

Kysely•2y ago•

5 replies

Marcus - noxify

Complex Query Builder from dynamic input - Typescript issue

Hi, it's me - again

I'm currently trying to migrate our current query builder.

It seems that everything is working as expected, but I get some type errors and I'm not sure how to fix this - Maybe you have an idea how to fix this / what the rootcause is.

The query is compiled and I can see it in my vitest & in the playground ( you have to open the web console )

Playground link: https://kyse.link/8cA4l

It's based on:
* https://kysely.dev/docs/recipes/expressions#creating-reusable-helpers
* https://kysely.dev/docs/recipes/expressions#conditional-expressions

Expressions | Kysely

An Expression is the basic type-safe query building block in Kysely. Pretty much all methods accept expressions as inputs. Most internal classes like SelectQueryBuilder and RawBuilder (the return value of the sql tag) are expressions themselves.

Solution

Hey

This is a bit harder to grok.

What's immediate is that conditionBuilderconditionBuilder might return undefinedundefined and .where.where expects something.
Also, .flatMap(identity).flatMap(identity) could be changed to .flat().flat().
conditionBuilderconditionBuilder should return ExpressionWrapper<KyselyDatabase<TableDefinition>, "tablename", SqlBool>ExpressionWrapper<KyselyDatabase<TableDefinition>, "tablename", SqlBool>.
filtersfilters should be of type ExpressionWrapper<KyselyDatabase<TableDefinition>, "tablename", SqlBool>[]ExpressionWrapper<KyselyDatabase<TableDefinition>, "tablename", SqlBool>[]
getFieldQuerygetFieldQuery should also return ExpressionWrapper<KyselyDatabase<TableDefinition>, "tablename", SqlBool>ExpressionWrapper<KyselyDatabase<TableDefinition>, "tablename", SqlBool>
You can as anyas any getFieldQuerygetFieldQuery's return values.
https://kyse.link/FIsu8

Jump to solution