Retire bear-cache-log in favour of bear-observe - #217
Conversation
The skill lived here because the cache log does, and bear-observe lived in BEAR.EventSourcing because the invoker that nests those scopes under resource_request does. That split followed package ownership, not the question a user asks, and the two skills duplicated their opening, their xstep procedure and their reporting checklist while competing for the same trigger phrases. Everything specific to this package — the event vocabulary, the declaration tables, the app-vs-library triage — is now in bear-observe, which also writes and proves the wiring instead of teaching it by hand. The READMEs point there. https://github.com/bearsunday/BEAR.EventSourcing/blob/1.x/skills/bear-observe/SKILL.md
|
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 (4)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe local ChangesObservation skill documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This change retires the duplicated local skill and updates both README links and the changelog consistently. No current merge-readiness risk remains. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 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 |
|
@coderabbitai review |
✅ Action performedReview finished.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
1 participant
skills/bear-cache-log/is folded into bear-observe in BEAR.EventSourcing, which installs the observation context and proves it from the bindings rather than teaching the wiring by hand. Both READMEs now link there.Merge bearsunday/BEAR.EventSourcing#18 first — the README link 404s until it lands.