Retire the automated review loop

[Neonforge98]

Summary

Retires the orchestrator's automated review loop end to end, per the decision that the durable run pipeline's in_review status projection is the review model going forward. Stacked on the dead-command removal PR.

Backend

• on_session_complete now always transitions to Completed — no more automatic review phase; on_review_complete / on_review_failed are deleted and TransitionResult shrinks to RetryAgent | Completed | Failed | Ignored (auto-retry on failure is kept)
• The session-terminal loop no longer launches review/fix sessions, no longer extracts structured review feedback, and no longer writes awaiting-user inbox notifications; launch_phase_session is retry-only
• orchestrator_cancel / orchestrator_retry / orchestrator_get_status / orchestrator_create_follow_up commands are removed along with the follow_up and branch_health modules — their only consumers were the review panel actions and the stale-phase watchdog, both retired here
• OrchestratorPhase::Review / AwaitingUser enum variants and stored review_feedback stay for legacy data deserialization; the durable pipeline keeps writing current_phase through its own start path

Frontend

• useWorkItemOrchestrator (with useAutoReview, useStaleSessionDetection, the collab-lock module, and ownership helpers) is deleted; the two read-only things the detail surface actually needed — the active session projection and PR-creation repo paths — move to a small derived hook, useWorkItemActiveSession
• The ReviewFeedbackPanel (Fix and rerun / Accept as-is / Create follow-up / Cancel) and the reviewer config section are removed; ChangedFilesList no longer threads review comments
• The reviewer property field (properties rail row, dropdown, and handlers) is removed; the orphaned Agent Settings panel that hosted the review/follow-up switches is deleted outright
• ORCHESTRATOR_COMMAND and its error helper are deleted

Failed-run retry remains available through the linked-session row action, which drives the durable run pipeline.

• The review-loop locale keys (agent settings switches, review feedback panel strings, reviewer picker) are removed from all 13 locales

Verification

• cargo test -p project_management orchestrator suites: 44 tests passing (review-transition tests rewritten to assert unconditional completion); cargo check clean on agent_core and the app crate
• npm run typecheck clean; vitest on the Work Items module: 40 files / 201 tests passing; workspace clippy clean
• Live-tested on a dev build: work item detail renders without the reviewer row; a discussion comment enqueued a durable run that resumed the linked session, posted its receipt, and completed on the first attempt — the session terminal path produced no review session, no awaiting-user state, and the item was projected to In Review by the run pipeline as designed