Refactor signFromChildMultisig to reduce noisy terminal output

[maurelian]

The signFromChildMultisig function in MultisigTask.sol currently returns VmSafe.AccountAccess[] and Action[] which are then printed to the terminal.

This creates unnecessarily noisy output that makes it harder to focus on the important information needed for signing.

[TheOnma]

Hello @maurelian, I would love to work on this, can I be assigned this task?

[Cass402]

Hello @maurelian,
I've identified that the noisy terminal output is primarily due to the call to the print function within the simulateRun function. I propose removing (or commenting out) the print call in simulateRun to reduce unnecessary logs during simulation, which should allow users to focus on the critical signing information.

Does this change meet the expectations for resolving the issue, or would you prefer a more selective reduction of the output instead of removing it entirely? I’m happy to adjust the implementation based on your feedback.

Below is the relevant code snippet on where I believe the issue stems from:

        Action[] memory actions = build();
        VmSafe.AccountAccess[] memory accountAccesses = simulate(signatures, actions);
        validate(accountAccesses, actions);
        print(actions, accountAccesses, optionalChildMultisig, true); // This is what might be unnecessarily log the output to the console

[nsancheznez]

may I work on this

[JoseValperga]

Hi team,

I'd like to apply to work on this improvement. I have experience with Solidity and improving the clarity of terminal output to help users focus on the most relevant information.

I'm confident I can help make this flow cleaner and more user-friendly without affecting the contract’s functionality. Happy to jump in if selected.

Thanks for the opportunity.
Best regards,
Jose Valperga

[Mau1401]

Hi I'm an student from the ETH Kipu from Costa Rica and I would like to work on these issue to gain hands-on experience with solidity.