Skip to content

docs(migration): record U-E0-02 to U-E0-04 status and the #15018 follow-up - #15263

Merged
fpliger merged 3 commits into
v2from
coord-status-e0-02
Oct 20, 2026
Merged

fpliger merged 3 commits into
v2from
coord-status-e0-02

Conversation

@fpliger

@fpliger fpliger commented Oct 20, 2026 •

Copy link
Copy Markdown

Coordinator status micro-PR for migration-tracking/plans/kilo-v2-migration-plan.md.

Row Before After Evidence
U-E0-02 todo done #15261 merged (68536a63da). The RPC contract moved to Schema and the adapter to kilo-plugin. A round-trip test runs through the generated promise client. bun run generate leaves no drift. Shared patches went from 30 files and 305 marked lines to 29 and 304
U-E0-03 todo review: #15259 PR open
U-E1-07 active active Adds the placement follow-up for the #15018 lines, delegated to its author by the user's decision. The lines are rung-4 patches under strategy §5 but registered as rung 3
§A10 Q-G3-02 open open Adds the #15261 finding. Plugin RPC methods are not in the generated OpenAPI or SDK. Clients call them through the generic rpc.call route

After merge, the user can close #15173 (U-E0-02). This PR does not close it.

Checks: bun script/upstream/progress.ts --check reports the map is up to date. No Changeset.

Second commit, after origin/v2 was merged into this branch:

Row Before After Evidence
U-E0-03 review: #15259 done #15259 merged (d011f04bdf)
U-E0-04 active done on evidence Only check.yml, test.yml and kilo-v2.yml remain. They run on every PR and on pushes to v2 (bun run check and the package tests). The upstream publish and deploy workflows are deleted, and check-workflows.ts guards the list. No remaining workflow publishes or deploys. All three were green on the last two pushes to v2

After merge, the user can close #15175 (U-E0-03) and #15176 (U-E0-04).

@github-actions

github-actions Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

➖ 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 kilo_port annotations, 0 new core-internal imports.

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.

v2 migration progress map at this commit

Full map · how cells close

| 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 |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.

@kilo-code-bot

kilo-code-bot Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • migration-tracking/plans/kilo-v2-migration-plan.md

Verification notes: All evidence in the incremental commit checks out against the repository. Only check.yml, test.yml and kilo-v2.yml exist; all three have pull_request triggers and push triggers for v2; check.yml runs bun run check; test.yml runs the package tests, script/publish.ts --dry-run and disables the e2e path (affected is if: false, so e2e never runs); kilo-v2.yml builds and uploads artifacts only. script/check-workflows.ts lists exactly 28 removed upstream workflows and fails on unexpected ones. AGENTS.md carries a single marked Kilo block with a rung-4 shared-patches.json entry, and v2-baseline.ts takes the pin from pinned(), asserts ancestry and mirrors the three workflows. U-E0-02's paths (packages/schema/src/kilocode/gateway.ts, packages/kilo-plugin/src/gateway/index.ts, packages/kilo-cli/test/rpc.test.ts) exist, and U-E0-03's merge (d011f04bdf) is an ancestor of this branch. The U-E1-07 note matches the rung-3/rung-4 distinction in v2-host-strategy.md §5. bun script/upstream/progress.ts --check and CI statuses were not run (read-only review); the PR body records the check as passing.

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

Severity Count
CRITICAL 0
WARNING 0
SUGGESTION 1
Issue Details (click to expand)

SUGGESTION

File Line Issue
migration-tracking/plans/kilo-v2-migration-plan.md 438 U-E0-03 recorded as review: #15259, but #15259 merged before this PR's base commit, so the status is already stale
Files Reviewed (1 file)
  • migration-tracking/plans/kilo-v2-migration-plan.md - 1 issue

Verification notes: The U-E0-02 evidence checks out against the repository: #15261 merged as 68536a63da on 2026-10-19, the contract is at packages/schema/src/kilocode/gateway.ts, the adapter at packages/kilo-plugin/src/gateway/index.ts, the round-trip test at packages/kilo-cli/test/rpc.test.ts, and the shared-patch counts (30/305/51 → 29/304/50) match the shared-patches.json limits change in that merge. The Q-G3-02 note is consistent with rpc.call being the only generated route for plugin RPC. The #15018 placement claim matches the marked config lines, which import KiloConfigDiscovery and carry Kilo file names, consistent with rung 4 under v2-host-strategy.md §5. bun script/upstream/progress.ts --check was not run (read-only review).

Fix these issues in Kilo Cloud


Reviewed by deepseek-v4.1-flash-1017 · Input: 47.9K · Output: 10.8K · Cached: 663.8K

Review guidance: REVIEW.md from base branch v2

@fpliger fpliger changed the title docs(migration): record U-E0-02 done and the #15018 placement follow-up docs(migration): record U-E0-02 to U-E0-04 status and the #15018 follow-up Oct 20, 2026
@fpliger
fpliger merged commit 273b11c into v2 Oct 20, 2026
23 checks passed
@fpliger
fpliger deleted the coord-status-e0-02 branch October 20, 2026 01:26
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.

1 participant