fix: enhance MyPy execution and reporting with improved error handling and debug messages

Author: thought-operator

action.yml

@@ -147,37 +147,58 @@ runs:
 
     - name: Run MyPy
       run: |
-        echo "::debug file=action.yml,line=106::Running MyPy with options: ${{ inputs.mypy_options }}"
-        mypy ${{ inputs.mypy_options }} . 2>&1 | tee mypy_output.txt
-        echo "MYPY_EXIT_CODE=$?" >> $GITHUB_ENV
-        echo "::debug file=action.yml,line=110::MyPy completed with exit code ${{ env.MYPY_EXIT_CODE }}"
+        if [ -z "${{ inputs.mypy_options }}" ]; then
+          echo "::debug file=action.yml,line=105::No MyPy options provided, using default."
+          mypy . 2>&1 | tee mypy_output.txt
+
+          if [ $? -eq 0 ]; then
+            echo "::debug file=action.yml,line=108::MyPy completed successfully with default options."
+            echo "MYPY_EXIT_CODE=0" >> $GITHUB_ENV
+            exit 0
+          else
+            echo "::debug file=action.yml,line=110::MyPy found issues with default options."
+            echo "MYPY_EXIT_CODE=1" >> $GITHUB_ENV
+            exit 1
+          fi
+        else
+          echo "::debug file=action.yml,line=107::Using MyPy options: ${{ inputs.mypy_options }}"
+          mypy ${{ inputs.mypy_options }} . 2>&1 | tee mypy_output.txt
+          if [ $? -eq 0 ]; then
+            echo "::debug file=action.yml,line=112::MyPy completed successfully with custom options."
+            echo "MYPY_EXIT_CODE=0" >> $GITHUB_ENV
+            exit 0
+          else
+            echo "::debug file=action.yml,line=114::MyPy found issues with custom options."
+            echo "MYPY_EXIT_CODE=1" >> $GITHUB_ENV
+            exit 1
+          fi
+        fi
       shell: bash
       if: always()
 
     - name: Report MyPy results
       run: |
-          if [ "$MYPY_EXIT_CODE" == "0" ]; then
-            if [ -z "$(cat mypy_output.txt)" ]; then
-              echo "::notice file=action.yml,line=115::No type checking issues found."
-            else
-              echo "::warning file=action.yml,line=145::Mypy output present despite zero exit code."
-              echo "Mypy output present despite zero exit code." > mypy_output.txt
-            fi
-            echo "::notice file=action.yml,line=120::All type checks passed."
-          else
-            echo "::error file=action.yml,line=150::Type checking issues found."
-            if [ -z "$(cat mypy_output.txt)" ]; then
-              echo "::warning file=action.yml,line=153::No output from mypy, but exit code indicates issues."
-            fi
+        if [ "${MYPY_EXIT_CODE}" == "0" ]; then
+          export MYPY_OUTPUT=$(cat mypy_output.txt)
+
+          if [ -z "${MYPY_OUTPUT}" ]; then
+            echo "::warning file=action.yml,line=159::No mypy output but exit code indicates success."
           fi
-          echo "## MyPy Results :hammer:" >> $GITHUB_STEP_SUMMARY
-          echo "" >> $GITHUB_STEP_SUMMARY
-          echo '```text' >> $GITHUB_STEP_SUMMARY
-          cat mypy_output.txt >> $GITHUB_STEP_SUMMARY
-          echo '```' >> $GITHUB_STEP_SUMMARY
-          echo "" >> $GITHUB_STEP_SUMMARY
-          echo "::debug file=action.yml,line=138::Removing temporary mypy output file."
-          rm -f mypy_output.txt
+        else
+          echo "::error file=action.yml,line=159::Mypy issues found."
+        fi
+        echo "## MyPy Results :hammer:" >> $GITHUB_STEP_SUMMARY
+        echo "" >> $GITHUB_STEP_SUMMARY
+        echo '```text' >> $GITHUB_STEP_SUMMARY
+        cat mypy_output.txt >> $GITHUB_STEP_SUMMARY
+        echo '```' >> $GITHUB_STEP_SUMMARY
+        echo "" >> $GITHUB_STEP_SUMMARY
+        rm -f mypy_output.txt
+        if [[ ! -f mypy_output.txt ]]; then
+          echo "::debug file=action.yml,line=170::Temporary mypy output file removed successfully."
+        else
+          echo "::warning file=action.yml,line=174::Failed to remove temporary mypy output file."
+        fi
       shell: bash
       if: always()
 
@@ -284,26 +305,36 @@ runs:
           echo "::warning file=action.yml,line=274::Failed to add mypy badge. Directory may not exist."
         fi
         
-        echo "::notice file=action.yml,line=284::Badge files added to staging area."
+        git status --porcelain
+        if [ $? -eq 0 ]; then
+          echo "::debug file=action.yml,line=287::Git status checked successfully."
+        else
+          echo "::warning file=action.yml,line=287::Failed to check git status."
+        fi
 
-        echo "::debug file=action.yml,line=288::Checking for changes to commit."
-        
+        echo "::debug file=action.yml,line=295::Checking for changes to commit."
         git diff --staged --quiet
 
         if [ $? -eq 0 ]; then
-          echo "::notice file=action.yml,line=277::No changes to commit."
+          echo "::notice file=action.yml,line=295::No changes to commit."
           exit 0
         fi
-        echo "::notice file=action.yml,line=281::Committing and pushing badge changes to repository."
+        echo "::debug file=action.yml,line=302::Committing changes to repository."
         git commit -m "Update linting badges [skip ci]"
+        if [ $? -eq 0 ]; then
+          echo "::notice file=action.yml,line=302::Changes committed to repository."
+        else
+          echo "::warning file=action.yml,line=303::Failed to commit changes to repository."
+        fi
 
+        echo "::debug file=action.yml,line=310::Pushing changes to repository."
         git push
 
         if [ $? -eq 0 ]; then
-          echo "::notice file=action.yml,line=282::Changes committed and pushed to repository."
+          echo "::notice file=action.yml,line=310::Changes committed and pushed to repository."
         else
-          echo "::warning file=action.yml,line=283::Failed to push changes to repository."
-          echo "::notice file=action.yml,line=284::This may be due to insufficient permissions or a merge conflict."
+          echo "::warning file=action.yml,line=310::Failed to push changes to repository."
+          echo "::notice file=action.yml,line=312::This may be due to insufficient permissions or a merge conflict."
           echo ""
           exit 0
         fi