@redocly/openapi-core 1.25.8 → 1.25.10

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @redocly/openapi-core
 
+## 1.25.10
+
+### Patch Changes
+
+- Fixed `component-name-unique` problems to include correct location.
+- Fixed the `remove-x-internal` decorator, which was not removing the reference in the corresponding discriminator mapping while removing the original `$ref`.
+- Updated @redocly/config to v0.16.0.
+
+## 1.25.9
+
+### Patch Changes
+
+- Updated @redocly/config to v0.15.0.
+
 ## 1.25.8
 
 ### Patch Changes

package/lib/config/config.js

@@ -11,7 +11,7 @@ const utils_2 = require("./utils");
 const ref_utils_1 = require("../ref-utils");
 exports.IGNORE_FILE = '.redocly.lint-ignore.yaml';
 const IGNORE_BANNER = `# This file instructs Redocly's linter to ignore the rules contained for specific parts of your API.\n` +
-    `# See https://redoc.ly/docs/cli/ for more information.\n`;
+    `# See https://redocly.com/docs/cli/ for more information.\n`;
 function getIgnoreFilePath(configFile) {
     if (configFile) {
         return (0, utils_1.doesYamlFileExist)(configFile)

package/lib/decorators/common/remove-x-internal.js

@@ -4,22 +4,29 @@ exports.RemoveXInternal = void 0;
 const utils_1 = require("../../utils");
 const ref_utils_1 = require("../../ref-utils");
 const DEFAULT_INTERNAL_PROPERTY_NAME = 'x-internal';
-const RemoveXInternal = ({ internalFlagProperty }) => {
-    const hiddenTag = internalFlagProperty || DEFAULT_INTERNAL_PROPERTY_NAME;
-    function removeInternal(node, ctx) {
+const RemoveXInternal = ({ internalFlagProperty = DEFAULT_INTERNAL_PROPERTY_NAME, }) => {
+    function removeInternal(node, ctx, originalMapping) {
         const { parent, key } = ctx;
         let didDelete = false;
         if (Array.isArray(node)) {
             for (let i = 0; i < node.length; i++) {
                 if ((0, ref_utils_1.isRef)(node[i])) {
                     const resolved = ctx.resolve(node[i]);
-                    if (resolved.node?.[hiddenTag]) {
+                    if (resolved.node?.[internalFlagProperty]) {
+                        // First, remove the reference in the discriminator mapping, if it exists:
+                        if ((0, utils_1.isPlainObject)(parent.discriminator?.mapping)) {
+                            for (const mapping in parent.discriminator.mapping) {
+                                if (originalMapping?.[mapping] === node[i].$ref) {
+                                    delete parent.discriminator.mapping[mapping];
+                                }
+                            }
+                        }
                         node.splice(i, 1);
                         didDelete = true;
                         i--;
                     }
                 }
-                if (node[i]?.[hiddenTag]) {
+                if (node[i]?.[internalFlagProperty]) {
                     node.splice(i, 1);
                     didDelete = true;
                     i--;
@@ -28,15 +35,14 @@ const RemoveXInternal = ({ internalFlagProperty }) => {
         }
         else if ((0, utils_1.isPlainObject)(node)) {
             for (const key of Object.keys(node)) {
-                node = node;
                 if ((0, ref_utils_1.isRef)(node[key])) {
                     const resolved = ctx.resolve(node[key]);
-                    if (resolved.node?.[hiddenTag]) {
+                    if ((0, utils_1.isPlainObject)(resolved.node) && resolved.node?.[internalFlagProperty]) {
                         delete node[key];
                         didDelete = true;
                     }
                 }
-                if (node[key]?.[hiddenTag]) {
+                if ((0, utils_1.isPlainObject)(node[key]) && node[key]?.[internalFlagProperty]) {
                     delete node[key];
                     didDelete = true;
                 }
@@ -46,10 +52,16 @@ const RemoveXInternal = ({ internalFlagProperty }) => {
             delete parent[key];
         }
     }
+    let originalMapping = {};
     return {
+        DiscriminatorMapping: {
+            enter: (mapping) => {
+                originalMapping = structuredClone(mapping);
+            },
+        },
         any: {
             enter: (node, ctx) => {
-                removeInternal(node, ctx);
+                removeInternal(node, ctx, originalMapping);
             },
         },
     };

package/lib/resolve.js

@@ -101,7 +101,7 @@ class BaseResolver {
             }
             else {
                 if (fs.lstatSync(absoluteRef).isDirectory()) {
-                    throw new Error(`Expected a file but received a folder at ${absoluteRef}`);
+                    throw new Error(`Expected a file but received a folder at ${absoluteRef}.`);
                 }
                 const content = await fs.promises.readFile(absoluteRef, 'utf-8');
                 // In some cases file have \r\n line delimeters like on windows, we should skip it.
@@ -165,7 +165,7 @@ const resolvableScalarType = { name: 'scalar', properties: {} };
 async function resolveDocument(opts) {
     const { rootDocument, externalRefResolver, rootType } = opts;
     const resolvedRefMap = new Map();
-    const seedNodes = new Set(); // format "${type}::${absoluteRef}${pointer}"
+    const seenNodes = new Set(); // format "${type}::${absoluteRef}${pointer}"
     const resolvePromises = [];
     resolveRefsInParallel(rootDocument.parsed, rootDocument, '#/', rootType);
     let resolved;
@@ -182,10 +182,10 @@ async function resolveDocument(opts) {
                 return;
             }
             const nodeId = `${type.name}::${nodeAbsoluteRef}`;
-            if (seedNodes.has(nodeId)) {
+            if (seenNodes.has(nodeId)) {
                 return;
             }
-            seedNodes.add(nodeId);
+            seenNodes.add(nodeId);
             const [_, anchor] = Object.entries(node).find(([key]) => key === '$anchor') || [];
             if (anchor) {
                 anchorRefsMap.set(`#${anchor}`, node);

package/lib/rules/oas3/component-name-unique.js

@@ -34,27 +34,31 @@ const ComponentNameUnique = (options) => {
                     const resolvedRef = resolve(ref);
                     if (!resolvedRef.location)
                         return;
-                    addComponentFromAbsoluteLocation(typeName, resolvedRef.location.absolutePointer.toString());
+                    addComponentFromAbsoluteLocation(typeName, resolvedRef.location);
                 }
             },
         },
         Root: {
             leave(root, ctx) {
                 components.forEach((value, key, _) => {
-                    if (value.size > 1) {
+                    if (value.absolutePointers.size > 1) {
                         const component = getComponentFromKey(key);
                         const optionComponentName = getOptionComponentNameForTypeName(component.typeName);
-                        const definitions = Array.from(value)
-                            .map((v) => `- ${v}`)
-                            .join('\n');
-                        const problem = {
-                            message: `Component '${optionComponentName}/${component.componentName}' is not unique. It is defined at:\n${definitions}`,
-                        };
                         const componentSeverity = optionComponentName ? options[optionComponentName] : null;
-                        if (componentSeverity) {
-                            problem.forceSeverity = componentSeverity;
+                        for (const location of value.locations) {
+                            const definitions = Array.from(value.absolutePointers)
+                                .filter((v) => v !== location.absolutePointer.toString())
+                                .map((v) => `- ${v}`)
+                                .join('\n');
+                            const problem = {
+                                message: `Component '${optionComponentName}/${component.componentName}' is not unique. It is also defined at:\n${definitions}`,
+                                location: location,
+                            };
+                            if (componentSeverity) {
+                                problem.forceSeverity = componentSeverity;
+                            }
+                            ctx.report(problem);
                         }
-                        ctx.report(problem);
                     }
                 });
             },
@@ -63,28 +67,28 @@ const ComponentNameUnique = (options) => {
     if (options.schemas != 'off') {
         rule.NamedSchemas = {
             Schema(_, { location }) {
-                addComponentFromAbsoluteLocation(TYPE_NAME_SCHEMA, location.absolutePointer.toString());
+                addComponentFromAbsoluteLocation(TYPE_NAME_SCHEMA, location);
             },
         };
     }
     if (options.responses != 'off') {
         rule.NamedResponses = {
             Response(_, { location }) {
-                addComponentFromAbsoluteLocation(TYPE_NAME_RESPONSE, location.absolutePointer.toString());
+                addComponentFromAbsoluteLocation(TYPE_NAME_RESPONSE, location);
             },
         };
     }
     if (options.parameters != 'off') {
         rule.NamedParameters = {
             Parameter(_, { location }) {
-                addComponentFromAbsoluteLocation(TYPE_NAME_PARAMETER, location.absolutePointer.toString());
+                addComponentFromAbsoluteLocation(TYPE_NAME_PARAMETER, location);
             },
         };
     }
     if (options.requestBodies != 'off') {
         rule.NamedRequestBodies = {
             RequestBody(_, { location }) {
-                addComponentFromAbsoluteLocation(TYPE_NAME_REQUEST_BODY, location.absolutePointer.toString());
+                addComponentFromAbsoluteLocation(TYPE_NAME_REQUEST_BODY, location);
             },
         };
     }
@@ -98,15 +102,22 @@ const ComponentNameUnique = (options) => {
         }
         return componentName;
     }
-    function addFoundComponent(typeName, componentName, absoluteLocation) {
+    function addFoundComponent(typeName, componentName, location) {
         const key = getKeyForComponent(typeName, componentName);
-        const locations = components.get(key) ?? new Set();
-        locations.add(absoluteLocation);
-        components.set(key, locations);
+        const entry = components.get(key) ?? {
+            absolutePointers: new Set(),
+            locations: [],
+        };
+        const absoluteLocation = location.absolutePointer.toString();
+        if (!entry.absolutePointers.has(absoluteLocation)) {
+            entry.absolutePointers.add(absoluteLocation);
+            entry.locations.push(location);
+        }
+        components.set(key, entry);
     }
-    function addComponentFromAbsoluteLocation(typeName, absoluteLocation) {
-        const componentName = getComponentNameFromAbsoluteLocation(absoluteLocation);
-        addFoundComponent(typeName, componentName, absoluteLocation);
+    function addComponentFromAbsoluteLocation(typeName, location) {
+        const componentName = getComponentNameFromAbsoluteLocation(location.absolutePointer.toString());
+        addFoundComponent(typeName, componentName, location);
     }
 };
 exports.ComponentNameUnique = ComponentNameUnique;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@redocly/openapi-core",
-  "version": "1.25.8",
+  "version": "1.25.10",
   "description": "",
   "main": "lib/index.js",
   "engines": {
@@ -32,11 +32,11 @@
     "oas"
   ],
   "contributors": [
-    "Roman Hotsiy <roman@redoc.ly> (https://redoc.ly/)"
+    "Roman Hotsiy <roman@redocly.com> (https://redocly.com/)"
   ],
   "dependencies": {
     "@redocly/ajv": "^8.11.2",
-    "@redocly/config": "^0.12.1",
+    "@redocly/config": "^0.16.0",
     "colorette": "^1.2.0",
     "https-proxy-agent": "^7.0.4",
     "js-levenshtein": "^1.1.6",

package/src/__tests__/lint.test.ts

@@ -199,7 +199,6 @@ const testPortalConfig = parseYamlToDocument(
         editPage:
           baseUrl: https://test.com
       graphql:
-        pagination: section
         menu:
           requireExactGroups: false
           groups:

package/src/benchmark/benches/rebilly.yaml

@@ -10,9 +10,6 @@ info:
     name: Rebilly
     url: 'https://www.rebilly.com/api-license/'
   termsOfService: 'https://www.rebilly.com/terms-of-use/'
-  x-logo:
-    url: 'https://rebilly-core.redoc.ly/rb_apiLogo.svg'
-    backgroundColor: '#0033A0'
   description: >
     # Introduction
 

package/src/bundle.ts

@@ -398,7 +398,7 @@ function makeBundleVisitor(
       },
     },
     Root: {
-      enter(root: any, ctx: any) {
+      enter(root: any, ctx: UserContext) {
         rootLocation = ctx.location;
         if (version === SpecMajorVersion.OAS3) {
           components = root.components = root.components || {};
@@ -417,7 +417,7 @@ function makeBundleVisitor(
 
   if (version === SpecMajorVersion.OAS3) {
     visitor.DiscriminatorMapping = {
-      leave(mapping: Record<string, string>, ctx: any) {
+      leave(mapping: Record<string, string>, ctx: UserContext) {
         for (const name of Object.keys(mapping)) {
           const $ref = mapping[name];
           const resolved = ctx.resolve({ $ref });
@@ -446,7 +446,7 @@ function makeBundleVisitor(
 
   function saveComponent(
     componentType: string,
-    target: { node: any; location: Location },
+    target: { node: unknown; location: Location },
     ctx: UserContext
   ) {
     components[componentType] = components[componentType] || {};
@@ -464,8 +464,8 @@ function makeBundleVisitor(
   }
 
   function isEqualOrEqualRef(
-    node: any,
-    target: { node: any; location: Location },
+    node: unknown,
+    target: { node: unknown; location: Location },
     ctx: UserContext
   ) {
     if (
@@ -480,7 +480,7 @@ function makeBundleVisitor(
   }
 
   function getComponentName(
-    target: { node: any; location: Location },
+    target: { node: unknown; location: Location },
     componentType: string,
     ctx: UserContext
   ) {

package/src/config/config.ts

@@ -34,7 +34,7 @@ import type {
 export const IGNORE_FILE = '.redocly.lint-ignore.yaml';
 const IGNORE_BANNER =
   `# This file instructs Redocly's linter to ignore the rules contained for specific parts of your API.\n` +
-  `# See https://redoc.ly/docs/cli/ for more information.\n`;
+  `# See https://redocly.com/docs/cli/ for more information.\n`;
 
 function getIgnoreFilePath(configFile?: string): string | undefined {
   if (configFile) {

package/src/decorators/__tests__/remove-x-internal.test.ts

@@ -271,6 +271,103 @@ describe('oas3 remove-x-internal', () => {
 
     `);
   });
+
+  it('should remove $refs and the corresponding discriminator mapping', async () => {
+    const testDoc = parseYamlToDocument(
+      outdent`
+        openapi: 3.1.0
+        info: {}
+        paths:
+          /test:
+            post:
+              requestBody:
+                content:
+                  application/json:
+                    schema:
+                      $ref: '#/components/schemas/christmas-tree'
+        components:
+          schemas:
+            christmas-tree:
+              type: array
+              items:
+                discriminator:
+                  propertyName: type
+                  mapping:
+                    candy-cane: '#/components/schemas/candy-cane'
+                    popcorn: '#/components/schemas/popcorn'
+                    cranberry: '#/components/schemas/cranberry'
+                anyOf:
+                  - $ref: '#/components/schemas/candy-cane'
+                  - $ref: '#/components/schemas/popcorn'
+                  - $ref: '#/components/schemas/cranberry'
+            candy-cane:
+              x-internal: true
+              title: candy-cane
+              type: object
+              properties:
+                type:
+                  type: string
+                  enum: [candy-cane]
+            popcorn:
+              type: object
+              properties:
+                type:
+                  type: string
+                  enum: [popcorn]
+            cranberry:
+              type: object
+              properties:
+                type:
+                  type: string
+                  enum: [cranberry]
+      `
+    );
+    const { bundle: res } = await bundleDocument({
+      document: testDoc,
+      externalRefResolver: new BaseResolver(),
+      config: await makeConfig({ rules: {}, decorators: { 'remove-x-internal': 'on' } }),
+    });
+    expect(res.parsed).toMatchInlineSnapshot(`
+      openapi: 3.1.0
+      info: {}
+      paths:
+        /test:
+          post:
+            requestBody:
+              content:
+                application/json:
+                  schema:
+                    $ref: '#/components/schemas/christmas-tree'
+      components:
+        schemas:
+          christmas-tree:
+            type: array
+            items:
+              discriminator:
+                propertyName: type
+                mapping:
+                  popcorn: '#/components/schemas/popcorn'
+                  cranberry: '#/components/schemas/cranberry'
+              anyOf:
+                - $ref: '#/components/schemas/popcorn'
+                - $ref: '#/components/schemas/cranberry'
+          popcorn:
+            type: object
+            properties:
+              type:
+                type: string
+                enum:
+                  - popcorn
+          cranberry:
+            type: object
+            properties:
+              type:
+                type: string
+                enum:
+                  - cranberry
+
+    `);
+  });
 });
 
 describe('oas2 remove-x-internal', () => {

package/src/decorators/common/remove-x-internal.ts

@@ -6,23 +6,35 @@ import type { UserContext } from '../../walk';
 
 const DEFAULT_INTERNAL_PROPERTY_NAME = 'x-internal';
 
-export const RemoveXInternal: Oas3Decorator | Oas2Decorator = ({ internalFlagProperty }) => {
-  const hiddenTag: string = internalFlagProperty || DEFAULT_INTERNAL_PROPERTY_NAME;
-
-  function removeInternal(node: any, ctx: UserContext) {
+export const RemoveXInternal: Oas3Decorator | Oas2Decorator = ({
+  internalFlagProperty = DEFAULT_INTERNAL_PROPERTY_NAME,
+}) => {
+  function removeInternal(
+    node: unknown,
+    ctx: UserContext,
+    originalMapping: Record<string, string>
+  ) {
     const { parent, key } = ctx;
     let didDelete = false;
     if (Array.isArray(node)) {
       for (let i = 0; i < node.length; i++) {
         if (isRef(node[i])) {
           const resolved = ctx.resolve(node[i]);
-          if (resolved.node?.[hiddenTag]) {
+          if (resolved.node?.[internalFlagProperty]) {
+            // First, remove the reference in the discriminator mapping, if it exists:
+            if (isPlainObject(parent.discriminator?.mapping)) {
+              for (const mapping in parent.discriminator.mapping) {
+                if (originalMapping?.[mapping] === node[i].$ref) {
+                  delete parent.discriminator.mapping[mapping];
+                }
+              }
+            }
             node.splice(i, 1);
             didDelete = true;
             i--;
           }
         }
-        if (node[i]?.[hiddenTag]) {
+        if (node[i]?.[internalFlagProperty]) {
           node.splice(i, 1);
           didDelete = true;
           i--;
@@ -30,15 +42,14 @@ export const RemoveXInternal: Oas3Decorator | Oas2Decorator = ({ internalFlagPro
       }
     } else if (isPlainObject(node)) {
       for (const key of Object.keys(node)) {
-        node = node as any;
         if (isRef(node[key])) {
-          const resolved = ctx.resolve<any>(node[key]);
-          if (resolved.node?.[hiddenTag]) {
+          const resolved = ctx.resolve(node[key]);
+          if (isPlainObject(resolved.node) && resolved.node?.[internalFlagProperty]) {
             delete node[key];
             didDelete = true;
           }
         }
-        if (node[key]?.[hiddenTag]) {
+        if (isPlainObject(node[key]) && node[key]?.[internalFlagProperty]) {
           delete node[key];
           didDelete = true;
         }
@@ -50,10 +61,16 @@ export const RemoveXInternal: Oas3Decorator | Oas2Decorator = ({ internalFlagPro
     }
   }
 
+  let originalMapping: Record<string, string> = {};
   return {
+    DiscriminatorMapping: {
+      enter: (mapping: Record<string, string>) => {
+        originalMapping = structuredClone(mapping);
+      },
+    },
     any: {
       enter: (node, ctx) => {
-        removeInternal(node, ctx);
+        removeInternal(node, ctx, originalMapping);
       },
     },
   };

package/src/resolve.ts

@@ -126,7 +126,7 @@ export class BaseResolver {
         return new Source(absoluteRef, body, mimeType);
       } else {
         if (fs.lstatSync(absoluteRef).isDirectory()) {
-          throw new Error(`Expected a file but received a folder at ${absoluteRef}`);
+          throw new Error(`Expected a file but received a folder at ${absoluteRef}.`);
         }
         const content = await fs.promises.readFile(absoluteRef, 'utf-8');
         // In some cases file have \r\n line delimeters like on windows, we should skip it.
@@ -233,7 +233,7 @@ export async function resolveDocument(opts: {
 }): Promise<ResolvedRefMap> {
   const { rootDocument, externalRefResolver, rootType } = opts;
   const resolvedRefMap: ResolvedRefMap = new Map();
-  const seedNodes = new Set<string>(); // format "${type}::${absoluteRef}${pointer}"
+  const seenNodes = new Set<string>(); // format "${type}::${absoluteRef}${pointer}"
 
   const resolvePromises: Array<Promise<void>> = [];
   resolveRefsInParallel(rootDocument.parsed, rootDocument, '#/', rootType);
@@ -262,11 +262,11 @@ export async function resolveDocument(opts: {
       }
 
       const nodeId = `${type.name}::${nodeAbsoluteRef}`;
-      if (seedNodes.has(nodeId)) {
+      if (seenNodes.has(nodeId)) {
         return;
       }
 
-      seedNodes.add(nodeId);
+      seenNodes.add(nodeId);
 
       const [_, anchor] = Object.entries(node).find(([key]) => key === '$anchor') || [];
       if (anchor) {