Skip to content

Survey Phase 2, and fix what "done" means - #73

Merged
Seungpyo1007 merged 1 commit into
developfrom
docs/phase2-survey
Aug 7, 2026
Merged

Survey Phase 2, and fix what "done" means#73
Seungpyo1007 merged 1 commit into
developfrom
docs/phase2-survey

Conversation

@Seungpyo1007

Copy link
Copy Markdown
Member

Track B of the 0.3.0 plan, first deliverable. Survey only — no code changes.

The finding that matters more than the table

Two documents define "done" for Phase 2, they disagree, and neither matches the code:

  • DESIGN.md — Phase 2 is "fork Code-OSS", because visual effects were thought impossible through the extension API.
  • README — Phase 2 is "deepen renderer-level visual effects at the editor core", marked in progress.
  • Reality — this is not a fork. It is Electron 43 + React 18 + Monaco with its own renderer from the start. The reason the fork existed — no access to the renderer — never applied.

So Phase 2's original goal was met by another route. What remains is not the fork question but whether the four pillars in DESIGN.md §2 are actually on screen.

The table

Verdicts are from reading the code. △ means something similar exists but not where the design put it.

Pillar Verdict
2.3 Agent activity panel Done. Current action, plan view, tool log, token meter — all four filled by real paths.
2.1 Editing animation Glow and afterglow ✓, single undo step ✓, ghost cursor missing, streaming insertion △
2.2 Diff visualization Gutter colours △ (they track git, not the pending proposal), rationale △ (on the card only), CodeLens missing — zero registerCodeLensProvider calls
2.4 Multi-file overview Badges, progress dots and jump ✓; a bar list rather than the card grid △

Definition of done

Three things the design named that the code does not have:

  1. Ghost cursor — small, decoration-based, immediately visible.
  2. Rationale tooltiprationale already exists; it needs to hang on the changed line.
  3. CodeLens accept/reject — hunk selection logic already exists; this puts the buttons in the editor.

All three sit on the decoration layer built in #70.

The remaining △ items are places where the app already answers the same need a different way (full diff in its own tab, file overview as a list). Reshaping those to match the prose would be a separate UI decision, not Phase 2 completion. The fork item closes as not-applicable.

"Streaming insertion" is the one genuinely open question: today the animation replays already-complete text, so it has no relation to how fast the model actually writes. Streaming real tokens would mean changing the proposal/approval flow, not the visuals. Left as separate work.

무엇이 있고 무엇이 빠졌는지 모르는 채로는 "끝냈다" 를 말할 수 없다. 네 기둥
(DESIGN.md 2.1~2.4)을 기준으로 코드를 훑어 표로 정리한다.

먼저 나온 것은 표가 아니라 이것이다 — 끝의 정의가 두 문서에서 다르고 둘 다
현실과 다르다. DESIGN.md 의 Phase 2 는 "Code-OSS 포크로 이전" 이고 README 는
"에디터 코어 렌더러 레벨 시각 효과 심화" 인데, 이 앱은 포크가 아니라
Electron + React + Monaco 로 처음부터 자기 렌더러를 갖고 있다. 포크가 필요했던
이유가 애초에 없다. 원래 목표는 다른 방법으로 이미 달성됐고, 남은 것은 포크
여부가 아니라 네 기둥이 화면에 있느냐다.

판정:

- 2.3 작업 상태·계획 패널은 끝났다. 현재 행동·계획 뷰·툴 로그·토큰 미터
  네 항목 모두 실제 경로로 채워진다.
- 2.1 은 글로우와 잔광, undo 한 덩어리가 있고 커서 고스트가 없다.
  스트리밍 삽입은 완성된 텍스트의 재생이라 모델이 쓰는 속도와 무관하다.
- 2.2 는 거터 색이 git 기준이고, 사유는 카드에만 있으며, CodeLens 는
  registerCodeLensProvider 호출이 0건이다.
- 2.4 는 배지·진행 표시·점프가 있고 그리드가 아니라 막대 목록이다.

그래서 완료 조건을 셋으로 확정한다 — 커서 고스트, 변경 사유 툴팁, CodeLens
수락/거절. 설계가 이름으로 적었는데 코드에 없는 것들이고, 셋 다 데코레이션
계층 위에 얹힌다. 포크 항목은 아키텍처가 달라져 해당 없음으로 닫는다.

나머지 △ 는 설계와 다른 방식으로 이미 답이 있는 것들이라(전체 diff 는 별도
탭, 파일 개요는 목록) 문구에 맞춰 바꾸는 것은 Phase 2 완료가 아니라 별개의
UI 결정으로 남긴다.
@Seungpyo1007 Seungpyo1007 added this to the v0.3 (2026-09-30) milestone Aug 7, 2026
@Seungpyo1007 Seungpyo1007 added the documentation Improvements or additions to documentation label Aug 7, 2026
@Seungpyo1007 Seungpyo1007 self-assigned this Aug 7, 2026
@Seungpyo1007
Seungpyo1007 merged commit 7f039d0 into develop Aug 7, 2026
1 check passed
@Seungpyo1007
Seungpyo1007 deleted the docs/phase2-survey branch August 7, 2026 03:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant