[Refactor][Kiro] Migrate the kiro plugin from aws-sdk-go v1 to aws-sdk-go-v2
Los mantenedores suelen responder en 2 días
@ay5399 ya está trabajando en esto.
Desde el 14/10/2026.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 38/100
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Go
- Estrellas
- 3.2k
- Forks
- 824
- Merge medio
- 2 d 22 h
- PR fusionados (30 d)
- 43
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de apache/devlake
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
Los mantenedores suelen responder en 2 días
-
[Bug][Project] Invalid metric plugin options are accepted on project save and only fail at the next pipeline runPosiblemente ocupada @chenwei791129 la tomó hace 3 días. Abierto
Dificultad 3/5 Medio día Aptitud para principiantes 22/100
Los mantenedores suelen responder en 2 días
-
[Bug][Dashboard] Engineering Throughput PR time panels count PRs without a project_pr_metrics row as 0 hoursPosiblemente ocupada @chenwei791129 la tomó hace 3 días. Abierto
Dificultad 3/5 Medio día Aptitud para principiantes 22/100
Los mantenedores suelen responder en 2 días
-
[Bug][e2e] Several plugin e2e tests fail on a fresh database because they depend on tables created by other testsPosiblemente ocupada @chenwei791129 la tomó hace 3 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Los mantenedores suelen responder en 2 días
-
[Feature][dora] Allow filtering which pull requests count toward Lead Time for Changes by branch namePosiblemente ocupada @chenwei791129 la tomó hace 3 días. Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
Los mantenedores suelen responder en 2 días
Todos los issues de apache/devlake
Issues similares
-
goose sub-agent runs are never indexed: blame finds no session for any file a `delegate` child wroteAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
wakatime/wakatime-cli#1649 ·
Los mantenedores suelen responder en 1 día
-
PathBuffer.Pop corrupts later error locations when a key contains "." or "["Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
danielgtaylor/huma#1135 ·
Los mantenedores suelen responder en 1 día
-
needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
stacklok/toolhive-catalog#1734 ·
Los mantenedores suelen responder en 1 día
-
Docs: how to get more careful answers (Effort, reasoning models, Reasoning level, Deliberate)Abiertodocumentation priority:p1
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
yeixio/toskar-core#697 ·
Los mantenedores suelen responder en 1 día