From 2af566a6ad4818e2f82c80690503b0ccbdf86ce7 Mon Sep 17 00:00:00 2001 From: braveji18 Date: Thu, 20 Aug 2026 05:51:38 +0000 Subject: [PATCH] =?UTF-8?q?BIOP02-113=20#comment=20kkkim=20=EA=B2=80?= =?UTF-8?q?=ED=86=A0=20=EB=B0=98=EC=98=81=20=E2=80=94=20config=20=EC=A0=95?= =?UTF-8?q?=EB=A0=AC(=EC=8A=B9=EC=9D=B8)=20+=20v03=20=EB=B3=B4=EA=B0=95=20?= =?UTF-8?q?4=EA=B1=B4?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit kkkim(Leader) 코멘트 2건 반영. #11915(config 정렬 승인) · #11983(v03 검토). ■ #11915 — config 정렬 (Leader 승인 건) auto_review_config.json: ai_review.agents ["paper-critic","reviewer"] → ["paper-critic"] 단독. SKILL.md 5단계가 이미 paper-critic 단독인데 config가 안 따라와 폐기된 reviewer 를 부르던 불일치(GAP 2) 해소. ⚠️ 이 수정이 anti-self-reference 에 안 걸리는 이유 = Critic 이 스스로 정한 게 아니라 Leader 가 승인했기 때문. 같은 이유로 GAP 3·4·5 는 열려 있음. ■ #11983 — v03 보강 4건 (mermaid 10 → 13종) ① 세션-간 상태 지속 레이어 신설 (kkkim "가장 큰 누락") 그림10 + 전용 섹션 — HANDOFF/SESSION_LOG 2층(목적이 다름, 같은 턴에 함께 쓴다) · memory · PITFALLS_REGISTRY · 개인기록 vs 팀기록 분리 · 백업 규칙 · 실제 로그 유실 사고를 이 층의 실패 사례로 기록 ② Layer A 심화 — 그림5 신설: 5역할 체인(data→embedding→modeling→ therapeutic_evidence→critic) + experiments 5종 아티팩트 + A→B 산출물 계약 ("두 레이어를 잇는 것은 대화가 아니라 파일") ③ 헤드라인 재계산 게이트 미기계화를 GAP1 에 명시 — "CI의 숫자 드리프트는 JSON↔md 정합이지 raw 재산출이 아님". ⚠️ 실제 CI 신설은 Leader 승인 사안이라 문서 표시까지만(kkkim 지시 경계 준수). ④ 메타-학습 루프 명명 — 그림3 신설: 사고 → PITFALLS_REGISTRY 등재 → 재발방지 장치 강제 → 금지조항/스크립트/CI. 레지스트리 규칙② "교훈만 적고 장치가 없으면 재발한다"가 이 루프의 핵심. CI 의 게이트 mutation·공허통과 회귀가 이 루프의 산물임을 연결. ■ 그 밖 갭 6 → 5건(config 정렬 닫힘), 닫힌 루프 절에 두 번째 사례 추가. 그림 번호를 첫 언급 순으로 전면 재정렬(품질 기준선 규칙) — 1~13 중복·누락 0. /wrap·/mine 은 이 환경에서 실재 확인 실패 → 인용하지 않음. 검증: mermaid 13블록 구조 검사 전부 통과. PITFALLS_REGISTRY(29항목)· agents/ 5역할·memory/ 실재 확인 후 인용. Co-Authored-By: Claude Opus 5 --- agents/critic/auto_review_config.json | 2 +- ai_scientist/output_v03/README.md | 42 ++++--- ai_scientist/output_v03/index.html | 159 ++++++++++++++++++++++---- 3 files changed, 166 insertions(+), 37 deletions(-) diff --git a/agents/critic/auto_review_config.json b/agents/critic/auto_review_config.json index 7938c9d..ec383db 100644 --- a/agents/critic/auto_review_config.json +++ b/agents/critic/auto_review_config.json @@ -55,7 +55,7 @@ }, "ai_review": { - "agents": ["paper-critic", "reviewer"], + "agents": ["paper-critic"], "checklist": "agents/critic/checklist_v1.md", "prompt_template": "agents/critic/AI_REVIEW_PROMPT.md", "independent_passes": 2, diff --git a/ai_scientist/output_v03/README.md b/ai_scientist/output_v03/README.md index b5a8e00..75fac5b 100644 --- a/ai_scientist/output_v03/README.md +++ b/ai_scientist/output_v03/README.md @@ -1,6 +1,6 @@ # output_v03 — AI Scientist 설계서 시각화 (2026-08-20 기준) -`ai_scientist/`의 설계서 7개(README + 01~06)를 **mermaid 10종 + HTML** 단일 페이지로 시각화한 결과물. +`ai_scientist/`의 설계서 7개(README + 01~06)를 **mermaid 13종 + HTML** 단일 페이지로 시각화한 결과물. | 파일 | 내용 | |---|---| @@ -26,28 +26,43 @@ CI 검증기가 **3종 → 7종**으로 늘었고, 그중 둘(`게이트 mutatio |---|---|---| | CI 검증기 | 3종 | **7종** (게이트 mutation · 공허통과 회귀 · split 누수 · 국영문 정합 추가) | | 검수의 성격 | 결과를 검증 | **검증기 자신을 검증** | -| 미해결 갭 | 5건 | **6건** + **1건 닫힘**(스키마) | +| 미해결 갭 | 5건 | **5건** + **2건 닫힘**(스키마 · config 정렬) | | 03 문서 반영 | 라우팅 + 계약 | + **품질 기준선** · **실행·보고 계약** · **mock 분기** | | 04 문서 반영 | 티어 게이트 | + **`required_followups`**(노동 ≠ 판단) | | 01 문서 | "원고 아직 없음" | 🔴 **정정** — 원고 존재 | -| 다이어그램 | 12종 | **10종**(재구성: 닫힌 루프·mock 분기·mutation 신규) | +| 다이어그램 | 12종 | **13종** — 닫힌 루프 · **메타학습 루프** · **Layer A 상세** · mock 분기 · mutation · **세션 지속** 신규 | -## 다이어그램 10종 +## 다이어그램 13종 | # | 그림 | 대응 | |---|---|---| | 1 | 전체 흐름 + **실행 모드 분기**(mock 포함) | README·03 | | **2** | **⭐ 닫힌 루프** — 발견→분리→결정→도구화→CI | 신규 | -| 3 | 2-레이어 + 사람 게이트 3개 | 01 | -| 4 | **권한 강제 지도** — 도구로 막힌 곳 vs 말로만 | 02 (GAP 3) | -| 5 | **mock 분기** — 데모를 실 결과처럼 넘기지 않는 장치 | 03 | -| 6 | **검수 3층 — CI 7종 상세** | 04 | -| **7** | **⭐ 검증기를 검증하는 층**(mutation / 공허통과) | 04 · 신규 | -| 8 | 협업 흐름 | 05 | -| 9 | 교차검수 owner≠reviewer | 05 | -| 10 | 설계 계보 | 06 | +| **3** | **⭐ 메타-학습 루프** — 사고→PITFALLS_REGISTRY→**재발방지 장치** | 신규 (kkkim ④) | +| 4 | 2-레이어 + 사람 게이트 3개 | 01 | +| **5** | **⭐ 레이어 A 상세** — 5역할 체인 + **A→B 산출물 계약** | 신규 (kkkim ②) | +| 6 | **권한 강제 지도** — 도구로 막힌 곳 vs 말로만 | 02 (GAP 3) | +| 7 | mock 분기 — 데모를 실 결과처럼 넘기지 않는 장치 | 03 | +| 8 | 검수 3층 — CI 7종 상세 | 04 | +| 9 | **검증기를 검증하는 층**(mutation / 공허통과) | 04 | +| **10** | **⭐ 세션을 넘는 상태 인계** — HANDOFF/SESSION_LOG 2층 + memory | 신규 (kkkim ①) | +| 11 | 협업 흐름 | 05 | +| 12 | 교차검수 owner≠reviewer | 05 | +| 13 | 설계 계보 | 06 | + +## kkkim(Leader) 검토 반영 — #11983 (2026-08-20) + +| 지적 | 반영 | +|---|---| +| ① **세션-간 상태 지속 레이어가 없다**(가장 큰 누락) | ✅ 신설 섹션 + 그림 10 — HANDOFF/SESSION_LOG 2층 · memory · 백업 규칙 · 로그 유실 사고 | +| ② **Layer A가 얕다** | ✅ 그림 5 신설 — 5역할 체인(data→embedding→modeling→therapeutic_evidence→critic) + **A→B 산출물 계약** | +| ③ 헤드라인 재계산 게이트가 CI에 없음 | ✅ GAP 1에 명시 — *"CI의 숫자 드리프트는 JSON↔md 정합이지 raw 재산출이 아님"*. **CI화는 Leader 승인 사안으로 남김** | +| ④ incident→레지스트리→가드 루프에 이름 붙이기 | ✅ 그림 3 신설 — **메타-학습 루프** | + +> ③의 경계는 kkkim 지적대로다 — **문서에 갭으로 적는 것까지는 지금, 실제 CI 신설은 별도 승인.** +> `/wrap`·`/mine`은 이 환경에서 실재를 확인하지 못해 **인용하지 않았다.** -## 갭 6건은 전부 실물 재검증했다 +## 갭 5건은 전부 실물 재검증했다 v03 작성 시점에 **문서를 믿지 않고 리포를 열어** 재확인했다. 그 결과 v02가 갭으로 적었던 **스키마 항목은 닫혀 있어** "닫힌 루프" 절로 옮겼다. @@ -58,6 +73,7 @@ v03 작성 시점에 **문서를 믿지 않고 리포를 열어** 재확인했 | `.claude/agents/*.md` frontmatter | `manuscript-writer`는 `Write` 보유 · `venue-reviewer`는 `tools:` 미선언 | | `schemas/critic_report.schema.json` | `critic_status` enum 3값 — "식별 불가" 없음 | | `schemas/hypothesis.schema.json` | 3필드 **정식 등재됨** → 갭 아님(닫힘) | +| `auto_review_config.json` | `ai_review.agents` → **`["paper-critic"]` 정렬 완료**(kkkim 승인 #11915) → 갭 아님(닫힘) | | `.github/workflows/critic-validators.yml` | blocking 스텝 **7개** | ## 특징 diff --git a/ai_scientist/output_v03/index.html b/ai_scientist/output_v03/index.html index 119ee3a..31d1de3 100644 --- a/ai_scientist/output_v03/index.html +++ b/ai_scientist/output_v03/index.html @@ -134,13 +134,14 @@