Align Cicero Server API with Cicero CLI

[jeromesimeon]

Is your feature request related to a problem? Please describe.
Cicero server only supports a single version of the cicero trigger call through the /trigger endpoint. It might be interesting to extend the cicero server API to the rest of the commands (e.g., draft invoke parse).

Other improvements could include the ability to pass or create the template from a string (binary encoded) or JSON.

[adrijshikhar]

i would like to work on this issue.

[jeromesimeon]

i would like to work on this issue.

that's great! I think there might be already some work being done by @martinhalford though, so this may need some coordination.

[martinhalford]

Hi Ardrij,
I'm about to submit a PR today but very happy to see what you do and we can take the best approach.
Cheers,
...Martin

[jeromesimeon]

Hi Ardrij,
I'm about to submit a PR today but very happy to see what you do and we can take the best approach.
Cheers,
...Martin

An initial PR would be great. We can then refine and there will likely be some follow up work in extensions / testing / documentation.

[martinhalford]

Pull request submitted. I've implemented /draft and /parse. If anyone wants to have a crack at the other commands (e.g. /invoke) then go for it.

[jeromesimeon]

Pull request submitted. I've implemented /draft and /parse. If anyone wants to have a crack at the other commands (e.g. /invoke) then go for it.

Thanks! I've merged it.

I'm leaving this issue open for now, as I think there could be further improvements as mentioned in comments.

[ankushbhardwxj]

@jeromesimeon I would like to implement the other commands , if this issue is available 👍

[jeromesimeon]

@martinhalford Can we consider this issue closed, or is there more work to do on the Cicero server API?