Skip to content

fix(stovepipe): complete project results when recording is abandoned - #989

Open
mnoah1 wants to merge 1 commit into
mainfrom
mnoah1/stovepipe-abandoned-results-complete
Open

mnoah1 wants to merge 1 commit into
mainfrom
mnoah1/stovepipe-abandoned-results-complete

Conversation

@mnoah1

@mnoah1 mnoah1 commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Why?

After record-stage work was abandoned, project_results_complete stayed false and the UI continued to report that project result recording was in progress.

What?

Project record_abandoned events into the existing completion timestamp. Preserve the first projected completion time and any recorded facts and project counts; keep project_facts_recorded distinct from abandonment. Cover event replay, concurrent projection updates, and empty or partial results through both status endpoints.

Existing abandoned requests require replaying their retained abandonment event after rollout to repair the summary.

Test Plan

Passed the request-log, record-controller, and RPC-controller unit tests and the Docker-based Stovepipe integration suite. Also passed make fmt, make lint, make check-tidy, and make check-gazelle.

## Why?

After record-stage work was abandoned, project_results_complete stayed false and the UI continued to report that project result recording was in progress.

## What?

Project record_abandoned events into the existing completion timestamp. Preserve the first projected completion time and any recorded facts and project counts; keep project_facts_recorded distinct from abandonment. Cover event replay, concurrent projection updates, and empty or partial results through both status endpoints.

Existing abandoned requests require replaying their retained abandonment event after rollout to repair the summary.

## Test Plan

Passed the request-log, record-controller, and RPC-controller unit tests and the Docker-based Stovepipe integration suite. Also passed make fmt, make lint, make check-tidy, and make check-gazelle.

## Issue

None.
@mnoah1
mnoah1 marked this pull request as ready for review October 29, 2026 14:36
@mnoah1
mnoah1 requested review from a team, behinddwalls and sbalabanov as code owners October 29, 2026 14:36

This branch has not been deployed

No deployments
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.

1 participant