eslint-plugin-class-validator-type-match 1.6.0 → 2.0.0

package/dist/rules/decorator-type-match.js

@@ -879,6 +879,7 @@ exports.default = createRule({
                  * - Not initialized (no = value)
                  * - Don't have undefined in their type
                  * Should use the definite assignment assertion (!)
+                 * Note: Properties with | null still require ! unless they also have ?
                  */
                 const isOptionalProperty = node.optional === true;
                 const hasInitializer = node.value !== undefined;

package/dist/rules/optional-decorator-match.js

@@ -322,10 +322,11 @@ exports.default = createRule({
                 }
                 /**
                  * Priority 3: Null in type with @IsOptional() (likely incorrect)
-                 * Note: ? adds undefined, not null
-                 * Only flag if there's null but NO undefined (neither explicit nor from ?)
+                 * @IsOptional() works with undefined, not null
+                 * Flag if there's null in the type (with or without ?)
+                 * However, if it already has undefined explicitly, we won't flag it as user might intentionally want both
                  */
-                if (hasDecorator && hasNull && !hasUndefined && !isOptionalProperty) {
+                if (hasDecorator && hasNull && !hasUndefined) {
                     const propertyTypeWithoutNull = getPropertyTypeWithoutNull(node);
                     issues.push({
                         messageId: 'nullUnionIncorrect',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-class-validator-type-match",
-  "version": "1.6.0",
+  "version": "2.0.0",
   "description": "ESLint plugin to ensure class-validator decorators match TypeScript type annotations",
   "keywords": [
     "eslint",