@redocly/openapi-core 1.29.0 → 1.31.0

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @redocly/openapi-core
 
+## 1.31.0
+
+## 1.30.0
+
+### Patch Changes
+
+- Updated `operation-tag-defined` built-in rule to verify tags are defined on the operation prior to matching them to a global tag.
+
 ## 1.29.0
 
 ### Minor Changes
@@ -26,7 +34,7 @@
 
 ### Patch Changes
 
-- Added support for Arazzo version 1.0.1 in Spot validation rules.
+- Added support for Arazzo version 1.0.1 in Respect validation rules.
 
 ## 1.28.1
 
@@ -67,13 +75,13 @@
 
 ### Patch Changes
 
-- Updated Spot validation rules.
+- Updated Respect validation rules.
 
 ## 1.26.1
 
 ### Patch Changes
 
-- Removed the `no-actions-type-end` Spot rule.
+- Removed the `no-actions-type-end` Respect rule.
 - Removed unused lodash.isequal dependency.
 
 ## 1.26.0
@@ -196,7 +204,7 @@
 
 ### Minor Changes
 
-- Added Spot and Arazzo rules: `no-criteria-xpath`, `no-actions-type-end`, `criteria-unique`.
+- Added Respect and Arazzo rules: `no-criteria-xpath`, `no-actions-type-end`, `criteria-unique`.
 
 ### Patch Changes
 

package/lib/config/all.js

@@ -222,7 +222,7 @@ const all = {
         'stepId-unique': 'error',
         'sourceDescription-name-unique': 'error',
         'sourceDescriptions-not-empty': 'error',
-        'spot-supported-versions': 'off',
+        'respect-supported-versions': 'off',
         'workflowId-unique': 'error',
         'workflow-dependsOn': 'error',
     },

package/lib/config/minimal.js

@@ -198,7 +198,7 @@ const minimal = {
         'step-onFailure-unique': 'off',
         'stepId-unique': 'error',
         'sourceDescription-name-unique': 'off',
-        'spot-supported-versions': 'off',
+        'respect-supported-versions': 'off',
         'workflowId-unique': 'error',
         'workflow-dependsOn': 'off',
     },

package/lib/config/recommended-strict.js

@@ -198,7 +198,7 @@ const recommendedStrict = {
         'stepId-unique': 'error',
         'sourceDescription-name-unique': 'error',
         'sourceDescriptions-not-empty': 'error',
-        'spot-supported-versions': 'off',
+        'respect-supported-versions': 'off',
         'workflowId-unique': 'error',
         'workflow-dependsOn': 'error',
     },

package/lib/config/recommended.js

@@ -198,7 +198,7 @@ const recommended = {
         'stepId-unique': 'error',
         'sourceDescription-name-unique': 'error',
         'sourceDescriptions-not-empty': 'error',
-        'spot-supported-versions': 'off',
+        'respect-supported-versions': 'off',
         'workflowId-unique': 'error',
         'workflow-dependsOn': 'error',
     },

package/lib/config/spec.js

@@ -12,7 +12,7 @@ const spec = {
     async3Rules: {},
     arazzo1Rules: {
         'sourceDescription-type': 'error',
-        'spot-supported-versions': 'off',
+        'respect-supported-versions': 'off',
         'workflowId-unique': 'error',
         'stepId-unique': 'error',
         'sourceDescription-name-unique': 'error',

package/lib/logger.d.ts

@@ -1,5 +1,7 @@
 import * as colorette from 'colorette';
-export { options as colorOptions } from 'colorette';
+export declare const colorOptions: {
+    enabled: boolean;
+};
 export declare const colorize: typeof colorette;
 declare class Logger {
     protected stderr(str: string): boolean;
@@ -8,3 +10,4 @@ declare class Logger {
     error(str: string): boolean | void;
 }
 export declare const logger: Logger;
+export {};

package/lib/logger.js

@@ -4,8 +4,9 @@ exports.logger = exports.colorize = exports.colorOptions = void 0;
 const colorette = require("colorette");
 const env_1 = require("./env");
 const utils_1 = require("./utils");
-var colorette_1 = require("colorette");
-Object.defineProperty(exports, "colorOptions", { enumerable: true, get: function () { return colorette_1.options; } });
+// eslint-disable-next-line @typescript-eslint/ban-ts-comment
+// @ts-ignore this works but some types are not working
+exports.colorOptions = colorette.options;
 exports.colorize = new Proxy(colorette, {
     get(target, prop) {
         if (env_1.isBrowser) {

package/lib/rules/arazzo/index.js

@@ -5,7 +5,7 @@ const struct_1 = require("../common/struct");
 const assertions_1 = require("../common/assertions");
 const sourceDescription_type_1 = require("../arazzo/sourceDescription-type");
 const sourceDescriptions_not_empty_1 = require("./sourceDescriptions-not-empty");
-const spot_supported_versions_1 = require("../spot/spot-supported-versions");
+const respect_supported_versions_1 = require("../respect/respect-supported-versions");
 const workflowId_unique_1 = require("./workflowId-unique");
 const stepId_unique_1 = require("./stepId-unique");
 const sourceDescriptions_name_unique_1 = require("./sourceDescriptions-name-unique");
@@ -14,13 +14,13 @@ const parameters_unique_1 = require("./parameters-unique");
 const step_onSuccess_unique_1 = require("./step-onSuccess-unique");
 const step_onFailure_unique_1 = require("./step-onFailure-unique");
 const requestBody_replacements_unique_1 = require("./requestBody-replacements-unique");
-const no_criteria_xpath_1 = require("../spot/no-criteria-xpath");
+const no_criteria_xpath_1 = require("../respect/no-criteria-xpath");
 const criteria_unique_1 = require("./criteria-unique");
 exports.rules = {
     struct: struct_1.Struct,
     assertions: assertions_1.Assertions,
     'sourceDescription-type': sourceDescription_type_1.SourceDescriptionType,
-    'spot-supported-versions': spot_supported_versions_1.SpotSupportedVersions,
+    'respect-supported-versions': respect_supported_versions_1.RespectSupportedVersions,
     'workflowId-unique': workflowId_unique_1.WorkflowIdUnique,
     'stepId-unique': stepId_unique_1.StepIdUnique,
     'sourceDescription-name-unique': sourceDescriptions_name_unique_1.SourceDescriptionsNameUnique,

package/lib/rules/arazzo/requestBody-replacements-unique.js

@@ -4,23 +4,28 @@ exports.RequestBodyReplacementsUnique = void 0;
 const RequestBodyReplacementsUnique = () => {
     const seenReplacements = new Set();
     return {
-        RequestBody: {
-            enter(requestBody, { report, location }) {
-                if (!requestBody.replacements)
-                    return;
-                for (const replacement of requestBody.replacements) {
-                    if (seenReplacements.has(replacement.target)) {
-                        report({
-                            message: 'Every `replacement` in `requestBody` must be unique.',
-                            location: location.child([
-                                'replacements',
-                                requestBody.replacements.indexOf(replacement),
-                                `target`,
-                            ]),
-                        });
+        Step: {
+            leave() {
+                seenReplacements.clear();
+            },
+            RequestBody: {
+                enter(requestBody, { report, location }) {
+                    if (!requestBody.replacements)
+                        return;
+                    for (const replacement of requestBody.replacements) {
+                        if (seenReplacements.has(replacement.target)) {
+                            report({
+                                message: 'Every `replacement` in `requestBody` must be unique.',
+                                location: location.child([
+                                    'replacements',
+                                    requestBody.replacements.indexOf(replacement),
+                                    `target`,
+                                ]),
+                            });
+                        }
+                        seenReplacements.add(replacement.target);
                     }
-                    seenReplacements.add(replacement.target);
-                }
+                },
             },
         },
     };

package/lib/rules/common/operation-tag-defined.js

@@ -8,7 +8,7 @@ const OperationTagDefined = () => {
             definedTags = new Set((root.tags ?? []).map((t) => t.name));
         },
         Operation(operation, { report, location }) {
-            if (operation.tags) {
+            if (operation?.tags) {
                 for (let i = 0; i < operation.tags.length; i++) {
                     if (!definedTags.has(operation.tags[i])) {
                         report({
@@ -18,6 +18,12 @@ const OperationTagDefined = () => {
                     }
                 }
             }
+            else {
+                report({
+                    message: `Operation tags should be defined`,
+                    location: location.key(),
+                });
+            }
         },
     };
 };

package/lib/rules/{spot → respect}/no-criteria-xpath.js

@@ -10,7 +10,7 @@ const NoCriteriaXpath = () => {
                 }
                 if (criteria?.type?.type === 'xpath' || criteria?.type === 'xpath') {
                     report({
-                        message: 'The `xpath` type criteria is not supported by Spot.',
+                        message: 'The `xpath` type criteria is not supported by Respect.',
                         location: location.child(['type']),
                     });
                 }

package/lib/rules/respect/respect-supported-versions.d.ts

@@ -0,0 +1,2 @@
+import type { Arazzo1Rule } from '../../visitors';
+export declare const RespectSupportedVersions: Arazzo1Rule;

package/lib/rules/{spot/spot-supported-versions.js → respect/respect-supported-versions.js}

@@ -1,16 +1,16 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.SpotSupportedVersions = void 0;
+exports.RespectSupportedVersions = void 0;
 const arazzo_1 = require("../../typings/arazzo");
 const utils_1 = require("../../utils");
-const SpotSupportedVersions = () => {
-    const supportedVersions = arazzo_1.ARAZZO_VERSIONS_SUPPORTED_BY_SPOT.join(', ');
+const RespectSupportedVersions = () => {
+    const supportedVersions = arazzo_1.ARAZZO_VERSIONS_SUPPORTED_BY_RESPECT.join(', ');
     return {
         Root: {
             enter(root, { report, location }) {
-                if (!arazzo_1.ARAZZO_VERSIONS_SUPPORTED_BY_SPOT.includes(root.arazzo)) {
+                if (!arazzo_1.ARAZZO_VERSIONS_SUPPORTED_BY_RESPECT.includes(root.arazzo)) {
                     report({
-                        message: `Only ${supportedVersions} Arazzo ${(0, utils_1.pluralize)('version is', arazzo_1.ARAZZO_VERSIONS_SUPPORTED_BY_SPOT.length)} supported by Spot.`,
+                        message: `Only ${supportedVersions} Arazzo ${(0, utils_1.pluralize)('version is', arazzo_1.ARAZZO_VERSIONS_SUPPORTED_BY_RESPECT.length)} supported by Respect.`,
                         location: location.child('arazzo'),
                     });
                 }
@@ -18,4 +18,4 @@ const SpotSupportedVersions = () => {
         },
     };
 };
-exports.SpotSupportedVersions = SpotSupportedVersions;
+exports.RespectSupportedVersions = RespectSupportedVersions;

package/lib/types/redocly-yaml.d.ts

@@ -9,7 +9,7 @@ declare const builtInAsync2Rules: readonly ["info-contact", "info-license-strict
 declare const builtInAsync3Rules: readonly ["info-contact", "info-license-strict", "operation-operationId", "tag-description", "tags-alphabetical", "channels-kebab-case", "no-channel-trailing-slash"];
 export type BuiltInAsync2RuleId = typeof builtInAsync2Rules[number];
 export type BuiltInAsync3RuleId = typeof builtInAsync3Rules[number];
-declare const builtInArazzo1Rules: readonly ["sourceDescription-type", "workflowId-unique", "stepId-unique", "sourceDescription-name-unique", "sourceDescriptions-not-empty", "workflow-dependsOn", "parameters-unique", "step-onSuccess-unique", "step-onFailure-unique", "spot-supported-versions", "requestBody-replacements-unique", "no-criteria-xpath", "criteria-unique"];
+declare const builtInArazzo1Rules: readonly ["sourceDescription-type", "workflowId-unique", "stepId-unique", "sourceDescription-name-unique", "sourceDescriptions-not-empty", "workflow-dependsOn", "parameters-unique", "step-onSuccess-unique", "step-onFailure-unique", "respect-supported-versions", "requestBody-replacements-unique", "no-criteria-xpath", "criteria-unique"];
 export type BuiltInArazzo1RuleId = typeof builtInArazzo1Rules[number];
 declare const oas2NodeTypesList: readonly ["Root", "Tag", "TagList", "ExternalDocs", "SecurityRequirement", "SecurityRequirementList", "Info", "Contact", "License", "Paths", "PathItem", "Parameter", "ParameterList", "ParameterItems", "Operation", "Example", "ExamplesMap", "Examples", "Header", "Responses", "Response", "Schema", "Xml", "SchemaProperties", "NamedSchemas", "NamedResponses", "NamedParameters", "NamedSecuritySchemes", "SecurityScheme", "TagGroup", "TagGroups", "EnumDescriptions", "Logo", "XCodeSample", "XCodeSampleList", "XServer", "XServerList"];
 export type Oas2NodeType = typeof oas2NodeTypesList[number];

package/lib/types/redocly-yaml.js

@@ -139,7 +139,7 @@ const builtInArazzo1Rules = [
     'parameters-unique',
     'step-onSuccess-unique',
     'step-onFailure-unique',
-    'spot-supported-versions',
+    'respect-supported-versions',
     'requestBody-replacements-unique',
     'no-criteria-xpath',
     'criteria-unique',

package/lib/typings/arazzo.d.ts

@@ -128,4 +128,4 @@ export interface ArazzoDefinition {
     };
 }
 export declare const VERSION_PATTERN: RegExp;
-export declare const ARAZZO_VERSIONS_SUPPORTED_BY_SPOT: string[];
+export declare const ARAZZO_VERSIONS_SUPPORTED_BY_RESPECT: string[];

package/lib/typings/arazzo.js

@@ -1,5 +1,5 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.ARAZZO_VERSIONS_SUPPORTED_BY_SPOT = exports.VERSION_PATTERN = void 0;
+exports.ARAZZO_VERSIONS_SUPPORTED_BY_RESPECT = exports.VERSION_PATTERN = void 0;
 exports.VERSION_PATTERN = /^1\.0\.\d+(-.+)?$/;
-exports.ARAZZO_VERSIONS_SUPPORTED_BY_SPOT = ['1.0.1'];
+exports.ARAZZO_VERSIONS_SUPPORTED_BY_RESPECT = ['1.0.1'];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@redocly/openapi-core",
-  "version": "1.29.0",
+  "version": "1.31.0",
   "description": "",
   "main": "lib/index.js",
   "engines": {
@@ -28,6 +28,7 @@
     "OpenAPI linter",
     "Swagger linter",
     "AsyncAPI linter",
+    "Arazzo linter",
     "oas"
   ],
   "contributors": [

package/src/config/__tests__/__snapshots__/config-resolvers.test.ts.snap

@@ -9,10 +9,10 @@ exports[`resolveConfig should ignore minimal from the root and read local file 1
     "no-criteria-xpath": "off",
     "parameters-unique": "error",
     "requestBody-replacements-unique": "warn",
+    "respect-supported-versions": "off",
     "sourceDescription-name-unique": "error",
     "sourceDescription-type": "error",
     "sourceDescriptions-not-empty": "error",
-    "spot-supported-versions": "off",
     "step-onFailure-unique": "warn",
     "step-onSuccess-unique": "warn",
     "stepId-unique": "error",
@@ -238,10 +238,10 @@ exports[`resolveStyleguideConfig should resolve extends with local file config w
     "no-criteria-xpath": "off",
     "parameters-unique": "error",
     "requestBody-replacements-unique": "warn",
+    "respect-supported-versions": "off",
     "sourceDescription-name-unique": "error",
     "sourceDescription-type": "error",
     "sourceDescriptions-not-empty": "error",
-    "spot-supported-versions": "off",
     "step-onFailure-unique": "warn",
     "step-onSuccess-unique": "warn",
     "stepId-unique": "error",

package/src/config/all.ts

@@ -222,7 +222,7 @@ const all: PluginStyleguideConfig<'built-in'> = {
     'stepId-unique': 'error',
     'sourceDescription-name-unique': 'error',
     'sourceDescriptions-not-empty': 'error',
-    'spot-supported-versions': 'off',
+    'respect-supported-versions': 'off',
     'workflowId-unique': 'error',
     'workflow-dependsOn': 'error',
   },

package/src/config/minimal.ts

@@ -198,7 +198,7 @@ const minimal: PluginStyleguideConfig<'built-in'> = {
     'step-onFailure-unique': 'off',
     'stepId-unique': 'error',
     'sourceDescription-name-unique': 'off',
-    'spot-supported-versions': 'off',
+    'respect-supported-versions': 'off',
     'workflowId-unique': 'error',
     'workflow-dependsOn': 'off',
   },

package/src/config/recommended-strict.ts

@@ -198,7 +198,7 @@ const recommendedStrict: PluginStyleguideConfig<'built-in'> = {
     'stepId-unique': 'error',
     'sourceDescription-name-unique': 'error',
     'sourceDescriptions-not-empty': 'error',
-    'spot-supported-versions': 'off',
+    'respect-supported-versions': 'off',
     'workflowId-unique': 'error',
     'workflow-dependsOn': 'error',
   },

package/src/config/recommended.ts

@@ -198,7 +198,7 @@ const recommended: PluginStyleguideConfig<'built-in'> = {
     'stepId-unique': 'error',
     'sourceDescription-name-unique': 'error',
     'sourceDescriptions-not-empty': 'error',
-    'spot-supported-versions': 'off',
+    'respect-supported-versions': 'off',
     'workflowId-unique': 'error',
     'workflow-dependsOn': 'error',
   },

package/src/config/spec.ts

@@ -12,7 +12,7 @@ const spec: PluginStyleguideConfig<'built-in'> = {
   async3Rules: {},
   arazzo1Rules: {
     'sourceDescription-type': 'error',
-    'spot-supported-versions': 'off',
+    'respect-supported-versions': 'off',
     'workflowId-unique': 'error',
     'stepId-unique': 'error',
     'sourceDescription-name-unique': 'error',

package/src/logger.ts

@@ -2,7 +2,9 @@ import * as colorette from 'colorette';
 import { isBrowser } from './env';
 import { identity } from './utils';
 
-export { options as colorOptions } from 'colorette';
+// eslint-disable-next-line @typescript-eslint/ban-ts-comment
+// @ts-ignore this works but some types are not working
+export const colorOptions = colorette.options;
 
 export const colorize = new Proxy(colorette, {
   get(target: typeof colorette, prop: string): typeof identity {

package/src/rules/arazzo/__tests__/no-criteria-xpath.test.ts

@@ -90,7 +90,7 @@ describe('Arazzo no-criteria-xpath', () => {
               "source": "arazzo.yaml",
             },
           ],
-          "message": "The \`xpath\` type criteria is not supported by Spot.",
+          "message": "The \`xpath\` type criteria is not supported by Respect.",
           "ruleId": "no-criteria-xpath",
           "severity": "error",
           "suggest": [],
@@ -103,7 +103,7 @@ describe('Arazzo no-criteria-xpath', () => {
               "source": "arazzo.yaml",
             },
           ],
-          "message": "The \`xpath\` type criteria is not supported by Spot.",
+          "message": "The \`xpath\` type criteria is not supported by Respect.",
           "ruleId": "no-criteria-xpath",
           "severity": "error",
           "suggest": [],

package/src/rules/arazzo/__tests__/{spot-supported-versions.test.ts → respect-supported-versions.test.ts}

@@ -3,7 +3,7 @@ import { lintDocument } from '../../../lint';
 import { parseYamlToDocument, replaceSourceWithRef, makeConfig } from '../../../../__tests__/utils';
 import { BaseResolver } from '../../../resolve';
 
-describe('Arazzo spot-supported-versions', () => {
+describe('Arazzo respect-supported-versions', () => {
   const documentWithUnsupportedVersion = parseYamlToDocument(
     outdent`
       arazzo: '1.0.2'
@@ -67,7 +67,7 @@ describe('Arazzo spot-supported-versions', () => {
       externalRefResolver: new BaseResolver(),
       document: documentWithUnsupportedVersion,
       config: await makeConfig({
-        rules: { 'spot-supported-versions': 'error' },
+        rules: { 'respect-supported-versions': 'error' },
       }),
     });
 
@@ -81,8 +81,8 @@ describe('Arazzo spot-supported-versions', () => {
               "source": "arazzo.yaml",
             },
           ],
-          "message": "Only 1.0.1 Arazzo version is supported by Spot.",
-          "ruleId": "spot-supported-versions",
+          "message": "Only 1.0.1 Arazzo version is supported by Respect.",
+          "ruleId": "respect-supported-versions",
           "severity": "error",
           "suggest": [],
         },
@@ -95,7 +95,7 @@ describe('Arazzo spot-supported-versions', () => {
       externalRefResolver: new BaseResolver(),
       document: documentWithSupportedVersion,
       config: await makeConfig({
-        rules: { 'spot-supported-versions': 'error' },
+        rules: { 'respect-supported-versions': 'error' },
       }),
     });
 

package/src/rules/arazzo/index.ts

@@ -2,7 +2,7 @@ import { Struct } from '../common/struct';
 import { Assertions } from '../common/assertions';
 import { SourceDescriptionType } from '../arazzo/sourceDescription-type';
 import { SourceDescriptionsNotEmpty } from './sourceDescriptions-not-empty';
-import { SpotSupportedVersions } from '../spot/spot-supported-versions';
+import { RespectSupportedVersions } from '../respect/respect-supported-versions';
 import { WorkflowIdUnique } from './workflowId-unique';
 import { StepIdUnique } from './stepId-unique';
 import { SourceDescriptionsNameUnique } from './sourceDescriptions-name-unique';
@@ -11,7 +11,7 @@ import { ParametersUnique } from './parameters-unique';
 import { StepOnSuccessUnique } from './step-onSuccess-unique';
 import { StepOnFailureUnique } from './step-onFailure-unique';
 import { RequestBodyReplacementsUnique } from './requestBody-replacements-unique';
-import { NoCriteriaXpath } from '../spot/no-criteria-xpath';
+import { NoCriteriaXpath } from '../respect/no-criteria-xpath';
 import { CriteriaUnique } from './criteria-unique';
 
 import type { Arazzo1Rule } from '../../visitors';
@@ -21,7 +21,7 @@ export const rules: Arazzo1RuleSet<'built-in'> = {
   struct: Struct as Arazzo1Rule,
   assertions: Assertions as Arazzo1Rule,
   'sourceDescription-type': SourceDescriptionType,
-  'spot-supported-versions': SpotSupportedVersions,
+  'respect-supported-versions': RespectSupportedVersions,
   'workflowId-unique': WorkflowIdUnique,
   'stepId-unique': StepIdUnique,
   'sourceDescription-name-unique': SourceDescriptionsNameUnique,

package/src/rules/arazzo/requestBody-replacements-unique.ts

@@ -5,23 +5,28 @@ export const RequestBodyReplacementsUnique: Arazzo1Rule = () => {
   const seenReplacements = new Set();
 
   return {
-    RequestBody: {
-      enter(requestBody, { report, location }: UserContext) {
-        if (!requestBody.replacements) return;
+    Step: {
+      leave() {
+        seenReplacements.clear();
+      },
+      RequestBody: {
+        enter(requestBody, { report, location }: UserContext) {
+          if (!requestBody.replacements) return;
 
-        for (const replacement of requestBody.replacements) {
-          if (seenReplacements.has(replacement.target)) {
-            report({
-              message: 'Every `replacement` in `requestBody` must be unique.',
-              location: location.child([
-                'replacements',
-                requestBody.replacements.indexOf(replacement),
-                `target`,
-              ]),
-            });
+          for (const replacement of requestBody.replacements) {
+            if (seenReplacements.has(replacement.target)) {
+              report({
+                message: 'Every `replacement` in `requestBody` must be unique.',
+                location: location.child([
+                  'replacements',
+                  requestBody.replacements.indexOf(replacement),
+                  `target`,
+                ]),
+              });
+            }
+            seenReplacements.add(replacement.target);
           }
-          seenReplacements.add(replacement.target);
-        }
+        },
       },
     },
   };

package/src/rules/common/__tests__/operation-tag-defined.test.ts

@@ -0,0 +1,68 @@
+import { outdent } from 'outdent';
+import { lintDocument } from '../../../lint';
+import { parseYamlToDocument, replaceSourceWithRef, makeConfig } from '../../../../__tests__/utils';
+import { BaseResolver } from '../../../resolve';
+
+describe('Oas3 operation-tag-defined', () => {
+  it('should not report on operation object if at least one tag is defined', async () => {
+    const document = parseYamlToDocument(
+      outdent`
+          openapi: 3.0.4
+          tags:
+            - name: a
+          paths:
+            /some:
+              get:
+                tags:
+                  - a
+        `,
+      'foobar.yaml'
+    );
+
+    const results = await lintDocument({
+      externalRefResolver: new BaseResolver(),
+      document,
+      config: await makeConfig({ rules: { 'operation-tag-defined': 'error' } }),
+    });
+
+    expect(replaceSourceWithRef(results)).toMatchInlineSnapshot(`[]`);
+  });
+
+  it('should report on operation object if no tags are defined', async () => {
+    const document = parseYamlToDocument(
+      outdent`
+      openapi: 3.0.4
+      tags:
+        - name: a
+      paths:
+        /some:
+          get:
+        `,
+      'foobar.yaml'
+    );
+
+    const results = await lintDocument({
+      externalRefResolver: new BaseResolver(),
+      document,
+      config: await makeConfig({ rules: { 'operation-tag-defined': 'error' } }),
+    });
+
+    expect(replaceSourceWithRef(results)).toMatchInlineSnapshot(`
+      [
+        {
+          "location": [
+            {
+              "pointer": "#/paths/~1some/get",
+              "reportOnKey": true,
+              "source": "foobar.yaml",
+            },
+          ],
+          "message": "Operation tags should be defined",
+          "ruleId": "operation-tag-defined",
+          "severity": "error",
+          "suggest": [],
+        },
+      ]
+    `);
+  });
+});

package/src/rules/common/operation-tag-defined.ts

@@ -11,7 +11,7 @@ export const OperationTagDefined: Oas3Rule | Oas2Rule = () => {
       definedTags = new Set((root.tags ?? []).map((t) => t.name));
     },
     Operation(operation: Oas2Operation | Oas3Operation, { report, location }: UserContext) {
-      if (operation.tags) {
+      if (operation?.tags) {
         for (let i = 0; i < operation.tags.length; i++) {
           if (!definedTags.has(operation.tags[i])) {
             report({
@@ -20,6 +20,11 @@ export const OperationTagDefined: Oas3Rule | Oas2Rule = () => {
             });
           }
         }
+      } else {
+        report({
+          message: `Operation tags should be defined`,
+          location: location.key(),
+        });
       }
     },
   };

package/src/rules/oas3/__tests__/no-invalid-media-type-examples.test.ts

@@ -3,6 +3,8 @@ import { lintDocument } from '../../../lint';
 import { parseYamlToDocument, replaceSourceWithRef, makeConfig } from '../../../../__tests__/utils';
 import { BaseResolver } from '../../../resolve';
 
+jest.setTimeout(10000);
+
 describe('no-invalid-media-type-examples', () => {
   it('should report on invalid example', async () => {
     const document = parseYamlToDocument(