feat: add quick fix to add @Suppress directive

spec/model/implementation/validation-utils.ts

@@ -81,7 +81,7 @@ export function expectQuickFix(
     const result = validate(component);
     const quickfixes = result.messages.flatMap((m) => m.quickFixes);
     expect(quickfixes.map((q) => q.description)).to.include(expectedDescription);
-    const matchingQuickfixes = quickfixes.filter((q) => q.description);
+    const matchingQuickfixes = quickfixes.filter((q) => q.description === expectedDescription);
     expect(matchingQuickfixes).to.have.a.lengthOf(1);
     const quickfix = matchingQuickfixes[0];
     const changeSet = quickfix.getChangeSet();

spec/model/validation/suppress/quick-fix.spec.ts

@@ -0,0 +1,218 @@
+import { expectQuickFix } from '../../implementation/validation-utils';
+import gql from 'graphql-tag';
+import { Project } from '../../../../src/project/project';
+
+describe('@suppress quick fix', () => {
+    it('generates a quick fix on type level', () => {
+        const project = new Project([
+            gql`
+                type Stuff @rootEntity {
+                    foo: String
+                }
+                # comment before type
+                type Child @childEntity {
+                    # comment in type
+                    stuff: Int
+                }
+            `.loc!.source,
+        ]);
+        expectQuickFix(
+            project,
+            'Suppress this warning',
+            `
+                type Stuff @rootEntity {
+                    foo: String
+                }
+                # comment before type
+                type Child @childEntity @suppress(warnings: UNUSED) {
+                    # comment in type
+                    stuff: Int
+                }
+            `,
+        );
+    });
+
+    it('generates a quick fix on enum level', () => {
+        const project = new Project([
+            gql`
+                type Stuff @rootEntity {
+                    value: myenum
+                }
+
+                enum myenum {
+                    NAME1
+                    NAME2
+                }
+            `.loc!.source,
+        ]);
+        expectQuickFix(
+            project,
+            'Suppress this warning',
+            `
+                type Stuff @rootEntity {
+                    value: myenum
+                }
+
+                enum myenum @suppress(warnings: NAMING) {
+                    NAME1
+                    NAME2
+                }
+            `,
+        );
+    });
+
+    it('generates a quick fix on enum value level', () => {
+        const project = new Project([
+            gql`
+                type Stuff @rootEntity {
+                    value: MyEnum
+                }
+
+                enum MyEnum {
+                    NAME1
+                    name2
+                }
+            `.loc!.source,
+        ]);
+        expectQuickFix(
+            project,
+            'Suppress this warning',
+            `
+                type Stuff @rootEntity {
+                    value: MyEnum
+                }
+
+                enum MyEnum {
+                    NAME1
+                    name2 @suppress(warnings: NAMING)
+                }
+            `,
+        );
+    });
+
+    it('generates a quick fix on field level', () => {
+        const project = new Project([
+            gql`
+                type Stuff @rootEntity {
+                    _key: String @key # after field
+                    key: String
+                }
+            `.loc!.source,
+        ]);
+        expectQuickFix(
+            project,
+            'Suppress this warning',
+            `
+                type Stuff @rootEntity {
+                    _key: String @key @suppress(warnings: DEPRECATED) # after field
+                    key: String
+                }
+            `,
+        );
+    });
+
+    it('generates a quick fix if there already is an empty @suppress directive', () => {
+        const project = new Project([
+            gql`
+                type Stuff @rootEntity {
+                    _key: String @key @suppress
+                    key: String
+                }
+            `.loc!.source,
+        ]);
+        expectQuickFix(
+            project,
+            'Suppress this warning',
+            `
+                type Stuff @rootEntity {
+                    _key: String @key @suppress(warnings: DEPRECATED)
+                    key: String
+                }
+            `,
+        );
+    });
+
+    it('generates a quick fix if there already is a @suppress directive with a different arg', () => {
+        const project = new Project([
+            gql`
+                type Stuff @rootEntity {
+                    _key: String @key @suppress(infos: NO_TYPE_CHECKS)
+                    key: String
+                }
+            `.loc!.source,
+        ]);
+        expectQuickFix(
+            project,
+            'Suppress this warning',
+            `
+                type Stuff @rootEntity {
+                    _key: String @key @suppress(infos: NO_TYPE_CHECKS, warnings: DEPRECATED)
+                    key: String
+                }
+            `,
+        );
+    });
+
+    it('generates a quick fix if there already is a @suppress directive with a single code', () => {
+        const project = new Project([
+            gql`
+                type Stuff @rootEntity {
+                    _key: String @key @suppress(warnings: UNUSED)
+                    key: String
+                }
+            `.loc!.source,
+        ]);
+        expectQuickFix(
+            project,
+            'Suppress this warning',
+            `
+                type Stuff @rootEntity {
+                    _key: String @key @suppress(warnings: [UNUSED, DEPRECATED])
+                    key: String
+                }
+            `,
+        );
+    });
+
+    it('generates a quick fix if there already is a @suppress directive with a single code as a list', () => {
+        const project = new Project([
+            gql`
+                type Stuff @rootEntity {
+                    _key: String @key @suppress(warnings: [UNUSED])
+                    key: String
+                }
+            `.loc!.source,
+        ]);
+        expectQuickFix(
+            project,
+            'Suppress this warning',
+            `
+                type Stuff @rootEntity {
+                    _key: String @key @suppress(warnings: [UNUSED, DEPRECATED])
+                    key: String
+                }
+            `,
+        );
+    });
+
+    it('generates a quick fix if there already is a @suppress directive with a multiple codes as a list', () => {
+        const project = new Project([
+            gql`
+                type Stuff @rootEntity {
+                    _key: String @key @suppress(warnings: [UNUSED, NAMING])
+                    key: String
+                }
+            `.loc!.source,
+        ]);
+        expectQuickFix(
+            project,
+            'Suppress this warning',
+            `
+                type Stuff @rootEntity {
+                    _key: String @key @suppress(warnings: [UNUSED, NAMING, DEPRECATED])
+                    key: String
+                }
+            `,
+        );
+    });
+});

src/model/change-set/change-set.ts

@@ -15,6 +15,8 @@ export class ChangeSet {
         this.textChanges = changes.filter((c) => c instanceof TextChange);
         this.appendChanges = changes.filter((c) => c instanceof AppendChange);
     }
+
+    static EMPTY = new ChangeSet([]);
 }
 
 /**

src/model/create-model.ts

@@ -124,7 +124,7 @@ import { parseI18nConfigs } from './parse-i18n';
 import { parseModuleConfigs } from './parse-modules';
 import { parseTTLConfigs } from './parse-ttl';
 import { ValidationContext, ValidationMessage } from './validation';
-import { WarningCode } from '../schema/message-codes';
+import { WarningCode } from './validation/suppress/message-codes';
 
 export function createModel(parsedProject: ParsedProject, options: ModelOptions = {}): Model {
     const validationContext = new ValidationContext();

src/model/implementation/enum-type.ts

@@ -6,7 +6,7 @@ import { EnumValueLocalization } from './i18n';
 import { Model } from './model';
 import { TypeBase } from './type-base';
 import memorize from 'memorize-decorator';
-import { WarningCode } from '../../schema/message-codes';
+import { WarningCode } from '../validation/suppress/message-codes';
 
 export class EnumType extends TypeBase {
     constructor(input: EnumTypeConfig, model: Model) {

src/model/implementation/field.ts

@@ -55,7 +55,7 @@ import { Relation, RelationSide } from './relation';
 import { RolesSpecifier } from './roles-specifier';
 import { InvalidType, ObjectType, Type } from './type';
 import { ValueObjectType } from './value-object-type';
-import { WarningCode } from '../../schema/message-codes';
+import { WarningCode } from '../validation/suppress/message-codes';
 
 export interface SystemFieldConfig extends FieldConfig {
     readonly isSystemField?: boolean;

src/model/implementation/flex-search.ts

@@ -4,7 +4,7 @@ import { ModelComponent, ValidationContext } from '../validation/validation-cont
 import { FlexSearchPrimarySortClauseConfig } from '../config';
 import { RootEntityType } from './root-entity-type';
 import { Severity, ValidationMessage } from '../validation';
-import { WarningCode } from '../../schema/message-codes';
+import { WarningCode } from '../validation/suppress/message-codes';
 
 export const IDENTITY_ANALYZER = 'identity';
 export const NORM_CI_ANALYZER = 'norm_ci';

src/model/implementation/roles-specifier.ts

@@ -1,7 +1,7 @@
 import { ModelComponent, ValidationContext } from '../validation/validation-context';
 import { RolesSpecifierConfig } from '../config';
 import { ValidationMessage } from '../validation';
-import { WarningCode } from '../../schema/message-codes';
+import { WarningCode } from '../validation/suppress/message-codes';
 import { Type } from './type';
 import { Field } from './field';
 

src/model/implementation/root-entity-type.ts

@@ -35,7 +35,7 @@ import { RolesSpecifier } from './roles-specifier';
 import { ScalarType } from './scalar-type';
 import { TimeToLiveType } from './time-to-live';
 import { EffectiveModuleSpecification } from './modules/effective-module-specification';
-import { WarningCode } from '../../schema/message-codes';
+import { WarningCode } from '../validation/suppress/message-codes';
 
 export class RootEntityType extends ObjectTypeBase {
     private readonly permissions: PermissionsConfig & {};

src/model/implementation/time-to-live.ts

@@ -11,7 +11,7 @@ import { RootEntityType } from './root-entity-type';
 import { ScalarType } from './scalar-type';
 import { Type } from './type';
 import { FieldPath } from './field-path';
-import { WarningCode } from '../../schema/message-codes';
+import { WarningCode } from '../validation/suppress/message-codes';
 
 export class TimeToLiveType implements ModelComponent {
     readonly cascadeFields: ReadonlyArray<FieldPath> = [];

src/model/implementation/type-base.ts

@@ -11,7 +11,7 @@ import { EffectiveModuleSpecification } from './modules/effective-module-specifi
 import { MODULES_DIRECTIVE } from '../../schema/constants';
 import { TypeModuleSpecification } from './modules/type-module-specification';
 import { Type } from './type';
-import { WarningCode } from '../../schema/message-codes';
+import { WarningCode } from '../validation/suppress/message-codes';
 
 export abstract class TypeBase implements ModelComponent {
     readonly name: string;