CowDogMoo / bcp / 19992301636

44%

main: 53%

Pull #3

github

refactor: migrate to aws-sdk-go-v2 and improve error handling

**Added:**

- Implement mock smithy.APIError for testing retry logic with AWS SDK v2 in
  `pkg/transfer/transfer_test.go`
- Add utility functions for uploading files and directories to S3 using AWS SDK v2
- Add SSM command execution and AWS CLI presence check functions using AWS SDK v2
- Add improved error handling and fallback logic in logging package for log
  write failures

**Changed:**

- Migrate all AWS SDK usage from aws-sdk-go v1 to aws-sdk-go-v2 across codebase,
  including S3, SSM, and EC2 interactions
- Update all AWS service calls to use context, new config loading, and new
  response types as per AWS SDK v2
- Refactor S3 and SSM logic in `pkg/completion/aws.go` and `cmd/list.go` for
  compatibility with AWS SDK v2 and improved pagination
- Update retry logic in transfer package to handle smithy.APIError from AWS SDK v2
- Refactor log output functions to attempt fallback or panic on log write
  failures, ensuring critical errors are not silently ignored
- Update test cleanup in multiple test files to log temp dir removal failures
  rather than silently ignoring errors
- Bump indirect dependencies and update `go.mod`/`go.sum` to reflect SDK v2
  adoption and remove unused v1 dependencies

**Removed:**

- Remove all usage of aws-sdk-go v1, related helpers, and dependencies from
  `go.mod` and `go.sum`
- Remove dependency on external `awsutils` helper modules in transfer logic
- Eliminate all v1-specific error handling branches in favor of smithy error
  types from AWS SDK v2

324 of 740 new or added lines in 8 files covered. (43.78%)

334 of 758 relevant lines covered (44.06%)

2.33 hits per line