Skip to content

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

Merged
liam-ilan merged 1 commit into
mainfrom
liam-ilan/databricks-genie-one-variables-copy
Oct 24, 2026
Merged

liam-ilan merged 1 commit into
mainfrom
liam-ilan/databricks-genie-one-variables-copy

Conversation

@liam-ilan

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

Copy link
Copy Markdown
Collaborator

Updates the setup form copy for the Databricks Genie One plugin variables.

Changes

  • DATABRICKS_HOST: title is now "Workspace URL", with the description "Your workspace URL or Genie MCP URL. Only the hostname is kept." and the example https://dbc-a1b2c3d4-e5f6.cloud.databricks.com. The old "Workspace hostname" wording is gone from the manifest.
  • CLIENT_ID: added the example "Paste from your Databricks OAuth app".
  • CLIENT_SECRET: added the example "Leave blank for a public client".
  • Bumped the plugin to 1.0.2 and added a changelog entry.

Notes

  • Property keys stay DATABRICKS_HOST, CLIENT_ID, and CLIENT_SECRET, because mcp.json reads ${DATABRICKS_HOST}, ${CLIENT_ID}, and ${CLIENT_SECRET}.
  • No type, required, additionalProperties, writeOnly, or format values changed. CLIENT_SECRET stays out of required, since Databricks app connections can be public clients.
  • node scripts/validate-plugins.mjs passes.
  • The plugin README still describes DATABRICKS_HOST as a bare hostname without https://. This PR leaves the README as is.

Note

Low Risk
Manifest-only copy and version bump; MCP wiring and schema constraints are unchanged.

Overview
Bumps the Databricks Genie One plugin to 1.0.2 and refreshes only the setup-form labels and placeholders in plugin.json—no changes to variable keys, types, or required fields.

DATABRICKS_HOST is now labeled Workspace URL, with copy that users can paste a full workspace or Genie MCP URL (hostname is retained). Examples were added for all three fields (DATABRICKS_HOST, CLIENT_ID, CLIENT_SECRET) to guide OAuth setup, including noting that CLIENT_SECRET can stay blank for public clients. CHANGELOG documents the release.

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

@liam-ilan liam-ilan self-assigned this Oct 24, 2026
@liam-ilan
liam-ilan marked this pull request as ready for review October 24, 2026 02:21
@liam-ilan
liam-ilan merged commit 6dbc3ff into main Oct 24, 2026
2 checks passed

@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: Form URL copy breaks connection
    • The setup form and changelog now ask for a bare workspace hostname, matching the literal https://${DATABRICKS_HOST} substitution that has no hostname extraction.

Create PR

Or push these changes by commenting:

@cursor push 46b3f6bb57
Preview (46b3f6bb57)
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,10 +41,10 @@
     "properties": {
       "DATABRICKS_HOST": {
         "type": "string",
-        "title": "Workspace URL",
-        "description": "Your workspace URL or Genie MCP URL. Only the hostname is kept.",
+        "title": "Workspace hostname",
+        "description": "Bare workspace hostname. Leave out https://, any path, and any port.",
         "examples": [
-          "https://dbc-a1b2c3d4-e5f6.cloud.databricks.com"
+          "dbc-a1b2c3d4-e5f6.cloud.databricks.com"
         ]
       },
       "CLIENT_ID": {

diff --git a/third_party/databricks-genie-one/CHANGELOG.md b/third_party/databricks-genie-one/CHANGELOG.md
--- a/third_party/databricks-genie-one/CHANGELOG.md
+++ b/third_party/databricks-genie-one/CHANGELOG.md
@@ -4,7 +4,7 @@
 
 ## 1.0.2 — setup form copy
 
-- Plugin variables: `DATABRICKS_HOST` is now titled "Workspace URL" and accepts a workspace URL or Genie MCP URL (only the hostname is kept). Added placeholder examples for `DATABRICKS_HOST`, `CLIENT_ID`, and `CLIENT_SECRET`. Variable keys, types, and required fields are unchanged; `CLIENT_SECRET` stays optional for public clients.
+- Plugin variables: `DATABRICKS_HOST` is titled "Workspace hostname" and takes only the bare hostname (leave out `https://`, any path, and any port). Added placeholder examples for `DATABRICKS_HOST`, `CLIENT_ID`, and `CLIENT_SECRET`. Variable keys, types, and required fields are unchanged; `CLIENT_SECRET` stays optional for public clients.
 
 ## 1.0.1 — Genie icon

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

Reviewed by Cursor Bugbot for commit e27dafe. Configure here.

"description": "Your workspace URL or Genie MCP URL. Only the hostname is kept.",
"examples": [
"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.

Form URL copy breaks connection

High Severity · Logic Bug

The setup form now titles DATABRICKS_HOST as Workspace URL, claims a workspace or Genie MCP URL is accepted and only the hostname is kept, and shows an https:// example. That value is still substituted verbatim into https://${DATABRICKS_HOST}/... with no hostname extraction, so following the form produces a malformed MCP URL and the plugin cannot connect.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e27dafe. 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