Skip to content

feat(idgen): add configurable prefix for operator event IDs#832

Closed
alexluong wants to merge 1 commit intomainfrom
feat/opevents-idgen-prefix
Closed

feat(idgen): add configurable prefix for operator event IDs#832
alexluong wants to merge 1 commit intomainfrom
feat/opevents-idgen-prefix

Conversation

@alexluong
Copy link
Copy Markdown
Collaborator

Summary

  • Add IDGEN_OPERATOR_EVENT_PREFIX config to support prefixed IDs for operator events
  • Add OperatorEvent() method to IDGenerator following the same pattern as Event/Destination/Attempt
  • Update opevents.Emitter to use idgen.OperatorEvent() instead of idgen.String()

Test plan

  • Unit tests added for OperatorEvent() (default UUID, prefix, nanoid+prefix, independence from event prefix)
  • Verify operator events emitted with configured prefix
  • Verify backward compat: no prefix by default

🤖 Generated with Claude Code

Add OperatorEventPrefix support to the ID generation system, following
the same pattern as Event, Destination, and Attempt prefixes. The
emitter now uses idgen.OperatorEvent() instead of idgen.String().

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
outpost-docs Ready Ready Preview, Comment Apr 13, 2026 9:09pm
outpost-website Ready Ready Preview, Comment Apr 13, 2026 9:09pm

Request Review

@alexbouchardd
Copy link
Copy Markdown
Contributor

I don't believe that's necessary given that those IDs are internal and we can decide on the ID format. I would close

@alexluong alexluong closed this Apr 13, 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.

2 participants