src/csv-loader/tests/parseCsv-typeDeclarations.test.ts

@@ -247,7 +247,7 @@ describe("parseCsv - type declarations", () => {
     expect(result.typeDefinition).toContain("export type IntentEffects =");
     // IntentEffect should reference IntentEffectTarget, not expand it
     expect(result.typeDefinition).toContain(
-      "[IntentEffectTarget, string, int]",
+      "[IntentEffectTarget; string; int]",
     );
     // IntentEffects should reference IntentEffect, not expand it
     expect(result.typeDefinition).toContain("IntentEffect[]");
@@ -281,32 +281,9 @@ describe("parseCsv - type declarations", () => {
     });
 
     expect(typeResult.typeDefinition).toContain("export type Type =");
-    expect(typeResult.typeDefinition).toContain("[Type, int][]");
+    expect(typeResult.typeDefinition).toContain("[Type; int][]");
     expect(typeResult.typeDefinition).toContain(
-      "readonly items: [Type, int][];",
-    );
-  });
-
-  it("should resolve @table references inside declared types to capitalized type names", () => {
-    const csv = [
-      "# type Entry = [@user]",
-      "id,entry",
-      "string,Entry",
-      "1,1",
-    ].join("\n");
-
-    const result = parseCsv(csv, {
-      emitTypes: true,
-      resourceName: "entries",
-      currentFilePath: path.join(fixturesDir, "entries.csv"),
-      resolveReferences: false,
-    });
-
-    expect(result.typeDefinition).toContain("export type Entry = [User]");
-    expect(result.typeDefinition).toContain("readonly entry: Entry;");
-    // It should also import the User type
-    expect(result.typeDefinition).toContain(
-      "import type { User } from './user.csv'",
+      "readonly items: [Type; int][];",
     );
   });
 });

src/csv-loader/type-gen.ts

@@ -2,20 +2,6 @@ import * as path from "path";
 import { schemaToTypeString } from "../index.js";
 import type { PropertyConfig, TypeDeclaration } from "./types.js";
 
-function resolveReferencesInSchemaString(
-  schemaString: string,
-  resourceNames: Map<string, string>,
-): string {
-  return schemaString
-    .replace(/@([a-zA-Z0-9\-_]+)(\[\])?/g, (_match, tableName, arraySuffix) => {
-      const typeName =
-        resourceNames.get(tableName) ||
-        tableName.charAt(0).toUpperCase() + tableName.slice(1);
-      return arraySuffix ? `${typeName}[]` : typeName;
-    })
-    .replace(/; ?/g, ", ");
-}
-
 /**
  * Generate TypeScript interface for the CSV data
  */
@@ -65,7 +51,7 @@ export function generateTypeDefinition(
       ? typeDeclarations
           .map(
             (decl) =>
-              `export type ${decl.name} = ${decl.schemaString ? resolveReferencesInSchemaString(decl.schemaString, resourceNames) : schemaToTypeString(decl.schema, resourceNames)};`,
+              `export type ${decl.name} = ${decl.schemaString ?? schemaToTypeString(decl.schema, resourceNames)};`,
           )
           .join("\n") + "\n\n"
       : "";
@@ -74,9 +60,8 @@ export function generateTypeDefinition(
     .map((config) => {
       const typeStr = config.declaredTypeName
         ? config.declaredTypeName
-        : config.schemaString
-          ? resolveReferencesInSchemaString(config.schemaString, resourceNames)
-          : schemaToTypeString(config.schema, resourceNames);
+        : (config.schemaString ??
+          schemaToTypeString(config.schema, resourceNames));
       return `    readonly ${config.name}: ${typeStr};`;
     })
     .join("\n");