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 |
Verified supersession closure, not abandonment.
Child exact
9a30272ce33566d9a9c781f868443aac7edafed5carried only two effective paths versus its historical merge base:config/actions_queue_health_repositories.jsonandtests/test_actions_queue_health_contract.py, preserving concurrentContextualWisdomLab/LineageWeaveandContextualWisdomLab/disksageenrollment.Canonical successor #2241
74613cffdc575fb87c2a9c7cb3efcb9036ebce09explicitly consolidatedLineageWeave,OriginWeave,disksage,noema, andquarantine-sandbox-runtimeon refreshed #1150 and has now merged into protected.github/mainas991b85410270f2374ec1c30967ab6e7de9221fb8. Fresh file-by-file verification on that protected exact proves complete succession of this PR's valid delta/tests:config/actions_queue_health_repositories.jsoncontains bothLineageWeaveanddisksageand additionally preserves the successor enrollments (OriginWeave,noema,quarantine-sandbox-runtime);tests/test_actions_queue_health_contract.pyrequires the same combined allowlist, again with the stronger successor set.Therefore #2201 has no remaining unique valid delta, fixture, contract, or evidence to restack. Historical CodeQL RED remains provenance for the now-repaired inherited central source but does not require keeping a duplicate child open. Closure is allowed specifically because protected #2241/main fully supersedes the two-path child without dropping either enrollment. No force update, destructive rebase, source copy, self-approval, or gate weakening was used.