Skip to content

Expose CI log step metadata#501

Merged
121watts merged 1 commit intomainfrom
codex/dep-4399-step-log-metadata-cli-builder
May 6, 2026
Merged

Expose CI log step metadata#501
121watts merged 1 commit intomainfrom
codex/dep-4399-step-log-metadata-cli-builder

Conversation

@121watts
Copy link
Copy Markdown
Contributor

@121watts 121watts commented May 6, 2026

Summary

  • regenerate the CI API proto with step_key, optional step_id, and optional step_name on log lines
  • keep streaming de-dupe keyed on the stable Depot step_key
  • include step_key, step_id, and step_name in JSON and NDJSON log output while leaving plain log output unchanged. this is particularly useful for visualizing and parsing parallel steps.

Verification

  • make generate
  • gofmt -w pkg/api/ci.go pkg/api/ci_test.go pkg/cmd/ci/logs.go pkg/cmd/ci/logs_test.go
  • go test ./pkg/api ./pkg/cmd/ci
  • go test ./...
  • git diff --check

Depends on: https://github.com/depot/api/pull/3637
Linear: DEP-4399


Note

Medium Risk
Changes the LogLine protobuf schema and JSON log output fields (renaming the primary correlation field and adding new metadata), which may impact downstream consumers expecting the old step_id semantics.

Overview
Updates CI log lines to carry a stable Depot correlation key (step_key) plus optional user-authored step_id/step_name in the v1 protobuf (ci.proto regen).

Adjusts log streaming de-dupe to key on step_key instead of step_id, and extends depot ci logs --output json/NDJSON events to include step_key, step_id, and step_name while leaving plain text output unchanged (tests updated accordingly).

Reviewed by Cursor Bugbot for commit 3f084b7. Bugbot is set up for automated code reviews on this repo. Configure here.

@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 6, 2026

@121watts 121watts merged commit 65c7626 into main May 6, 2026
12 checks passed
@121watts 121watts deleted the codex/dep-4399-step-log-metadata-cli-builder branch May 6, 2026 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants