diff utilities/vampipe-convert.cpp @ 24:533ca5ca3404

More on reading and writing json messages

--- a/utilities/vampipe-convert.cpp	Fri May 20 18:05:02 2016 +0100
+++ b/utilities/vampipe-convert.cpp	Mon May 23 16:09:25 2016 +0100
@@ -10,31 +10,11 @@
 using namespace json11;
 using namespace vampipe;
 
-// Accepting JSON objects with two fields, "type" and "payload". The
+// Accepting JSON objects with two fields, "type" and "content". The
 // "type" string corresponds to the JSON schema filename
-// (e.g. "outputdescriptor") and the "payload" is the JSON object
+// (e.g. "outputdescriptor") and the "content" is the JSON object
 // encoded with that schema.
 
-Json
-json_input(string input)
-{
-    string err;
-    Json j = Json::parse(input, err);
-    if (err != "") {
-	throw VampJson::Failure("invalid json: " + err);
-    }
-    if (!j.is_object()) {
-	throw VampJson::Failure("object expected at top level");
-    }
-    if (!j["type"].is_string()) {
-	throw VampJson::Failure("string expected for type field");
-    }
-    if (!j["payload"].is_object()) {
-	throw VampJson::Failure("object expected for payload field");
-    }
-    return j;
-}
-
 class PreservingPluginHandleMapper : public PluginHandleMapper
 {
 public:
@@ -56,112 +36,63 @@
     Vamp::Plugin *m_plugin;
 };
 
-void 
-handle_input(::capnp::MallocMessageBuilder &message, string input)
-{
-    string err;
-
-    Json j = json_input(input);
-    string type = j["type"].string_value();
-    Json payload = j["payload"];
-
-    if (type == "configurationrequest") {
-	auto req = message.initRoot<ConfigurationRequest>();
-	PreservingPluginHandleMapper mapper;
-	VampnProto::buildConfigurationRequest
-	    (req, VampJson::toConfigurationRequest(payload, mapper), mapper);
-
-    } else if (type == "configurationresponse") {
-	auto resp = message.initRoot<ConfigurationResponse>();
-	VampnProto::buildConfigurationResponse
-	    (resp, VampJson::toConfigurationResponse(payload));
-
-    } else if (type == "feature") {
-	auto f = message.initRoot<Feature>();
-	VampnProto::buildFeature
-	    (f, VampJson::toFeature(payload));
-
-    } else if (type == "featureset") {
-	auto fs = message.initRoot<FeatureSet>();
-	VampnProto::buildFeatureSet
-	    (fs, VampJson::toFeatureSet(payload));
-
-    } else if (type == "loadrequest") {
-	auto req = message.initRoot<LoadRequest>();
-	VampnProto::buildLoadRequest
-	    (req, VampJson::toLoadRequest(payload));
-	
-    } else if (type == "loadresponse") {
-	auto resp = message.initRoot<LoadResponse>();
-	PreservingPluginHandleMapper mapper;
-	VampnProto::buildLoadResponse
-	    (resp, VampJson::toLoadResponse(payload, mapper), mapper);
-
-    } else if (type == "outputdescriptor") {
-	auto od = message.initRoot<OutputDescriptor>();
-	VampnProto::buildOutputDescriptor
-	    (od, VampJson::toOutputDescriptor(payload));
-
-    } else if (type == "parameterdescriptor") {
-	auto pd = message.initRoot<ParameterDescriptor>();
-	VampnProto::buildParameterDescriptor
-	    (pd, VampJson::toParameterDescriptor(payload));
-
-    } else if (type == "pluginconfiguration") {
-	auto pc = message.initRoot<PluginConfiguration>();
-	auto config = VampJson::toPluginConfiguration(payload);
-	VampnProto::buildPluginConfiguration(pc, config);
-
-    } else if (type == "pluginstaticdata") {
-	auto pc = message.initRoot<PluginStaticData>();
-	auto sd = VampJson::toPluginStaticData(payload);
- 	VampnProto::buildPluginStaticData(pc, sd);
-
-    } else if (type == "processrequest") {
-	auto p = message.initRoot<ProcessRequest>();
-	PreservingPluginHandleMapper mapper;
-	VampnProto::buildProcessRequest
-	    (p, VampJson::toProcessRequest(payload, mapper), mapper);
-
-    } else if (type == "realtime") {
-	auto b = message.initRoot<RealTime>();
-	VampnProto::buildRealTime
-	    (b, VampJson::toRealTime(payload));
-	
-    } else {
-	throw VampJson::Failure("unknown or unsupported JSON schema type " +
-				type);
-    }
-}
-
 void usage()
 {
     string myname = "vampipe-convert";
     cerr << "\n" << myname <<
-	": Convert Vamp request and response messages between formats\n\n"
-	"    Usage:  " << myname << " -i <informat> -o <outformat>\n\n"
-	"Where <informat> and <outformat> may be \"json\" or \"capnp\".\n"
-	"Messages are read from stdin and written to stdout.\n" << endl;
+	": Validate and convert Vamp request and response messages\n\n"
+	"    Usage: " << myname << " [-i <informat>] [-o <outformat>] request\n"
+	"           " << myname << " [-i <informat>] [-o <outformat>] response\n\n"
+	"    where\n"
+	"       <informat>: the format to read from stdin\n"
+	"           (\"json\" or \"capnp\", default is \"json\")\n"
+	"       <outformat>: the format to convert to and write to stdout\n"
+	"           (\"json\" or \"capnp\", default is \"json\")\n"
+	"       request|response: whether to expect Vamp request or response messages\n\n"
+	"If <informat> and <outformat> differ, convert from <informat> to <outformat>.\n"
+	"If <informat> and <outformat> are the same, just check validity of incoming\n"
+	"messages and pass them to output.\n\n";
+
     exit(2);
 }
 
 class RequestOrResponse
 {
 public:
+    enum Direction {
+	Request, Response
+    };
     enum Type {
 	List, Load, Configure, Process, Finish, Eof
     };
+
+    static Type typeForName(string name) {
+	if (name == "list") return List;
+	else if (name == "load") return Load;
+	else if (name == "configure") return Configure;
+	else if (name == "process") return Process;
+	else if (name == "finish") return Finish;
+	else if (name == "eof") return Eof;
+	else {
+	    throw runtime_error("unknown or unexpected request/response type \"" +
+				name + "\"");
+	}
+    }
+    
     RequestOrResponse() : // nothing by default
+	direction(Request),
 	type(Eof),
 	success(false),
 	finishPlugin(0) { }
 
+    Direction direction;
     Type type;
     bool success;
     string errorText;
 
     PreservingPluginHandleMapper mapper;
-    
+
+    vector<Vamp::HostExt::PluginStaticData> listResponse;
     Vamp::HostExt::LoadRequest loadRequest;
     Vamp::HostExt::LoadResponse loadResponse;
     Vamp::HostExt::ConfigurationRequest configurationRequest;
@@ -170,96 +101,219 @@
     Vamp::HostExt::ProcessResponse processResponse;
     Vamp::Plugin *finishPlugin;
     Vamp::HostExt::ProcessResponse finishResponse;
-    
 };
 
+Json
+convertRequestJson(string input)
+{
+    string err;
+    Json j = Json::parse(input, err);
+    if (err != "") {
+	throw VampJson::Failure("invalid json: " + err);
+    }
+    if (!j.is_object()) {
+	throw VampJson::Failure("object expected at top level");
+    }
+    if (!j["type"].is_string()) {
+	throw VampJson::Failure("string expected for type field");
+    }
+    if (!j["content"].is_object()) {
+	throw VampJson::Failure("object expected for content field");
+    }
+    return j;
+}
+
+Json
+convertResponseJson(string input)
+{
+    string err;
+    Json j = Json::parse(input, err);
+    if (err != "") {
+	throw VampJson::Failure("invalid json: " + err);
+    }
+    if (!j.is_object()) {
+	throw VampJson::Failure("object expected at top level");
+    }
+    if (!j["success"].is_bool()) {
+	throw VampJson::Failure("bool expected for success field");
+    }
+    if (!j["content"].is_object()) {
+	throw VampJson::Failure("object expected for content field");
+    }
+    return j;
+}
+
 RequestOrResponse
-readInputJson()
+readRequestJson()
 {
     RequestOrResponse rr;
+    rr.direction = RequestOrResponse::Request;
+
+    string input;
+    if (!getline(cin, input)) {
+	rr.type = RequestOrResponse::Eof;
+	return rr;
+    }
+    
+    Json j = convertRequestJson(input);
+    string type = j["type"].string_value();
+    rr.type = RequestOrResponse::typeForName(type);
+
+    if (rr.type == RequestOrResponse::Load) {
+	rr.loadRequest = VampJson::toVampRequest_Load(j);
+
+    } else if (rr.type == RequestOrResponse::Configure) {
+	rr.configurationRequest = VampJson::toVampRequest_Configure(j, rr.mapper);
+
+    } else if (rr.type == RequestOrResponse::Process) {
+	rr.processRequest = VampJson::toVampRequest_Process(j, rr.mapper);
+
+    } else if (rr.type == RequestOrResponse::Finish) {
+	rr.finishPlugin = VampJson::toVampRequest_Finish(j, rr.mapper);
+    }
+
+    return rr;
+}
+
+void
+writeRequestJson(RequestOrResponse &rr)
+{
+    Json j;
+
+    if (rr.type == RequestOrResponse::List) {
+	j = VampJson::fromVampRequest_List();
+	
+    } else if (rr.type == RequestOrResponse::Load) {
+	j = VampJson::fromVampRequest_Load(rr.loadRequest);
+	
+    } else if (rr.type == RequestOrResponse::Configure) {
+	j = VampJson::fromVampRequest_Configure(rr.configurationRequest, rr.mapper);
+	
+    } else if (rr.type == RequestOrResponse::Process) {
+	j = VampJson::fromVampRequest_Process(rr.processRequest, rr.mapper);
+	
+    } else if (rr.type == RequestOrResponse::Finish) {
+	j = VampJson::fromVampRequest_Finish(rr.finishPlugin, rr.mapper);
+    }
+
+    cout << j.dump() << endl;
+}
+
+RequestOrResponse
+readResponseJson()
+{
+    RequestOrResponse rr;
+    rr.direction = RequestOrResponse::Response;
+
     string input;
     if (!getline(cin, input)) {
 	rr.type = RequestOrResponse::Eof;
 	return rr;
     }
 
-    Json j = json_input(input);
+    Json j = convertResponseJson(input);
     string type = j["type"].string_value();
+    rr.type = RequestOrResponse::typeForName(type);
 
-    if (type == "list") {
-	rr.type = RequestOrResponse::List;
+    if (rr.type == RequestOrResponse::List) {
+	rr.listResponse = VampJson::toVampResponse_List(j);
+	
+    } else if (rr.type == RequestOrResponse::Load) {
+	rr.loadResponse = VampJson::toVampResponse_Load(j, rr.mapper);
 
-    } else if (type == "load") {
-	//!!! ah, we need a way to know whether we're dealing with a request or response here
-	rr.type = RequestOrResponse::Load;
-	rr.loadRequest = VampJson::toLoadRequest(j["content"]);
+    } else if (rr.type == RequestOrResponse::Configure) {
+	rr.configurationResponse = VampJson::toVampResponse_Configure(j);
 
-    } else if (type == "configure") {
-	rr.type = RequestOrResponse::Configure;
-	rr.configurationRequest =
-	    VampJson::toConfigurationRequest(j["content"], rr.mapper);
+    } else if (rr.type == RequestOrResponse::Process) {
+	rr.processResponse = VampJson::toVampResponse_Process(j);
 
-    } else if (type == "process") {
-	rr.type = RequestOrResponse::Process;
-	rr.processRequest =
-	    VampJson::toProcessRequest(j["content"], rr.mapper);
-
-    } else if (type == "finish") {
-	rr.type = RequestOrResponse::Finish;
-	//!!! VampJsonify
-	rr.finishPlugin = rr.mapper.handleToPlugin(j["content"]["pluginHandle"].int_value());
-
-    } else {
-	throw runtime_error("unknown or unexpected request/response type \"" +
-			    type + "\"");
+    } else if (rr.type == RequestOrResponse::Finish) {
+	rr.finishResponse = VampJson::toVampResponse_Finish(j);
     }
 
     return rr;
 }
 
+void
+writeResponseJson(RequestOrResponse &rr)
+{
+    Json j;
+
+    if (rr.type == RequestOrResponse::List) {
+	j = VampJson::fromVampResponse_List("", rr.listResponse);
+	
+    } else if (rr.type == RequestOrResponse::Load) {
+	j = VampJson::fromVampResponse_Load(rr.loadResponse, rr.mapper);
+    }
+
+    cout << j.dump() << endl;
+}
+
 RequestOrResponse
-readInput(string format)
+readInput(string format, RequestOrResponse::Direction direction)
 {
     if (format == "json") {
-	return readInputJson();
+	if (direction == RequestOrResponse::Request) {
+	    return readRequestJson();
+	} else {
+	    return readResponseJson();
+	}
     } else {
 	throw runtime_error("unknown or unimplemented format \"" + format + "\"");
     }
 }
 
 void
-writeOutput(string format, const RequestOrResponse &rr)
+writeOutput(string format, RequestOrResponse &rr)
 {
-    throw runtime_error("writeOutput not implemented yet");
-    
+    if (format == "json") {
+	if (rr.direction == RequestOrResponse::Request) {
+	    writeRequestJson(rr);
+	} else {
+	    writeResponseJson(rr);
+	}
+    } else {
+	throw runtime_error("unknown or unimplemented format \"" + format + "\"");
+    }
 }
 
 int main(int argc, char **argv)
 {
-    if (argc != 5) {
+    if (argc < 2) {
 	usage();
     }
 
-    string informat, outformat;
+    string informat = "json", outformat = "json";
+    RequestOrResponse::Direction direction;
+    bool haveDirection = false;
     
-    for (int i = 1; i + 1 < argc; ++i) {
+    for (int i = 1; i < argc; ++i) {
 
 	string arg = argv[i];
+	bool final = (i + 1 == argc);
 	
 	if (arg == "-i") {
-	    if (informat != "") usage();
+	    if (informat != "" || final) usage();
 	    else informat = argv[++i];
 
 	} else if (arg == "-o") {
-	    if (outformat != "") usage();
+	    if (outformat != "" || final) usage();
 	    else outformat = argv[++i];
 
+	} else if (arg == "request") {
+	    direction = RequestOrResponse::Request;
+	    haveDirection = true;
+
+	} else if (arg == "response") {
+	    direction = RequestOrResponse::Response;
+	    haveDirection = true;
+	    
 	} else {
 	    usage();
 	}
     }
 
-    if (informat == "" || outformat == "") {
+    if (informat == "" || outformat == "" || !haveDirection) {
 	usage();
     }
 
@@ -267,7 +321,7 @@
 
 	try {
 
-	    RequestOrResponse rr = readInput(informat);
+	    RequestOrResponse rr = readInput(informat, direction);
 	    if (rr.type == RequestOrResponse::Eof) break;
 	    writeOutput(outformat, rr);