Skip to content

feat(stovepipe): List open issues in pages - #924

Merged
prathshenoy merged 1 commit into
mainfrom
codex/stovepipe-open-issue-list
Oct 23, 2026
Merged

prathshenoy merged 1 commit into
mainfrom
codex/stovepipe-open-issue-list

Conversation

@prathshenoy

@prathshenoy prathshenoy commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Summary

What:

  • List currently open issues in bounded, cursor-based pages within each validation queue, preserving byte-exact task and unit ordering.
  • Reject open issues without a task identity while preserving task-wide issues with no unit.

Why:

  • Enable issue reconciliation to discover existing open issues through primary-key reads without secondary indexes or loading every issue at once.

Test Plan

  • Add unit tests.
  • Verify page boundaries, queue isolation, identifier limits, and continuation after deletion against MySQL.

Revert Plan

  • Revert this PR. It changes storage reads and input validation without a schema migration or enabling runtime writers.

Issue Links

Stack

  1. feat(stovepipe): Add issue lifecycle history #923
  2. feat(stovepipe): List open issues in pages #924
  3. feat(stovepipe): Define issue-tracking handoff #925

@prathshenoy
prathshenoy force-pushed the codex/stovepipe-open-issue-list branch 2 times, most recently from f99ace7 to 660c3e1 Compare October 23, 2026 04:00
@prathshenoy
prathshenoy added this pull request to stack #927 October 23, 2026 14:28
@prathshenoy
prathshenoy marked this pull request as ready for review October 23, 2026 14:28
@prathshenoy
prathshenoy force-pushed the codex/stovepipe-open-issue-list branch from 660c3e1 to 1da0bcc Compare October 23, 2026 16:32
Base automatically changed from codex/stovepipe-issue-history-storage to main October 23, 2026 16:49
@prathshenoy
prathshenoy force-pushed the codex/stovepipe-open-issue-list branch from 1da0bcc to b189d84 Compare October 23, 2026 16:50

@mnoah1 mnoah1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's use a similar materializer pattern to the request/queue summaries

Comment thread stovepipe/extension/storage/README.md
@prathshenoy
prathshenoy added this pull request to the merge queue Oct 23, 2026
Merged via the queue into main with commit f75843b Oct 23, 2026
33 of 36 checks passed
@prathshenoy
prathshenoy deleted the codex/stovepipe-open-issue-list branch October 23, 2026 20:16
DivyanshGarg380 pushed a commit to DivyanshGarg380/submitqueue that referenced this pull request Oct 24, 2026
## Summary

**What**:

- Persist immutable opening and resolution history, retaining the
associated build and timestamp after an issue resolves or reopens.

**Why**:

- Preserve issue lifecycle history separately from current open issues
and per-build observations, without enabling reconciliation or repair
tracking yet.

## Test Plan

- [x] Add unit tests.
- [x] Verify history retention, duplicate replay, concurrent creation,
and identifier boundaries against MySQL.

## Revert Plan

- Revert this PR. No runtime writer is enabled; the additive table can
remain unused without affecting validation.

## Issue Links

- [CODEM-607](https://linear.app/uber/issue/CODEM-607/build-analysis)

## Stack

1. uber#923
1. uber#924
1. uber#925
DivyanshGarg380 pushed a commit to DivyanshGarg380/submitqueue that referenced this pull request Oct 24, 2026
## Summary

**What**:

- Define queue-scoped notifications for new failure observations and
successful final analysis, without enabling publishers or consumers.
- Expose persisted failures for one build in bounded, cursor-based pages
with required build and task identities.

**Why**:

- Prepare ordered issue reconciliation while keeping build polling
independent and messages small, without adding tables or serialized
failure collections.

## Test Plan

- [x] Add unit tests.
- [x] Verify pagination, queue/build isolation, byte-exact ordering, and
identifier boundaries against MySQL.

## Revert Plan

- Revert this PR. The contracts and reads are not wired into runtime
workflows; no schema migration or data removal is required.

## Issue Links

- [CODEM-607](https://linear.app/uber/issue/CODEM-607/build-analysis)

## Stack

1. uber#923
1. uber#924
1. uber#925
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