feat: support annotation output format in harness - #18
Conversation
|
Hey @Julian, I have updated python-jschon harness for annotation support. |
|
Nice! Will have a look (looks like I didn't have notifications on) |
|
It seems smoke tests send a basic payload without the new "output" field, which caused a TypeError. Should we pass a default value |
|
That would sound like an upstream bug in the main repo -- it shouldn't be possible to send invalid requests to harnesses so if that's happening from smoke something very strange is happening. We likely should smoke test both kinds of input as well. |
|
I think you are right, this is an upstream issue. The smoke test in Bowtie on main does send Also you're right that the smoke test will always just send |
This updates the python-jschon harness to support the new output argument.
Changes
cmd_runto accept the new output parameteroutput == "annotations", uses jschon'sresult.output("basic")to extract annotations and transforms them into Bowtie's expected annotation formatoutput == "flag", returns the standard{"valid": bool}response as before