Skip to content

fix(vscode): replace opencode references with kilo.jsonc in settings UI#7845

Open
kilo-code-bot[bot] wants to merge 2 commits intomainfrom
fix/mcp-settings-opencode-references
Open

fix(vscode): replace opencode references with kilo.jsonc in settings UI#7845
kilo-code-bot[bot] wants to merge 2 commits intomainfrom
fix/mcp-settings-opencode-references

Conversation

@kilo-code-bot
Copy link
Copy Markdown
Contributor

@kilo-code-bot kilo-code-bot bot commented Mar 28, 2026

Summary

  • Replace all opencode.json / opencode config file references with kilo.jsonc in the VS Code extension settings UI i18n strings across all 18 locale files
  • Add hint text in the MCP empty state message suggesting users can ask the agent to add an MCP server for them
  • Affected i18n keys: settings.agentBehaviour.mcpEmpty, settings.agentBehaviour.workflows.description, settings.agentBehaviour.workflows.empty, dialog.plugins.empty, error.chain.checkConfig

Related: #7839

kilo-code-bot bot and others added 2 commits March 28, 2026 09:04
…UI i18n strings

Replace 'opencode.json' and 'opencode config file' references with 'kilo.jsonc'
across all 18 i18n locale files for MCP empty state, plugins empty state,
error config check, and workflow description/empty state messages. Also add
text suggesting users can ask the agent to add an MCP server for them.
"settings.agentBehaviour.addMcp.url.placeholder": "e.g. http://localhost:3000/sse",
"settings.agentBehaviour.mcpEmpty": "No MCP servers configured. Edit the opencode config file to add MCP servers.",
"settings.agentBehaviour.mcpEmpty":
"No MCP servers configured. Edit kilo.jsonc to add MCP servers, or ask the agent to add one for you.",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

WARNING: This hard-codes the wrong config filename for the VS Code extension

The extension still resolves and detects MCP config from kilo.json (packages/kilo-vscode/src/services/marketplace/paths.ts and packages/kilo-vscode/src/services/marketplace/detection.ts), so telling users here to edit kilo.jsonc can send them to a file that this UI will not pick up. This same mcpEmpty copy was changed in every locale, so the mismatch will be user-visible across all translations.

@kilo-code-bot
Copy link
Copy Markdown
Contributor Author

kilo-code-bot bot commented Mar 28, 2026

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/i18n/en.ts 1119 mcpEmpty now points users to kilo.jsonc, but the VS Code extension still resolves/detects MCP config from kilo.json paths; the same mismatch was introduced across all localized mcpEmpty strings.
Other Observations (not in diff)

None.

Files Reviewed (19 files)
  • packages/kilo-vscode/tests/visual-regression.spec.ts-snapshots/settings/agent-behaviour-workflows-empty-chromium-linux.png
  • packages/kilo-vscode/webview-ui/src/i18n/ar.ts
  • packages/kilo-vscode/webview-ui/src/i18n/br.ts
  • packages/kilo-vscode/webview-ui/src/i18n/bs.ts
  • packages/kilo-vscode/webview-ui/src/i18n/da.ts
  • packages/kilo-vscode/webview-ui/src/i18n/de.ts
  • packages/kilo-vscode/webview-ui/src/i18n/en.ts - 1 warning
  • packages/kilo-vscode/webview-ui/src/i18n/es.ts
  • packages/kilo-vscode/webview-ui/src/i18n/fr.ts
  • packages/kilo-vscode/webview-ui/src/i18n/ja.ts
  • packages/kilo-vscode/webview-ui/src/i18n/ko.ts
  • packages/kilo-vscode/webview-ui/src/i18n/nl.ts
  • packages/kilo-vscode/webview-ui/src/i18n/no.ts
  • packages/kilo-vscode/webview-ui/src/i18n/pl.ts
  • packages/kilo-vscode/webview-ui/src/i18n/ru.ts
  • packages/kilo-vscode/webview-ui/src/i18n/th.ts
  • packages/kilo-vscode/webview-ui/src/i18n/tr.ts
  • packages/kilo-vscode/webview-ui/src/i18n/zh.ts
  • packages/kilo-vscode/webview-ui/src/i18n/zht.ts

Reviewed by gpt-5.4-20260305 · 1,049,271 tokens

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.

0 participants