Skip to content

app-server: Emit thread archive/unarchive notifications#12030

Merged
iceweasel-oai merged 1 commit intomainfrom
ruslan/archive-unarchive-events
Feb 17, 2026
Merged

app-server: Emit thread archive/unarchive notifications#12030
iceweasel-oai merged 1 commit intomainfrom
ruslan/archive-unarchive-events

Conversation

@euroelessar
Copy link
Contributor

  • Add v2 server notifications thread/archived and thread/unarchived with a threadId payload.
  • Wire new events into thread/archive and thread/unarchive success paths.
  • Update app-server protocol/schema/docs accordingly.

Testing:

  • Updated archive/unarchive end-to-end tests to verify both notifications are emitted with the expected thread id payload.

@euroelessar euroelessar requested a review from owenlin0 February 17, 2026 19:54
@etraut-openai etraut-openai added the oai PRs contributed by OpenAI employees label Feb 17, 2026
Add v2 server notifications thread/archived and thread/unarchived with a threadId payload, wire them into thread/archive and thread/unarchive success paths, and update app-server protocol/schema/docs accordingly.
Also extend the existing v2 archive/unarchive e2e tests to assert both notifications are emitted with the correct thread id.

Testing:
- Updated archive/unarchive end-to-end tests to verify both notifications are emitted with the expected thread id payload.
@euroelessar euroelessar force-pushed the ruslan/archive-unarchive-events branch from a133edb to 7290763 Compare February 17, 2026 21:27
@iceweasel-oai iceweasel-oai merged commit 31cbebd into main Feb 17, 2026
61 of 69 checks passed
@iceweasel-oai iceweasel-oai deleted the ruslan/archive-unarchive-events branch February 17, 2026 22:54
@github-actions github-actions bot locked and limited conversation to collaborators Feb 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

oai PRs contributed by OpenAI employees

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants