123123213weqw opened a new pull request, #5850: URL: https://github.com/apache/rocketmq-dashboard/pull/5850
## Why `TestMessageQueryCompletenessWarningTest` (yaml cases) is red on current `rocketmq-studio`: the yaml round-trip turns every number into a quoted string (`skippedCount: "5"`, `size: "3"`). Root cause: `internal/studio/client.go` decodes results with `decoder.UseNumber()` so int64 offsets/timestamps beyond 2^53 survive the JSON round-trip — but `json.Number` is a string type and yaml.v3 has no special case for it, so `output.YAML` quotes every number it prints. ## What changed - `output.YAML` now walks the value tree and converts `json.Number` leaves to `int64` when parseable (large offsets keep their exact digits), else `float64`, else the raw string. ## Validation - Red-first: the pre-existing failing yaml subtests of `cmd/catalog_renderers_test.go` now pass without touching the test. - New `internal/output/yaml_numbers_test.go`: numbers render unquoted, an offset of 2^53+1 round-trips with exact digits, floats stay floats. - `gofmt`, `go vet`, `go build ./...` and the full `go test ./...` suite (7 packages) pass. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
