rename directive and add errors

Author: danadajian

src/config.ts

@@ -82,8 +82,14 @@ export const configSchema = object({
           union([
             string(),
             object({
+              /**
+               * The name of the annotation to replace the directive with.
+               */
               annotationName: string(),
-              retainArguments: array(string()),
+              /**
+               * The arguments to retain from the directive. Can be INT, FLOAT, STRING, BOOLEAN, or ENUM.
+               */
+              argumentsToRetain: array(string()),
             }),
           ]),
         ),

src/helpers/build-directive-annotations.ts

@@ -60,13 +60,19 @@ export function buildDirectiveAnnotations(
       const kotlinAnnotations =
         directiveReplacementFromConfig.kotlinAnnotations.map((item) => {
           if (typeof item === "string") return item;
-          const directiveArguments = item.retainArguments
+          const directiveArguments = item.argumentsToRetain
             ?.map((argumentToRetain) => {
               const argumentValueNode = directive.arguments?.find(
                 (argument) => argument.name.value === argumentToRetain,
               )?.value;
-              if (!argumentValueNode) throw new Error();
-              if (!("value" in argumentValueNode)) throw new Error();
+              if (!argumentValueNode)
+                throw new Error(
+                  `Argument ${argumentToRetain} was provided in argumentsToRetain config but was not found in directive ${directiveName}`,
+                );
+              if (!("value" in argumentValueNode))
+                throw new Error(
+                  `Directive argument ${argumentToRetain} in directive ${directiveName} has an unsupported type. Only INT, FLOAT, STRING, BOOLEAN, and ENUM are supported.`,
+                );
               const argumentValue =
                 argumentValueNode.kind === "StringValue"
                   ? `"${argumentValueNode.value}"`

test/unit/should_honor_directiveReplacements_config/codegen.config.ts

@@ -7,11 +7,11 @@ export default {
       kotlinAnnotations: ["@SomeAnnotation1", "@SomeAnnotation2"],
     },
     {
-      directive: "directive2",
+      directive: "directiveWithArgs",
       kotlinAnnotations: [
         {
           annotationName: "SomeAnnotation3",
-          retainArguments: ["arg1", "arg2"],
+          argumentsToRetain: ["arg1", "arg2"],
         },
       ],
     },

test/unit/should_honor_directiveReplacements_config/schema.graphql

@@ -1,8 +1,8 @@
 directive @directive1 on OBJECT | UNION
-directive @directive2(arg1: String, arg2: Int) on OBJECT | UNION
+directive @directiveWithArgs(arg1: String, arg2: Int) on OBJECT | UNION
 
 "A description for MyDirectiveType"
-type MyDirectiveType @directive1 @directive2(arg1: "arg1", arg2: 0) {
+type MyDirectiveType @directive1 @directiveWithArgs(arg1: "arg1", arg2: 0) {
   field: String
 }