fix(): address comments feedback

Author: Karnaukhov-kh

packages/angular-mcp-server/src/lib/tools/ds/component/utils/deprecated-css-helpers.ts

@@ -33,7 +33,6 @@ export async function getDeprecatedCssClasses(
 
   const module = await import(absPath);
 
-  // Pure ESM: use export default
   const dsComponents = module.default;
 
   if (!Array.isArray(dsComponents)) {

packages/angular-mcp-server/src/lib/validation/ds-components-file-loader.validation.ts

@@ -21,12 +21,10 @@ export function validateDsComponent(rawComponent: unknown): DsComponent {
 }
 
 export function validateDsComponentsArray(rawData: unknown): DsComponentsArray {
-  // First check if it's an array
   if (!Array.isArray(rawData)) {
     throw new Error(`Expected array of components, received ${typeof rawData}`);
   }
 
-  // Validate each component individually for better error messages
   const validatedComponents: DsComponent[] = [];
   for (let i = 0; i < rawData.length; i++) {
     try {
@@ -37,7 +35,6 @@ export function validateDsComponentsArray(rawData: unknown): DsComponentsArray {
     }
   }
 
-  // Final validation with the array schema for completeness
   const arrayValidation =
     DsComponentsArraySchema.safeParse(validatedComponents);
   if (!arrayValidation.success) {
@@ -71,10 +68,8 @@ export async function loadAndValidateDsComponentsFile(
     const fileUrl = pathToFileURL(absPath).toString();
     const module = await import(fileUrl);
 
-    // Pure ESM: use export default
     const rawData = module.default;
 
-    // Use the granular validation functions
     return validateDsComponentsArray(rawData);
   } catch (ctx) {
     if (
@@ -83,7 +78,7 @@ export async function loadAndValidateDsComponentsFile(
         ctx.message.includes('Expected array of components') ||
         ctx.message.includes('Component at index'))
     ) {
-      throw ctx; // Re-throw validation errors as-is
+      throw ctx;
     }
     throw new Error(
       `Failed to load configuration file: ${(ctx as Error).message}`,

packages/angular-mcp-server/src/lib/validation/ds-components-file.validation.ts

@@ -16,7 +16,6 @@ export async function validateDeprecatedCssClassesFile(
     const fileUrl = pathToFileURL(deprecatedCssClassesAbsPath).toString();
     const module = await import(fileUrl);
 
-    // Pure ESM: use export default
     dsComponents = module.default;
   } catch (ctx) {
     throw new Error(
@@ -29,7 +28,6 @@ export async function validateDeprecatedCssClassesFile(
     );
   }
 
-  // Validate the schema
   const validation = DsComponentsArraySchema.safeParse(dsComponents);
   if (!validation.success) {
     const actualType = Array.isArray(dsComponents)

packages/shared/typescript-ast-utils/src/index.ts

@@ -1,6 +1,5 @@
 export * from './lib/constants.js';
 export * from './lib/utils.js';
 
-// Explicit exports for commonly used functions
 export { removeQuotes } from './lib/utils.js';
 export { QUOTE_REGEX } from './lib/constants.js';