Output when using `query` with `-op dump -strip-ansi -format json` doesn’t strip ANSI

[plasticine]

Hey there! 👋 This CLI seems awesome — very exciting!

I have encountered what I think is a bug when using query ... -op dump -strip-ansi -format json.

Anyway, here is a snippet of output;

{"row_number":474,"timestamp":1763676757181,"content":"\u001b[90m# BUILDKIT_PROGRESS added\u001b[0m","group":"--- :docker: Building services with bake root_bundled\r","flags":1}
{"row_number":475,"timestamp":1763676757181,"content":"\u001b[90m# DOCKER_BUILDKIT added\u001b[0m","group":"--- :docker: Building services with bake root_bundled\r","flags":1}
{"row_number":476,"timestamp":1763676757181,"content":"~~~ Running agent post-command hook","group":"~~~ Running agent post-command hook","flags":3}
{"row_number":477,"timestamp":1763676757181,"content":"\u001b[90m$\u001b[0m /etc/buildkite-agent/hooks/post-command","group":"~~~ Running agent post-command hook","flags":1}
{"row_number":478,"timestamp":1763676758126,"content":"\u001b[90m# PATH changed\u001b[0m","group":"~~~ Running agent post-command hook","flags":1}
{"row_number":479,"timestamp":1763676758127,"content":"~~~ Running agent pre-exit hook","group":"~~~ Running agent pre-exit hook","flags":3}
{"row_number":480,"timestamp":1763676758127,"content":"\u001b[90m$\u001b[0m /etc/buildkite-agent/hooks/pre-exit","group":"~~~ Running agent pre-exit hook","flags":1}
{"row_number":481,"timestamp":1763676758811,"content":"\u001b[90m# PATH changed\u001b[0m","group":"~~~ Running agent pre-exit hook","flags":1}
{"row_number":482,"timestamp":1763676758811,"content":"~~~ Running plugin docker pre-exit hook","group":"~~~ Running plugin docker pre-exit hook","flags":3}
{"row_number":483,"timestamp":1763676758811,"content":"\u001b[90m$\u001b[0m /var/lib/buildkite-agent/plugins/snip/hooks/pre-exit","group":"~~~ Running plugin docker pre-exit hook","flags":1}

I think there are a few issues here;

• Most significantly I would have expected content to have had ANSI escapes stripped given -strip-ansi
• Less of an issue but it would be kinda amazing if the timestamp were a formatted as ISO8601

Cheers, keen to hear your thoughts. 🤙

[wolfeidau]

@plasticine great questions.

1. Yes, the JSON formatting doesn't currently strip ANSI escape codes.
2. The timestamp is format is def not ideal for humans.

I have mostly been using this CLI to debug and shape the integration into https://github.com/buildkite/buildkite-mcp-server. I do this from time to time to understand issues in job logs.

I wanted to see if people found it useful before PR'ing it into the bk/cli.

What are you using it for specifically?

[plasticine]

@wolfeidau rough plan is to build some tooling that can pull a whole bunch of disparate log sources together with the build log to provide a single place to correlate test execution with stuff happening in supporting systems on a common timeline.

Mostly just bashing rocks together at this stage.