fix: Remove ExpectedBucketOwner parameter from create_bucket method

The ExpectedBucketOwner parameter is not supported by the S3 create_bucket
API call. It is only valid for operations on existing buckets.

This fix removes the parameter from both us-east-1 and regional bucket
creation calls, resolving API errors during bucket creation.

Changes:
- Removed ExpectedBucketOwner from create_bucket() calls in s3.py
- Updated test assertions to match corrected API calls

Fixes #330
Related to PR #331

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: 2 people

src/bedrock_agentcore_starter_toolkit/services/s3.py

@@ -67,13 +67,9 @@ def create_s3_bucket(bucket_name: str, region: str, account_id: str) -> str:
 
     try:
         if region == "us-east-1":
-            s3.create_bucket(Bucket=bucket_name, ExpectedBucketOwner=account_id)
+            s3.create_bucket(Bucket=bucket_name)
         else:
-            s3.create_bucket(
-                Bucket=bucket_name,
-                CreateBucketConfiguration={"LocationConstraint": region},
-                ExpectedBucketOwner=account_id,
-            )
+            s3.create_bucket(Bucket=bucket_name, CreateBucketConfiguration={"LocationConstraint": region})
 
         s3.put_bucket_lifecycle_configuration(
             Bucket=bucket_name,

tests/services/test_s3.py

@@ -124,7 +124,7 @@ def test_create_bucket_us_east_1(self, mock_boto3_client):
         result = create_s3_bucket("test-bucket", "us-east-1", "123456789012")
 
         assert result == "test-bucket"
-        mock_s3.create_bucket.assert_called_once_with(Bucket="test-bucket", ExpectedBucketOwner="123456789012")
+        mock_s3.create_bucket.assert_called_once_with(Bucket="test-bucket")
         mock_s3.put_bucket_lifecycle_configuration.assert_called_once()
 
     @patch("bedrock_agentcore_starter_toolkit.services.s3.boto3.client")
@@ -137,9 +137,7 @@ def test_create_bucket_other_region(self, mock_boto3_client):
 
         assert result == "test-bucket"
         mock_s3.create_bucket.assert_called_once_with(
-            Bucket="test-bucket",
-            CreateBucketConfiguration={"LocationConstraint": "us-west-2"},
-            ExpectedBucketOwner="123456789012",
+            Bucket="test-bucket", CreateBucketConfiguration={"LocationConstraint": "us-west-2"}
         )
         mock_s3.put_bucket_lifecycle_configuration.assert_called_once()