Repository navigation
docs(wish): record re-review #1 of #2817, refresh the fix disposition, retire the UI entries - #2837
Conversation
…, retire the UI entries Orchestrator write for re-review gaps M13/M14: the dual-mode Orca wish now carries the re-review #1 evidence block (pinned f45d634, FIX-FIRST on the new H7 SessionStart-hook gap), a disposition that names every merged and open fix PR incl. the two main-side hotfixes, and the recorded decision that there is no Genie UI besides the Orca integration. INDEX entries for genie-ui-bridge, genie-ui-dash and genie-boards-ui record the retirement. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018QrkgYMEEhrWTUo5E7Nkjg
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 440ef26772
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - [genie-ui-bridge — DESIGN](brainstorms/genie-ui-bridge/DESIGN.md) → [WISH](wishes/genie-ui-bridge/WISH.md) — **RETIRED 2026-08-30** (#2834; Felipe: "theres no more genie ui, only 'ui' is orca integration" — `genie ui-bridge` is now a one-line refusal stub, the MCP transport modules and UI-BRIDGE.md are deleted). Prior record — **SHIPPED 2026-07-21**: G1 `aadbae83` + G2 `2ff71abe` (ui-bridge command, protocol 1.0) execution-reviewed SHIP; PR [#2610](https://github.com/automagik-dev/genie/pull/2610) MERGED to dev `600c18fe` (CI 16/16) + post-merge scratch-client QA. Unblocked genie-ui-dash G4/G5 (merge preceded both reviews); Felipe ratified channel/home/timing via explicit picker | ||
| - [genie-ui — DESIGN](brainstorms/genie-ui/DESIGN.md) → [WISH](wishes/genie-ui/WISH.md) — **SUPERSEDED as the UI direction 2026-07-21**: shipped the fresh-substrate browser shell (merged PR #2609), but Felipe rejected that substrate after seeing it live; the UI direction moved to [genie-ui-dash](wishes/genie-ui-dash/WISH.md). Salvage value retained: the ~1,270-LoC chat backend (no PTY imports) remains conductor-wish substrate. | ||
| - [genie-ui-dash — WISH](wishes/genie-ui-dash/WISH.md) — direct wish (no brainstorm; Felipe-ratified verbatim), plan SHIP + amendment re-review SHIP; **ALL FIVE GROUPS EXECUTED 2026-07-21** on private `khal-os/genie-desktop` branch `genie-dash` @ `9bf43fc`: base+build proof · agent-adapter contract (zero-behavior-change proven) · codex/hermes/rlmx adapters + picker · genie kanban as ui-bridge client · hire roster via roster_hire (single write path). IN_PROGRESS pending Felipe GUI QA + rlmx live spawn; polish-pass LOW backlog consolidated in the G5 disposition. | ||
| - [genie-ui-dash — WISH](wishes/genie-ui-dash/WISH.md) — direct wish (no brainstorm; Felipe-ratified verbatim), plan SHIP + amendment re-review SHIP; **ALL FIVE GROUPS EXECUTED 2026-07-21** on private `khal-os/genie-desktop` branch `genie-dash` @ `9bf43fc`: base+build proof · agent-adapter contract (zero-behavior-change proven) · codex/hermes/rlmx adapters + picker · genie kanban as ui-bridge client · hire roster via roster_hire (single write path). **RETIRED 2026-08-30** with its ui-bridge client (Felipe: no more Genie UI; Orca integration is the only UI) — was IN_PROGRESS pending Felipe GUI QA + rlmx live spawn; polish-pass LOW backlog consolidated in the G5 disposition. |
There was a problem hiding this comment.
When an orchestrator next evaluates the still-APPROVED khal-rebrand wish, this retirement record conflicts with .genie/wishes/genie-ui-dash/WISH.md, whose header and durable-status note still declare IN_PROGRESS (lines 5 and 356–358). More importantly, khal-rebrand/WISH.md explicitly forbids checkout until genie-ui-dash reaches SHIPPED (lines 57 and 329), which a retired wish can no longer do. Update the retired wish and its dependents—or retire the dependent track as well—rather than changing only the INDEX entry, otherwise the documented execution DAG remains permanently blocked.
Useful? React with 👍 / 👎.
* ci(commitlint): pin the four #2817 review-wave squash subjects Three squash-merge headers (#2828, #2829, #2837) crossed 100 characters only because GitHub appended the PR suffix, and #2830's subject is capitalised ("Orca-mode"). They are already on shared dev, so — matching every prior exception in this file — pin the exact full subjects instead of rewriting history. The rolling promotion #2817 fails Commit Messages on that range until this lands. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013gGxGgKskzyzr1HRUB6cV3 * ci(commitlint): also pin the #2840 squash subject Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013gGxGgKskzyzr1HRUB6cV3 --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Orchestrator ledger write for re-review gaps M13/M14 (touches only
.genie/). WISH.md: re-review #1 evidence block (pinnedf45d634b8, FIX-FIRST on the new H7 SessionStart-hook gap; all prior HIGHs verified closed; 3983 tests pass), refreshed fix disposition naming every merged/open PR incl. the main-side hotfixes #2822/#2833, and the recorded decision (Felipe, 2026-08-30: "theres no more genie ui, only 'ui' is orca integration"). INDEX: wish entry updated;genie-ui-bridge,genie-ui-dash,genie-boards-uinow record the retirement (#2834).bun run wishes:lint→ OK.🤖 Generated with Claude Code
https://claude.ai/code/session_018QrkgYMEEhrWTUo5E7Nkjg