feat(aws-iac-mcp-server): CDK best practices tool

[vishaalmehrishi]

Summary

Adding the CDK best practices tool.

Note: The starting point for the best practices is a summary of the official guidance. I'd like input on this and figured that the PR is a good place to have the discussion.

Changes

Adding a new IaC MCP server tool for CDK best practices.

User experience

AI agents will be able to serve customer requests while consulting CDK best practices for better results.

Checklist

If your change doesn't seem to apply, please leave them unchecked.

• 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? (Y/N)

RFC issue number:

Checklist:

• Migration process documented
• Implement warnings (if it can live side by side)

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

❌ Patch coverage is 90.90909% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 90.36%. Comparing base (7bace1f) to head (295c7df).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...rver/awslabs/aws_iac_mcp_server/tools/cdk_tools.py	66.66%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1832      +/-   ##
==========================================
- Coverage   90.36%   90.36%   -0.01%     
==========================================
  Files         810      811       +1     
  Lines       62171    62182      +11     
  Branches    10042    10042              
==========================================
+ Hits        56182    56191       +9     
- Misses       3734     3735       +1     
- Partials     2255     2256       +1     

☔ 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.

[praneetap]

Nothing on using L2s over L1?

[vishaalmehrishi]

Nothing on using L2s over L1?

This is not in the official guidance, so it was not included.

Using L2s over L1s requires customers to use L2s across their stack (or with the newer changes coming, understand how to make L1s and L2s work together if there is no L2 for the service they use). I can try to add something about this, but if a customer goes looking for official guidance on this, they will not find any.

[kdbrogan]

LGTM

[vishaalmehrishi]

After some internal discussion, I have updated the CDK best practices guidance using the official sources we discussed, plus some opinionated guidance (the potential use of CDK Nag).

I renamed the file (cdk_best_practices_official -> cdk_best_practices) since this is no longer completely official CDK guidance, which is why you will see a new file in the diff.