Skip to content

fix(console): the runtime login modal hands over an npx-prefixed command - #1820

Merged
spacedragon merged 1 commit into
mainfrom
dev/yulong/chiton-2
Sep 7, 2026
Merged

fix(console): the runtime login modal hands over an npx-prefixed command#1820
spacedragon merged 1 commit into
mainfrom
dev/yulong/chiton-2

Conversation

@spacedragon

Copy link
Copy Markdown
Contributor

What

The runtime "Login required" modal handed over agentconnect auth --runtime <id>, then spent
one of its four bullets explaining how to run it when agentconnect is not on that host's
PATH (npx -y @agentconnect.md/cli), plus --runtime/--instance flag trivia.

Put the npx prefix in the command itself — it is correct whether or not the CLI is on PATH —
and drop the two bullets that were working around its absence. The modal now shows the
command and two lines: what it does, and when sessions pick the credential up.

Verified

  • pnpm --filter @agentconnect.md/web test src/components/console/views/ClusterDetailView.test.tsx — 23 passed (the assertion on the handed-over command moved with it)
  • pnpm --filter @agentconnect.md/web typecheck — clean

The "Log in to <runtime>" modal handed over `agentconnect auth --runtime <id>`
and then spent a bullet explaining how to run it when `agentconnect` is not on
that host's PATH. Put the `npx -y @agentconnect.md/cli` prefix in the command
itself, which is correct either way, and drop that bullet along with the
`--runtime`/`--instance` flag trivia the operator does not need to log in.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

@agentconnect-md-test agentconnect-md-test Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed f39856b. No blocking regressions found. The npx command matches the CLI’s published executable and preserves forwarding of auth --runtime to the active daemon.

Validation: all 23 ClusterDetailView tests passed, web typecheck passed, and git diff --check passed. Tests and typecheck ran through the installed binaries after pnpm’s setup hook encountered the protected Git config.

sent by review-bot (Codex · gpt-6-astra) · open in session

@spacedragon
spacedragon merged commit b15e0c1 into main Sep 7, 2026
13 checks passed
@spacedragon
spacedragon deleted the dev/yulong/chiton-2 branch September 7, 2026 05:54
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