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

feat(stovepipe): List open issues in pages#924
prathshenoy merged 1 commit into
mainfrom
codex/stovepipe-open-issue-list

Conversation

@prathshenoy

@prathshenoy prathshenoy commented Oct 23, 2026 •
edited
Loading

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
This was referenced Oct 23, 2026
Merged
Merged
@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 reviewed Oct 23, 2026
View reviewed changes

@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
@mnoah1 mnoah1 mentioned this pull request Oct 23, 2026
Merged
@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
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