Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

abctl cost --by currency: a withheld (mixed) cell is unexplained on that axis

Abierto Apto para principiantes
  • ¿Algo no está bien?
#1,177 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
90/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
go
Área
cli, testing-qa

Línea de trabajo

Comienza en cmd/abctl/cmd_cost.go alrededor de las líneas 1321-1324 e inspecciona TestRunCost_ByCurrencyWithholdsACappedRowOnAMixedWindow para ver el fixture existente. Ejecuta esa prueba para el caso de ventana mixta, fila limitada y --by currency; estará terminado cuando la celda retenida vaya acompañada de la explicación “these rows hold” y la prueba cubra este comportamiento.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Found by the round-3 strict review of #1153. Introduced by that PR's own review round 2 (26a1ad75), so it ships with it.

Before 26a1ad75 no cell on the currency axis could be (mixed): the byUnit arm took the row label unconditionally. It now gates on isReportedUnit, so a capped row (one minute past maxLabelsPerMinute, which carries (other) on every axis) in a mixed window falls through to default and renders (mixed). But the explanatory caveat is gated if !byUnit && !labelled — cmd/abctl/cmd_cost.go:1324 — so on --by currency it is suppressed:

  CURRENCY                             REQUESTS     TOKENS           COST
  USD                                      1000          0        $146.36
  (other)                                    57          0        (mixed)
  credits                                    57          0   0.08 credits

these rows hold is absent. The reader followed the advice in the summary above the table and one row is still withheld with nothing here saying why. Two rules from the PR's own prose:

  • cmd_cost.go:1321-1323, three lines up: "A column of (mixed) with no explanation reads as a defect in the tool."
  • mixedCostCell's godoc: "the first wants a pricing entry, the second wants --by currency" — prescribing what the reader already did.

The candidate fix is unguarded. Mutant HOLE_bycurrency_withheld_cell_unexplained applies it (gate on !labelled alone) and SURVIVES — no test outcome changes. TestRunCost_ByCurrencyWithholdsACappedRowOnAMixedWindow already carries the fixture; extend it rather than adding a fourth --by currency test.

Reachable set: a genuinely mixed window and a minute past maxLabelsPerMinute and --by currency.

Assisted-By: Claude (Anthropic AI) [email protected]

Lenguaje dominante
Go
Estrellas
13
Forks
40
Merge medio
12 h 3 min
PR fusionados (30 d)
199

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de rossoctl/cortex

Todos los issues de rossoctl/cortex

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.