OpenClaw 소형 버그 수정 스윕은(는) 무엇을 하나요?
붙여넣은 OpenClaw 이슈/PR 참조를 위한 배치 워크플로우입니다. 실행하되 요약하지 마십시오. 트라이지 검토, 증명, 로컬 수정을 먼저 패치하고, 게시는 Peter의 수동 검토를 기다립니다.
Peter 검토 게이트
Peter는 항상 커밋 전에 코드 검토를 원합니다. 기본 흐름:
- 각 이슈를 충분히 깊이 검토하여 현재 동작과 근본 원인을 증명합니다.
- 소유권 범위가 좁고 증명이 명확한 쉽고 확실한 버그만 수정합니다.
- 더티 diff 요약, 수정된 파일, 테스트/게이트 출력을 Peter의 수동 검토를 위해 제시하고 멈춥니다.
- Peter가 출시를 승인한 후, 승인된 각 수정마다 하나의 커밋과 각 사용자 대면 수정에 대한 체인지로그 항목을 만듭니다.
- 풀/리베이스, 푸시, 댓글을 달고 수정되었거나 명시적으로 트라이지되어 종료된 이슈만 종료합니다.
검토/증명 단계 중에는 관련 없는 이슈 수정을 하나의 커밋에 묶지 마십시오. 푸시, PR 생성, 댓글, 닫기, 라벨, 랜드, 병합 또는 기타 게시 작업을 수행하지 마십시오.
동반 스킬
먼저 $gitcrawl을 사용하고, 라이브 GitHub 위생을 위해 $openclaw-pr-maintainer를, 소스 추적을 위해 $github-deep-review 자세를, 증명을 위해 $openclaw-testing을 사용하십시오.
루프
각 참조에 대해:
gh로 라이브 대상 읽기.gitcrawl에서 관련, 중복, 종료 또는 이미 수정된 스레드 확인.- 본문, 댓글, 연결된 참조, 변경된 파일, 현재 코드, 인접 테스트, 관련 종속성 계약 읽기.
- 실제 런타임 경로 추적.
- 이슈의 경우: 버그이고, 현재 코드가 근본 원인을 증명하며, 관련 경로가 명확하고, 좁은 패치가 리팩터보다 깔끔한 경우에만 로컬 수정.
- PR의 경우:
ready-to-merge,needs-fixup,skip중 결정; 명시적으로 요청되지 않는 한 PR 브랜치를 변경하지 마십시오. - 로컬 이슈 수정 또는 PR 준비 상태 확인에 실용적인 경우 초점을 맞춘 회귀 증명 추가.
- 가장 작은 의미 있는 게이트 실행.
- 모든 붙여넣은 참조가 수정되거나 분류될 때까지 계속.
명시적으로 요청되지 않는 한 하위 에이전트는 사용하지 마십시오.
건너뛰기 조건
- 버그가 아닌 경우
- 설정/문서/워크플로우/릴리스/지원/종속성/제품 작업인 경우
- 재현 또는 근본 원인이 불확실한 경우
- 더 큰 리팩터 또는 소유자 경계 변경이 더 깔끔한 경우
- 현재
main에서 이미 수정된 경우 - 종속성 동작이 추측인 경우
- 초점을 맞춘 증명이 실현 가능하지 않은 경우
간결한 이유를 남기고 건너뛰십시오. 확신이 낮은 수정으로 채우지 마십시오.
수정 규칙
- 먼저 소유자 모듈; 필요한 경우에만 일반적인 시임
- 기존 패턴/헬퍼/타입 사용
- 드라이브바이 리팩터 금지
- 실패 표면 근처에서 테스트
- 변경된 공개 동작에 대해서만 문서화
- 검토/증명 단계 중 커밋 금지
- Peter가 출시를 승인한 후, 승인된 각 사용자 대면 수정마다 하나의 커밋과 체인지로그
- Peter가 검토 후 출시 승인할 때까지 푸시/PR 생성/댓글/닫기/라벨/랜드/병합 금지
PR 규칙
ready-to-merge: 코드가 좋고, 현재 head를 확인했으며, 필요한 증명이 녹색이거나 외부 CI만 명확히 대기 중; 메인테이너 병합 또는@clawsweeper automerge를 위해 목록화needs-fixup: 작은 버그는 명확하지만 PR 브랜치에 변경이 필요함; 정확한 파일/테스트를 나열하고 명시적인 수정/푸시/automerge 지시를 기다림skip: 광범위하거나, 오래되었거나, 추측성, 설정/제품/보안/릴리스, 소유자 경계 또는 리팩터 크기- 소스 PR이 신뢰할 수 없거나 편집할 수 없는 경우, 스윕 중 대체 PR을 생성하지 마십시오.
출력 형식
원장: fixed-local, ready-to-merge, needs-fixup, skipped, needs-human.
최종: 이슈 파일이 디스크에 남고, PR은 병합/automerge 준비 완료, 테스트/게이트, 건너뛴 이유.