Skip to content

perf: make DKG memory refresh demand driven - #6

Open
branarakic-agent wants to merge 3 commits into
feat/local-reputation-providerfrom
perf/dkg-memory-refresh-control
Open

perf: make DKG memory refresh demand driven#6
branarakic-agent wants to merge 3 commits into
feat/local-reputation-providerfrom
perf/dkg-memory-refresh-control

Conversation

@branarakic-agent

Copy link
Copy Markdown
Collaborator

Summary

  • stop automatic 30-second DKG memory and trust polling
  • disable automatic retry/reconnect/focus amplification for expensive DKG reads
  • keep a 30-second local stale window
  • add explicit retry and refresh controls to the Web of Trust panel

Tests

  • Biome on changed files
  • TypeScript typecheck
  • focused DKG policy/provider tests
  • full desktop suite: 4,756 tests

Stacked on #5 because it changes the DKG memory/trust UI introduced there.

@branarakic-agent branarakic-agent left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review pass completed at 5a3558a. I found no new correctness blocker in this PR-specific delta. Heavy DKG queries no longer poll, retry, refetch on focus, or refetch on reconnect; successful reads remain briefly cached; mutations still invalidate active queries; and the Trust UI exposes explicit retry/refresh controls. The focused policy test matches that behavior. This remains stacked on #4 and #5, so it should only merge after their review blockers are fixed and inherited CI is green.

Signed-off-by: branarakic <branimir.rakic@origin-trail.com>
@branarakic
branarakic force-pushed the feat/local-reputation-provider branch from 8981a80 to e7eabeb Compare August 12, 2026 23:59
@branarakic
branarakic force-pushed the perf/dkg-memory-refresh-control branch from d463729 to 829d885 Compare August 12, 2026 23:59
Signed-off-by: branarakic <branimir.rakic@origin-trail.com>
Signed-off-by: branarakic <branimir.rakic@origin-trail.com>
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