feat(legal): add fail-closed due diligence coverage - #445
Closed
zhengdaqi wants to merge 30 commits into
Closed
Conversation
…olution and improved error handling - Introduced functions to resolve the appropriate bash executable based on the environment, improving compatibility across platforms. - Updated the update application logic to utilize the resolved bash executable for executing update scripts. - Enhanced error handling to provide more informative messages when the update process fails, particularly regarding bash executable issues. - Added new service functions for managing update runtime errors and restart commands. - Implemented unit tests to verify the new functionality and ensure robust error handling.
…adjustments - Added new styles for the settings content header to improve typography. - Updated various components to refine spacing and alignment for a more cohesive layout. - Adjusted font sizes and line heights across settings sections for better readability. - Enhanced sidebar navigation with consistent padding and hover effects for improved user experience. - Modified general settings section to streamline visual hierarchy and spacing.
…ation - Renamed settings components for clarity and consistency. - Introduced new settings navigation logic to enhance user experience. - Added unit tests for the new navigation functionality. - Removed deprecated constants and hooks to streamline the codebase. - Updated styles for improved layout and readability across settings sections.
feat: introduce redesigned settings center and harden cross-platform updates
…w functionality - Added `getPdfNavigationMode` utility to determine navigation type based on file extension. - Introduced `PdfOutline` and `resolvePdfOutline` for handling PDF outlines and navigation. - Enhanced `PdfDocumentPreview` component with navigation controls and improved rendering logic. - Updated `CodeEditorBinaryFile` to support PDF navigation and expand/collapse functionality. - Added unit tests for new PDF navigation features and outline resolution. - Improved UI elements for better user experience in document previews.
- Added `findPdfSearchMatches` and `renderPdfSearchHighlights` functions for searching and highlighting text in PDF documents. - Created unit tests for the search functions to ensure accurate match detection and rendering. - Updated `PdfDocumentPreview` component to integrate search functionality, allowing for text highlighting and selection. - Enhanced text handling in the PDF viewer to support search results and improve user interaction.
…tion - Added `handleTabsClose` method to manage closing multiple tabs atomically and selecting the nearest remaining tab. - Enhanced `EditorSidebar` and `CodeEditorTabBar` components to support closing all tabs with context menu options. - Updated UI to include confirmation prompts for unsaved changes when closing tabs. - Added unit tests to verify the new tab closing behavior and ensure proper state management. - Localized new tab actions in English and Chinese for improved user experience.
…ed input handling - Updated `ChatHistorySearchBar` to manage input state with draft queries and handle IME composition events. - Implemented composition start and end handlers to preserve input during Chinese IME usage. - Refactored keydown event handling to improve search navigation and escape functionality. - Added unit tests for preserving IME composition in `MainAreaV2` to ensure correct behavior during search interactions.
- Updated search highlight styles for improved visibility and user experience. - Refactored `highlightSearchMatches` to support multiple highlighted occurrences and distinguish the active match. - Enhanced unit tests to verify correct highlighting behavior in chat history and PDF search components. - Introduced new CSS variables for consistent highlight styling across different themes.
feat(ui): enhance document previews, tab controls, and search
Collaborator
Author
|
Superseded by clean stacked PR #447. PR #445 had a criss-cross merge-base with #437 after both branches merged main independently, which inflated the diff to 190 files. #447 is based directly on the updated #437 head and has a 39-file legal-only diff. This PR is retained only as historical context and rollback reference. |
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
2 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.
Summary
This stacked PR adds a legal due-diligence Domain Plugin and Skill on top of the generic Agent Harness introduced by #437. It uses hook-driven dynamic context and state-based milestones to guide legal work, then blocks completion until deterministic source, fact, issue, authority, coverage, and deliverable checks pass.
The implementation remains entirely inside
products/legalplus focused tests. It does not add case-specific logic or change PilotDeck Core.Changes
legal-coverageplugin with legal-only activation, dynamic prompt injection, model-request state metadata, and fail-closed Stop handling.conduct-legal-due-diligenceSkill and explicit canonical data contracts for sources, facts, matrices, issues, authorities, and final coverage.createLocalGatewaytests for dynamic milestones, bounded correction, Stop-time I/O failure, malformed-config recovery, and unsafe session paths.QA & Evidence
.omo/evidence/20260723-legal-coverage-v1/post-review-fix-gates.txt..omo/evidence/20260723-legal-coverage-v1/post-review-fix-gates.txt..omo/evidence/20260723-legal-coverage-v1/hard-case-validator-summary.txt..omo/evidence/20260723-legal-coverage-v1/hard-verifier-summary.txt..omo/evidence/20260723-legal-coverage-v1/false-positive-rejection-summary.txt..omo/evidence/20260723-legal-coverage-v1/recovery-v12-summary.txt..omo/evidence/20260723-legal-coverage-v1/final-code-review-v3.txtand.omo/evidence/20260723-legal-coverage-v1/final-manual-qa-v3.txt.Risks & Residuals
.omo/evidence/20260723-legal-coverage-v1/judge-connectivity.txt.Stack
codex/legal-runtime-e2e->main)codex/legal-coverage-v1->codex/legal-runtime-e2e