Skip to content

docs: clarify marketplace installs - #11639

Merged
markijbema merged 10 commits into
mainfrom
mark/clarify-marketplace
Jun 26, 2026
Merged

docs: clarify marketplace installs#11639
markijbema merged 10 commits into
mainfrom
mark/clarify-marketplace

Conversation

@markijbema

Copy link
Copy Markdown
Contributor

What

Clarify Marketplace installs in the app and docs.

  • Add Marketplace intro copy and docs links in the VS Code webview.
  • Expand the install modal with item-type explanations, project/global scope guidance, destination paths, and MCP/project warnings.
  • Keep the install success modal open long enough to show where the item was installed.
  • Add a Marketplace docs page covering agents, skills, MCP servers, scopes, changed files, removal, and MCP permissions.
  • Add localized strings for the new Marketplace copy.

Why

The Marketplace did not explain what MCP means, how project vs global installs differ, or what files are changed. This makes install decisions unclear, especially for MCP servers that may require credentials or expose powerful tools.

Validation

  • bun test tests/unit/i18n-keys.test.ts tests/unit/marketplace-panel-arch.test.ts from packages/kilo-vscode
  • bun run typecheck from packages/kilo-vscode
  • bun run lint from packages/kilo-vscode
  • bun run build-storybook from packages/kilo-vscode
  • bun run typecheck from packages/kilo-docs
  • bun run script/check-md-table-padding.ts from repo root
  • bun run knip && bun run check-kilocode-change from packages/kilo-vscode

@markijbema
markijbema force-pushed the mark/clarify-marketplace branch from 942b6cc to 39f25ee Compare June 24, 2026 16:03
@markijbema
markijbema changed the base branch from main to docs/marketplace-page June 24, 2026 16:03
Base automatically changed from docs/marketplace-page to main June 24, 2026 16:16
@markijbema
markijbema marked this pull request as ready for review June 25, 2026 13:50
@markijbema
markijbema enabled auto-merge June 25, 2026 13:51
Comment thread packages/kilo-vscode/webview-ui/src/components/marketplace/InstallModal.tsx Outdated
@kilo-code-bot

kilo-code-bot Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: 1 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0

Fix these issues in Kilo Cloud

Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-vscode/webview-ui/src/components/marketplace/InstallModal.tsx 136 The new installed-at copy still hardcodes Unix-style global paths, so Windows users are told the item was installed to a location that does not exist.
Files Reviewed (33 files)
  • .changeset/clear-marketplace-installs.md - 0 issues
  • packages/kilo-docs/lychee.toml - 0 issues
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/marketplace/empty-list-chromium-linux.png - 0 issues
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/marketplace/install-mcp-modal-chromium-linux.png - 0 issues
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/marketplace/mixed-list-with-items-chromium-linux.png - 0 issues
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/marketplace/relevant-items-chromium-linux.png - 0 issues
  • packages/kilo-docs/source-links.md - 0 issues
  • packages/kilo-i18n/src/ar.ts - 0 issues
  • packages/kilo-i18n/src/br.ts - 0 issues
  • packages/kilo-i18n/src/bs.ts - 0 issues
  • packages/kilo-i18n/src/da.ts - 0 issues
  • packages/kilo-i18n/src/de.ts - 0 issues
  • packages/kilo-i18n/src/en.ts - 0 issues
  • packages/kilo-i18n/src/es.ts - 0 issues
  • packages/kilo-i18n/src/fr.ts - 0 issues
  • packages/kilo-i18n/src/it.ts - 0 issues
  • packages/kilo-i18n/src/ja.ts - 0 issues
  • packages/kilo-i18n/src/ko.ts - 0 issues
  • packages/kilo-i18n/src/nl.ts - 0 issues
  • packages/kilo-i18n/src/no.ts - 0 issues
  • packages/kilo-i18n/src/pl.ts - 0 issues
  • packages/kilo-i18n/src/ru.ts - 0 issues
  • packages/kilo-i18n/src/th.ts - 0 issues
  • packages/kilo-i18n/src/tr.ts - 0 issues
  • packages/kilo-i18n/src/uk.ts - 0 issues
  • packages/kilo-i18n/src/zh.ts - 0 issues
  • packages/kilo-i18n/src/zht.ts - 0 issues
  • packages/kilo-vscode/webview-ui/src/components/marketplace/InstallModal.tsx - 1 issue
  • packages/kilo-vscode/webview-ui/src/components/marketplace/MarketplaceListView.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/src/components/marketplace/MarketplaceView.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/src/components/marketplace/marketplace.css - 0 issues
  • packages/kilo-vscode/webview-ui/src/context/language.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/src/stories/marketplace.stories.tsx - 0 issues
Previous Review Summaries (3 snapshots, latest commit 61e87f0)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 61e87f0)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 files)
  • packages/kilo-vscode/webview-ui/src/components/marketplace/InstallModal.tsx

Previous review (commit 5d5b2b9)

Status: 1 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-vscode/webview-ui/src/components/marketplace/InstallModal.tsx 128 The new success-path snapshot still records Unix-style paths, so Windows users are told the item was installed to a path that does not exist.

Fix these issues in Kilo Cloud

Files Reviewed (2 files)
  • packages/kilo-docs/lychee.toml - 0 issues
  • packages/kilo-vscode/webview-ui/src/components/marketplace/InstallModal.tsx - 1 issue

Previous review (commit a4b458d)

Status: 1 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-vscode/webview-ui/src/components/marketplace/InstallModal.tsx 262 Success state can report the wrong install destination if scope or method changes while the request is still in flight.

Fix these issues in Kilo Cloud

Files Reviewed (31 files)
  • .changeset/clear-marketplace-installs.md - 0 issues
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/marketplace/empty-list-chromium-linux.png - 0 issues
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/marketplace/install-mcp-modal-chromium-linux.png - 0 issues
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/marketplace/mixed-list-with-items-chromium-linux.png - 0 issues
  • packages/kilo-docs/source-links.md - 0 issues
  • packages/kilo-i18n/src/ar.ts - 0 issues
  • packages/kilo-i18n/src/br.ts - 0 issues
  • packages/kilo-i18n/src/bs.ts - 0 issues
  • packages/kilo-i18n/src/da.ts - 0 issues
  • packages/kilo-i18n/src/de.ts - 0 issues
  • packages/kilo-i18n/src/en.ts - 0 issues
  • packages/kilo-i18n/src/es.ts - 0 issues
  • packages/kilo-i18n/src/fr.ts - 0 issues
  • packages/kilo-i18n/src/it.ts - 0 issues
  • packages/kilo-i18n/src/ja.ts - 0 issues
  • packages/kilo-i18n/src/ko.ts - 0 issues
  • packages/kilo-i18n/src/nl.ts - 0 issues
  • packages/kilo-i18n/src/no.ts - 0 issues
  • packages/kilo-i18n/src/pl.ts - 0 issues
  • packages/kilo-i18n/src/ru.ts - 0 issues
  • packages/kilo-i18n/src/th.ts - 0 issues
  • packages/kilo-i18n/src/tr.ts - 0 issues
  • packages/kilo-i18n/src/uk.ts - 0 issues
  • packages/kilo-i18n/src/zh.ts - 0 issues
  • packages/kilo-i18n/src/zht.ts - 0 issues
  • packages/kilo-vscode/webview-ui/src/components/marketplace/InstallModal.tsx - 1 issue
  • packages/kilo-vscode/webview-ui/src/components/marketplace/MarketplaceListView.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/src/components/marketplace/MarketplaceView.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/src/components/marketplace/marketplace.css - 0 issues
  • packages/kilo-vscode/webview-ui/src/context/language.tsx - 0 issues
  • packages/kilo-vscode/webview-ui/src/stories/marketplace.stories.tsx - 0 issues

Reviewed by gpt-5.4-20260305 · Input: 328.1K · Output: 15.4K · Cached: 719.9K

Review guidance: REVIEW.md from base branch main

@markijbema
markijbema disabled auto-merge June 25, 2026 15:37
Mark IJbema and others added 2 commits June 25, 2026 17:54
# Conflicts:
#	packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/marketplace/empty-list-chromium-linux.png
#	packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/marketplace/mixed-list-with-items-chromium-linux.png
#	packages/kilo-vscode/webview-ui/src/components/marketplace/MarketplaceListView.tsx
@markijbema
markijbema merged commit 1a51b09 into main Jun 26, 2026
28 checks passed
@markijbema
markijbema deleted the mark/clarify-marketplace branch June 26, 2026 08:21
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
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.

2 participants