feat: implement new rule to check illogical usage of composition keywords

feat: add proper check for mutually exclusive schemas in oneOf

feat: add checks for object min/max props, required props and additionlProperties

feat: add rule to the oas3_2 spec

refactor: functions naming and returning type

refactor: change if statement

refactor: move areDuplicatedSchemas function to utility and change naming

feat: add new rule no-illogical-any-of-usage

chore: add new rule to type

fix: getEffectiveBounds functionality, pattern check and mutualExclusibity check

feat: add functionality to report more then 1 problem

chore: improve warning messages

feat: add proper checks, refactor and change error messages

fix: error message in areDuplicatedSchemas

refactor: move logic into main function, add helper function, rename utility function

refactor: change naming and functionality for duplicated shcemas function

refactor: nullable function and return statement

fix: nullable type detection

refactor: add utility functions

refactor: remove functions, clean code, refactor additionalProperties

refactor: functions naming, if statement

chore: remove oneOf/anyOf/allOf separate rules and add one to handle different behavior

feat: add proper handling for additionalProperties

chore: update type

feat: add functionality to handle anyOf and allOf checks

Author: AlbinaBlazhko17

packages/core/src/config/all.ts

@@ -117,6 +117,7 @@ const all: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'error',
     'tags-alphabetical': 'error',
     'no-duplicated-tag-names': 'error',
+    'no-illogical-composition-keywords': 'error',
   },
   oas3_1Rules: {
     'array-parameter-serialization': 'error',
@@ -179,6 +180,7 @@ const all: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'error',
     'tags-alphabetical': 'error',
     'no-duplicated-tag-names': 'error',
+    'no-illogical-composition-keywords': 'error',
   },
   oas3_2Rules: {
     'array-parameter-serialization': 'error',
@@ -241,6 +243,7 @@ const all: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'error',
     'tags-alphabetical': 'error',
     'no-duplicated-tag-names': 'error',
+    'no-illogical-composition-keywords': 'error',
   },
   async2Rules: {
     'channels-kebab-case': 'error',

packages/core/src/config/minimal.ts

@@ -108,6 +108,7 @@ const minimal: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'warn',
     'tags-alphabetical': 'off',
     'no-duplicated-tag-names': 'off',
+    'no-illogical-composition-keywords': 'off',
   },
   oas3_1Rules: {
     'array-parameter-serialization': 'off',
@@ -164,6 +165,7 @@ const minimal: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'warn',
     'tags-alphabetical': 'off',
     'no-duplicated-tag-names': 'off',
+    'no-illogical-composition-keywords': 'off',
   },
   oas3_2Rules: {
     'array-parameter-serialization': 'off',
@@ -220,6 +222,7 @@ const minimal: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'warn',
     'tags-alphabetical': 'off',
     'no-duplicated-tag-names': 'off',
+    'no-illogical-composition-keywords': 'off',
   },
   async2Rules: {
     'channels-kebab-case': 'off',

packages/core/src/config/recommended-strict.ts

@@ -108,6 +108,7 @@ const recommendedStrict: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'error',
     'tags-alphabetical': 'off',
     'no-duplicated-tag-names': 'error',
+    'no-illogical-composition-keywords': 'error',
   },
   oas3_1Rules: {
     'array-parameter-serialization': 'off',
@@ -164,6 +165,7 @@ const recommendedStrict: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'error',
     'tags-alphabetical': 'off',
     'no-duplicated-tag-names': 'error',
+    'no-illogical-composition-keywords': 'error',
   },
   oas3_2Rules: {
     'array-parameter-serialization': 'off',
@@ -220,6 +222,7 @@ const recommendedStrict: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'error',
     'tags-alphabetical': 'off',
     'no-duplicated-tag-names': 'error',
+    'no-illogical-composition-keywords': 'error',
   },
   async2Rules: {
     'channels-kebab-case': 'off',

packages/core/src/config/recommended.ts

@@ -108,6 +108,7 @@ const recommended: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'warn',
     'tags-alphabetical': 'off',
     'no-duplicated-tag-names': 'warn',
+    'no-illogical-composition-keywords': 'warn',
   },
   oas3_1Rules: {
     'array-parameter-serialization': 'off',
@@ -164,6 +165,7 @@ const recommended: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'warn',
     'tags-alphabetical': 'off',
     'no-duplicated-tag-names': 'warn',
+    'no-illogical-composition-keywords': 'warn',
   },
   oas3_2Rules: {
     'array-parameter-serialization': 'off',
@@ -220,6 +222,7 @@ const recommended: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'warn',
     'tags-alphabetical': 'off',
     'no-duplicated-tag-names': 'warn',
+    'no-illogical-composition-keywords': 'warn',
   },
   async2Rules: {
     'channels-kebab-case': 'off',

packages/core/src/config/spec.ts

@@ -108,6 +108,7 @@ const spec: RawGovernanceConfig<'built-in'> = {
     'tag-description': 'off',
     'tags-alphabetical': 'off',
     'nullable-type-sibling': 'error',
+    'no-illogical-composition-keywords': 'off',
   },
   oas3_1Rules: {
     'array-parameter-serialization': 'off',
@@ -164,6 +165,7 @@ const spec: RawGovernanceConfig<'built-in'> = {
     'spec-components-invalid-map-name': 'error',
     'tag-description': 'off',
     'tags-alphabetical': 'off',
+    'no-illogical-composition-keywords': 'error',
   },
   oas3_2Rules: {
     'array-parameter-serialization': 'off',
@@ -220,6 +222,7 @@ const spec: RawGovernanceConfig<'built-in'> = {
     'spec-components-invalid-map-name': 'error',
     'tag-description': 'off',
     'tags-alphabetical': 'off',
+    'no-illogical-composition-keywords': 'error',
   },
   async2Rules: {
     'channels-kebab-case': 'off',

packages/core/src/rules/oas3/index.ts

@@ -55,6 +55,7 @@ import { NoRequiredSchemaPropertiesUndefined } from '../common/no-required-schem
 import { NoSchemaTypeMismatch } from '../common/no-schema-type-mismatch.js';
 import { NoDuplicatedTagNames } from '../common/no-duplicated-tag-names.js';
 import { NullableTypeSibling } from './nullable-type-sibling.js';
+import { NoIllogicalCompositionKeywords } from './no-illogical-composition-keywords.js';
 
 import type { Oas3RuleSet } from '../../oas-types.js';
 import type { Oas3Rule } from '../../visitors.js';
@@ -118,6 +119,7 @@ export const rules: Oas3RuleSet<'built-in'> = {
   'no-schema-type-mismatch': NoSchemaTypeMismatch as Oas3Rule,
   'no-duplicated-tag-names': NoDuplicatedTagNames as Oas3Rule,
   'nullable-type-sibling': NullableTypeSibling,
+  'no-illogical-composition-keywords': NoIllogicalCompositionKeywords,
 };
 
 export const preprocessors = {};