Derive method task inputs and outputs from the function signature

[woutdenolf]

PR summary

Generate a task class with inputs/outputs for task_type = "method" based on the function signature.

Pydantic models generated when possible. Fall-back to the non-typed input_names, optional_input_names, n_required_positional_inputs and output_names.

AI Disclosure

• No AI used
• AI tool Clause used for generate prototype

[codecov]

Codecov Report

❌ Patch coverage is 95.16129% with 6 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/ewokscore/methodtask.py	95.72%	5 Missing ⚠️
src/ewokscore/inittask.py	66.66%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[woutdenolf]

@loichuder In the current proto-type when the return type annotation is of a certain kind (pydantic model, named tuple, dataclass, TypeDict), we unpack the outputs, there is no opt-out.

Choices we have:

1. Add a new task_type. Costly since several places need to learn about the new type (ewokscore, all engine bindings, ewoksweb, ewoksjob). Workflow author decides.
2. Introduce a decorator to opt-in or opt-out unpacking. The task author decides.
3. Only unpack BaseOutputModel.

The first would be inline with method vs. ppfmethod for example. Any method could be either and the workflow author decides. But it is more costly though. Perhaps the way we implement task types needs to be revisited.

[loichuder]

Perhaps I jumped the gun and there will be no situation where someone wants to opt-out from this since it seems way more practical.

Should we more forward with unconditional unpacking and revisit if someone complains?

[woutdenolf]

I think it is was a fair remark and just like ppfmethod changes the way we are using python functions we should do the same (imo) for unpackedmethod (or whatever we call it).

Since it is another task_type (doesn't happen every day but it will again) I think this is also a simple use case to validate the approach proposed in #508.

In any case, all this came from a taste PR, not a real need so not urgent. Next major release.