messages: created_at never stamped on save #25

New issue

Closed

opened 2026-04-19 08:25:32 +00:00 by zaelgohary · 0 comments

zaelgohary commented

2026-04-19 08:25:32 +00:00

Member

The server-side chatmessage_trigger_save_pre trigger in crates/hero_osis_server/src/communication/server/rpc.rs was a no-op, so every saved message kept created_at = 0 / updated_at = 0.

This surfaced in the messaging UI as 00:00 times on every message bubble and an Invalid Date / Jan 1 1970 entry in the conversation list when the server data was first loaded.

Expected: every persisted ChatMessage has a non-zero creation time the first time it is stored, and updated_at refreshes on every subsequent save.

Proposed fix: stamp created_at (when 0) and updated_at in the central chatmessage_trigger_save_pre trigger — one hook covers send_message, reply_to_message, add_reaction, and any future handler.

The server-side `chatmessage_trigger_save_pre` trigger in `crates/hero_osis_server/src/communication/server/rpc.rs` was a no-op, so every saved message kept `created_at = 0` / `updated_at = 0`. This surfaced in the messaging UI as `00:00` times on every message bubble and an `Invalid Date` / `Jan 1 1970` entry in the conversation list when the server data was first loaded. **Expected**: every persisted `ChatMessage` has a non-zero creation time the first time it is stored, and `updated_at` refreshes on every subsequent save. **Proposed fix**: stamp `created_at` (when 0) and `updated_at` in the central `chatmessage_trigger_save_pre` trigger — one hook covers `send_message`, `reply_to_message`, `add_reaction`, and any future handler.

zaelgohary referenced this issue from a commit

2026-04-19 08:26:32 +00:00

communication: add list_messages RPC and stamp message timestamps

~~zaelgohary referenced this issue 2026-04-19 08:26:46 +00:00~~

communication: list_messages + stamp timestamps #27

zaelgohary referenced this issue

2026-04-19 08:31:56 +00:00

communication: list_messages + stamp timestamps #27

zaelgohary changed title from ~~bug: ChatMessage.created_at / updated_at never stamped on save~~ to messages: created_at never stamped on save

2026-04-19 08:32:18 +00:00

zaelgohary referenced this issue from a commit

2026-04-19 09:26:04 +00:00

tests: end-to-end messaging regression

~~zaelgohary referenced this issue from lhumina_code/hero_os 2026-04-19 09:26:23 +00:00~~

tests: end-to-end messaging regression #71

zaelgohary closed this issue

2026-04-19 09:27:33 +00:00

zaelgohary referenced this issue from a commit

2026-04-19 09:43:36 +00:00

tests: end-to-end messaging regression