Skip to content

feat(stovepipe): Define project analysis stage - #1066

Open
prathshenoy wants to merge 1 commit into
prath.shenoy/refactor/stovepipe-validation-factsfrom
prath.shenoy/feat/stovepipe-project-analysis
Open

prathshenoy wants to merge 1 commit into
prath.shenoy/refactor/stovepipe-validation-factsfrom
prath.shenoy/feat/stovepipe-project-analysis

Conversation

@prathshenoy

@prathshenoy prathshenoy commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Summary

What:

  • Define the terminal project-analysis message and controller dependencies without activating the stage.

Why:

  • Establish a request-scoped boundary for project processing after terminal build analysis completes.

Test Plan

  • Add unit tests.

Revert Plan

  • Revert this PR. The stage is not registered and the existing pipeline is unchanged.

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

## Summary
**What**:
- Define the terminal project-analysis message and controller dependencies without activating the stage.

**Why**:
- Establish a request-scoped stage for project processing before moving the existing implementation or changing the running pipeline.

## Test Plan
- [x] Add unit tests.

## Revert Plan
- Revert this PR. The stage is not registered and the existing pipeline is unchanged.

## Issue Links
- [CODEM-632](https://linear.app/uber/issue/CODEM-632/projects-issues)
@prathshenoy
prathshenoy force-pushed the prath.shenoy/feat/stovepipe-project-analysis branch from 6857225 to c3eee68 Compare November 10, 2026 21:52
@prathshenoy
prathshenoy removed this pull request from stack #1067 November 10, 2026 23:23
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
prathshenoy added this pull request to stack #1071 November 10, 2026 23:24
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

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