Fix sequence operator bug (#322)

* Fix sequence bug

* Fix seq

* 0.20.3

package.json

@@ -1,6 +1,6 @@
 {
   "name": "edgedb",
-  "version": "0.20.2",
+  "version": "0.20.3",
   "description": "The official Node.js client library for EdgeDB",
   "homepage": "https://edgedb.com/docs",
   "author": "EdgeDB <info@edgedb.com>",

qb/playground.ts

@@ -7,19 +7,7 @@ import e, * as types from "./dbschema/edgeql-js/index";
 
 async function run() {
   const {client} = await setupTests();
-  const query = e.select(e.Villain, () => ({
-    id: true,
-    name: true,
-    nemesis: nemesis => {
-      const nameLen = e.len(nemesis.name);
-      return {
-        name: true,
-        nameLen,
-        nameLen2: nameLen,
-      };
-    },
-  }));
-
+  const query = e.select("Hello world!");
   console.log(query.toEdgeQL());
   const result = await query.run(client);
   console.log(result);

qb/test/select.test.ts

@@ -1116,6 +1116,10 @@ test("select with enums", async () => {
   expect(result.length).toEqual(2);
 });
 
+test("filter by sequence", async () => {
+  await e.op(e.Bag.seqField, "=", 1).run(client);
+});
+
 // Modifier methods removed for now, until we can fix typescript inference
 // problems / excessively deep errors
 

src/reflection/generators/generateOperatorTypes.ts

@@ -63,7 +63,6 @@ export function generateOperators({
 }: GeneratorParams) {
   const typeSpecificities = getTypesSpecificity(types, casts);
   const implicitCastableRootTypes = getImplicitCastableRootTypes(casts);
-
   const code = dir.getPath("operators");
 
   code.addImport({$: true}, "edgedb");

src/reflection/queries/getTypes.ts

@@ -5,7 +5,6 @@ import {StrictMap} from "../strictMap";
 export type UUID = string;
 
 export type Pointer = {
-  // cardinality: "One" | "Many";
   real_cardinality: Cardinality;
   kind: "link" | "property";
   name: string;
@@ -14,7 +13,6 @@ export type Pointer = {
   is_computed: boolean;
   is_readonly: boolean;
   has_default: boolean;
-  is_seq: boolean;
   pointers: ReadonlyArray<Pointer> | null;
 };
 
@@ -38,6 +36,7 @@ export type TypeProperties<T extends TypeKind> = {
 
 export type ScalarType = TypeProperties<"scalar"> & {
   is_abstract: boolean;
+  is_seq: boolean;
   bases: ReadonlyArray<{id: UUID}>;
   // ancestors: ReadonlyArray<{id: UUID}>;
   enum_values: ReadonlyArray<string> | null;
@@ -75,20 +74,20 @@ export type Type = PrimitiveType | ObjectType;
 
 export type Types = StrictMap<UUID, Type>;
 
+export const nonCastableTypes = new Set<string>([
+  // numberType.id
+]);
+
 const numberType: ScalarType = {
   id: "00000000-0000-0000-0000-0000000001ff",
   name: "std::number",
   is_abstract: false,
+  is_seq: false,
   kind: "scalar",
   enum_values: null,
   material_id: null,
   bases: [],
 };
-
-export const nonCastableTypes = new Set<string>([
-  // numberType.id
-]);
-
 export const typeMapping = new Map([
   [
     "00000000-0000-0000-0000-000000000103", // int16
@@ -139,7 +138,7 @@ export async function getTypes(
               'unknown',
 
       [IS ScalarType].enum_values,
-
+      is_seq := 'std::sequence' in [IS ScalarType].ancestors.name,
       # for sequence (abstract type that has non-abstract ancestor)
       single material_id := (
         SELECT x := Type[IS ScalarType].ancestors
@@ -221,6 +220,9 @@ export async function getTypes(
         if (typeMapping.has(type.id)) {
           type.castOnlyType = typeMapping.get(type.id)!.id;
         }
+        if (type.is_seq) {
+          type.castOnlyType = numberType.id;
+        }
         // if (type.material_id) {
         //   type.material_id =
         //     typeMapping.get(type.material_id)?.id ?? type.material_id;