bugfix: force required args (#21)

thuoe · Mar 18, 2024 · 847d6fd · 847d6fd
1 parent 71b880a
commit 847d6fd
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/directives/cache.ts b/src/directives/cache.ts
@@ -27,7 +27,7 @@ const inMemoryCache: CachingImpl = {
 
 const cacheDirective = ({ directiveName = 'cache', cache = inMemoryCache }: Partial<Params> = {}) => {
   return {
-    cacheDirectiveTypeDefs: `directive @${directiveName}(key: String, ttl: Int) on FIELD_DEFINITION`,
+    cacheDirectiveTypeDefs: `directive @${directiveName}(key: String!, ttl: Int!) on FIELD_DEFINITION`,
     cacheDirectiveTransformer: (schema: GraphQLSchema) => mapSchema(schema, {
       [MapperKind.OBJECT_FIELD]: fieldConfig => {
         const { resolve = defaultFieldResolver } = fieldConfig

diff --git a/src/directives/encode.ts b/src/directives/encode.ts
@@ -4,7 +4,7 @@ import { GraphQLError, GraphQLSchema, defaultFieldResolver } from 'graphql'
 
 const encodingDirective = (directiveName: string = 'encode') => {
   return {
-    encodingDirectiveTypeDefs: `directive @${directiveName}(method: String) on FIELD_DEFINITION`,
+    encodingDirectiveTypeDefs: `directive @${directiveName}(method: String!) on FIELD_DEFINITION`,
     encodingDirectiveTransformer: (schema: GraphQLSchema) => mapSchema(schema, {
       [MapperKind.OBJECT_FIELD]: fieldConfig => {
         const encodingDirective = fetchDirective<{ method: string }>(schema, fieldConfig, directiveName)

diff --git a/src/directives/regex.ts b/src/directives/regex.ts
@@ -5,7 +5,7 @@ import { fetchDirective } from '@src/utils'
 
 const regexDirective = (directiveName: string = 'regex') => {
   return {
-    regexDirectiveTypeDefs: `directive @${directiveName}(pattern: String) on FIELD_DEFINITION`,
+    regexDirectiveTypeDefs: `directive @${directiveName}(pattern: String!) on FIELD_DEFINITION`,
     regexDirectiveTransformer: (schema: GraphQLSchema) => mapSchema(schema, {
       [MapperKind.OBJECT_FIELD]: fieldConfig => {
         const regexDirective = fetchDirective<{ pattern: string }>(schema, fieldConfig, directiveName)