[minor][engg]: exclude comments and blank lines from PR size line count

[Copilot]

• Add countCodeLines(patch, fallback) helper to .github/workflows/pr-size-check-reusable.yml that parses a unified-diff patch and skips blank lines, // single-line comments, and /* … */ multi-line block comments
• Replace f.additions + f.deletions with countCodeLines in the totalLines sum
• Update per-file table rows to display comment-stripped line counts instead of raw +additions / -deletions
• Update the job-summary status label wording to say "code lines" for clarity
• Cache codeLinesChanged once per included file to avoid re-scanning the same patch multiple times
• Sort summary table by codeLinesChanged (code lines) instead of raw additions + deletions

[azure-pipelines]

This pull request does not update CHANGELOG.md.

Please consider if this change would be noticeable to a partner or user and either update CHANGELOG.md or resolve this conversation.

[Copilot]

Pull request overview

Updates the PR Size Check reusable workflow to count only “real code” lines changed (excluding blank lines and // / /* */ comment-only lines) when enforcing the max-lines threshold, so cosmetic changes don’t inflate PR size.

Changes:

• Added a countCodeLines(patch, rawLines) helper to scan GitHub unified diff patches and count only non-comment, non-blank added/removed lines.
• Switched the overall line-limit calculation from additions + deletions to the helper-derived “code lines” count (with fallback to raw counts when patch is missing).
• Updated the summary table and status/failure messages to reflect “code lines changed” semantics.

[Copilot]

The summary table is now labeled in code lines, but sortedIncluded is still ordered by raw additions + deletions. Impact: the “top N files by changes” table may be misleading (files with many comment/blank-line edits can sort to the top even if they have few counted code lines). Recommendation: sort by the same countCodeLines(...) (or a cached codeLinesChanged) value that you display in the table.

[ameyapat]

@copilot apply changes based on this feedback

[Copilot]

Fixed in 205fdc5. countCodeLines is now called once per file and the result cached as f.codeLinesChanged. Both the totalLines reduction and the table rows read from that cached value. The summary table sort also now uses codeLinesChanged so the ranking is consistent with what's displayed.