Improvement Idea: restrict dynamic prompt args list as much as possible to prompt

[metachris]

Opening this for discussion.

I understand the prompt needs to read generic input strings. I think it would be beneficial to add function signatures as soon as possible, and restrict this dynamic args list to prompt.py as much as possible.

For instance neo.Prompt.Commands.LoadSmartContract.LoadContract: Instead of def LoadContract(args): it could/should be def LoadContract(path, params, return_type, needs_storage):

This would make the code a lot easier to understand, and also more resilient to errors and bugs.

[localhuman]

Agreed, would be easier to document as well :)

[ixje]

needs milestone please

[ixje]

is this still a valid issue/something to keep in mind? @metachris

[metachris]

Yes, I think this would still be an improvement.

[ixje]

Cleaning up issues and we're moving to a different CLI command structure.