Skip to content

[1/N Status & List APIs] docs: RFC for Status and List APIs#314

Open
albertywu wants to merge 1 commit into
mainfrom
wua/status-list-api-rfc
Open

[1/N Status & List APIs] docs: RFC for Status and List APIs#314
albertywu wants to merge 1 commit into
mainfrom
wua/status-list-api-rfc

Conversation

@albertywu

@albertywu albertywu commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Add a SubmitQueue RFC defining gateway Status and List API contracts and general design principles.

Test Plan

make fmt && make build && make test && make check-mocks && make e2e-test

Issues

Stack

  1. @ [1/N Status & List APIs] docs: RFC for Status and List APIs #314
  2. [2/N Status & List APIs] feat(storage): add request read-model schema #321
  3. [3/N Status & List APIs] feat(storage): add request read-model stores #322
  4. feat(gateway): persist request context on Land #327

Copilot AI review requested due to automatic review settings July 8, 2026 19:14
@albertywu albertywu marked this pull request as ready for review July 8, 2026 19:16
@albertywu albertywu requested review from a team, behinddwalls and sbalabanov as code owners July 8, 2026 19:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a new SubmitQueue RFC that proposes gateway Status and List API contracts and the supporting gateway-owned, write-time materialized read model, plus links it from the RFC index.

Changes:

  • Introduce RFC detailing Status (sqid or change URI) and List (queue + admission-time range) proto contracts and semantics.
  • Define a gateway-owned projection/write flow using query-shaped tables (request_summary, request_summary_by_queue, request_by_change_uri) and optimistic concurrency.
  • Add the new RFC to doc/rfc/index.md.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
doc/rfc/submitqueue/status-list-api.md New RFC specifying proposed API contracts and the gateway-owned read model + consistency semantics.
doc/rfc/index.md Adds the new RFC to the index (and touches the surrounding list).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread doc/rfc/index.md Outdated
Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
@albertywu albertywu changed the title docs: propose status and list APIs docs: RFC for Status and List APIs Jul 8, 2026
Copilot AI review requested due to automatic review settings July 8, 2026 20:35
@albertywu albertywu force-pushed the wua/status-list-api-rfc branch from c578263 to 017d801 Compare July 8, 2026 20:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
Copilot AI review requested due to automatic review settings July 8, 2026 21:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
Copilot AI review requested due to automatic review settings July 8, 2026 21:58
@albertywu albertywu force-pushed the wua/status-list-api-rfc branch from c14e380 to 12fd1a4 Compare July 8, 2026 21:58
@albertywu albertywu requested a review from behinddwalls July 8, 2026 21:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@albertywu albertywu force-pushed the wua/status-list-api-rfc branch from 12fd1a4 to da2056f Compare July 8, 2026 22:06
Copilot AI review requested due to automatic review settings July 8, 2026 22:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
Comment thread doc/rfc/submitqueue/status-list-api.md Outdated
Copilot AI review requested due to automatic review settings July 8, 2026 22:12
@albertywu albertywu force-pushed the wua/status-list-api-rfc branch from da2056f to 84e1d59 Compare July 8, 2026 22:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Summary:
Add a SubmitQueue RFC defining gateway Status and List API contracts, query-shaped materialized read models, and consistency semantics.

Test Plan:
git diff --check

Revert Plan:
Revert this commit.

API Changes:
Proposal only. Documents a breaking Status response shape and a new List RPC.

Monitoring and Alerts:
N/A
@albertywu albertywu force-pushed the wua/status-list-api-rfc branch from 84e1d59 to 593cda9 Compare July 8, 2026 22:26
Copilot AI review requested due to automatic review settings July 8, 2026 22:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

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.

3 participants