Skip to content

feat(stovepipe): Record resolved project names - #860

Closed
prathshenoy wants to merge 1 commit into
mainfrom
wt/projectstatus-list
Closed

prathshenoy wants to merge 1 commit into
mainfrom
wt/projectstatus-list

Conversation

@prathshenoy

@prathshenoy prathshenoy commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Summary

What:

  • Record a durable completion signal with the names produced by project-status resolution.
  • Preserve empty results so integrations can distinguish completion from missing data.

Why:

  • Let integrations act only after the resolved project-fact set is complete.
  • Avoid rerunning project-status resolution to discover recorded names.

Test Plan

  • Verify nonempty and empty resolution results record one completion signal with deterministic project-name ordering.
  • Verify malformed completion data is rejected and recording completes before integrations are notified.

Revert Plan

  • Revert this PR. The completion signal has no schema migration or irreversible data change.

Issues

@prathshenoy
prathshenoy force-pushed the wt/projectstatus-list branch from 055b0e5 to 1f0a747 Compare October 14, 2026 17:19
Summary:
Intent:
- Give asynchronous hooks a durable signal that project-status resolution has completed.
- Preserve the resolved project identities without requiring integrations to rerun resolution.

Changes:
- Record a project-facts completion occurrence for every terminal validation, including empty result sets.
- Store sorted project identities and their count in the completion occurrence metadata.
- Add helpers that construct and decode the completion occurrence.

Test Plan:
`go test ./stovepipe/...` covers nonempty and empty completion manifests, deterministic ordering, malformed metadata, and record-stage sequencing.

Revert Plan:
Revert this PR.

---

<sub>Generated by the 🪄 [pr-create](https://sg.uberinternal.com/code.uber.internal/uber-code/devexp-agent-marketplace/-/blob/claude-code/plugins/dev/uber-dev/skills/pr-create/SKILL.md) skill in devexp-agent-marketplace</sub>
@prathshenoy
prathshenoy force-pushed the wt/projectstatus-list branch from 1f0a747 to 533d67a Compare October 14, 2026 18:08
@prathshenoy prathshenoy changed the title feat(stovepipe): Record completed project fact sets feat(stovepipe): Record resolved project names Oct 14, 2026
@prathshenoy
prathshenoy marked this pull request as ready for review October 14, 2026 18:27
@prathshenoy
prathshenoy enabled auto-merge October 14, 2026 18:36
@prathshenoy
prathshenoy added this pull request to the merge queue Oct 14, 2026
@mnoah1
mnoah1 removed this pull request from the merge queue due to a manual request Oct 14, 2026
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