Skip to content

[databricks-genie-one] Update setup form copy - #1112

Draft
liam-ilan wants to merge 1 commit into
mainfrom
liam-ilan/databricks-genie-one-variables-properties
Draft

liam-ilan wants to merge 1 commit into
mainfrom
liam-ilan/databricks-genie-one-variables-properties

Conversation

@liam-ilan

@liam-ilan liam-ilan commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator

Update Databricks Genie One setup form copy

The setup form asks for a bare "Workspace hostname" and gives no hints for the client ID or secret fields.

Changes

  • Rename the host field to "Workspace URL" and explain that a workspace URL or Genie link works, since only the hostname is kept.
  • Add example text to the host, client ID, and client secret fields.
  • Tell people they can leave the client secret blank for a public client.

Note

Low Risk
Copy-only changes to plugin variable metadata; no runtime or auth logic is modified.

Overview
Updates the Databricks Genie One plugin setup schema so users get clearer guidance when connecting OAuth.

The host field is renamed from Workspace hostname to Workspace URL, with copy noting that a full workspace or Genie MCP URL is fine because only the hostname is kept. Placeholders were added for host (example https://…), client ID (paste from OAuth app), and client secret (leave blank for a public client).

Reviewed by Cursor Bugbot for commit 8918f14. Bugbot is set up for automated code reviews on this repo. Configure here.

@liam-ilan liam-ilan self-assigned this Oct 28, 2026

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Host copy invites invalid MCP URLs
    • Nothing in the plugin trims a pasted URL down to its hostname before ${DATABRICKS_HOST} goes into mcp.json, so I changed the DATABRICKS_HOST title, description and placeholder back to asking for a bare hostname without https://, path or port, matching the README.

Create PR

Or push these changes by commenting:

@cursor push a8fc3c3710
Preview (a8fc3c3710)
diff --git a/third_party/databricks-genie-one/.cursor-plugin/plugin.json b/third_party/databricks-genie-one/.cursor-plugin/plugin.json
--- a/third_party/databricks-genie-one/.cursor-plugin/plugin.json
+++ b/third_party/databricks-genie-one/.cursor-plugin/plugin.json
@@ -41,9 +41,9 @@
     "properties": {
       "DATABRICKS_HOST": {
         "type": "string",
-        "title": "Workspace URL",
-        "description": "Your workspace URL or Genie MCP URL. Only the hostname is kept.",
-        "placeholder": "https://dbc-a1b2c3d4-e5f6.cloud.databricks.com"
+        "title": "Workspace hostname",
+        "description": "Bare workspace hostname. Leave out https://, any path, and any port.",
+        "placeholder": "dbc-a1b2c3d4-e5f6.cloud.databricks.com"
       },
       "CLIENT_ID": {
         "type": "string",

You can send follow-ups to the cloud agent here.

Reviewed by Cursor Bugbot for commit 8918f14. Configure here.

"description": "For example, dbc-a1b2c3d4-e5f6.cloud.databricks.com"
"title": "Workspace URL",
"description": "Your workspace URL or Genie MCP URL. Only the hostname is kept.",
"placeholder": "https://dbc-a1b2c3d4-e5f6.cloud.databricks.com"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Host copy invites invalid MCP URLs

High Severity · Logic Bug

The setup form now asks for a workspace or Genie MCP URL and shows an https:// example, but DATABRICKS_HOST is still interpolated as a bare hostname in mcp.json. Pasting a URL produces an invalid MCP endpoint, and nothing in this plugin keeps only the hostname.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 8918f14. Configure here.

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