Survey Phase 2, and fix what "done" means - #73
Merged
Conversation
무엇이 있고 무엇이 빠졌는지 모르는 채로는 "끝냈다" 를 말할 수 없다. 네 기둥 (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 결정으로 남긴다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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:
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.
registerCodeLensProvidercallsDefinition of done
Three things the design named that the code does not have:
rationalealready exists; it needs to hang on the changed line.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.