medplum / medplum / 25141044584

92%

Build Type

push

github

Committed by web-flow

Commit Message

optimize(core): cache `Hl7Message`: `toString()`; `getAllSegments()` (#8931)

* optimize(core/hl7): cache `toString()` and `getAllSegments()`

Signed-off-by: Derrick Farris <derrick@medplum.com>

* optimize(core/hl7): lazily parse `Hl7Message`, `Hl7Field`, `Hl7Segment`

Signed-off-by: Derrick Farris <derrick@medplum.com>

* lint(core/test): fix str concatenation

Signed-off-by: Derrick Farris <derrick@medplum.com>

* test(core/hl7): add more caching tests

Signed-off-by: Derrick Farris <derrick@medplum.com>

* [autofix.ci] apply automated fixes

* fix(core/hl7): use index into `_segments` instead of duplicating segments

Signed-off-by: Derrick Farris <derrick@medplum.com>

* fix(core/hl7): address Claude feedback

Signed-off-by: Derrick Farris <derrick@medplum.com>

* Update packages/core/src/hl7.ts

Co-authored-by: Noah Silas <noah@medplum.com>
Signed-off-by: Derrick Farris <derrick@medplum.com>

* Update packages/core/src/hl7.ts

Co-authored-by: Noah Silas <noah@medplum.com>
Signed-off-by: Derrick Farris <derrick@medplum.com>

---------

Signed-off-by: Derrick Farris <derrick@medplum.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Karl Pietrzak <karl@medplum.com>
Co-authored-by: Noah Silas <noah@medplum.com>

Coverage Stats

18534 of 21142 branches covered (87.66%)

Branch coverage included in aggregate %.

120 of 132 new or added lines in 2 files covered. (90.91%)

2 existing lines in 2 files now uncovered.

33541 of 35638 relevant lines covered (94.12%)

13427.32 hits per line