feat: restructure AWS connector with focused core services

[mattzcarey]

Restructured AWS connector removing unnecessary tools and focusing on core AWS services:

• ✅ Lambda function management (list, get, invoke with sync/async/dry-run)
• ✅ CloudWatch Logs (filter events, list groups)
• ✅ ECS management (clusters, services, tasks)
• ✅ EC2 instance listing
• ✅ Cost analysis APIs
• ✅ Production-ready native fetch implementation
• ✅ Comprehensive test coverage

Closes #61

🤖 Generated with Claude Code

---

Summary by cubic

Restructures the AWS connector around core services and improves reliability and test coverage, addressing #61. Adds Lambda sync/async/dry-run invocation, CloudWatch Logs, and ECS management while keeping EC2 and Cost Explorer.

• New Features

  • Lambda: list/get/invoke with RequestResponse, Event, and DryRun.
  • CloudWatch Logs: filter log events and list log groups.
  • ECS: list clusters/services/tasks and describe services.
  • Native fetch with AWS SigV4 and comprehensive tests.

• Migration

  • Removed S3 tools.
  • Replace aws_get_cloudwatch_metrics with aws_get_cloudwatch_logs; update inputs to use logGroupName, startTime/endTime (ms), filterPattern, and limit.
  • Version 2.0.0; Lambda invoke adds optional invocationType (defaults to RequestResponse).

[Copilot]

Pull Request Overview

This PR restructures the AWS connector by focusing on core AWS services and improving the API design. The changes replace CloudWatch metrics functionality with CloudWatch logs and add comprehensive ECS management capabilities.

• Replaces CloudWatch metrics tool with CloudWatch logs functionality
• Adds comprehensive ECS management tools (clusters, services, tasks)
• Enhances Lambda invocation with support for sync/async/dry-run modes
• Adds comprehensive test coverage for all tools

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
packages/mcp-connectors/src/connectors/aws.ts	Updates version to 2.0.0, replaces CloudWatch metrics with logs, adds ECS tools, enhances Lambda invocation
packages/mcp-connectors/src/connectors/aws.test.ts	Adds comprehensive test coverage for all AWS connector tools and error handling

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Bumping the version to 2.0.0 indicates breaking changes, but the change from CloudWatch metrics to logs should be documented in the PR description or changelog to help users understand what functionality was removed.

[cubic-dev-ai]

2 issues found across 2 files

_{React with 👍 or 👎 to teach cubic. You can also tag @cubic-dev-ai to give feedback, ask questions, or re-run the review.}

[cubic-dev-ai]

Avoid committing hard-coded AWS access keys; use obviously fake placeholders to prevent false positives from secret scanners

Prompt for AI agents

Address the following comment on packages/mcp-connectors/src/connectors/aws.test.ts at line 34:

<comment>Avoid committing hard-coded AWS access keys; use obviously fake placeholders to prevent false positives from secret scanners</comment>

<file context>
@@ -0,0 +1,484 @@
+import { beforeEach, describe, expect, test, vi } from &#39;vitest&#39;;
+import { AwsConnectorConfig } from &#39;./aws&#39;;
+
+// Mock fetch globally
+const mockFetch = vi.fn();
+global.fetch = mockFetch;
+
+// Mock crypto.subtle for AWS signing
+global.crypto = {
</file context>

[cubic-dev-ai]

Replacing the whole global.crypto object with a partial stub can break other code that expects the full WebCrypto API

Prompt for AI agents

Address the following comment on packages/mcp-connectors/src/connectors/aws.test.ts at line 9:

<comment>Replacing the whole global.crypto object with a partial stub can break other code that expects the full WebCrypto API</comment>

<file context>
@@ -0,0 +1,484 @@
+import { beforeEach, describe, expect, test, vi } from &#39;vitest&#39;;
+import { AwsConnectorConfig } from &#39;./aws&#39;;
+
+// Mock fetch globally
+const mockFetch = vi.fn();
+global.fetch = mockFetch;
+
+// Mock crypto.subtle for AWS signing
+global.crypto = {
</file context>