fix: prisma schema generation issue with default() using auth()

fixes #274

Author: ymc9

packages/sdk/src/prisma/prisma-schema-generator.ts

@@ -18,6 +18,7 @@ import {
     InvocationExpr,
     isArrayExpr,
     isDataModel,
+    isExpression,
     isInvocationExpr,
     isLiteralExpr,
     isModel,
@@ -31,7 +32,7 @@ import {
     StringLiteral,
     type AstNode,
 } from '@zenstackhq/language/ast';
-import { getAllAttributes, getAllFields, isDelegateModel } from '@zenstackhq/language/utils';
+import { getAllAttributes, getAllFields, isAuthInvocation, isDelegateModel } from '@zenstackhq/language/utils';
 import { AstUtils } from 'langium';
 import { match } from 'ts-pattern';
 import { ModelUtils, ZModelCodeGenerator } from '..';
@@ -242,8 +243,8 @@ export class PrismaSchemaGenerator {
 
         const attributes = field.attributes
             .filter((attr) => this.isPrismaAttribute(attr))
-            // `@default` with calling functions from plugin is handled outside Prisma
-            .filter((attr) => !this.isDefaultWithPluginInvocation(attr))
+            // `@default` using `auth()` is handled outside Prisma
+            .filter((attr) => !this.isDefaultWithAuthInvocation(attr))
             .filter(
                 (attr) =>
                     // when building physical schema, exclude `@default` for id fields inherited from delegate base
@@ -260,7 +261,7 @@ export class PrismaSchemaGenerator {
         return result;
     }
 
-    private isDefaultWithPluginInvocation(attr: DataFieldAttribute) {
+    private isDefaultWithAuthInvocation(attr: DataFieldAttribute) {
         if (attr.decl.ref?.name !== '@default') {
             return false;
         }
@@ -270,7 +271,7 @@ export class PrismaSchemaGenerator {
             return false;
         }
 
-        return AstUtils.streamAst(expr).some((node) => isInvocationExpr(node) && this.isFromPlugin(node.function.ref));
+        return AstUtils.streamAst(expr).some((node) => isExpression(node) && isAuthInvocation(node));
     }
 
     private isFromPlugin(node: AstNode | undefined) {

packages/testtools/src/project.ts

@@ -2,7 +2,7 @@ import fs from 'node:fs';
 import path from 'node:path';
 import tmp from 'tmp';
 
-export function createTestProject() {
+export function createTestProject(zmodelContent?: string) {
     const { name: workDir } = tmp.dirSync({ unsafeCleanup: true });
 
     fs.mkdirSync(path.join(workDir, 'node_modules'));
@@ -63,5 +63,9 @@ export function createTestProject() {
         ),
     );
 
+    if (zmodelContent) {
+        fs.writeFileSync(path.join(workDir, 'schema.zmodel'), zmodelContent);
+    }
+
     return workDir;
 }

tests/regression/test/issue-204/regression.test.ts

@@ -1,11 +1,9 @@
-import { describe, it } from 'vitest';
+import { it } from 'vitest';
 import { type Configuration, ShirtColor } from './models';
 
-describe('Issue 204 regression tests', () => {
-    it('tests issue 204', () => {
-        const config: Configuration = { teamColors: [ShirtColor.Black, ShirtColor.Blue] };
-        console.log(config.teamColors?.[0]);
-        const config1: Configuration = {};
-        console.log(config1);
-    });
+it('tests issue 204', () => {
+    const config: Configuration = { teamColors: [ShirtColor.Black, ShirtColor.Blue] };
+    console.log(config.teamColors?.[0]);
+    const config1: Configuration = {};
+    console.log(config1);
 });

tests/regression/test/issue-274/regression.test.ts

@@ -0,0 +1,29 @@
+import { createTestProject } from '@zenstackhq/testtools';
+import { execSync } from 'child_process';
+import { it } from 'vitest';
+
+it('tests issue 274', async () => {
+    const dir = await createTestProject(`
+
+datasource db {
+  provider = 'sqlite'
+  url      = "file:./test.db"
+}
+
+model Comment {  
+  id            String       @id
+  author        User?     @relation(fields: [authorId], references: [id])
+  authorId      String?   @default(auth().id) 
+}
+
+model User {
+    id            String       @id
+    email         String
+    comments      Comment[]    
+}
+`);
+
+    process.chdir(dir);
+
+    execSync('node node_modules/@zenstackhq/cli/dist/index.js migrate dev --name init');
+});