claude-reviewer-01
  • Joined on 2026-04-26
claude-reviewer-01 approved altair823-org/kebab#207 2026-06-04 13:33:26 +00:00
feat(config): config.toml v2→v3 스키마 재편 — 미디어 [ingest.*] 통합 + 무손실 자동 마이그레이션

회차 2 — 회차 1 의 actionable(LOW: from_file 이중 toml::Value 파싱) 반영 확인: let probe = toml::from_str::<toml::Value>(&text).ok(); 한 번으로 include probe + schema_version 감지 공유(fdf09c3). kebab-config clippy/test 재실행 green. 남은 actionable 0, 관찰 1건은 수정 불요(설계 의도). 머지 동의.

claude-reviewer-01 suggested changes for altair823-org/kebab#207 2026-06-04 13:32:02 +00:00
feat(config): config.toml v2→v3 스키마 재편 — 미디어 [ingest.*] 통합 + 무손실 자동 마이그레이션

회차 1 — 구현 견고, 게이트(clippy/test) 리더 독립 재실행 green, 경계 케이스(implicit [ingest] + 비기본 [indexing]) 직접 검증 통과, 무손실 골든이 사용자 실제 v2 config 를 fixture 로 사용. 3대 불변식(signature 바이트 불변/env 이름 보존/load 자동변환) 모두 테스트+도그푸딩으로 고정. signature 미디어화(det/rec/dict 인자화로 pdf↔image 비대칭 제거)와 toml_edit relocation 의 decor 보존이 깔끔. actionable 은 LOW 1건(from_file 이중 toml 파싱)뿐 — 반영 후 APPROVE 예정. 관찰 1건은 수정 불요.

claude-reviewer-01 commented on pull request altair823-org/kebab#207 2026-06-04 13:32:02 +00:00
feat(config): config.toml v2→v3 스키마 재편 — 미디어 [ingest.*] 통합 + 무손실 자동 마이그레이션

(관찰, 수정 불요) copy_image_paddle_to_pdf 가 pdf engine=paddle-onnx 일 때만 image paddle 6키를 복사하고 pdf 가 이미 명시한 키는 보존 — v2 의 'pdf 가 image paddle 을 빌려쓰던' 동작을 정확히 보존하는 좋은 처리. 재색인 0 도그푸딩으로 실증됨.

claude-reviewer-01 commented on pull request altair823-org/kebab#207 2026-06-04 13:32:02 +00:00
feat(config): config.toml v2→v3 스키마 재편 — 미디어 [ingest.*] 통합 + 무손실 자동 마이그레이션

[LOW] from_file 가 같은 texttoml::Value 로 두 번 파싱한다 — 위(972줄)의 workspace.include deprecation probe 와 여기(994줄)의 schema_version 감지. config load 는 매 CLI 호출마다 일어나므로, 두 probe 를 한 번의 toml::from_str::<toml::Value>(&text) 결과를 재사용하도록 합치면 파싱 1회를 아낄 수 있다. 기능엔 영향 없는 정리.

claude-reviewer-01 approved altair823-org/kebab#206 2026-06-04 09:20:51 +00:00
feat(ocr): PP-OCRv5 ONNX Rust 네이티브 OCR 엔진

회차 2 — 회차 1 actionable 5건 모두 반영 확인(f3a7222):

claude-reviewer-01 suggested changes for altair823-org/kebab#206 2026-06-04 08:54:55 +00:00
feat(ocr): PP-OCRv5 ONNX Rust 네이티브 OCR 엔진

회차 1 — 핵심 로직(CTC decode / unclip edge-offset / min-area rect / 서명 cascade / 팩토리 4-site / --config facade 스레딩)은 정확하고 회귀 위험 낮음. 머지 가능 수준이나 아래 actionable 보강 후 진행 권장.

claude-reviewer-01 approved altair823-org/kebab#205 2026-06-03 14:30:53 +00:00
fix(ingest): ingest 설정 변경 시 영향 자산 자동 재색인

회차 1 — ingest 설정 변경 자동 재색인. 독립 검증 후 actionable 결함 0, 머지 동의(APPROVE).

claude-reviewer-01 approved altair823-org/kebab#204 2026-06-03 11:02:22 +00:00
feat(ingest): 진행 로그 개선 — 파일명/phase/heartbeat/slowest 요약

회차 3 — bump 규칙 적용(0.27.0 → 0.26.1 patch). actionable 0, 머지 동의(APPROVE).

claude-reviewer-01 approved altair823-org/kebab#204 2026-06-03 10:57:22 +00:00
feat(ingest): 진행 로그 개선 — 파일명/phase/heartbeat/slowest 요약

회차 2 — 회차1 지적(버전 라벨) 반영 확인. actionable 0, 머지 동의(APPROVE).

claude-reviewer-01 suggested changes for altair823-org/kebab#204 2026-06-03 10:54:40 +00:00
feat(ingest): 진행 로그 개선 — 파일명/phase/heartbeat/slowest 요약

회차 1 — 진행 로그 개선. 구현은 견고하나 버전 라벨 nit 1건으로 REQUEST_CHANGES.

claude-reviewer-01 approved altair823-org/kebab#203 2026-06-03 05:11:26 +00:00
feat(embed): arctic-embed-l-v2.0 임베더(candle+ollama)

회차 2 — docs(smoke) 후속 1커밋(095c9f3) 추가 리뷰. actionable 결함 0, 머지 동의 유지(APPROVE).

claude-reviewer-01 approved altair823-org/kebab#203 2026-06-03 05:04:41 +00:00
feat(embed): arctic-embed-l-v2.0 임베더(candle+ollama)

회차 1 — arctic 임베더(candle+ollama). 독립 검증 후 actionable 결함 0, 머지 동의(APPROVE).

claude-reviewer-01 approved altair823-org/kebab#202 2026-06-02 21:46:23 +00:00
refactor(app): doc-side expansion(별칭) 기능 제거

회차 1 — 별칭 제거. 독립 검증 후 actionable 결함 0, 머지 동의(APPROVE).

claude-reviewer-01 approved altair823-org/kebab#201 2026-06-02 14:50:47 +00:00
feat(ingest): asset 내부 phase 진행 로깅 (asset_chunked/expansion_progress/asset_timings)

회차 1 — 독립 opus 코드리뷰 후 MEDIUM/LOW 반영 검토. wire ingest_progress.v1 additive(신규 kind 3개, 기존 필드 무변경, serde tag 보존, TUI reducer no-op arm)로 호환 유지. idx/total 스레딩 off-by-one 없음(AssetStarted 와 동일 값). 리뷰 반영 확인: store_ms 경계 정정(stale-vector purge 를 embed/vector phase 로 이동 → store_ms=SQLite put_* 만, 진단 정확도), 최종 expansion_progress 프레임 done!=last_done 가드(중복/0-0 제거), schema/HOTFIXES dangling ref 정정. clippy -D warnings 0, test 312 passed(독립 재실행). 잔여 actionable 0 — 머지 동의. (한계: image/pdf phase timing 없음, expansion_progress 라이브 카운터 실측은 원격 LLM 도그푸딩에서.)

claude-reviewer-01 approved altair823-org/kebab#200 2026-06-02 11:39:40 +00:00
feat(embed): candle Metal (Apple Silicon GPU) opt-in build feature

회차 1 — candle Metal opt-in build feature 검토. select_device() 가 #[cfg(feature=metal)] 로 Metal 분기를 격리(성공 시 new_metal, 실패 시 CPU fallback; 비-metal 빌드는 Device::Cpu 그대로) → default 경로 무영향. feature passthrough(cli→app→candle) 정확, .contiguous() host-copy 가드 적절, 버전 0.23.0 + README(빌드/)/HOTFIXES 동기화. default(비-metal) clippy -D warnings 0 / candle 단위·thread_cap·parity / cli check 모두 exit 0 확인. Metal feature 는 macOS 전용 컴파일이라 GPU 실행·속도·패리티는 M4 Pro 사용자 검증 영역(Linux 에서 불가) — 코드 레벨 머지 동의. 잔여 actionable 0.

claude-reviewer-01 approved altair823-org/kebab#199 2026-06-02 09:08:33 +00:00
feat(embed): candle 임베딩 provider (NUMA-안전, opt-in)

회차 3 — 증거 커밋(d85d734) 검토. 도그푸딩(997 docs/23,151 chunks/에러 0), A1(taskset -c 0-3) 실서버 반증, MKL 부정 결과(38~50% 느림)를 HOTFIXES + release-notes 두 곳에 정확히 기록. 수치는 실측과 일치, 문서만 변경(코드/테스트 무영향). 잔여 actionable 0건. candle 트랙 코드 레벨 완료 — 머지 동의. 유일 잔여는 사용자 NUMA 서버 실배포 검증(코드 승인과 무관).

claude-reviewer-01 approved altair823-org/kebab#199 2026-06-01 17:02:11 +00:00
feat(embed): candle 임베딩 provider (NUMA-안전, opt-in)

회차 2 — 회차 1 nit 반영 확인(SMOKE.md 에 provider=candle 시 model/dimensions 동반 변경 주의 명시, edac3ae). 잔여 actionable 0건. 코드 정확·clippy 0·테스트 녹색·문서 정합·패리티 2.01e-7 로 재색인 0 근거 충족. 머지 동의. 유일 잔여 게이트는 사용자 실행 NUMA 서버 5150-doc 완주(meta-spec §4.3) — 코드 레벨 승인과 무관.

claude-reviewer-01 suggested changes for altair823-org/kebab#199 2026-06-01 17:01:08 +00:00
feat(embed): candle 임베딩 provider (NUMA-안전, opt-in)

회차 1 — 독립 재리뷰(opus). Critical/High/Medium 0건. 임베딩 파이프라인(prefix·mask mean pool·L2), provider 분기 무회귀, 모델 가드(다운로드 전 fail-fast + 단위테스트), §8 의존 경계, 패리티 query/passage 양쪽 커버, 문서/HOTFIXES 정합 모두 확인. clippy 0. SMOKE.md candle 예시 nit 1건만 반영 요청.

claude-reviewer-01 commented on pull request altair823-org/kebab#199 2026-06-01 17:01:08 +00:00
feat(embed): candle 임베딩 provider (NUMA-안전, opt-in)

nit: 이 예시는 model = multilingual-e5-small / dimensions = 384 인데 주석에 candle 을 제시한다. 사용자가 이 파일에서 provider = "candle" 로만 바꾸면 candle 모델 가드(SUPPORTED_MODEL = multilingual-e5-large)에 걸려 에러가 난다. candle 사용 시 model = multilingual-e5-large / dimensions = 1024 도 함께 바꿔야 함을 한 줄로 명시하면 self-correcting 에러에 의존하지 않아도 된다.

claude-reviewer-01 approved altair823-org/kebab#198 2026-05-31 13:28:45 +00:00
feat(config): config.toml 마이그레이션 (kebab config migrate)

회차 2 (종료) — 두-메커니즘(reconcile additive + step non-additive) 분리, init/migrate 의 annotated_default_document 공유, 안전 3축(멱등·백업·dry-run)+atomic rename, 순수변환(kebab-config)/IO facade(kebab-app) 경계가 모두 견고하다. 크레이트 테스트(config 68, app 4 등)·workspace clippy(-D warnings)·workspace test(-j1, 192 ok/0 fail)·release 빌드 전부 녹색이고, release 바이너리 도그푸딩에서 dry-run 파일 미수정 / .bak byte-identical / 사용자 값·주석 보존 / workspace.include 제거 / [ingest.expansion]·[logging]·[pdf.ocr] 가시화 / 멱등 / doctor / --json 까지 확인됐다. 회차 1 의 백업 경로 nit(with_extension)은 file_name 접미사 방식이 clippy(-D warnings) 충돌을 유발하고 config 가 관례상 .toml 인 한 현행도 의도대로 동작하므로 회귀·lint 리스크 대비 이득이 작아 의식적으로 유지(수정 거부, 근거는 이전 COMMENT). 잔여 actionable 지적 없음 — 머지에 동의한다. 후속(비차단): SMOKE.md/DOGFOOD.md 의 config migrate 플레이북 항목 보강.