Skip to content

feat(stovepipe): Wire the project analysis stage - #1065

Draft
prathshenoy wants to merge 1 commit into
prath.shenoy/feat/stovepipe-project-analysis-dlqfrom
prath.shenoy/feat/stovepipe-project-analysis-wiring
Draft

feat(stovepipe): Wire the project analysis stage#1065
prathshenoy wants to merge 1 commit into
prath.shenoy/feat/stovepipe-project-analysis-dlqfrom
prath.shenoy/feat/stovepipe-project-analysis-wiring

Conversation

@prathshenoy

@prathshenoy prathshenoy commented Nov 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

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

  • 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

Stack

  1. refactor(stovepipe): Extract fact persistence #1057
  2. feat(stovepipe): Define project analysis stage #1066
  3. feat(stovepipe): Implement project processing #1058
  4. feat(stovepipe): Recover project analysis failures #1062
  5. feat(stovepipe): Wire the project analysis stage #1065

@prathshenoy
prathshenoy added this pull request to stack #1059 November 9, 2026 22:25
This was referenced Nov 9, 2026
Open
Open
Merged
Open
Closed
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from af15e62 to 7ea0407 Compare November 10, 2026 15:42
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from 7ea0407 to ef261e4 Compare November 10, 2026 16:27
@prathshenoy
prathshenoy removed this pull request from stack #1059 November 10, 2026 16:28
@prathshenoy prathshenoy mentioned this pull request Nov 10, 2026
Open
1 task done
@prathshenoy
prathshenoy added this pull request to stack #1067 November 10, 2026 16:44
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch 2 times, most recently from 056eb27 to e9309c3 Compare November 10, 2026 17:04
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from e9309c3 to 485d3b1 Compare November 10, 2026 17:08
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from 485d3b1 to 73696e4 Compare November 10, 2026 17:13
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from 73696e4 to cfe1551 Compare November 10, 2026 17:55
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch 2 times, most recently from 070b536 to 406f777 Compare November 10, 2026 21:17
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from 406f777 to c57d38d Compare November 10, 2026 21:21
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from c57d38d to 3b850df Compare November 10, 2026 21:31
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from 3b850df to a96159e Compare November 10, 2026 21:42
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch 2 times, most recently from 9709f62 to ba03173 Compare November 10, 2026 21:52
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from ba03173 to 08dbfb2 Compare November 10, 2026 22:15
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch 2 times, most recently from 1556bc3 to eb2ea69 Compare November 10, 2026 23:14
@prathshenoy
prathshenoy removed this pull request from stack #1067 November 10, 2026 23:23
@prathshenoy
prathshenoy changed the base branch from prath.shenoy/feat/stovepipe-project-completion to prath.shenoy/feat/stovepipe-project-analysis-dlq November 10, 2026 23:23
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from eb2ea69 to 4504f5f Compare November 10, 2026 23:23
prathshenoy added a commit that referenced this pull request Nov 10, 2026
@prathshenoy
feat(stovepipe): Wire the project analysis stage
4504f5f 
## 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
prathshenoy added this pull request to stack #1071 November 10, 2026 23:24
@prathshenoy
feat(stovepipe): Wire the project analysis stage
65d8709 
## 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
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from 4504f5f to 65d8709 Compare November 10, 2026 23: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

Reviewers

@behinddwalls behinddwalls Awaiting requested review from behinddwalls behinddwalls will be requested when the pull request is marked ready for review behinddwalls is a code owner
@sbalabanov sbalabanov Awaiting requested review from sbalabanov sbalabanov will be requested when the pull request is marked ready for review sbalabanov is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@prathshenoy