@@ -18,15 +18,15 @@ run_codenarc() {
1818 includes_arg=" "
1919 [ -n " $INPUT_SOURCE_FILES " ] && includes_arg=" -includes=${INPUT_SOURCE_FILES} "
2020
21- echo " 🔍 Executando CodeNarc..."
21+ echo " 🔍 Executando CodeNarc para análise estática ..."
2222 java -jar /lib/codenarc-all.jar \
2323 -report=" json:${CODENARC_JSON} " \
2424 -rulesetfiles=" ${INPUT_RULESETFILES} " \
2525 -basedir=" ." \
26- $includes_arg
27-
26+ $includes_arg > /dev/null 2>&1
27+
2828 echo " "
29- echo " 📋 Violações encontradas:"
29+ echo " 📋 Processando violações encontradas:"
3030 echo " "
3131 convert_json_to_compact
3232 cat " $CODENARC_COMPACT "
@@ -55,7 +55,7 @@ run_reviewdog() {
5555 [ ! -s " $CODENARC_COMPACT " ] && return
5656
5757 echo " 📤 Enviando resultados para reviewdog..."
58-
58+
5959 if [ " ${INPUT_REPORTER} " = " local" ]; then
6060 < " $CODENARC_COMPACT " reviewdog \
6161 -efm=" %f:%l:%m" \
@@ -92,8 +92,8 @@ run_reviewdog() {
9292
9393generate_git_diff () {
9494 if [ -n " $GITHUB_BASE_SHA " ] && [ -n " $GITHUB_HEAD_SHA " ]; then
95- git fetch origin " $GITHUB_BASE_SHA " --depth=1 2 > /dev/null || true
96- git fetch origin " $GITHUB_HEAD_SHA " --depth=1 2 > /dev/null || true
95+ git fetch origin " $GITHUB_BASE_SHA " --depth=1 > /dev/null 2>&1 || true
96+ git fetch origin " $GITHUB_HEAD_SHA " --depth=1 > /dev/null 2>&1 || true
9797 git diff -U0 " $GITHUB_BASE_SHA " " $GITHUB_HEAD_SHA " -- ' *.groovy'
9898 else
9999 git diff -U0 HEAD~1 -- ' *.groovy'
@@ -130,7 +130,7 @@ build_changed_lines_cache() {
130130is_changed () {
131131 local file=" $1 "
132132 local line=" $2 "
133-
133+
134134 if [ -z " $line " ]; then
135135 [ -f " $CHANGED_FILES_CACHE " ] && grep -qF " $file " " $CHANGED_FILES_CACHE " && return 0
136136 return 1
@@ -159,14 +159,12 @@ extract_p1_violations() {
159159}
160160
161161check_blocking_rules () {
162- echo " 🔎 Verificando violações bloqueantes (priority 1)..."
163-
164- [ ! -f " $CODENARC_JSON " ] && echo " ❌ Resultado do CodeNarc não encontrado. Não é possível verificar P1s." && return 1
162+ echo " 🔎 Verificando violações bloqueantes (P1)..."
163+ [ ! -f " $CODENARC_JSON " ] && echo " ❌ Erro: Resultado do CodeNarc não encontrado. Não é possível verificar P1s." && return 1
165164
166165 p1_violations=$( extract_p1_violations)
167-
168166 if [ -z " $p1_violations " ]; then
169- echo " ✅ Nenhuma P1 detectada → merge permitido"
167+ echo " ✅ Nenhuma violação P1 detectada → merge permitido"
170168 return 0
171169 fi
172170
@@ -178,49 +176,35 @@ check_blocking_rules() {
178176 echo " "
179177
180178 if [ " ${INPUT_REPORTER} " = " local" ]; then
181- echo " 🏠 Modo local - todas as P1s são bloqueantes"
179+ echo " 🏠 Modo de execução local: todas as violações P1 são bloqueantes. "
182180 echo " 💡 Corrija as violações antes de prosseguir."
183181 exit 1
184182 fi
185183
186- echo " ⚠️ Verificando se P1s estão em linhas alteradas..."
184+ echo " ⚠️ Analisando se as P1s estão em linhas alteradas..."
187185 build_changed_lines_cache
188186
189187 if [ ! -s " $ALL_DIFF " ]; then
190- echo " ⚠️ Diff vazio - considerando todas as P1s como bloqueantes (sem informações de linhas alteradas) ."
191- echo " 💡 Corrija as violações ou use o bypass autorizado."
188+ echo " ⚠️ Diff vazio: Sem informações de linhas alteradas. Todas as P1s são consideradas bloqueantes ."
189+ echo " 💡 Corrija as violações ou use um bypass autorizado."
192190 exit 1
193191 fi
194-
195- echo " 📝 Debug - Linhas alteradas:"
196- cat " $CHANGED_LINES_CACHE " 2> /dev/null || echo " (cache vazio)"
197- echo " 📝 Debug - Arquivos alterados:"
198- cat " $CHANGED_FILES_CACHE " 2> /dev/null || echo " (cache vazio)"
199- echo " "
200192
201193 found_blocking=0
202194 while IFS=: read -r file line rest; do
203195 [ -z " $file " ] && continue
204-
205- echo " 🔍 Verificando violação: $file :$line "
206196
207197 if [ -z " $line " ]; then
208- echo " → Violação a nível de arquivo."
209198 if is_changed " $file " " " ; then
210- echo " ⛔ BLOQUEANDO: $file ( nível de arquivo) : $rest "
199+ echo " 🚨 BLOQUEADO: Violação P1 a nível de arquivo encontrada no arquivo alterado : $file "
211200 found_blocking=1
212201 break
213- else
214- echo " → Arquivo não foi alterado no diff, ignorando P1."
215202 fi
216203 else
217- echo " → Violação a nível de linha."
218204 if is_changed " $file " " $line " ; then
219- echo " ⛔ BLOQUEANDO: $file :$line : $rest "
205+ echo " 🚨 BLOQUEADO: Violação P1 encontrada na linha alterada: $file :$line "
220206 found_blocking=1
221207 break
222- else
223- echo " → Linha não está no diff, ignorando P1."
224208 fi
225209 fi
226210 done << EOF
@@ -229,12 +213,12 @@ EOF
229213
230214 if [ $found_blocking -eq 1 ]; then
231215 echo " "
232- echo " 🚨 Violações P1 críticas encontradas em linhas alteradas. Merge bloqueado ."
233- echo " 💡 Corrija as violações ou use o bypass autorizado."
216+ echo " 🚨 Merge bloqueado: Violações P1 críticas encontradas em código alterado ."
217+ echo " 💡 Corrija as violações antes de prosseguir com o merge ou use o bypass autorizado."
234218 exit 1
235219 fi
236220
237- echo " ✅ P1s existem mas fora das linhas alteradas → merge permitido"
221+ echo " ✅ Todas as violações P1 estão fora das linhas alteradas → merge permitido"
238222}
239223
240224if [ -n " ${GITHUB_WORKSPACE} " ]; then
@@ -248,4 +232,4 @@ run_codenarc
248232run_reviewdog
249233check_blocking_rules
250234
251- echo " 🏁 Concluído com sucesso"
235+ echo " 🏁 Análise de CodeNarc concluída com sucesso. "
0 commit comments