Skip to content

feat: enable per-workspace model selection#9

Closed
ThomasK33 wants to merge 1 commit into
thomask33/10-01-add_slash_command_suggestionsfrom
thomask33/10-01-add_workspace_model_selection
Closed

feat: enable per-workspace model selection#9
ThomasK33 wants to merge 1 commit into
thomask33/10-01-add_slash_command_suggestionsfrom
thomask33/10-01-add_workspace_model_selection

Conversation

@ThomasK33

Copy link
Copy Markdown
Member

Implement workspace model persistence and command handling.
Load metadata on mount, watch for updates, and save model changes.
Route model updates through new IPC channel and AI service logic.
Provide provider adapters, model catalog helpers, and UI feedback.
Extend slash command parsing, suggestions, and tests for /model flow.

Implement workspace model persistence and command handling.
Load metadata on mount, watch for updates, and save model changes.
Route model updates through new IPC channel and AI service logic.
Provide provider adapters, model catalog helpers, and UI feedback.
Extend slash command parsing, suggestions, and tests for /model flow.

@ammario ammario left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a specific vision for how to do this so I don't think I want to merge the change. For one, I want to send the model via sendMessage in an effort to leave the IPC as minimal as possible.

For two, I want to use the convenient models.json (provided by lite LLM) for abbreviation expansion.

@ThomasK33 ThomasK33 closed this Oct 2, 2025
@ThomasK33 ThomasK33 deleted the thomask33/10-01-add_workspace_model_selection branch October 2, 2025 10:47
duynguyen020304 pushed a commit to duynguyen020304/mux that referenced this pull request May 10, 2026
…RY, tests

Batch 1 — Compile Errors & Correctness:
- Add TOGGLE_KANBAN to KEYBIND_LABELS and KEYBIND_GROUPS (coder#1)
- Fix Ctrl+K conflict → Ctrl+Shift+K to avoid PREV_WORKSPACE collision (coder#2)
- Add TOGGLE_KANBAN dispatch handler in App.tsx (coder#3)
- Fix dirty-cache in promoteQueuedTask: move flag clearing after moveTask (coder#5)
- Wrap 4 IPC helpers in KanbanBoard with try/catch (coder#4)

Batch 2 — Consistency & DRY:
- Deduplicate KanbanColumnSchema: import from canonical orpc/schemas (coder#6)
- Remove unused KanbanBoardData import from kanbanStorage.test (coder#7)
- Remove dead KANBAN_STORAGE_KEYS constant (coder#8)
- Replace console.warn with log helper in kanbanStorage (coder#9)
- Replace useEffect form reset with key prop in TaskDetailModal (coder#10)
- Remove unsafe {} as KanbanTask cast, use EMPTY_FORM pattern (coder#11)
- Add loading state + error logging to KanbanView (coder#12, coder#13)
- Add KANBAN_STATUS_LABELS helper, use in TaskDetailModal (coder#14)
- Validate column membership in reorderTasks (coder#15)
- Use KanbanTaskPriority type import in TaskCreateModal (coder#16)

Batch 3 — Tests:
- Add 4 tests for updateColumn (coder#17)
- Add 3 tests for promoteQueuedTask (coder#18)
- Rename misleading test 'rejects empty title' → 'accepts empty title' (coder#19)

43 tests pass (was 36). No typecheck or lint regressions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants