Skip to content

Feat: Added kimi support#163

Closed
buriy wants to merge 1 commit intovakovalskii:mainfrom
buriy:feat/kimi-support
Closed

Feat: Added kimi support#163
buriy wants to merge 1 commit intovakovalskii:mainfrom
buriy:feat/kimi-support

Conversation

@buriy
Copy link
Copy Markdown
Contributor

@buriy buriy commented Apr 10, 2026

No description provided.

@buriy buriy force-pushed the feat/kimi-support branch 7 times, most recently from 67d9b8f to fb6f745 Compare April 10, 2026 17:07
@vakovalskii
Copy link
Copy Markdown
Owner

Conflicts after merging recent PRs (#165 Kilo, #177 Linux focus, #172 WSL). Please rebase on latest main.

@vakovalskii
Copy link
Copy Markdown
Owner

Still conflicts. Please rebase on latest main.

@buriy buriy force-pushed the feat/kimi-support branch from fb6f745 to a3bc2ec Compare April 17, 2026 21:14
@buriy
Copy link
Copy Markdown
Contributor Author

buriy commented Apr 17, 2026

@vakovalskii rebased

@buriy
Copy link
Copy Markdown
Contributor Author

buriy commented Apr 17, 2026

also pls see/check active sessions search speedup w/ caching : using /proc instead of lsof + 5 seconds caching

Copy link
Copy Markdown
Collaborator

@NovakPAai NovakPAai left a comment

Choose a reason for hiding this comment

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

Code Review

⚠️ Installation command may be incorrect — README and app.js both show:

pip install kimi-cli

But Kimi CLI (MoonshotAI) appears to be a Python package, so pip is correct per the GitHub repo. However, the app.js install entry also adds uv tool install kimi-cli as an alternative — that's a good addition. Please double-check the package name on PyPI to make sure it matches exactly.

⚠️ Agent count mismatch across docs — the PR updates different files with different counts:

  • ARCHITECTURE.md: "7 agents" (adds Kimi)
  • docs/README_RU.md: "6 агентов"
  • docs/README_ZH.md: "6 个代理"
  • README.md: table header unchanged

All four files should agree on the same count.

⚠️ ARCHITECTURE.md diagram still shows "reads from 6 locations" but the PR adds ~/.kimi/sessions/ as a 7th location. The count in the ASCII diagram needs to be updated too.

ℹ️ The kimi -r <sessionId> resume command — please confirm this is the correct CLI flag for Kimi Code. The similar Codex uses codex resume, Claude uses --resume. If the flag is wrong, the "Resume" button in the UI and the terminal integration will silently do nothing useful.

@vakovalskii
Copy link
Copy Markdown
Owner

Thanks for the contribution! Closing for now — we're keeping the agent roster focused (Claude Code, Codex, Cursor, OpenCode, Kiro + adding Qwen and GitHub Copilot Chat this round). Kimi is on the radar but we don't want to grow past ~7-8 agents without a pluggable agent API. Happy to revisit if there's strong user demand — feel free to reopen or discuss in an issue.

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.

3 participants