feat(iac-mcp-server): Iac mcp further improvements

[kumvprat]

Fixes

Summary

Changes

Refactored the AWS IaC MCP Server with comprehensive code organization improvements and Pydantic integration for enhanced maintainability and type safety:

Code Organization & Structure:

• Modularized architecture into logical directories: models/, tools/, utils/
• Eliminated monolithic files by breaking them into focused, single-responsibility modules
• Standardized naming conventions across the entire codebase

Pydantic Integration:

• Introduced type-safe data models for all tool inputs/outputs
• Enhanced input validation and error handling with runtime type checking
• Streamlined JSON serialization with proper type conversion

Tool Separation:

• validation_tools.py - CloudFormation template validation using cfn-lint
• compliance_tools.py - Security compliance checking using cfn-guard
• deployment_tools.py - CloudFormation deployment troubleshooting
• cdk_tools.py - CDK documentation and sample search tools

File Reorganization:

• Moved utility modules (failure_cases.py, sanitizer.py) to utils/ directory
• Removed deprecated deployment_troubleshooter.py (functionality moved to deployment_tools.py)
• Updated all import paths to reflect new modular structure

User Experience

Before: The AWS IaC MCP Server had a monolithic structure with mixed concerns, making it difficult to maintain, extend, and debug. Type safety was limited, and code organization was inconsistent.

After: Users now benefit from:

• Better maintainability: Clear separation of concerns makes code easier to understand and modify
• Enhanced type safety: Pydantic models provide runtime validation and better IDE support with autocomplete
• Consistent patterns: Standardized approach across all tools and modules
• Improved error handling: Better validation and more informative error messages
• Easier testing: Modular structure enables more focused unit tests
• Future extensibility: Well-organized structure makes adding new tools straightforward

Technical Details

• 28 files changed: 1,283 additions, 1,120 deletions (net code reduction through efficiency)
• No breaking changes: All existing tool interfaces remain unchanged
• Backward compatibility: Existing configurations and usage patterns continue to work
• Performance: No performance impact, improved error handling
• Dependencies: No new external dependencies added

Checklist

• I have reviewed the contributing guidelines
• I have performed a self-review of this change
• Changes have been tested
• Changes are documented

Is this a breaking change? N

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.33%. Comparing base (2f689ec) to head (230c9b4).
⚠️ Report is 46 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1840      +/-   ##
==========================================
- Coverage   90.36%   90.33%   -0.04%     
==========================================
  Files         810      799      -11     
  Lines       62171    61735     -436     
  Branches    10042     9975      -67     
==========================================
- Hits        56181    55766     -415     
+ Misses       3734     3724      -10     
+ Partials     2256     2245      -11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[github-actions]

This pull request is now marked as stale because it hasn't seen activity for a while. Add a comment or it will be closed soon. If you wish to exclude this issue from being marked as stale, add the "backlog" label.