@redocly/cli 1.2.1 → 1.4.0

package/lib/utils.js

@@ -20,7 +20,7 @@ var __rest = (this && this.__rest) || function (s, e) {
     return t;
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.cleanRawInput = exports.cleanArgs = exports.sendTelemetry = exports.cleanColors = exports.checkIfRulesetExist = exports.sortTopLevelKeysForOas = exports.loadConfigAndHandleErrors = exports.isSubdir = exports.exitWithError = exports.printUnusedWarnings = exports.getOutputFileName = exports.printConfigLintTotals = exports.printLintTotals = exports.HandledError = exports.handleError = exports.pluralize = exports.writeYaml = exports.readYaml = exports.promptUser = exports.saveBundle = exports.dumpBundle = exports.CircularJSONNotSupportedError = exports.langToExt = exports.escapeLanguageName = exports.pathToFilename = exports.printExecutionTime = exports.getExecutionTime = exports.getFallbackApisOrExit = void 0;
+exports.cleanRawInput = exports.cleanArgs = exports.sendTelemetry = exports.cleanColors = exports.checkIfRulesetExist = exports.sortTopLevelKeysForOas = exports.loadConfigAndHandleErrors = exports.isSubdir = exports.exitWithError = exports.printUnusedWarnings = exports.getOutputFileName = exports.printConfigLintTotals = exports.printLintTotals = exports.HandledError = exports.handleError = exports.pluralize = exports.getAndValidateFileExtension = exports.writeJson = exports.writeYaml = exports.writeToFileByExtension = exports.readYaml = exports.promptUser = exports.saveBundle = exports.dumpBundle = exports.CircularJSONNotSupportedError = exports.langToExt = exports.escapeLanguageName = exports.pathToFilename = exports.printExecutionTime = exports.getExecutionTime = exports.getFallbackApisOrExit = void 0;
 const fetch_with_timeout_1 = require("./fetch-with-timeout");
 const path_1 = require("path");
 const colorette_1 = require("colorette");
@@ -31,6 +31,7 @@ const readline = require("readline");
 const stream_1 = require("stream");
 const child_process_1 = require("child_process");
 const openapi_core_1 = require("@redocly/openapi-core");
+const config_1 = require("@redocly/openapi-core/lib/config");
 const types_1 = require("./types");
 const utils_1 = require("@redocly/openapi-core/lib/utils");
 const update_version_notifier_1 = require("./update-version-notifier");
@@ -45,7 +46,7 @@ function getFallbackApisOrExit(argsApis, config) {
         const filteredInvalidEntrypoints = res.filter(({ path }) => !isApiPathValid(path));
         if (isNotEmptyArray(filteredInvalidEntrypoints)) {
             for (const { path } of filteredInvalidEntrypoints) {
-                process.stderr.write(colorette_1.yellow(`\n${path_1.relative(process.cwd(), path)} ${colorette_1.red(`does not exist or is invalid.\n\n`)}`));
+                process.stderr.write((0, colorette_1.yellow)(`\n${(0, path_1.relative)(process.cwd(), path)} ${(0, colorette_1.red)(`does not exist or is invalid.\n\n`)}`));
             }
             exitWithError('Please provide a valid path.');
         }
@@ -54,7 +55,7 @@ function getFallbackApisOrExit(argsApis, config) {
 }
 exports.getFallbackApisOrExit = getFallbackApisOrExit;
 function getConfigDirectory(config) {
-    return config.configFile ? path_1.dirname(config.configFile) : process.cwd();
+    return config.configFile ? (0, path_1.dirname)(config.configFile) : process.cwd();
 }
 function isNotEmptyArray(args) {
     return Array.isArray(args) && !!args.length;
@@ -64,11 +65,11 @@ function isApiPathValid(apiPath) {
         exitWithError('Path cannot be empty.');
         return;
     }
-    return fs.existsSync(apiPath) || openapi_core_1.isAbsoluteUrl(apiPath) ? apiPath : undefined;
+    return fs.existsSync(apiPath) || (0, openapi_core_1.isAbsoluteUrl)(apiPath) ? apiPath : undefined;
 }
 function fallbackToAllDefinitions(apis, config) {
     return Object.entries(apis).map(([alias, { root }]) => ({
-        path: openapi_core_1.isAbsoluteUrl(root) ? root : path_1.resolve(getConfigDirectory(config), root),
+        path: (0, openapi_core_1.isAbsoluteUrl)(root) ? root : (0, path_1.resolve)(getConfigDirectory(config), root),
         alias,
     }));
 }
@@ -80,14 +81,14 @@ function getAliasOrPath(config, aliasOrPath) {
             path: aliasOrPath,
             // find alias by path, take the first match
             alias: (_c = (_b = Object.entries(config.apis).find(([_alias, api]) => {
-                return path_1.resolve(api.root) === path_1.resolve(aliasOrPath);
+                return (0, path_1.resolve)(api.root) === (0, path_1.resolve)(aliasOrPath);
             })) === null || _b === void 0 ? void 0 : _b[0]) !== null && _c !== void 0 ? _c : undefined,
         };
 }
 function expandGlobsInEntrypoints(args, config) {
     return __awaiter(this, void 0, void 0, function* () {
         return (yield Promise.all(args.map((aliasOrPath) => __awaiter(this, void 0, void 0, function* () {
-            return glob.hasMagic(aliasOrPath) && !openapi_core_1.isAbsoluteUrl(aliasOrPath)
+            return glob.hasMagic(aliasOrPath) && !(0, openapi_core_1.isAbsoluteUrl)(aliasOrPath)
                 ? (yield glob.__promisify__(aliasOrPath)).map((g) => getAliasOrPath(config, g))
                 : getAliasOrPath(config, aliasOrPath);
         })))).flat();
@@ -101,7 +102,7 @@ function getExecutionTime(startedAt) {
 exports.getExecutionTime = getExecutionTime;
 function printExecutionTime(commandName, startedAt, api) {
     const elapsed = getExecutionTime(startedAt);
-    process.stderr.write(colorette_1.gray(`\n${api}: ${commandName} processed in ${elapsed}\n\n`));
+    process.stderr.write((0, colorette_1.gray)(`\n${api}: ${commandName} processed in ${elapsed}\n\n`));
 }
 exports.printExecutionTime = printExecutionTime;
 function pathToFilename(path, pathSeparator) {
@@ -152,7 +153,7 @@ function dumpBundle(obj, format, dereference) {
         }
     }
     else {
-        return openapi_core_1.stringifyYaml(obj, {
+        return (0, openapi_core_1.stringifyYaml)(obj, {
             noRefs: !dereference,
             lineWidth: -1,
         });
@@ -160,7 +161,7 @@ function dumpBundle(obj, format, dereference) {
 }
 exports.dumpBundle = dumpBundle;
 function saveBundle(filename, output) {
-    fs.mkdirSync(path_1.dirname(filename), { recursive: true });
+    fs.mkdirSync((0, path_1.dirname)(filename), { recursive: true });
     fs.writeFileSync(filename, output);
 }
 exports.saveBundle = saveBundle;
@@ -195,19 +196,47 @@ function promptUser(query, hideUserInput = false) {
 }
 exports.promptUser = promptUser;
 function readYaml(filename) {
-    return openapi_core_1.parseYaml(fs.readFileSync(filename, 'utf-8'), { filename });
+    return (0, openapi_core_1.parseYaml)(fs.readFileSync(filename, 'utf-8'), { filename });
 }
 exports.readYaml = readYaml;
+function writeToFileByExtension(data, filePath, noRefs) {
+    const ext = getAndValidateFileExtension(filePath);
+    if (ext === 'json') {
+        writeJson(data, filePath);
+        return;
+    }
+    writeYaml(data, filePath, noRefs);
+}
+exports.writeToFileByExtension = writeToFileByExtension;
 function writeYaml(data, filename, noRefs = false) {
-    const content = openapi_core_1.stringifyYaml(data, { noRefs });
+    const content = (0, openapi_core_1.stringifyYaml)(data, { noRefs });
     if (process.env.NODE_ENV === 'test') {
         process.stderr.write(content);
         return;
     }
-    fs.mkdirSync(path_1.dirname(filename), { recursive: true });
+    fs.mkdirSync((0, path_1.dirname)(filename), { recursive: true });
     fs.writeFileSync(filename, content);
 }
 exports.writeYaml = writeYaml;
+function writeJson(data, filename) {
+    const content = JSON.stringify(data, null, 2);
+    if (process.env.NODE_ENV === 'test') {
+        process.stderr.write(content);
+        return;
+    }
+    fs.mkdirSync((0, path_1.dirname)(filename), { recursive: true });
+    fs.writeFileSync(filename, content);
+}
+exports.writeJson = writeJson;
+function getAndValidateFileExtension(fileName) {
+    const ext = fileName.split('.').pop();
+    if (['yaml', 'yml', 'json'].includes(ext)) {
+        return ext;
+    }
+    process.stderr.write((0, colorette_1.yellow)(`Unsupported file extension: ${ext}. Using yaml.\n`));
+    return 'yaml';
+}
+exports.getAndValidateFileExtension = getAndValidateFileExtension;
 function pluralize(label, num) {
     if (label.endsWith('is')) {
         [label] = label.split(' ');
@@ -228,10 +257,12 @@ function handleError(e, ref) {
             return exitWithError(`Failed to parse API description at ${ref}:\n\n  - ${e.message}.`);
         case CircularJSONNotSupportedError: {
             return exitWithError(`Detected circular reference which can't be converted to JSON.\n` +
-                `Try to use ${colorette_1.blue('yaml')} output or remove ${colorette_1.blue('--dereferenced')}.`);
+                `Try to use ${(0, colorette_1.blue)('yaml')} output or remove ${(0, colorette_1.blue)('--dereferenced')}.`);
         }
         case SyntaxError:
             return exitWithError(`Syntax error: ${e.message} ${(_b = (_a = e.stack) === null || _a === void 0 ? void 0 : _a.split('\n\n')) === null || _b === void 0 ? void 0 : _b[0]}`);
+        case config_1.ConfigValidationError:
+            return exitWithError(e.message);
         default: {
             exitWithError(`Something went wrong when processing ${ref}:\n\n  - ${e.message}.`);
         }
@@ -243,34 +274,32 @@ class HandledError extends Error {
 exports.HandledError = HandledError;
 function printLintTotals(totals, definitionsCount) {
     const ignored = totals.ignored
-        ? colorette_1.yellow(`${totals.ignored} ${pluralize('problem is', totals.ignored)} explicitly ignored.\n\n`)
+        ? (0, colorette_1.yellow)(`${totals.ignored} ${pluralize('problem is', totals.ignored)} explicitly ignored.\n\n`)
         : '';
     if (totals.errors > 0) {
-        process.stderr.write(colorette_1.red(`❌ Validation failed with ${totals.errors} ${pluralize('error', totals.errors)}${totals.warnings > 0
+        process.stderr.write((0, colorette_1.red)(`❌ Validation failed with ${totals.errors} ${pluralize('error', totals.errors)}${totals.warnings > 0
             ? ` and ${totals.warnings} ${pluralize('warning', totals.warnings)}`
             : ''}.\n${ignored}`));
     }
     else if (totals.warnings > 0) {
-        process.stderr.write(colorette_1.green(`Woohoo! Your API ${pluralize('description is', definitionsCount)} valid. 🎉\n`));
-        process.stderr.write(colorette_1.yellow(`You have ${totals.warnings} ${pluralize('warning', totals.warnings)}.\n${ignored}`));
+        process.stderr.write((0, colorette_1.green)(`Woohoo! Your API ${pluralize('description is', definitionsCount)} valid. 🎉\n`));
+        process.stderr.write((0, colorette_1.yellow)(`You have ${totals.warnings} ${pluralize('warning', totals.warnings)}.\n${ignored}`));
     }
     else {
-        process.stderr.write(colorette_1.green(`Woohoo! Your API ${pluralize('description is', definitionsCount)} valid. 🎉\n${ignored}`));
+        process.stderr.write((0, colorette_1.green)(`Woohoo! Your API ${pluralize('description is', definitionsCount)} valid. 🎉\n${ignored}`));
     }
     if (totals.errors > 0) {
-        process.stderr.write(colorette_1.gray(`run \`redocly lint --generate-ignore-file\` to add all problems to the ignore file.\n`));
+        process.stderr.write((0, colorette_1.gray)(`run \`redocly lint --generate-ignore-file\` to add all problems to the ignore file.\n`));
     }
     process.stderr.write('\n');
 }
 exports.printLintTotals = printLintTotals;
 function printConfigLintTotals(totals) {
     if (totals.errors > 0) {
-        process.stderr.write(colorette_1.red(`❌ Your config has ${totals.errors} ${pluralize('error', totals.errors)}${totals.warnings > 0
-            ? ` and ${totals.warnings} ${pluralize('warning', totals.warnings)}`
-            : ''}.\n`));
+        process.stderr.write((0, colorette_1.red)(`❌ Your config has ${totals.errors} ${pluralize('error', totals.errors)}.`));
     }
     else if (totals.warnings > 0) {
-        process.stderr.write(colorette_1.yellow(`You have ${totals.warnings} ${pluralize('warning', totals.warnings)}.\n`));
+        process.stderr.write((0, colorette_1.yellow)(`⚠️ Your config has ${totals.warnings} ${pluralize('warning', totals.warnings)}.\n`));
     }
 }
 exports.printConfigLintTotals = printConfigLintTotals;
@@ -280,21 +309,21 @@ function getOutputFileName(entrypoint, entries, output, ext) {
     }
     let outputFile = output;
     if (entries > 1) {
-        ext = ext || path_1.extname(entrypoint).substring(1);
+        ext = ext || (0, path_1.extname)(entrypoint).substring(1);
         if (!types_1.outputExtensions.includes(ext)) {
             throw new Error(`Invalid file extension: ${ext}.`);
         }
-        outputFile = path_1.join(output, path_1.basename(entrypoint, path_1.extname(entrypoint))) + '.' + ext;
+        outputFile = (0, path_1.join)(output, (0, path_1.basename)(entrypoint, (0, path_1.extname)(entrypoint))) + '.' + ext;
     }
     else {
         if (output) {
-            ext = ext || path_1.extname(output).substring(1);
+            ext = ext || (0, path_1.extname)(output).substring(1);
         }
-        ext = ext || path_1.extname(entrypoint).substring(1);
+        ext = ext || (0, path_1.extname)(entrypoint).substring(1);
         if (!types_1.outputExtensions.includes(ext)) {
             throw new Error(`Invalid file extension: ${ext}.`);
         }
-        outputFile = path_1.join(path_1.dirname(outputFile), path_1.basename(outputFile, path_1.extname(outputFile))) + '.' + ext;
+        outputFile = (0, path_1.join)((0, path_1.dirname)(outputFile), (0, path_1.basename)(outputFile, (0, path_1.extname)(outputFile))) + '.' + ext;
     }
     return { outputFile, ext };
 }
@@ -302,13 +331,13 @@ exports.getOutputFileName = getOutputFileName;
 function printUnusedWarnings(config) {
     const { preprocessors, rules, decorators } = config.getUnusedRules();
     if (rules.length) {
-        process.stderr.write(colorette_1.yellow(`[WARNING] Unused rules found in ${colorette_1.blue(config.configFile || '')}: ${rules.join(', ')}.\n`));
+        process.stderr.write((0, colorette_1.yellow)(`[WARNING] Unused rules found in ${(0, colorette_1.blue)(config.configFile || '')}: ${rules.join(', ')}.\n`));
     }
     if (preprocessors.length) {
-        process.stderr.write(colorette_1.yellow(`[WARNING] Unused preprocessors found in ${colorette_1.blue(config.configFile || '')}: ${preprocessors.join(', ')}.\n`));
+        process.stderr.write((0, colorette_1.yellow)(`[WARNING] Unused preprocessors found in ${(0, colorette_1.blue)(config.configFile || '')}: ${preprocessors.join(', ')}.\n`));
     }
     if (decorators.length) {
-        process.stderr.write(colorette_1.yellow(`[WARNING] Unused decorators found in ${colorette_1.blue(config.configFile || '')}: ${decorators.join(', ')}.\n`));
+        process.stderr.write((0, colorette_1.yellow)(`[WARNING] Unused decorators found in ${(0, colorette_1.blue)(config.configFile || '')}: ${decorators.join(', ')}.\n`));
     }
     if (rules.length || preprocessors.length) {
         process.stderr.write(`Check the spelling and verify the added plugin prefix.\n`);
@@ -316,7 +345,7 @@ function printUnusedWarnings(config) {
 }
 exports.printUnusedWarnings = printUnusedWarnings;
 function exitWithError(message) {
-    process.stderr.write(colorette_1.red(message) + '\n\n');
+    process.stderr.write((0, colorette_1.red)(message) + '\n\n');
     throw new HandledError(message);
 }
 exports.exitWithError = exitWithError;
@@ -324,14 +353,14 @@ exports.exitWithError = exitWithError;
  * Checks if dir is subdir of parent
  */
 function isSubdir(parent, dir) {
-    const relativePath = path_1.relative(parent, dir);
-    return !!relativePath && !/^..($|\/)/.test(relativePath) && !path_1.isAbsolute(relativePath);
+    const relativePath = (0, path_1.relative)(parent, dir);
+    return !!relativePath && !/^..($|\/)/.test(relativePath) && !(0, path_1.isAbsolute)(relativePath);
 }
 exports.isSubdir = isSubdir;
 function loadConfigAndHandleErrors(options = {}) {
     return __awaiter(this, void 0, void 0, function* () {
         try {
-            return yield openapi_core_1.loadConfig(options);
+            return yield (0, openapi_core_1.loadConfig)(options);
         }
         catch (e) {
             handleError(e, '');
@@ -398,7 +427,7 @@ function sortOas3Keys(document) {
 }
 function checkIfRulesetExist(rules) {
     const ruleset = Object.assign(Object.assign(Object.assign({}, rules.oas2), rules.oas3_0), rules.oas3_0);
-    if (utils_1.isEmptyObject(ruleset)) {
+    if ((0, utils_1.isEmptyObject)(ruleset)) {
         exitWithError('⚠️ No rules were configured. Learn how to configure rules: https://redocly.com/docs/cli/rules/');
     }
 }
@@ -425,7 +454,7 @@ function sendTelemetry(argv, exit_code, has_config) {
                 command,
                 arguments: cleanArgs(args),
                 node_version: process.version,
-                npm_version: child_process_1.execSync('npm -v').toString().replace('\n', ''),
+                npm_version: (0, child_process_1.execSync)('npm -v').toString().replace('\n', ''),
                 version: update_version_notifier_1.version,
                 exit_code,
                 environment: process.env.REDOCLY_ENVIRONMENT,
@@ -433,7 +462,7 @@ function sendTelemetry(argv, exit_code, has_config) {
                 raw_input: cleanRawInput(process.argv.slice(2)),
                 has_config,
             };
-            yield fetch_with_timeout_1.default(`https://api.redocly.com/registry/telemetry/cli`, {
+            yield (0, fetch_with_timeout_1.default)(`https://api.redocly.com/registry/telemetry/cli`, {
                 method: 'POST',
                 headers: {
                     'content-type': 'application/json',
@@ -457,7 +486,7 @@ function cleanString(value) {
     if (!value) {
         return value;
     }
-    if (openapi_core_1.isAbsoluteUrl(value)) {
+    if ((0, openapi_core_1.isAbsoluteUrl)(value)) {
         return value.split('://')[0] + '://url';
     }
     if (isFile(value)) {

package/lib/wrapper.js

@@ -20,15 +20,15 @@ function commandWrapper(commandHandler) {
         let hasConfig;
         let telemetry;
         try {
-            if (argv.config && !openapi_core_1.doesYamlFileExist(argv.config)) {
-                utils_1.exitWithError('Please, provide valid path to the configuration file');
+            if (argv.config && !(0, openapi_core_1.doesYamlFileExist)(argv.config)) {
+                (0, utils_1.exitWithError)('Please, provide valid path to the configuration file');
             }
-            const config = (yield utils_1.loadConfigAndHandleErrors({
+            const config = (yield (0, utils_1.loadConfigAndHandleErrors)({
                 configPath: argv.config,
                 customExtends: argv.extends,
                 region: argv.region,
                 files: argv.files,
-                processRawConfig: lint_1.lintConfigCallback(argv, update_version_notifier_1.version),
+                processRawConfig: (0, lint_1.lintConfigCallback)(argv, update_version_notifier_1.version),
             }));
             telemetry = config.telemetry;
             hasConfig = !config.styleguide.recommendedFallback;
@@ -41,7 +41,7 @@ function commandWrapper(commandHandler) {
         }
         finally {
             if (process.env.REDOCLY_TELEMETRY !== 'off' && telemetry !== 'off') {
-                yield utils_1.sendTelemetry(argv, code, hasConfig);
+                yield (0, utils_1.sendTelemetry)(argv, code, hasConfig);
             }
             process.once('beforeExit', () => {
                 process.exit(code);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@redocly/cli",
-  "version": "1.2.1",
+  "version": "1.4.0",
   "description": "",
   "license": "MIT",
   "bin": {
@@ -36,7 +36,7 @@
     "Roman Hotsiy <roman@redoc.ly> (https://redoc.ly/)"
   ],
   "dependencies": {
-    "@redocly/openapi-core": "1.2.1",
+    "@redocly/openapi-core": "1.4.0",
     "chokidar": "^3.5.1",
     "colorette": "^1.2.0",
     "core-js": "^3.32.1",
@@ -59,6 +59,6 @@
     "@types/react-dom": "^17.0.0 || ^18.2.7",
     "@types/semver": "^7.5.0",
     "@types/yargs": "17.0.5",
-    "typescript": "^4.0.3"
+    "typescript": "^5.2.2"
   }
 }

package/src/__mocks__/@redocly/openapi-core.ts

@@ -31,6 +31,7 @@ export const doesYamlFileExist = jest.fn();
 export const bundleDocument = jest.fn(() => Promise.resolve({ problems: {} }));
 export const detectSpec = jest.fn();
 export const isAbsoluteUrl = jest.fn();
+export const stringifyYaml = jest.fn((data) => data);
 
 export class BaseResolver {
   cache = new Map<string, Promise<Document | ResolveError>>();

package/src/__mocks__/utils.ts

@@ -17,3 +17,5 @@ export const writeYaml = jest.fn();
 export const loadConfigAndHandleErrors = jest.fn(() => ConfigFixture);
 export const checkIfRulesetExist = jest.fn();
 export const sortTopLevelKeysForOas = jest.fn((document) => document);
+export const getAndValidateFileExtension = jest.fn((fileName: string) => fileName.split('.').pop());
+export const writeToFileByExtension = jest.fn();

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

@@ -1,11 +1,12 @@
 import { handleJoin } from '../../commands/join';
-import { exitWithError, writeYaml } from '../../utils';
+import { exitWithError, writeToFileByExtension, writeYaml } from '../../utils';
 import { yellow } from 'colorette';
 import { detectSpec } from '@redocly/openapi-core';
 import { loadConfig } from '../../__mocks__/@redocly/openapi-core';
 import { ConfigFixture } from '../fixtures/config';
 
 jest.mock('../../utils');
+
 jest.mock('colorette');
 
 describe('handleJoin fails', () => {
@@ -80,7 +81,7 @@ describe('handleJoin fails', () => {
     );
   });
 
-  it('should call writeYaml function', async () => {
+  it('should call writeToFileByExtension function', async () => {
     (detectSpec as jest.Mock).mockReturnValue('oas3_0');
     await handleJoin(
       {
@@ -90,10 +91,14 @@ describe('handleJoin fails', () => {
       'cli-version'
     );
 
-    expect(writeYaml).toHaveBeenCalledWith(expect.any(Object), 'openapi.yaml', expect.any(Boolean));
+    expect(writeToFileByExtension).toHaveBeenCalledWith(
+      expect.any(Object),
+      'openapi.yaml',
+      expect.any(Boolean)
+    );
   });
 
-  it('should call writeYaml function for OpenAPI 3.1', async () => {
+  it('should call writeToFileByExtension function for OpenAPI 3.1', async () => {
     (detectSpec as jest.Mock).mockReturnValue('oas3_1');
     await handleJoin(
       {
@@ -103,10 +108,14 @@ describe('handleJoin fails', () => {
       'cli-version'
     );
 
-    expect(writeYaml).toHaveBeenCalledWith(expect.any(Object), 'openapi.yaml', expect.any(Boolean));
+    expect(writeToFileByExtension).toHaveBeenCalledWith(
+      expect.any(Object),
+      'openapi.yaml',
+      expect.any(Boolean)
+    );
   });
 
-  it('should call writeYaml function with custom output file', async () => {
+  it('should call writeToFileByExtension function with custom output file', async () => {
     (detectSpec as jest.Mock).mockReturnValue('oas3_0');
     await handleJoin(
       {
@@ -117,7 +126,28 @@ describe('handleJoin fails', () => {
       'cli-version'
     );
 
-    expect(writeYaml).toHaveBeenCalledWith(expect.any(Object), 'output.yml', expect.any(Boolean));
+    expect(writeToFileByExtension).toHaveBeenCalledWith(
+      expect.any(Object),
+      'output.yml',
+      expect.any(Boolean)
+    );
+  });
+
+  it('should call writeToFileByExtension function with json file extension', async () => {
+    (detectSpec as jest.Mock).mockReturnValue('oas3_0');
+    await handleJoin(
+      {
+        apis: ['first.json', 'second.yaml'],
+      },
+      ConfigFixture as any,
+      'cli-version'
+    );
+
+    expect(writeToFileByExtension).toHaveBeenCalledWith(
+      expect.any(Object),
+      'openapi.json',
+      expect.any(Boolean)
+    );
   });
 
   it('should call skipDecorators and skipPreprocessors', async () => {

package/src/__tests__/utils.test.ts

@@ -12,6 +12,10 @@ import {
   HandledError,
   cleanArgs,
   cleanRawInput,
+  getAndValidateFileExtension,
+  writeYaml,
+  writeJson,
+  writeToFileByExtension,
 } from '../utils';
 import {
   ResolvedApi,
@@ -19,11 +23,13 @@ import {
   isAbsoluteUrl,
   ResolveError,
   YamlParseError,
+  stringifyYaml,
 } from '@redocly/openapi-core';
 import { blue, red, yellow } from 'colorette';
-import { existsSync, statSync } from 'fs';
+import { existsSync, statSync, writeFileSync } from 'fs';
 import * as path from 'path';
 import * as process from 'process';
+import * as utils from '../utils';
 
 jest.mock('os');
 jest.mock('colorette');
@@ -108,22 +114,14 @@ describe('printConfigLintTotals', () => {
 
   it('should print errors if such exist', () => {
     printConfigLintTotals(totalProblemsMock);
-    expect(process.stderr.write).toHaveBeenCalledWith('❌ Your config has 1 error.\n');
-    expect(redColoretteMocks).toHaveBeenCalledWith('❌ Your config has 1 error.\n');
-  });
-
-  it('should print warnign and error', () => {
-    printConfigLintTotals({ ...totalProblemsMock, warnings: 2 });
-    expect(process.stderr.write).toHaveBeenCalledWith(
-      '❌ Your config has 1 error and 2 warnings.\n'
-    );
-    expect(redColoretteMocks).toHaveBeenCalledWith('❌ Your config has 1 error and 2 warnings.\n');
+    expect(process.stderr.write).toHaveBeenCalledWith('❌ Your config has 1 error.');
+    expect(redColoretteMocks).toHaveBeenCalledWith('❌ Your config has 1 error.');
   });
 
   it('should print warnign if no error', () => {
     printConfigLintTotals({ ...totalProblemsMock, errors: 0, warnings: 2 });
-    expect(process.stderr.write).toHaveBeenCalledWith('You have 2 warnings.\n');
-    expect(yellowColoretteMocks).toHaveBeenCalledWith('You have 2 warnings.\n');
+    expect(process.stderr.write).toHaveBeenCalledWith('⚠️ Your config has 2 warnings.\n');
+    expect(yellowColoretteMocks).toHaveBeenCalledWith('⚠️ Your config has 2 warnings.\n');
   });
 
   it('should print nothing if no error and no warnings', () => {
@@ -562,4 +560,42 @@ describe('cleanRawInput', () => {
       'redocly lint file-json --format stylish --extends=minimal --skip-rule operation-4xx-response'
     );
   });
+
+  describe('validateFileExtension', () => {
+    it('should return current file extension', () => {
+      expect(getAndValidateFileExtension('test.json')).toEqual('json');
+    });
+
+    it('should return yaml and print warning if file extension does not supported', () => {
+      const stderrMock = jest.spyOn(process.stderr, 'write').mockImplementation(() => true);
+      (yellow as jest.Mock<any, any>).mockImplementation((text: string) => text);
+
+      expect(getAndValidateFileExtension('test.xml')).toEqual('yaml');
+      expect(stderrMock).toHaveBeenCalledWith(`Unsupported file extension: xml. Using yaml.\n`);
+    });
+  });
+
+  describe('writeToFileByExtension', () => {
+    beforeEach(() => {
+      jest.spyOn(process.stderr, 'write').mockImplementation(jest.fn());
+      (yellow as jest.Mock<any, any>).mockImplementation((text: string) => text);
+    });
+
+    afterEach(() => {
+      jest.restoreAllMocks();
+    });
+
+    it('should call stringifyYaml function', () => {
+      writeToFileByExtension('test data', 'test.yaml');
+      expect(stringifyYaml).toHaveBeenCalledWith('test data', { noRefs: false });
+      expect(process.stderr.write).toHaveBeenCalledWith(`test data`);
+    });
+
+    it('should call JSON.stringify function', () => {
+      const stringifySpy = jest.spyOn(JSON, 'stringify').mockImplementation((data) => data);
+      writeToFileByExtension('test data', 'test.json');
+      expect(stringifySpy).toHaveBeenCalledWith('test data', null, 2);
+      expect(process.stderr.write).toHaveBeenCalledWith(`test data`);
+    });
+  });
 });

package/src/commands/join.ts

@@ -17,6 +17,7 @@ import {
   bundleDocument,
   Referenced,
   isRef,
+  RuleSeverity,
 } from '@redocly/openapi-core';
 
 import {
@@ -24,9 +25,10 @@ import {
   printExecutionTime,
   handleError,
   printLintTotals,
-  writeYaml,
   exitWithError,
   sortTopLevelKeysForOas,
+  getAndValidateFileExtension,
+  writeToFileByExtension,
 } from '../utils';
 import { isObject, isString, keysOf } from '../js-utils';
 import {
@@ -64,21 +66,24 @@ export type JoinOptions = {
   output?: string;
   config?: string;
   extends?: undefined;
-  'lint-config'?: undefined;
+  'lint-config'?: RuleSeverity;
 };
 
 export async function handleJoin(argv: JoinOptions, config: Config, packageVersion: string) {
   const startedAt = performance.now();
+
   if (argv.apis.length < 2) {
     return exitWithError(`At least 2 apis should be provided. \n\n`);
   }
 
+  const fileExtension = 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.yaml',
+    output: specFilename = `openapi.${fileExtension}`,
   } = argv;
 
   const usedTagsOptions = [
@@ -228,7 +233,8 @@ export async function handleJoin(argv: JoinOptions, config: Config, packageVersi
     return exitWithError(`Please fix conflicts before running ${yellow('join')}.`);
   }
 
-  writeYaml(sortTopLevelKeysForOas(joinedDef), specFilename, noRefs);
+  writeToFileByExtension(sortTopLevelKeysForOas(joinedDef), specFilename, noRefs);
+
   printExecutionTime('join', startedAt, specFilename);
 
   function populateTags({

package/src/commands/lint.ts

@@ -9,6 +9,7 @@ import {
   makeDocumentFromString,
   stringifyYaml,
 } from '@redocly/openapi-core';
+import { ConfigValidationError } from '@redocly/openapi-core/lib/config';
 import {
   checkIfRulesetExist,
   exitWithError,
@@ -134,7 +135,7 @@ export function lintConfigCallback(
     const configContent = makeDocumentFromString(stringYaml, configPath);
     const problems = await lintConfig({
       document: configContent,
-      severity: argv['lint-config'] as ProblemSeverity,
+      severity: (argv['lint-config'] || 'warn') as ProblemSeverity,
     });
 
     const fileTotals = getTotals(problems);
@@ -147,5 +148,9 @@ export function lintConfigCallback(
     });
 
     printConfigLintTotals(fileTotals);
+
+    if (fileTotals.errors > 0) {
+      throw new ConfigValidationError();
+    }
   };
 }