fix(ci): Improve PR coverage check workflow accuracy (#8692)

- Add exclusions for apps/Standalone and apps/docs (no tests)
- Add exclusions for manifest and swagger directories (config/data files)
- Use full relative path matching instead of basename for accurate file matching in lcov reports
- Add explicit list of packages that require coverage
- Fail when files in covered packages have no coverage data
- Report skipped files for packages not configured for coverage

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: takyyon

.github/workflows/pr-coverage.yml

@@ -100,6 +100,14 @@ jobs:
             **/node_modules/**
             **/dist/**
             **/build/**
+            # Standalone app (dev environment, not covered by tests)
+            apps/Standalone/**
+            # Documentation site
+            apps/docs/**
+            # Manifest and swagger files (configuration/data, not logic)
+            **/manifest/**
+            **/manifests/**
+            **/swagger/**
 
       - name: Check coverage on changed files
         id: coverage-check
@@ -141,6 +149,10 @@ jobs:
           UNCOVERED_FILES=""
           COVERED_FILES=""
           PARTIAL_FILES=""
+          SKIPPED_FILES=""
+
+          # Define packages that require coverage (libs with vitest configs)
+          COVERED_PACKAGES="libs/designer libs/designer-ui libs/designer-v2 libs/logic-apps-shared libs/data-mapper libs/data-mapper-v2 libs/chatbot libs/a2a-core libs/vscode-extension apps/vs-code-designer apps/vs-code-react apps/iframe-app"
 
           for file in $CHANGED_FILES; do
             # Skip if file doesn't exist (might have been deleted)
@@ -153,11 +165,24 @@ jobs:
             PKG_DIR=$(echo "$file" | sed 's|/src/.*||')
             LCOV_FILE="$PKG_DIR/coverage/lcov.info"
 
+            # Check if this package requires coverage
+            PKG_REQUIRES_COVERAGE=false
+            for pkg in $COVERED_PACKAGES; do
+              if [ "$PKG_DIR" = "$pkg" ]; then
+                PKG_REQUIRES_COVERAGE=true
+                break
+              fi
+            done
+
             if [ -f "$LCOV_FILE" ]; then
-              # Check if this file is mentioned in the coverage report
-              if grep -q "SF:.*$(basename $file)" "$LCOV_FILE" 2>/dev/null; then
-                # Get line coverage for this file
-                FILE_SECTION=$(awk "/SF:.*$(basename $file)/,/end_of_record/" "$LCOV_FILE" 2>/dev/null || true)
+              # Get the absolute path pattern to match in lcov (lcov uses paths relative to package root or absolute)
+              # Extract the path after PKG_DIR to match against lcov SF entries
+              FILE_RELATIVE_PATH=$(echo "$file" | sed "s|^$PKG_DIR/||")
+
+              # Check if this file is mentioned in the coverage report using the relative path
+              if grep -q "SF:.*${FILE_RELATIVE_PATH}$" "$LCOV_FILE" 2>/dev/null; then
+                # Get line coverage for this file - use the full relative path for accurate matching
+                FILE_SECTION=$(awk "/SF:.*${FILE_RELATIVE_PATH//\//\\/}\$/,/end_of_record/" "$LCOV_FILE" 2>/dev/null || true)
 
                 if [ -n "$FILE_SECTION" ]; then
                   LINES_FOUND=$(echo "$FILE_SECTION" | grep "^LF:" | cut -d: -f2 || echo "0")
@@ -177,9 +202,12 @@ jobs:
               else
                 UNCOVERED_FILES="$UNCOVERED_FILES\n❌ \`$file\` - Not covered by tests"
               fi
+            elif [ "$PKG_REQUIRES_COVERAGE" = true ]; then
+              # Package should have coverage but doesn't - this is a problem
+              UNCOVERED_FILES="$UNCOVERED_FILES\n❌ \`$file\` - No coverage data (package should have tests)"
             else
-              # No coverage file found for this package - might be acceptable
-              echo "ℹ️ No coverage data for package: $PKG_DIR" >> $GITHUB_STEP_SUMMARY
+              # Package doesn't require coverage (not in the list)
+              SKIPPED_FILES="$SKIPPED_FILES\n⏭️ \`$file\` - Package not configured for coverage"
             fi
           done
 
@@ -198,6 +226,12 @@ jobs:
             echo "" >> $GITHUB_STEP_SUMMARY
           fi
 
+          if [ -n "$SKIPPED_FILES" ]; then
+            echo "#### ⏭️ Skipped (package not configured for coverage):" >> $GITHUB_STEP_SUMMARY
+            echo -e "$SKIPPED_FILES" >> $GITHUB_STEP_SUMMARY
+            echo "" >> $GITHUB_STEP_SUMMARY
+          fi
+
           if [ -n "$UNCOVERED_FILES" ]; then
             echo "#### ❌ Not Covered:" >> $GITHUB_STEP_SUMMARY
             echo -e "$UNCOVERED_FILES" >> $GITHUB_STEP_SUMMARY