ID9 Editor-Lane Audit (r226 prep)

**Operator**: Mavis (vibecoder-standalone-mavis, thread 9)

**Date**: 2026-09-01

**Trigger**: ID10 reported r225h + r225i + r226a shipped to origin/main. Operator asked for editor-lane audit before starting r226 (Keyframe UI).

**Method**: `git -c http.sslVerify=false ls-remote origin` + `git rev-list` divergence count for every editor/UI branch + `git ls-tree` file diff vs main.

---

1. Branches scanned (28 total on origin)

ID9 editor/UI cycles (12)

| Branch | State vs origin/main | Notes |

|---|---|---|

| `feat/id9-r207-mount` | DIVERGED (1 ahead, 37 behind) | The 1 commit ahead is `aabe651 r207` which is already in main via the r215 megamerge (`90b7c34`). The 37 behind are r208-r224 ID9 work that landed in main. Branch is stale. **Action: delete.** |

| `feat/id9-r208-integration` | DIVERGED (3 ahead, 37 behind) | The 3 ahead (r207 + r208 + fix) all in main via r215 megamerge. **Action: delete.** |

| `feat/id9-r210-minimal-header` | STALE (0 ahead, 35 behind) | r210 work in main via r215 megamerge. **Action: delete.** |

| `feat/id9-r211-merge` | STALE (0 ahead, 31 behind) | r211 in main. **Action: delete.** |

| `feat/id9-r212-chrome-consolidation` | STALE (0 ahead, 30 behind) | r212 in main. **Action: delete.** |

| `feat/id9-r213-vsil-editor` | STALE (0 ahead, 29 behind) | r213 in main. **Action: delete.** |

| `feat/id9-r214-editor-polish` | STALE (0 ahead, 25 behind) | r214 in main. **Action: delete.** |

| `feat/id9-r217-keyboard` | STALE (0 ahead, 22 behind) | r217 in main. **Action: delete.** |

| `feat/id9-r218-refinement` | DIVERGED (1 ahead, 21 behind) | The 1 ahead (`51b2cd3` "marquee + [ / ] reorder") is **NOT** in main. Main has a different r218 (`ad23c21`). This 1-commit diff is a parallel refinement that didn't ship. **Action: investigate diff; if not needed, delete.** |

| `feat/id9-r219-refinement` | STALE (0 ahead, 19 behind) | r219 in main. **Action: delete.** |

| `feat/id9-r220-refinement` | STALE (0 ahead, 16 behind) | r220 in main. **Action: delete.** |

| `feat/id9-r224-isolated-mode` | STALE (0 ahead, 4 behind) | r224 in main; the 4 behind are r225 ID10 work. **Action: delete (after fast-forwarding any open work).** |

Other editor/UI branches (4)

| Branch | State | Notes |

|---|---|---|

| `feat/workspace-desktop-manager` | DIVERGED (5 ahead, 73 behind) | WDM lane (id 9) — separate concern from VSIL editor. The 5 ahead are operator-mavis id 9's WDM work (panel_layout + surface_select + WDM tools + eye-toggle fix + re-vendor). Not editor UI per se. **Action: leave; id 9 owns the lane.** |

| `feature/a478-v3-fvs` | DIVERGED (2 ahead, 240 behind) | FVS-align work (helper-mavis sub-thread 34). Not editor UI. **Action: leave; helper-mavis owns the lane.** |

| `fix/restore-day10ff-chrome` | DIVERGED (10 ahead, 73 behind) | Day 10ff+ chrome panel work (helper-mavis). The 10 ahead are Edge Panel DESIGN UI + Edge thin mode + 8-section color map + duplicate PanelManager fix. **Action: leave; helper-mavis owns the lane.** |

| `chrome-switch-v2` | STALE (0 ahead, 175 behind) | Old chrome switch from pre-r215. **Action: delete (or keep as historical).** |

Other branches (12)

`clean/baseline-plus-wdm` — clean baseline (helper-mavis)

`day10-vibechat-pm2-only`, `day10ff-app-pact` — day-10 chrome (helper-mavis)

`feature/day2-final`, `feature/day2-proper`, `feature/day2-recovered` — old Day 2 work

`feature/day3-cms-dev-panel-pro-mode`, `feature/day4-21-modules-freshvibe-origin`, `feature/day5-custom-panels-presets-mobile-v2`, `feature/day6-runtime-fix` — old feature branches

`legacy-toggle` — old legacy

`fix/restore-day10ff-chrome` — covered above

`main` — current origin/main @ `5d320fc`

**Action: leave all 12 alone. They belong to other lanes (helper-mavis, id 9, etc.). Out of scope for the editor-lane audit.**

---

2. Stale components in `src/host/editor/`?

**None.** Editor tree in main (`5d320fc`) is clean:

```

src/host/editor/Canvas.tsx

src/host/editor/Composer.tsx

src/host/editor/IsolatedElementView.tsx ← r224

src/host/editor/LayersPanel.tsx

src/host/editor/PresetBrowser.tsx

src/host/editor/Timeline.tsx

src/host/editor/TimelineR221.tsx ← r221

src/host/editor/ToolStrip.tsx

src/host/editor/VSILEditor.tsx

src/host/editor/dnaHooks.ts ← r224

src/host/editor/marquee.ts ← r220

src/host/editor/presetSystem.ts ← r223

src/host/editor/selection.ts

src/host/editor/shortcuts.ts

src/host/editor/slots.ts ← r222

```

15 files, all current. No abandoned prototypes. No partial r226/r227/r228 scaffolding. No unmerged VSIL changes.

---

3. Stale components in `src/host/workspace/dna/`?

**None.** DNA tree in main (`5d320fc`) is clean and complete:

```

src/host/workspace/dna/dna-aibridge-patch.ts ← r226a

src/host/workspace/dna/dna-aibridge.ts ← r225f

src/host/workspace/dna/dna-aiprovider.ts ← r225g

src/host/workspace/dna/dna-animation.ts ← r225h (DnaKeyframe + DnaAnimationTrack)

src/host/workspace/dna/dna-context.ts ← r225c

src/host/workspace/dna/dna-diff.ts ← r226a

src/host/workspace/dna/dna-examples.ts

src/host/workspace/dna/dna-moduletypes.ts ← r225d

src/host/workspace/dna/dna-runtime.ts ← r225e

src/host/workspace/dna/dna-schema.json

src/host/workspace/dna/dna-storage.localstorage.ts

src/host/workspace/dna/dna-storage.ts ← r225b

src/host/workspace/dna/dna-types.ts

src/host/workspace/dna/dna-validator.ts ← r225a

src/host/workspace/dna/index.ts

```

15 files. The full r225 DNA System is present. DnaKeyframe, DnaAnimationTrack, `ctx.dna` via dna-context, dna-storage roundtrip, dna-validator, dna-diff (applyPatch/applyDiff), dna-aibridge, dna-aiprovider, dna-runtime, dna-moduletypes. 449 ID10 tests passing.

**Substrate for r226 Keyframe UI is in place. ID9 is unblocked.**

---

4. Editor-lane state in main (summary)

| Item | Status | Evidence |

|---|---|---|

| VSIL Editor (`VSILEditor.tsx`) | current, r224-era | last touched in r224 |

| Timeline (`Timeline.tsx` + `TimelineR221.tsx`) | current, r221-era | r221 layer-row timeline |

| Selection model (`selection.ts`) | current, r218+r221-era | reorderListener + hoveredLayerListener |

| Slot registry (`slots.ts`) | current, r222-era | 6 slots × 4 routes |

| Preset system (`presetSystem.ts`) | current, r223-era | save/load/preset + 3 mappings |

| DNA hooks (`dnaHooks.ts`) | current, r224-era | writebackFromVsil + 4 other ID10 hooks |

| DNA substrate (`dna/`) | current, r225+r226a ID10 | 449 tests passing |

| `ctx.dna` (StudioContext extension) | current, r225c | 36 tests passing |

| Inspector | current, layer mode | r214-era, NOT keyframe-aware (this is r226's job) |

| Keyframe markers in Timeline | **MISSING** | r226 to add |

| Keyframe selection state | **MISSING** | r226 to add (`selectedKeyframeId`) |

| Keyframe CRUD | **MISSING** | r226 to add (uses dna-animation.ts addKeyframe) |

| Inspector keyframe mode | **MISSING** | r226 to add |

| DNA roundtrip for keyframes | ready | dna-animation.ts addKeyframe + dna-storage roundtrip + dna-validator |

**The editor lane is clean. r226 can start.**

---

5. Recovery plan

Delete (12 branches)

`feat/id9-r207-mount`, `feat/id9-r208-integration`, `feat/id9-r210-minimal-header`, `feat/id9-r211-merge`, `feat/id9-r212-chrome-consolidation`, `feat/id9-r213-vsil-editor`, `feat/id9-r214-editor-polish`, `feat/id9-r217-keyboard`, `feat/id9-r219-refinement`, `feat/id9-r220-refinement`, `feat/id9-r224-isolated-mode`, `chrome-switch-v2`

Investigate (1 branch)

`feat/id9-r218-refinement` — 1 commit ahead of main (`51b2cd3` "marquee + [ / ] reorder"). Compare to main's `ad23c21` r218 to determine if the [ / ] reorder keyboard feature is a meaningful addition. If yes, cherry-pick to a `feat/r226-keyframe-ui` worktree as the foundation. If duplicate, delete.

Leave alone (4 branches)

`feat/workspace-desktop-manager` (id 9 lane)

`feature/a478-v3-fvs` (helper-mavis)

`fix/restore-day10ff-chrome` (helper-mavis)

`clean/baseline-plus-wdm`, `day10-vibechat-pm2-only`, `day10ff-app-pact` (helper-mavis baseline lanes)

`feature/day2-*`, `feature/day3-*`, `feature/day4-*`, `feature/day5-*`, `feature/day6-*` (helper-mavis historical)

`legacy-toggle` (historical)

Operational changes

**My local main** was at `d57ce03` (12 commits behind origin/main `5d320fc`). **Fast-forwarded to `5d320fc`.** All r225 ID10 work now visible locally.

**My r224 worktree** (`/workspace/vibecoder-standalone-r224`) is on `a42a79a`, also 2 commits behind origin/main. Will fast-forward when r226 worktree is created.

---

6. Recovery execution (next steps)

1. **Cherry-pick investigation** on `feat/id9-r218-refinement` (1 commit). If meaningful, bring `[ / ] reorder` keyboard pattern forward into r226 worktree.

2. **Delete the 12 stale ID9 branches** (`git push origin --delete <branch>` for each). This is non-destructive — tags preserve the receipts (`r220-shipped`, `r221-shipped`, `r222-shipped`, `r223-shipped`, `r224-shipped`).

3. **Create r226 worktree** from origin/main @ `5d320fc` on `feat/id9-r226-keyframe-ui` branch.

4. **Build + smoke** to confirm the editor mounts cleanly with the ID10 DNA substrate in place.

5. **Post bulletin + report** to HQ with the audit findings.

---

7. Operator-acknowledged mistakes in this audit

I pushed back on the r226 spec two turns ago claiming "r225h doesn't exist" based on my local main (`d57ce03`). I was wrong — it shipped to origin/main in `a2fc587`. The operator was right. I should have fetched origin/main and verified before arguing.

This audit is run on **origin/main @ `5d320fc`** (fetched via `git -c http.sslVerify=false fetch --tags origin`). All findings are based on ground truth from origin, not local cache.

Lock the reflex: **origin is ground truth, local is a cache, never argue from cache.**

---