Skip to content

feat(sql-editor): install a missing driver and move run checks into the editor - #542

Merged
huyplb merged 2 commits into
mainfrom
cursor/install-missing-driver
Oct 22, 2026
Merged

huyplb merged 2 commits into
mainfrom
cursor/install-missing-driver

Conversation

@huyplb

@huyplb huyplb commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • The schema browser and a query error offer Install driver when the selected credential's package is missing, the same install the credential form already runs.
  • Each statement's play button and include checkbox sit on the left of that statement, so the list between the editor and the results no longer takes a vertical band.

Test plan

  • Select a credential whose driver is not installed and confirm Install driver appears on the schema error and the result error.
  • Open a multi-statement query and confirm each statement has a checkbox and play button on its first line, and the statement list under the editor is gone.
  • Check a box and confirm toolbar Run includes that cell; with nothing checked, Run still uses the cell at the caret.

huyplb and others added 2 commits October 21, 2026 11:43
Selecting Redis or another credential with no driver package only showed the error. The schema browser and the result error now offer the same install the credential form already runs.

Co-authored-by: Cursor <cursoragent@cursor.com>
The statement list between the editor and the results used a band of vertical space. Each cell's play button and include checkbox now sit on that statement's first line.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cursor

cursor Bot commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: 88567d2c-182d-4a80-95a7-a02e404eead1)

@github-actions

Copy link
Copy Markdown
Contributor

Test IDs

Added (3)

  • sql-install-driver-{…}
  • sql-install-driver-{…}-banner
  • sql-statement-gutter

{…} is a part filled in at run time. The full tree: docs/testing/TEST_IDS.md.

@huyplb huyplb changed the title Cursor/install missing driver feat(sql-editor): install a missing driver and move run checks into the editor Oct 22, 2026
@huyplb
huyplb merged commit 9bff58b into main Oct 22, 2026
11 of 13 checks passed
@huyplb
huyplb deleted the cursor/install-missing-driver branch October 22, 2026 19:04
huyplb added a commit that referenced this pull request Oct 25, 2026
fix: follow-ups from reviewing #542 and #538 — run rail, Install driver, Db2 docs
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.

1 participant