docs: plan Kimi Code history import - #1
Conversation
🤖 CodeAnt AI — Review Status
|
Thanks for using CodeAnt! 🎉We're free for open-source projects. if you're enjoying it, help us grow by sharing. Share on X · |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_8a05266f-f076-4337-af50-e7f944f5444f) |
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewTip of the day💡 Did you know, you can switch off images and animations for a plain-text comment |
PR Summary by QodoPlan Kimi Code historical import with contract gate and fixture-first testing
AI Description
Diagram
High-Level Assessment
Files changed (1)
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3ac55ad393
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (1 file)
Previous Review Summaries (5 snapshots, latest commit bc37d1d)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit bc37d1d)Status: 1 Warning Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
Files Reviewed (4 files)
Fix these issues in Kilo Cloud Previous review (commit 0a4cf79)Status: 1 Warning Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
Files Reviewed (1 file)
Fix these issues in Kilo Cloud Previous review (commit ed3a855)Status: 2 Warnings Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
Files Reviewed (1 file)
Fix these issues in Kilo Cloud Previous review (commit 168ea6f)Status: No Issues Found | Recommendation: Merge Files Reviewed (1 file)
Review conducted by stepfun/step-3.7-flash:free (medium-reasoning model). Resolution of Previous FindingsAll 3 WARNINGs from the prior review at
A new "Documentation lifecycle" section correctly prescribes promotion to Previous review (commit 3ac55ad)Status: 3 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
Files Reviewed (1 file)
Review conducted by stepfun/step-3.7-flash:free (medium-reasoning model). Additional ContextThe Reviewed by step-3.7-flash:free · Input: 83.7K · Output: 3.9K · Cached: 266.6K |
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bc37d1d491
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ct.ThrowIfCancellationRequested(); | ||
| if (!Guid.TryParse(candidate.DashedId, out var guid)) continue; | ||
| var sessionId = guid.ToString("N"); | ||
| if (!seen.Add(sessionId) || (sessionFilter is not null && !string.Equals(sessionId, sessionFilter, StringComparison.Ordinal))) continue; |
There was a problem hiding this comment.
Deduplicate Kimi candidates only after filtering
When the same session UUID exists under both supported roots and the first .kimi-code copy does not match --cwd while the legacy .kimi copy does, this marks the UUID as seen before reading metadata and applying the filter. Since .kimi-code candidates are enumerated first, the matching legacy copy is then skipped, causing the requested session to disappear from the import; mark the UUID only after accepting a candidate, or group duplicate candidates before filtering.
Useful? React with 👍 / 👎.
| [](#) | ||
|
|
||
| **Kurrent Capacitor** (`kcap`) records your coding-agent sessions and forwards them to a Capacitor server, where a real-time dashboard and PR-review tools surface the context no diff can give you: *why* code changed, what alternatives were weighed, and how it was actually built. It works across nine agents — Claude Code, Codex, Cursor, GitHub Copilot, Gemini, Kiro, Pi, OpenCode, and Antigravity — capturing the full picture: session lifecycle, transcripts, subagent trees, tool calls, and token usage. | ||
| **Kurrent Capacitor** (`kcap`) records your coding-agent sessions and forwards them to a Capacitor server, where a real-time dashboard and PR-review tools surface the context no diff can give you: *why* code changed, what alternatives were weighed, and how it was actually built. It works across ten agents — Claude Code, Codex, Cursor, GitHub Copilot, Gemini, Kiro, Kimi Code, Pi, OpenCode, and Antigravity — capturing the full picture: session lifecycle, transcripts, subagent trees, tool calls, and token usage. |
There was a problem hiding this comment.
Qualify Kimi support as historical-only
For Kimi users, this opening claim groups Kimi with the agents that kcap records automatically and says it captures the complete lifecycle in real time, but this change deliberately adds only historical import: Kimi is absent from the harness catalog and plugin/setup paths, and the later Kimi section explicitly says no plugin or hook is installed. This can lead users to expect new Kimi sessions to be recorded after setup, so the headline should distinguish historical-only Kimi support until live capture is implemented.
Useful? React with 👍 / 👎.
|
|
||
| ## Documentation lifecycle | ||
|
|
||
| This is an unexecuted implementation plan and remains in |
There was a problem hiding this comment.
[WARNING]: Plan doc is stale — claims "unexecuted" but implementation is already in this PR
Line 300 says "This is an unexecuted implementation plan", yet KimiImportSource.cs now contains the child-stream import implementation (ImportChildrenAsync, AttachesChildContentOnReplay => true, failOnError: true), Program.cs registers KimiImportSource, VendorSelection.cs exposes --kimi, and integration/unit tests verify the behavior. The plan's own "Documentation lifecycle" section prescribes removal once executed; this file should be deleted and its validated architecture/contract moved to docs/superpowers/specs/ with execution evidence recorded in docs/CHANGES.md.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
The plan was written into kurrent-io/kcap-cli's own docs/superpowers/plans/ directory, which is upstream's design-doc folder (referenced by their CLAUDE.md), not ours. Phase 1 — the historical importer — is implemented and tested, so the plan has served its purpose here. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KbSTrWJs9TnWBjhXwH8rpn
Thanks for using CodeAnt! 🎉We're free for open-source projects. if you're enjoying it, help us grow by sharing. Share on X · |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_07914f3d-de4c-4340-bead-af3f3ac1c3ec) |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e5facd35fe
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| if (!await PostAsync(ctx.HttpClient, ctx.BaseUrl, "session-start/kimi", start, ct)) return ImportOutcome.Failed; | ||
| var startLine = classification.Status switch { ImportCommand.ClassificationStatus.Partial => classification.ResumeFromLine, ImportCommand.ClassificationStatus.AlreadyLoaded => classification.TotalLines, _ => 0 }; | ||
| int sent; | ||
| try { sent = await SessionImporter.SendTranscriptBatches(ctx.HttpClient, ctx.BaseUrl, classification.SessionId, path, null, startLine, vendor: Vendor, failOnError: true); } |
There was a problem hiding this comment.
Exclude incomplete JSONL tails from strict batches
When an active Kimi process is partway through appending its final JSON object—or a crashed session leaves a truncated tail—SendTranscriptBatches treats that unterminated nonblank text as a complete line and this strict send makes the server reject the batch during normalization. That prevents even the valid records earlier in the same batch from being imported, and a permanently truncated session can never succeed; validate or defer an incomplete tail while preserving its physical line index.
Useful? React with 👍 / 👎.
|
Reviewed the diff directly (checks are meaningless today — org-wide Actions billing block). Major diff-vs-claim mismatch — this is the headline finding. The PR title is "docs: plan Kimi Code history import" and the (human-written) User description says explicitly:
That is not what's in the diff. The actual changeset is +727/-23 and includes a complete, working importer:
The PR's own auto-generated CodeAnt-AI description (further down the body) correctly describes this as a shipped feature (" Recommendation: do not merge. Reconcile the title/description with the actual diff first — if the intent is really to ship the Kimi importer now, retitle/rebody the PR accordingly and let it get reviewed as an implementation PR (the code itself looks reasonably careful — path-layout handling for both Kimi Code and legacy Kimi, idempotent resume-from-watermark, child-lifecycle repair without resending accepted content — but that's exactly the kind of change that needs an implementation review, not a docs-only rubber stamp). If the intent was genuinely to land only the plan first, this PR needs to be split. No CI signal used in this assessment (checks are known-broken org-wide today) — this is a diff-content finding, independent of that. |
User description
Summary
Validation
This PR intentionally contains no importer implementation and is the review gate before implementation begins.
CodeAnt-AI Description
Import Kimi Code session history, including child agents
What Changed
kcap importnow discovers Kimi Code sessions from both current and legacy transcript locations.--kimiselects Kimi history imports without adding Kimi to live setup or hook installation.Impact
✅ Kimi Code history appears in the dashboard✅ Child-agent transcripts remain attached to their parent session✅ Fewer duplicate uploads and recoverable child import failures💡 Usage Guide
Checking Your Pull Request
Every time you make a pull request, our system automatically looks through it. We check for security issues, mistakes in how you're setting up your infrastructure, and common code problems. We do this to make sure your changes are solid and won't cause any trouble later.
Talking to CodeAnt AI
Got a question or need a hand with something in your pull request? You can easily get in touch with CodeAnt AI right here. Just type the following in a comment on your pull request, and replace "Your question here" with whatever you want to ask:
This lets you have a chat with CodeAnt AI about your pull request, making it easier to understand and improve your code.
Example
Preserve Org Learnings with CodeAnt
You can record team preferences so CodeAnt AI applies them in future reviews. Reply directly to the specific CodeAnt AI suggestion (in the same thread) and replace "Your feedback here" with your input:
This helps CodeAnt AI learn and adapt to your team's coding style and standards.
Example
Retrigger review
Ask CodeAnt AI to review the PR again, by typing:
Check Your Repository Health
To analyze the health of your code repository, visit our dashboard at https://app.codeant.ai. This tool helps you identify potential issues and areas for improvement in your codebase, ensuring your repository maintains high standards of code health.