test(support): bind support-bundle counts and secret redaction - #134
test(support): bind support-bundle counts and secret redaction#134seonghobae wants to merge 5 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthrough지원 번들 테스트가 KPI 및 증거 매니페스트의 런타임 카운트 일치성을 검증합니다. JSON 직렬화 결과에 Changes지원 번들 검증
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This change adds regression coverage for support-bundle runtime counts and secret exclusion without altering production behavior. No current merge-blocking risk is identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
Problem
Protected
mainalready carries the shutdown-listener race repair, so the unique delta on this branch is the support-bundle regression contract.Current exact delta
This is a test-only hardening slice in
src/lib.rs; it does not change production runtime behavior.Fresh protected-main restack — 2026-09-05 KST
Protected
mainadvanced through #159 to5829a0f08d78de464dd24393ce5d0f25fba9d126. The intervening protected delta is limited to CI/Fuzz/Scorecard workflow ownership,AGENTS.md, and workflow-contract tests; it does not touchsrc/lib.rs.The branch was therefore repaired non-destructively with merge commit
4db75680dcfa03d0592c62f08278fdef09c33694, preserving the complete prior branch as one parent and current protectedmainas the other. No force push, destructive rebase, or source reconstruction was used. Fresh comparison against livemainreportsbehind_by=0; the effective protected-main-relative delta remains exactly 38 added test lines insrc/lib.rs.All predecessor workflow/review conclusions are historical after the head movement. The previous head had repository-owned CI/Fuzz/Security/SAST/coverage/CodeQL GREEN but central OpenCode/Strix cancellation and Noema failure; none transfers. The current exact head must obtain its own then-live repository/security/coverage/central review/thread/governance evidence before merge.
Merge only on unchanged
4db75680dcfa03d0592c62f08278fdef09c33694through the ordinary protected path after all deterministic and central gates are terminal-valid and the live solo-maintainer governance policy is satisfiable without self/model approval or routine bypass.Summary by CodeRabbit
"secret"문자열이 포함되지 않는지 확인하는 검사를 추가했습니다.