Repository navigation
docs(migration): reset the migration docs to features and principles - #15566
Conversation
Plan at epic and feature level only, keep live state in GitHub, keep only stable content in the repo (#15294).
Code Review SummaryStatus: No Issues Found | Recommendation: Merge The incremental change since the last review is the fix commit No images were added by this change, and no product code is touched, so there is no memory-leak surface to check. Files Reviewed (2 files)
Previous Review Summary (commit 6c9379c)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit 6c9379c)Status: 1 Issue Found | Recommendation: Address before merge Overview
Issue Details (click to expand)SUGGESTION
This PR deletes The PR body's stale-reference sweep says the only remaining hits are the two Files Reviewed (109 files)
Reviewed by deepseek-flash-latest · Input: 36.8K · Output: 5.6K · Cached: 472.8K Review guidance: REVIEW.md from base branch |
# Conflicts: # script/upstream/shared-patches.json
Five dropped dispositions cited the removed kilo-v2-migration-plan.md; they now name the commit that still has it and the marketplace epic #15307.
|
➖ v2 migration progress: 6% → 6% (0 done, 0 total, 0 dropped; 594/9819 cells) Placement: 5 new marked lines in shared files (5 patch), 0 new Ratchets: 27 core-internal imports (base 27); shared patches by rung 35 seam, 25 patch (base 35 seam, 25 patch). No change to the progress map. |
What and why
Implements the stricter version of #15294. The detailed plan (137 units, Part B status cells, many overlapping docs) kept pulling work into doc updates and filled agent context. Planning now happens at epic and feature level only, live state stays in GitHub (#15297, epics, milestones M1 to M4, Project 42), and the repo keeps only stable content. Docs only; no product code changes.
Files
Added
migration-tracking/principles.md: the one place for the technical rules (direction, placement order, clients, identity and config names, behavioral reference). It overrides any other doc.migration-tracking/plan.md: objective, working principles, milestones, epics by milestone with a one-line intent each, out of scope, where things live. No units, no status.migration-tracking/technical-notes/v1-v2-client-breaking-changes.md: taken from docs(migration): list v1 to v2 client breaking changes #15291 (without itsbun.lockchange). Edits: unit IDs replaced by epic links; theplan-foundationdecision record replaced by a link toprinciples.md; "the Kilo RPCs on v2 today are …" replaced by a pointer topackages/schema/src/kilocode/. Checked against the code one39551c9b0:kilocode.provider-usage(get,refresh) andkilocode.marketplace(list,install,uninstall) now exist, so those five rows and the count table are updated.Rewritten
migration-tracking/README.md: what lives where (about 15 lines).Kept
migration-tracking/progress/: generator output unchanged except fivedroppedmarketplace notes indispositions.json(and the README lines generated from them). They cited the removedkilo-v2-migration-plan.md; they now keep U-E2-23 and namee39551c9b0:migration-tracking/plans/kilo-v2-migration-plan.mdand epic MCP and marketplace #15307.technical-notes/baseline/pinned-v2-baseline.md(three pointers to removed docs dropped) andtechnical-notes/script/v2-baseline.ts.Removed
plans/,discovery/,marker-audit/,test-plans/,github-drafts/,assets/, and everything intechnical-notes/except the three kept files (92 files).Pointers updated
AGENTS.md(Kilo block): points toplan.md,principles.md, Kilo v2 migration #15297 and the skills, and adds the line "Before any v2 change, readmigration-tracking/principles.md; it overrides any other doc".shared-patches.json: entry text updated,markedLines+1 (466 after mergingorigin/v2, which added two).v2-migrationskill: sources of truth now point toplan.md,principles.md, Kilo v2 migration #15297 and the kept references. Removed: the G1/G2/G3 strategy, the restated placement ladder and direction rule, "One plan unit per PR", thefacadedisposition, thebun run extensionnote, and Phased Plan - Port Kilo onto OpenCode V2 #13750.## Progress mapis unchanged, along with isolation, self-test, debugging, test policy and evidence.v2-feature-placement:principles.mdis the source of truth; unit IDs removed and the procedure kept.kilocode-merge-minimizer,.kilo/agent/upstream-merge.md,script/upstream/README.md: pointers fixed. Rung 5 (branding) now matchesv2-feature-placement: Kilo-owned automated renaming through the fix(kilo-cli): apply Kilo functional branding #15417 adapter, not "the rename script during syncs". The stale "the host does not exist yet" note and unit IDs are removed from the restated order.script/check-shared-patches.ts,script/check-opencode-annotations.ts,script/upstream/placement.ts,script/upstream/utils/config.ts; a test fixture path inscript/upstream/utils/config.test.ts.Rule moved from
v2-fork-conventions.mdprettier --writeis not run over Kilo documents. No skill had this rule, so it now sits inv2-migration, "Working rules". The other rules in that file are already in the skills,script/upstream/README.mdor the baseline doc, or are dropped (the dev-fix triage lane and thevirtuapatch note forkilo-vscode).Where carried content landed
principles.mdsections 1 to 4 (identity uses the wording of decision 12; its open points are named on the matching epics inplan.md)..opencode/contributor config, which decision 7 also lists, is included.kilo, the npm dist-tag); package naming (D-1). Each can come back as an issue when needed.Archive tag
Not created or pushed: the user declined. The old docs remain at
e39551c9b0(named inmigration-tracking/README.md). To tag that commit later:Checks
bun script/check-shared-patches.ts --worktreeand without--worktree: okbun script/check-opencode-annotations.ts --worktreeand without: okbun script/upstream/progress.ts --check: up to date (no generator output change)bun run script/check-workflows.ts: okscript/upstream:bun install --frozen-lockfile,bun test(92 pass, 0 fail),bun run typecheckokbun run check: 29/29 tasks passedrgpattern from the task, run withgit grep -P): outsideprogress/, the only hits are the twotechnical-notes/baseline/logspaths inv2-baseline.ts. They point to the script's own gitignored log directory, not to a removed file.Follow-ups (not in this PR)
plan.md(Anil offered on chore(migration): plan by feature, decide now, prioritize code #15294).facadedisposition from the progress generator is a small tooling issue.