### Is there an existing issue for this?

- [X] I have searched the existing issues

### Current behaviour

crash

### Expected behaviour

no crash

### Steps to reproduce

* Compile 79e8247ddb166f9b980f40249b7372a502402a4d
* Type `migratewallet`

### Relevant log output

```
wallet/scriptpubkeyman.cpp:1915: std::optional<MigrationData> wallet::LegacyScriptPubKeyMan::MigrateToDescriptor(): Assertion `IsMine(desc_spk) != ISMINE_NO' failed.
Aborted (core dumped)
```


### How did you obtain Bitcoin Core

Compiled from source

### What version of Bitcoin Core are you using?

current master

### Operating system and version

Linux

### Machine specifications

_No response_
}
    const bool has_legacy_spkman(GetScriptPubKeyMan(OutputTypeLEGACY, /*internal
    if (has_legacy_spkman && any_pkh) {
        // Currently pkh is the only type supported by the LEGACY spkman
        return OutputTypeLEGACY;
    }
    if (has_bech32m_spkman) {
        return OutputTypeBECH32M;
    }
    if (has_bech32_spkman) {
        return OutputTypeBECH32;
    }
    // else use m_default_address_type for change
    return m_default_address_type;
}
void CWalCommitTransaction(CTransactionRef tx, mapValue_t mapValue, stdvector<stdpair<stdstring, stdstring>> orderForm)
{
    LOCK(cs_wal);
    WalLogPrintf("CommitTransaction\n%s", tx->ToString()); // NOLINT(bitcoin-unterminated-logprintf)
    // Add tx to wal, because if it has change it's also ours,
    // otheise just for transaction history.
    CWalTx* wtx = AddToWal(tx, TxStateInactive{}, [&](CWalTx& wtx, bool new_tx) {
        CHECK_NONFATAL(wtx.mapValue.empty());
        CHECK_NONFATAL(wtx.vOrderForm.empty());
        wtx.mapValue = stdmove(mapVal
        wtx.vOrderForm = stdmove(orderForm);
        wtx.fTimeReceivedIsTxTime = true;
        wtx.fFromMe = true;
        return true;
    });