Remove unused binary option from plugin configs

Remove the unimplemented "binary" option from all plugin config files.
Setting this option to something other than "nodePackage" would
supposedly run a binary and feed it with the stdout of the factorio
proccess, but that was never implemented.

CHANGELOG.md

@@ -58,6 +58,7 @@ Version 2.0.0
   instance is now three separate commands.
 - Removed oddball limits to slaves.json size.
 - Moved slave specific configuration into its own configuration file.
+- Removed unused binary option from plugin config.
 
 ### Breaking Changes
 

client.js

@@ -690,7 +690,7 @@ async function instanceManagement(slaveConfig, instance, instanceconfig, server,
 		if(!global.subscribedFiles) {
 			global.subscribedFiles = {};
 		}
-		if(pluginConfig.binary == "nodePackage" && pluginConfig.enabled){
+		if (pluginConfig.enabled) {
 			// require plugin class and execute it
 			let pluginClass = require(path.resolve(pluginConfig.pluginPath, "index"));
 			plugins[i] = new pluginClass(combinedConfig, async function(data, callback){

sharedPlugins/UPSdisplay/config.js

@@ -1,16 +1,11 @@
 /*
 	Example config file for example Clusterio plugin
-	Binary will be spawned using child_process.spawn and get input from stdin and text from stdout will be ran as
-	commands in the factorio server.
 */
 
 module.exports = {
 	// Name of package. For display somewhere I guess.
 	name: "UPSdisplay",
 	version: "2.0.0",
-	// Binary entrypoint for plugin. Don't let it crash. Stdout is sent to game as server chat (run /c commands from here for interface)
-	// Make sure its cross platform somehow.
-	binary: "nodePackage",
 	description: "Internal clusterio plugin to get the worlds UPS and display it on master",
 	// We'll send everything in this file to your stdin. Beware.
 	scriptOutputFileSubscription: "UPSdisplay.txt",

sharedPlugins/clusterioMod/config.js

@@ -5,7 +5,6 @@ module.exports = {
 	// Name of package. For display somewhere I guess.
 	name: "clusterioMod",
 	version: "1.0.0",
-	binary: "nodePackage",
 	description: "Server side handling of functionality in the clusterio lua mod",
 	masterPlugin: "masterPlugin.js",
 }

sharedPlugins/globalChat/config.js

@@ -5,6 +5,5 @@ module.exports = {
 	// Name of package. For display somewhere I guess.
 	name: "globalChat",
 	version: "1.0.0",
-	binary: "nodePackage",
 	description: "Clusterio plugin to allow for chat between instances.",
 }

sharedPlugins/inventoryImports/config.js

@@ -10,9 +10,6 @@ module.exports = {
 	// Name of package. For display somewhere I guess.
 	name: "inventoryImports",
 	version: "1.0.0",
-	// Binary entrypoint for plugin. Don't let it crash. Stdout is sent to game as server chat (run /c commands from here for interface)
-	// Make sure its cross platform somehow.
-	binary: "nodePackage",
 	arguments: ["index.js"],
 	description: "Clusterio plugin to fill character logistic slots with master imports",
 	// We'll send everything in this file to your stdin. Beware.

sharedPlugins/researchSync/config.js

@@ -4,9 +4,6 @@ module.exports = {
     // Name of package. For display somewhere I guess.
     name: "researchSync",
     version: "1.0.0",
-    // Binary entrypoint for plugin. Don't let it crash. Stdout is sent to game as server chat (run /c commands from here for interface)
-    // Make sure its cross platform somehow.
-    binary: "nodePackage",
     arguments: ["index.js"],
     description: "syncs research across servers",
     // We'll send everything in this file to your stdin. Beware.

sharedPlugins/statisticsExporter/config.js

@@ -4,7 +4,6 @@
 module.exports = {
 	name: "statisticsExporter",
 	version: "1.0.0",
-	binary: "nodePackage",
 	description: "Gathers ingame stats such as production, consumption, kills and buildings placed",
 	masterPlugin: "masterPlugin.js"
 }