Proper SwiftUI

Pattern flagged for review

struct AnimalDetailView: View {
     let animal: Animal

     var body: some View {
         habitatSection
     }

     @ViewBuilder
     private var habitatSection: some View {
         AnimalInfoSection(
             header: "Habitat",
             info: animal.habitatName
         )
     }
}

A possible improvement

// In AnimalDetailView.body:
AnimalHabitatSection(
     habitatName: animal.habitatName
)

struct AnimalHabitatSection: View {
     let habitatName: String

     var body: some View {
         AnimalInfoSection(
             header: "Habitat",
             info: habitatName
         )
     }
}

Extract the section into a separate view struct, passing it only the information it needs to display its contents. When the parent updates, a new instance of AnimalHabitatSection is created. If its habitat name hasn't changed, SwiftUI can skip the execution of its body.

Pattern flagged for review

ForEach(walks) { walk in
     if walk.isShort {
         Text(walk.name)
     }
}

A possible improvement

ForEach(shortWalks) { walk in
     Text(walk.name)
}

Prepare shortWalks when the source data or filter changes. ForEach then receives a collection with known membership and produces exactly one row per element.