Conversation
Contributor
🔍 Cypress Test ResultsStatus: ✅ All tests passed 📊 Test Suite Details
|
Contributor
🧪 Type Coverage92.91% ✅
███████████████████░ 92.91% Top hotspots (untyped occurrences)
|
laura-borghesi-lum00n
approved these changes
Feb 19, 2026
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.
This pull request updates the project's type coverage reporting workflow, modernizes dependencies, and improves documentation for DICOM data handling. The most significant changes include switching to a new type coverage toolchain, updating Node.js and Cypress dependencies, and clarifying how to use metadata and pixel buffers from PACS in Larvitar.
Type Coverage Reporting Overhaul:
typescript-coverage-reporttool and its associated script (.github/scripts/ts-coverage-summary.js) with a new workflow based ontype-coverageand a custom reporting script (.github/scripts/typecov-badge-md.js). The new system generates both text and JSON reports, then creates a Markdown summary with top untyped hotspots. The GitHub Actions workflow and PR comment logic were updated to match the new output. [1] [2] [3] [4] [5]package.jsonscripts and dependencies to removetypescript-coverage-report, addtype-coverage, and ensure consistent version pinning for@hyzyla/pdfiumandpatch-package. [1] [2] [3] [4] [5]Testing and Cypress Updates:
cypress-image-snapshotto@simonsmith/cypress-image-snapshotthroughout the Cypress config, plugins, and support files for better maintenance and compatibility. [1] [2] [3]cypress/e2e/base.cy.js.Documentation Improvements:
convertQidoMetadataand the Single Frame Loader. [1] [2]Other Codebase Updates:
HSVtoRGBfunction by wrapping assignments in parentheses for clarity and correctness.3.8.7.