fix: inject and optional decorators should allow key to be undefined (#…

…2329)
Urigo · Mar 22, 2023 · df7e2db · df7e2db
1 parent e454a1d
commit df7e2db
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/.changeset/lazy-bulldogs-cry.md b/.changeset/lazy-bulldogs-cry.md
@@ -0,0 +1,5 @@
+---
+'graphql-modules': patch
+---
+
+TypeScript 5 support for the Inject and Optional decorators
diff --git a/packages/graphql-modules/src/di/decorators.ts b/packages/graphql-modules/src/di/decorators.ts
@@ -51,6 +51,13 @@ export function Injectable(options?: ProviderOptions): ClassDecorator {
   };
 }
 
+// https://github.com/microsoft/TypeScript/issues/52435
+type ParameterDecorator = (
+  target: Object,
+  propertyKey: string | symbol | undefined,
+  parameterIndex: number
+) => void;
+
 export function Optional(): ParameterDecorator {
   return (target, _, index) => {
     ensureReflect();