Add concurrent transaction tests for DynamoDB adapter with AWS SDK v2 migration and golangci-lint fixes

[Copilot]

This PR adds comprehensive test coverage for DynamoDB adapter transactions under concurrent load, migrates from the deprecated AWS SDK v1 to AWS SDK v2, and resolves all golangci-lint issues to ensure CI passes.

Problem

The existing DynamoDB adapter transaction tests only covered single-threaded scenarios and were using the deprecated AWS SDK v1, which triggered SA1019 staticcheck warnings. Additionally, golangci-lint was failing due to version compatibility issues and unsafe type assertions, preventing CI from passing. Given that distributed systems often handle concurrent requests, it was important to verify that the transaction implementation correctly handles:

• Multiple concurrent transactions without data corruption
• Proper isolation when multiple goroutines access shared resources
• Atomicity guarantees under concurrent load

Solution

New Test Coverage

Added two new test cases following the existing codebase patterns:

TestDynamoDB_TransactWriteItems_Concurrent

• Spawns 100 concurrent goroutines
• Each performs independent transactions with unique keys
• Verifies that all transactions complete successfully without interference
• Tests basic concurrent operation handling

TestDynamoDB_TransactWriteItems_Concurrent_Conflicting

• Spawns 50 concurrent goroutines competing for shared resources
• Each transaction updates the same set of keys atomically
• Verifies transaction isolation and data consistency
• Ensures that despite concurrent access, each key maintains consistent state

AWS SDK v2 Migration

Migrated all DynamoDB test code from AWS SDK v1 to v2 to resolve deprecation warnings:

• Updated imports from github.com/aws/aws-sdk-go/* to github.com/aws/aws-sdk-go-v2/*
• Replaced deprecated session.NewSession with config.LoadDefaultConfig
• Updated endpoint configuration to use service-specific BaseEndpoint instead of global resolver
• Updated attribute value types to use the new types.AttributeValue interface
• Updated method calls to remove WithContext suffix (modern SDK v2 pattern)
• Replaced pointer-based access patterns with type assertions for attribute values

golangci-lint Fixes

Resolved all linting issues to ensure CI passes:

• Updated golangci-lint to v2.4.0 for Go 1.25.0 compatibility
• Fixed type assertion safety - Replaced unsafe type assertions with checked assertions using the ok pattern
• Applied formatting fixes - Resolved gci linter formatting issues
• Verified compatibility - All linters now pass with 0 issues

Testing

Both new tests and all existing tests continue to pass, confirming no regressions were introduced. The migration maintains full functional compatibility while using the modern, supported AWS SDK.

Impact

These tests provide confidence that the DynamoDB adapter's transaction handling is robust under concurrent load, which is critical for production deployment in distributed environments. The SDK migration ensures the codebase uses supported, non-deprecated dependencies, while the golangci-lint fixes ensure the CI pipeline runs successfully.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[github-actions]

🚫 [golangci] _{reported by reviewdog 🐶}
SA1019: "github.com/aws/aws-sdk-go/aws/credentials" is deprecated: aws-sdk-go is deprecated. Use aws-sdk-go-v2. See https://aws.amazon.com/blogs/developer/announcing-end-of-support-for-aws-sdk-for-go-v1-on-july-31-2025/. (staticcheck)

elastickv/adapter/dynamodb_test.go

Line 10 in 6905c19

"github.com/aws/aws-sdk-go/aws/credentials"

[bootjp]

@copilot 対応してください

[bootjp]

@copilot fix
#205 (comment)

[bootjp]

@copilot
🚫 [golangci] reported by reviewdog 🐶
SA1019: "github.com/aws/aws-sdk-go/aws/credentials" is deprecated: aws-sdk-go is deprecated. Use aws-sdk-go-v2. See https://aws.amazon.com/blogs/developer/announcing-end-of-support-for-aws-sdk-for-go-v1-on-july-31-2025/. (staticcheck)

[Copilot]

@copilot
🚫 [golangci] reported by reviewdog 🐶
SA1019: "github.com/aws/aws-sdk-go/aws/credentials" is deprecated: aws-sdk-go is deprecated. Use aws-sd...

Fixed the SA1019 deprecation warning by migrating from AWS SDK v1 to v2. Updated all imports and usage patterns to use the modern AWS SDK v2 API. The staticcheck warning is now resolved. Commit: da8aa3a

[bootjp]

@copilot golangci-lintをパスさせてください

[Copilot]

@copilot golangci-lintをパスさせてください

golangci-lint is now passing successfully with 0 issues. Fixed the type assertion checks and updated to compatible golangci-lint version for Go 1.25.0. Commit: db94f9d

[bootjp]

LGTM