Commit Graph

2 Commits

Author SHA1 Message Date
HAL9000 1295dea0ac fix(tui): PermissionsScreen diff mode cycle uses split/auto per spec #1449
Changed DiffDisplayMode enum values from side_by_side/context to split/auto
as required by specification §29570, §30139, §30391.

- Renamed SIDE_BY_SIDE="side_by_side" → SPLIT="split" in models.py
- Renamed CONTEXT="context" → AUTO="auto" in models.py
- Renamed side_by_side_diff() → split_diff() in models.py
- Renamed context_diff() → auto_diff() in models.py
- Updated _DIFF_MODE_CYCLE in screen.py to use SPLIT and AUTO
- Updated all BDD feature scenarios and step definitions
- Fixed broken Behave step parameter renames (restored standard 'context' param)

Closes #1449

---
Automated by CleverAgents Bot
Supervisor: Implementation | Agent: task-implementor
2026-05-30 02:34:32 -04:00
freemo 4a18a15ca5 feat(tui): implement PermissionsScreen with diff view
CI / lint (push) Failing after 20s
CI / build (push) Successful in 19s
CI / quality (push) Has been cancelled
CI / helm (push) Successful in 22s
CI / typecheck (push) Has been cancelled
CI / e2e_tests (push) Has been cancelled
CI / integration_tests (push) Has been cancelled
CI / coverage (push) Has been cancelled
CI / docker (push) Has been cancelled
CI / security (push) Has been cancelled
CI / unit_tests (push) Has been cancelled
CI / status-check (push) Has been cancelled
CI / benchmark-publish (push) Has been cancelled
CI / benchmark-regression (push) Has been cancelled
Implement PermissionsScreen with diff view for tool permission requests.
- 3 diff display modes (unified, side-by-side, context)
- Allow/reject keyboard bindings
- Permission decision persistence
- 62 BDD scenarios covering all components

ISSUES CLOSED: #996

Co-authored-by: Jeffrey Phillips Freeman <the@jeffreyfreeman.me>
Co-committed-by: Jeffrey Phillips Freeman <the@jeffreyfreeman.me>
2026-04-02 17:08:13 +00:00