Skip to content

docs: record QA follow-ups in the review register - #49

Open
GCdePaula wants to merge 2 commits into
mainfrom
docs/register-qa-followups
Open

GCdePaula wants to merge 2 commits into
mainfrom
docs/register-qa-followups

Conversation

@GCdePaula

Copy link
Copy Markdown
Member

Adds the work selected from the QA report assessment, checked on 2026-09-24 against 7e47145: bound the canonical direct-input fridge (superseding the capacity investigation entry), validate batch-open and danger timing together, support HTTPS/WSS in the Rust SDK, separate the ingress and egress listeners, and fix the pinned testsi cycle target. The header now allows sections to carry their own verification stamps.

Adds the work selected from the QA report assessment, checked on
2026-09-24 against 7e47145: bound the canonical direct-input fridge
(superseding the capacity investigation entry), validate batch-open and
danger timing together, support HTTPS/WSS in the Rust SDK, separate the
ingress and egress listeners, and fix the pinned testsi cycle target.
The header now allows sections to carry their own verification stamps.
The QA preamble said its entries were proposed work, not current
behavior, which misdescribed the batch-size SSZ discrepancy kept there
as selected work. Each entry describes current behavior; only its
direction and next steps are proposed. The listener follow-up now names
every public route from the API contract, including GET /nonce and
GET /domain, so a listener split cannot leave them on the internal side.
@GCdePaula
GCdePaula force-pushed the docs/register-qa-followups branch from 408d29b to 36ff34e Compare October 1, 2026 14:05

@stephenctw stephenctw left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@GCdePaula
GCdePaula requested a review from jplgarcia October 2, 2026 20:31
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