Skip to content

[Brook] feat(wit): parse the @nexus.long-poll operation annotation - #37

Closed
moetemp wants to merge 4 commits into
moe/brook/v3/n1-offset-fieldsfrom
moe/brook/v3/n2-long-poll-parsing
Closed

moetemp wants to merge 4 commits into
moe/brook/v3/n1-offset-fieldsfrom
moe/brook/v3/n2-long-poll-parsing

Conversation

@moetemp

@moetemp moetemp commented Oct 21, 2026 •

Copy link
Copy Markdown
Owner

What changed?

This PR parses the @nexus.long-poll operation directive and checks its fields when the WIT loads.

Why?

Brook's ReadRecords is a long poll. The upstream long-poll PR (#55) carries the same code, and n7 merges it.

How did you test it?

The parser checks are covered by #55.

@moetemp moetemp changed the title [Brook] Parsed the @nexus.long-poll operation annotation on the WIT path. [Brook] Parse the @nexus.long-poll operation annotation on the WIT path Oct 22, 2026
@moetemp moetemp changed the title [Brook] Parse the @nexus.long-poll operation annotation on the WIT path feat(brook): parse the @nexus.long-poll operation annotation on the WIT path Oct 22, 2026
@moetemp moetemp changed the title feat(brook): parse the @nexus.long-poll operation annotation on the WIT path [Brook] feat(wit): parse the @nexus.long-poll operation annotation Oct 22, 2026
@moetemp

moetemp commented Oct 25, 2026

Copy link
Copy Markdown
Owner Author

Closing as superseded. The long-poll parsing goes upstream with UPSTREAM-9 (#55). Plan: https://www.notion.so/4048fc567738814990b7c796ac28fe3f

@moetemp moetemp closed this Oct 25, 2026
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