Skip to content

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

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

feat(stovepipe): Wire the project analysis stage#1065
prathshenoy wants to merge 1 commit into
mainfrom
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
Merged
Merged
Merged
Merged
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
Merged
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 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
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from 4504f5f to 65d8709 Compare November 10, 2026 23:36
prathshenoy added a commit that referenced this pull request Nov 10, 2026
@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 marked this pull request as ready for review November 12, 2026 18:23
@prathshenoy
prathshenoy added this pull request to the merge queue Nov 12, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue because a pull request earlier in the stack was removed Nov 12, 2026
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from 65d8709 to 41fbce5 Compare November 12, 2026 18:51
prathshenoy added a commit that referenced this pull request Nov 12, 2026
@prathshenoy
feat(stovepipe): Wire the project analysis stage
41fbce5 
## 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 enabled stacked auto-merge November 12, 2026 18:52
@prathshenoy
prathshenoy disabled stacked auto-merge November 12, 2026 19:03
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis-wiring branch from 41fbce5 to ca69ccb Compare November 12, 2026 19:06
prathshenoy added a commit that referenced this pull request Nov 12, 2026
@prathshenoy
feat(stovepipe): Wire the project analysis stage
ca69ccb 
## 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 ca69ccb to 41fbce5 Compare November 12, 2026 19:08
Base automatically changed from prath.shenoy/feat/stovepipe-project-analysis-dlq to main November 12, 2026 19:11
@prathshenoy
feat(stovepipe): Wire the project analysis stage
736785e 
## 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 41fbce5 to 736785e Compare November 12, 2026 19:12
@prathshenoy
prathshenoy enabled stacked auto-merge November 12, 2026 19:29

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

@mnoah1 mnoah1 mnoah1 approved these changes
@behinddwalls behinddwalls Awaiting requested review from behinddwalls behinddwalls is a code owner
@sbalabanov sbalabanov Awaiting requested review from sbalabanov sbalabanov is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@prathshenoy @mnoah1