View disappear action is not triggered when the view is disappeared

[johnqiang-cao]

Description

We have code in the view:
.onDisappear {
store.send(.viewDisappeared)
}
But viewDisappeared action is not triggered when the view is unloaded.

Checklist

• I have determined whether this bug is also reproducible in a vanilla SwiftUI project.
• If possible, I've reproduced the issue using the main branch of this package.
• This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior

The .viewDisappeared action should be triggered.

Actual behavior

The .viewDisappeared action is not triggered.

Reproducing project

ResourceCleanup 2.zip

The Composable Architecture version information

1.16.1

Destination operating system

iOS 18

Xcode version information

XCode 16.1

Swift Compiler version information

No response

[mbrandonw]

Hi @johnqiang-cao, this is to be expected, and there is even a purple runtime displaying in Xcode letting you know what you are doing isn't correct:

🟣 A "forEach" at "ComposableArchitecture/CaseReducer.swift:52" received an action for a missing element. …

Action:
ContentFeature.Path.Action.child(.viewDisappeared)

This is generally considered an application logic error, and can happen for a few reasons:

You cannot send an action in the onDisappear of a view because by the time that view modifier is invoked the feature's state has been nil'd out, and hence the reducer for the child feature cannot even be run.

What is it you are wanting to do in the onDisappear of the view?

If you really need to do this, you can always add it inside the navigationDestination:

} destination: { destination in
  switch destination.case {
  case .child(let childStore):
    ChildView(store: childStore)
      .onDisappear { store.send(.onChildDisappear…) }
  }
}

Since this isn't an issue with the library I am going to convert it to a discussion. Please feel free to continue the conversation over there!

[johnqiang-cao]

Hi @mbrandonw , thanks for response.
In the ChildFeature, we do have two state properties which are stuck in memory and cause memory leak if not clean:

@Reducer
struct ChildFeature: Sendable {
    @ObservableState
    struct State: ReducerState {
        var childModel = [ChildModel?](repeating: nil, count: 6)
        var customScenes = [ViewType: ImagingScene]()
        var imagingCache: ImagingCache = .init()
    } 

In Reducer body, we have action:

        case .viewDisappeared:
                state.boneModel = [ChildModel?](repeating: nil, count: 6)
                state.customScenes = [:]
                return .run { [state] _ in
                    await state.imagingCache.purgeLoaderCache()
               }

When the Child view is unloaded, from ChildView's .onDisappear, call store.send(.viewDisappeared) to avoid memory leak.

I do not think I can move them up to navigationDestination (with .onChildDisappear) since ChildModel and customScenes are sit inside ChildFeature.

Previously, above .viewDisappeard action was triggered and memory was clean. Recently we upgraded to XCode 16.1 and then found out .viewDisappeard action is not triggered anymore.

Any suggestion? Thanks.