fix: super-linter (#6)

* fix: super-linter - .env and javascript_standard

* fix: super-linter - pre-commit

* fix: installGitHook gradle task

Author: DBFabianWenzel

.env.template

@@ -1,2 +1,2 @@
 # It will force the user to add an e-mail for this project, before committing.
-COMMIT_MAIL=[email protected]
+COMMIT_MAIL=[email protected]

.github/workflows/linter.yml

@@ -32,9 +32,11 @@ jobs:
           VALIDATE_CHECKOV: false
           VALIDATE_JSCPD: false
           VALIDATE_KOTLIN: false
+          VALIDATE_JAVASCRIPT_PRETTIER: false
           VALIDATE_JSON_PRETTIER: false
           VALIDATE_MARKDOWN_PRETTIER: false
           VALIDATE_YAML_PRETTIER: false
+          VALIDATE_BASH_EXEC: false
 
       - name: Set up JDK
         uses: actions/[email protected]

build.gradle.kts

@@ -4,12 +4,20 @@ plugins {
     alias(libs.plugins.jetbrains.kotlin.android) apply false
     alias(libs.plugins.compose.compiler) apply false
 }
-task<Copy>("installGitHook") {
+task("installGitHook") {
     delete(".git/hooks/pre-commit")
+    copy {
+        filePermissions {
+            user {
+                read = true
+                execute = true
+            }
+            other.execute = false
+        }
 
-    fileMode = 0x777
-    from(File(rootProject.rootDir, "scripts/pre-commit"))
-    into(File(rootProject.rootDir, ".git/hooks"))
+        from("${rootProject.rootDir}/scripts/pre-commit")
+        into("${rootProject.rootDir}/.git/hooks")
+    }
 }
 
-tasks.getByPath(":app:preBuild").dependsOn(":installGitHook")
+tasks.getByPath("buildEnvironment").dependsOn(":installGitHook")

gradle/libs.versions.toml

@@ -1,12 +1,12 @@
 [versions]
-agp = "8.7.1"
+agp = "8.7.2"
 kotlin = "2.0.21"
-coreKtx = "1.13.1"
+coreKtx = "1.15.0"
 junit = "4.13.2"
 junitVersion = "1.2.1"
 espressoCore = "3.6.1"
 appcompat = "1.7.0"
-composeBom = "2024.10.00"
+composeBom = "2024.10.01"
 material = "1.12.0"
 
 [libraries]

scripts/check-commit-mail.js

@@ -0,0 +1,28 @@
+#!/usr/bin/env node
+import * as ChildProcess from 'node:child_process'
+import * as process from 'node:process'
+
+const checkCommitMail = () => {
+  console.warn('Check COMMIT_MAIL')
+  if (!process.env.COMMIT_MAIL) {
+    console.error(
+      "No COMMIT_MAIL set in .env, please take a look at the file '.env.template'"
+    )
+    process.exit(1)
+  }
+
+  const currentMail = ChildProcess.execSync('git config user.email')
+    .toString()
+    .trim()
+    .toLowerCase()
+  const commitMail = process.env.COMMIT_MAIL.trim().toLowerCase()
+  if (currentMail !== commitMail) {
+    console.error(`currentMail: ${currentMail} !== initialMail: ${commitMail}`)
+    console.error(
+      `Please set your commit user mail for this project like: 'git config user.email '${commitMail}'`
+    )
+    process.exit(1)
+  }
+}
+
+checkCommitMail()

scripts/check-commit-mail.mjs

@@ -5,13 +5,13 @@ git stash -q --keep-index
 echo "Running git pre-commit hook"
 
 ./gradlew lint &&
-node --env-file=.env ./scripts/check-commit-mail.js &&
-node ./scripts/validate-branch-name.js
+	node --env-file=.env ./scripts/check-commit-mail.mjs &&
+	node ./scripts/validate-branch-name.mjs
 
 RESULT=$?
 
 git stash pop -q
 
 # return 1 exit code if running checks fails
 [ $RESULT -ne 0 ] && exit 1
-exit 0
+exit 0

scripts/pre-commit

@@ -0,0 +1,17 @@
+#!/usr/bin/env node
+import * as ChildProcess from 'node:child_process'
+import * as process from 'node:process'
+
+const checkValidBranchName = () => {
+  console.warn('Check VALID_BRANCH_NAME')
+
+  const currentBranchName = ChildProcess.execSync('git symbolic-ref --short HEAD')
+    .toString()
+    .trim()
+
+  const regex = /^(feature|fix|hotfix|release)\/.+$/g
+  const found = currentBranchName.match(regex) !== null
+  process.exit(found ? 0 : 1)
+}
+
+checkValidBranchName()