Skip to content

feat(webui): Integrate Monaco editor for future Presto SQL query input.#1108

Merged
hoophalab merged 16 commits into
y-scope:mainfrom
davemarco:sqlbox
Jul 24, 2025
Merged

feat(webui): Integrate Monaco editor for future Presto SQL query input.#1108
hoophalab merged 16 commits into
y-scope:mainfrom
davemarco:sqlbox

Conversation

@davemarco
Copy link
Copy Markdown
Contributor

@davemarco davemarco commented Jul 16, 2025

Description

PR adds the monaco editor to the webui for SQL input, and adds support for highlighting and autocomplete for sql syntax like SELECT, WHERE etc.

I also added a "run" button, which seemed more appropriate than "search" since we support more queries with sql.

They only render when presto is set as query engine in settings.

Checklist

  • The PR satisfies the contribution guidelines.
  • This is a breaking change and that has been indicated in the PR title, OR this isn't a
    breaking change.
  • Necessary docs have been updated, OR no docs need to be updated.

Validation performed

Editor loads. To validate yourself, you must change from ClpQueryEngine from native to presto in settings.json in public folder.

Summary by CodeRabbit

  • New Features

    • Introduced a SQL editor with syntax highlighting and autocomplete for SQL queries.
    • Added a dedicated SQL query input and "Run" button for Presto query engine workflows.
    • UI now dynamically switches between native and Presto query controls based on configuration.
  • Enhancements

    • Added configuration options to select the query engine type in settings.
    • Improved editor responsiveness and user experience with dynamic sizing and tooltips.
  • Chores

    • Updated dependencies to support Monaco Editor integration.
    • Adjusted build configuration for optimized Monaco Editor loading.

Loading
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.

3 participants