Finding 1: local store failure now propagates via map_err instead of
unwrap_or_default, so a broken managed-agents store can never silently
reclassify every local instance as relay-only. Added
local_store_failure_propagates_not_silently_dropped test + #[allow]
on the deliberate complement demo line.
Finding 2: unknown relay instances now have a top-level entry point in
UnifiedAgentsSection (relay-unknown-agents-group). InstancesSheet gains
a showUnknown prop defaulting to false in persona-scoped mode and true
in global-unknown mode (persona=null), so opening Duncan's Sheet does
not show unrelated unknown rows.
Finding 3: profile test replaced the tautological assertion with real
ones: expect(fetchedB).toBe(true) and getByTestId('user-profile-panel')
visible.
Finding 4: presence test seeds presenceOverrides for both duplicate
pubkeys via MockBridgeOptions and asserts distinct Online/Offline badges.
Finding 5: verify_snapshot_for_trust and reduce_snapshot_query_result
extracted as pub(super) helpers; load_archive_snapshot delegates to
reduce_snapshot_query_result; all four trust-arm tests drive production
helpers instead of re-declaring match arms inline.
Co-authored-by: Will Pfleger <pfleger.will@gmail.com>
Signed-off-by: Will Pfleger <pfleger.will@gmail.com>
