Skip to content

fix: describe the whole run in the trace header - #133

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-11450-make-the-trace-header-describe-the-whole-run
Oct 27, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-11450-make-the-trace-header-describe-the-whole-run

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

The trace header shows a local start time. Cost is an em dash until a span reports one, and a reported zero still shows $0. A caller can pass the whole-run step count when the loaded page is shorter than the run.

Verified with bun run check and bun run build.

Closes CL-11450

A paged caller can pass the run's step total. Cost stays blank until
a span reports one. The start time is a local date and time.
@linear-code

linear-code Bot commented Oct 25, 2026

Copy link
Copy Markdown

CL-11450

@TheGreatAxios
TheGreatAxios marked this pull request as ready for review October 25, 2026 03:40
@corbits-triage

corbits-triage Bot commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Waiting on a maintainer to approve: approve.

@TheGreatAxios

Copy link
Copy Markdown
Contributor Author

Storybook · before and after

Polish on ff6d038: a parseable start time is a local date and a local time joined with a middle dot, so the mono line does not pick up the word “at”. A clock string such as 09:14:02 is unchanged.

Gist: https://gist.github.com/TheGreatAxios/9fce55ceb929bd902ecdeff9fedf744f

Before

Observability / Views / App, after opening “Review a pull request”. repo.review · 09:14:02, cost $0.11, 17 steps.

Before

After

The same app story is identical, because that fixture is a clock time.

After, app

Observability / Views / Run / Parseable start, ISO 2026-10-24T15:04:00.000Z: repo.review · Oct 24 · 8:04 AM (hour follows the machine time zone). No word “at”.

After, parseable start

The step list and the timing breakdown do not mount the header. Those shots matched before and after.

@TheGreatAxios
TheGreatAxios merged commit e4a1616 into main Oct 27, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant