Remove LLM Functionality from Package

[LeonNissen]

Remove LLM Functionality from Package

♻️ Current situation & Problem

Remove the LLM summary and interpretation logic from this package, as it uses application-specific functionality that does not belong within the package

📝 Code of Conduct & Contributing Guidelines

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

• I agree to follow the Code of Conduct and Contributing Guidelines.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 259 lines in your changes missing coverage. Please review.

Project coverage is 33.50%. Comparing base (d368a1e) to head (961b287).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
.../SpeziFHIR/Extensions/FHIRResource+Flattener.swift	0.00%	258 Missing ⚠️
...urces/SpeziFHIRHealthKit/FHIRStore+HealthKit.swift	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
- Coverage   40.68%   33.50%   -7.18%     
==========================================
  Files          20       21       +1     
  Lines        1207     1466     +259     
==========================================
  Hits          491      491              
- Misses        716      975     +259     

Files with missing lines	Coverage Δ
...urces/SpeziFHIRHealthKit/FHIRStore+HealthKit.swift	0.00% <0.00%> (ø)
.../SpeziFHIR/Extensions/FHIRResource+Flattener.swift	0.00% <0.00%> (ø)

---

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 d368a1e...961b287. Read the comment docs.

[LeonNissen]

We've decided to remove the LLM functionality from SpeziFHIR and move it over to the application.

@philippzagar I saw your changes in #23. I have transferred them over into LLMonFHIR in: StanfordBDHG/LLMonFHIR#53

Feel free to review this pull request, however, it should only remove "application logic" 💯

[philippzagar]

@LeonNissen Like with the SpeziChat PR, the commits aren’t properly signed, which might cause a problem later on when we try to merge them.

How do we plan to support applications that depend on SpeziFHIR and the current SpeziFHIRLLM target functionality, such as LLMonFHIR and OwnYourData? Probably you and @PSchmiedmayer already have a plan there? Maybe a separate SPM package as a whole?

[PSchmiedmayer]

@philippzagar We discussed it and there will be a need for a dedicated module & package for that type of functionality in the long run. The placement in SpeziFHIR is not ideal and as we move to tagging a 1.0 for this package we need to further define its scope & focus. For now, removing the LLM parts will help us to clean things up. The elements are better positioned in the relevant apps or a dedicated package.

[philippzagar]

@PSchmiedmayer 100% agree, the SpeziFHIRLLM target was always intended as a kind of dumping ground for all FHIR-related LLM functionality, therefore less than ideal.
Great to see that improve 🚀