Bricks: split PackageResolver.MaterializeTwinAsync
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Open Gaya/MASS4.Bricks/PackageResolver.Twins.cs and read MaterializeTwinAsync and its callees. Apply the pending cached-bake fix first, then introduce a private value holding the parameters and move the index/cached/published path into a separate async step. 'Done' is the full suite green with no test changes and each changed method at complexity ≤8 and CRAP ≤15.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Current State
PackageResolver.MaterializeTwinAsync (Gaya/MASS4.Bricks/PackageResolver.Twins.cs) scored CRAP 32 in the store contract client batch (96a5587): one async body reads the lock, the package store, the registry index and the variant choice. The pending cached-bake fix splits out CachedTwin, CheckedTwinAsync and AvailableTwinAsync, which brings it to complexity 14 at the 15 limit, with AvailableTwinAsync taking eleven parameters. Branches from static-lambda caching, the strict-lock exit, the index fetch and the cached path still add up in one state machine.
Tasks
- Carry the source id, ranges, requester, depth, existing lock and locked bake in one private value instead of separate parameters
- Keep the strict-lock exit in a synchronous
MaterializeTwinAsyncand move the index/cached/published path into its own async step - Pass that value to
AvailableTwinAsyncinstead of its parameter list
Acceptance Criteria
- Behavior unchanged: no test edited, full suite green
- Every changed method has complexity ≤ 8 and CRAP ≤ 15
Related
- Lands after the cached/locked bake compatibility and advisory fix
- Lenguaje dominante
- C#
- Estrellas
- 1
- Forks
- 1
- Merge medio
- 1 h 46 min
- PR fusionados (30 d)
- 41
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la 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 MASS4ORG/Turian
-
c:assets p:low size:XS t:task
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
c:engine p:medium size:S t:enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Los mantenedores suelen responder en 1 día
-
c:gaya p:high size:XS t:bug
Dificultad 3/5 1-2 días Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
c:build c:editor p:medium size:XL t:feature
Dificultad 5/5 Más de una semana Aptitud para principiantes 5/100
Los mantenedores suelen responder en 1 día
-
c:build p:medium size:XL t:refactor
Dificultad 5/5 Más de una semana Aptitud para principiantes 8/100
Los mantenedores suelen responder en 1 día
Todos los issues de MASS4ORG/Turian
Issues similares
-
Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
dotnet/roslynator#1859 ·
Los mantenedores suelen responder en 1 día
-
area:jobads-cv BE P3
Dificultad 1/5 1-3 horas Aptitud para principiantes 74/100
klasolsson81/jobbliggaren#2240 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
k2-fsa/sherpa-onnx#4100 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
:watch: Not Triaged
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día