Add better WorkflowSummary type

[ahmedhamidawan]

Related to #19777 (?)
Follows up on #19294 (comment)
Fixes #19681

Creates a WorkflowSummary type which we use in cases where StoredWorkflowDetailed is not needed (or is rather not the actual return from the backend in fact).

Then also adds the AnyWorkflow type to generalize usage:

export type AnyWorkflow = WorkflowSummary | StoredWorkflowDetailed;

Moves methods from workflows.services to the client/src/api/workflows.ts file

I moved some functions that have a typed response from workflows.services to workflows.ts to make it more uniform with what we do in other places (e.g.: datasets.ts).

How to test the changes?

(Select all options that apply)

• I've included appropriate automated tests.
• This is a refactoring of components with existing test coverage.
• Instructions for manual testing are as follows:
  1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

• I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

[ahmedhamidawan]

Does this improve on the concerns here @mvdbeek ?

[ahmedhamidawan]

I moved some functions that have a typed response from workflows.services to workflows.ts to make it more uniform with what we do in other places (e.g.: datasets.ts).

@davelopez Does what I have done here make sense?

[davelopez]

Does what I have done here make sense?

Yes, it does to me :)

[jmchilton]

Current errors: 10
Baseline errors: 11
vue-tsc error count did not increase.

xref #19681

[ahmedhamidawan]

@jmchilton I think this takes care of what #19681 reports then 🤔 ?

[mvdbeek]

Does this improve on the concerns here @mvdbeek ?

No. You'll want to have an extremely good reason to cast a defined type to something else. I think this should be limited to responses from non-fastapi endpoints.

[mvdbeek]

I think this should be limited to responses from non-fastapi endpoints.

I had forgotten that we have fastapi endpoints without defined models. This is fine then.

We don't have types in the first place so this is all moot.

[mvdbeek]

Thanks @ahmedhamidawan!