Repository navigation
feat(stovepipe): Implement project processing - #1058
Open
prathshenoy wants to merge 1 commit into
Open
prathshenoy wants to merge 1 commit into
prathshenoy wants to merge 1 commit into
Conversation
1 task done
This was referenced Nov 9, 2026
prathshenoy
force-pushed
the
prath.shenoy/refactor/stovepipe-project-results
branch
2 times, most recently
from
November 10, 2026 16:27
3886604 to
2daca0c
Compare
1 task done
prathshenoy
changed the title
prathshenoy
changed the base branch from
prath.shenoy/refactor/stovepipe-validation-facts
to
prath.shenoy/feat/stovepipe-project-analysis
November 10, 2026 16:43
prathshenoy
force-pushed
the
prath.shenoy/refactor/stovepipe-project-results
branch
from
November 10, 2026 16:57
2daca0c to
95049d4
Compare
prathshenoy
force-pushed
the
prath.shenoy/refactor/stovepipe-project-results
branch
from
November 10, 2026 17:04
95049d4 to
152fc51
Compare
prathshenoy
force-pushed
the
prath.shenoy/refactor/stovepipe-project-results
branch
2 times, most recently
from
November 10, 2026 17:13
258c237 to
b2d9c1e
Compare
prathshenoy
force-pushed
the
prath.shenoy/refactor/stovepipe-project-results
branch
7 times, most recently
from
November 10, 2026 21:45
90a6a3b to
3a36b76
Compare
**What**: - Add project-status recording and issue attribution to the project-analysis stage without activating it. - Retain project facts and associations before handing completed analysis to repository recording. **Why**: - Give project analysis ownership of the project-processing implementation so status and attribution can finish together after terminal build analysis. - [x] Add unit tests. - Revert this PR. The controller remains unwired and repository recording still owns the running pipeline. - [CODEM-632](https://linear.app/uber/issue/CODEM-632/projects-issues)
prathshenoy
force-pushed
the
prath.shenoy/refactor/stovepipe-project-results
branch
from
November 10, 2026 21:52
3a36b76 to
1315d01
Compare
prathshenoy
marked this pull request as ready for review
November 10, 2026 22:03
prathshenoy
requested review from
a team,
behinddwalls and
sbalabanov
as code owners
November 10, 2026 22:03
prathshenoy
added a commit
that referenced
this pull request
Nov 10, 2026
## Summary **What**: - Run project analysis after terminal build analysis and before repository recording. - Mark project analysis finished after project statuses and issue attribution are recorded, or processing is abandoned. **Why**: - Ensure attribution uses final build issues and repository recording receives the project results needed to distinguish partial failures from repository-wide failures. - Keep request summaries from reporting completion while project analysis is still running, and repair incomplete summary updates before handing off to repository recording. ## Test Plan - [x] Add unit tests. ## Revert Plan - Revert this PR after pausing intake and draining or reconciling pending project-analysis work, because the revert removes its consumers. ## Issue Links - [CODEM-632](https://linear.app/uber/issue/CODEM-632/projects-issues) ## Stack 1. #1057 1. #1066 1. #1058 1. #1062 1. #1065
prathshenoy
added a commit
that referenced
this pull request
Nov 10, 2026
## Summary **What**: - Run project analysis after terminal build analysis and before repository recording. - Mark project analysis finished after project statuses and issue attribution are recorded, or processing is abandoned. **Why**: - Ensure attribution uses final build issues and repository recording receives the project results needed to distinguish partial failures from repository-wide failures. - Keep request summaries from reporting completion while project analysis is still running, and repair incomplete summary updates before handing off to repository recording. ## Test Plan - [x] Add unit tests. ## Revert Plan - Revert this PR after pausing intake and draining or reconciling pending project-analysis work, because the revert removes its consumers. ## Issue Links - [CODEM-632](https://linear.app/uber/issue/CODEM-632/projects-issues) ## Stack 1. #1057 1. #1066 1. #1058 1. #1062 1. #1065
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
What:
Why:
Test Plan
Revert Plan
Issue Links
Stack