[PR #17] [MERGED] Ensure PascalCase serialization for MCP tools types #175

New issue

Closed

opened 2026-03-15 11:51:46 +03:00 by kerem · 0 comments

kerem commented

2026-03-15 11:51:46 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/awslabs/iam-policy-autopilot/pull/17
Author: @adpaco-aws
Created: 11/21/2025
Status: ✅ Merged
Merged: 11/24/2025
Merged by: @adpaco-aws

Base: prerelease ← Head: fix-serialize-mcp-tools

📝 Commits (4)

6bf9c2a Ensure PascalCase serialization for MCP tools types
4cb13c0 Fix MCP tools unit tests
192df1a Merge branch 'prerelease' into fix-serialize-mcp-tools
3cfd08c Merge branch 'prerelease' into fix-serialize-mcp-tools

📊 Changes

4 files changed (+109 additions, -16 deletions)

View changed files

📝 iam-policy-autopilot-mcp-server/src/tools/fix_access_denied.rs (+45 -0)
📝 iam-policy-autopilot-mcp-server/src/tools/generate_policy.rs (+31 -0)
📝 iam-policy-autopilot-mcp-server/src/tools/generate_policy_for_access_denied.rs (+25 -0)
📝 iam-policy-autopilot-mcp-server/tests/mcp_server_integration_test.rs (+8 -16)

📄 Description

Description of changes: Adds PascalCase serialization for the following MCP tool input/output types:

GeneratePoliciesInput
GeneratePoliciesOutput
FixAccessDeniedInput
FixAccessDeniedOutput and FixResult
GeneratePolicyForAccessDeniedInput
GeneratePolicyForAccessDeniedOutput

Added tests to verify serialization of these types.

Note: We are making updates to ensure PascalCase serialization across our public types, this PR covers the MCP packages.

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

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/awslabs/iam-policy-autopilot/pull/17 **Author:** [@adpaco-aws](https://github.com/adpaco-aws) **Created:** 11/21/2025 **Status:** ✅ Merged **Merged:** 11/24/2025 **Merged by:** [@adpaco-aws](https://github.com/adpaco-aws) **Base:** `prerelease` ← **Head:** `fix-serialize-mcp-tools` --- ### 📝 Commits (4) - [`6bf9c2a`](https://github.com/awslabs/iam-policy-autopilot/commit/6bf9c2a30aceadac4e64257ce5b2f3b3a32c1480) Ensure PascalCase serialization for MCP tools types - [`4cb13c0`](https://github.com/awslabs/iam-policy-autopilot/commit/4cb13c035503f1d6efbff025a850e53b1dba6e1e) Fix MCP tools unit tests - [`192df1a`](https://github.com/awslabs/iam-policy-autopilot/commit/192df1a2e055a461b6a45be8906078ba4cf9ca39) Merge branch 'prerelease' into fix-serialize-mcp-tools - [`3cfd08c`](https://github.com/awslabs/iam-policy-autopilot/commit/3cfd08c406bdedfe315e19b6aa56cf8129854f17) Merge branch 'prerelease' into fix-serialize-mcp-tools ### 📊 Changes **4 files changed** (+109 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `iam-policy-autopilot-mcp-server/src/tools/fix_access_denied.rs` (+45 -0) 📝 `iam-policy-autopilot-mcp-server/src/tools/generate_policy.rs` (+31 -0) 📝 `iam-policy-autopilot-mcp-server/src/tools/generate_policy_for_access_denied.rs` (+25 -0) 📝 `iam-policy-autopilot-mcp-server/tests/mcp_server_integration_test.rs` (+8 -16) </details> ### 📄 Description *Description of changes:* Adds PascalCase serialization for the following MCP tool input/output types: - `GeneratePoliciesInput` - `GeneratePoliciesOutput` - `FixAccessDeniedInput` - `FixAccessDeniedOutput` and `FixResult` - `GeneratePolicyForAccessDeniedInput` - `GeneratePolicyForAccessDeniedOutput` Added tests to verify serialization of these types. Note: We are making updates to ensure PascalCase serialization across our public types, this PR covers the MCP packages. By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>