From 4ac7bb187d172dcc5f4fcea36bcd63983750d325 Mon Sep 17 00:00:00 2001
From: Pulkit Kathuria <kevincobain2000@gmail.com>
Date: Sat, 18 May 2024 16:55:27 +0900
Subject: [PATCH] (feat) support same metric types (#20)

* (feat) support same metric types

* (ui) debug code

* (fix) tests fix

* (fix) package name
---
 app/frontend/package.json                     |   2 +-
 .../src/components/ReadmeDashboard.astro      | 125 +++++++++++-------
 app/pkg/badge_handler_test.go                 |   8 +-
 app/pkg/chart.go                              |  30 +++++
 app/pkg/chart_handler.go                      |  34 ++++-
 app/pkg/chart_handler_test.go                 |   8 +-
 app/pkg/pr_handler_test.go                    |   8 +-
 app/pkg/readme_handler.go                     |   1 +
 app/pkg/upload_handler.go                     |   4 +
 app/build.sh => build.sh                      |   2 +-
 10 files changed, 160 insertions(+), 62 deletions(-)
 rename app/build.sh => build.sh (80%)

diff --git a/app/frontend/package.json b/app/frontend/package.json
index 5606ab5..1a07ac5 100644
--- a/app/frontend/package.json
+++ b/app/frontend/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "instachart-frontend",
+  "name": "coveritup-frontend",
   "type": "module",
   "version": "1.0.0",
   "scripts": {
diff --git a/app/frontend/src/components/ReadmeDashboard.astro b/app/frontend/src/components/ReadmeDashboard.astro
index 716c569..d8d95b6 100644
--- a/app/frontend/src/components/ReadmeDashboard.astro
+++ b/app/frontend/src/components/ReadmeDashboard.astro
@@ -8,7 +8,7 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
   </p>
 
   <p
-    class="text-violet-200 font-bold text-center text-xl select-none"
+    class="text-violet-200 font-mono mt-3 font-bold text-center text-xl select-none"
     id="org-repo-name"
   >
   </p>
@@ -24,7 +24,9 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
   </p>
   <form class="text-left text-black select-none">
     <label for="countries" class="block mt-2 text-slate-500 text-sm"
-      >Choose a branch</label
+      >Choose from <span
+        class="font-bold text-slate-300"
+        x-text="branches.length"></span> branches</label
     >
 
     <select
@@ -55,23 +57,7 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
             :src="`//${url}/table?org=${org}&repo=${repo}&branch=${branch}&theme=dark`"
           />
         </template>
-        <table class="w-full select-none mb-5">
-          <tbody class="text-left">
-            <tr>
-              <th class="text-slate-700">Types</th>
-              <td class="font-semibold font-mono"
-                ><span x-text="types.length"></span></td
-              >
-            </tr>
-            <tr>
-              <th class="text-slate-700">Branches</th>
-              <td class="font-semibold font-mono"
-                ><span x-text="branches.length"></span></td
-              >
-            </tr>
-          </tbody>
-        </table>
-        <code class="font-mono text-slate-700 break-words text-sm">
+        <code class="font-mono text-slate-500 break-words text-sm">
           ![<span x-text="types[0].name"></span>](https://<span x-text="url"
           ></span>/table?org=<span x-text="types[0].org"></span>&repo=<span
             x-text="types[0].repo"></span>&branch=<span x-text="types[0].branch"
@@ -101,29 +87,29 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
         <table class="w-full">
           <tbody class="text-left">
             <tr>
-              <th class="text-slate-700 font-mono text-sm">?org</th>
-              <th class="text-slate-700 font-mono text-sm">=</th>
+              <th class="text-slate-400 font-mono text-sm">?org</th>
+              <th class="text-slate-400 font-mono text-sm">=</th>
               <td class="font-thin text-sm font-mono"
                 ><code x-text="org"></code></td
               >
             </tr>
             <tr>
-              <th class="text-slate-700 font-mono text-sm">&repo</th>
-              <th class="text-slate-700 font-mono text-sm">=</th>
+              <th class="text-slate-400 font-mono text-sm">&repo</th>
+              <th class="text-slate-400 font-mono text-sm">=</th>
               <td class="font-thin text-sm font-mono"
                 ><code x-text="repo"></code></td
               >
             </tr>
             <tr>
-              <th class="text-slate-700 font-mono text-sm">&branch</th>
-              <th class="text-slate-700 font-mono text-sm">=</th>
+              <th class="text-slate-400 font-mono text-sm">&branch</th>
+              <th class="text-slate-400 font-mono text-sm">=</th>
               <td class="font-thin text-sm font-mono"
                 ><code x-text="branch"></code></td
               >
             </tr>
             <tr>
-              <th class="text-slate-700 font-mono text-sm">&type</th>
-              <th class="text-slate-700 font-mono text-sm">=</th>
+              <th class="text-slate-400 font-mono text-sm">&type</th>
+              <th class="text-slate-400 font-mono text-sm">=</th>
               <td class="font-thin text-sm font-mono"
                 ><code x-text="types[0].name"></code></td
               >
@@ -134,28 +120,28 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
         <table class="w-full">
           <tbody class="text-left">
             <tr>
-              <th class="text-slate-700 font-mono text-sm">&width</th>
-              <th class="text-slate-700 font-mono text-sm">=</th>
+              <th class="text-slate-400 font-mono text-sm">&width</th>
+              <th class="text-slate-400 font-mono text-sm">=</th>
               <td class="font-mono text-sm">chart width</td>
             </tr>
             <tr>
-              <th class="text-slate-700 font-mono text-sm">&height</th>
-              <th class="text-slate-700 font-mono text-sm">=</th>
+              <th class="text-slate-400 font-mono text-sm">&height</th>
+              <th class="text-slate-400 font-mono text-sm">=</th>
               <td class="font-mono text-sm">chart height</td>
             </tr>
             <tr>
-              <th class="text-slate-700 font-mono text-sm">&line</th>
-              <th class="text-slate-700 font-mono text-sm">=</th>
+              <th class="text-slate-400 font-mono text-sm">&line</th>
+              <th class="text-slate-400 font-mono text-sm">=</th>
               <td class="font-mono text-sm">fill or nofill</td>
             </tr>
             <tr>
-              <th class="text-slate-700 font-mono text-sm">&theme</th>
-              <th class="text-slate-700 font-mono text-sm">=</th>
+              <th class="text-slate-400 font-mono text-sm">&theme</th>
+              <th class="text-slate-400 font-mono text-sm">=</th>
               <td class="font-mono text-sm">dark or light</td>
             </tr>
             <tr>
-              <th class="text-slate-700 font-mono text-sm">&output</th>
-              <th class="text-slate-700 font-mono text-sm">=</th>
+              <th class="text-slate-400 font-mono text-sm">&output</th>
+              <th class="text-slate-400 font-mono text-sm">=</th>
               <td class="font-mono text-sm">svg or png</td>
             </tr>
           </tbody>
@@ -180,9 +166,9 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
   >
     <h2 class="text-white font-bold text-xl pb-2 select-none"># Badges</h2>
     <p class="pb-6 font-semibold text-sm select-none">
-      Best Suited for README.md
+      Best Suited for README.md. Badges for each type.
     </p>
-    <dl class="grid grid-cols-1 gap-x-4 gap-y-8 sm:grid-cols-2">
+    <dl class="grid grid-cols-1 gap-x-4 gap-y-8 lg:grid-cols-3">
       <template class="mx-auto" x-for="t in types">
         <div class="sm:col-span-1 bg-slate-900 rounded-md p-5">
           <dt class="text-white font-bold select-none pb-2" x-text="t.name">
@@ -194,7 +180,7 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
                 :src="`//${url}/badge?org=${org}&repo=${repo}&branch=${branch}&type=${t.name}`"
               />
             </div>
-            <code class="font-mono text-slate-700 break-words text-sm">
+            <code class="font-mono text-slate-500 break-words text-sm">
               ![<span x-text="t.name"></span>](https://<span x-text="url"
               ></span>/badge?org=<span x-text="org"></span>&repo=<span
                 x-text="repo"></span>&branch=<span x-text="branch"
@@ -210,7 +196,7 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
   >
     <h2 class="text-white font-bold text-xl pb-2 select-none"># Mini Charts</h2>
     <p class="pb-6 font-semibold text-sm select-none">
-      Best Suited for README.md
+      Best Suited for README.md. Mini charts for each type.
     </p>
 
     <div class="text-center">
@@ -228,7 +214,7 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
         <div class="text-gray-500 bg-slate-900 mb-3 rounded-md p-5">
           <div class="text-white font-bold select-none pb-2" x-text="t.name">
           </div>
-          <code class="font-mono text-slate-700 break-words text-sm">
+          <code class="font-mono text-slate-500 break-words text-sm">
             ![<span x-text="t.name"></span>](https://<span x-text="url"
             ></span>/chart?org=<span x-text="org"></span>&repo=<span
               x-text="repo"></span>&branch=<span x-text="branch"
@@ -247,7 +233,7 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
       Visualize the full chart for trends on a branch over time for a specific
       type
     </p>
-    <dl class="grid grid-cols-1 gap-x-4 gap-y-8 sm:grid-cols-2">
+    <dl class="grid grid-cols-1 gap-x-4 gap-y-8 lg:grid-cols-3">
       <template class="mx-auto" x-for="t in types">
         <div class="sm:col-span-1 bg-slate-900 rounded-md p-5">
           <dt class="text-white font-bold select-none pb-2" x-text="t.name">
@@ -259,7 +245,7 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
                 :src="`//${url}/chart?org=${org}&repo=${repo}&branch=${branch}&type=${t.name}&theme=dark&line=fill`"
               />
             </div>
-            <code class="font-mono text-slate-700 break-words text-sm">
+            <code class="font-mono text-slate-500 break-words text-sm">
               ![<span x-text="t.name"></span>](https://<span x-text="url"
               ></span>/chart?org=<span x-text="org"></span>&repo=<span
                 x-text="repo"></span>&branch=<span x-text="branch"
@@ -270,6 +256,39 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
       </template>
     </dl>
   </div>
+  <div
+    class="pl-5 pr-5 pt-10 pb-10 text-slate-400 bg-slate-800 mt-24 rounded-lg"
+  >
+    <h2 class="text-white font-bold text-xl pb-2 select-none">
+      # Comparison Charts for Types
+    </h2>
+    <p class="pb-6 font-semibold text-sm select-none">
+      Visualize comparison charts for types for latest scores for same metrics
+    </p>
+    <dl class="grid grid-cols-1 gap-x-4 gap-y-8 lg:grid-cols-2">
+        <template class="mx-auto" x-for="(ts, metric) in groupedTypes" :key="metric">
+            <template x-if="ts.length > 1">
+            <div class="sm:col-span-1 bg-slate-900 rounded-md p-5">
+              <dt class="text-white font-bold select-none pb-2" x-text="metric">
+              </dt>
+              <dd class="text-gray-600">
+                <div class="pb-2 select-none flex justify-center items-center">
+                  <img
+                    is:raw
+                    :src="`//${url}/chart?org=${org}&repo=${repo}&branch=${branch}&types=${ts.map(t => t.name).join(',')}&theme=dark`"
+                    alt="Chart"
+                  />
+                </div>
+                <code class="font-mono text-slate-500 break-words text-sm">
+                    ![<span x-text="ts.map(t => t.name).join(',')"></span>](https://<span x-text="url"></span>/chart?org=<span x-text="org"></span>&repo=<span x-text="repo"></span>&branch=<span x-text="branch"></span>&types=<span x-text="ts.map(t => t.name).join(',')"></span>)
+                  </code>
+              </dd>
+            </div>
+          </template>
+          </template>
+      </dl>
+  </div>
+
   <div
     class="pl-5 pr-5 pt-10 pb-10 text-slate-400 bg-slate-800 mt-24 rounded-lg"
   >
@@ -291,7 +310,7 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
                 :src="`//${url}/chart?org=${org}&repo=${repo}&branches=all&type=${t.name}&theme=dark&line=fill`"
               />
             </div>
-            <code class="font-mono text-slate-700 break-words text-sm">
+            <code class="font-mono text-slate-500 break-words text-sm">
               ![<span x-text="t.name"></span>](https://<span x-text="url"
               ></span>/chart?org=<span x-text="org"></span>&repo=<span
                 x-text="repo"></span>&branches=all&type=<span x-text="t.name"
@@ -311,7 +330,7 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
     <p class="pb-6 font-semibold text-sm select-none">
       Visualize comparison charts for users over time for a specific type
     </p>
-    <dl class="grid grid-cols-1 gap-x-4 gap-y-8 sm:grid-cols-2">
+    <dl class="grid grid-cols-1 gap-x-4 gap-y-8 lg:grid-cols-3">
       <template class="mx-auto" x-for="t in types">
         <div class="sm:col-span-1 bg-slate-900 rounded-md p-5">
           <dt class="text-white font-bold select-none pb-2" x-text="t.name">
@@ -323,7 +342,7 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
                 :src="`//${url}/chart?org=${org}&repo=${repo}&users=all&type=${t.name}&theme=dark&line=fill`"
               />
             </div>
-            <code class="font-mono text-slate-700 break-words text-sm">
+            <code class="font-mono text-slate-500 break-words text-sm">
               ![<span x-text="t.name"></span>](https://<span x-text="url"
               ></span>/chart?org=<span x-text="org"></span>&repo=<span
                 x-text="repo"></span>&branch=<span x-text="branch"
@@ -343,6 +362,16 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
     window.history.pushState({ path: url.toString() }, "", url.toString());
     window.location.reload();
   };
+  const groupByMetric = (types) => {
+    return types.reduce((acc, type) => {
+      const metric = type.metric;
+      if (!acc[metric]) {
+        acc[metric] = [];
+      }
+      acc[metric].push(type);
+      return acc;
+    }, {});
+  };
 
   const currentURL = window.location.origin;
 
@@ -370,6 +399,8 @@ const baseURL = import.meta.env.PUBLIC_BASE_URL;
         branch = data.branch;
         updateBranch();
       }
+
+      data.groupedTypes = groupByMetric(data.types);
       data.url = baseURL.replace("https://", "").replace("http://", "");
       data.org = org;
       data.repo = repo;
diff --git a/app/pkg/badge_handler_test.go b/app/pkg/badge_handler_test.go
index c3ddb4e..082677e 100644
--- a/app/pkg/badge_handler_test.go
+++ b/app/pkg/badge_handler_test.go
@@ -39,25 +39,25 @@ func TestGetBadgeErrors(t *testing.T) {
 		{
 			Name:        "422",
 			Query:       `?repo=repo&branch=branch&type=type`,
-			ContentType: "application/json; charset=UTF-8",
+			ContentType: "application/json",
 			Status:      http.StatusUnprocessableEntity,
 		},
 		{
 			Name:        "422",
 			Query:       `?org=org&branch=branch&type=type`,
-			ContentType: "application/json; charset=UTF-8",
+			ContentType: "application/json",
 			Status:      http.StatusUnprocessableEntity,
 		},
 		{
 			Name:        "422",
 			Query:       `?org=org&repo=repo&type=type`,
-			ContentType: "application/json; charset=UTF-8",
+			ContentType: "application/json",
 			Status:      http.StatusUnprocessableEntity,
 		},
 		{
 			Name:        "422",
 			Query:       `?org=org&repo=repo&branch=branch`,
-			ContentType: "application/json; charset=UTF-8",
+			ContentType: "application/json",
 			Status:      http.StatusUnprocessableEntity,
 		},
 	}
diff --git a/app/pkg/chart.go b/app/pkg/chart.go
index b71ffe2..b57513b 100644
--- a/app/pkg/chart.go
+++ b/app/pkg/chart.go
@@ -157,6 +157,36 @@ func (e *Chart) GetInstaChartForUser(req *ChartRequest, t *models.Type) ([]byte,
 	return line.Get(xData, yyData, names, cReq)
 }
 
+// bar chart for all types with type names on x-axis and scores on y-axis
+func (e *Chart) GetInstaChartForTypes(req *ChartRequest, branch string, types []*models.Type) ([]byte, error) {
+	cReq := e.makeChartRequest(req, types[0])
+	bar := instachart.NewBarChart()
+
+	xData := []string{}
+	yData := []float64{}
+	zData := []float64{}
+	names := []string{}
+	var hasErr error
+	for _, t := range types {
+		ret, err := e.coverageModel.GetLatestBranchScore(req.Org, req.Repo, branch, t.Name)
+		if err != nil {
+			hasErr = err
+			break
+		}
+		xData = append(xData, t.Name)
+		yData = append(yData, ret.Score)
+		zData = append(zData, ret.Score)
+		names = append(names, t.Name)
+	}
+	if hasErr != nil {
+		return nil, hasErr
+	}
+	yyData := [][]float64{yData}
+	zzData := [][]float64{zData}
+
+	return bar.GetStacked(xData, yyData, zzData, names, cReq)
+}
+
 // bar chart with branch names on x-axis and scores on y-axis
 func (e *Chart) GetInstaChartForBranches(req *ChartRequest, t *models.Type) ([]byte, error) {
 	cReq := e.makeChartRequest(req, t)
diff --git a/app/pkg/chart_handler.go b/app/pkg/chart_handler.go
index f7b0031..5d093a9 100644
--- a/app/pkg/chart_handler.go
+++ b/app/pkg/chart_handler.go
@@ -2,7 +2,9 @@ package pkg
 
 import (
 	"net/http"
+	"strings"
 
+	"github.com/kevincobain2000/action-coveritup/models"
 	"github.com/labstack/echo/v4"
 	"github.com/mcuadros/go-defaults"
 )
@@ -23,7 +25,8 @@ type ChartRequest struct {
 	Branch     string `json:"branch" query:"branch" validate:"ascii" message:"ascii branch is required"`
 	User       string `json:"user" query:"user" validate:"ascii,excludes=/" message:"ascii user is required"`
 	BaseBranch string `json:"base_branch" query:"base_branch" validate:"ascii" message:"ascii base_branch is required"`
-	Type       string `json:"type" query:"type" validate:"ascii,required,excludes=/" message:"ascii type is required"`
+	Type       string `json:"type" query:"type" validate:"ascii,excludes=/" message:"ascii type is required"`
+	Types      string `json:"types" query:"types" validate:"ascii,excludes=/" message:"ascii types are required"`
 	Branches   string `json:"branches" query:"branches" validate:"ascii" message:"ascii branches is required"`
 	Users      string `json:"users" query:"users" validate:"ascii" message:"ascii users is required"`
 	PRNum      int    `json:"pr_num" query:"pr_num"`
@@ -49,6 +52,35 @@ func (h *ChartHandler) Get(c echo.Context) error {
 	if err != nil {
 		return echo.NewHTTPError(http.StatusUnprocessableEntity, msgs)
 	}
+	if req.Types == "" && req.Type == "" {
+		return echo.NewHTTPError(http.StatusUnprocessableEntity, "types or type is required")
+	}
+	if req.Types != "" {
+		if req.Branch == "" {
+			return echo.NewHTTPError(http.StatusUnprocessableEntity, "branch for types is required")
+		}
+		types := []*models.Type{}
+		for _, t := range strings.Split(req.Types, ",") {
+			tt, err := h.Chart.GetType(t)
+			if err != nil {
+				return echo.NewHTTPError(http.StatusUnprocessableEntity, err.Error())
+			}
+			if tt.ID == 0 {
+				return echo.NewHTTPError(http.StatusNotFound, "type from types not found")
+			}
+			types = append(types, tt)
+		}
+		for i := 1; i < len(types); i++ {
+			if types[i].Metric != types[0].Metric {
+				return echo.NewHTTPError(http.StatusUnprocessableEntity, "types must have same metric")
+			}
+		}
+		buf, err := h.Chart.GetInstaChartForTypes(req, req.Branch, types)
+		if err != nil {
+			return echo.NewHTTPError(http.StatusInternalServerError, err)
+		}
+		return ResponseMedia(c, buf, req.Output)
+	}
 
 	t, err := h.Chart.GetType(req.Type)
 	if err != nil {
diff --git a/app/pkg/chart_handler_test.go b/app/pkg/chart_handler_test.go
index 33b03b4..77cff84 100644
--- a/app/pkg/chart_handler_test.go
+++ b/app/pkg/chart_handler_test.go
@@ -33,24 +33,24 @@ func TestGetChartErrors(t *testing.T) {
 			Name:        "404",
 			Query:       `?org=org&repo=repo&branch=branch&type=type`,
 			Status:      http.StatusNotFound,
-			ContentType: "application/json; charset=UTF-8",
+			ContentType: "application/json",
 		},
 		{
 			Name:        "422",
 			Query:       `?repo=repo&branch=branch&type=type`,
-			ContentType: "application/json; charset=UTF-8",
+			ContentType: "application/json",
 			Status:      http.StatusUnprocessableEntity,
 		},
 		{
 			Name:        "422",
 			Query:       `?org=org&branch=branch&type=type`,
-			ContentType: "application/json; charset=UTF-8",
+			ContentType: "application/json",
 			Status:      http.StatusUnprocessableEntity,
 		},
 		{
 			Name:        "422",
 			Query:       `?org=org&repo=repo&branch=branch`,
-			ContentType: "application/json; charset=UTF-8",
+			ContentType: "application/json",
 			Status:      http.StatusUnprocessableEntity,
 		},
 	}
diff --git a/app/pkg/pr_handler_test.go b/app/pkg/pr_handler_test.go
index bede2a2..a00eaaa 100644
--- a/app/pkg/pr_handler_test.go
+++ b/app/pkg/pr_handler_test.go
@@ -32,25 +32,25 @@ func TestPRErrors(t *testing.T) {
 		{
 			Name:        "409",
 			Query:       `?org=org&repo=repo&branch=branch&base_branch=master&pr_num=2&type=type`,
-			ContentType: "application/json; charset=UTF-8",
+			ContentType: "application/json",
 			Status:      http.StatusConflict,
 		},
 		{
 			Name:        "422",
 			Query:       `?org=org&branch=branch&type=type`,
-			ContentType: "application/json; charset=UTF-8",
+			ContentType: "application/json",
 			Status:      http.StatusUnprocessableEntity,
 		},
 		{
 			Name:        "422",
 			Query:       `?org=org&repo=repo&type=type`,
-			ContentType: "application/json; charset=UTF-8",
+			ContentType: "application/json",
 			Status:      http.StatusUnprocessableEntity,
 		},
 		{
 			Name:        "422",
 			Query:       `?org=org&repo=repo&branch=branch`,
-			ContentType: "application/json; charset=UTF-8",
+			ContentType: "application/json",
 			Status:      http.StatusUnprocessableEntity,
 		},
 	}
diff --git a/app/pkg/readme_handler.go b/app/pkg/readme_handler.go
index 09d3057..b530ed1 100644
--- a/app/pkg/readme_handler.go
+++ b/app/pkg/readme_handler.go
@@ -49,6 +49,7 @@ func (h *ReadmeHandler) Get(c echo.Context) error {
 	if err != nil {
 		return echo.NewHTTPError(http.StatusInternalServerError, err)
 	}
+
 	res := &ReadmeResponse{
 		Types:    types,
 		Branches: branches,
diff --git a/app/pkg/upload_handler.go b/app/pkg/upload_handler.go
index d06309d..f5de701 100644
--- a/app/pkg/upload_handler.go
+++ b/app/pkg/upload_handler.go
@@ -3,6 +3,7 @@ package pkg
 import (
 	"net/http"
 	"os"
+	"strings"
 
 	"github.com/labstack/echo/v4"
 	"github.com/mcuadros/go-defaults"
@@ -43,6 +44,9 @@ func (h *UploadHandler) Post(c echo.Context) error {
 	if err != nil {
 		return echo.NewHTTPError(http.StatusUnprocessableEntity, msgs)
 	}
+	if strings.Contains(req.Type, ",") {
+		return echo.NewHTTPError(http.StatusUnprocessableEntity, "type cannot contain comma")
+	}
 
 	// set by default to api.github and "" on tests
 	if os.Getenv("GITHUB_API") != "" {
diff --git a/app/build.sh b/build.sh
similarity index 80%
rename from app/build.sh
rename to build.sh
index cc56ecd..2511d25 100755
--- a/app/build.sh
+++ b/build.sh
@@ -1,6 +1,6 @@
 #! /bin/bash
 
-cd frontend
+cd app/frontend
 npm install
 npm run build
 cd ..