Skip to content

feat(coverage): add Git tracking awareness for coverage reports#2557

Draft
brynary wants to merge 1 commit intomainfrom
feat/coverage-git-tracking-awareness
Draft

feat(coverage): add Git tracking awareness for coverage reports#2557
brynary wants to merge 1 commit intomainfrom
feat/coverage-git-tracking-awareness

Conversation

@brynary
Copy link
Copy Markdown
Member

@brynary brynary commented Dec 5, 2025

Summary

  • Add informational display showing Git repository path and untracked files in coverage reports
  • Implement GitTrackingInfo struct and get_git_tracking_info() function to check if files are tracked in Git
  • Add untracked_files and git_repo_path fields to the Report struct
  • Display Git repository line in COVERAGE DATA section and untracked files section (with red asterisks)
  • Update display limits from 20/10 to 50/50 for missing/untracked files

This is purely informational and does NOT affect validation (which remains disk-based only).

Test plan

  • All existing unit tests pass (232 tests in qlty-coverage)
  • All integration tests pass (including coverage and without_git tests)
  • Test fixtures updated to reflect new Git Repository line in output
  • Manual testing: verify Git repository path displays correctly
  • Manual testing: verify untracked files are displayed with red asterisks
  • Manual testing: verify "not found" displays when not in a Git repository

🤖 Generated with Claude Code

Add informational display showing Git repository path and untracked files
in coverage reports. This helps users understand when coverage files are
tracked in Git vs only present on disk.

Changes:
- Add GitTrackingInfo struct and get_git_tracking_info() function to check
  if files are tracked in Git
- Add untracked_files and git_repo_path fields to the Report struct
- Update processor to check Git tracking for files that exist on disk
- Add Git repository line to COVERAGE DATA section output
- Add untracked files display section showing files on disk but not in Git
- Update display limits from 20/10 to 50/50 for missing/untracked files
- Update test fixtures to include new Git Repository line in output

This is purely informational and does NOT affect validation (which remains
disk-based only).

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

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings December 5, 2025 20:40
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 5, 2025

No issue mentions found. Please mention an issue in the pull request description.

Use GitHub automation to close the issue when a PR is merged

@qltysh
Copy link
Copy Markdown
Contributor

qltysh bot commented Dec 5, 2025

❌ 1 blocking issue (3 total)

Tool Category Rule Count
ripgrep Lint style("NOTE:").bold() 1
qlty Structure Function with high complexity (count = 28): compute 1
qlty Structure Deeply nested control flow (level = 5) 1

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants