Conversation
| mod openapi; | ||
| mod routes; | ||
| mod state; | ||
| mod theme_css; |
There was a problem hiding this comment.
The module declaration mod theme_css; is added but the actual theme_css.rs file is not included in this PR. This will cause a compilation error as the module cannot be found.
| mod theme_css; |
| Json(ThemeVariablesResponse { variables }) | ||
| } | ||
|
|
||
| #[derive(Deserialize, ToSchema)] |
There was a problem hiding this comment.
The theme_css module is referenced but not included in this PR. The function generate_mcp_theme_variables() is called but its implementation is missing from the changeset. This will cause a compilation error.
# Conflicts: # ui/desktop/src/api/index.ts # ui/desktop/src/components/ChatInput.tsx # ui/desktop/src/components/McpApps/McpAppRenderer.tsx # ui/desktop/src/components/McpApps/types.ts # ui/desktop/src/components/McpApps/useSandboxBridge.ts # ui/desktop/src/components/ToolCallArguments.tsx # ui/desktop/src/components/ToolCallWithResponse.tsx # ui/desktop/src/components/recipes/shared/RecipeFormFields.tsx # ui/desktop/src/components/settings/chat/GoosehintsModal.tsx # ui/desktop/src/styles/main.css
|
Closing in favor of a cleaner two-part PR strategy that incorporates this work: #7275 — theme-tokens — theme token infrastructure + CSS class rename (~130 TSX files) The CSS variable rename and main.css restructuring are included in #7275. The Rust CSS parser (theme_css.rs) and server-side theme endpoints were replaced by a TypeScript-first approach, so those pieces didn't carry over. Thanks for kicking this off! |
Summary
Adapt to MCP colors fully - demonstrate it works by having changeable theme colors: