Skip to content

fix(agent-manager): reference update-from-base in introduction - #13753

Merged
marius-kilocode merged 7 commits into
mainfrom
agent-manager-intro-base-update
Sep 4, 2026
Merged

marius-kilocode merged 7 commits into
mainfrom
agent-manager-intro-base-update

Conversation

@marius-kilocode

@marius-kilocode marius-kilocode commented Sep 3, 2026 •

Copy link
Copy Markdown
Collaborator

What Problem This Solves

Beginners can reach conflicts through either Agent Manager return path: Apply can conflict in Local, while a pull request can require conflict resolution on GitHub. The onboarding did not explain that Kilo can resolve those conflicts in the worktree before either step.

Why This Change Was Made

Add Resolve conflicts with Kilo /update-from-base as a visible onboarding item. Its explanation tells users to run the command in that worktree session before applying changes to Local or merging a pull request. Kilo then merges the latest base changes and resolves conflicts in the isolated worktree first.

User Impact

New Agent Manager sessions now teach where conflict resolution should happen and how to delegate it to Kilo. Compact layouts open the explanation in a popover; larger layouts show it inline. The copy is localized in all 21 Agent Manager locales.

Evidence

  • Host and webview type checks, lint, bundling, Knip, and the ownership-marker guard pass.
  • 33 focused i18n and slash-command tests pass.
  • The Linux visual-regression workflow regenerated and passed the Agent Manager introduction baseline. The final image was reviewed locally.
  • Verified the compact onboarding item and popover plus the larger inline layout in isolated VS Code. No command was executed. The isolated instance was cleaned up.
  • VS Code logged only its existing Node url.parse() deprecation warning during the manual check.

Agent Manager onboarding explaining how Kilo resolves conflicts in a worktree before Apply or pull request merge

@kilo-code-bot

kilo-code-bot Bot commented Sep 3, 2026 •

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (22 files)
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/agentmanager/introduction-chromium-linux.png
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ar.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/br.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/bs.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/da.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/de.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/en.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/es.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/fa.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/fr.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/it.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ja.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ko.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/nl.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/no.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/pl.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ru.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/th.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/tr.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/uk.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/zh.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/zht.ts
Previous Review Summary (commit 0225da3)

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

Previous review (commit 0225da3)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (21 files)
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ar.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/br.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/bs.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/da.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/de.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/en.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/es.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/fa.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/fr.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/it.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ja.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ko.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/nl.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/no.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/pl.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ru.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/th.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/tr.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/uk.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/zh.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/zht.ts

Reviewed by grok-4.6 · Input: 81.2K · Output: 7.2K · Cached: 263.2K

Review guidance: REVIEW.md from base branch main

@marius-kilocode
marius-kilocode force-pushed the agent-manager-intro-base-update branch from a160a5f to a780b41 Compare September 4, 2026 08:15
@marius-kilocode
marius-kilocode force-pushed the agent-manager-intro-base-update branch from 6df58ee to 0d5ca6e Compare September 4, 2026 08:35
@marius-kilocode
marius-kilocode force-pushed the agent-manager-intro-base-update branch from 0225da3 to 2829b2f Compare September 4, 2026 08:52
@marius-kilocode
marius-kilocode force-pushed the agent-manager-intro-base-update branch from 1838e35 to 7ba7941 Compare September 4, 2026 08:59
@marius-kilocode
marius-kilocode merged commit 0f3ad38 into main Sep 4, 2026
27 checks passed
@marius-kilocode
marius-kilocode deleted the agent-manager-intro-base-update branch September 4, 2026 09:41
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