feat(kanban): separate builder corrections from owner asks - #2
Open
moonweave wants to merge 2 commits into
Open
Conversation
૮ >ﻌ< ა ci reviewrunning on 41f9c35 — feat(kanban): separate builder corrections from owner asks waiting for jobs to start… |
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.
#1을 대체한다. 내용은 같고, 무관한 파일 둘이 빠졌다.무엇을 하나
needs_input은 그 자체로 "대표에게 묻는다"는 계약이 아니다. 리뷰어가 빌더를 코드·증거 수정으로 되돌려보낸 행도 같은 큐에 들어간다. 그 행들의 다음 행동은 사람의 답이 아니라 수정이다.needs_work:와REVIEW_R<n>_HOLD를 세는needs_input_needs_work_rows집계를 추가한다. 기존needs_input_awaiting_review_rows와 같은 형태이고 같은 범위(block_kind,blocked/triage)의 진부분집합이며, 개수만 내보낸다 — id·제목·reason 은 함수 밖으로 나가지 않는다.판정은 현재 유효한 block 이벤트 기준이다. 제목이나 나이 추측이 아니다.
왜 필요한가
2026-08-15 실측. 알림이 대표에게
Frontend-only Release — 5건이 결정을 기다립니다라고 보냈다. 21건 전수 조사 결과 대표가 답할 것은 0건이었다 — 3건은 리뷰어의 코드 결함 지적, 나머지는 기술 실패이거나 이미 후속 카드가 대신한 잔해였다.대표 대기열이 5배 부풀면 그 채널은 곧 안 읽힌다. 그러면 진짜 정지가 조용히 지나간다.
#1과의 차이#1은contributors/emails/katie@lightpanda.io와mamagnus00@gmail.com에 매핑을 추가했다. 확인해보니 현재origin/main대비 이 PR 범위의 저자는 본인 6건과 teknium 1건(CI가 건너뜀)뿐이고, 그 두 사람의 커밋은 범위에 없다. 불필요할 뿐 아니라 남의 GitHub 핸들을 대신 주장하는 파일이라 뺐다.contributors/emails/choimoonyoung3631@gmail.com은 남겼다 — 그 이메일은 실제로 범위 안에 있고 upstream 에 매핑이 없어 CI 가 요구한다.검증
베이스 대비
82 passed → 83 passed, 실패 2건은 집합이 동일하며 이 변경과 무관하다.양성 대조 —
_NEEDS_WORK_REASON_PREFIX를 매칭되지 않는 값으로 바꾸면 새 테스트test_board_stats_separates_builder_corrections_from_owner_questions가 실패한다.남은 것
이 PR 은
codex/kanban-stats-rebased-20260813(upstream NousResearch#85266) 위에 쌓여 있다. 그것이 먼저 들어가야 한다.소비자 쪽 변경은 대시보드 저장소 PR NousResearch#18 이다. 구형 코어가 이 필드를 내보내지 않으면 해당 부분집합은 숨기지 않고 대표 대기열에 그대로 남는다 — 검증되지 않은 가정으로 감추지 않는 방향이다.