fix: remove unused components was skipping schemas with a top-level a…

…llOf. fixes #1209

.changeset/light-games-clap.md

@@ -0,0 +1,6 @@
+---
+'@redocly/openapi-core': patch
+'@redocly/cli': patch
+---
+
+Fixed an issue where remove-unused-components would not remove schemas which had a top-level allOf

packages/core/src/rules/oas2/__tests__/remove-unused-components.test.ts

@@ -152,4 +152,77 @@ describe('oas2 remove-unused-components', () => {
  },
  });
  });
+
+ it('should remove unused components even if they have a root allOf', async () => {
+ const document = parseYamlToDocument(
+ outdent`
+ swagger: '2.0'
+ paths:
+ /pets:
+ get:
+ produces:
+ - application/json
+ parameters: []
+ responses:
+ '200':
+ schema:
+ $ref: '#/definitions/Used'
+ operationId: listPets
+ summary: List all pets
+ definitions:
+ Unused:
+ allOf:
+ - type: object
+ properties:
+ one:
+ type: string
+ - type: object
+ properties:
+ two:
+ type: string
+ Used:
+ properties:
+ link:
+ $ref: '#/definitions/InnerUsed/properties/link'
+ type: object
+ `,
+ 'foobar.yaml'
+ );
+
+ const results = await bundleDocument({
+ externalRefResolver: new BaseResolver(),
+ document,
+ config: await makeConfig({}),
+ removeUnusedComponents: true,
+ });
+
+ expect(results.bundle.parsed).toEqual({
+ swagger: '2.0',
+ definitions: {
+ Used: {
+ properties: {
+ link: { $ref: '#/definitions/InnerUsed/properties/link' },
+ },
+ type: 'object',
+ },
+ },
+ paths: {
+ '/pets': {
+ get: {
+ produces: ['application/json'],
+ parameters: [],
+ summary: 'List all pets',
+ operationId: 'listPets',
+ responses: {
+ '200': {
+ schema: {
+ $ref: '#/definitions/Used',
+ },
+ },
+ },
+ },
+ },
+ },
+ });
+ });
 });

packages/core/src/rules/oas2/remove-unused-components.ts

@@ -61,10 +61,8 @@ export const RemoveUnusedComponents: Oas2Rule = () => {
  },
  },
  NamedSchemas: {
- Schema(schema, { location, key }) {
- if (!schema.allOf) {
- registerComponent(location, 'definitions', key.toString());
- }
+ Schema(_schema, { location, key }) {
+ registerComponent(location, 'definitions', key.toString());
  },
  },
  NamedParameters: {

packages/core/src/rules/oas3/__tests__/remove-unused-components.test.ts

@@ -168,4 +168,81 @@ describe('oas3 remove-unused-components', () => {
  },
  });
  });
+
+ it('should remove unused components even if they have a root allOf', async () => {
+ const document = parseYamlToDocument(
+ outdent`
+ openapi: "3.0.0"
+ paths:
+ /pets:
+ get:
+ summary: List all pets
+ operationId: listPets
+ responses:
+ '200':
+ content:
+ application/json:
+ schema:
+ $ref: '#/components/schemas/Used'
+ components:
+ schemas:
+ Unused:
+ allOf:
+ - type: object
+ properties:
+ one:
+ type: string
+ - type: object
+ properties:
+ two:
+ type: string
+ Used:
+ type: object
+ properties:
+ link:
+ type: string
+ `,
+ 'foobar.yaml'
+ );
+
+ const results = await bundleDocument({
+ externalRefResolver: new BaseResolver(),
+ document,
+ config: await makeConfig({}),
+ removeUnusedComponents: true,
+ });
+
+ expect(results.bundle.parsed).toEqual({
+ openapi: '3.0.0',
+ paths: {
+ '/pets': {
+ get: {
+ summary: 'List all pets',
+ operationId: 'listPets',
+ responses: {
+ '200': {
+ content: {
+ 'application/json': {
+ schema: {
+ $ref: '#/components/schemas/Used',
+ },
+ },
+ },
+ },
+ },
+ },
+ },
+ },
+ components: {
+ schemas: {
+ Used: {
+ type: 'object',
+ properties: {
+ link: { type: 'string' },
+ },
+ },
+ },
+ },
+ });
+ });
 });

packages/core/src/rules/oas3/remove-unused-components.ts

@@ -65,10 +65,8 @@ export const RemoveUnusedComponents: Oas3Rule = () => {
  },
  },
  NamedSchemas: {
- Schema(schema, { location, key }) {
- if (!schema.allOf) {
- registerComponent(location, 'schemas', key.toString());
- }
+ Schema(_schema, { location, key }) {
+ registerComponent(location, 'schemas', key.toString());
  },
  },
  NamedParameters: {