Add Vue linting (#13447)

* Add Vue linting

Turns out the .vue files were not linted at all, so I added that as well
as re-indented the file to 2-space and fixed all reasonable issues that
cam up except one case of a unintended side effect for which I have no
idea how to fix it, so the rule was disabled.

* misc tweaks

* update lockfile

* use overrides to include .vue files

* treat warnings as errors on lint-frontend

* also treat stylelint warnings as errors

* use equal sign syntax

Co-authored-by: Lauris BH <lauris@nix.lv>
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>

.eslintrc

@@ -11,6 +11,10 @@ parserOptions:
 plugins:
   - eslint-plugin-unicorn
   - eslint-plugin-import
+  - eslint-plugin-vue
+
+extends:
+  - plugin:vue/recommended
 
 env:
   es2021: true
@@ -24,7 +28,7 @@ globals:
   u2fApi: false
 
 overrides:
-  - files: ["web_src/**/*.js"]
+  - files: ["web_src/**/*.js", "web_src/**/*.vue"]
     env:
       browser: true
       jquery: true
@@ -387,6 +391,11 @@ rules:
   use-isnan: [2]
   valid-typeof: [2, {requireStringLiterals: true}]
   vars-on-top: [0]
+  vue/attributes-order: [0]
+  vue/component-definition-name-casing: [0]
+  vue/html-closing-bracket-spacing: [0]
+  vue/max-attributes-per-line: [0]
+  vue/one-component-per-file: [0]
   wrap-iife: [2, inside]
   wrap-regex: [0]
   yield-star-spacing: [2, after]

Makefile

@@ -312,8 +312,8 @@ lint: lint-frontend lint-backend
 
 .PHONY: lint-frontend
 lint-frontend: node_modules
-	npx eslint web_src/js build webpack.config.js
+	npx eslint --max-warnings=0 web_src/js build webpack.config.js
-	npx stylelint web_src/less
+	npx stylelint --max-warnings=0 web_src/less
 
 .PHONY: lint-backend
 lint-backend: golangci-lint revive vet

package-lock.json

@@ -5195,6 +5195,26 @@
         }
       }
     },
+    "eslint-plugin-vue": {
+      "version": "7.1.0",
+      "resolved": "https://registry.npmjs.org/eslint-plugin-vue/-/eslint-plugin-vue-7.1.0.tgz",
+      "integrity": "sha512-9dW7kj8/d2IkDdgNpvIhJdJ3XzU3x4PThXYMzWt49taktYnGyrTY6/bXCYZ/VtQKU9kXPntPrZ41+8Pw0Nxblg==",
+      "dev": true,
+      "requires": {
+        "eslint-utils": "^2.1.0",
+        "natural-compare": "^1.4.0",
+        "semver": "^7.3.2",
+        "vue-eslint-parser": "^7.1.1"
+      },
+      "dependencies": {
+        "semver": {
+          "version": "7.3.2",
+          "resolved": "https://registry.npmjs.org/semver/-/semver-7.3.2.tgz",
+          "integrity": "sha512-OrOb32TeeambH6UrhtShmF7CRDqhL6/5XpPNp2DuRH6+9QLw/orhp72j87v8Qa1ScDkvrrBNpZcDejAirJmfXQ==",
+          "dev": true
+        }
+      }
+    },
     "eslint-scope": {
       "version": "4.0.3",
       "resolved": "https://registry.npmjs.org/eslint-scope/-/eslint-scope-4.0.3.tgz",
@@ -14005,6 +14025,49 @@
         "v-tooltip": "^2.0.0-rc.32"
       }
     },
+    "vue-eslint-parser": {
+      "version": "7.1.1",
+      "resolved": "https://registry.npmjs.org/vue-eslint-parser/-/vue-eslint-parser-7.1.1.tgz",
+      "integrity": "sha512-8FdXi0gieEwh1IprIBafpiJWcApwrU+l2FEj8c1HtHFdNXMd0+2jUSjBVmcQYohf/E72irwAXEXLga6TQcB3FA==",
+      "dev": true,
+      "requires": {
+        "debug": "^4.1.1",
+        "eslint-scope": "^5.0.0",
+        "eslint-visitor-keys": "^1.1.0",
+        "espree": "^6.2.1",
+        "esquery": "^1.0.1",
+        "lodash": "^4.17.15"
+      },
+      "dependencies": {
+        "acorn": {
+          "version": "7.4.1",
+          "resolved": "https://registry.npmjs.org/acorn/-/acorn-7.4.1.tgz",
+          "integrity": "sha512-nQyp0o1/mNdbTO1PO6kHkwSrmgZ0MT/jCCpNiwbUjGoRN4dlBhqJtoQuCnEOKzgTVwg0ZWiCoQy6SxMebQVh8A==",
+          "dev": true
+        },
+        "eslint-scope": {
+          "version": "5.1.1",
+          "resolved": "https://registry.npmjs.org/eslint-scope/-/eslint-scope-5.1.1.tgz",
+          "integrity": "sha512-2NxwbF/hZ0KpepYN0cNbo+FN6XoK7GaHlQhgx/hIZl6Va0bF45RQOOwhLIy8lQDbuCiadSLCBnH2CFYquit5bw==",
+          "dev": true,
+          "requires": {
+            "esrecurse": "^4.3.0",
+            "estraverse": "^4.1.1"
+          }
+        },
+        "espree": {
+          "version": "6.2.1",
+          "resolved": "https://registry.npmjs.org/espree/-/espree-6.2.1.tgz",
+          "integrity": "sha512-ysCxRQY3WaXJz9tdbWOwuWr5Y/XrPTGX9Kiz3yoUXwW0VZ4w30HTkQLaGx/+ttFjF8i+ACbArnB4ce68a9m5hw==",
+          "dev": true,
+          "requires": {
+            "acorn": "^7.1.1",
+            "acorn-jsx": "^5.2.0",
+            "eslint-visitor-keys": "^1.1.0"
+          }
+        }
+      }
+    },
     "vue-hot-reload-api": {
       "version": "2.3.4",
       "resolved": "https://registry.npmjs.org/vue-hot-reload-api/-/vue-hot-reload-api-2.3.4.tgz",

package.json

@@ -54,6 +54,7 @@
     "eslint": "7.11.0",
     "eslint-plugin-import": "2.22.1",
     "eslint-plugin-unicorn": "23.0.0",
+    "eslint-plugin-vue": "7.1.0",
     "stylelint": "13.7.2",
     "stylelint-config-standard": "20.0.0",
     "svgo": "1.3.2",

web_src/js/components/ActivityHeatmap.vue

@@ -1,79 +1,71 @@
 <template>
-    <div>
+  <div>
-        <div v-show="isLoading">
+    <div v-show="isLoading">
-            <slot name="loading"></slot>
+      <slot name="loading"/>
-        </div>
-        <h4 class="total-contributions" v-if="!isLoading">
-            {{ totalContributions }} total contributions in the last 12 months
-        </h4>
-        <calendar-heatmap v-show="!isLoading" :locale="locale" :no-data-text="locale.no_contributions" :tooltip-unit="locale.contributions" :end-date="endDate" :values="values" :range-color="colorRange"/>
     </div>
+    <h4 v-if="!isLoading" class="total-contributions">
+      {{ values.length }} total contributions in the last 12 months
+    </h4>
+    <calendar-heatmap
+      v-show="!isLoading"
+      :locale="locale"
+      :no-data-text="locale.no_contributions"
+      :tooltip-unit="locale.contributions"
+      :end-date="endDate"
+      :values="values"
+      :range-color="colorRange"
+    />
+  </div>
 </template>
-
 <script>
 import {CalendarHeatmap} from 'vue-calendar-heatmap';
 const {AppSubUrl, heatmapUser} = window.config;
 
 export default {
-    name: "ActivityHeatmap",
+  name: 'ActivityHeatmap',
-    components: {
+  components: {CalendarHeatmap},
-        CalendarHeatmap
+  data: () => ({
+    isLoading: true,
+    colorRange: [],
+    endDate: null,
+    values: [],
+    suburl: AppSubUrl,
+    user: heatmapUser,
+    locale: {
+      contributions: 'contributions',
+      no_contributions: 'No contributions',
     },
-    data() {
+  }),
-        return {
+  mounted() {
-            isLoading: true,
+    this.colorRange = [
-            colorRange: [],
+      this.getColor(0),
-            endDate: null,
+      this.getColor(1),
-            values: [],
+      this.getColor(2),
-            totalContributions: 0,
+      this.getColor(3),
-            suburl: AppSubUrl,
+      this.getColor(4),
-            user: heatmapUser,
+      this.getColor(5)
-            locale: {
+    ];
-                contributions: 'contributions',
+    this.endDate = new Date();
-                no_contributions: 'No contributions',
+    this.loadHeatmap(this.user);
-            },
+  },
-        };
+  methods: {
+    async loadHeatmap(userName) {
+      const res = await fetch(`${this.suburl}/api/v1/users/${userName}/heatmap`);
+      const data = await res.json();
+      this.values = data.map(({contributions, timestamp}) => {
+        return {date: new Date(timestamp * 1000), count: contributions};
+      });
+      this.isLoading = false;
     },
-    mounted() {
+    getColor(idx) {
-        this.colorRange = [
+      const el = document.createElement('div');
-            this.getColor(0),
+      el.className = `heatmap-color-${idx}`;
-            this.getColor(1),
+      document.body.appendChild(el);
-            this.getColor(2),
+      const color = getComputedStyle(el).backgroundColor;
-            this.getColor(3),
+      document.body.removeChild(el);
-            this.getColor(4),
+      return color;
-            this.getColor(5)
+    }
-        ];
+  },
-        this.endDate = new Date();
+};
-        this.loadHeatmap(this.user);
-    },
-    methods: {
-        loadHeatmap(userName) {
-            const self = this;
-            $.get(`${this.suburl}/api/v1/users/${userName}/heatmap`, (chartRawData) => {
-                const chartData = [];
-                for (let i = 0; i < chartRawData.length; i++) {
-                    self.totalContributions += chartRawData[i].contributions;
-                    chartData[i] = {date: new Date(chartRawData[i].timestamp * 1000), count: chartRawData[i].contributions};
-                }
-                self.values = chartData;
-                self.isLoading = false;
-            });
-        },
-        getColor(idx) {
-            const el = document.createElement('div');
-            el.className = `heatmap-color-${idx}`;
-            document.body.appendChild(el);
-
-            const color = getComputedStyle(el).backgroundColor;
-
-            document.body.removeChild(el);
-
-            return color;
-        }
-    },
-}
 </script>
-
+<style scoped/>
-<style scoped>
-
-</style>

web_src/js/components/ActivityTopAuthors.vue

@@ -1,102 +1,101 @@
 <template>
-    <div>
+  <div>
-        <div class="activity-bar-graph" ref="style" style="width:0px;height:0px"></div>
+    <div class="activity-bar-graph" ref="style" style="width:0px;height:0px"/>
-        <div class="activity-bar-graph-alt" ref="altStyle" style="width:0px;height:0px"></div>
+    <div class="activity-bar-graph-alt" ref="altStyle" style="width:0px;height:0px"/>
-        <vue-bar-graph
+    <vue-bar-graph
-            :points="graphData"
+      :points="graphData"
-            :show-x-axis="true"
+      :show-x-axis="true"
-            :show-y-axis="false"
+      :show-y-axis="false"
-            :show-values="true"
+      :show-values="true"
-            :width="graphWidth"
+      :width="graphWidth"
-            :bar-color="colors.barColor"
+      :bar-color="colors.barColor"
-            :text-color="colors.textColor"
+      :text-color="colors.textColor"
-            :text-alt-color="colors.textAltColor"
+      :text-alt-color="colors.textAltColor"
-            :height="100"
+      :height="100"
-            :label-height="20"
+      :label-height="20"
-        >
+    >
-            <template v-slot:label="opt">
+      <template #label="opt">
-                <g v-for="(author, idx) in authors" :key="author.position">
+        <g v-for="(author, idx) in authors" :key="author.position">
-                    <a
+          <a
-                        v-if="opt.bar.index === idx && author.home_link !== ''"
+            v-if="opt.bar.index === idx && author.home_link !== ''"
-                        :href="author.home_link"
+            :href="author.home_link"
-                    >
+          >
-                        <image
+            <image
-                            :x="`${opt.bar.midPoint - 10}px`"
+              :x="`${opt.bar.midPoint - 10}px`"
-                            :y="`${opt.bar.yLabel}px`"
+              :y="`${opt.bar.yLabel}px`"
-                            height="20"
+              height="20"
-                            width="20"
+              width="20"
-                            :href="author.avatar_link"
+              :href="author.avatar_link"
-                        />
+            />
-                    </a>
+          </a>
-                    <image
+          <image
-                        v-else-if="opt.bar.index === idx"
+            v-else-if="opt.bar.index === idx"
-                        :x="`${opt.bar.midPoint - 10}px`"
+            :x="`${opt.bar.midPoint - 10}px`"
-                        :y="`${opt.bar.yLabel}px`"
+            :y="`${opt.bar.yLabel}px`"
-                        height="20"
+            height="20"
-                        width="20"
+            width="20"
-                        :href="author.avatar_link"
+            :href="author.avatar_link"
-                    />
+          />
-                </g>
+        </g>
-            </template>
+      </template>
-            <template v-slot:title="opt">
+      <template #title="opt">
-                <tspan v-for="(author, idx) in authors" :key="author.position"><tspan v-if="opt.bar.index === idx">{{ author.name }}</tspan></tspan>
+        <tspan v-for="(author, idx) in authors" :key="author.position">
-            </template>
+          <tspan v-if="opt.bar.index === idx">
-        </vue-bar-graph>
+            {{ author.name }}
-    </div>
+          </tspan>
+        </tspan>
+      </template>
+    </vue-bar-graph>
+  </div>
 </template>
-
 <script>
 import VueBarGraph from 'vue-bar-graph';
 
 export default {
-    components: {
+  components: {VueBarGraph},
-        VueBarGraph,
+  props: {
+    data: {type: Array, default: () => []},
+  },
+  data: () => ({
+    colors: {
+      barColor: 'green',
+      textColor: 'black',
+      textAltColor: 'white',
     },
-    props: {
+  }),
-        data: { type: Array, default: () => [] },
+  computed: {
-    },
+    graphData() {
-    mounted() {
+      return this.data.map((item) => {
-        const st = window.getComputedStyle(this.$refs.style);
-        const stalt = window.getComputedStyle(this.$refs.altStyle);
-
-        this.colors.barColor = st.backgroundColor;
-        this.colors.textColor = st.color;
-        this.colors.textAltColor = stalt.color;
-    },
-    data() {
         return {
-            colors: {
+          value: item.commits,
-                barColor: 'green',
+          label: item.name,
-                textColor: 'black',
-                textAltColor: 'white',
-            },
         };
+      });
     },
-    computed: {
+    authors() {
-        graphData() {
+      return this.data.map((item, idx) => {
-            return this.data.map((item) => {
+        return {
-                return {
+          position: idx + 1,
-                    value: item.commits,
+          ...item,
-                    label: item.name,
+        };
-                };
+      });
-            });
-        },
-        authors() {
-            return this.data.map((item, idx) => {
-                return {
-                    position: idx+1,
-                    ...item,
-                }
-            });
-        },
-        graphWidth() {
-            return this.data.length * 40;
-        },
     },
-    methods: {
+    graphWidth() {
-        hasHomeLink(i) {
+      return this.data.length * 40;
-            return this.graphData[i].homeLink !== '' && this.graphData[i].homeLink !== null;
+    },
-        },
+  },
-    }
+  mounted() {
-}
+    const st = window.getComputedStyle(this.$refs.style);
-</script>
+    const stalt = window.getComputedStyle(this.$refs.altStyle);
+
+    this.colors.barColor = st.backgroundColor;
+    this.colors.textColor = st.color;
+    this.colors.textAltColor = stalt.color;
+  },
+  methods: {
+    hasHomeLink(i) {
+      return this.graphData[i].homeLink !== '' && this.graphData[i].homeLink !== null;
+    },
+  }
+};
+</script>

web_src/js/index.js

@@ -2687,7 +2687,15 @@ function initVueComponents() {
       .replace(/height="[0-9]+"/, 'v-bind:height="size"')
       .replace(/width="[0-9]+"/, 'v-bind:width="size"');
 
-    Vue.component(name, {props: ['size'], template});
+    Vue.component(name, {
+      props: {
+        size: {
+          type: String,
+          default: '16',
+        },
+      },
+      template,
+    });
   }
 
   const vueDelimeters = ['${', '}'];
@@ -2710,7 +2718,7 @@ function initVueComponents() {
       },
       organizations: {
         type: Array,
-        default: []
+        default: () => [],
       },
       isOrganization: {
         type: Boolean,
@@ -3050,17 +3058,19 @@ function initVueApp() {
   initVueComponents();
 
   new Vue({
-    delimiters: ['${', '}'],
     el,
-    data: {
+    delimiters: ['${', '}'],
-      searchLimit: Number((document.querySelector('meta[name=_search_limit]') || {}).content),
-      suburl: AppSubUrl,
-      uid: Number((document.querySelector('meta[name=_context_uid]') || {}).content),
-      activityTopAuthors: window.ActivityTopAuthors || [],
-    },
     components: {
       ActivityTopAuthors,
     },
+    data: () => {
+      return {
+        searchLimit: Number((document.querySelector('meta[name=_search_limit]') || {}).content),
+        suburl: AppSubUrl,
+        uid: Number((document.querySelector('meta[name=_context_uid]') || {}).content),
+        activityTopAuthors: window.ActivityTopAuthors || [],
+      };
+    },
   });
 }
 
@@ -3105,24 +3115,30 @@ function initFilterBranchTagDropdown(selector) {
     });
     $data.remove();
     new Vue({
-      delimiters: ['${', '}'],
       el: this,
+      delimiters: ['${', '}'],
       data,
+      computed: {
+        filteredItems() {
+          const items = this.items.filter((item) => {
+            return ((this.mode === 'branches' && item.branch) || (this.mode === 'tags' && item.tag)) &&
+              (!this.searchTerm || item.name.toLowerCase().includes(this.searchTerm.toLowerCase()));
+          });
 
-      beforeMount() {
+          // no idea how to fix this so linting rule is disabled instead
-        const vm = this;
+          this.active = (items.length === 0 && this.showCreateNewBranch ? 0 : -1); // eslint-disable-line vue/no-side-effects-in-computed-properties
-
+          return items;
-        this.noResults = vm.$el.getAttribute('data-no-results');
+        },
-        this.canCreateBranch = vm.$el.getAttribute('data-can-create-branch') === 'true';
+        showNoResults() {
-
+          return this.filteredItems.length === 0 && !this.showCreateNewBranch;
-        document.body.addEventListener('click', (event) => {
+        },
-          if (vm.$el.contains(event.target)) {
+        showCreateNewBranch() {
-            return;
+          if (!this.canCreateBranch || !this.searchTerm || this.mode === 'tags') {
+            return false;
           }
-          if (vm.menuVisible) {
+
-            Vue.set(vm, 'menuVisible', false);
+          return this.items.filter((item) => item.name.toLowerCase() === this.searchTerm.toLowerCase()).length === 0;
-          }
+        }
-        });
       },
 
       watch: {
@@ -3133,30 +3149,16 @@ function initFilterBranchTagDropdown(selector) {
         }
       },
 
-      computed: {
+      beforeMount() {
-        filteredItems() {
+        this.noResults = this.$el.getAttribute('data-no-results');
-          const vm = this;
+        this.canCreateBranch = this.$el.getAttribute('data-can-create-branch') === 'true';
 
-          const items = vm.items.filter((item) => {
+        document.body.addEventListener('click', (event) => {
-            return ((vm.mode === 'branches' && item.branch) || (vm.mode === 'tags' && item.tag)) &&
+          if (this.$el.contains(event.target)) return;
-              (!vm.searchTerm || item.name.toLowerCase().includes(vm.searchTerm.toLowerCase()));
+          if (this.menuVisible) {
-          });
+            Vue.set(this, 'menuVisible', false);
-
-          vm.active = (items.length === 0 && vm.showCreateNewBranch ? 0 : -1);
-
-          return items;
-        },
-        showNoResults() {
-          return this.filteredItems.length === 0 && !this.showCreateNewBranch;
-        },
-        showCreateNewBranch() {
-          const vm = this;
-          if (!this.canCreateBranch || !vm.searchTerm || vm.mode === 'tags') {
-            return false;
           }
-
+        });
-          return vm.items.filter((item) => item.name.toLowerCase() === vm.searchTerm.toLowerCase()).length === 0;
-        }
       },
 
       methods: {
@@ -3169,15 +3171,12 @@ function initFilterBranchTagDropdown(selector) {
           window.location.href = item.url;
         },
         createNewBranch() {
-          if (!this.showCreateNewBranch) {
+          if (!this.showCreateNewBranch) return;
-            return;
-          }
           $(this.$refs.newBranchForm).trigger('submit');
         },
         focusSearchField() {
-          const vm = this;
           Vue.nextTick(() => {
-            vm.$refs.searchField.focus();
+            this.$refs.searchField.focus();
           });
         },
         getSelected() {
@@ -3194,15 +3193,12 @@ function initFilterBranchTagDropdown(selector) {
         },
         scrollToActive() {
           let el = this.$refs[`listItem${this.active}`];
-          if (!el || el.length === 0) {
+          if (!el || !el.length) return;
-            return;
-          }
           if (Array.isArray(el)) {
             el = el[0];
           }
 
           const cont = this.$refs.scrollContainer;
-
           if (el.offsetTop < cont.scrollTop) {
             cont.scrollTop = el.offsetTop;
           } else if (el.offsetTop + el.clientHeight > cont.scrollTop + cont.clientHeight) {
@@ -3210,49 +3206,41 @@ function initFilterBranchTagDropdown(selector) {
           }
         },
         keydown(event) {
-          const vm = this;
+          if (event.keyCode === 40) { // arrow down
-          if (event.keyCode === 40) {
-            // arrow down
             event.preventDefault();
 
-            if (vm.active === -1) {
+            if (this.active === -1) {
-              vm.active = vm.getSelectedIndexInFiltered();
+              this.active = this.getSelectedIndexInFiltered();
             }
 
-            if (vm.active + (vm.showCreateNewBranch ? 0 : 1) >= vm.filteredItems.length) {
+            if (this.active + (this.showCreateNewBranch ? 0 : 1) >= this.filteredItems.length) {
               return;
             }
-            vm.active++;
+            this.active++;
-            vm.scrollToActive();
+            this.scrollToActive();
-          }
+          } else if (event.keyCode === 38) { // arrow up
-          if (event.keyCode === 38) {
-            // arrow up
             event.preventDefault();
 
-            if (vm.active === -1) {
+            if (this.active === -1) {
-              vm.active = vm.getSelectedIndexInFiltered();
+              this.active = this.getSelectedIndexInFiltered();
             }
 
-            if (vm.active <= 0) {
+            if (this.active <= 0) {
               return;
             }
-            vm.active--;
+            this.active--;
-            vm.scrollToActive();
+            this.scrollToActive();
-          }
+          } else if (event.keyCode === 13) { // enter
-          if (event.keyCode === 13) {
-            // enter
             event.preventDefault();
 
-            if (vm.active >= vm.filteredItems.length) {
+            if (this.active >= this.filteredItems.length) {
-              vm.createNewBranch();
+              this.createNewBranch();
-            } else if (vm.active >= 0) {
+            } else if (this.active >= 0) {
-              vm.selectItem(vm.filteredItems[vm.active]);
+              this.selectItem(this.filteredItems[this.active]);
             }
-          }
+          } else if (event.keyCode === 27) { // escape
-          if (event.keyCode === 27) {
-            // escape
             event.preventDefault();
-            vm.menuVisible = false;
+            this.menuVisible = false;
           }
         }
       }