Skip to content

docs(kilo-docs): document upstream v1.17.13 behavior - #12900

Merged
johnnyeric merged 1 commit into
mainfrom
johnnyeric/docs-upstream-v1-17-13
Aug 5, 2026
Merged

docs(kilo-docs): document upstream v1.17.13 behavior#12900
johnnyeric merged 1 commit into
mainfrom
johnnyeric/docs-upstream-v1-17-13

Conversation

@johnnyeric

@johnnyeric johnnyeric commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Issue

Related to #12695. This is a documentation follow-up for the user-facing changes imported by that upstream merge; there is no standalone issue.

Context

Audit and align Kilo's documentation and merge changeset with the behavior that actually shipped from OpenCode v1.17.9 through v1.17.13. The documentation must not present upstream-only v2 infrastructure, transparent fixes, existing Kilo checkpoint behavior, or unsupported interfaces as new public features.

Implementation

Document the bindable Move Session and diff viewer actions, remote skill versioning and atomic refresh semantics, and MCP server instructions and resources. Preserve Kilo's underscore MCP tool naming and correct the merge changeset to exclude the pre-existing snapshot/revert flow and unsupported kilo run --mini interface.

Screenshots / Video

N/A — documentation-only changes.

How to Test

Manual/local verification

  • Agent: ran bun run lint in packages/kilo-docs (0 errors; existing warnings remain).
  • Agent: ran bun run typecheck in packages/kilo-docs.
  • Agent: ran bun run test in packages/kilo-docs (22 tests passed).
  • Agent: ran the Markdown table-padding guard for the four touched documentation pages and git diff --check.
  • Agent: the pre-push repository typecheck completed successfully (28/28 tasks).

Reviewer test steps

  1. Compare the changeset and updated pages with the Kilo behavior imported by Opencode Merge v1.17.10-v1.17.13 #12695.
  2. Confirm the pages document only shipped user-facing behavior and retain Kilo's underscore MCP tool naming.
  3. Confirm the generated CLI reference still does not expose kilo run --mini.

Blocked checks and substitute verification

  • None.

Checklist

  • Issue linked above, or exception explained
  • Tests/verification described
  • Screenshots/video included for visual changes, or marked N/A
  • Changeset considered for user-facing changes
  • I personally reviewed the diff and can explain the changes, including any AI-assisted work.

@johnnyeric
johnnyeric marked this pull request as ready for review August 5, 2026 14:29
@kilo-code-bot

kilo-code-bot Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

lgtm — docs-only PR. Every documented claim was verified against the shipped implementation:

  • /move and /diff slash commands exist (packages/tui/src/component/prompt/index.tsx, diff-viewer.tsx)
  • diff_open / session_move keybinds exist and default to none (unbound), and <leader> syntax is valid (packages/tui/src/config/keybind.ts)
  • Diff viewer sources match: working tree, main branch, last turn
  • MCP server instructions are injected into the system prompt and omitted when every tool from that server is denied (session/system.ts)
  • list_mcp_resources, list_mcp_resource_templates, read_mcp_resource tool names and their read-permission mapping are correct (session/tools.ts, permission/index.ts)
  • Remote skill version field is optional and refreshes via staged download + atomic swap, keeping the previous cache on failure (skill/discovery.ts)
  • Changeset removals are accurate: Kilo keeps underscore MCP tool naming (server_tool, not mcp__server__tool), kilo run --mini is explicitly rejected (--mini only works without the run subcommand), and the snapshot/revert flow pre-existed this merge
Files Reviewed (5 files)
  • .changeset/opencode-v1-17-9-to-v1-17-13.md
  • packages/kilo-docs/pages/automate/mcp/using-in-cli.md
  • packages/kilo-docs/pages/automate/mcp/using-in-kilo-code.md
  • packages/kilo-docs/pages/code-with-ai/platforms/cli.md
  • packages/kilo-docs/pages/customize/skills.md

Reviewed by kimi-k3 · Input: 105.7K · Output: 14.7K · Cached: 1.4M

Review guidance: REVIEW.md from base branch main

@johnnyeric
johnnyeric merged commit efbae40 into main Aug 5, 2026
24 checks passed
@johnnyeric
johnnyeric deleted the johnnyeric/docs-upstream-v1-17-13 branch August 5, 2026 14:57
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…eam-v1-17-13

docs(kilo-docs): document upstream v1.17.13 behavior
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