fix(ui): accordion connection-info + demo password prefill#874
Closed
buggtb wants to merge 2 commits into
Closed
Conversation
Three polish issues caught on the live demo login screen: - The Agent API access panel was a wall of text below the sign-in form. ApiAccessAdmin now wraps each section in a <details> styled as a card, collapsed by default. Component takes a defaultOpen prop for the admin tab to pre-expand if desired. - Demo-mode LoginForm prefilled "admin" username but left password blank. Now sets password=admin once the capabilities probe confirms demoMode, so a visitor can land on demo.saiku.bi and click Sign in immediately instead of digging the credential out of the panel below the form. - DXT download install-row block re-confirmed under the MCP accordion; source already had it but the stale dist masked it on first deploy.
Claude Desktop rejected the previous bundle with:
"Invalid enum value. Expected 'python' | 'node' | 'binary',
received 'remote', Required, Required, Unrecognized key(s)
in object: 'transport'"
DXT's schema has no first-class remote/HTTP server mode — only
local stdio servers of type python/node/binary. So the bundle now
ships a Node shim alongside the manifest:
- manifest.json: server.type=node, entry_point=server.js,
mcp_config={command: "npx", args: ["-y", "mcp-remote@latest", URL]}
- server.js: tiny Node script that spawns `npx mcp-remote URL`
as a fallback for Claude builds that ignore mcp_config
The mcp-remote npm package bridges stdio MCP ↔ remote HTTP MCP
servers. Claude Desktop installs the DXT, runs the shim, and the
shim proxies the user's stdio MCP frames to our /mcp URL.
Tests updated: now asserts server.type=node, mcp_config.args
contains the URL, and server.js is present in the zip with the
URL embedded.
5 tasks
Contributor
Author
|
Superseded by #875. The DXT manifest fix here uses The UI accordion fix here also lands in #875 (commit Closing in favour of the larger combined PR. |
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
1 participant
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.
Three polish issues from the live demo login screen review. See commit message.