nsaspy/zara
1
0
Fork 0
Code Issues 95 Pull requests 18 Projects Releases Packages Wiki Activity Actions

feat(desktop): redesign Copilot with Signal Cabin UI #295

Closed
nsaspy wants to merge 1 commit from codex/copilot-redesign into master
pull from: codex/copilot-redesign
merge into: nsaspy:master
Conversation 2 Commits 1 Files changed 13 +1192 -46
nsaspy commented 2026-09-04 22:44:14 +00:00
Owner
Copy link

Summary

  • Redesigns the Copilot surface with the Signal Cabin UI
  • Adds theme token support across desktop chat widgets and windows

Changes

  • zara/desktop/theme.py: major expansion (+376 lines)
  • zara/desktop/windows/{chat,quick,status}.py, zara/desktop/chat_widgets.py, zara/desktop/app.py
  • t/test_quick_copilot.py: new tests

Merged from local worktree Zarathushtra-copilot-redesign.

## Summary - Redesigns the Copilot surface with the Signal Cabin UI - Adds theme token support across desktop chat widgets and windows ## Changes - `zara/desktop/theme.py`: major expansion (+376 lines) - `zara/desktop/windows/{chat,quick,status}.py`, `zara/desktop/chat_widgets.py`, `zara/desktop/app.py` - `t/test_quick_copilot.py`: new tests Merged from local worktree `Zarathushtra-copilot-redesign`.
feat(desktop): redesign Copilot with Signal Cabin UI
Some checks failed
CI / test (pull_request) Failing after 11s
Details
CI / shared mic / Ubuntu 24.04 (pull_request) Failing after 4m53s
Details
CI / shared mic / Arch Linux (pull_request) Failing after 10m48s
Details
92cb156972
nsaspy commented 2026-09-04 22:52:41 +00:00
Author
Owner
Copy link

Review triage (2026-08-29): the Signal Cabin UI work has real test coverage (theme/quick-copilot/full-chat) but collides add/add with the design system that landed separately on master (.impeccable/design.json, DESIGN.md, PRODUCT.md). Two competing design-direction documents cannot both ship.

Blocking decision (repo owner call): which design system is canonical. If master's is, this PR needs its design docs rebased into that system rather than replacing it; theme.py +376 and the window changes then need a fresh review against current desktop state. Keeping open pending that decision; behind P0 #134 in the queue.

Review triage (2026-08-29): the Signal Cabin UI work has real test coverage (theme/quick-copilot/full-chat) but collides add/add with the design system that landed separately on master (.impeccable/design.json, DESIGN.md, PRODUCT.md). Two competing design-direction documents cannot both ship. Blocking decision (repo owner call): which design system is canonical. If master's is, this PR needs its design docs rebased into that system rather than replacing it; theme.py +376 and the window changes then need a fresh review against current desktop state. Keeping open pending that decision; behind P0 #134 in the queue.
nsaspy commented 2026-09-04 22:52:41 +00:00
Author
Owner
Copy link

Closing as obsolete implementation. The desktop/runtime/config surfaces have moved substantially since this Signal Cabin branch was cut, and the PR is non-mergeable. Keep the visual ideas as historical reference only; any desktop refresh should be reapplied against current master and the current hook/config/runtime ownership rather than reviving this head.

Closing as obsolete implementation. The desktop/runtime/config surfaces have moved substantially since this Signal Cabin branch was cut, and the PR is non-mergeable. Keep the visual ideas as historical reference only; any desktop refresh should be reapplied against current master and the current hook/config/runtime ownership rather than reviving this head.
nsaspy closed this pull request 2026-09-05 01:42:23 +00:00
Some checks failed
CI / test (pull_request) Failing after 11s
Details
CI / shared mic / Ubuntu 24.04 (pull_request) Failing after 4m53s
Details
CI / shared mic / Arch Linux (pull_request) Failing after 10m48s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug
Something isn't working

  
documentation
Improvements or additions to documentation

  
duplicate
This issue or pull request already exists

  
enhancement
New feature or request

  
good first issue
Good for newcomers

  
help wanted
Extra attention is needed

  
invalid
This doesn't seem right

  
question
Further information is requested

  
wontfix
This will not be worked on

No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
nsaspy/zara!295
No description provided.