Revert "fix(ui): improve routine properties panel and history UX" #5723

PR
PR description

Thinking Path

  • Paperclip orchestrates AI agents through visible, governable task and routine workflows.
  • The routines UI includes the routine detail page, properties panel, history tab, and shared sidebar components.
  • PR #5703 changed that workflow by widening the routine properties panel and moving revision inspection/comparison into dialogs.
  • The product direction for that change is being paused for now, so the safest path is a direct revert instead of partial edits.
  • This pull request reverts merge commit 74cb560c41305ac3283067d1ec8d3060ffdc28cb from #5703.
  • The benefit is restoring the prior routines UI behavior while keeping the revert easy to review and re-apply later if needed.

What Changed

  • Reverted #5703: fix(ui): improve routine properties panel and history UX.
  • Restored the previous routine properties panel sizing, panel context API, routine detail layout, and routine history rendering behavior.
  • Removed the reverted sidebar pane test additions and restored the previous focused routine history test expectations.

Verification

  • git diff --check origin/master..HEAD
  • pnpm vitest run ui/src/components/RoutineHistoryTab.test.tsx
  • pnpm --filter @paperclipai/ui typecheck

Screenshots

This is a direct revert of #5703. The visual state after this PR corresponds to the "Before" screenshots from #5703, and the state being removed corresponds to the "After" screenshots from #5703.

Trigger Panel Width

Before this revert After this revert
triggers-before-this-revert triggers-after-this-revert

History Panel

Before this revert After this revert
history-before-this-revert history-after-this-revert

Risks

  • Low technical risk: this is a clean Git revert of a recently merged UI-only PR.
  • Product risk: the routine properties panel and revision history return to the older, narrower workflow that #5703 was improving.
  • Re-application risk: future work that wants the #5703 behavior back should re-apply it deliberately rather than cherry-picking around this revert.

For core feature work, check ROADMAP.md first and discuss it in #dev before opening the PR. Feature PRs that overlap with planned core work may need to be redirected — check the roadmap first. See CONTRIBUTING.md.

Model Used

  • OpenAI Codex, GPT-5 coding agent, tool-enabled with local shell and GitHub CLI access. Context window size was not exposed in this session.

Checklist

  • I have included a thinking path that traces from project context to this change
  • I have specified the model used (with version and capability details)
  • I have checked ROADMAP.md and confirmed this PR does not duplicate planned core work
  • I have run tests locally and they pass
  • I have added or updated tests where applicable
  • If this change affects the UI, I have included before/after screenshots
  • I have updated relevant documentation to reflect my changes
  • I have considered and documented any risks above
  • I will address all Greptile and reviewer comments before requesting merge
CUT
cutter bot commented just now

Cutter Summary

The first step's screenshots show the properties panel extending past the right edge of the viewport, clipping the tab row and trigger card. The remaining steps exercise an inline history list with preview pane and a banner with return/restore actions, which on this SHA are rendered as a single-column list and a read-only dialog.