[appkit] `Navigator.pop` is called without properly checking if the modal has been opened

[AlexV525]

SDK Affected

AppKit

Describe the bug

reown_flutter/packages/reown_appkit/lib/modal/appkit_modal_impl.dart

Lines 1404 to 1411 in ac096cb

	// If we aren't open, then we can't and shouldn't close
	_close();
	if (_context != null) {
	final canPop = Navigator.of(_context!, rootNavigator: true).canPop();
	if (canPop) {
	Navigator.of(_context!, rootNavigator: true).pop();
	}
	}

The closeModal method calls navigation popping without checking if the modal has opened.

To Reproduce

// Select and connect a target wallet directly without opening the modal.
_appKitModal.selectWallet(targetWallet);
await _appKitModal.connectSelectedWallet();

Expected behavior

The navigation operation should not be called if the modal is not open.

[linear]

APKTN-697

[ignaciosantise]

Hey @AlexV525 thanks for reporting. What issues is this behavior causing you?

[AlexV525]

Hey @AlexV525 thanks for reporting. What issues is this behavior causing you?

As the top comment described: when the modal was not opened directly, e.g., connecting a specific wallet without using the modal, it will unexpectedly pop the current route.