Extract plugin version from manifest

build/manifest/main.go

@@ -13,9 +13,13 @@ import (
 const PluginIdGoFileTemplate = `package main
 
 const PluginId = "%s"
+const PluginVersion = "%s"
 `
 
-const PluginIdJsFileTemplate = `export default '%s';
+const PluginIdJsFileTemplate = `export default {
+    PluginId: '%s',
+    PluginVersion: '%s',
+};
 `
 
 func main() {
@@ -30,9 +34,12 @@ func main() {
 
 	cmd := os.Args[1]
 	switch cmd {
-	case "plugin_id":
+	case "id":
 		dumpPluginId(manifest)
 
+	case "version":
+		dumpPluginVersion(manifest)
+
 	case "has_server":
 		if manifest.HasServer() {
 			fmt.Printf("true")
@@ -81,25 +88,30 @@ func dumpPluginId(manifest *model.Manifest) {
 	fmt.Printf("%s", manifest.Id)
 }
 
+// dumpPluginVersion writes the plugin version from the given manifest to standard out
+func dumpPluginVersion(manifest *model.Manifest) {
+	fmt.Printf("%s", manifest.Version)
+}
+
 // applyManifest propagates the plugin_id into the server and webapp folders, as necessary
 func applyManifest(manifest *model.Manifest) error {
 	if manifest.HasServer() {
 		if err := ioutil.WriteFile(
-			"server/plugin_id.go",
+			"server/manifest.go",
-			[]byte(fmt.Sprintf(PluginIdGoFileTemplate, manifest.Id)),
+			[]byte(fmt.Sprintf(PluginIdGoFileTemplate, manifest.Id, manifest.Version)),
 			0644,
 		); err != nil {
-			return errors.Wrap(err, "failed to write server/plugin_id.go")
+			return errors.Wrap(err, "failed to write server/manifest.go")
 		}
 	}
 
 	if manifest.HasWebapp() {
 		if err := ioutil.WriteFile(
-			"webapp/src/plugin_id.js",
+			"webapp/src/manifest.js",
-			[]byte(fmt.Sprintf(PluginIdJsFileTemplate, manifest.Id)),
+			[]byte(fmt.Sprintf(PluginIdJsFileTemplate, manifest.Id, manifest.Version)),
 			0644,
 		); err != nil {
-			return errors.Wrap(err, "failed to open webapp/src/plugin_id.js")
+			return errors.Wrap(err, "failed to open webapp/src/manifest.js")
 		}
 	}
 

build/setup.mk

@@ -8,7 +8,7 @@ endif
 $(shell cd build/manifest && $(GO) build -o ../bin/manifest)
 
 # Extract the plugin id from the manifest.
-PLUGIN_ID ?= $(shell build/bin/manifest plugin_id)
+PLUGIN_ID ?= $(shell build/bin/manifest id)
 ifeq ($(PLUGIN_ID),)
     $(error "Cannot parse id from $(MANIFEST_FILE)")
 endif

server/manifest.go

@@ -1,3 +1,4 @@
 package main
 
 const PluginId = "com.mattermost.sample-plugin"
+const PluginVersion = "0.0.1"

webapp/src/index.js

@@ -1,4 +1,4 @@
-import PluginId from './plugin_id';
+import Manifest from './manifest';
 
 export default class Plugin {
     // eslint-disable-next-line no-unused-vars
@@ -7,4 +7,4 @@ export default class Plugin {
     }
 }
 
-window.registerPlugin(PluginId, new Plugin());
+window.registerPlugin(Manifest.PluginId, new Plugin());

webapp/src/manifest.js

@@ -0,0 +1,4 @@
+export default {
+    PluginId: 'com.mattermost.sample-plugin',
+    PluginVersion: '0.0.1',
+};

webapp/src/plugin_id.js

@@ -1 +0,0 @@
-export default 'com.mattermost.sample-plugin';