-
Notifications
You must be signed in to change notification settings - Fork 0
Doghouse: implement real CodeRabbit state modeling or trim the promise #9
Copy link
Copy link
Open
Labels
enhancementNew feature or requestNew feature or request
Description
Follow-up to #4.
Why
Doghouse’s docs and domain model promise reviewer-specific state for CodeRabbit, including states like:
- actively reviewing
- cooled down and requestable
- rate-limited
- paused behind manual rearm controls
But the current runtime does not emit CODERABBIT_STATE blockers or events.
That leaves a docs/runtime mismatch in one of the product’s most differentiated promises.
Current behavior
Relevant files:
doghouse/README.mddoghouse/playbacks.mddoghouse/flight-recorder-brief.mdsrc/doghouse/core/domain/blocker.pysrc/doghouse/adapters/github/gh_cli_adapter.py
The enum and design docs talk about CodeRabbit-specific state, but the GitHub adapter currently only models:
- unresolved threads
- checks
- approval state
- merge conflicts
Desired outcome
Make the runtime and the docs agree.
Two acceptable resolutions:
- Implement real CodeRabbit state detection for the currently detectable/actionable states.
- Trim the docs/playbacks/product claims back to what Doghouse actually knows today.
Acceptance criteria
- No stale promise remains in the docs.
- If implemented,
CODERABBIT_STATEis emitted through a tested runtime path. - If not implemented yet, the docs explicitly stop claiming that Doghouse can distinguish cooldown / pause / manual rearm states today.
- The result stays subordinate to human review state instead of eclipsing it.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request