Merge develop into master#2857
Merged
Merged
Conversation
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
## Changes - Wrap editor.deleteBackward and deleteForward methods - Catch "Cannot get the parent path of the root path" error - Known issue with @platejs/list-classic when removing list items 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Enhance policy editor with image and table support
## Changes ### Fix 1: Project deletion foreign key constraint error - Moved `recordUseCaseDeletion()` to run BEFORE `deleteProjectByIdQuery()` - The use_case_change_history table has a foreign key to projects(id) - Previously the code tried to insert a history record after the project was deleted - Now the deletion is recorded while the project still exists ### Fix 2: Null project owner causing 500 error - Added null check for project.owner before querying user name - Added check for empty query results - Returns "Unassigned" as owner name when owner is null or user doesn't exist - Prevents "Cannot read properties of undefined (reading 'full_name')" error 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
## Changes - Changed default view mode from "card" to "table" in ProjectsList component - Users will now see the row/table view by default on /overview page 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
## Changes - Fixed cut-off right border by removing negative margin - Fixed column alignment so sidebar aligns at top with main content - Changed default height to auto with max-height constraint - Updated border styling to match design system (#d0d5dd, 4px radius) ## Benefits - History sidebar now displays properly without border cutoff - Two-column layout aligns correctly at the top - Consistent styling with rest of application 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
… into llm-evals-module
## Changes - Added vendorRisks prop to ITableWithPlaceholderProps interface - Updated TableWithPlaceholder to accept and use vendorRisks data - Added getVendorRiskCount helper function to count risks per vendor - Updated Risk column to show "View risks (N)" with count when risks exist - Passed vendorRisks from Vendors page to TableWithPlaceholder component ## Benefits - Users can see at a glance how many risks are associated with each vendor - No additional API calls required - uses existing vendorRisks data - Clicking still opens the full vendor risks dialog Closes #2834
## Changes - Added modelRisks prop to ModelInventoryTableProps interface - Added RISKS column to model inventory table between SECURITY ASSESSMENT and STATUS - Added getModelRiskCount helper function to count risks per model - Display "N risks" when model has risks, "No risks" otherwise - Passed modelRisksData from ModelInventory page to table component ## Benefits - Users can see at a glance how many risks are associated with each model - Consistent with vendor table which also shows risk counts
## Changes - Fixed lexical declaration in case block (Vendors/index.tsx) - Removed unnecessary 'theme' dependency from useMemo (WithPlaceholder) - Added missing 'onCheckModelHasRisks' to useMemo dependencies (modelInventoryTable) ## Benefits - Lint checks now pass for all modified files - No errors, only pre-existing warnings remain
The latest improvements made to the EvalsDashboard and EvalServer
## Changes - Created new ModelRisksDialog component to display model risks in a modal - Updated model inventory table RISKS column to use VWLink component - Clicking "View risks (N)" now opens a dialog showing all risks for that model - Dialog displays risk name, category, risk level, status, and owner - Includes pagination for models with many risks ## Benefits - Consistent UX with vendor risks dialog - Users can quickly view model risks without navigating away - Risk count is visible at a glance in the table
## Changes - Changed risk display format from "View risks (N)" to "N risks" - Show "No risks" as plain text (without VWLink) when count is 0 - Applied consistent format to both Vendors and Model Inventory tables ## Benefits - Cleaner, more concise display format - Visual distinction between items with risks (clickable link) and without risks (plain text) - Consistent risk display across both vendor and model inventory views 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
## Changes
- Changed route from "/model-risks" to "/modelRisks?filter=active" to match the server API
- Added handling for different response formats (array vs {data: array})
## Issue
The dialog was showing "Failed to load model risks" because it was calling a non-existent endpoint
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
## Changes - Added users fetch alongside model risks fetch - Created userMap to map user IDs to names - Updated Owner column to display user's full name (name + surname) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
## Changes - Made risks column sortable in Vendors table - Added full sorting support to Model Inventory table (provider, model, version, approver, security assessment, risks, status, status date) - Added full sorting support to Model Risks table (risk name, model name, category, risk level, status, owner, target date) ## Features - Click column headers to sort ascending/descending - Sorting state persisted in localStorage - Visual indicators show sort direction (chevron icons) - Third click resets to default unsorted state 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
## Changes - Added sorted column background highlighting to Model Inventory table - First column uses #e8e8e8 when sorted, #fafafa otherwise - Other columns use #f5f5f5 when sorted, inherit otherwise - Added sorted column background highlighting to Model Risks table - Same color pattern as other tables in the codebase - Fixed ModelRisksDialog to reset pagination when dialog reopens ## Consistency - Matches the same visual pattern used in training, vendors, and other tables 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
## Changes - Added ability to click on a row in ModelRisksDialog to edit the risk - Added edit modal integration with NewModelRisk component - Added API call to update model risk on save - Added hover effect and cursor pointer on rows - Refresh risks list after successful edit ## Consistency - Matches the same pattern used in VendorRisksDialog for editing vendor risks 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
## Issue The global table style had textTransform: "capitalize" which was automatically capitalizing the first character of all text in table cells. ## Changes - Changed textTransform from "capitalize" to "none" in tables.ts for the body row style 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Show vendor risk count in vendors table
## Changes - Make VerifyWise logo smaller (20x18 from 32x30) - Make VerifyWise text smaller (13px font size, 8px version) - Move sidebar toggle button from edge to right side of logo/version - Remove floating toggle button from sidebar border - Adjust heart icon animation for smaller logo size - Add padding to right side of header for balance ## Benefits - Cleaner, more compact sidebar header - Toggle button is integrated into the header row - Better visual balance with smaller elements
## Changes - Hide VerifyWise text when sidebar is collapsed (using conditional render) - Center logo and toggle button when collapsed - Remove margin-left auto on toggle when collapsed - Use justifyContent to center items in collapsed state ## Benefits - Toggle button stays within collapsed sidebar area - Clean centered layout when collapsed
Fix v1.8 UI issues and error handling
This reverts commit 93633e6.
Add missing `import traceback` that is used at line 268 for error logging. Without this import, a NameError would occur when an evaluation fails. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Added new rul to stop the user from running eval with empty dataset and expanded the UI.
Notes tab UX enhancements
Escape single quotes in topic titles, subtopic titles, questions, and hints to prevent SQL syntax errors when text contains apostrophes (e.g., "you're"). Single quotes are now properly escaped as '' for PostgreSQL compatibility.
Fix SQL syntax error in assessment tracker migration
## Changes - Update Docker build context to repository root to access EvaluationModule - Copy built-in datasets to /app/datasets in Docker image - Add Docker-aware path resolution for built-in datasets - Separate path handling for user uploads vs built-in datasets ## Benefits - Built-in datasets now load correctly in deployed environment - User uploads continue to work in local development - Maintains security checks for both path types 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Fix EvalServer dataset paths for Docker and local development
## Changes - Add ENCRYPTION_KEY environment variable to backend and worker services in docker-compose.yml - Add commented ENCRYPTION_KEY documentation to .env.dev and .env.prod ## Problem LLM API keys stored in the database were being decrypted with the wrong key in Docker deployments, causing 401 Unauthorized errors when calling LLM APIs (e.g., Mistral). ## Root cause The ENCRYPTION_KEY environment variable was not being passed to Docker containers. Without it, the backend falls back to a hardcoded default key, which differs from the key used to encrypt the stored API keys. ## Backward compatibility - ENCRYPTION_KEY is commented out in env files, so upgrades continue using default - New installations can optionally set a custom key - Warning added about re-saving API keys if the key is changed 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add ENCRYPTION_KEY to Docker environment configuration
…ory-table-issue Add handling for missing change history tables
gorkem-bwl
approved these changes
Dec 19, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Describe your changes
Write your issue number after "Fixes "
Enter the corresponding issue number after "Fixes #"
Please ensure all items are checked off before requesting a review: