Skip to content

feat(stovepipe): expose retained history event metadata - #954

Merged
mnoah1 merged 4 commits into
mainfrom
mnoah1/stovepipe-history-metadata
Oct 30, 2026
Merged

mnoah1 merged 4 commits into
mainfrom
mnoah1/stovepipe-history-metadata

Conversation

@mnoah1

@mnoah1 mnoah1 commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Why?

Expose context already retained with Stovepipe history occurrences.

What?

  • Add typed event payloads for build IDs on request-state, build-triggered, build-finished, and build-analysis-abandoned occurrences; validation degree on validation-fact-recorded; and project-fact count on project-facts-recorded.
  • Omit payloads when metadata is unavailable or malformed, preserving valid zero values. Regenerate Go and TypeScript bindings.
  • Read existing metadata only; no SQL changes or new event writers. Independent of the history-milestones PR.

Test Plan

Passed mapper, handler, and server tests; TypeScript generation and host type checks; and formatting, lint, tidy, and Gazelle checks.

Revert Plan

Revert this PR to remove typed history event metadata from the API.

Issue Links

None — no issue is linked to this history API enhancement.

@mnoah1
mnoah1 marked this pull request as ready for review October 27, 2026 17:31
@mnoah1
mnoah1 requested review from a team, behinddwalls and sbalabanov as code owners October 27, 2026 17:31
@mnoah1
mnoah1 enabled auto-merge October 30, 2026 18:51
@mnoah1
mnoah1 added this pull request to the merge queue Oct 30, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 30, 2026
@mnoah1
mnoah1 enabled auto-merge October 30, 2026 19:41
@mnoah1
mnoah1 added this pull request to the merge queue Oct 30, 2026
Merged via the queue into main with commit 57b2852 Oct 30, 2026
21 checks passed
@mnoah1
mnoah1 deleted the mnoah1/stovepipe-history-metadata branch October 30, 2026 19:53
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.

2 participants