@redocly/cli 1.10.6 → 1.11.0

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # @redocly/cli
 
+## 1.11.0
+
+### Minor Changes
+
+- Removed additional operations from the `join` command; use `lint` and/or `bundle` for operations such as `lint` and `decorate`.
+- Removed lint support from the bundle command to support a wider range of use cases. Users should update to [run lint and bundle separately](https://redocly.com/docs/cli/guides/lint-and-bundle/).
+- Added support for a `github-actions` output format for the `lint` command to annotate reported problems on files when used in a GitHub Actions workflow.
+
+### Patch Changes
+
+- Fixed [`no-invalid-media-type-examples`](https://redocly.com/docs/cli/rules/no-invalid-media-type-examples/) rule `externalValue` example validation.
+- Updated @redocly/openapi-core to v1.11.0.
+
 ## 1.10.6
 
 ### Patch Changes

package/lib/__tests__/commands/bundle.test.js

@@ -28,18 +28,15 @@ describe('bundle', () => {
         jest.spyOn(process.stderr, 'write').mockImplementation(() => true);
     });
     afterEach(() => {
-        openapi_core_1.lint.mockClear();
         openapi_core_1.bundle.mockClear();
         openapi_core_1.getTotals.mockReset();
     });
-    it('bundles definitions w/o linting', () => __awaiter(void 0, void 0, void 0, function* () {
+    it('bundles definitions', () => __awaiter(void 0, void 0, void 0, function* () {
         const apis = ['foo.yaml', 'bar.yaml'];
         yield (0, wrapper_1.commandWrapper)(bundle_1.handleBundle)({
             apis,
             ext: 'yaml',
-            format: 'codeframe',
         });
-        expect(openapi_core_1.lint).toBeCalledTimes(0);
         expect(openapi_core_1.bundle).toBeCalledTimes(apis.length);
     }));
     it('exits with code 0 when bundles definitions', () => __awaiter(void 0, void 0, void 0, function* () {
@@ -47,39 +44,20 @@ describe('bundle', () => {
         yield (0, wrapper_1.commandWrapper)(bundle_1.handleBundle)({
             apis,
             ext: 'yaml',
-            format: 'codeframe',
         });
         yield (exitCb === null || exitCb === void 0 ? void 0 : exitCb());
         expect(processExitMock).toHaveBeenCalledWith(0);
     }));
-    it('bundles definitions w/ linting', () => __awaiter(void 0, void 0, void 0, function* () {
-        const apis = ['foo.yaml', 'bar.yaml', 'foobar.yaml'];
-        openapi_core_1.getTotals.mockReturnValue({
-            errors: 0,
-            warnings: 0,
-            ignored: 0,
-        });
-        yield (0, wrapper_1.commandWrapper)(bundle_1.handleBundle)({
-            apis,
-            ext: 'yaml',
-            format: 'codeframe',
-            lint: true,
-        });
-        expect(openapi_core_1.lint).toBeCalledTimes(apis.length);
-        expect(openapi_core_1.bundle).toBeCalledTimes(apis.length);
-    }));
-    it('exits with code 0 when bundles definitions w/linting w/o errors', () => __awaiter(void 0, void 0, void 0, function* () {
+    it('exits with code 0 when bundles definitions w/o errors', () => __awaiter(void 0, void 0, void 0, function* () {
         const apis = ['foo.yaml', 'bar.yaml', 'foobar.yaml'];
         yield (0, wrapper_1.commandWrapper)(bundle_1.handleBundle)({
             apis,
             ext: 'yaml',
-            format: 'codeframe',
-            lint: true,
         });
         yield (exitCb === null || exitCb === void 0 ? void 0 : exitCb());
         expect(processExitMock).toHaveBeenCalledWith(0);
     }));
-    it('exits with code 1 when bundles definitions w/linting w/errors', () => __awaiter(void 0, void 0, void 0, function* () {
+    it('exits with code 1 when bundles definitions w/errors', () => __awaiter(void 0, void 0, void 0, function* () {
         const apis = ['foo.yaml'];
         openapi_core_1.getTotals.mockReturnValue({
             errors: 1,
@@ -89,10 +67,7 @@ describe('bundle', () => {
         yield (0, wrapper_1.commandWrapper)(bundle_1.handleBundle)({
             apis,
             ext: 'yaml',
-            format: 'codeframe',
-            lint: true,
         });
-        expect(openapi_core_1.lint).toBeCalledTimes(apis.length);
         yield (exitCb === null || exitCb === void 0 ? void 0 : exitCb());
         expect(processExitMock).toHaveBeenCalledWith(1);
     }));
@@ -104,8 +79,6 @@ describe('bundle', () => {
         yield (0, wrapper_1.commandWrapper)(bundle_1.handleBundle)({
             apis,
             ext: 'json',
-            format: 'codeframe',
-            lint: false,
         });
         expect(miscellaneous_1.handleError).toHaveBeenCalledTimes(1);
         expect(miscellaneous_1.handleError).toHaveBeenCalledWith(new Error('Invalid definition'), 'invalid.json');
@@ -120,8 +93,6 @@ describe('bundle', () => {
         yield (0, wrapper_1.commandWrapper)(bundle_1.handleBundle)({
             apis,
             ext: 'yaml',
-            format: 'codeframe',
-            lint: false,
         });
         expect(miscellaneous_1.handleError).toHaveBeenCalledTimes(0);
     }));

package/lib/__tests__/commands/join.test.js

@@ -94,17 +94,6 @@ describe('handleJoin', () => {
         expect(config.styleguide.skipDecorators).toHaveBeenCalled();
         expect(config.styleguide.skipPreprocessors).toHaveBeenCalled();
     }));
-    it('should not call skipDecorators and skipPreprocessors', () => __awaiter(void 0, void 0, void 0, function* () {
-        openapi_core_1.detectSpec.mockReturnValue('oas3_0');
-        yield (0, join_1.handleJoin)({
-            apis: ['first.yaml', 'second.yaml'],
-            decorate: true,
-            preprocess: true,
-        }, config_1.ConfigFixture, 'cli-version');
-        const config = (0, openapi_core_2.loadConfig)();
-        expect(config.styleguide.skipDecorators).not.toHaveBeenCalled();
-        expect(config.styleguide.skipPreprocessors).not.toHaveBeenCalled();
-    }));
     it('should handle join with prefix-components-with-info-prop and null values', () => __awaiter(void 0, void 0, void 0, function* () {
         openapi_core_1.detectSpec.mockReturnValue('oas3_0');
         yield (0, join_1.handleJoin)({

package/lib/commands/bundle.d.ts

@@ -1,16 +1,13 @@
-import { Config, OutputFormat } from '@redocly/openapi-core';
+import { Config } from '@redocly/openapi-core';
 import type { OutputExtensions, Skips } from '../types';
 export type BundleOptions = {
     apis?: string[];
-    'max-problems'?: number;
     extends?: string[];
     config?: string;
-    format?: OutputFormat;
     output?: string;
     ext: OutputExtensions;
     dereferenced?: boolean;
     force?: boolean;
-    lint?: boolean;
     metafile?: string;
     'remove-unused-components'?: boolean;
     'keep-url-references'?: boolean;

package/lib/commands/bundle.js

@@ -34,44 +34,15 @@ function handleBundle(argv, config, version) {
             ((_c = (_b = (_a = config.rawConfig) === null || _a === void 0 ? void 0 : _a.styleguide) === null || _b === void 0 ? void 0 : _b.decorators) === null || _c === void 0 ? void 0 : _c.hasOwnProperty('remove-unused-components'));
         const apis = yield (0, miscellaneous_1.getFallbackApisOrExit)(argv.apis, config);
         const totals = { errors: 0, warnings: 0, ignored: 0 };
-        const maxProblems = argv['max-problems'];
-        const deprecatedOptions = [
-            'lint',
-            'format',
-            'skip-rule',
-            'extends',
-            'max-problems',
-        ];
+        const deprecatedOptions = [];
         (0, miscellaneous_2.checkForDeprecatedOptions)(argv, deprecatedOptions);
         for (const { path, alias } of apis) {
             try {
                 const startedAt = perf_hooks_1.performance.now();
                 const resolvedConfig = (0, openapi_core_1.getMergedConfig)(config, alias);
                 const { styleguide } = resolvedConfig;
-                styleguide.skipRules(argv['skip-rule']);
                 styleguide.skipPreprocessors(argv['skip-preprocessor']);
                 styleguide.skipDecorators(argv['skip-decorator']);
-                if (argv.lint) {
-                    (0, miscellaneous_1.checkIfRulesetExist)(styleguide.rules);
-                    if (config.styleguide.recommendedFallback) {
-                        process.stderr.write(`No configurations were provided -- using built in ${(0, colorette_1.blue)('recommended')} configuration by default.\n\n`);
-                    }
-                    const results = yield (0, openapi_core_1.lint)({
-                        ref: path,
-                        config: resolvedConfig,
-                    });
-                    const fileLintTotals = (0, openapi_core_1.getTotals)(results);
-                    totals.errors += fileLintTotals.errors;
-                    totals.warnings += fileLintTotals.warnings;
-                    totals.ignored += fileLintTotals.ignored;
-                    (0, openapi_core_1.formatProblems)(results, {
-                        format: argv.format || 'codeframe',
-                        totals: fileLintTotals,
-                        version,
-                        maxProblems: maxProblems,
-                    });
-                    (0, miscellaneous_1.printLintTotals)(fileLintTotals, 2);
-                }
                 process.stderr.write((0, colorette_1.gray)(`bundling ${path}...\n`));
                 const _f = yield (0, openapi_core_1.bundle)({
                     config: resolvedConfig,
@@ -96,8 +67,7 @@ function handleBundle(argv, config, version) {
                 totals.warnings += fileTotals.warnings;
                 totals.ignored += fileTotals.ignored;
                 (0, openapi_core_1.formatProblems)(problems, {
-                    format: argv.format || 'codeframe',
-                    maxProblems: maxProblems,
+                    format: 'codeframe',
                     totals: fileTotals,
                     version,
                 });

package/lib/commands/join.d.ts

@@ -2,9 +2,6 @@ import { Config } from '@redocly/openapi-core';
 import type { RuleSeverity } from '@redocly/openapi-core';
 export type JoinOptions = {
     apis: string[];
-    lint?: boolean;
-    decorate?: boolean;
-    preprocess?: boolean;
     'prefix-tags-with-info-prop'?: string;
     'prefix-tags-with-filename'?: boolean;
     'prefix-components-with-info-prop'?: string;

package/lib/commands/join.js

@@ -28,7 +28,6 @@ function handleJoin(argv, config, packageVersion) {
         if (argv.apis.length < 2) {
             return (0, miscellaneous_1.exitWithError)(`At least 2 apis should be provided. \n\n`);
         }
-        (0, miscellaneous_1.checkForDeprecatedOptions)(argv, ['lint']);
         const fileExtension = (0, miscellaneous_1.getAndValidateFileExtension)(argv.output || argv.apis[0]);
         const { 'prefix-components-with-info-prop': prefixComponentsWithInfoProp, 'prefix-tags-with-filename': prefixTagsWithFilename, 'prefix-tags-with-info-prop': prefixTagsWithInfoProp, 'without-x-tag-groups': withoutXTagGroups, output: specFilename = `openapi.${fileExtension}`, } = argv;
         const usedTagsOptions = [
@@ -42,22 +41,18 @@ function handleJoin(argv, config, packageVersion) {
         const apis = yield (0, miscellaneous_1.getFallbackApisOrExit)(argv.apis, config);
         const externalRefResolver = new openapi_core_1.BaseResolver(config.resolve);
         const documents = yield Promise.all(apis.map(({ path }) => externalRefResolver.resolveDocument(null, path, true)));
-        if (!argv.decorate) {
-            const decorators = new Set([
-                ...Object.keys(config.styleguide.decorators.oas3_0),
-                ...Object.keys(config.styleguide.decorators.oas3_1),
-                ...Object.keys(config.styleguide.decorators.oas2),
-            ]);
-            config.styleguide.skipDecorators(Array.from(decorators));
-        }
-        if (!argv.preprocess) {
-            const preprocessors = new Set([
-                ...Object.keys(config.styleguide.preprocessors.oas3_0),
-                ...Object.keys(config.styleguide.preprocessors.oas3_1),
-                ...Object.keys(config.styleguide.preprocessors.oas2),
-            ]);
-            config.styleguide.skipPreprocessors(Array.from(preprocessors));
-        }
+        const decorators = new Set([
+            ...Object.keys(config.styleguide.decorators.oas3_0),
+            ...Object.keys(config.styleguide.decorators.oas3_1),
+            ...Object.keys(config.styleguide.decorators.oas2),
+        ]);
+        config.styleguide.skipDecorators(Array.from(decorators));
+        const preprocessors = new Set([
+            ...Object.keys(config.styleguide.preprocessors.oas3_0),
+            ...Object.keys(config.styleguide.preprocessors.oas3_1),
+            ...Object.keys(config.styleguide.preprocessors.oas2),
+        ]);
+        config.styleguide.skipPreprocessors(Array.from(preprocessors));
         const bundleResults = yield Promise.all(documents.map((document) => (0, openapi_core_1.bundleDocument)({
             document,
             config: config.styleguide,
@@ -70,7 +65,7 @@ function handleJoin(argv, config, packageVersion) {
             if (fileTotals.errors) {
                 (0, openapi_core_1.formatProblems)(problems, {
                     totals: fileTotals,
-                    version: document.parsed.version,
+                    version: packageVersion,
                 });
                 (0, miscellaneous_1.exitWithError)(`❌ Errors encountered while bundling ${(0, colorette_1.blue)(document.source.absoluteRef)}: join will not proceed.\n`);
             }
@@ -91,11 +86,6 @@ function handleJoin(argv, config, packageVersion) {
                 return (0, miscellaneous_1.exitWithError)(`${e.message}: ${(0, colorette_1.blue)(document.source.absoluteRef)}`);
             }
         }
-        if (argv.lint) {
-            for (const document of documents) {
-                yield validateApi(document, config.styleguide, externalRefResolver, packageVersion);
-            }
-        }
         const joinedDef = {};
         const potentialConflicts = {
             tags: {},
@@ -561,19 +551,6 @@ function getInfoPrefix(info, prefixArg, type) {
         (0, miscellaneous_1.exitWithError)(`${(0, colorette_1.yellow)(`prefix-${type}-with-info-prop`)} argument value length should not exceed 50 characters. \n\n`);
     return info[prefixArg].replaceAll(/\s/g, '_');
 }
-function validateApi(document, config, externalRefResolver, packageVersion) {
-    return __awaiter(this, void 0, void 0, function* () {
-        try {
-            const results = yield (0, openapi_core_1.lintDocument)({ document, config, externalRefResolver });
-            const fileTotals = (0, openapi_core_1.getTotals)(results);
-            (0, openapi_core_1.formatProblems)(results, { format: 'stylish', totals: fileTotals, version: packageVersion });
-            (0, miscellaneous_1.printLintTotals)(fileTotals, 2);
-        }
-        catch (err) {
-            (0, miscellaneous_1.handleError)(err, document.parsed);
-        }
-    });
-}
 function replace$Refs(obj, componentsPrefix) {
     (0, split_1.crawl)(obj, (node) => {
         if (node.$ref && typeof node.$ref === 'string' && (0, split_1.startsWithComponents)(node.$ref)) {

package/lib/index.js

@@ -12,6 +12,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 Object.defineProperty(exports, "__esModule", { value: true });
 require("./utils/assert-node-version");
 const yargs = require("yargs");
+const colors = require("colorette");
 const types_1 = require("./types");
 const openapi_core_1 = require("@redocly/openapi-core");
 const preview_docs_1 = require("./commands/preview-docs");
@@ -92,9 +93,6 @@ yargs
     demandOption: true,
 })
     .option({
-    lint: { description: 'Lint descriptions', type: 'boolean', default: false, hidden: true },
-    decorate: { description: 'Run decorators', type: 'boolean', default: false },
-    preprocess: { description: 'Run preprocessors', type: 'boolean', default: false },
     'prefix-tags-with-info-prop': {
         description: 'Prefix tags with property value from info object.',
         requiresArg: true,
@@ -129,7 +127,34 @@ yargs
         choices: ['warn', 'error', 'off'],
         default: 'warn',
     },
+    lint: {
+        hidden: true,
+        deprecated: true,
+    },
+    decorate: {
+        hidden: true,
+        deprecated: true,
+    },
+    preprocess: {
+        hidden: true,
+        deprecated: true,
+    },
 }), (argv) => {
+    const DEPRECATED_OPTIONS = ['lint', 'preprocess', 'decorate'];
+    const DECORATORS_DOCUMENTATION_LINK = 'https://redocly.com/docs/cli/decorators/#decorators';
+    const JOIN_COMMAND_DOCUMENTATION_LINK = 'https://redocly.com/docs/cli/commands/join/#join';
+    DEPRECATED_OPTIONS.forEach((option) => {
+        if (argv[option]) {
+            process.stdout.write(`${colors.red(`Option --${option} is no longer supported. Please review join command documentation ${JOIN_COMMAND_DOCUMENTATION_LINK}.`)}`);
+            process.stdout.write('\n\n');
+            if (['preprocess', 'decorate'].includes(option)) {
+                process.stdout.write(`${colors.red(`If you are looking for decorators, please review the decorators documentation ${DECORATORS_DOCUMENTATION_LINK}.`)}`);
+                process.stdout.write('\n\n');
+            }
+            yargs.showHelp();
+            process.exit(1);
+        }
+    });
     process.env.REDOCLY_CLI_COMMAND = 'join';
     (0, wrapper_1.commandWrapper)(join_1.handleJoin)(argv);
 })
@@ -328,6 +353,7 @@ yargs
             'checkstyle',
             'codeclimate',
             'summary',
+            'github-actions',
         ],
         default: 'codeframe',
     },
@@ -377,28 +403,11 @@ yargs
         description: 'Output file.',
         alias: 'o',
     },
-    format: {
-        description: 'Use a specific output format.',
-        choices: ['stylish', 'codeframe', 'json', 'checkstyle'],
-        hidden: true,
-    },
-    'max-problems': {
-        requiresArg: true,
-        description: 'Reduce output to a maximum of N problems.',
-        type: 'number',
-        hidden: true,
-    },
     ext: {
         description: 'Bundle file extension.',
         requiresArg: true,
         choices: types_1.outputExtensions,
     },
-    'skip-rule': {
-        description: 'Ignore certain rules.',
-        array: true,
-        type: 'string',
-        hidden: true,
-    },
     'skip-preprocessor': {
         description: 'Ignore certain preprocessors.',
         array: true,
@@ -423,12 +432,6 @@ yargs
         description: 'Path to the config file.',
         type: 'string',
     },
-    lint: {
-        description: 'Lint API descriptions',
-        type: 'boolean',
-        default: false,
-        hidden: true,
-    },
     metafile: {
         description: 'Produce metadata about the bundle',
         type: 'string',
@@ -455,7 +458,35 @@ yargs
         choices: ['warn', 'error', 'off'],
         default: 'warn',
     },
+    format: {
+        hidden: true,
+        deprecated: true,
+    },
+    lint: {
+        hidden: true,
+        deprecated: true,
+    },
+    'skip-rule': {
+        hidden: true,
+        deprecated: true,
+        array: true,
+        type: 'string',
+    },
+    'max-problems': {
+        hidden: true,
+        deprecated: true,
+    },
 }), (argv) => {
+    const DEPRECATED_OPTIONS = ['lint', 'format', 'skip-rule', 'max-problems'];
+    const LINT_AND_BUNDLE_DOCUMENTATION_LINK = 'https://redocly.com/docs/cli/guides/lint-and-bundle/#lint-and-bundle-api-descriptions-with-redocly-cli';
+    DEPRECATED_OPTIONS.forEach((option) => {
+        if (argv[option]) {
+            process.stdout.write(`${colors.red(`Option --${option} is no longer supported. Please use separate commands, as described in the ${LINT_AND_BUNDLE_DOCUMENTATION_LINK}.`)}`);
+            process.stdout.write('\n\n');
+            yargs.showHelp();
+            process.exit(1);
+        }
+    });
     process.env.REDOCLY_CLI_COMMAND = 'bundle';
     (0, wrapper_1.commandWrapper)(bundle_1.handleBundle)(argv);
 })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@redocly/cli",
-  "version": "1.10.6",
+  "version": "1.11.0",
   "description": "",
   "license": "MIT",
   "bin": {
@@ -36,7 +36,7 @@
     "Roman Hotsiy <roman@redoc.ly> (https://redoc.ly/)"
   ],
   "dependencies": {
-    "@redocly/openapi-core": "1.10.6",
+    "@redocly/openapi-core": "1.11.0",
     "abort-controller": "^3.0.0",
     "chokidar": "^3.5.1",
     "colorette": "^1.2.0",

package/src/__tests__/commands/bundle.test.ts

@@ -1,4 +1,4 @@
-import { lint, bundle, getTotals, getMergedConfig } from '@redocly/openapi-core';
+import { bundle, getTotals, getMergedConfig } from '@redocly/openapi-core';
 
 import { BundleOptions, handleBundle } from '../../commands/bundle';
 import { handleError } from '../../utils/miscellaneous';
@@ -25,21 +25,18 @@ describe('bundle', () => {
   });
 
   afterEach(() => {
-    (lint as jest.Mock).mockClear();
     (bundle as jest.Mock).mockClear();
     (getTotals as jest.Mock).mockReset();
   });
 
-  it('bundles definitions w/o linting', async () => {
+  it('bundles definitions', async () => {
     const apis = ['foo.yaml', 'bar.yaml'];
 
     await commandWrapper(handleBundle)({
       apis,
       ext: 'yaml',
-      format: 'codeframe',
     } as Arguments<BundleOptions>);
 
-    expect(lint).toBeCalledTimes(0);
     expect(bundle).toBeCalledTimes(apis.length);
   });
 
@@ -49,48 +46,25 @@ describe('bundle', () => {
     await commandWrapper(handleBundle)({
       apis,
       ext: 'yaml',
-      format: 'codeframe',
     } as Arguments<BundleOptions>);
 
     await exitCb?.();
     expect(processExitMock).toHaveBeenCalledWith(0);
   });
 
-  it('bundles definitions w/ linting', async () => {
-    const apis = ['foo.yaml', 'bar.yaml', 'foobar.yaml'];
-
-    (getTotals as jest.Mock).mockReturnValue({
-      errors: 0,
-      warnings: 0,
-      ignored: 0,
-    });
-
-    await commandWrapper(handleBundle)({
-      apis,
-      ext: 'yaml',
-      format: 'codeframe',
-      lint: true,
-    } as Arguments<BundleOptions>);
-
-    expect(lint).toBeCalledTimes(apis.length);
-    expect(bundle).toBeCalledTimes(apis.length);
-  });
-
-  it('exits with code 0 when bundles definitions w/linting w/o errors', async () => {
+  it('exits with code 0 when bundles definitions w/o errors', async () => {
     const apis = ['foo.yaml', 'bar.yaml', 'foobar.yaml'];
 
     await commandWrapper(handleBundle)({
       apis,
       ext: 'yaml',
-      format: 'codeframe',
-      lint: true,
     } as Arguments<BundleOptions>);
 
     await exitCb?.();
     expect(processExitMock).toHaveBeenCalledWith(0);
   });
 
-  it('exits with code 1 when bundles definitions w/linting w/errors', async () => {
+  it('exits with code 1 when bundles definitions w/errors', async () => {
     const apis = ['foo.yaml'];
 
     (getTotals as jest.Mock).mockReturnValue({
@@ -102,11 +76,8 @@ describe('bundle', () => {
     await commandWrapper(handleBundle)({
       apis,
       ext: 'yaml',
-      format: 'codeframe',
-      lint: true,
     } as Arguments<BundleOptions>);
 
-    expect(lint).toBeCalledTimes(apis.length);
     await exitCb?.();
     expect(processExitMock).toHaveBeenCalledWith(1);
   });
@@ -121,8 +92,6 @@ describe('bundle', () => {
     await commandWrapper(handleBundle)({
       apis,
       ext: 'json',
-      format: 'codeframe',
-      lint: false,
     } as Arguments<BundleOptions>);
 
     expect(handleError).toHaveBeenCalledTimes(1);
@@ -141,8 +110,6 @@ describe('bundle', () => {
     await commandWrapper(handleBundle)({
       apis,
       ext: 'yaml',
-      format: 'codeframe',
-      lint: false,
     } as Arguments<BundleOptions>);
 
     expect(handleError).toHaveBeenCalledTimes(0);

package/src/__tests__/commands/join.test.ts

@@ -165,23 +165,6 @@ describe('handleJoin', () => {
     expect(config.styleguide.skipPreprocessors).toHaveBeenCalled();
   });
 
-  it('should not call skipDecorators and skipPreprocessors', async () => {
-    (detectSpec as jest.Mock).mockReturnValue('oas3_0');
-    await handleJoin(
-      {
-        apis: ['first.yaml', 'second.yaml'],
-        decorate: true,
-        preprocess: true,
-      },
-      ConfigFixture as any,
-      'cli-version'
-    );
-
-    const config = loadConfig();
-    expect(config.styleguide.skipDecorators).not.toHaveBeenCalled();
-    expect(config.styleguide.skipPreprocessors).not.toHaveBeenCalled();
-  });
-
   it('should handle join with prefix-components-with-info-prop and null values', async () => {
     (detectSpec as jest.Mock).mockReturnValue('oas3_0');