Fix ARCHITECTURE.md Event trait definition #242

New issue

Closed

opened 2025-12-28 19:59:08 -08:00 by jwilger · 0 comments

jwilger commented

2025-12-28 19:59:08 -08:00

(Migrated from github.com)

Issue

The Event trait documentation in docs/ARCHITECTURE.md:374-376 shows an incomplete trait definition:

pub trait Event: Clone + Send + 'static {
    fn stream_id(&self) -> &StreamId;
}

Actual Trait (src/command.rs:202)

pub trait Event: Clone + Send + Serialize + DeserializeOwned + 'static {
    fn stream_id(&self) -> &StreamId;
    fn event_type_name(&self) -> EventTypeName;
    fn all_type_names() -> Vec<EventTypeName>;
}

Fix Required

Update ARCHITECTURE.md to show complete trait definition including:

Serialize + DeserializeOwned bounds
event_type_name() method
all_type_names() method

Acceptance Criteria

ARCHITECTURE.md Event trait definition matches actual implementation
All trait bounds documented
All methods documented

Migrated from beads issue: eventcore-6am

## Issue The Event trait documentation in `docs/ARCHITECTURE.md:374-376` shows an incomplete trait definition: ```rust pub trait Event: Clone + Send + 'static { fn stream_id(&self) -> &StreamId; } ``` ## Actual Trait (src/command.rs:202) ```rust pub trait Event: Clone + Send + Serialize + DeserializeOwned + 'static { fn stream_id(&self) -> &StreamId; fn event_type_name(&self) -> EventTypeName; fn all_type_names() -> Vec<EventTypeName>; } ``` ## Fix Required Update ARCHITECTURE.md to show complete trait definition including: - `Serialize + DeserializeOwned` bounds - `event_type_name()` method - `all_type_names()` method ## Acceptance Criteria - [ ] ARCHITECTURE.md Event trait definition matches actual implementation - [ ] All trait bounds documented - [ ] All methods documented --- *Migrated from beads issue: eventcore-6am*