use user field name when getting linked rows

src/codegen/makeGetter.spec.ts

@@ -8,7 +8,17 @@ function run(field: Partial<FieldDefinition> = {}): string {
     {
       id: 1,
       name: "table_name",
-      fields: [],
+      fields: [
+        {
+          id: 2,
+          name: "field_name",
+          table_id: 0,
+          order: 0,
+          type: "",
+          primary: false,
+          read_only: false,
+        },
+      ],
     },
   ]);
 }
@@ -98,6 +108,14 @@ describe("makeGetter", () => {
       },
       "this.getLinkedRows",
     ],
+    [
+      {
+        type: "link_row",
+        link_row_table_id: 1,
+        link_row_related_field_id: 2,
+      },
+      `"field_name"`,
+    ],
   ])("%s => `%s`", (field, expected) => {
     expect(run(field)).toContain(expected);
   });

src/codegen/makeGetter.ts

@@ -3,10 +3,7 @@ import { FieldDefinition, ListFieldsResponse } from "../index.js";
 import { getRawType } from "./getRawType.js";
 import { toCamelCase } from "./toCamelCase.js";
 
-function getForeignTable(
-  field: FieldDefinition,
-  tables: Table[],
-): { id: number; name: string } {
+function getForeignTable(field: FieldDefinition, tables: Table[]): Table {
   if (!field.link_row_table_id) {
     throw new Error("link_row_table_id is missing");
   }
@@ -59,7 +56,12 @@ function getBody(field: FieldDefinition, tables: Table[]): string {
 
   if (field.type === "link_row") {
     const foreignTable = getForeignTable(field, tables);
-    return `return this.getLinkedRows(${field.link_row_table_id}, ${field.link_row_related_field_id}, ${foreignTable.name}Row);`;
+    const tableId = field.link_row_table_id;
+    const fieldId = field.link_row_related_field_id;
+    const foreignField = foreignTable.fields.find((f) => f.id === fieldId);
+    const fieldName = foreignField?.name;
+    const className = `${foreignTable.name}Row`;
+    return `return this.getLinkedRows(${tableId}, "${fieldName}", ${className});`;
   }
 
   if (field.type === "date" || field.formula_type === "date") {

src/row.spec.ts

@@ -4,7 +4,7 @@ import { describe, it, expect, vi } from "vitest";
 
 class MyRow extends Row {
   public async doSomething(): Promise<Row<RowType, Factory>[]> {
-    return this.getLinkedRows(1, 2, MyRow);
+    return this.getLinkedRows(1, "the_field", MyRow);
   }
 }
 
@@ -31,7 +31,7 @@ describe("Row", () => {
         filters: expect.objectContaining({
           filters: expect.arrayContaining([
             expect.objectContaining({
-              field: 2,
+              field: "the_field",
             }),
           ]),
         }),

src/row.ts

@@ -52,17 +52,16 @@ export abstract class Row<
 
   protected getLinkedRows<T extends Row, R extends RowType, F extends Factory>(
     tableId: number,
-    fieldId: number,
+    field: string,
     defaultClass: RowClass<R, F>,
   ): Promise<T[]> {
     return this.repository.getMany(tableId, defaultClass, {
-      user_field_names: false,
       filters: {
         filter_type: "AND",
         filters: [
           {
             type: "link_row_has",
-            field: fieldId,
+            field,
             value: this.getId().toString(),
           },
         ],

tsconfig.dev.json

@@ -1,7 +1,8 @@
 {
   "extends": "./tsconfig.json",
   "compilerOptions": {
-    "noEmit": true
+    "noEmit": true,
+    "outDir": null
   },
-  "exclude": ["dist", ".baserowrc"]
+  "exclude": ["./dist/**/*", ".baserowrc"]
 }

tsconfig.json

@@ -14,7 +14,7 @@
     "types": ["node"]
   },
   "exclude": [
-    "dist",
+    "./dist/**/*",
     "**/*.spec.ts",
     "vitest.config.ts",
     "vitest.setup.ts",