Skip to content

FHIRPath+Date improvements#100

Merged
lukaskollmer merged 4 commits intomainfrom
lukas/fhir-improvements
Feb 18, 2026
Merged

FHIRPath+Date improvements#100
lukaskollmer merged 4 commits intomainfrom
lukas/fhir-improvements

Conversation

@lukaskollmer
Copy link
Member

FHIRPath+Date improvements

♻️ Current situation & Problem

Currently, only Date values can be produced from a FHIRPath.

This is problematic since a path can express a time-only value, which cannot be represented in a good/sensible/correct way using a Date.

⚙️ Release Notes

  • FHIRPath can now produce DateComponents values. Depending on the input (date/time/dateTime), an appropriate subset of the components is populated.
  • Improvements to FHIRPath parsing when extracting Date/DateComponents values.

📚 Documentation

n/a

✅ Testing

yes

Code of Conduct & Contributing Guidelines

By creating and submitting this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

@lukaskollmer lukaskollmer self-assigned this Feb 18, 2026
@lukaskollmer lukaskollmer added the enhancement New feature or request label Feb 18, 2026
@codecov
Copy link

codecov bot commented Feb 18, 2026

Codecov Report

❌ Patch coverage is 88.88889% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.87%. Comparing base (b56f6f2) to head (ea0652d).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
Sources/FHIRPathParser/DateTimeLiteralParser.swift 75.00% 2 Missing ⚠️
...ces/FHIRPathParser/DateComponents+Operations.swift 75.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #100      +/-   ##
==========================================
+ Coverage   83.87%   85.87%   +2.01%     
==========================================
  Files          28       29       +1     
  Lines         378      382       +4     
==========================================
+ Hits          317      328      +11     
+ Misses         61       54       -7     
Files with missing lines Coverage Δ
Sources/FHIRPathParser/Date+FHIRPathValue.swift 100.00% <100.00%> (ø)
...rces/FHIRPathParser/DateExpressionEvaluation.swift 55.56% <100.00%> (+1.99%) ⬆️
Sources/FHIRPathParser/FHIRPathExpression.swift 100.00% <ø> (ø)
...IRPathParserTests/DateTimeLiteralParserTests.swift 100.00% <100.00%> (ø)
...ests/FHIRPathParserTests/FHIRPathParserTests.swift 100.00% <100.00%> (ø)
...ces/FHIRPathParser/DateComponents+Operations.swift 83.34% <75.00%> (-16.66%) ⬇️
Sources/FHIRPathParser/DateTimeLiteralParser.swift 81.49% <75.00%> (+0.72%) ⬆️

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b56f6f2...ea0652d. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lukaskollmer lukaskollmer merged commit 67cca59 into main Feb 18, 2026
10 checks passed
@lukaskollmer lukaskollmer deleted the lukas/fhir-improvements branch February 18, 2026 20:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments