feat(pkg-r): add ggsql visualization tool#224
Merged
Conversation
cd14057 to
5731d0a
Compare
cpsievert
commented
Apr 29, 2026
cpsievert
commented
Apr 29, 2026
f52f440 to
afbbfba
Compare
cpsievert
commented
Apr 30, 2026
8b342a8 to
6b7e981
Compare
cpsievert
commented
May 1, 2026
cpsievert
commented
May 1, 2026
aa52aa6 to
8a1928a
Compare
8a1928a to
b79b71e
Compare
For TblSqlSource, execute_query() returns a lazy tbl_sql which ggsql_register() can't handle. Materialize it first, matching the existing pattern in app_obj().
This comment was marked as outdated.
This comment was marked as outdated.
Add a new Visualizations vignette, a Data visualization section to the Tools vignette, a navbar entry in pkgdown, and a mention in the README, mirroring the documentation added for the Python package in PR #201.
Add screenshots for bar chart, scatter plot, fullscreen, and show-query views. Wire them into the Visualizations vignette and the README.
6 tasks
Resolve conflicts between the ggsql/visualize tool additions on this branch and main's "filter" rename + suggestion UI changes: - DESCRIPTION: keep both Remotes (ggsql-r and shinychat/pkg-r) - QueryChat.R: combine "filter"/"visualize" in arg_match values - querychat_module.R: use main's heading-grouped GREETING_PROMPT, keep branch's restore_viz_widgets helper - prompt.md: drop nested-list suggestion format per main's change - QueryChat.Rd: merge filter terminology with visualize/session params
ggsql is now on CRAN — remove posit-dev/ggsql-r from Remotes and update vignette install instructions to use install.packages().
bslib 0.11.0 supports the "ggsql" language in input_code_editor, so we can require it directly and remove the runtime packageVersion check.
R CMD check warns about non-ASCII characters in R code files. Replace em dash with double hyphen in the ggsql tool description string.
Replace 32-bit sample.int with random_hex(8) for collision-resistant ID generation, matching the Python implementation's approach.
Make the viz container a fill carrier so the ggsql widget properly fills the aspect-ratio-driven height instead of using a fixed height. Also remove the tools default from mod_server() and gracefully skip viz widget restoration when ggsql is not installed.
The "passes the namespaced DOM id into the footer" test was inside the collapse_validation_errors() describe block but actually exercises tool_visualize_dashboard(). Move it to the correct group.
Add conditional mustache blocks for tool-aware suggestion groups that Python already has: "Analyze the data" gated on has_tool_query, new "Visualize the data" group gated on has_tool_visualize.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
R port of #201. Adds an opt-in
visualizetool to querychat's R package, enabling LLM-generated data visualizations via ggsql. When enabled, the LLM can write ggsql queries (SQL +VISUALISEclause) that are executed and rendered as interactive charts inline in the chat.Usage
The
visualizetool is opt-in — include"visualize"in thetoolsvector alongside"filter"and/or"update"to enable it.What it looks like
Charts render inline in the chat with a collapsible footer showing the ggsql query (with syntax highlighting), a copy button, and save options:
Key changes
visualizetool (opt-in viatools = c("query", "visualize")): The LLM writes a full ggsql query, which is executed against the data source and rendered using ggsql's native Shiny bindings (ggsql::renderGgsql/ggsql::ggsqlOutput).querychat_viz.R): Uses DataSource for SQL execution (preserving database pushdown), then feeds results into ggsql's DuckDB reader for VISUALISE processing.pkg-r/inst/htmldep/viz.{js,css}are checked-in package assets, but the shared source/build logic for them already lives onmainunderjs/(seejs/build.mjs). This PR is adding the R package's generated copies, not introducing a second implementation of that browser logic.V8+rsvg) and sent back to the LLM so it can see what it produced.collapsedparameter onquerychat_query: Preparatory queries (e.g., inspecting data before visualizing) can start collapsed so the chart remains the focal point.Test plan
execute_ggsql,extract_visualise_table,has_layer_level_sourcetool_visualize_dashboard(creation, callback, error handling)truncate_errorcollapsedparameter on query tool