Skip to content

fix(honcho): dedupe repeated message flushes#18536

Open
erosika wants to merge 1 commit into
NousResearch:mainfrom
erosika:fix/honcho-skip-synthetic-sync
Open

fix(honcho): dedupe repeated message flushes#18536
erosika wants to merge 1 commit into
NousResearch:mainfrom
erosika:fix/honcho-skip-synthetic-sync

Conversation

@erosika

@erosika erosika commented May 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add bounded per-session batch fingerprints inside the Honcho session manager
  • skip repeated Honcho uploads from stale async queue/retry/session-end flush paths after a successful sync
  • keep the fix scoped to the Honcho plugin only

Test Plan

  • scripts/run_tests.sh tests/honcho_plugin/test_honcho_idempotency.py tests/honcho_plugin/test_session.py::TestConcludeToolDispatch::test_sync_turn_strips_leaked_memory_context_before_honcho_ingest tests/honcho_plugin/test_async_memory.py -q

@erosika erosika force-pushed the fix/honcho-skip-synthetic-sync branch from 8bb5f4a to 802ec88 Compare May 1, 2026 19:52
@erosika erosika changed the title fix(honcho): skip synthetic review turn sync fix(honcho): dedupe repeated message flushes May 1, 2026
@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/plugins Plugin system and bundled plugins labels May 1, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Implements part of #9404. Related to #17153 (gateway-restart dedup). This adds batch fingerprint dedup for async/retry/session-end flush paths.

@alt-glitch

Copy link
Copy Markdown
Collaborator

Implements part of #9404

@erosika erosika force-pushed the fix/honcho-skip-synthetic-sync branch from 802ec88 to 237a499 Compare May 21, 2026 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants