[Refactor][Kiro] Migrate the kiro plugin from aws-sdk-go v1 to aws-sdk-go-v2
Les mainteneurs répondent en général sous 3 jours
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 38/100
Piste de recherche
Start with the listed files under backend/plugins/kiro: api/test_connection.go, tasks/identity_client.go, tasks/s3_client.go, and tasks/s3_file_collector.go; read their current SDK usage and the named test doubles in the issue. Run the plugin unit tests, then migrate the clients and tests to aws-sdk-go-v2. Done means those tests pass, the v1 module is removed from backend/go.mod, and the temporary exclusion is removed from backend/.golangci.yaml; a real AWS/Kiro run is also requested.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
What and why to refactor
backend/plugins/kiro is the only consumer of github.com/aws/aws-sdk-go
(v1). The AWS SDK for Go v1 reached
end-of-support on 2025-07-31;
it no longer receives bug or security fixes. Its final release, v1.55.8, marks
every package as deprecated, so staticcheck reports SA1019 for each import:
plugins/kiro/api/test_connection.go:24-25 aws, service/s3
plugins/kiro/tasks/identity_client.go:21-24 aws, aws/credentials, aws/session, service/identitystore
plugins/kiro/tasks/s3_client.go:25-28 aws, aws/credentials, aws/session, service/s3
plugins/kiro/tasks/s3_file_collector.go:24-25 aws, service/s3
This is what turned the lint job red on the Dependabot group #9192. It is
tolerated for now by a narrowly scoped lint exclusion (see the linked PR), but
that only hides the debt.
Describe the solution you'd like
Move the plugin to aws-sdk-go-v2:
aws/session+aws/credentials→config.LoadDefaultConfig(ctx, config.WithRegion(…), config.WithCredentialsProvider(credentials.NewStaticCredentialsProvider(…)))service/s3→github.com/aws/aws-sdk-go-v2/service/s3(ListObjectsV2,
GetObject, …; request/response types move tos3/types, calls take a
context.Context)service/identitystore→github.com/aws/aws-sdk-go-v2/service/identitystore- adapt the test doubles in
tasks/s3_client_test.go,
tasks/s3_file_collector_test.goandtasks/discovery_test.goto the v2
client interfaces - afterwards: drop
github.com/aws/aws-sdk-gofrombackend/go.modand remove
the temporary exclusion frombackend/.golangci.yaml
The new modules are Apache-2.0 licensed. Dependabot's existing
go-minor-patch group (patterns: ["*"]) already bundles the many v2
sub-modules, so no extra configuration is needed.
@warren830 — you authored the kiro plugin (#9073, #9076, #9082). Would you be
willing to take this on, or review a migration PR? The plugin talks to real
S3 buckets and IAM Identity Center, which I can't test without a Kiro/AWS
setup, so a run against a real environment by someone who has one would be
essential.
Related issues
- #9192 (Dependabot
go-minor-patchgroup, red because ofSA1019) - PR with the temporary lint exclusion: #9197
- Tracking issue for removing aws-sdk-go v1 from the build: #9196
Additional context
The plugin's unit tests (api, impl, models, tasks) pass with
v1.55.8 from #9192, so this is not urgent breakage — it is about not staying
on an unsupported SDK.
- Langage dominant
- Go
- Étoiles
- 3.2k
- Forks
- 830
- Merge moyen
- 2 j 22 h
- PR mergées (30 j)
- 49
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Aucun guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de apache/devlake
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
apache/devlake#9246 · 1 réaction ·
Les mainteneurs répondent en général sous 3 jours
-
[Bug][Blueprint] Cron-triggered pipelines use a sanitized plan after PATCH /blueprints, gitextractor fails with masked credentialsPeut-être pris @Thanatos9404 l’a pris il y a 4 jours. Ouverte
Difficulté 4/5 1-2 jours Accessibilité débutants 56/100
Les mainteneurs répondent en général sous 3 jours
-
Difficulté 3/5 Une demi-journée Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 3 jours
-
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
Les mainteneurs répondent en général sous 3 jours
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
Les mainteneurs répondent en général sous 3 jours
Toutes les issues de apache/devlake
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
eugenioenko/ttt#1206 ·
Les mainteneurs répondent en général sous 1 jour
-
t:urlDecodeUni does not map U+3002 and U+FF61 to a dot, unlike ModSecurityPeut-être pris @fzipi l’a pris aujourd’hui. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
peasant-labs/peasant#738 ·
Les mainteneurs répondent en général sous 1 jour
-
Duplicate review still bridges two books through a bundle whose first part has a colon subtitlePeut-être pris @vavallee l’a pris aujourd’hui. Ouvertearea:library-scan bug priority:low
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour