medplum / medplum / 28599406767

92%

push

github

feat(server): improve history tombstones with metadata (#9632)

* Use minimal delete tombstones without Provenance per FHIR spec

Store minimal delete tombstones in _History with meta.deleted and other metadata.
Delete history entries return HTTP 410 as required
by the FHIR specification.

Signed-off-by: Karl Pietrzak <karl@medplum.com>

* [autofix.ci] apply automated fixes

* better handling of meta.deleted

Signed-off-by: Karl Pietrzak <karl@medplum.com>

* Move restore test helpers into repo.test.ts

These helpers were only used by tests and did not wire up any runtime
restore operation, so colocate them with the test that exercises them.

Signed-off-by: Karl Pietrzak <karl@medplum.com>

* Parse history content once for tombstone checks (#9645)

Add parseHistoryContent helper and use it in readHistory, readVersion,
and search result hydration to avoid double JSON.parse calls.

Signed-off-by: Karl Pietrzak <karl@medplum.com>

* fixes

Signed-off-by: Karl Pietrzak <karl@medplum.com>

* Simplify parseHistoryContent and remove unused isDeleteTombstone

Address review feedback: parseHistoryContent now returns Resource
directly (no null, no one-off ParsedHistoryContent type), callers
branch on parsed.meta?.deleted, and the now-unused isDeleteTombstone
helper and its test are removed.

Co-authored-by: Karl Pietrzak <The-Alchemist@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* [autofix.ci] apply automated fixes

* inline test methods

Signed-off-by: Karl Pietrzak <karl@medplum.com>

* use WithId

Signed-off-by: Karl Pietrzak <karl@medplum.com>

* extracted variable to make the diff smaller

Signed-off-by: Karl Pietrzak <karl@medplum.com>

* test fix

Signed-off-by: Karl Pietrzak <karl@medplum.com>

---------

Signed-off-by: Karl Pietrzak <karl@medplum.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.git... (continued)

21630 of 24574 branches covered (88.02%)

Branch coverage included in aggregate %.

25 of 25 new or added lines in 3 files covered. (100.0%)

38556 of 40906 relevant lines covered (94.26%)

12559.06 hits per line