@redocly/openapi-core 1.1.0 → 1.2.0

package/lib/types/redocly-yaml.js

@@ -61,6 +61,8 @@ const builtInRulesList = [
     'required-string-property-missing-min-length',
     'spec-strict-refs',
     'component-name-unique',
+    'channels-kebab-case',
+    'no-channel-trailing-slash',
 ];
 const nodeTypesList = [
     'any',
@@ -122,6 +124,7 @@ const nodeTypesList = [
     'XCodeSampleList',
     'WebhooksMap',
     'SpecExtension',
+    'Message',
 ];
 const ConfigStyleguide = {
     properties: {

package/lib/typings/asyncapi.d.ts

@@ -0,0 +1,21 @@
+export interface Async2Definition {
+    asyncapi: string;
+    info?: Async2Info;
+}
+export interface Async2Info {
+    title: string;
+    version: string;
+    description?: string;
+    termsOfService?: string;
+    contact?: Async2Contact;
+    license?: Async2License;
+}
+export interface Async2Contact {
+    name?: string;
+    url?: string;
+    email?: string;
+}
+export interface Async2License {
+    name: string;
+    url?: string;
+}

package/lib/typings/asyncapi.js

@@ -0,0 +1,2 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });

package/lib/visitors.d.ts

@@ -4,6 +4,7 @@ import { NormalizedNodeType } from './types';
 import type { Stack } from './utils';
 import type { UserContext, ResolveResult, ProblemSeverity } from './walk';
 import type { Location } from './ref-utils';
+import { Async2Definition } from './typings/asyncapi';
 export declare type SkipFunctionContext = Pick<UserContext, 'location' | 'rawNode' | 'resolve' | 'rawLocation'>;
 export declare type VisitFunction<T> = (node: T, ctx: UserContext & {
     ignoreNextVisitorsOnNode: () => void;
@@ -134,16 +135,24 @@ declare type Oas2FlatVisitor = {
     SecurityScheme?: VisitFunctionOrObject<Oas2SecurityScheme>;
     SpecExtension?: VisitFunctionOrObject<unknown>;
 };
+declare type Async2FlatVisitor = {
+    Root?: VisitFunctionOrObject<Async2Definition>;
+};
 declare type Oas3NestedVisitor = {
     [T in keyof Oas3FlatVisitor]: Oas3FlatVisitor[T] extends Function ? Oas3FlatVisitor[T] : Oas3FlatVisitor[T] & NestedVisitor<Oas3NestedVisitor>;
 };
 declare type Oas2NestedVisitor = {
     [T in keyof Oas2FlatVisitor]: Oas2FlatVisitor[T] extends Function ? Oas2FlatVisitor[T] : Oas2FlatVisitor[T] & NestedVisitor<Oas2NestedVisitor>;
 };
+declare type Async2NestedVisitor = {
+    [T in keyof Async2FlatVisitor]: Async2FlatVisitor[T] extends Function ? Async2FlatVisitor[T] : Async2FlatVisitor[T] & NestedVisitor<Async2NestedVisitor>;
+};
 export declare type Oas3Visitor = BaseVisitor & Oas3NestedVisitor & Record<string, VisitFunction<any> | NestedVisitObject<any, Oas3NestedVisitor>>;
 export declare type Oas2Visitor = BaseVisitor & Oas2NestedVisitor & Record<string, VisitFunction<any> | NestedVisitObject<any, Oas2NestedVisitor>>;
+export declare type Async2Visitor = BaseVisitor & Async2NestedVisitor & Record<string, VisitFunction<any> | NestedVisitObject<any, Async2NestedVisitor>>;
 export declare type Oas3TransformVisitor = BaseVisitor & Oas3FlatVisitor & Record<string, VisitFunction<any> | VisitObject<any>>;
 export declare type Oas2TransformVisitor = BaseVisitor & Oas2FlatVisitor & Record<string, VisitFunction<any> | VisitObject<any>>;
+export declare type Async2TransformVisitor = BaseVisitor & Async2FlatVisitor & Record<string, VisitFunction<any> | VisitObject<any>>;
 export declare type NestedVisitor<T> = Exclude<T, 'any' | 'ref' | 'Root'>;
 export declare type NormalizedOasVisitors<T extends BaseVisitor> = {
     [V in keyof T]-?: {
@@ -162,10 +171,13 @@ export declare type NormalizedOasVisitors<T extends BaseVisitor> = {
 };
 export declare type Oas3Rule = (options: Record<string, any>) => Oas3Visitor | Oas3Visitor[];
 export declare type Oas2Rule = (options: Record<string, any>) => Oas2Visitor | Oas2Visitor[];
+export declare type Async2Rule = (options: Record<string, any>) => Async2Visitor | Async2Visitor[];
 export declare type Oas3Preprocessor = (options: Record<string, any>) => Oas3TransformVisitor;
 export declare type Oas2Preprocessor = (options: Record<string, any>) => Oas2TransformVisitor;
+export declare type Async2Preprocessor = (options: Record<string, any>) => Async2TransformVisitor;
 export declare type Oas3Decorator = (options: Record<string, any>) => Oas3TransformVisitor;
 export declare type Oas2Decorator = (options: Record<string, any>) => Oas2TransformVisitor;
+export declare type Async2Decorator = (options: Record<string, any>) => Async2TransformVisitor;
 export declare type OasRule = Oas3Rule;
 export declare type OasPreprocessor = Oas3Preprocessor;
 export declare type OasDecorator = Oas3Decorator;

package/lib/walk.d.ts

@@ -5,7 +5,7 @@ import type { NormalizedNodeType } from './types';
 import type { RuleSeverity } from './config';
 import { Location } from './ref-utils';
 import { ResolveError, YamlParseError, Source } from './resolve';
-import { OasVersion } from './oas-types';
+import { SpecVersion } from './oas-types';
 declare type NonUndefined = string | number | boolean | symbol | bigint | object | Record<string, any>;
 export declare type ResolveResult<T extends NonUndefined> = {
     node: T;
@@ -27,7 +27,7 @@ export declare type UserContext = {
     type: NormalizedNodeType;
     key: string | number;
     parent: any;
-    oasVersion: OasVersion;
+    oasVersion: SpecVersion;
     getVisitorData: () => Record<string, unknown>;
 };
 export declare type Loc = {
@@ -65,7 +65,7 @@ export declare type NormalizedProblem = {
 };
 export declare type WalkContext = {
     problems: NormalizedProblem[];
-    oasVersion: OasVersion;
+    oasVersion: SpecVersion;
     visitorsData: Record<string, Record<string, unknown>>;
     refTypes?: Map<string, NormalizedNodeType>;
 };

package/package.json

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

package/src/__tests__/lint.test.ts

@@ -5,7 +5,7 @@ import { lintFromString, lintConfig, lintDocument } from '../lint';
 import { BaseResolver } from '../resolve';
 import { loadConfig } from '../config/load';
 import { parseYamlToDocument, replaceSourceWithRef, makeConfig } from '../../__tests__/utils';
-import { detectOpenAPI } from '../oas-types';
+import { detectSpec } from '../oas-types';
 
 describe('lint', () => {
   it('lintFromString should work', async () => {
@@ -61,12 +61,12 @@ describe('lint', () => {
           path-http-verbs-order: error
           boolean-parameter-prefixes: off
           rule/operation-summary-length:
-            subject: 
+            subject:
               type: Operation
               property: summary
             message: Operation summary should start with an active verb
             assertions:
-              local/checkWordsCount: 
+              local/checkWordsCount:
                 min: 3
       theme:
         openapi:
@@ -189,8 +189,8 @@ describe('lint', () => {
   it('lintConfig should work with legacy fields - referenceDocs', async () => {
     const document = parseYamlToDocument(
       outdent`
-        apis: 
-          entry: 
+        apis:
+          entry:
             root: ./file.yaml
         rules:
           operation-2xx-response: warn
@@ -304,11 +304,41 @@ describe('lint', () => {
       `,
       ''
     );
-    expect(() => detectOpenAPI(testDocument.parsed)).toThrow(
+    expect(() => detectSpec(testDocument.parsed)).toThrow(
       `Invalid OpenAPI version: should be a string but got "number"`
     );
   });
 
+  it('detect unsupported OpenAPI version', () => {
+    const testDocument = parseYamlToDocument(
+      outdent`
+        openapi: 1.0.4
+      `,
+      ''
+    );
+    expect(() => detectSpec(testDocument.parsed)).toThrow(`Unsupported OpenAPI version: 1.0.4`);
+  });
+
+  it('detect unsupported AsyncAPI version', () => {
+    const testDocument = parseYamlToDocument(
+      outdent`
+        asyncapi: 1.0.4
+      `,
+      ''
+    );
+    expect(() => detectSpec(testDocument.parsed)).toThrow(`Unsupported AsyncAPI version: 1.0.4`);
+  });
+
+  it('detect unsupported spec format', () => {
+    const testDocument = parseYamlToDocument(
+      outdent`
+        notapi: 3.1.0
+      `,
+      ''
+    );
+    expect(() => detectSpec(testDocument.parsed)).toThrow(`Unsupported specification`);
+  });
+
   it("spec rule shouldn't throw an error for named callback", async () => {
     const document = parseYamlToDocument(
       outdent`

package/src/bundle.ts

@@ -1,12 +1,9 @@
 import isEqual = require('lodash.isequal');
 import { BaseResolver, resolveDocument, Document, ResolvedRefMap, makeRefId } from './resolve';
 import { Oas3Rule, normalizeVisitors, Oas3Visitor, Oas2Visitor } from './visitors';
-import { Oas3Types } from './types/oas3';
-import { Oas2Types } from './types/oas2';
-import { Oas3_1Types } from './types/oas3_1';
 import { NormalizedNodeType, normalizeTypes, NodeType } from './types';
 import { WalkContext, walkDocument, UserContext, ResolveResult, NormalizedProblem } from './walk';
-import { detectOpenAPI, openAPIMajor, OasMajorVersion } from './oas-types';
+import { detectSpec, getTypes, getMajorSpecVersion, SpecMajorVersion } from './oas-types';
 import { isAbsoluteUrl, isRef, Location, refBaseName } from './ref-utils';
 import { initRules } from './config/rules';
 import { reportUnresolvedRef } from './rules/no-unresolved-refs';
@@ -93,27 +90,20 @@ export async function bundleDocument(opts: {
     removeUnusedComponents = false,
     keepUrlRefs = false,
   } = opts;
-  const oasVersion = detectOpenAPI(document.parsed);
-  const oasMajorVersion = openAPIMajor(oasVersion);
-  const rules = config.getRulesForOasVersion(oasMajorVersion);
+  const specVersion = detectSpec(document.parsed);
+  const specMajorVersion = getMajorSpecVersion(specVersion);
+  const rules = config.getRulesForOasVersion(specMajorVersion);
   const types = normalizeTypes(
-    config.extendTypes(
-      customTypes ?? oasMajorVersion === OasMajorVersion.Version3
-        ? oasVersion === OasVersion.Version3_1
-          ? Oas3_1Types
-          : Oas3Types
-        : Oas2Types,
-      oasVersion
-    ),
+    config.extendTypes(customTypes ?? getTypes(specVersion), specVersion),
     config
   );
 
-  const preprocessors = initRules(rules as any, config, 'preprocessors', oasVersion);
-  const decorators = initRules(rules as any, config, 'decorators', oasVersion);
+  const preprocessors = initRules(rules as any, config, 'preprocessors', specVersion);
+  const decorators = initRules(rules as any, config, 'decorators', specVersion);
 
   const ctx: BundleContext = {
     problems: [],
-    oasVersion: oasVersion,
+    oasVersion: specVersion,
     refTypes: new Map<string, NormalizedNodeType>(),
     visitorsData: {},
   };
@@ -123,7 +113,7 @@ export async function bundleDocument(opts: {
       severity: 'error',
       ruleId: 'remove-unused-components',
       visitor:
-        oasMajorVersion === OasMajorVersion.Version2
+        specMajorVersion === SpecMajorVersion.OAS2
           ? RemoveUnusedComponentsOas2({})
           : RemoveUnusedComponentsOas3({}),
     });
@@ -157,7 +147,7 @@ export async function bundleDocument(opts: {
         severity: 'error',
         ruleId: 'bundler',
         visitor: makeBundleVisitor(
-          oasMajorVersion,
+          specMajorVersion,
           dereference,
           skipRedoclyRegistryRefs,
           document,
@@ -188,9 +178,9 @@ export async function bundleDocument(opts: {
   };
 }
 
-export function mapTypeToComponent(typeName: string, version: OasMajorVersion) {
+export function mapTypeToComponent(typeName: string, version: SpecMajorVersion) {
   switch (version) {
-    case OasMajorVersion.Version3:
+    case SpecMajorVersion.OAS3:
       switch (typeName) {
         case 'Schema':
           return 'schemas';
@@ -213,7 +203,7 @@ export function mapTypeToComponent(typeName: string, version: OasMajorVersion) {
         default:
           return null;
       }
-    case OasMajorVersion.Version2:
+    case SpecMajorVersion.OAS2:
       switch (typeName) {
         case 'Schema':
           return 'definitions';
@@ -224,13 +214,22 @@ export function mapTypeToComponent(typeName: string, version: OasMajorVersion) {
         default:
           return null;
       }
+    case SpecMajorVersion.Async2:
+      switch (typeName) {
+        case 'Schema':
+          return 'schemas';
+        case 'Parameter':
+          return 'parameters';
+        default:
+          return null;
+      }
   }
 }
 
 // function oas3Move
 
 function makeBundleVisitor(
-  version: OasMajorVersion,
+  version: SpecMajorVersion,
   dereference: boolean,
   skipRedoclyRegistryRefs: boolean,
   rootDocument: Document,
@@ -282,16 +281,16 @@ function makeBundleVisitor(
     Root: {
       enter(root: any, ctx: any) {
         rootLocation = ctx.location;
-        if (version === OasMajorVersion.Version3) {
+        if (version === SpecMajorVersion.OAS3) {
           components = root.components = root.components || {};
-        } else if (version === OasMajorVersion.Version2) {
+        } else if (version === SpecMajorVersion.OAS2) {
           components = root;
         }
       },
     },
   };
 
-  if (version === OasMajorVersion.Version3) {
+  if (version === SpecMajorVersion.OAS3) {
     visitor.DiscriminatorMapping = {
       leave(mapping: Record<string, string>, ctx: any) {
         for (const name of Object.keys(mapping)) {
@@ -345,7 +344,7 @@ function makeBundleVisitor(
     components[componentType] = components[componentType] || {};
     const name = getComponentName(target, componentType, ctx);
     components[componentType][name] = target.node;
-    if (version === OasMajorVersion.Version3) {
+    if (version === SpecMajorVersion.OAS3) {
       return `#/components/${componentType}/${name}`;
     } else {
       return `#/${componentType}/${name}`;

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

@@ -6,6 +6,11 @@ StyleguideConfig {
   "_usedVersions": Set {},
   "configFile": undefined,
   "decorators": Object {
+    "async2": Object {
+      "oas2": Object {},
+      "oas3_0": Object {},
+      "oas3_1": Object {},
+    },
     "oas2": Object {
       "oas2": Object {},
       "oas3_0": Object {},
@@ -39,6 +44,11 @@ StyleguideConfig {
   "pluginPaths": Array [],
   "plugins": Array [],
   "preprocessors": Object {
+    "async2": Object {
+      "oas2": Object {},
+      "oas3_0": Object {},
+      "oas3_1": Object {},
+    },
     "oas2": Object {
       "oas2": Object {},
       "oas3_0": Object {},
@@ -97,6 +107,20 @@ StyleguideConfig {
   },
   "recommendedFallback": false,
   "rules": Object {
+    "async2": Object {
+      "oas2": Object {
+        "no-empty-servers": "error",
+        "operation-summary": "error",
+      },
+      "oas3_0": Object {
+        "no-empty-servers": "error",
+        "operation-summary": "error",
+      },
+      "oas3_1": Object {
+        "no-empty-servers": "error",
+        "operation-summary": "error",
+      },
+    },
     "oas2": Object {
       "oas2": Object {
         "no-empty-servers": "error",

package/src/config/__tests__/config.test.ts

@@ -1,4 +1,4 @@
-import { OasVersion } from '../../oas-types';
+import { SpecVersion } from '../../oas-types';
 import { Config, StyleguideConfig } from '../config';
 import { getMergedConfig } from '../utils';
 import { doesYamlFileExist } from '../../utils';
@@ -117,6 +117,7 @@ describe('getMergedConfig', () => {
           "_usedVersions": Set {},
           "configFile": "redocly.yaml",
           "decorators": Object {
+            "async2": Object {},
             "oas2": Object {},
             "oas3_0": Object {},
             "oas3_1": Object {},
@@ -127,6 +128,7 @@ describe('getMergedConfig', () => {
           "pluginPaths": Array [],
           "plugins": Array [],
           "preprocessors": Object {
+            "async2": Object {},
             "oas2": Object {},
             "oas3_0": Object {},
             "oas3_1": Object {},
@@ -140,6 +142,9 @@ describe('getMergedConfig', () => {
           },
           "recommendedFallback": false,
           "rules": Object {
+            "async2": Object {
+              "operation-summary": "warn",
+            },
             "oas2": Object {
               "operation-summary": "warn",
             },
@@ -217,6 +222,7 @@ describe('getMergedConfig', () => {
           "_usedVersions": Set {},
           "configFile": "redocly.yaml",
           "decorators": Object {
+            "async2": Object {},
             "oas2": Object {},
             "oas3_0": Object {},
             "oas3_1": Object {},
@@ -227,6 +233,7 @@ describe('getMergedConfig', () => {
           "pluginPaths": Array [],
           "plugins": Array [],
           "preprocessors": Object {
+            "async2": Object {},
             "oas2": Object {},
             "oas3_0": Object {},
             "oas3_1": Object {},
@@ -242,6 +249,10 @@ describe('getMergedConfig', () => {
           },
           "recommendedFallback": false,
           "rules": Object {
+            "async2": Object {
+              "no-empty-servers": "error",
+              "operation-summary": "error",
+            },
             "oas2": Object {
               "no-empty-servers": "error",
               "operation-summary": "error",
@@ -279,19 +290,19 @@ describe('StyleguideConfig.extendTypes', () => {
   };
   it('should call only oas3 types extension', () => {
     const styleguideConfig = new StyleguideConfig(testRawConfigStyleguide);
-    styleguideConfig.extendTypes({}, OasVersion.Version3_0);
+    styleguideConfig.extendTypes({}, SpecVersion.OAS3_0);
     expect(oas3).toHaveBeenCalledTimes(1);
     expect(oas2).toHaveBeenCalledTimes(0);
   });
   it('should call only oas2 types extension', () => {
     const styleguideConfig = new StyleguideConfig(testRawConfigStyleguide);
-    styleguideConfig.extendTypes({}, OasVersion.Version2);
+    styleguideConfig.extendTypes({}, SpecVersion.OAS2);
     expect(oas3).toHaveBeenCalledTimes(0);
     expect(oas2).toHaveBeenCalledTimes(1);
   });
   it('should throw error if for oas version different from 2 and 3', () => {
     const styleguideConfig = new StyleguideConfig(testRawConfigStyleguide);
-    expect(() => styleguideConfig.extendTypes({}, 'something else' as OasVersion)).toThrowError(
+    expect(() => styleguideConfig.extendTypes({}, 'something else' as SpecVersion)).toThrowError(
       'Not implemented'
     );
   });

package/src/config/builtIn.ts

@@ -3,6 +3,7 @@ import all from './all';
 import minimal from './minimal';
 import { rules as oas3Rules } from '../rules/oas3';
 import { rules as oas2Rules } from '../rules/oas2';
+import { rules as async2Rules } from '../rules/async2';
 import { preprocessors as oas3Preprocessors } from '../rules/oas3';
 import { preprocessors as oas2Preprocessors } from '../rules/oas2';
 import { decorators as oas3Decorators } from '../decorators/oas3';
@@ -24,14 +25,17 @@ export const defaultPlugin: Plugin = {
   rules: {
     oas3: oas3Rules,
     oas2: oas2Rules,
+    async2: async2Rules,
   } as CustomRulesConfig,
   preprocessors: {
     oas3: oas3Preprocessors,
     oas2: oas2Preprocessors,
+    async2: {},
   },
   decorators: {
     oas3: oas3Decorators,
     oas2: oas2Decorators,
+    async2: {},
   },
   configs: builtInConfigs,
 };

package/src/config/config-resolvers.ts

@@ -141,8 +141,8 @@ export function resolvePlugins(
       };
 
       if (pluginModule.rules) {
-        if (!pluginModule.rules.oas3 && !pluginModule.rules.oas2) {
-          throw new Error(`Plugin rules must have \`oas3\` or \`oas2\` rules "${p}.`);
+        if (!pluginModule.rules.oas3 && !pluginModule.rules.oas2 && !pluginModule.rules.async2) {
+          throw new Error(`Plugin rules must have \`oas3\`, \`oas2\` or \`async2\` rules "${p}.`);
         }
         plugin.rules = {};
         if (pluginModule.rules.oas3) {
@@ -151,11 +151,18 @@ export function resolvePlugins(
         if (pluginModule.rules.oas2) {
           plugin.rules.oas2 = prefixRules(pluginModule.rules.oas2, id);
         }
+        if (pluginModule.rules.async2) {
+          plugin.rules.async2 = prefixRules(pluginModule.rules.async2, id);
+        }
       }
       if (pluginModule.preprocessors) {
-        if (!pluginModule.preprocessors.oas3 && !pluginModule.preprocessors.oas2) {
+        if (
+          !pluginModule.preprocessors.oas3 &&
+          !pluginModule.preprocessors.oas2 &&
+          !pluginModule.preprocessors.async2
+        ) {
           throw new Error(
-            `Plugin \`preprocessors\` must have \`oas3\` or \`oas2\` preprocessors "${p}.`
+            `Plugin \`preprocessors\` must have \`oas3\`, \`oas2\` or \`async2\` preprocessors "${p}.`
           );
         }
         plugin.preprocessors = {};
@@ -165,11 +172,20 @@ export function resolvePlugins(
         if (pluginModule.preprocessors.oas2) {
           plugin.preprocessors.oas2 = prefixRules(pluginModule.preprocessors.oas2, id);
         }
+        if (pluginModule.preprocessors.async2) {
+          plugin.preprocessors.async2 = prefixRules(pluginModule.preprocessors.async2, id);
+        }
       }
 
       if (pluginModule.decorators) {
-        if (!pluginModule.decorators.oas3 && !pluginModule.decorators.oas2) {
-          throw new Error(`Plugin \`decorators\` must have \`oas3\` or \`oas2\` decorators "${p}.`);
+        if (
+          !pluginModule.decorators.oas3 &&
+          !pluginModule.decorators.oas2 &&
+          !pluginModule.decorators.async2
+        ) {
+          throw new Error(
+            `Plugin \`decorators\` must have \`oas3\`, \`oas2\` or \`async2\` decorators "${p}.`
+          );
         }
         plugin.decorators = {};
         if (pluginModule.decorators.oas3) {
@@ -178,6 +194,9 @@ export function resolvePlugins(
         if (pluginModule.decorators.oas2) {
           plugin.decorators.oas2 = prefixRules(pluginModule.decorators.oas2, id);
         }
+        if (pluginModule.decorators.async2) {
+          plugin.decorators.async2 = prefixRules(pluginModule.decorators.async2, id);
+        }
       }
 
       if (pluginModule.assertions) {