feat: verify with the JSONSchema by default

Author: ldez

action.yml

@@ -24,7 +24,7 @@ inputs:
     required: false
   verify:
     description: "if set to true and the action verify the configuration file against the JSONSchema"
-    default: 'false'
+    default: 'true'
     required: false
   only-new-issues:
     description: "if set to true and the action runs on a pull request - the action outputs only newly found issues"

src/run.ts

@@ -137,17 +137,7 @@ async function runLint(binPath: string, patchPath: string): Promise<void> {
     cmdArgs.cwd = path.resolve(workingDirectory)
   }
 
-  if (core.getBooleanInput(`verify`, { required: true })) {
-    let cmdVerify = `${binPath} config verify`
-    if (userArgsMap.get("config")) {
-      cmdVerify += ` --config=${userArgsMap.get("config")}`
-    }
-
-    core.info(`Running [${cmdVerify}] in [${cmdArgs.cwd || process.cwd()}] ...`)
-
-    const res = await execShellCommand(cmdVerify, cmdArgs)
-    printOutput(res)
-  }
+  await runVerify(binPath, userArgsMap, cmdArgs)
 
   const cmd = `${binPath} run ${addedArgs.join(` `)} ${userArgs}`.trimEnd()
 
@@ -173,6 +163,39 @@ async function runLint(binPath: string, patchPath: string): Promise<void> {
   core.info(`Ran golangci-lint in ${Date.now() - startedAt}ms`)
 }
 
+async function runVerify(binPath: string, userArgsMap: Map<string, string>, cmdArgs: ExecOptions): Promise<void> {
+  const cfgPath = await getConfigPath(binPath, userArgsMap, cmdArgs)
+  const verify = core.getBooleanInput(`verify`, { required: true })
+
+  if (!verify || !cfgPath) {
+    return
+  }
+
+  let cmdVerify = `${binPath} config verify`
+  if (userArgsMap.get("config")) {
+    cmdVerify += ` --config=${userArgsMap.get("config")}`
+  }
+
+  core.info(`Running [${cmdVerify}] in [${cmdArgs.cwd || process.cwd()}] ...`)
+
+  const res = await execShellCommand(cmdVerify, cmdArgs)
+  printOutput(res)
+}
+
+async function getConfigPath(binPath: string, userArgsMap: Map<string, string>, cmdArgs: ExecOptions): Promise<string> {
+  let cmdConfigPath = `${binPath} config path`
+  if (userArgsMap.get("config")) {
+    cmdConfigPath += ` --config=${userArgsMap.get("config")}`
+  }
+
+  try {
+    const resPath = await execShellCommand(cmdConfigPath, cmdArgs)
+    return resPath.stdout.trim()
+  } catch {
+    return ``
+  }
+}
+
 export async function run(): Promise<void> {
   try {
     const { binPath, patchPath } = await core.group(`prepare environment`, prepareEnv)