Skip to content

Repoint kbagent plugin install to the AI Kit marketplace - #1095

Open
claude[bot] wants to merge 17 commits into
mainfrom
agent/kbagent-marketplace-repoint
Open

Repoint kbagent plugin install to the AI Kit marketplace#1095
claude[bot] wants to merge 17 commits into
mainfrom
agent/kbagent-marketplace-repoint

Conversation

@claude

@claude claude Bot commented Aug 23, 2026

Copy link
Copy Markdown

Brings the kbagent docs in line with what the product shows on Settings → Developer settings → Agentic CLI, after each AI client was hand-tested. Then corrected in review: two claims that came out of that testing were wrong.

Everything that changed

# What Before After Asked by Why
1 Marketplace and plugin keboola/cli, kbagent@keboola-agent-cli keboola/ai-kit, kbagent@keboola-claude-kit, plus /plugin marketplace remove keboola-agent-cli for anyone on the old one product change The plugin moved; nothing migrates readers off the old marketplace, so only copy can
2 Client coverage Claude Code, Cursor, Claude Desktop; "no kbagent plugin exists" for VS Code, Codex, Devin Per-client routes for Claude Code, Claude Desktop, Cursor, VS Code, ChatGPT app; Devin dropped hand testing All five install the same plugin. VS Code and Codex read the Claude plugin format, so the old claim was false. Devin is hidden from the product surface
3 Setup order for non-Claude-Code clients mixed Terminal half once (install, project add, doctor), then the client's own plugin UI hand testing project add prompts for a token with hidden input, which needs a real TTY. It cannot be handed to an agent
4 kbagent doctor last step, after the plugin steps with the install and connect steps it checks hand testing It only looks in ~/.claude/plugins/cache, so it is blind to a Cursor or VS Code install and cannot vouch for the plugin half
5 The payoff ask /kbagent list my projects kbagent list my projects, no leading slash hand testing Claude Desktop answers /kbagent with "Unknown command"
6 /kbagent:setup the shortcut for every client, and it "signs you in" Claude Code only, and it starts a browser sign-in you finish in a terminal @Iamfle4ka 1, @KaroEverling 1 auth login needs a human at a browser; keboola/cli#704 marks every client, Claude Code included, as unable to finish it in-chat
7 Connecting a project one route, auth login --register-projects both routes labelled: auth login + auth register-projects for the browser, project add --token for a static token @Iamfle4ka 1 project add resolves --token, then KBC_TOKEN, then a hidden prompt. A reader with no token had nothing to type
8 AI Kit page three plugins, no kbagent entry, a developer plugin seven plugins, kbagent with its source and clients, no developer @Iamfle4ka 3 The live marketplace ships seven and has never had a developer; its link 404'd
9 Token creation the Agentic CLI page only that, plus a Settings → API Tokens fallback @Iamfle4ka 4 The Agentic CLI page is behind a feature flag, so the only route given was a dead end for most readers
10 cli/index.md quickstart required --token, said kbagent "never needs a browser" signs in the way Step 2 now does @Iamfle4ka 5 Rewriting Step 2 without it left the two pages contradicting each other
11 keboola-cli "the unrelated legacy kbc CLI" a separate project-review toolkit built on the older kbc sync CLI @Iamfle4ka 6 It is a current v1.1.1 plugin Keboola publishes, not the old CLI. The warning still belongs, since it sits beside kbagent in every list
12 keboola/cli#704 link in customer-facing prose in an HTML comment only @KaroEverling 2 It resolves publicly and lands a reader on an open issue about the flow they are following
13 Windows install one command plus a fallback when a rate-limited GitHub API call leaves $ver empty @Iamfle4ka NB The empty variable surfaces as a bare uv 404, which reads as nothing to do with rate limiting
14 Internal comments VERIFY(owner: Padak) and a #704 repro note contradicting the prose removed; provenance comments rescoped @Iamfle4ka NB They shipped into public AI-facing markdown, and attested reviews that did not cover this content
15 "Skip the terminal steps" tip unconditional, and forked Step 2's example conditional on project list, defers to Step 2 @Iamfle4ka NB It contradicted every client section's "do the terminal steps above"
16 Vendored command reference v0.76.1, with three skip-next markers for auth v0.91.0, no skips @michalsevcik The skips only existed because the reference predated the auth group
17 kbagent tool offered in three pages as a way to call MCP tools removed follow-on from 16 The string does not appear anywhere in v0.91.0. Nothing replaced it, so deletion is the fix
18 kbagent doctor --fix "repairs what it safely can", twice doctor reports, the manual steps below fix follow-on from 16 doctor takes no options at all in v0.91.0
19 --version sample kbagent v0.66.0 kbagent v0.91.0 @michalsevcik It trailed the shipped version by 25 releases
20 Beta label on cli/index.md :::caution[Beta] removed @michalsevcik Nothing else labels kbagent beta: not keboola.com, not keboola/cli's README, and the product badges the connect card NEW. This reverses one of @jordanrburger's conditions for merging #1015 (PRDCT-490) — see the comment thread, reverting is one commit

Still open

  • kbagent list my projects keeps a skip-next. It is a chat prompt, not a command, so the gate will never know it.
  • /cli/for-agents/ was not renamed. It returns 200 and keboola.com's hero links to it, so /cli/ai-agents/ would 404.
  • chore: sync CLI command reference (v0.91.0) #1044 carries the same v0.91.0 bump as row 16 and will conflict or go redundant.
  • The same wrong sign-in sentence shipped to keboola.com with keboola_com#531; fixed in keboola_com#541.

npm run build clean (361 pages) · node scripts/check-cli-reference.mjs 0 findings · base merged from main, no conflicts.

🤖 Generated with Claude Code

The kbagent Claude Code plugin now ships from the keboola/ai-kit
marketplace (keboola-claude-kit) instead of the CLI's own marketplace
(keboola-agent-cli).

- cli/for-agents: update the install code block to
  `/plugin marketplace add keboola/ai-kit` +
  `/plugin install kbagent@keboola-claude-kit`, reword the lede that
  called it "the CLI's own marketplace", and refresh the marketplace
  name in the page's provenance comment.
- ai/ai-kit: list `kbagent` alongside the other plugins in the
  "enable the plugins you need" block.

The CLI's on-disk config directory (~/.config/keboola-agent-cli) and
the CLI install instructions themselves are unchanged.
@vercel

vercel Bot commented Aug 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
connection-docs Ready Ready Preview Aug 27, 2026 9:05am

Request Review

The CLI docs were behind the product: they pointed Windows readers at a
git+https uv install, led with a Storage API token when browser login now
needs none, and said nothing about per-client setup at all.

- cli/getting-started: replace the Windows tab with the canonical
  PowerShell path from keboola/cli's README (winget uv, release wheel,
  `uv tool update-shell`, open a new shell), plus the Git-bash and
  self-contained-zip alternatives and the package-manager warning. Add
  the POSIX PATH note. Rewrite Step 2 to lead with
  `auth login --register-projects` and demote the token to the CI /
  headless route, prompting for it instead of passing `--token`.
- cli/for-agents: add a "Set up your client" section covering Claude
  Code, Claude Desktop, the plugin-less clients (Cursor, VS Code, Codex,
  Devin Desktop) and a plain terminal, and describe `/kbagent:setup` and
  why a plugin slash command cannot be skipped or run in a shell.
- ai/ai-kit: qualify the kbagent install with its marketplace and link
  across to the per-client steps.

Claude Desktop's chat accepting /kbagent:setup is flagged with a VERIFY
comment; it has not been checked against a live build.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
eruveo and others added 2 commits August 26, 2026 12:33
Tested against live clients on 2026-08-26, which contradicts what the
previous commit assumed: Cursor has a marketplace, Claude Desktop has no
slash-command surface at all, and /kbagent:setup completes nowhere.

- Claude Desktop: no /plugin and no /kbagent:setup ("Unknown command" and
  "/plugin isn't available in this environment"), so its section is the
  manual flow. Menu path corrected to Customise -> Plugins -> Add -> Add
  from marketplace, install via the plus button on the card titled
  "Kbagent". Drops the VERIFY comment the fact replaces.
- Cursor: promoted from the plugin-less group to its own section, with
  the Import from Github full-URL requirement and the useless
  [invalid_argument] Error the short form returns.
- /kbagent:setup: demoted to a shortcut that stops at the sign-in and
  hands you a terminal command, with keboola/cli#704 as the tracking
  issue for in-chat login. The manual sequence is the happy path again.
- Adds the once-per-machine sign-in tip (all clients share the local
  config) and a line on asking the agent in plain language, which the
  plugin's skill handles.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
eruveo added 3 commits August 26, 2026 15:24
Hand-testing each client showed the plugin installs everywhere, not
only in Claude Code: Claude Desktop, Cursor, VS Code and the ChatGPT
app all read the Claude plugin format that keboola/ai-kit publishes.
The page claimed the opposite for VS Code and Codex, so a reader on
those clients was told to skip the plugin that would have worked.

Each client reaches it differently (short repo form vs full GitHub
URL, a command palette entry, Developer mode), so the routes are
written out per client rather than generalised. The terminal half is
shared, because install, connect and doctor are the same everywhere
and all three need a real TTY.

Also drops the Devin section (hidden from the product's own setup
surface) and adds the marketplace-move step for readers who followed
the old keboola/cli instructions.
Step 2 sent readers to Project Settings → API Tokens to mint a Full
Access token by hand. The UI now has a page that builds the whole
`project add` command for the reader's project and stack, and can put
a scoped read-only token in it, so the docs point there instead of
walking through a token form they cannot fill in for you.

Interactive setups need no token at all, which was the bigger gap:
`project add` signs you in through the browser. The token route stays
for CI and the features that still require a static token.
The install block already had kbagent in it, but the page still
described three plugins and only documented those three, so the one
plugin readers arrive for had no entry. It also claimed
keboola-claude-kit is the only marketplace Keboola operates, which we
cannot vouch for; what matters is that it is the one to install from.

The CLI landing page called the same thing "the Claude Code plugin".
@Iamfle4ka

Copy link
Copy Markdown
Collaborator

Reviewed against keboola/cli main, the live keboola/ai-kit marketplace, and issue #704. Mechanics are fine (check:cli and build pass, skip-next placements correct, Windows wheel/zip names match v0.91.0). But several factual claims contradict the PR's own sources — requesting changes.

Blocking

  1. project add does not sign you in through the browser (cli/getting-started.mdx Step 2, repeated in cli/for-agents.md). In keboola/cli main, project_add resolves a token as --tokenKBC_TOKEN → hidden TTY prompt → error; no browser flow. Browser sign-in is the separate kbagent auth login (docs/auth.md). A tokenless reader hits a hidden token prompt — and this PR deleted the token-creation walkthrough. Document auth login + auth register-projects as the interactive path.

  2. /kbagent:setup "signs you in" contradicts its own citation (cli/for-agents.md). kbagent: /kbagent:setup cannot complete login in-chat, in any client cli#704 says login can't be completed in any client — its table marks Claude Code "finishes: no — hands auth login to the user's terminal", and the PR's own HTML comment admits it was reproduced in Claude Code. Claude Code can run the command; no client finishes the sign-in in-chat.

  3. ai/ai-kit/index.md doesn't match the marketplace it now anchors to: live keboola-claude-kit ships seven plugins, none named developer — so "four plugins", /plugin install developer, and the plugins/developer link (404) are all wrong. (The Developer section is pre-existing, but this PR is what pins the page to this marketplace and asserts the count.)

  4. Token creation is a dead end behind a feature flag (getting-started.mdx, "When you need a token"): the only route given is the Agentic CLI settings page, flagged off until launch (and permanently for one tenant), while docs deploy on merge. Add a fallback: "If you don't see Agentic CLI, create a token under Project Settings → API Tokens (details)." Also delete the now-orphaned public/cli/token-create.png.

  5. cli/index.md left contradicting the rewritten Step 2: the "in one minute" block still requires --token YOUR_TOKEN and the page still says kbagent "never needs a browser". (Pre-existing lines this PR didn't touch — but rewriting Step 2 without them is what makes the two pages contradict each other; fixing them is in this PR's scope.)

  6. keboola-cli mischaracterized (for-agents.md): it's not "the unrelated legacy kbc CLI" — it's a current v1.1.1 review/management toolkit built on kbc. Suggest: "a separate project-review toolkit built on the older kbc sync CLI — for the agent interface, pick kbagent."

Non-blocking

  • Windows recommended path fails silently when the unauthenticated GitHub API call is rate-limited/proxied ($ver empty → malformed wheel URL → baffling uv 404). Add one fallback line.
  • Internal comments ship into the public AI-facing markdown (<slug>/index.md): drop the VERIFY(owner: Padak) note and the Python backend size added #704 repro comment that contradicts the visible prose above it.
  • The "skip the terminal steps if already signed in" tip contradicts every client section's unconditional "Do the terminal steps above" — make step 1 conditional; and the project add example forks getting-started's with a different alias, so defer to Step 2 instead of restating it.
  • Provenance comments now attest reviews that didn't happen (Padak's v0.66.1 review covered keboola-agent-cli, not keboola-claude-kit; "verified 2026-07-13" now covers 2026-08-26 claims). Update both.

@KaroEverling

KaroEverling commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

@eruveo two things in src/content/docs/cli/for-agents.md. Rest of the PR looks good.

1. Three lines disagree about Claude Code

Line 31: /kbagent:setup "installs the CLI if it is missing, signs you in, registers every project..."

Line 33: "The other four clients... also cannot finish the sign-in it performs" (implies Claude Code can)

Line 35, your HTML comment: "the login hand-off back to a terminal was reproduced in Claude Code, Cursor and Claude Desktop"

keboola/cli#704, which line 33 cites, has a table saying Claude Code → finishes → no.

So line 31 says one thing, and lines 33/35 plus #704 say another. Which is right? You hand-tested this, I didn't, and #704 was only filed this morning so it may already be stale. One of the two is wrong either way.

If #704 is right, line 31 could read:

...installs the CLI if it is missing, starts the browser sign-in, and registers every project you can reach. You finish the sign-in in your terminal.

2. The keboola/cli#704 link is customer-facing

keboola/cli is public, so it resolves. A reader clicking it lands on an open issue saying "The documented 'three commands and you are set up' flow is really 'three commands, then go to a terminal anyway'."

for-agents.md on main links to zero GitHub issues today.

Line 33 also reads oddly: it says these clients can't run the command, then explains they can't finish that command's sign-in. Maybe just:

The other four clients cannot run it. Install the CLI and connect your project in a terminal first, then add the plugin from your client's UI.

Keep #704 in the HTML comment on line 35 where customers won't see it.

Not urgent

Nothing is live. main and help.keboola.com/cli/for-agents/ have zero mentions of /kbagent:setup, so this section is all new here.

Also: base is BEHIND main, and public/cli/token-create.png is now unreferenced (you noted that already).

Same sentence elsewhere

keboola_com#531 step 04 says "signs you in through the browser", and #538 on top of it repeats it. Both unmerged, so whatever you decide can go into all three.

🤖 Generated with Claude Code

eruveo added 5 commits August 27, 2026 08:55
The branch's four CLI pages were written against an older main; merging now so
the review fixes land on top of current content rather than diverging further.
Step 2 told readers `project add` would sign them in through the browser. It
does not. In keboola/cli main `project add` resolves a token from `--token`,
then `KBC_TOKEN`, then a hidden TTY prompt, and errors out. A reader arriving
without a token hit that prompt with nothing on the page telling them what to
type, and this branch had already removed the token-creation walkthrough.

Browser sign-in is the separate `auth login` + `auth register-projects` pair,
so document both routes and say which is which.

The Agentic CLI settings page is feature-flagged off until launch while these
docs deploy on merge, so the token route now also names Settings → API Tokens
as the way in for anyone who cannot see it. Drops the orphaned screenshot the
removed walkthrough used, and the internal VERIFY note that was shipping into
the public page.
The /kbagent:setup section said the command "signs you in", while the sentence
below it and the page's own HTML comment said the browser hands back to a
terminal. keboola/cli#704 settles it: no client completes the sign-in in-chat,
its table marks Claude Code "finishes: no". The command starts the browser
sign-in and the reader finishes it in their terminal.

Drops the customer-facing link to #704. It is an open issue whose title reads
as an admission that the documented flow does not work, and this page links no
other GitHub issues. The reference stays in the HTML comment.

The "other four clients" sentence said they cannot run the command and then
explained they cannot finish its sign-in, so it now just says they cannot run
it and points at the terminal route.

Also:
- `keboola-cli` is a current v1.1.1 project-review toolkit built on the older
  `kbc` sync CLI, not "the unrelated legacy kbc CLI".
- The terminal steps deferred to getting-started's Step 2 rather than forking
  its `project add` example under a different alias, and they no longer repeat
  the "no token needed interactively" claim that `project add` does not honour.
- The sign-in tip contradicted every client section's unconditional "Do the
  terminal steps above", so it is now conditional on `project list`.
- Two provenance comments were attesting reviews that did not cover what sits
  under them: Padak's v0.66.1 review predates the `keboola-claude-kit`
  marketplace, and "verified 2026-07-13" was covering 2026-08-26 claims.
Two pre-existing lines here disagreed with getting-started once Step 2 grew a
browser-login route. The "in one minute" block demanded `--token YOUR_TOKEN`
from a reader who has no token yet, and the page claimed kbagent "never needs
a browser" one section later.

The quickstart now signs in the way Step 2 does, and the browser sentence says
what is actually true: one browser round-trip to sign in, or a Storage API
token to skip it.
This branch pins the page to keboola/ai-kit and asserts a plugin count, so the
count has to be right. The live `keboola-claude-kit` marketplace ships seven
plugins, and none of them is named `developer`. The page promised four, told
readers to run `/plugin install developer`, and linked `plugins/developer`,
which 404s.

Documents the four plugins that were missing (keboola-cli, keboola-git,
sl-toolkit, powerbi-to-sl), drops the Developer Plugin section along with the
intro and best-practice sentences that referred to it, and qualifies every
install string with `@keboola-claude-kit` so they work as written.
@eruveo

eruveo commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

@Iamfle4ka @KaroEverling both reviews were right on the load-bearing claims. All twelve items are addressed across four commits. Base was behind main; merged (170b666, no conflicts).

On the /kbagent:setup contradiction you both raised: #704 is right and "signs you in" was wrong. Took @KaroEverling's wording. The command starts the browser sign-in, the reader finishes it in their terminal.

Iamfle4ka — blocking

# Item Done
1 project add is not the browser path Step 2 now documents both routes: auth login + auth register-projects (with --all) as the interactive path, project add --token as the static-token path, and says which resolves what.
2 /kbagent:setup "signs you in" Now "starts the browser sign-in… You finish the sign-in in your terminal."
3 AI Kit doesn't match the marketplace Four → seven plugins, no developer. Dropped that section and the intro/best-practice sentences referring to it; documented keboola-cli, keboola-git, sl-toolkit, powerbi-to-sl; every install string now carries @keboola-claude-kit.
4 Token creation dead-ends behind a flag Added the fallback to Settings → API Tokens (/management/project/tokens/) for anyone who can't see Agentic CLI. public/cli/token-create.png deleted.
5 cli/index.md contradicts Step 2 Quickstart signs in the way Step 2 does instead of demanding --token YOUR_TOKEN; "never needs a browser" replaced with what's actually true.
6 keboola-cli mischaracterized Took your wording: a separate project-review toolkit built on the older kbc sync CLI, and pick kbagent for the agent interface. Confirmed v1.1.1 in the live marketplace manifest.

Iamfle4ka — non-blocking

All four done: Windows $ver fallback (says to check echo $ver and set it by hand, since the empty-var failure surfaces as a bare uv 404); VERIFY(owner: Padak) gone and the #704 repro comment rewritten so it no longer contradicts the prose above it; both provenance comments now scope what they actually attest (Padak's v0.66.1 review predates keboola-claude-kit, so the install string is sourced to the marketplace manifest instead); the sign-in tip is conditional on project list and the terminal steps defer to Step 2 rather than forking its example under a different alias.

KaroEverling

  1. Resolved as above.
  2. Customer-facing Python backend size added #704 link removed from the prose; reference kept in the HTML comment only. Took your rewrite of the "other four clients" sentence verbatim — you were right that it said they can't run the command and then explained they can't finish its sign-in.

One thing I did not do, deliberately

Documenting auth login needs the gate's vendored _data/cli/command-reference.md to know the auth group. The checked-in copy is v0.76.1, which predates it, so check:cli flagged the new commands.

I bumped it to v0.91.0 locally to see what happens, then reverted: #1044 (chore: sync CLI command reference (v0.91.0)) is already open with exactly that bump, and it surfaces two unrelated regressions this PR has no business fixing — kbagent tool no longer exists in v0.91.0 (referenced in ai/mcp-server, cli/commands.md, cli/for-agents.md) and kbagent doctor --fix is gone (cli/troubleshooting.md ×2). I'd be guessing at what replaced them.

So the three new auth lines carry kbagent-check: skip-next with a comment naming #1044. Drop those three skips when #1044 merges. Worth noting #1044 will go red on the four tool/--fix findings until someone resolves them.

Spotted in passing, not changed

cli/getting-started still shows kbagent v0.66.0 in the --version sample while the marketplace ships kbagent 0.91.0. Neither of you raised it and it's a captured console sample, so I left it.

@KaroEverling on keboola_com#531 / #538: outside this repo, so I can't touch them from here. The wording to carry over is "starts the browser sign-in… You finish the sign-in in your terminal."

Checks

npm run build ✅ (361 pages) · node scripts/check-cli-reference.mjs ✅ 0 findings · all four changed pages render, including the restructured step lists and the #step-2--connect-your-project anchor. No em dashes added.

(The Astro dev server on :4321 was 500ing on every page including the homepage and untouched routes after the merge pulled in the new content files. Stale server, not this branch — verified against a fresh build of the same tree.)

The `auth login` lines carried `kbagent-check: skip-next` because the checked-in
reference was v0.76.1, which predates the `auth` group. Bumping it to v0.91.0
removes the reason for all three skips.

The two regressions that bump exposes both resolve by deletion rather than by
guessing at a replacement, which is what made them look risky:

- `kbagent tool` is gone, and "tool" appears nowhere in the v0.91.0 reference.
  Nothing succeeded it, so the three places offering it as a way to call MCP
  tools were offering a command that does not exist.
- `kbagent doctor` now takes no options at all, so `--fix` cannot be described
  as repairing anything. Troubleshooting says what doctor reports and leaves the
  fixes to the manual steps already listed under it.

The `--version` sample moves to v0.91.0 with the reference, and its provenance
comment stops pinning a version it no longer shows. The remaining skips are the
chat prompts, which are prompts rather than commands and always will be.
Removed on Michal's call. This reverses one of Jordan's two conditions for
merging #1015 (PRDCT-490, commit 968e0cb), so it is called out in the PR body
rather than left to be found in a diff.

Nothing else Keboola ships labels kbagent beta: the keboola.com /cli page does
not, keboola/cli's README does not, and the product badges the connect card NEW
where Kai gets a Beta badge. The docs carried the label alone.
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.

4 participants