fix(queue): preserve concurrent DiskSage and LineageWeave enrollments - #2201
seonghobae wants to merge 6 commits into
Conversation
Integrate concurrent bounded queue-health enrollments without overwriting either sibling. The allowlist and its exact-equality contract now retain both ContextualWisdomLab/disksage and ContextualWisdomLab/LineageWeave. This is a non-force two-parent descendant of #2196401a6ea and #2200c4054ee; collector behavior and workflow policy are unchanged.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughActions 큐 상태 점검 저장소 허용 목록에 ChangesActions 큐 상태 점검 허용 목록
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The PR adds the two intended queue-health repositories and keeps the contract test aligned; no current merge-blocking risk is evident. 🚥 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 |
|
Focused exact-source verification on @coderabbitai review |
|
✅ Action performedReview finished.
|
|
Noema enrollment gap found against this integration exact I preserved this PR as the integration parent and opened Draft child #2202 rather than another sibling against #1150. #2202 is a two-commit RED→repair stack: test-only |
Canonical queue-health child integration for concurrent DiskSage and LineageWeave enrollment. This PR owns only the combined allowlist/contract delta.
Current child exact remains
9a30272ce33566d9a9c781f868443aac7edafed5. Its historical CodeQL failure remains valid RED provenance from inherited central source; successful Security/SAST checks on that old exact do not offset the CodeQL failure.The inherited source defect has now been canonically repaired:
.github#2234@2acb72681a9acd146eb938b8d97c4d67d9ae7e9cmerged into protected.github/mainas8c77327a52883cbb7a9a698a67c20b3c2bec6700. Do not duplicate that repair in this child.Canonical queue-health parent #1150 has been reconciled by ordinary two-parent ancestry onto that protected main. New parent exact is
6f9f2b4d9d1223355a417ba48d8a11500f267986, descending from prior owner42bb922f...and protected main8c77327.... Against current main it is ahead-only / behind 0 with 12 effective queue-health paths. Fresh #1150 checks are nonterminal: Security35214786186, Python Security35214786233, SAST35214786215, and CodeQL35214786187are still queued/pending.This child has not yet been reconciled onto
6f9f2b4.... Its current non-mergeable state is a repair finding, not a Close reason. Correct order is: #1150 current exact reaches terminal acceptance; then normally reconcile #2201 while preserving exactly the two enrollment paths; then obtain fresh #2201 security/static-analysis evidence. No previous GREEN transfers.Keep Draft. Do not use forced branch updates, destructive rebases, blind reruns, source-copy of the central repair, wholesale conflict-side selection, synthetic success, runner-selector churn, or gate weakening.