diff --git a/.yoi/tickets/open/20260607-001651-workspace-panel-remove-direct-pod-send/artifacts/.gitkeep b/.yoi/tickets/closed/20260607-001651-workspace-panel-remove-direct-pod-send/artifacts/.gitkeep similarity index 100% rename from .yoi/tickets/open/20260607-001651-workspace-panel-remove-direct-pod-send/artifacts/.gitkeep rename to .yoi/tickets/closed/20260607-001651-workspace-panel-remove-direct-pod-send/artifacts/.gitkeep diff --git a/.yoi/tickets/open/20260607-001651-workspace-panel-remove-direct-pod-send/item.md b/.yoi/tickets/closed/20260607-001651-workspace-panel-remove-direct-pod-send/item.md similarity index 96% rename from .yoi/tickets/open/20260607-001651-workspace-panel-remove-direct-pod-send/item.md rename to .yoi/tickets/closed/20260607-001651-workspace-panel-remove-direct-pod-send/item.md index 5223c61f..e995a7eb 100644 --- a/.yoi/tickets/open/20260607-001651-workspace-panel-remove-direct-pod-send/item.md +++ b/.yoi/tickets/closed/20260607-001651-workspace-panel-remove-direct-pod-send/item.md @@ -2,13 +2,13 @@ id: 20260607-001651-workspace-panel-remove-direct-pod-send slug: workspace-panel-remove-direct-pod-send title: Remove workspace panel direct Pod send -status: open +status: closed kind: task priority: P1 labels: [tui, panel, companion, cleanup] -workflow_state: ready +workflow_state: done created_at: 2026-06-07T00:16:51Z -updated_at: 2026-06-07T01:33:27Z +updated_at: 2026-06-07T02:00:58Z assignee: null legacy_ticket: null --- diff --git a/.yoi/tickets/closed/20260607-001651-workspace-panel-remove-direct-pod-send/resolution.md b/.yoi/tickets/closed/20260607-001651-workspace-panel-remove-direct-pod-send/resolution.md new file mode 100644 index 00000000..cf117145 --- /dev/null +++ b/.yoi/tickets/closed/20260607-001651-workspace-panel-remove-direct-pod-send/resolution.md @@ -0,0 +1,19 @@ +Implemented and merged. + +Summary: +- Removed the workspace panel path that treated the `Companion` composer target as direct selected-Pod `Method::Run` sending. +- Non-empty Companion submit now preserves the draft and reports that the real Companion lifecycle is not connected yet. +- Preserved Pod open/attach behavior through empty Enter / `o`. +- Preserved Ticket Intake composer launch behavior. +- Updated Pod row action/key-hint behavior away from direct-send semantics. +- Updated focused tests for the new behavior. + +Merged implementation: +- Child commit: `393cde9 tui: remove panel direct pod send` +- Main merge commit: `merge: remove panel direct pod send` + +Validation after merge: +- `cargo test -p tui multi_pod --lib` +- `cargo test -p tui workspace_panel --lib` +- `cargo fmt --check` +- `git diff --check` \ No newline at end of file diff --git a/.yoi/tickets/open/20260607-001651-workspace-panel-remove-direct-pod-send/thread.md b/.yoi/tickets/closed/20260607-001651-workspace-panel-remove-direct-pod-send/thread.md similarity index 67% rename from .yoi/tickets/open/20260607-001651-workspace-panel-remove-direct-pod-send/thread.md rename to .yoi/tickets/closed/20260607-001651-workspace-panel-remove-direct-pod-send/thread.md index 752b2a99..db3f372d 100644 --- a/.yoi/tickets/open/20260607-001651-workspace-panel-remove-direct-pod-send/thread.md +++ b/.yoi/tickets/closed/20260607-001651-workspace-panel-remove-direct-pod-send/thread.md @@ -84,3 +84,59 @@ Scope: The child should commit implementation work in the child worktree and report diff/tests. Merge, review, Ticket closure, and cleanup remain with the parent/human workflow. --- + + + +## Review: approve + +Approved after reviewing the child implementation and post-merge validation. + +Review notes: +- The workspace panel no longer sends non-empty Companion composer input to the selected Pod via direct `Method::Run`. +- Non-empty Companion submit now keeps the draft and reports that the real Companion lifecycle is not connected yet. +- Empty Enter and `o` still open/attach the selected Pod. +- Ticket Intake Enter still builds the Intake launch request and is not treated as direct selected-Pod send. +- Pod rows now advertise open behavior rather than send behavior. + +Validation run after merge: +- `cargo test -p tui multi_pod --lib` +- `cargo test -p tui workspace_panel --lib` +- `cargo fmt --check` +- `git diff --check` + +--- + + + +## State changed + +Ticket closed; workflow_state set to done. + + +--- + + + +## Closed + +Implemented and merged. + +Summary: +- Removed the workspace panel path that treated the `Companion` composer target as direct selected-Pod `Method::Run` sending. +- Non-empty Companion submit now preserves the draft and reports that the real Companion lifecycle is not connected yet. +- Preserved Pod open/attach behavior through empty Enter / `o`. +- Preserved Ticket Intake composer launch behavior. +- Updated Pod row action/key-hint behavior away from direct-send semantics. +- Updated focused tests for the new behavior. + +Merged implementation: +- Child commit: `393cde9 tui: remove panel direct pod send` +- Main merge commit: `merge: remove panel direct pod send` + +Validation after merge: +- `cargo test -p tui multi_pod --lib` +- `cargo test -p tui workspace_panel --lib` +- `cargo fmt --check` +- `git diff --check` + +---