[TS] Use 'override' when overriding class items

crates/bindings-typescript/src/lib/schema.ts

@@ -110,32 +110,21 @@ export function tablesToSchema<
         // UntypedTableDef expects mutable array; idxs are readonly, spread to copy.
         indexes: [
           ...schema.idxs.map(
-            (idx: Infer<typeof RawIndexDefV9>): IndexOpts<any> =>
-              ({
+            (idx: Infer<typeof RawIndexDefV9>): IndexOpts<any> => {
+              const columnIds =
+                idx.algorithm.tag === 'Direct'
+                  ? [idx.algorithm.value]
+                  : idx.algorithm.value;
+              const columns = columnIds.map(i => colNameList[i]);
+              return {
                 name: idx.accessorName,
-                unique: schema.tableDef.constraints
-                  .map(c => {
-                    if (idx.algorithm.tag == 'BTree') {
-                      return c.data.value.columns.every(col => {
-                        const idxColumns = idx.algorithm.value;
-                        if (Array.isArray(idxColumns)) {
-                          return idxColumns.includes(col);
-                        } else {
-                          return col === idxColumns;
-                        }
-                      });
-                    }
-                  })
-                  .includes(true),
+                unique: schema.tableDef.constraints.some(c =>
+                  c.data.value.columns.every(col => columnIds.includes(col))
+                ),
                 algorithm: idx.algorithm.tag.toLowerCase() as 'btree',
-                columns: (() => {
-                  const cols =
-                    idx.algorithm.tag === 'Direct'
-                      ? [idx.algorithm.value]
-                      : idx.algorithm.value;
-                  return cols.map(i => colNameList[i]);
-                })(),
-              }) as IndexOpts<any>
+                columns,
+              } as IndexOpts<any>;
+            }
           ),
         ],
       } as const;

crates/codegen/src/typescript.rs

@@ -430,7 +430,7 @@ impl Lang for TypeScript {
 
         writeln!(out);
         writeln!(out, "/** Creates a new {{@link SubscriptionBuilder}} to configure a subscription to the remote SpacetimeDB instance. */");
-        writeln!(out, "subscriptionBuilder = (): SubscriptionBuilder => {{");
+        writeln!(out, "override subscriptionBuilder = (): SubscriptionBuilder => {{");
         out.indent(1);
         writeln!(out, "return new SubscriptionBuilder(this);");
 

crates/codegen/tests/snapshots/codegen__codegen_typescript.snap

@@ -1,6 +1,5 @@
 ---
 source: crates/codegen/tests/codegen.rs
-assertion_line: 37
 expression: outfiles
 ---
 "add_player_reducer.ts" = '''
@@ -587,7 +586,7 @@ export class DbConnection extends __DbConnectionImpl<typeof REMOTE_MODULE> {
   };
 
   /** Creates a new {@link SubscriptionBuilder} to configure a subscription to the remote SpacetimeDB instance. */
-  subscriptionBuilder = (): SubscriptionBuilder => {
+  override subscriptionBuilder = (): SubscriptionBuilder => {
     return new SubscriptionBuilder(this);
   };
 }