fix(progress): eliminate duplicate TTY frame per asset #119

altair823 · 2026-05-07T13:15:51Z

altair823 commented

2026-05-07 13:15:51 +00:00

원인

set_position()과 set_message()가 각각 독립적으로 update_and_draw()를 호출 → TTY scrollback에 파일당 2줄 출력.

두 번째 ingest 실행 시 1줄로 보이는 이유: 이미 ingested된 파일들이 빠르게 처리되어 indicatif 내부 draw rate limiter가 연속 draw를 병합.

수정

AssetStarted arm에서:

set_draw_target(hidden()) — draw 억제
set_message() + set_position() — 상태 업데이트
set_draw_target(stderr()) — draw 복원
tick() — 단일 프레임 명시 출력

테스트

cargo test -p kebab-cli 통과
TTY 모드에서 파일당 정확히 1줄만 출력되는 것 확인

## 원인 `set_position()`과 `set_message()`가 각각 독립적으로 `update_and_draw()`를 호출 → TTY scrollback에 파일당 2줄 출력. 두 번째 ingest 실행 시 1줄로 보이는 이유: 이미 ingested된 파일들이 빠르게 처리되어 indicatif 내부 draw rate limiter가 연속 draw를 병합. ## 수정 `AssetStarted` arm에서: 1. `set_draw_target(hidden())` — draw 억제 2. `set_message()` + `set_position()` — 상태 업데이트 3. `set_draw_target(stderr())` — draw 복원 4. `tick()` — 단일 프레임 명시 출력 ## 테스트 - `cargo test -p kebab-cli` 통과 - TTY 모드에서 파일당 정확히 1줄만 출력되는 것 확인

altair823 added 1 commit 2026-05-07 13:15:52 +00:00

fix(progress): eliminate duplicate TTY frame per asset 5be90cffec

set_position() and set_message() each call update_and_draw()
independently, producing two scrollback lines per file in TTY mode.
Suppress the draw target before the two updates, restore to stderr,
then call tick() to emit exactly one frame.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

altair823 merged commit 4475abbf4f into main

2026-05-07 13:17:36 +00:00

altair823 deleted branch fix/progress-duplicate-tty-frame

2026-05-07 13:17:37 +00:00

altair823 referenced this issue from a commit

2026-05-07 13:17:38 +00:00

Merge pull request 'fix(progress): eliminate duplicate TTY frame per asset' (#119) from fix/progress-duplicate-tty-frame into main

altair823 referenced this pull request

2026-05-07 13:28:53 +00:00

fix(progress): one draw per file — drop set_message in TTY AssetStarted #120

Sign in to join this conversation.