fix(tanstack,swr): create/upsert hooks shouldn't be generated for delegate models (#1567)

Author: ymc9

packages/plugins/swr/src/generator.ts

@@ -5,6 +5,7 @@ import {
     ensureEmptyDir,
     generateModelMeta,
     getDataModels,
+    isDelegateModel,
     requireOption,
     resolvePath,
     saveProject,
@@ -77,15 +78,17 @@ function generateModelHooks(
 
     const mutationFuncs: string[] = [];
 
+    // Note: delegate models don't support create and upsert operations
+
     // create is somehow named "createOne" in the DMMF
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    if (mapping.create || (mapping as any).createOne) {
+    if (!isDelegateModel(model) && (mapping.create || (mapping as any).createOne)) {
         const argsType = `Prisma.${model.name}CreateArgs`;
         mutationFuncs.push(generateMutation(sf, model, 'POST', 'create', argsType, false));
     }
 
     // createMany
-    if (mapping.createMany && supportCreateMany(model.$container)) {
+    if (!isDelegateModel(model) && mapping.createMany && supportCreateMany(model.$container)) {
         const argsType = `Prisma.${model.name}CreateManyArgs`;
         mutationFuncs.push(generateMutation(sf, model, 'POST', 'createMany', argsType, true));
     }
@@ -138,7 +141,7 @@ function generateModelHooks(
     // upsert
     // upsert is somehow named "upsertOne" in the DMMF
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    if (mapping.upsert || (mapping as any).upsertOne) {
+    if (!isDelegateModel(model) && (mapping.upsert || (mapping as any).upsertOne)) {
         const argsType = `Prisma.${model.name}UpsertArgs`;
         mutationFuncs.push(generateMutation(sf, model, 'POST', 'upsert', argsType, false));
     }

packages/plugins/tanstack-query/src/generator.ts

@@ -6,6 +6,7 @@ import {
     ensureEmptyDir,
     generateModelMeta,
     getDataModels,
+    isDelegateModel,
     requireOption,
     resolvePath,
     saveProject,
@@ -341,14 +342,16 @@ function generateModelHooks(
     });
     sf.addStatements(makeBaseImports(target, version));
 
+    // Note: delegate models don't support create and upsert operations
+
     // create is somehow named "createOne" in the DMMF
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    if (mapping.create || (mapping as any).createOne) {
+    if (!isDelegateModel(model) && (mapping.create || (mapping as any).createOne)) {
         generateMutationHook(target, sf, model.name, 'create', 'post', true);
     }
 
     // createMany
-    if (mapping.createMany && supportCreateMany(model.$container)) {
+    if (!isDelegateModel(model) && mapping.createMany && supportCreateMany(model.$container)) {
         generateMutationHook(target, sf, model.name, 'createMany', 'post', false, 'Prisma.BatchPayload');
     }
 
@@ -422,7 +425,7 @@ function generateModelHooks(
     // upsert
     // upsert is somehow named "upsertOne" in the DMMF
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    if (mapping.upsert || (mapping as any).upsertOne) {
+    if (!isDelegateModel(model) && (mapping.upsert || (mapping as any).upsertOne)) {
         generateMutationHook(target, sf, model.name, 'upsert', 'post', true);
     }
 

tests/integration/tests/enhancements/with-delegate/plugin-interaction.test.ts

@@ -23,19 +23,36 @@ describe('Polymorphic Plugin Interaction Test', () => {
         });
     });
 
+    it('swr', async () => {
+        const schema = `
+        ${POLYMORPHIC_SCHEMA}
+
+        plugin hooks {
+            provider = '@zenstackhq/swr'
+            output = '$projectRoot/hooks'
+        }
+        `;
+
+        await loadSchema(schema, {
+            compile: true,
+            extraDependencies: ['swr'],
+        });
+    });
+
     it('trpc', async () => {
         const schema = `
         ${POLYMORPHIC_SCHEMA}
 
         plugin trpc {
             provider = '@zenstackhq/trpc'
             output = '$projectRoot/routers'
+            generateClientHelpers = 'react'
         }
         `;
 
         await loadSchema(schema, {
             compile: true,
-            extraDependencies: ['@trpc/client', '@trpc/server'],
+            extraDependencies: ['@trpc/client', '@trpc/server', '@trpc/react-query'],
         });
     });
 });