Repository navigation
docs(migration): record U-E0-02 to U-E0-04 status and the #15018 follow-up - #15263
Conversation
|
➖ v2 migration progress: 0.9% → 0.9% (0 done, 0 total, 0 dropped; 86/9829 cells) Placement: 0 new marked lines in shared files, 0 new Ratchets: 9 core-internal imports (base 9); shared patches by rung 28 seam, 2 patch (base 28 seam, 2 patch). No change to the progress map. |
| | U-E0-02 | Kilo RPC and plugin scaffolding (`packages/schema/src/kilocode`, plugin `define`, generated client) | #15173 | pre-gate | U-E0-01 | native mechanism | n/a | yes H | 1 plugin | One sample Kilo RPC registered as a plugin round-trips through the generated client; `bun run generate` leaves no drift | todo | | ||
| | U-E0-03 | Re-land Kilo `AGENTS.md`; rewrite `v2-fork-conventions.md` and `script/v2-baseline.ts` (README, banners and drops done by this plan, Appendix D) | #15175 | pre-gate | — | needs-port | n/a | no | n/a | `AGENTS.md` names the integration branch `origin/v2`, BASE and branch rules; conventions carry the base pin, no-prefix branches, `services/www`; `v2-baseline.ts` runs at BASE (pin-is-ancestor assertion, mirrors `check.yml`/`test.yml`); fork conventions carry the plugin-first rung table (landed with this change) | todo | | ||
| | U-E0-02 | Kilo RPC and plugin scaffolding (`packages/schema/src/kilocode`, plugin `define`, generated client) | #15173 | pre-gate | U-E0-01 | native mechanism | n/a | yes H | 1 plugin | One sample Kilo RPC registered as a plugin round-trips through the generated client; `bun run generate` leaves no drift | done: #15261 merged (`68536a63da`, 2026-10-19). The Gateway RPC contract is in `packages/schema/src/kilocode/gateway.ts` and the adapter in `packages/kilo-plugin/src/gateway/`; `packages/kilo-cli/test/rpc.test.ts` calls it through the generated promise client against a host-booted server; `bun run generate` leaves no drift; shared patches 30 files, 305 marked, 51 unmarked before, 29, 304, 50 after | | ||
| | U-E0-03 | Re-land Kilo `AGENTS.md`; rewrite `v2-fork-conventions.md` and `script/v2-baseline.ts` (README, banners and drops done by this plan, Appendix D) | #15175 | pre-gate | — | needs-port | n/a | no | n/a | `AGENTS.md` names the integration branch `origin/v2`, BASE and branch rules; conventions carry the base pin, no-prefix branches, `services/www`; `v2-baseline.ts` runs at BASE (pin-is-ancestor assertion, mirrors `check.yml`/`test.yml`); fork conventions carry the plugin-first rung table (landed with this change) | review: #15259 | |
There was a problem hiding this comment.
SUGGESTION: U-E0-03 is already merged, so review: #15259 is stale
#15259 merged at 2026-10-20T00:12:45Z, and this PR's base is that merge commit (d011f04b), so the unit's deliverables (AGENTS.md, v2-fork-conventions.md, migration-tracking/technical-notes/script/v2-baseline.ts) are already on v2. Recording review understates the map at merge time. If the acceptance evidence holds, record done: with the merge SHA instead, and drop "PR open" from the PR body table.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (1 file)
Verification notes: All evidence in the incremental commit checks out against the repository. Only Previous Review Summary (commit bc16ba7)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit bc16ba7)Status: 1 Issue Found | Recommendation: Merge after addressing the SUGGESTION Overview
Issue Details (click to expand)SUGGESTION
Files Reviewed (1 file)
Verification notes: The U-E0-02 evidence checks out against the repository: #15261 merged as Reviewed by deepseek-v4.1-flash-1017 · Input: 47.9K · Output: 10.8K · Cached: 663.8K Review guidance: REVIEW.md from base branch |
Coordinator status micro-PR for
migration-tracking/plans/kilo-v2-migration-plan.md.tododone68536a63da). The RPC contract moved to Schema and the adapter tokilo-plugin. A round-trip test runs through the generated promise client.bun run generateleaves no drift. Shared patches went from 30 files and 305 marked lines to 29 and 304todoreview: #15259activeactiverpc.callrouteAfter merge, the user can close #15173 (U-E0-02). This PR does not close it.
Checks:
bun script/upstream/progress.ts --checkreports the map is up to date. No Changeset.Second commit, after
origin/v2was merged into this branch:review: #15259doned011f04bdf)activedoneon evidencecheck.yml,test.ymlandkilo-v2.ymlremain. They run on every PR and on pushes tov2(bun run checkand the package tests). The upstream publish and deploy workflows are deleted, andcheck-workflows.tsguards the list. No remaining workflow publishes or deploys. All three were green on the last two pushes tov2After merge, the user can close #15175 (U-E0-03) and #15176 (U-E0-04).