Skip to content

Conversation

@Evintkoo
Copy link
Contributor

@Evintkoo Evintkoo commented Oct 7, 2025

No description provided.

…oduct

- Simplify route matching logic and unify static file serving
- Add comprehensive asset support with enhanced security
- Modernize 404 page and improve caching strategy
- Remove separate dashboard/playground submodules
- Add comprehensive CPack installer configuration for cross-platform package generation
  - Support for Windows (NSIS, WiX, ZIP), macOS (Bundle, DMG), and Linux (DEB, RPM, TGZ)
  - Configure proper install targets for executables, libraries, and assets
  - Add platform-specific installer customization (icons, shortcuts, registry)
  - Include runtime directory creation for data and logs

- Refactor UI routing to use unified kolosal-product static directory
  - Consolidate playground and dashboard routes under single static folder
  - Improve static asset detection with comprehensive file type support
  - Add proper cache control headers based on content type (images: 1 week, CSS/JS: 1 hour, HTML: 5 minutes)
  - Implement ETag support for better client-side caching
  - Enhanced security with path traversal protection and Content-Type-Options header

- Modernize 404 error page with professional UI design
  - Add responsive layout with mobile support
  - Implement navigation links to main sections
  - Apply dark theme consistent with application branding

- Fix CUDA build configuration for Windows/MSVC
  - Add proper CUDA math library linking
  - Configure CUDA graphs and device runtime libraries
  - Resolve missing math function symbols on Windows builds

- Update CMake to mark test and example installs as OPTIONAL
  - Prevent build failures when test targets are not built

- Improve socket timeout handling with platform-specific implementations
  - Add proper error logging for Windows (WSAGetLastError) and Unix (errno)

- Disable PoDoFo PDF support by default (USE_PODOFO=OFF)

- Remove submodule references for kolosal-dashboard and kolosal-playground
  - Migrate to unified kolosal-product repository structure
- Update gemma3-1b model path to HuggingFace repository
- Update qwen3-embedding-0.6b model path to HuggingFace repository
- Remove server.log file
- Update kolosal-product submodule
- Enable PoDoFo PDF support by default
- Add advanced model loading form with GPU/memory settings
- Implement chat history with localStorage persistence
- Add streaming chat responses and generation parameters
- Support system instructions and multi-chat management
Copilot AI review requested due to automatic review settings October 7, 2025 02:31
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a complete modern UI system for the Kolosal Server, replacing the previous submodule-based approach with an integrated frontend that includes redesigned interfaces for model management, document retrieval, and interactive playground functionality.

  • Updates UI routing system to serve the new integrated frontend from static/kolosal-product/
  • Adds comprehensive component library with modern styling and improved user experience
  • Implements proper API configuration system with automatic endpoint detection

Reviewed Changes

Copilot reviewed 84 out of 93 changed files in this pull request and generated 6 comments.

File Description
src/server.cpp Adds proper socket timeout handling with cross-platform compatibility
src/routes/ui_routes.cpp Complete rewrite of UI routing to support new frontend structure and enhanced static file serving
static/kolosal-product/ New integrated frontend with playground, discover, model management, and retrieval interfaces
Various component files Modular frontend components for UI elements, API handling, and page functionality
Comments suppressed due to low confidence (1)

static/kolosal-product/retrieve/style/upload.scss:1

  • Corrected spelling of 'Reguler' to 'Regular'
@use '../../styles/variable' as *;

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +59 to +66
<div class="select" data-selected="Reguler Chunking" id="ChunkingForm">
<div class="select-trigger">
<h2 class="text-12px reguler">Reguler Chunking</h2>
<i class="ri-arrow-down-s-fill"></i>
</div>
<div class="select-list">
<div class="select-list-item">
<h2 class="text-12px reguler">Reguler Chunking</h2>
Copy link

Copilot AI Oct 7, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'Reguler' to 'Regular'

Suggested change
<div class="select" data-selected="Reguler Chunking" id="ChunkingForm">
<div class="select-trigger">
<h2 class="text-12px reguler">Reguler Chunking</h2>
<i class="ri-arrow-down-s-fill"></i>
</div>
<div class="select-list">
<div class="select-list-item">
<h2 class="text-12px reguler">Reguler Chunking</h2>
<div class="select" data-selected="Regular Chunking" id="ChunkingForm">
<div class="select-trigger">
<h2 class="text-12px reguler">Regular Chunking</h2>
<i class="ri-arrow-down-s-fill"></i>
</div>
<div class="select-list">
<div class="select-list-item">
<h2 class="text-12px reguler">Regular Chunking</h2>

Copilot uses AI. Check for mistakes.
@import url('https://fonts.googleapis.com/css2?family=IBM+Plex+Mono:ital,wght@0,100;0,200;0,300;0,400;0,500;0,600;0,700;1,100;1,200;1,300;1,400;1,500;1,600;1,700&family=Inter:ital,opsz,wght@0,14..32,100..900;1,14..32,100..900&display=swap');

// font weight
.reguler {
Copy link

Copilot AI Oct 7, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'reguler' to 'regular'

Suggested change
.reguler {
.regular {

Copilot uses AI. Check for mistakes.
padding: 12px;
}

.reguler {
Copy link

Copilot AI Oct 7, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'reguler' to 'regular'

Suggested change
.reguler {
.regular {

Copilot uses AI. Check for mistakes.
Comment on lines +59 to +66
<div class="select" data-selected="Reguler Chunking" id="ChunkingForm">
<div class="select-trigger">
<h2 class="text-12px reguler">Reguler Chunking</h2>
<i class="ri-arrow-down-s-fill"></i>
</div>
<div class="select-list">
<div class="select-list-item">
<h2 class="text-12px reguler">Reguler Chunking</h2>
Copy link

Copilot AI Oct 7, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'Reguler' to 'Regular'

Suggested change
<div class="select" data-selected="Reguler Chunking" id="ChunkingForm">
<div class="select-trigger">
<h2 class="text-12px reguler">Reguler Chunking</h2>
<i class="ri-arrow-down-s-fill"></i>
</div>
<div class="select-list">
<div class="select-list-item">
<h2 class="text-12px reguler">Reguler Chunking</h2>
<div class="select" data-selected="Regular Chunking" id="ChunkingForm">
<div class="select-trigger">
<h2 class="text-12px reguler">Regular Chunking</h2>
<i class="ri-arrow-down-s-fill"></i>
</div>
<div class="select-list">
<div class="select-list-item">
<h2 class="text-12px reguler">Regular Chunking</h2>

Copilot uses AI. Check for mistakes.
gap: 4px;
padding: 8px;
overflow-y: auto;
flex: 1;
Copy link

Copilot AI Oct 7, 2025

Choose a reason for hiding this comment

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

Missing flex: 1 0 0 shorthand could cause inconsistent behavior. Consider using the explicit shorthand for better cross-browser compatibility.

Suggested change
flex: 1;
flex: 1 0 0;

Copilot uses AI. Check for mistakes.
justify-content: flex-start;
align-items: stretch;
gap: 4px;
padding: 8px;
Copy link

Copilot AI Oct 7, 2025

Choose a reason for hiding this comment

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

Missing overflow-y: auto and flex: 1 properties that exist in the SCSS file. The compiled CSS appears to be missing some styles from the source.

Suggested change
padding: 8px;
padding: 8px;
overflow-y: auto;
flex: 1;

Copilot uses AI. Check for mistakes.
- Expand LoadingParameters with full llama.cpp common_params coverage
- Add RoPE, GPU, CPU, embedding, LoRA, multimodal, and server params
- Implement JSON serialization and validation for all parameters
- Map parameters to llama.cpp structures in inference implementation
- Disable PoDoFo PDF support in CMake
- Update config with local model paths
@Evintkoo Evintkoo closed this Oct 7, 2025
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