Basic Syntax Fixing

Author: notyashhh

tools/Mcp/src/services/toolsService.ts

@@ -5,12 +5,6 @@ import path from 'path';
 import { get, RequestOptions } from 'http';
 import { toolParameterSchema } from '../types.js';
 import { CodegenServer } from '../CodegenServer.js';
-import {
-    listSpecModules,
-    listProvidersForService,
-    listApiVersions,
-    resolveAutorestInputs
-} from './utils.js';
 
 export class ToolsService {
     private static _instance: ToolsService;
@@ -180,7 +174,7 @@ export class ToolsService {
     setupModuleStructure = async <Args extends ZodRawShape>(args: Args): Promise<string[]> => {
         try {
             // List available services with dropdown
-            const modules = await listSpecModules();
+            const modules = await utils.listSpecModules();
             const serviceResponse = await this._server!.elicitInput({
                 message: `Select an Azure service from the dropdown below:`,
                 requestedSchema: {
@@ -202,7 +196,7 @@ export class ToolsService {
             }
 
             // List providers for the selected service with dropdown
-            const providers = await listProvidersForService(selectedService);
+            const providers = await utils.listProvidersForService(selectedService);
             if (providers.length === 0) {
                 throw new Error(`No providers found for service '${selectedService}'`);
             }
@@ -228,7 +222,7 @@ export class ToolsService {
             }
 
             // List API versions with dropdown combining version and stability
-            const apiVersions = await listApiVersions(selectedService, selectedProvider);
+            const apiVersions = await utils.listApiVersions(selectedService, selectedProvider);
             const allVersions = [
                 ...apiVersions.stable.map(v => ({ version: v, stability: 'stable' as const })),
                 ...apiVersions.preview.map(v => ({ version: v, stability: 'preview' as const }))
@@ -269,7 +263,7 @@ export class ToolsService {
             const selectedStability = versionMatch[2] as 'stable' | 'preview';
 
             // Resolve Readme placeholder values based on Responses
-            const resolved = await resolveAutorestInputs({
+            const resolved = await utils.resolveAutorestInputs({
                 service: selectedService,
                 provider: selectedProvider,
                 stability: selectedStability,

tools/Mcp/src/services/utils.ts

@@ -82,9 +82,6 @@ export async function getSwaggerContentFromUrl(swaggerUrl: string): Promise<any>
     }
 }
 
-/**
- * GitHub helper: get latest commit SHA for azure-rest-api-specs main branch
- */
 export async function getSpecsHeadCommitSha(branch: string = 'main'): Promise<string> {
     const url = `${GITHUB_API_BASE}/repos/${REST_API_SPECS_OWNER}/${REST_API_SPECS_REPO}/branches/${branch}`;
     const res = await fetch(url);
@@ -95,9 +92,6 @@ export async function getSpecsHeadCommitSha(branch: string = 'main'): Promise<st
     return data?.commit?.sha as string;
 }
 
-/**
- * List top-level service directories under specification/
- */
 export async function listSpecModules(): Promise<string[]> {
     const url = `${GITHUB_API_BASE}/repos/${REST_API_SPECS_OWNER}/${REST_API_SPECS_REPO}/contents/specification`;
     const res = await fetch(url);
@@ -111,9 +105,6 @@ export async function listSpecModules(): Promise<string[]> {
         .sort((a: string, b: string) => a.localeCompare(b));
 }
 
-/**
- * Given a service (spec folder), list provider namespaces under resource-manager.
- */
 export async function listProvidersForService(service: string): Promise<string[]> {
     const url = `${GITHUB_API_BASE}/repos/${REST_API_SPECS_OWNER}/${REST_API_SPECS_REPO}/contents/specification/${service}/resource-manager`;
     const res = await fetch(url);
@@ -128,10 +119,6 @@ export async function listProvidersForService(service: string): Promise<string[]
         .sort((a: string, b: string) => a.localeCompare(b));
 }
 
-/**
- * For service + provider, list API version directories under stable/ and preview/.
- * Returns map: { stable: string[], preview: string[] }
- */
 export async function listApiVersions(service: string, provider: string): Promise<{ stable: string[]; preview: string[] }> {
     const base = `specification/${service}/resource-manager/${provider}`;
     const folders = ['stable', 'preview'] as const;
@@ -153,10 +140,6 @@ export async function listApiVersions(service: string, provider: string): Promis
     return result;
 }
 
-/**
- * For a given service/provider/version, find likely swagger files (.json) under that version path.
- * Returns array of repo-relative file paths (starting with specification/...).
- */
 export async function listSwaggerFiles(service: string, provider: string, stability: 'stable'|'preview', version: string): Promise<string[]> {
     const dir = `specification/${service}/resource-manager/${provider}/${stability}/${version}`;
     const url = `${GITHUB_API_BASE}/repos/${REST_API_SPECS_OWNER}/${REST_API_SPECS_REPO}/contents/${dir}`;
@@ -173,9 +156,6 @@ export async function listSwaggerFiles(service: string, provider: string, stabil
     return ordered;
 }
 
-/**
- * Resolve the four Autorest inputs given service, provider, and version path.
- */
 export async function resolveAutorestInputs(params: {
     service: string;
     provider: string;