fix(cli): fixes the issue that enhancer ignores zod schemas if it's explicitly configured in ZModel (#1632)

Author: ymc9

packages/schema/src/cli/plugin-runner.ts

@@ -183,12 +183,15 @@ export class PluginRunner {
         // 2. @core/enhancer
         const existingEnhancer = plugins.find((p) => p.provider === CorePlugins.Enhancer);
         if (existingEnhancer) {
+            // enhancer should load zod schemas if there're validation rules
+            existingEnhancer.options.withZodSchemas = hasValidation;
             corePlugins.push(existingEnhancer);
             plugins.splice(plugins.indexOf(existingEnhancer), 1);
         } else {
             if (options.defaultPlugins) {
                 corePlugins.push(
                     this.makeCorePlugin(CorePlugins.Enhancer, options.schemaPath, {
+                        // enhancer should load zod schemas if there're validation rules
                         withZodSchemas: hasValidation,
                     })
                 );

packages/testtools/src/schema.ts

@@ -336,9 +336,6 @@ export async function loadSchema(schema: string, options?: SchemaLoadOptions) {
                 prisma,
                 { user },
                 {
-                    policy,
-                    modelMeta,
-                    zodSchemas,
                     logPrismaQuery: opt.logPrismaQuery,
                     transactionTimeout: 1000000,
                     kinds: opt.enhancements,

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

@@ -0,0 +1,45 @@
+import { loadSchema } from '@zenstackhq/testtools';
+describe('issue 1562', () => {
+    it('regression', async () => {
+        const { enhance } = await loadSchema(
+            `
+generator client {
+    provider = "prisma-client-js"
+}
+
+datasource db {
+    provider = "sqlite"
+    url      = "file:./dev.db"
+}
+
+plugin zod {
+    provider = '@core/zod'
+}
+
+plugin enhancer {
+    provider = '@core/enhancer'
+    generatePermissionChecker = true
+}
+
+abstract model Base {
+    id        String   @id @default(uuid())
+    createdAt DateTime @default(now())
+    updatedAt DateTime @updatedAt()
+
+    // require login
+    @@allow('all', true)
+}
+
+model User extends Base {
+    name String @unique @regex('^[a-zA-Z0-9_]{3,30}$')
+
+    @@allow('read', true)
+}
+            `,
+            { addPrelude: false }
+        );
+
+        const db = enhance();
+        await expect(db.user.create({ data: { name: '1 2 3 4' } })).toBeRejectedByPolicy();
+    });
+});