Skip to content

fix(ci): preserve bounded sidecar diagnostics - #1978

Draft
seonghobae wants to merge 1 commit into
mainfrom
codex/sidecar-diagnostic-compat
Draft

fix(ci): preserve bounded sidecar diagnostics#1978
seonghobae wants to merge 1 commit into
mainfrom
codex/sidecar-diagnostic-compat

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

변경

CO #1053의 새 진단 형식이 중앙 sanitizer에서 사라지는 두 경로를 수정합니다. 실패한 공급자 시도의 실제 provider_status(100–599 또는 None)와 서버가 만든 request_id(소문자16진수32자리 또는 omitted 표식)를 보존합니다. 기존 로그 형식은 유지하고 원문 메시지·URL·본문은 계속 제외합니다. 요청 이벤트는 bare 형식 또는 서버 logger 접두사에서만 인정하며, 잘못된 필드·부분 매칭·여러 줄 입력은 거절합니다.

AGENTS.md에 producer 단위 테스트만으로 CI artifact 보존을 주장하지 말고 exact revision 쌍을 검증하라는 지침을 추가했습니다. 변경은 sanitizer·기존 테스트·AGENTS 세 파일입니다.

검증

  • 메인 독립 재현: 기존 consumer는 새 provider_status 로그를 None으로 버리고 요청 식별자를 출력에서 제거했습니다.
  • 새 회귀 검사: 33 passed, 101 deselected, 0.10초, -W error.
  • 기존 동작 포함 파일 전체: GITHUB_ACTIONS=true, -W error, 134 passed, 5.88초, exit0.
  • git diff --check 통과, 정상 push, head 5dad3fe.
  • 실제 CO logger73f89777287e2c0bcd21a630019a76512de99f02와 consumer blob9df99e1b7064ca4779a071cbd4c0d5b75afd8b82를 로컬에서 연결해425·429·503 각각 보존 확인. 실제 formatter를 거쳤고 원문/URL/본문 노출 없음, 본문 stream읽기0. 기록: /private/tmp/cwl-sidecar-compat-paired-evidence.md.

경계

로컬 fixture와 실제 logger 연결 검증이며 외부 공급자 요청·실제 CI artifact 게시·보호 병합·배포된 pin 갱신 증거는 아닙니다. 요청 식별자 생성 자체의 HTTP 경로는 이번 paired검증 범위가 아닙니다. 전체 저장소 suite나 coverage100%를 주장하지 않습니다. 원문 응답 재출력, 권한 확대, retry 또는 timeout 변경은 없습니다. CO #1053과 함께 보호된 버전으로 채택해야 실제 CI 진단이 개선됩니다.

Signed-off-by: Seongho Bae <me@seonghobae.me>
@coderabbitai

coderabbitai Bot commented Sep 6, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seonghobae

Copy link
Copy Markdown
Contributor Author

추가 독립 측정: exact head5dad3fe874f93b19b232f4452e7190b72655f18e, 전후 worktree clean. Actions 환경 -W error로 기존 preflight 파일134건이11.72초에 통과했습니다. sanitizer 한 모듈만 측정한 coverage는 실행 문장104/104, 분기50/50, 누락/부분분기0입니다. 기존 pragma로 제외된 main 진입점234–235행 두 줄은 그대로이며 새 제외 규칙을 추가하지 않았습니다. 따라서 제외 없는 전체 저장소100%나 모든 입력 조합 검증이라는 뜻은 아닙니다. 보고서 /tmp/cwl-sidecar-coverage.f1HySS/coverage.json. 새로운 테스트 조건을 통과시키려고 보안 필터·coverage 기준·기존 gate를 낮추지 않았습니다.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Exact-head 검증 추가: 5dad3fe874f93b19b232f4452e7190b72655f18e에서 GITHUB_ACTIONS=true pytest tests -q -W error 전체 실행이 2997 passed, 1 skipped, 21 subtests passed / 201.99초 / exit 0으로 종료했습니다. 실행 전후 HEAD 동일, 작업 트리 clean, stderr 0 bytes를 직접 확인했습니다. 로컬 원본 기록: /tmp/cwl-pr1978-full.6ZO4qH. 제외 1개를 통과로 계산하지 않습니다.

Hosted 검사는 별개입니다. Strix run 34045630626 job 101526823503은 internal_error HTTP 500(request_id 25970859f26b4662a6d2091496a515ad) 후 authoritative vulnerability analysis 불가로 exit 1입니다. CodeQL actions job 101526846114는 DISPATCH_OUTCOME=success, VERDICT_STATE=pending 후 의도된 exit 1이며 취약점 검출 결과가 아닙니다. 어느 것도 성공으로 간주하거나 수동 재실행으로 덮지 않았습니다. 보호 병합과 실제 배포 검증은 미완료입니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority: medium Normal-priority or P2 work status: draft Draft pull request type: bug Defect or incorrect behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant