fix(reliability): engram-write #38

Closed
will.anderson wants to merge 0 commits from improve/reliability-engram-write into main
Owner

Reliability fixes from automated audit.

Guards all fire-and-forget engram_node_full and engram_save calls with return-value checks and println logging. Adds api_persisted() read-back guard to handle_api_log_state_event and handle_api_tune_config. TODO comments added for non-atomic write patterns (#12, #13) and missing circuit breaker (#14).

Reliability fixes from automated audit. Guards all fire-and-forget engram_node_full and engram_save calls with return-value checks and println logging. Adds api_persisted() read-back guard to handle_api_log_state_event and handle_api_tune_config. TODO comments added for non-atomic write patterns (#12, #13) and missing circuit breaker (#14).
will.anderson added 1 commit 2026-06-22 16:51:34 +00:00
fix(reliability): engram-write — guard all fire-and-forget writes
Neuron Soul CI / build (pull_request) Has been cancelled
494d973a3b
Every engram_node_full call that dropped its return value now binds it
and emits a println on empty string. engram_save calls in consolidate,
heartbeat, and dharma-room-turn are checked for failure. The two API
handlers (log_state_event, tune_config) that skipped api_persisted()
now match the read-back-after-write contract used everywhere else in
neuron-api.el.

Files changed:
- chat.el: conv_history_persist, handle_dharma_room_turn, auto_persist
- soul.el: emit_session_start_event, seed_persona_from_env HTTP check
- memory.el: mem_save, mem_boot_count_inc
- neuron-api.el: handle_api_log_state_event, handle_api_tune_config,
  handle_api_consolidate (engram_save + session summary write)
- awareness.el: ise_post local-engram fallback path

TODO comments added for non-atomic patterns (issues #12, #13) and
the missing circuit breaker (#14) — these require new primitives.
will.anderson added 1 commit 2026-06-22 17:21:53 +00:00
fix(reliability): replace undefined session_exists with session_get check
Neuron Soul CI / build (pull_request) Failing after 13m25s
392d2416ec
will.anderson closed this pull request 2026-06-22 17:55:30 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: neuron-technologies/neuron#38