ICredbull.VaultParams should be broken up.

[lucasia]

https://github.com/credbull/credbull-defi/blob/8a910043673f8c3e5d7f68f6e5e496e138806786/packages/contracts/src/interface/ICredbull.sol#L12C1-L28C6

We have some great patterns with Plugins to extend Vault Behaviour and Factories to create different vault types. However, we are sharing the same VaultParams across all Vaults and Plugin types. This means the VaultParams will grow forever. :)

Let's split this VaultParams up, and push the logic into the VaultFactories. For example:

BaseVault only really needs the following:

 struct ContractRoles {
        address owner;
        address operator;
    }

    //Struct defining parameters for a vault
    struct BaseVaultParams {
        IERC20 asset;
        string shareName;
        string shareSymbol;
    }

Adding a WindowPlugin we need to also provide:

    // Window add-on
    struct WindowParams {
        uint256 opensAt;
        uint256 closesAt;
    }

    struct WindowVaultParams {
        WindowParams depositWindow;
        WindowParams matureWindow;
    }

This is likely a large refactoring across contracts, api, sdk and scripts. So, suggest we keep this in mind, but not start on it without improving our test cover and merge our scripts and sdk.

[lucasia]

Closed, see #48