encapsulate kludge property to fix invalid credential type error
đī¸ Tracking
N/A
đ Objective
The user state subject injects a property into the objects it stores so that the state provider emits for every update. This property escaped containment, which caused reflection over the stored object to break. Deleting the property before outputting it should preserve the update behavior without breaking reflection.
đĻŽ Reviewer guidelines
- đ (
:+1:) or similar for great changes - đ (
:memo:) or âšī¸ (:information_source:) for notes or general info - â (
:question:) for questions - đ¤ (
:thinking:) or đ (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion - đ¨ (
:art:) for suggestions / improvements - â (
:x:) or â ī¸ (:warning:) for more significant problems or concerns needing attention - đą (
:seedling:) or âģī¸ (:recycle:) for future improvements or indications of technical debt - â (
:pick:) for minor or nitpick changes
Checkmarx One â Scan Summary & Details â c49527d3-e424-448e-8562-6d043d1e0076
New Issues (2)
Checkmarx found the following issues in this Pull Request
| Severity | Issue | Source File / Package | Checkmarx Insight |
|---|---|---|---|
![]() |
CVE-2025-6191 | Npm-electron-36.3.1 | detailsDescription: The package google chrome versions prior to 137.0.7151.119 is vulnerable Integer overflow in V8.Attack Vector: NETWORK Attack Complexity: LOW ID: GX%2FrnqY1VT9ad4VyMwlWUaPbPZW%2F6y5iMvkFoQ9IiEc%3D |
![]() |
CVE-2025-6192 | Npm-electron-36.3.1 | detailsDescription: The package google chrome versions prior to 137.0.7151.119 is vulnerable Use after free in Profiler.Attack Vector: NETWORK Attack Complexity: LOW ID: Zn9dIKOodGUja0o9ZIkQVUv8gE2YGnQiRl27UDs0cUU%3D |
Quality Gate passed
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
Codecov Report
Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.
Please upload report for BASE (
main@71bc684). Learn more about missing BASE report. Report is 25 commits behind head on main.
:white_check_mark: All tests successful. No failed tests found.
| Files with missing lines | Patch % | Lines |
|---|---|---|
| libs/common/src/tools/state/user-state-subject.ts | 33.33% | 1 Missing and 1 partial :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## main #15225 +/- ##
=======================================
Coverage ? 36.92%
=======================================
Files ? 3225
Lines ? 93148
Branches ? 13987
=======================================
Hits ? 34394
Misses ? 57332
Partials ? 1422
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

