From e9f2f1d7aaebecf785b453806cbd55c2063b5aeb Mon Sep 17 00:00:00 2001 From: Vitor Pamplona Date: Fri, 5 Jun 2026 16:45:04 -0400 Subject: [PATCH] refactor(dm): tighten visibility of internal-only paging helpers MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - BackwardRelayPager.floorFor was public but only ever called inside the pager (and its same-module test) — narrow to internal. - RelayReachMarker composable was public but only rendered by RelayWindowLimitMarkers in the same file; the public entry points are RelayWindowLimitMarkers/Sentinels — make it private. Co-Authored-By: Claude Opus 4.8 (1M context) --- .../vitorpamplona/amethyst/commons/ui/feeds/RelayReachMarker.kt | 2 +- .../quartz/nip01Core/relay/client/paging/BackwardRelayPager.kt | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/commons/src/commonMain/kotlin/com/vitorpamplona/amethyst/commons/ui/feeds/RelayReachMarker.kt b/commons/src/commonMain/kotlin/com/vitorpamplona/amethyst/commons/ui/feeds/RelayReachMarker.kt index 4d4fc9e21b..f886f93383 100644 --- a/commons/src/commonMain/kotlin/com/vitorpamplona/amethyst/commons/ui/feeds/RelayReachMarker.kt +++ b/commons/src/commonMain/kotlin/com/vitorpamplona/amethyst/commons/ui/feeds/RelayReachMarker.kt @@ -196,7 +196,7 @@ fun RelayWindowLimitMarkers( * Reads e.g. "Relay sync: ✓ 8 · ↓ 1" or "Relay sync: ↓ nostr.wine". */ @Composable -fun RelayReachMarker(entries: List) { +private fun RelayReachMarker(entries: List) { if (entries.isEmpty()) return Row( diff --git a/quartz/src/jvmAndroid/kotlin/com/vitorpamplona/quartz/nip01Core/relay/client/paging/BackwardRelayPager.kt b/quartz/src/jvmAndroid/kotlin/com/vitorpamplona/quartz/nip01Core/relay/client/paging/BackwardRelayPager.kt index 21474a810f..094f4b749d 100644 --- a/quartz/src/jvmAndroid/kotlin/com/vitorpamplona/quartz/nip01Core/relay/client/paging/BackwardRelayPager.kt +++ b/quartz/src/jvmAndroid/kotlin/com/vitorpamplona/quartz/nip01Core/relay/client/paging/BackwardRelayPager.kt @@ -132,7 +132,7 @@ class BackwardRelayPager( val relayProgress: StateFlow> = _relayProgress.asStateFlow() /** The session-pinned floor for [key] — where its paging starts (just below the live tail). */ - fun floorFor(key: K): Long = pinnedFloor.getOrPut(key) { TimeUtils.now() - liveTailSeconds } + internal fun floorFor(key: K): Long = pinnedFloor.getOrPut(key) { TimeUtils.now() - liveTailSeconds } // --- Filter building support: the caller assembles the actual REQ from these. ---