Skip to content

Conversation

bgentry
Copy link
Contributor

@bgentry bgentry commented Jul 15, 2025

This integrates all of riverui's non-Pro queries into this library's driver system, which paves the way for riverui to support multiple driver backends (sqlite) and to add more Pro-specific UI extensions. These queries were generally imported as-is except for the sqlite variants which required some rework.

I also added a Client.Schema() accessor because I couldn't see any way for riverui to pull the schema from the river.Client in its constructor opts. Rather than add another param for it there, it seemed cleaner to just expose a reader method on the client. This all ends up being required if we want the UI to properly support custom schemas while also having it directly call into pilot and driver methods without having to expose all this functionality directly on the Client and bloating its API.

@bgentry bgentry force-pushed the bg-extended-riverui-integrations-via-driver branch from 392fc67 to 30dcec8 Compare July 19, 2025 16:51
@bgentry bgentry force-pushed the bg-extended-riverui-integrations-via-driver branch from 30dcec8 to 8dfcc87 Compare July 24, 2025 03:38
@bgentry bgentry merged commit 013878b into master Jul 24, 2025
10 checks passed
@bgentry bgentry deleted the bg-extended-riverui-integrations-via-driver branch July 24, 2025 03:41
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