@microsoft/m365-spec-parser 0.2.4-rc.0 → 0.2.4

package/dist/index.node.cjs.js

@@ -9,7 +9,6 @@ var fs = require('fs-extra');
 var path = require('path');
 var teamsManifest = require('@microsoft/teams-manifest');
 var crypto = require('crypto');
-var jsonSchemaRefParser = require('@apidevtools/json-schema-ref-parser');
 
 function _interopDefaultLegacy (e) { return e && typeof e === 'object' && 'default' in e ? e : { 'default': e }; }
 
@@ -81,8 +80,11 @@ exports.ErrorType = void 0;
     ErrorType["PostBodyContainMultipleMediaTypes"] = "post-body-contain-multiple-media-types";
     ErrorType["ResponseContainMultipleMediaTypes"] = "response-contain-multiple-media-types";
     ErrorType["ResponseJsonIsEmpty"] = "response-json-is-empty";
+    ErrorType["PostBodySchemaIsNotJson"] = "post-body-schema-is-not-json";
     ErrorType["PostBodyContainsRequiredUnsupportedSchema"] = "post-body-contains-required-unsupported-schema";
     ErrorType["ParamsContainRequiredUnsupportedSchema"] = "params-contain-required-unsupported-schema";
+    ErrorType["ParamsContainsNestedObject"] = "params-contains-nested-object";
+    ErrorType["RequestBodyContainsNestedObject"] = "request-body-contains-nested-object";
     ErrorType["ExceededRequiredParamsLimit"] = "exceeded-required-params-limit";
     ErrorType["NoParameter"] = "no-parameter";
     ErrorType["NoAPIInfo"] = "no-api-info";
@@ -102,7 +104,6 @@ exports.WarningType = void 0;
     WarningType["ConvertSwaggerToOpenAPI"] = "convert-swagger-to-openapi";
     WarningType["FuncDescriptionTooLong"] = "function-description-too-long";
     WarningType["OperationIdContainsSpecialCharacters"] = "operationid-contains-special-characters";
-    WarningType["UnsupportedAuthType"] = "unsupported-auth-type";
     WarningType["GenerateJsonDataFailed"] = "generate-json-data-failed";
     WarningType["Unknown"] = "unknown";
 })(exports.WarningType || (exports.WarningType = {}));
@@ -142,7 +143,6 @@ ConstantString.SwaggerNotSupported = "Swagger 2.0 is not supported. Please conve
 ConstantString.SpecVersionNotSupported = "Unsupported OpenAPI version %s. Please use version 3.0.x.";
 ConstantString.MultipleAuthNotSupported = "Multiple authentication methods are unsupported. Ensure all selected APIs use identical authentication.";
 ConstantString.OperationIdContainsSpecialCharacters = "Operation id '%s' in OpenAPI description document contained special characters and was renamed to '%s'.";
-ConstantString.AuthTypeIsNotSupported = "Unsupported authorization type in API '%s'. No authorization will be used.";
 ConstantString.UnsupportedSchema = "Unsupported schema in %s %s: %s";
 ConstantString.FuncDescriptionTooLong = "The description of the function '%s' is too long. The current length is %s characters, while the maximum allowed length is %s characters.";
 ConstantString.GenerateJsonDataFailed = "Failed to generate JSON data for api: %s due to %s.";
@@ -152,12 +152,17 @@ ConstantString.WrappedCardResponseLayout = "list";
 ConstantString.GetMethod = "get";
 ConstantString.PostMethod = "post";
 ConstantString.AdaptiveCardVersion = "1.5";
-ConstantString.AdaptiveCardSchema = "https://adaptivecards.io/schemas/adaptive-card.json";
+ConstantString.AdaptiveCardSchema = "http://adaptivecards.io/schemas/adaptive-card.json";
 ConstantString.AdaptiveCardType = "AdaptiveCard";
 ConstantString.TextBlockType = "TextBlock";
 ConstantString.ImageType = "Image";
 ConstantString.ContainerType = "Container";
-ConstantString.RegistrationIdPostfix = "REGISTRATION_ID";
+ConstantString.RegistrationIdPostfix = {
+    apiKey: "REGISTRATION_ID",
+    oauth2: "CONFIGURATION_ID",
+    http: "REGISTRATION_ID",
+    openIdConnect: "REGISTRATION_ID",
+};
 ConstantString.ResponseCodeFor20X = [
     "200",
     "201",
@@ -169,7 +174,6 @@ ConstantString.ResponseCodeFor20X = [
     "207",
     "208",
     "226",
-    "2XX",
     "default",
 ];
 ConstantString.AllOperationMethods = [
@@ -235,6 +239,17 @@ class SpecParserError extends Error {
 
 // Copyright (c) Microsoft Corporation.
 class Utils {
+    static hasNestedObjectInSchema(schema) {
+        if (this.isObjectSchema(schema)) {
+            for (const property in schema.properties) {
+                const nestedSchema = schema.properties[property];
+                if (this.isObjectSchema(nestedSchema)) {
+                    return true;
+                }
+            }
+        }
+        return false;
+    }
     static isObjectSchema(schema) {
         return schema.type === "object" || (!schema.type && !!schema.properties);
     }
@@ -247,32 +262,11 @@ class Utils {
     static isAPIKeyAuth(authScheme) {
         return authScheme.type === "apiKey";
     }
-    static isAPIKeyAuthButNotInCookie(authScheme) {
-        return authScheme.type === "apiKey" && authScheme.in !== "cookie";
-    }
     static isOAuthWithAuthCodeFlow(authScheme) {
         return !!(authScheme.type === "oauth2" &&
             authScheme.flows &&
             authScheme.flows.authorizationCode);
     }
-    static isNotSupportedAuth(authSchemeArray) {
-        if (authSchemeArray.length === 0) {
-            return false;
-        }
-        if (authSchemeArray.length > 0 && authSchemeArray.every((auths) => auths.length > 1)) {
-            return true;
-        }
-        for (const auths of authSchemeArray) {
-            if (auths.length === 1) {
-                if (Utils.isOAuthWithAuthCodeFlow(auths[0].authScheme) ||
-                    Utils.isBearerTokenAuth(auths[0].authScheme) ||
-                    Utils.isAPIKeyAuthButNotInCookie(auths[0].authScheme)) {
-                    return false;
-                }
-            }
-        }
-        return true;
-    }
     static getAuthArray(securities, spec) {
         var _a;
         const result = [];
@@ -297,20 +291,6 @@ class Utils {
         result.sort((a, b) => a[0].name.localeCompare(b[0].name));
         return result;
     }
-    static getAuthMap(spec) {
-        const authMap = {};
-        for (const url in spec.paths) {
-            for (const method in spec.paths[url]) {
-                const operation = spec.paths[url][method];
-                const authArray = Utils.getAuthArray(operation.security, spec);
-                if (authArray && authArray.length > 0) {
-                    const currentAuth = authArray[0][0];
-                    authMap[operation.operationId] = currentAuth;
-                }
-            }
-        }
-        return authMap;
-    }
     static getAuthInfo(spec) {
         let authInfo = undefined;
         for (const url in spec.paths) {
@@ -339,32 +319,26 @@ class Utils {
         let multipleMediaType = false;
         for (const code of ConstantString.ResponseCodeFor20X) {
             const responseObject = (_a = operationObject === null || operationObject === void 0 ? void 0 : operationObject.responses) === null || _a === void 0 ? void 0 : _a[code];
-            if (!responseObject) {
-                continue;
-            }
-            multipleMediaType = Utils.containMultipleMediaTypes(responseObject);
-            if (!allowMultipleMediaType && multipleMediaType) {
-                json = {};
-                continue;
-            }
-            const mediaObj = Utils.getJsonContentType(responseObject);
-            if (Object.keys(mediaObj).length > 0) {
-                json = mediaObj;
-                return { json, multipleMediaType };
-            }
-        }
-        return { json, multipleMediaType };
-    }
-    static getJsonContentType(responseObject) {
-        if (responseObject.content) {
-            for (const contentType of Object.keys(responseObject.content)) {
-                // json media type can also be "application/json; charset=utf-8"
-                if (contentType.indexOf("application/json") >= 0) {
-                    return responseObject.content[contentType];
+            if (responseObject === null || responseObject === void 0 ? void 0 : responseObject.content) {
+                for (const contentType of Object.keys(responseObject.content)) {
+                    // json media type can also be "application/json; charset=utf-8"
+                    if (contentType.indexOf("application/json") >= 0) {
+                        multipleMediaType = false;
+                        json = responseObject.content[contentType];
+                        if (Utils.containMultipleMediaTypes(responseObject)) {
+                            multipleMediaType = true;
+                            if (!allowMultipleMediaType) {
+                                json = {};
+                            }
+                        }
+                        else {
+                            return { json, multipleMediaType };
+                        }
+                    }
                 }
             }
         }
-        return {};
+        return { json, multipleMediaType };
     }
     static convertPathToCamelCase(path) {
         const pathSegments = path.split(/[./{]/);
@@ -401,7 +375,7 @@ class Utils {
         }
         return newStr;
     }
-    static checkServerUrl(servers, allowHttp = false) {
+    static checkServerUrl(servers) {
         const errors = [];
         let serverUrl;
         try {
@@ -424,7 +398,8 @@ class Utils {
                 data: servers,
             });
         }
-        else if (protocol !== "https:" && !(protocol === "http:" && allowHttp)) {
+        else if (protocol !== "https:") {
+            // Http server url is not supported
             const protocolString = protocol.slice(0, -1);
             errors.push({
                 type: exports.ErrorType.UrlProtocolNotSupported,
@@ -440,11 +415,10 @@ class Utils {
         let hasTopLevelServers = false;
         let hasPathLevelServers = false;
         let hasOperationLevelServers = false;
-        const allowHttp = options.projectType === exports.ProjectType.Copilot;
         if (spec.servers && spec.servers.length >= 1) {
             hasTopLevelServers = true;
             // for multiple server, we only use the first url
-            const serverErrors = Utils.checkServerUrl(spec.servers, allowHttp);
+            const serverErrors = Utils.checkServerUrl(spec.servers);
             errors.push(...serverErrors);
         }
         const paths = spec.paths;
@@ -452,7 +426,7 @@ class Utils {
             const methods = paths[path];
             if ((methods === null || methods === void 0 ? void 0 : methods.servers) && methods.servers.length >= 1) {
                 hasPathLevelServers = true;
-                const serverErrors = Utils.checkServerUrl(methods.servers, allowHttp);
+                const serverErrors = Utils.checkServerUrl(methods.servers);
                 errors.push(...serverErrors);
             }
             for (const method in methods) {
@@ -460,7 +434,7 @@ class Utils {
                 if (((_a = options.allowMethods) === null || _a === void 0 ? void 0 : _a.includes(method)) && operationObject) {
                     if ((operationObject === null || operationObject === void 0 ? void 0 : operationObject.servers) && operationObject.servers.length >= 1) {
                         hasOperationLevelServers = true;
-                        const serverErrors = Utils.checkServerUrl(operationObject.servers, allowHttp);
+                        const serverErrors = Utils.checkServerUrl(operationObject.servers);
                         errors.push(...serverErrors);
                     }
                 }
@@ -749,6 +723,22 @@ class Validator {
         }
         return result;
     }
+    validateResponse(method, path) {
+        const result = { isValid: true, reason: [] };
+        const operationObject = this.spec.paths[path][method];
+        const { json, multipleMediaType } = Utils.getResponseJson(operationObject);
+        if (this.options.projectType === exports.ProjectType.SME) {
+            // only support response body only contains “application/json” content type
+            if (multipleMediaType) {
+                result.reason.push(exports.ErrorType.ResponseContainMultipleMediaTypes);
+            }
+            else if (Object.keys(json).length === 0) {
+                // response body should not be empty
+                result.reason.push(exports.ErrorType.ResponseJsonIsEmpty);
+            }
+        }
+        return result;
+    }
     validateServer(method, path) {
         const result = { isValid: true, reason: [] };
         const serverObj = Utils.getServerObject(this.spec, method, path);
@@ -757,8 +747,8 @@ class Validator {
             result.reason.push(exports.ErrorType.NoServerInformation);
         }
         else {
-            const allowHttp = this.projectType === exports.ProjectType.Copilot;
-            const serverValidateResult = Utils.checkServerUrl([serverObj], allowHttp);
+            // server url should be absolute url with https protocol
+            const serverValidateResult = Utils.checkServerUrl([serverObj]);
             result.reason.push(...serverValidateResult.map((item) => item.type));
         }
         return result;
@@ -781,9 +771,6 @@ class Validator {
                     reason: [exports.ErrorType.MultipleAuthNotSupported],
                 };
             }
-            if (this.projectType === exports.ProjectType.Copilot) {
-                return { isValid: true, reason: [] };
-            }
             for (const auths of authSchemeArray) {
                 if (auths.length === 1) {
                     if ((this.options.allowAPIKeyAuth && Utils.isAPIKeyAuth(auths[0].authScheme)) ||
@@ -796,6 +783,114 @@ class Validator {
         }
         return { isValid: false, reason: [exports.ErrorType.AuthTypeIsNotSupported] };
     }
+    checkPostBodySchema(schema, isRequired = false) {
+        var _a;
+        const paramResult = {
+            requiredNum: 0,
+            optionalNum: 0,
+            isValid: true,
+            reason: [],
+        };
+        if (Object.keys(schema).length === 0) {
+            return paramResult;
+        }
+        const isRequiredWithoutDefault = isRequired && schema.default === undefined;
+        const isCopilot = this.projectType === exports.ProjectType.Copilot;
+        if (isCopilot && Utils.hasNestedObjectInSchema(schema)) {
+            paramResult.isValid = false;
+            paramResult.reason = [exports.ErrorType.RequestBodyContainsNestedObject];
+            return paramResult;
+        }
+        if (schema.type === "string" ||
+            schema.type === "integer" ||
+            schema.type === "boolean" ||
+            schema.type === "number") {
+            if (isRequiredWithoutDefault) {
+                paramResult.requiredNum = paramResult.requiredNum + 1;
+            }
+            else {
+                paramResult.optionalNum = paramResult.optionalNum + 1;
+            }
+        }
+        else if (Utils.isObjectSchema(schema)) {
+            const { properties } = schema;
+            for (const property in properties) {
+                let isRequired = false;
+                if (schema.required && ((_a = schema.required) === null || _a === void 0 ? void 0 : _a.indexOf(property)) >= 0) {
+                    isRequired = true;
+                }
+                const result = this.checkPostBodySchema(properties[property], isRequired);
+                paramResult.requiredNum += result.requiredNum;
+                paramResult.optionalNum += result.optionalNum;
+                paramResult.isValid = paramResult.isValid && result.isValid;
+                paramResult.reason.push(...result.reason);
+            }
+        }
+        else {
+            if (isRequiredWithoutDefault && !isCopilot) {
+                paramResult.isValid = false;
+                paramResult.reason.push(exports.ErrorType.PostBodyContainsRequiredUnsupportedSchema);
+            }
+        }
+        return paramResult;
+    }
+    checkParamSchema(paramObject) {
+        const paramResult = {
+            requiredNum: 0,
+            optionalNum: 0,
+            isValid: true,
+            reason: [],
+        };
+        if (!paramObject) {
+            return paramResult;
+        }
+        const isCopilot = this.projectType === exports.ProjectType.Copilot;
+        for (let i = 0; i < paramObject.length; i++) {
+            const param = paramObject[i];
+            const schema = param.schema;
+            if (isCopilot && Utils.hasNestedObjectInSchema(schema)) {
+                paramResult.isValid = false;
+                paramResult.reason.push(exports.ErrorType.ParamsContainsNestedObject);
+                continue;
+            }
+            const isRequiredWithoutDefault = param.required && schema.default === undefined;
+            if (isCopilot) {
+                if (isRequiredWithoutDefault) {
+                    paramResult.requiredNum = paramResult.requiredNum + 1;
+                }
+                else {
+                    paramResult.optionalNum = paramResult.optionalNum + 1;
+                }
+                continue;
+            }
+            if (param.in === "header" || param.in === "cookie") {
+                if (isRequiredWithoutDefault) {
+                    paramResult.isValid = false;
+                    paramResult.reason.push(exports.ErrorType.ParamsContainRequiredUnsupportedSchema);
+                }
+                continue;
+            }
+            if (schema.type !== "boolean" &&
+                schema.type !== "string" &&
+                schema.type !== "number" &&
+                schema.type !== "integer") {
+                if (isRequiredWithoutDefault) {
+                    paramResult.isValid = false;
+                    paramResult.reason.push(exports.ErrorType.ParamsContainRequiredUnsupportedSchema);
+                }
+                continue;
+            }
+            if (param.in === "query" || param.in === "path") {
+                if (isRequiredWithoutDefault) {
+                    paramResult.requiredNum = paramResult.requiredNum + 1;
+                }
+                else {
+                    paramResult.optionalNum = paramResult.optionalNum + 1;
+                }
+            }
+        }
+        return paramResult;
+    }
 }
 
 // Copyright (c) Microsoft Corporation.
@@ -805,6 +900,7 @@ class CopilotValidator extends Validator {
         this.projectType = exports.ProjectType.Copilot;
         this.options = options;
         this.spec = spec;
+        this.checkCircularReference();
     }
     validateSpec() {
         const result = { errors: [], warnings: [] };
@@ -830,6 +926,10 @@ class CopilotValidator extends Validator {
         if (!methodAndPathResult.isValid) {
             return methodAndPathResult;
         }
+        const circularReferenceResult = this.validateCircularReference(method, path);
+        if (!circularReferenceResult.isValid) {
+            return circularReferenceResult;
+        }
         const operationObject = this.spec.paths[path][method];
         // validate auth
         const authCheckResult = this.validateAuth(method, path);
@@ -841,6 +941,24 @@ class CopilotValidator extends Validator {
         // validate server
         const validateServerResult = this.validateServer(method, path);
         result.reason.push(...validateServerResult.reason);
+        // validate response
+        const validateResponseResult = this.validateResponse(method, path);
+        result.reason.push(...validateResponseResult.reason);
+        // validate requestBody
+        const requestBody = operationObject.requestBody;
+        const requestJsonBody = requestBody === null || requestBody === void 0 ? void 0 : requestBody.content["application/json"];
+        if (requestJsonBody) {
+            const requestBodySchema = requestJsonBody.schema;
+            if (!Utils.isObjectSchema(requestBodySchema)) {
+                result.reason.push(exports.ErrorType.PostBodySchemaIsNotJson);
+            }
+            const requestBodyParamResult = this.checkPostBodySchema(requestBodySchema, requestBody.required);
+            result.reason.push(...requestBodyParamResult.reason);
+        }
+        // validate parameters
+        const paramObject = operationObject.parameters;
+        const paramResult = this.checkParamSchema(paramObject);
+        result.reason.push(...paramResult.reason);
         if (result.reason.length > 0) {
             result.isValid = false;
         }
@@ -932,108 +1050,6 @@ class SMEValidator extends Validator {
         }
         return result;
     }
-    validateResponse(method, path) {
-        const result = { isValid: true, reason: [] };
-        const operationObject = this.spec.paths[path][method];
-        const { json, multipleMediaType } = Utils.getResponseJson(operationObject);
-        // only support response body only contains “application/json” content type
-        if (multipleMediaType) {
-            result.reason.push(exports.ErrorType.ResponseContainMultipleMediaTypes);
-        }
-        else if (Object.keys(json).length === 0) {
-            // response body should not be empty
-            result.reason.push(exports.ErrorType.ResponseJsonIsEmpty);
-        }
-        return result;
-    }
-    checkPostBodySchema(schema, isRequired = false) {
-        var _a;
-        const paramResult = {
-            requiredNum: 0,
-            optionalNum: 0,
-            isValid: true,
-            reason: [],
-        };
-        if (Object.keys(schema).length === 0) {
-            return paramResult;
-        }
-        const isRequiredWithoutDefault = isRequired && schema.default === undefined;
-        const isCopilot = this.projectType === exports.ProjectType.Copilot;
-        if (schema.type === "string" ||
-            schema.type === "integer" ||
-            schema.type === "boolean" ||
-            schema.type === "number") {
-            if (isRequiredWithoutDefault) {
-                paramResult.requiredNum = paramResult.requiredNum + 1;
-            }
-            else {
-                paramResult.optionalNum = paramResult.optionalNum + 1;
-            }
-        }
-        else if (Utils.isObjectSchema(schema)) {
-            const { properties } = schema;
-            for (const property in properties) {
-                let isRequired = false;
-                if (schema.required && ((_a = schema.required) === null || _a === void 0 ? void 0 : _a.indexOf(property)) >= 0) {
-                    isRequired = true;
-                }
-                const result = this.checkPostBodySchema(properties[property], isRequired);
-                paramResult.requiredNum += result.requiredNum;
-                paramResult.optionalNum += result.optionalNum;
-                paramResult.isValid = paramResult.isValid && result.isValid;
-                paramResult.reason.push(...result.reason);
-            }
-        }
-        else {
-            if (isRequiredWithoutDefault && !isCopilot) {
-                paramResult.isValid = false;
-                paramResult.reason.push(exports.ErrorType.PostBodyContainsRequiredUnsupportedSchema);
-            }
-        }
-        return paramResult;
-    }
-    checkParamSchema(paramObject) {
-        const paramResult = {
-            requiredNum: 0,
-            optionalNum: 0,
-            isValid: true,
-            reason: [],
-        };
-        if (!paramObject) {
-            return paramResult;
-        }
-        for (let i = 0; i < paramObject.length; i++) {
-            const param = paramObject[i];
-            const schema = param.schema;
-            const isRequiredWithoutDefault = param.required && schema.default === undefined;
-            if (param.in === "header" || param.in === "cookie") {
-                if (isRequiredWithoutDefault) {
-                    paramResult.isValid = false;
-                    paramResult.reason.push(exports.ErrorType.ParamsContainRequiredUnsupportedSchema);
-                }
-                continue;
-            }
-            if (schema.type !== "boolean" &&
-                schema.type !== "string" &&
-                schema.type !== "number" &&
-                schema.type !== "integer") {
-                if (isRequiredWithoutDefault) {
-                    paramResult.isValid = false;
-                    paramResult.reason.push(exports.ErrorType.ParamsContainRequiredUnsupportedSchema);
-                }
-                continue;
-            }
-            if (param.in === "query" || param.in === "path") {
-                if (isRequiredWithoutDefault) {
-                    paramResult.requiredNum = paramResult.requiredNum + 1;
-                }
-                else {
-                    paramResult.optionalNum = paramResult.optionalNum + 1;
-                }
-            }
-        }
-        return paramResult;
-    }
     validateParamCount(postBodyResult, paramResult) {
         const result = { isValid: true, reason: [] };
         const totalRequiredParams = postBodyResult.requiredNum + paramResult.requiredNum;
@@ -1735,7 +1751,7 @@ function inferProperties(card) {
 
 // Copyright (c) Microsoft Corporation.
 class ManifestUpdater {
-    static updateManifestWithAiPlugin(manifestPath, outputSpecPath, apiPluginFilePath, spec, options, authMap, existingPluginManifestInfo) {
+    static updateManifestWithAiPlugin(manifestPath, outputSpecPath, apiPluginFilePath, spec, options, authInfo, existingPluginManifestInfo) {
         return __awaiter(this, void 0, void 0, function* () {
             const manifest = yield fs__default['default'].readJSON(manifestPath);
             const apiPluginRelativePath = ManifestUpdater.getRelativePath(manifestPath, apiPluginFilePath);
@@ -1766,7 +1782,7 @@ class ManifestUpdater {
             }
             const appName = this.removeEnvs(manifest.name.short);
             const specRelativePath = ManifestUpdater.getRelativePath(manifestPath, outputSpecPath);
-            const [apiPlugin, warnings] = yield ManifestUpdater.generatePluginManifestSchema(spec, specRelativePath, apiPluginFilePath, appName, authMap, options, existingPluginManifestInfo);
+            const [apiPlugin, warnings] = yield ManifestUpdater.generatePluginManifestSchema(spec, specRelativePath, apiPluginFilePath, appName, authInfo, options, existingPluginManifestInfo);
             return [manifest, apiPlugin, warnings];
         });
     }
@@ -1789,14 +1805,29 @@ class ManifestUpdater {
             throw new SpecParserError(Utils.format(ConstantString.UnsupportedSchema, method, pathUrl, JSON.stringify(schema)), exports.ErrorType.UpdateManifestFailed);
         }
     }
-    static generatePluginManifestSchema(spec, specRelativePath, apiPluginFilePath, appName, authMap, options, existingPluginManifestInfo) {
+    static generatePluginManifestSchema(spec, specRelativePath, apiPluginFilePath, appName, authInfo, options, existingPluginManifestInfo) {
         var _a, _b, _c, _d;
         return __awaiter(this, void 0, void 0, function* () {
             const warnings = [];
             const functions = [];
-            const functionNamesMap = {};
+            const functionNames = [];
             const conversationStarters = [];
             const paths = spec.paths;
+            const pluginAuthObj = {
+                type: "None",
+            };
+            if (authInfo) {
+                if (Utils.isOAuthWithAuthCodeFlow(authInfo.authScheme)) {
+                    pluginAuthObj.type = "OAuthPluginVault";
+                }
+                else if (Utils.isBearerTokenAuth(authInfo.authScheme)) {
+                    pluginAuthObj.type = "ApiKeyPluginVault";
+                }
+                if (pluginAuthObj.type !== "None") {
+                    const safeRegistrationIdName = Utils.getSafeRegistrationIdEnvName(`${authInfo.name}_${ConstantString.RegistrationIdPostfix[authInfo.authScheme.type]}`);
+                    pluginAuthObj.reference_id = `\${{${safeRegistrationIdName}}}`;
+                }
+            }
             for (const pathUrl in paths) {
                 const pathItem = paths[pathUrl];
                 if (pathItem) {
@@ -1804,11 +1835,36 @@ class ManifestUpdater {
                     for (const method in operations) {
                         if (options.allowMethods.includes(method)) {
                             const operationItem = operations[method];
+                            const confirmationBodies = [];
                             if (operationItem) {
                                 const operationId = operationItem.operationId;
                                 const safeFunctionName = operationId.replace(/[^a-zA-Z0-9]/g, "_");
                                 const description = (_a = operationItem.description) !== null && _a !== void 0 ? _a : "";
                                 const summary = operationItem.summary;
+                                const paramObject = operationItem.parameters;
+                                const requestBody = operationItem.requestBody;
+                                if (paramObject) {
+                                    for (let i = 0; i < paramObject.length; i++) {
+                                        const param = paramObject[i];
+                                        const schema = param.schema;
+                                        ManifestUpdater.checkSchema(schema, method, pathUrl);
+                                        confirmationBodies.push(ManifestUpdater.getConfirmationBodyItem(param.name));
+                                    }
+                                }
+                                if (requestBody) {
+                                    const requestJsonBody = requestBody.content["application/json"];
+                                    const requestBodySchema = requestJsonBody.schema;
+                                    if (Utils.isObjectSchema(requestBodySchema)) {
+                                        for (const property in requestBodySchema.properties) {
+                                            const schema = requestBodySchema.properties[property];
+                                            ManifestUpdater.checkSchema(schema, method, pathUrl);
+                                            confirmationBodies.push(ManifestUpdater.getConfirmationBodyItem(property));
+                                        }
+                                    }
+                                    else {
+                                        throw new SpecParserError(Utils.format(ConstantString.UnsupportedSchema, method, pathUrl, JSON.stringify(requestBodySchema)), exports.ErrorType.UpdateManifestFailed);
+                                    }
+                                }
                                 let funcDescription = operationItem.description || operationItem.summary || "";
                                 if (funcDescription.length > ConstantString.FunctionDescriptionMaxLens) {
                                     warnings.push({
@@ -1842,66 +1898,19 @@ class ManifestUpdater {
                                     }
                                 }
                                 if (options.allowConfirmation && method !== ConstantString.GetMethod) {
-                                    const paramObject = operationItem.parameters;
-                                    const requestBody = operationItem.requestBody;
-                                    const confirmationBodies = [];
-                                    if (paramObject) {
-                                        for (let i = 0; i < paramObject.length; i++) {
-                                            const param = paramObject[i];
-                                            const schema = param.schema;
-                                            ManifestUpdater.checkSchema(schema, method, pathUrl);
-                                            confirmationBodies.push(ManifestUpdater.getConfirmationBodyItem(param.name));
-                                        }
-                                    }
-                                    if (requestBody) {
-                                        const requestJsonBody = Utils.getJsonContentType(requestBody);
-                                        const requestBodySchema = requestJsonBody.schema;
-                                        if (Utils.isObjectSchema(requestBodySchema)) {
-                                            for (const property in requestBodySchema.properties) {
-                                                const schema = requestBodySchema.properties[property];
-                                                ManifestUpdater.checkSchema(schema, method, pathUrl);
-                                                confirmationBodies.push(ManifestUpdater.getConfirmationBodyItem(property));
-                                            }
-                                        }
-                                        else {
-                                            throw new SpecParserError(Utils.format(ConstantString.UnsupportedSchema, method, pathUrl, JSON.stringify(requestBodySchema)), exports.ErrorType.UpdateManifestFailed);
-                                        }
+                                    if (!funcObj.capabilities) {
+                                        funcObj.capabilities = {};
                                     }
+                                    funcObj.capabilities.confirmation = {
+                                        type: "AdaptiveCard",
+                                        title: (_b = operationItem.summary) !== null && _b !== void 0 ? _b : description,
+                                    };
                                     if (confirmationBodies.length > 0) {
-                                        if (!funcObj.capabilities) {
-                                            funcObj.capabilities = {};
-                                        }
-                                        funcObj.capabilities.confirmation = {
-                                            type: "AdaptiveCard",
-                                            title: (_b = operationItem.summary) !== null && _b !== void 0 ? _b : description,
-                                        };
                                         funcObj.capabilities.confirmation.body = confirmationBodies.join("\n");
                                     }
                                 }
                                 functions.push(funcObj);
-                                const authInfo = authMap[operationId];
-                                let key = "None";
-                                let authName = "None";
-                                if (authInfo) {
-                                    if (Utils.isOAuthWithAuthCodeFlow(authInfo.authScheme)) {
-                                        key = "OAuthPluginVault";
-                                        authName = authInfo.name;
-                                    }
-                                    else if (Utils.isBearerTokenAuth(authInfo.authScheme) ||
-                                        Utils.isAPIKeyAuthButNotInCookie(authInfo.authScheme)) {
-                                        key = "ApiKeyPluginVault";
-                                        authName = authInfo.name;
-                                    }
-                                }
-                                if (functionNamesMap[key]) {
-                                    functionNamesMap[key].functionNames.push(safeFunctionName);
-                                }
-                                else {
-                                    functionNamesMap[key] = {
-                                        functionNames: [safeFunctionName],
-                                        authName: authName,
-                                    };
-                                }
+                                functionNames.push(safeFunctionName);
                                 const conversationStarterStr = (summary !== null && summary !== void 0 ? summary : description).slice(0, ConstantString.ConversationStarterMaxLens);
                                 if (conversationStarterStr) {
                                     conversationStarters.push(conversationStarterStr);
@@ -1911,12 +1920,6 @@ class ManifestUpdater {
                     }
                 }
             }
-            if (Object.keys(functionNamesMap).length === 0) {
-                functionNamesMap["None"] = {
-                    functionNames: [],
-                    authName: "None",
-                };
-            }
             let apiPlugin;
             if (yield fs__default['default'].pathExists(apiPluginFilePath)) {
                 apiPlugin = yield fs__default['default'].readJSON(apiPluginFilePath);
@@ -1952,35 +1955,24 @@ class ManifestUpdater {
                 const relativePath = ManifestUpdater.getRelativePath(existingPluginManifestInfo.manifestPath, existingPluginManifestInfo.specPath);
                 apiPlugin.runtimes = apiPlugin.runtimes.filter((runtime) => runtime.spec.url !== relativePath);
             }
-            for (const authType in functionNamesMap) {
-                const pluginAuthObj = {
-                    type: authType,
-                };
-                const authName = functionNamesMap[authType].authName;
-                if (pluginAuthObj.type !== "None") {
-                    const safeRegistrationIdName = Utils.getSafeRegistrationIdEnvName(`${authName}_${ConstantString.RegistrationIdPostfix}`);
-                    pluginAuthObj.reference_id = `\${{${safeRegistrationIdName}}}`;
-                }
-                const functionNamesInfo = functionNamesMap[authType];
-                const index = apiPlugin.runtimes.findIndex((runtime) => {
-                    var _a, _b;
-                    return runtime.spec.url === specRelativePath &&
-                        runtime.type === "OpenApi" &&
-                        ((_b = (_a = runtime.auth) === null || _a === void 0 ? void 0 : _a.type) !== null && _b !== void 0 ? _b : "None") === authType;
+            const index = apiPlugin.runtimes.findIndex((runtime) => {
+                var _a, _b;
+                return runtime.spec.url === specRelativePath &&
+                    runtime.type === "OpenApi" &&
+                    ((_b = (_a = runtime.auth) === null || _a === void 0 ? void 0 : _a.type) !== null && _b !== void 0 ? _b : "None") === pluginAuthObj.type;
+            });
+            if (index === -1) {
+                apiPlugin.runtimes.push({
+                    type: "OpenApi",
+                    auth: pluginAuthObj,
+                    spec: {
+                        url: specRelativePath,
+                    },
+                    run_for_functions: functionNames,
                 });
-                if (index === -1) {
-                    apiPlugin.runtimes.push({
-                        type: "OpenApi",
-                        auth: pluginAuthObj,
-                        spec: {
-                            url: specRelativePath,
-                        },
-                        run_for_functions: functionNamesInfo.functionNames,
-                    });
-                }
-                else {
-                    apiPlugin.runtimes[index].run_for_functions = functionNamesInfo.functionNames;
-                }
+            }
+            else {
+                apiPlugin.runtimes[index].run_for_functions = functionNames;
             }
             if (!apiPlugin.name_for_human) {
                 apiPlugin.name_for_human = appName;
@@ -2023,7 +2015,7 @@ class ManifestUpdater {
                     };
                     if (authInfo) {
                         const auth = authInfo.authScheme;
-                        const safeRegistrationIdName = Utils.getSafeRegistrationIdEnvName(`${authInfo.name}_${ConstantString.RegistrationIdPostfix}`);
+                        const safeRegistrationIdName = Utils.getSafeRegistrationIdEnvName(`${authInfo.name}_${ConstantString.RegistrationIdPostfix[authInfo.authScheme.type]}`);
                         if (Utils.isAPIKeyAuth(auth) || Utils.isBearerTokenAuth(auth)) {
                             composeExtension.authorization = {
                                 authType: "apiSecretServiceAuth",
@@ -2153,7 +2145,6 @@ class SpecParser {
         };
         this.pathOrSpec = pathOrDoc;
         this.parser = new SwaggerParser__default['default']();
-        this.refParser = new jsonSchemaRefParser.$RefParser();
         this.options = Object.assign(Object.assign({}, this.defaultOptions), (options !== null && options !== void 0 ? options : {}));
     }
     /**
@@ -2167,15 +2158,12 @@ class SpecParser {
                 let hash = "";
                 try {
                     yield this.loadSpec();
-                    if (!this.refParser.$refs.circular) {
+                    if (!this.parser.$refs.circular) {
                         yield this.parser.validate(this.spec);
                     }
                     else {
-                        // The following code hangs for Graph API, support will be added when SwaggerParser is updated.
-                        /*
                         const clonedUnResolveSpec = JSON.parse(JSON.stringify(this.unResolveSpec));
-                        await this.parser.validate(clonedUnResolveSpec);
-                        */
+                        yield this.parser.validate(clonedUnResolveSpec);
                     }
                 }
                 catch (e) {
@@ -2203,7 +2191,7 @@ class SpecParser {
                     };
                 }
                 // Remote reference not supported
-                const refPaths = this.refParser.$refs.paths();
+                const refPaths = this.parser.$refs.paths();
                 // refPaths [0] is the current spec file path
                 if (refPaths.length > 1) {
                     errors.push({
@@ -2338,7 +2326,7 @@ class SpecParser {
                     throw new SpecParserError(ConstantString.CancelledMessage, exports.ErrorType.Cancelled);
                 }
                 const clonedUnResolveSpec = JSON.parse(JSON.stringify(newUnResolvedSpec));
-                const newSpec = yield this.deReferenceSpec(clonedUnResolveSpec);
+                const newSpec = (yield this.parser.dereference(clonedUnResolveSpec));
                 return [newUnResolvedSpec, newSpec];
             }
             catch (err) {
@@ -2349,12 +2337,6 @@ class SpecParser {
             }
         });
     }
-    deReferenceSpec(spec) {
-        return __awaiter(this, void 0, void 0, function* () {
-            const result = yield this.refParser.dereference(spec);
-            return result;
-        });
-    }
     /**
      * Generates and update artifacts from the OpenAPI specification file. Generate Adaptive Cards, update Teams app manifest, and generate a new OpenAPI specification file.
      * @param manifestPath A file path of the Teams app manifest file to update.
@@ -2372,6 +2354,7 @@ class SpecParser {
                 const newSpecs = yield this.getFilteredSpecs(filter, signal);
                 const newUnResolvedSpec = newSpecs[0];
                 const newSpec = newSpecs[1];
+                const authInfo = Utils.getAuthInfo(newSpec);
                 const paths = newUnResolvedSpec.paths;
                 for (const pathUrl in paths) {
                     const operations = paths[pathUrl];
@@ -2379,22 +2362,15 @@ class SpecParser {
                         const operationItem = operations[method];
                         const operationId = operationItem.operationId;
                         const containsSpecialCharacters = /[^a-zA-Z0-9_]/.test(operationId);
-                        if (containsSpecialCharacters) {
-                            operationItem.operationId = operationId.replace(/[^a-zA-Z0-9]/g, "_");
-                            result.warnings.push({
-                                type: exports.WarningType.OperationIdContainsSpecialCharacters,
-                                content: Utils.format(ConstantString.OperationIdContainsSpecialCharacters, operationId, operationItem.operationId),
-                                data: operationId,
-                            });
-                        }
-                        const authArray = Utils.getAuthArray(operationItem.security, newSpec);
-                        if (Utils.isNotSupportedAuth(authArray)) {
-                            result.warnings.push({
-                                type: exports.WarningType.UnsupportedAuthType,
-                                content: Utils.format(ConstantString.AuthTypeIsNotSupported, operationId),
-                                data: operationId,
-                            });
+                        if (!containsSpecialCharacters) {
+                            continue;
                         }
+                        operationItem.operationId = operationId.replace(/[^a-zA-Z0-9]/g, "_");
+                        result.warnings.push({
+                            type: exports.WarningType.OperationIdContainsSpecialCharacters,
+                            content: Utils.format(ConstantString.OperationIdContainsSpecialCharacters, operationId, operationItem.operationId),
+                            data: operationId,
+                        });
                     }
                 }
                 yield this.saveFilterSpec(outputSpecPath, newUnResolvedSpec);
@@ -2407,8 +2383,7 @@ class SpecParser {
                         specPath: this.pathOrSpec,
                     }
                     : undefined;
-                const authMap = Utils.getAuthMap(newSpec);
-                const [updatedManifest, apiPlugin, warnings] = yield ManifestUpdater.updateManifestWithAiPlugin(manifestPath, outputSpecPath, pluginFilePath, newSpec, this.options, authMap, existingPluginManifestInfo);
+                const [updatedManifest, apiPlugin, warnings] = yield ManifestUpdater.updateManifestWithAiPlugin(manifestPath, outputSpecPath, pluginFilePath, newSpec, this.options, authInfo, existingPluginManifestInfo);
                 result.warnings.push(...warnings);
                 yield fs__default['default'].outputJSON(manifestPath, updatedManifest, { spaces: 4 });
                 yield fs__default['default'].outputJSON(pluginFilePath, apiPlugin, { spaces: 4 });
@@ -2500,7 +2475,7 @@ class SpecParser {
                     this.isSwaggerFile = true;
                 }
                 const clonedUnResolveSpec = JSON.parse(JSON.stringify(this.unResolveSpec));
-                this.spec = yield this.deReferenceSpec(clonedUnResolveSpec);
+                this.spec = (yield this.parser.dereference(clonedUnResolveSpec));
             }
         });
     }