refactor: split IPC handler monolith, add tests, harden security, and improve architecture#34
Merged
NeuroKoder3 merged 3 commits intomainfrom Mar 19, 2026
Merged
Conversation
… improve architecture - Split 2000-line electron/ipc/handlers.cjs into 9 domain-specific modules (auth, entities, admin, license, barriers, ahhq, labs, clinical, operations) with shared session state and utilities in shared.cjs - Add 33 comprehensive business logic tests covering priority scoring, donor matching, FHIR validation/import, notification rules, password validation, and entity helper functions - Fix FHIR parameter mismatch (fhir_bundle -> fhir_data) in FHIRImporter - Add React ErrorBoundary component wrapping the entire app - Add .env.example documenting all environment variables - Add docs/ARCHITECTURE.md with system diagrams and module map - Harden LICENSE_FAIL_OPEN to also check app.isPackaged (prevents leak to production) - Harden default credential seeding (suppress console output in production) - Remove unused moment dependency, standardize on date-fns - Convert utils/index.ts to .js for codebase consistency - Upgrade apiClient.js with safeApiCall error handling utility - Add test execution step to CI workflow - Fix pre-existing lint error in LabsPanel.jsx Made-with: Cursor
…etter-sqlite3 compiles Made-with: Cursor
…LE_VERSION mismatch Made-with: Cursor
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
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
Test plan
Made with Cursor