refactor - Move Search Logic from Search.execute() to FhirEngine.search()

[itstanany]

Describe the Issue

• For Improving separation of concerns and single responsibility, I see search logic should be moved from Saerch.execute extension function to the fhir engine search method

Would you like to work on the issue?
Y

[jingtang10]

thanks for raising this - i actually think it might be cleaner to move the whole thing to the search pacakge... i see a future where the data access (CRUD), search, and sync APIs live in different packages.

can you see how that'll look like?

[jingtang10]

I commented here: #2541 (comment)

the search

https://github.com/itstanany/android-fhir/blob/601ebe18cd3b3d663fb4396dcf8058c60875c3eb/engine/src/main/java/com/google/android/fhir/FhirEngine.kt#L117

and count

https://github.com/itstanany/android-fhir/blob/601ebe18cd3b3d663fb4396dcf8058c60875c3eb/engine/src/main/java/com/google/android/fhir/FhirEngine.kt#L160

apis are defined in the com.google.android.fhir package. I think it might make more sense for them to be moved to the com.google.android.fhir.search package