@pothos/plugin-directives - npm Package Compare versions

Comparing version 3.9.1 to 3.9.2

CHANGELOG.md

 # Change Log
 
+## 3.9.2
+
+### Patch Changes
+
+- 0eb6cb94: Fix bug that ignored directives added directly via extensions in some locations
+
 ## 3.9.1
@@ -4,0 +10,0 @@

esm/index.js

@@ -23,5 +23,5 @@ /* eslint-disable no-param-reassign */ import './global-types.js';
     onInputFieldConfig(fieldConfig) {
-        var _fieldConfig_extensions;
+        var _fieldConfig_extensions, _fieldConfig_extensions1;
         const options = fieldConfig.pothosOptions;
-        if (!options.directives) {
+        if (!options.directives && !((_fieldConfig_extensions = fieldConfig.extensions) === null || _fieldConfig_extensions === void 0 ? void 0 : _fieldConfig_extensions.directives)) {
             return fieldConfig;
@@ -33,3 +33,3 @@ }
                 ...fieldConfig.extensions,
-                directives: this.normalizeDirectives(this.mergeDirectives((_fieldConfig_extensions = fieldConfig.extensions) === null || _fieldConfig_extensions === void 0 ? void 0 : _fieldConfig_extensions.directives, options.directives))
+                directives: this.normalizeDirectives(this.mergeDirectives((_fieldConfig_extensions1 = fieldConfig.extensions) === null || _fieldConfig_extensions1 === void 0 ? void 0 : _fieldConfig_extensions1.directives, options.directives))
             }
@@ -39,5 +39,5 @@ };
     onEnumValueConfig(valueConfig) {
-        var _valueConfig_extensions;
+        var _valueConfig_extensions, _valueConfig_extensions1;
         const options = valueConfig.pothosOptions;
-        if (!options.directives) {
+        if (!options.directives && !((_valueConfig_extensions = valueConfig.extensions) === null || _valueConfig_extensions === void 0 ? void 0 : _valueConfig_extensions.directives)) {
             return valueConfig;
@@ -49,3 +49,3 @@ }
                 ...valueConfig.extensions,
-                directives: this.normalizeDirectives(this.mergeDirectives((_valueConfig_extensions = valueConfig.extensions) === null || _valueConfig_extensions === void 0 ? void 0 : _valueConfig_extensions.directives, options.directives))
+                directives: this.normalizeDirectives(this.mergeDirectives((_valueConfig_extensions1 = valueConfig.extensions) === null || _valueConfig_extensions1 === void 0 ? void 0 : _valueConfig_extensions1.directives, options.directives))
             }
@@ -55,5 +55,5 @@ };
     onTypeConfig(typeConfig) {
-        var _typeConfig_extensions;
+        var _typeConfig_extensions, _typeConfig_extensions1;
         const options = typeConfig.pothosOptions;
-        if (!options.directives) {
+        if (!options.directives && !((_typeConfig_extensions = typeConfig.extensions) === null || _typeConfig_extensions === void 0 ? void 0 : _typeConfig_extensions.directives)) {
             return typeConfig;
@@ -65,3 +65,3 @@ }
                 ...typeConfig.extensions,
-                directives: this.normalizeDirectives(this.mergeDirectives((_typeConfig_extensions = typeConfig.extensions) === null || _typeConfig_extensions === void 0 ? void 0 : _typeConfig_extensions.directives, options.directives))
+                directives: this.normalizeDirectives(this.mergeDirectives((_typeConfig_extensions1 = typeConfig.extensions) === null || _typeConfig_extensions1 === void 0 ? void 0 : _typeConfig_extensions1.directives, options.directives))
             }
@@ -68,0 +68,0 @@ };

lib/index.js

@@ -92,5 +92,5 @@ /* eslint-disable no-param-reassign */ "use strict";
     onInputFieldConfig(fieldConfig) {
-        var _fieldConfig_extensions;
+        var _fieldConfig_extensions, _fieldConfig_extensions1;
         const options = fieldConfig.pothosOptions;
-        if (!options.directives) {
+        if (!options.directives && !((_fieldConfig_extensions = fieldConfig.extensions) === null || _fieldConfig_extensions === void 0 ? void 0 : _fieldConfig_extensions.directives)) {
             return fieldConfig;
@@ -102,3 +102,3 @@ }
                 ...fieldConfig.extensions,
-                directives: this.normalizeDirectives(this.mergeDirectives((_fieldConfig_extensions = fieldConfig.extensions) === null || _fieldConfig_extensions === void 0 ? void 0 : _fieldConfig_extensions.directives, options.directives))
+                directives: this.normalizeDirectives(this.mergeDirectives((_fieldConfig_extensions1 = fieldConfig.extensions) === null || _fieldConfig_extensions1 === void 0 ? void 0 : _fieldConfig_extensions1.directives, options.directives))
             }
@@ -108,5 +108,5 @@ };
     onEnumValueConfig(valueConfig) {
-        var _valueConfig_extensions;
+        var _valueConfig_extensions, _valueConfig_extensions1;
         const options = valueConfig.pothosOptions;
-        if (!options.directives) {
+        if (!options.directives && !((_valueConfig_extensions = valueConfig.extensions) === null || _valueConfig_extensions === void 0 ? void 0 : _valueConfig_extensions.directives)) {
             return valueConfig;
@@ -118,3 +118,3 @@ }
                 ...valueConfig.extensions,
-                directives: this.normalizeDirectives(this.mergeDirectives((_valueConfig_extensions = valueConfig.extensions) === null || _valueConfig_extensions === void 0 ? void 0 : _valueConfig_extensions.directives, options.directives))
+                directives: this.normalizeDirectives(this.mergeDirectives((_valueConfig_extensions1 = valueConfig.extensions) === null || _valueConfig_extensions1 === void 0 ? void 0 : _valueConfig_extensions1.directives, options.directives))
             }
@@ -124,5 +124,5 @@ };
     onTypeConfig(typeConfig) {
-        var _typeConfig_extensions;
+        var _typeConfig_extensions, _typeConfig_extensions1;
         const options = typeConfig.pothosOptions;
-        if (!options.directives) {
+        if (!options.directives && !((_typeConfig_extensions = typeConfig.extensions) === null || _typeConfig_extensions === void 0 ? void 0 : _typeConfig_extensions.directives)) {
             return typeConfig;
@@ -134,3 +134,3 @@ }
                 ...typeConfig.extensions,
-                directives: this.normalizeDirectives(this.mergeDirectives((_typeConfig_extensions = typeConfig.extensions) === null || _typeConfig_extensions === void 0 ? void 0 : _typeConfig_extensions.directives, options.directives))
+                directives: this.normalizeDirectives(this.mergeDirectives((_typeConfig_extensions1 = typeConfig.extensions) === null || _typeConfig_extensions1 === void 0 ? void 0 : _typeConfig_extensions1.directives, options.directives))
             }
@@ -137,0 +137,0 @@ };

package.json

 {
   "name": "@pothos/plugin-directives",
-  "version": "3.9.1",
+  "version": "3.9.2",
   "description": "Directive plugin for Pothos, enables using graphql-tools based directives with Pothos",
@@ -40,3 +40,3 @@ "main": "./lib/index.js",
   "devDependencies": {
-    "@graphql-tools/utils": "^9.1.1",
+    "@graphql-tools/utils": "^9.2.1",
     "graphql": "16.6.0",
@@ -46,3 +46,3 @@ "graphql-rate-limit-directive": "^2.0.3",
     "rate-limiter-flexible": "^2.4.1",
-    "@pothos/core": "3.23.1",
+    "@pothos/core": "3.24.1",
     "@pothos/test-utils": "1.4.7"
@@ -49,0 +49,0 @@ },

src/index.ts

@@ -45,3 +45,3 @@ /* eslint-disable no-param-reassign */
 
-    if (!options.directives) {
+    if (!options.directives && !fieldConfig.extensions?.directives) {
       return fieldConfig;
@@ -67,3 +67,3 @@ }
 
-    if (!options.directives) {
+    if (!options.directives && !valueConfig.extensions?.directives) {
       return valueConfig;
@@ -89,3 +89,3 @@ }
 
-    if (!options.directives) {
+    if (!options.directives && !typeConfig.extensions?.directives) {
       return typeConfig;
@@ -92,0 +92,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

118004

increased by1.58%

No dependency changes